Skip to content

PS-1049 add branch-build annotation to pods#672

Merged
SorchaAbel merged 1 commit into
mainfrom
PS-1049-branch-annotation
Aug 18, 2025
Merged

PS-1049 add branch-build annotation to pods#672
SorchaAbel merged 1 commit into
mainfrom
PS-1049-branch-annotation

Conversation

@SorchaAbel
Copy link
Copy Markdown
Contributor

@SorchaAbel SorchaAbel commented Aug 18, 2025

Description

Customer request: enables querying OTEL traces by branch name. This allows distinguishing between test failures on main branch (which is not usually good) vs WIP branches (which is expected).

The OTEL collector automatically picks up pod annotations, so this will make branch information available in traces alongside existing build-url annotation.

Context

Support and #671

Changes

Added BuildBranchAnnotation constant in internal/controller/config/config.go
Set annotation in internal/controller/scheduler/scheduler.go following existing pattern

Deployment

Safe

Rollback

Safe to rollback

@SorchaAbel SorchaAbel requested a review from a team as a code owner August 18, 2025 06:06
Copy link
Copy Markdown
Contributor

@DrJosh9000 DrJosh9000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems obviously correct to me ✅

@SorchaAbel SorchaAbel merged commit 1ae3132 into main Aug 18, 2025
1 check passed
@SorchaAbel SorchaAbel deleted the PS-1049-branch-annotation branch August 18, 2025 06:11
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