Skip to content

Standardize annotation prefix: use config.kubernetes.io instead of config.k8s.io#4663

Open
aravindtga wants to merge 1 commit into
kptdev:mainfrom
Nordix:standardize-resource-id-annotation-prefix
Open

Standardize annotation prefix: use config.kubernetes.io instead of config.k8s.io#4663
aravindtga wants to merge 1 commit into
kptdev:mainfrom
Nordix:standardize-resource-id-annotation-prefix

Conversation

@aravindtga

Copy link
Copy Markdown
Contributor

Description

  • Rename the internal kpt-resource-id annotation from internal.config.k8s.io/kpt-resource-id to internal.config.kubernetes.io/kpt-resource-id to align with the config.kubernetes.io prefix used consistently across kyaml and kpt.
  • This resolves the last kpt-owned annotation using the short config.k8s.io prefix. The annotation is fully transient (added before a selector/exclusion pipeline step and removed immediately after), so this change has no user-visible impact.

Fixes #1257

Type of Change

  • Bug fix
  • Tests

Checklist

  • Code follows project style guidelines
  • Self-reviewed changes
  • Tests added/updated
  • Documentation added/updated
  • All tests and gating checks pass

AI Disclosure

  • I have used AI in the creation of this PR.

If so, please describe how:

  • Kiro to investigate the issue, identify the inconsistency, and create PR message.

…urce-id

Signed-off-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
@netlify

netlify Bot commented Jul 24, 2026

Copy link
Copy Markdown

Deploy Preview for kptdocs ready!

Name Link
🔨 Latest commit 5fdad44
🔍 Latest deploy log https://app.netlify.com/projects/kptdocs/deploys/6a63676724d0a50008f62f44
😎 Deploy Preview https://deploy-preview-4663--kptdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud

Copy link
Copy Markdown

@aravindtga aravindtga self-assigned this Jul 24, 2026
@aravindtga aravindtga changed the title Standardize annotation prefix: use config.kubernetes.io for kpt-reso… Standardize annotation prefix: use config.kubernetes.io instead of config.k8s.io Jul 24, 2026
@aravindtga
aravindtga marked this pull request as ready for review July 24, 2026 13:31
@aravindtga
aravindtga requested review from a team July 24, 2026 13:31
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Jul 24, 2026
@dosubot dosubot Bot added the lgtm label Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Resolve inconsistent use of config.k8s.io and config.kubernetes.io

2 participants