Looks like a bug. Want to try and write a (failing) test for this @ausmanlumeris?

Facing the same issue at my day job. I can volunteer writing a failing test case.

@dblock @ausmanlumeris Well it's actually not a bug. We are supposed to use bulk from client and not from helper.
Basically all you need to do is

docs = [{"delete": {"_index": "<index name", "_id": "<id>"}}]           
client.bulk(docs)

This shall pass what we are trying to do. Coming to the bulk in helper it is only for the purpose of bulk indexing. It just takes your data object as it is and pushes it to index. so it actually isn't detecting your operation. (Maybe the name of that bulk method inside helper is confusing us ? Or we can have some comments on the separation of bulk from client and helper) But for what we intend to do the above solution works the best.

@dblock I have also updated the PR with passing test case with client implementation so that we have a sanity on it's working. Will be happy to coordinate further in case if there is something else we need to do.

This shall pass what we are trying to do. Coming to the bulk in helper it is only for the purpose of bulk indexing. It just takes your data object as it is and pushes it to index. so it actually isn't detecting your operation. (Maybe the name of that bulk method inside helper is confusing us ? Or we can have some comments on the separation of bulk from client and helper) But for what we intend to do the above solution works the best.

Does it make sense to add support for deletions in the bulk helper or do something in code that prevents future errors?

Either way do take a look at the documentation, maybe a few words added can help the next person?

Either way do take a look at the documentation, maybe a few words added can help the next person?

Yes, Actually documentation made me realize that we were using it in wrong way. There is no mention of helpers in documentation https://opensearch.org/docs/latest/clients/python-low-level/ and also it makes more sense to use client instead.

Does it make sense to add support for deletions in the bulk helper

Ah, I am not sure. I would like to learn why helpers exist ? What's the purpose of helpers in codebase ? Becase we are allowing same methods in client. Maybe it could be an abstraction somewhere? Cannot learn about them from documentations anywhere. This will then give a better idea if we should add delete to it or not.

@Harshil-Jani It's all legacy from when OpenSearch forked 3 years ago. I am 100% with you that helpers should really not exist and be wrappers on top of existing APIs. If you want to undertake that kind of change, please do!

That said, progress > perfection, so if you think you can contribute something/anything that would have prevented you from wasting a lot of time debugging a non-existent problem that would be very much appreciated.