Skip to content

feat: add namespace metadata labels with pr info#172

Merged
vigneshrajsb merged 2 commits intomainfrom
codex/namespace-metadata-labels
Apr 16, 2026
Merged

feat: add namespace metadata labels with pr info#172
vigneshrajsb merged 2 commits intomainfrom
codex/namespace-metadata-labels

Conversation

@vigneshrajsb
Copy link
Copy Markdown
Contributor

@vigneshrajsb vigneshrajsb commented Apr 16, 2026

What changed

  • add namespace labels for PR metadata during namespace create and update
  • split repository metadata into lfc/org and lfc/repo labels
  • add focused coverage for namespace metadata behavior using generic test data

Why

  • make it easier to query and debug Lifecycle namespaces directly with label selectors

Before / After

Before:

metadata:
  labels:
    app.kubernetes.io/managed-by: lifecycle
    lfc/uuid: <build-uuid>
    lfc/type: <static|ephemeral>
    lfc/ttl-enable: <true|false>
    # and TTL timestamp labels when enabled

After:

metadata:
  labels:
    app.kubernetes.io/managed-by: lifecycle
    lfc/uuid: <build-uuid>
    lfc/type: <static|ephemeral>
    lfc/ttl-enable: <true|false>
    lfc/org: <repo-owner>
    lfc/repo: <repo-name>
    lfc/pull-request: <pr-number>
    lfc/author: <github-login>
    # and TTL timestamp labels when enabled

Impact

  • namespaces created or updated by Lifecycle now include lfc/org, lfc/repo, lfc/pull-request, and lfc/author

Validation

  • pnpm exec jest src/server/lib/__tests__/namespaceMetadata.test.ts --runInBand
  • pnpm exec jest src/server/services/__tests__/build.test.ts --runInBand

@vigneshrajsb vigneshrajsb marked this pull request as ready for review April 16, 2026 23:11
@vigneshrajsb vigneshrajsb requested a review from a team as a code owner April 16, 2026 23:11
@vigneshrajsb vigneshrajsb changed the title [codex] add namespace metadata labels feat: add namespace metadata labels with pr info Apr 16, 2026
@vigneshrajsb vigneshrajsb merged commit 094262e into main Apr 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants