Skip to content

feat(run-integration-test)!: Improve action internals - #121

Merged
NickLarsenNZ merged 29 commits into
mainfrom
run-integration-test-otel
Aug 11, 2026
Merged

feat(run-integration-test)!: Improve action internals#121
NickLarsenNZ merged 29 commits into
mainfrom
run-integration-test-otel

Conversation

@NickLarsenNZ

@NickLarsenNZ NickLarsenNZ commented Aug 4, 2026

Copy link
Copy Markdown
Member

This PR contains the following changes:

  • Breaking: Add otlp-bearer-token input with is used to authenticate against our OTLP receiver.
  • Upgrade otel-operator and collector
  • Add local overlay for testing
  • Improve and fix the integration-test Slack notification
  • Enrich telemetry with more information

Warning

Downstream workflows will need to supply the otlp-bearer-token.

Testing with stackabletech/zookeeper-operator#1067

@NickLarsenNZ NickLarsenNZ self-assigned this Aug 4, 2026
@NickLarsenNZ
NickLarsenNZ requested a review from Techassi August 4, 2026 16:49
@NickLarsenNZ NickLarsenNZ changed the title chore(run-integration-test/kustomize): Upgrade otel-operator and collector, set bearer auth chore(run-integration-test): Add OTLP Bearer Auth Aug 4, 2026
@NickLarsenNZ
NickLarsenNZ force-pushed the run-integration-test-otel branch from 94d08fa to a503a30 Compare August 4, 2026 17:14
It is automatically set by GitHub Actions
I won't link it here, so people don't go snooping from the upstream issue linked as a comment in this commit.
Techassi
Techassi previously approved these changes Aug 4, 2026

@Techassi Techassi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice, looking good!

@NickLarsenNZ NickLarsenNZ changed the title chore(run-integration-test): Add OTLP Bearer Auth feat(run-integration-test)!: Add OTLP Bearer Auth Aug 4, 2026
Techassi
Techassi previously approved these changes Aug 4, 2026
…s.error_mode dynamic at runtime

The means we won't have to change action hashes just to check which things we might be missing.
@Techassi Techassi changed the title feat(run-integration-test)!: Add OTLP Bearer Auth feat(run-integration-test)!: Improve action internals Aug 6, 2026
Techassi and others added 5 commits August 6, 2026 22:36
It seems like that runs which were queued but never picked up to
run result in the .conclusion field to show an empty string. We
now ignore those when calculation the total attempts.

In a follow-up PR, we ideally want to re-fetch more previous attempts
if we detect one ore more runs with an empty conclusion to still get
the "x out of 5" health status.
@Techassi
Techassi force-pushed the run-integration-test-otel branch from 1d2389a to 57fb892 Compare August 11, 2026 10:40
@NickLarsenNZ
NickLarsenNZ merged commit 98f29ca into main Aug 11, 2026
12 checks passed
@NickLarsenNZ
NickLarsenNZ deleted the run-integration-test-otel branch August 11, 2026 11:07
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