Skip to content

chore(spartan): run setup-otel-resource inside aztec containers#12582

Closed
Maddiaa0 wants to merge 3 commits intomasterfrom
md/move-otel-into-container
Closed

chore(spartan): run setup-otel-resource inside aztec containers#12582
Maddiaa0 wants to merge 3 commits intomasterfrom
md/move-otel-into-container

Conversation

@Maddiaa0
Copy link
Member

@Maddiaa0 Maddiaa0 commented Mar 7, 2025

Overview

After hotfix #12580 kind-smoke-test time shot up again as there is another init container.

This just runs the script within the service getting the time back down to 4 minutes

@Maddiaa0 Maddiaa0 requested a review from alexghr March 7, 2025 17:41
- "/bin/bash"
- "-c"
- |
source /scripts/setup-otel-resource.sh
Copy link
Contributor

Choose a reason for hiding this comment

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

FWIW this is sourcing the script file, not the generated config

@alexghr
Copy link
Contributor

alexghr commented Mar 13, 2025

The helm template approach looks good to me but I think we need to execute the script and then load the config file (unless I missed where the script was modified to just export env vars directly)

@Maddiaa0
Copy link
Member Author

stale

@Maddiaa0 Maddiaa0 closed this Jul 24, 2025
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