Skip to content

feat: standardize Kueue config and restore trace propagation#15

Merged
tecarter94 merged 1 commit into
sbomer-project:mainfrom
tecarter94:kueue-improvements
Apr 29, 2026
Merged

feat: standardize Kueue config and restore trace propagation#15
tecarter94 merged 1 commit into
sbomer-project:mainfrom
tecarter94:kueue-improvements

Conversation

@tecarter94
Copy link
Copy Markdown
Member

Align Kueue Helm config with syft-generator.
Move ResourceFlavor to sbomer-platform.
Restore manual trace context propagation in TaskReconciler.

Align Kueue Helm config with syft-generator.
Move ResourceFlavor to sbomer-platform.
Restore manual trace context propagation in TaskReconciler.
ObjectMapper objectMapper;

@Inject
Tracer tracer;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Out of curiosity, what is the difference in using Tracer vs using the @WithSpan?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good question! https://github.com/sbomer-project/sbomer-engineering-hub/blob/main/docs/modules/ROOT/pages/observability-standards.adoc#kubernetes-resource-change

Basically if we just rely on @WithSpan we end up with an orphaned TaskReconciler root span because TaskReconciler is triggered by an async Kubernetes event - it has no knowledge of the parent span. It would end up disconnected to the generation request that triggered it and become part of a new trace.

Copy link
Copy Markdown
Contributor

@thavlice thavlice left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@mertbugrabicak mertbugrabicak left a comment

Choose a reason for hiding this comment

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

LGTM!

@tecarter94 tecarter94 merged commit ea45c39 into sbomer-project:main Apr 29, 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.

3 participants