Skip to content

Generate template only after secrets are created - #771

Merged
mcalmer merged 2 commits into
uyuni-project:mainfrom
aaannz:fix_missing_secrets_on_upgrade
Apr 21, 2026
Merged

Generate template only after secrets are created#771
mcalmer merged 2 commits into
uyuni-project:mainfrom
aaannz:fix_missing_secrets_on_upgrade

Conversation

@aaannz

@aaannz aaannz commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

What does this PR change?

  • Generate template only after secrets are created
  • Overwrite existing secrets during installation
  • Properly remove all secrets on uninstall
  • Increase the healthcheck maximum wait time to 5 minutes

Codespace

Check if you already have a running container clicking on Running CodeSpace

Create CodeSpace About billing for Github Codespaces CodeSpace Billing Summary CodeSpace Limit

Test coverage

  • No tests: add explanation

  • DONE

Links

Issue(s): https://github.com/SUSE/spacewalk/issues/30380

  • DONE

Changelogs

Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository

If you don't need a changelog check, please mark this checkbox:

  • No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)

Before you merge

Check How to branch and merge properly!

Comment thread mgradm/shared/podman/podman.go
Comment thread shared/connection.go
To avoid reusing old passwords, we need to force recreating user
secrets if they are already present.

Uninstall also did not remove those secrets, fixed.

This commit also adjust heatlhcheck timeout to 5 minutes as the
server container can now take a while until it is ready.
@aaannz
aaannz force-pushed the fix_missing_secrets_on_upgrade branch from 1436235 to 7e1b6ed Compare April 21, 2026 08:59
@sonarqubecloud

Copy link
Copy Markdown

@mcalmer
mcalmer merged commit 90aa21a into uyuni-project:main Apr 21, 2026
12 checks 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