Skip to content

Refresh scaleup_api_server secret.#1218

Merged
garloff merged 1 commit into
mainfrom
fix/refresh-scaleup
Jun 17, 2026
Merged

Refresh scaleup_api_server secret.#1218
garloff merged 1 commit into
mainfrom
fix/refresh-scaleup

Conversation

@garloff

@garloff garloff commented Jun 17, 2026

Copy link
Copy Markdown
Member

It's not a secret secret, so this is uncritical.
We use it as a canary since #1190.
As the webhook was broken before, we would not run the check and post pipelines on it. We should now do so, so refresh the secret.

It's not a secret secret, so this is uncritical.
We use it as a canary since #1190.
As the webhook was broken before, we would not run the check and post
pipelines on it. We should now do so, so refresh the secret.

Signed-off-by: Kurt Garloff <kurt@garloff.de>
@garloff garloff self-assigned this Jun 17, 2026
@garloff garloff added the housekeeping Internal housekeeping label Jun 17, 2026
@garloff garloff requested a review from fzakfeld June 17, 2026 05:34
@garloff

garloff commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

A note on this one:

  • zuul-merger does complex things when it calculates the future state
  • Somehow the fact that the webhook was not working for a while made it work with an outdated state. This may be what broke Add ScaleUp KaaS Offering to Compliance Monitor #1178
  • The change (which is just reencoding the same value) does touch the secure.yaml file so it gets in sync again
  • I'd like to observe the check (scs-check-adr-syntax - already ran successfully) and the post (scs-check-scs2) pipelines to succeed. The latter should run after this PR has seen the needed approvals.
  • Ideally, upon merging, the hourly pipeline (scs-check-scs2-main) starts working again

@fzakfeld fzakfeld left a comment

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.

Thanks for the fix, if other (actual) secrets are broken as well, let me know

@garloff

garloff commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

Well, your approval did not trigger the scs-check-scs2 pipeline, unfortunately. But that's a different thing to look at.
I'll merge and see if this fixes our hourly pipeline.

@garloff garloff merged commit b642307 into main Jun 17, 2026
7 checks passed
@garloff garloff deleted the fix/refresh-scaleup branch June 17, 2026 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

housekeeping Internal housekeeping

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants