fix(endpoints): remove old region tags

[OremGLG]

Description

Delete old region tags
secret-1
service
secret-2

Fixes b/347825448

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

• I have followed Sample Format Guide
• pom.xml parent set to latest shared-configuration
• Appropriate changes to README are included in PR
• These samples need a new API enabled in testing projects to pass (let us know which ones)
• These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
• Tests pass: mvn clean verify required
• Lint passes: mvn -P lint checkstyle:check required
• Static Analysis: mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
• This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
• This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
• Please merge this PR for me once it is approved

[code-review-assist]

Hi team,

I'm currently reviewing this pull request and will provide feedback shortly. In the meantime, here's a summary for you and other reviewers to quickly get up to speed:

This pull request, submitted by OremGLG, aims to remove outdated region tags from the endpoints/getting-started/k8s/esp_echo_http.yaml file. The intent is to clean up the file and improve maintainability. This addresses issue b/347825448.

Here's a breakdown of the changes:

• File Modified: endpoints/getting-started/k8s/esp_echo_http.yaml
• Changes: The pull request removes several lines containing old region tags, specifically those marked with comments [START secret-1], [END secret-1], [START service], [END service], [START secret-2], and [END secret-2]. These tags were previously used for documentation purposes but are no longer needed.
  • Lines 42-47 were removed from the first change block.
  • Lines 61-62 and 66-71 were removed from the second change block.

The author has indicated that all required tests and lint checks have passed.

---

A little haiku to lighten the mood:

Tags removed with care,
Code now cleaner, bright, and new,
Review, then merge on!

[code-review-assist]

This PR removes redundant region tags, which is good for clarity. The suggested comments improve the readability further. Also, please ensure the checklist items are accurately marked before merging.

[snippet-bot]

Here is the summary of changes.

You are about to delete 3 region tags.

• endpoints/getting-started/k8s/esp_echo_http.yaml:42, tag secret-1
• endpoints/getting-started/k8s/esp_echo_http.yaml:50, tag service
• endpoints/getting-started/k8s/esp_echo_http.yaml:66, tag secret-2

---

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

• Refresh this comment