[DOCS] Remove [testenv="gold+"] attributes

[edh-oss]

Changes:

• Removes several [testenv="gold+"] attributes from the docs. gold+ is not a valid subscription level or testenv value.
• Moves two [testenv="basic"] attributes to the file header. This makes the testenv placement consistent and fixes the yml file generated from docs/reference/snapshot-restore/register-repository.asciidoc.

Original text

This PR is doing three things:

• It allows for the testenv attribute to be set to "gold+"
• It makes the placement of testenv attribute settings consistent, always in doc headers (as 328 out of 332 currently do)
• It fixes the yml file generated from docs/reference/snapshot-restore/register-repository.asciidoc

Current code in RestTestsFromSnippetsTask allows for testenv to have one of four values: basic, gold, platinum or enterprise. In the asciidoc files, five are in use. The fifth is "gold+", which appears in eight files under "docs/reference/commands". None of those eight files contains a test snippet, so the value of testenv is never checked and never used. If any of them were to contain a test snippet, an exception would result. This PR assumes that "gold+" actually is a valid value for testenv, and updates RestTestsFromSnippetsTask accordingly.

There are 332 asciidoc files that set the testenv attribute. 328 of them put that statement in the doc header. I've updated the other four. For three of the four, the generated yaml is unchanged. The fourth of those files is register-repository.asciidoc.

In register-repository.asciidoc, the testenv attribute is set between the first test snippet and a series of corresponding "TEST[continued]" snippets. This placement means that, at least for the purposes of yaml generation, setting the attribute does nothing. If it's set at the top of the file, in the document header, then "xpack" is added to each snippets' list of skipped features. I believe the latter to be the expected behavior, and this CR makes the corresponding update.

[elasticmachine]

Pinging @elastic/es-docs (Team:Docs)

[elasticmachine]

Pinging @elastic/es-delivery (Team:Delivery)

[jrodewig]

Thanks @edh-oss. This looks great!

The fifth is "gold+", which appears in eight files under "docs/reference/commands". None of those eight files contains a test snippet, so the value of testenv is never checked and never used.

I pushed a commit to remove the gold+ value. The testenv values align with our subscription levels, and there isn't a gold+ or equivalent level.

I also removed [role="xpack"] attributes from any edited files. With elastic/docs#2213, these attributes no longer do anything in 7.11+ branches. There are other [role="xpack"] instances throughout the docs, but we can remove those as a separate effort.

[jrodewig]

@elasticmachine ok to test

[jrodewig]

Apologies for the extra commits and noise. I initially removed some [testenv="basic"] instances too. We now run our snippet tests in a basic env by default so I don't think these serve much purpose aside from annotating the yml. However, I could be wrong, and it's worth discussing this more in another issue/PR.

Thanks again @edh-oss.

[jrodewig]

@elasticmachine update branch

[jrodewig]

@elasticmachine update branch

[jrodewig]

@elasticmachine update branch

[jrodewig]

@elasticmachine run elasticsearch-ci/part-2