Align on and rename repository and helm chart name

[simitt]

Current names:

• Repo: apm-mutating-webhook: it is defining what it is rather than what it does. Knowledge that this is a mutating webhook is not required for usage, and should be treated as an implementation detail, rather than being surfaced in the name.
• Helm: apm-agent-auto-attach: too verbous; focus only on apm-agents, where longer term it might contain something related to apm-server?

Suggestions:

Everyone who wants to suggest a name, please update the list below. @chrisdistasio @alex-fedotyev would appreciate your inputs.

Repo

suggestion	comment	actively considered
apm-k8s-attacher	keep aligned with apm prefix, mention k8s	yes

Helm

suggestion	comment	actively considered
apm-attacher	k8s would be redundant for helm chart	yes

[simitt]

@felixbarny @axw @Mpdreamz @AlexanderWert @alex-fedotyev @chrisdistasio @graphaelli any concerns about the suggested names?
If not, I suggest we move forward with them by tomorrow, to unblock Marc from recording the screencast with the new names.

[simitt]

We settled on the names apm-k8s-attacher and apm-attacher. Next steps are to rename the repository and the helm chart.

[jackshirazi]

The helm name was changed (not sure when)

$ helm search repo
NAME                            CHART VERSION   APP VERSION     DESCRIPTION
elastic/apm-attacher            0.1.0                           A Helm chart installing the Elastic APM mutatin...

Who has permission to change the repo name?

[simitt]

Who has permission to change the repo name?

That should be the @elastic/observablt-robots team.

[jackshirazi]

Repo name has been changed. I've updated the doc links in #84 . After that is merged we need a new release with a version increment, and then I'll test it's all working and the docs are correct. Not sure how to do a release here - probably worth updating the release section in the README

[jackshirazi]

Note that the last release (v0.2.0) didn't update the charts at elastic helm:

$ helm repo add elastic https://helm.elastic.co
$ helm search repo
NAME                            CHART VERSION   APP VERSION     DESCRIPTION
elastic/apm-attacher            0.1.0                           A Helm chart installing the Elastic APM mutatin...

presumably because the chart is still saying 0.1.0

[amannocci]

PRs for rename:

• https://github.com/elastic/infra/pull/40325
• chore: rename apm-mutating-webhook to apm-k8s-attacher apm-agent-nodejs#3787
• chore: rename apm-mutating-webhook to apm-k8s-attacher apm-agent-java#3463
• chore: rename apm-mutating-webhook to apm-k8s-attacher apm#843
• https://github.com/elastic/observability-dev/pull/3019
• chore: rename apm-mutating-webhook to apm-k8s-attacher apm-server#12222
• chore: rename apm-mutating-webhook to apm-k8s-attacher docs#2869
• https://github.com/elastic/ci-agent-images/pull/465
• https://github.com/elastic/observability-robots/pull/1940
• https://github.com/elastic/cloud/pull/122540
• https://github.com/elastic/release-eng/pull/870
• https://github.com/elastic/seceng/pull/6408
• https://github.com/elastic/tech-writing-guidelines/pull/147

[estolfo]

Here are some outstanding rename tasks:

• the doc file and build (which might be able to stay the same but perhaps should change?)
• the artifact and directory name: "elastic/apm-mutating-webhook" in

  apm-k8s-attacher/skaffold.yaml

  Line 8 in 46433a6

  - image: elastic/apm-mutating-webhook

  and module github.com/elastic/apm-mutating-webhook in

  apm-k8s-attacher/go.mod

  Line 1 in 46433a6

  module github.com/elastic/apm-mutating-webhook

  , etc.
• Finally various link and repo refs that will work because of GitHub redirects, for a while at least.

[trentm]

Here are some outstanding rename tasks:
...

These from doing this search: https://github.com/search?q=org%3Aelastic%20apm-mutating-webhook&type=code
(I suppose the results from that might differ based on repo access permissions.)

Two of @amannocci's PRs are not yet merged, which covers a few of these search hits.

[trentm]

There are still a few remaining code hits for https://github.com/search?q=org%3Aelastic%20apm-mutating-webhook&type=code but nothing that impacts users. I think this renaming is complete.