Changes method from HEAD to OPTIONS for ping in rest-spec-api

[picandocodigo]

Fixes #97613

Related:

• Updates the method of ping from HEAD to OPTIONS  elasticsearch-specification#5138

[elasticsearchmachine]

Pinging @elastic/es-core-infra (Team:Core/Infra)

[ldematte]

Hello @picandocodigo, I'm not sure what's the intent of this PR.
Changing the spec will not change the implementation of the REST action (which after a quick search I think is https://github.com/elastic/elasticsearch/blob/main/modules/rest-root/src/main/java/org/elasticsearch/rest/root/RestMainAction.java#L33), so this actually will not fix the linked issue.
In order to review and give you pointers, we'd need to know how do you plan to proceed. Do you plan to submit multiple PRs maybe?

[ldematte]

Also, please avoid to force push changes: we usually don't do that, as it makes reviews more difficult and changes hard to trace (losing history)

[ldematte]

A final note: changing the method is a breaking change, so probably it's worth discussing implications and options before proceeding (e.g. on the linked issue, pinging us on our slack channel, or joining us at our weekly sync if this is urgent).

[picandocodigo]

Hi @ldematte, thanks for looking into this. Initially I thought this change would involve much less and be simpler. After discussing this more with the team, we agreed we won't go ahead with changing the method and keep using HEAD. The force push changes were from rebasing to the latest main, but I'll have this in mind for future Pull Requests. Thanks!

[ldematte]

No prob!