opensearch - Remove CurrentState attribute

[ichekaldin]

SUMMARY

This is necessary because describe_domain_config method returns both CurrentState and DesiredState while update_domain_config method only expects DesiredState.

Fixes #2161.

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

opensearch

ADDITIONAL INFORMATION

[softwarefactory-project-zuul]

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/583c9489b40945f6b4c9e0ce8dcfce69

❌ ansible-galaxy-importer FAILURE in 5m 21s (non-voting)
✔️ build-ansible-collection SUCCESS in 11m 15s
✔️ ansible-test-splitter SUCCESS in 4m 23s
Skipped 22 jobs

[markuman]

Can you also please add an integration test that covers this case? https://github.com/ansible-collections/community.aws/tree/main/tests/integration/targets/opensearch

Otherwise LGTM

[ichekaldin]

@markuman, as far as I can tell, integration tests for opensearch are disabled.

I'm fairly certain the existing tests would catch it by failing.

I'm not exactly sure when this piece of Opensearch functionality was introduced. I suspect it happened between April 25, 2024 (which is the most recent successful build of my Ansible automation that touched Opensearch) and now.

[markuman]

@markuman, as far as I can tell, integration tests for opensearch are disabled.

Ah ok. I'll run the opensearch test locally later this day and give you a feedback.

[ichekaldin]

@markuman, any luck with the testing?