Skip to content

Make uyuni-server dependent on uyuni-db#782

Merged
aaannz merged 2 commits into
uyuni-project:mainfrom
aaannz:add_weak_dependency
May 14, 2026
Merged

Make uyuni-server dependent on uyuni-db#782
aaannz merged 2 commits into
uyuni-project:mainfrom
aaannz:add_weak_dependency

Conversation

@aaannz

@aaannz aaannz commented May 4, 2026

Copy link
Copy Markdown
Contributor

What does this PR change?

Add as a weak dependency so that server is not shut down in case of database maintenance.

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

  • Unit tests were updated

  • DONE

Links

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

  • 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!

@aaannz
aaannz requested a review from cbosdo May 4, 2026 10:16
Comment thread mgradm/shared/templates/attestationServiceTemplate.go Outdated

@mcalmer mcalmer 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.

Please check also for filesystem dependency.
When /var/lib/containers/storage/volumes/ is a mounted filesystem, it must be available before the uyuni-db is starting.
I saw a logfile where uyuni-db reported "no such file or directory"

@aaannz
aaannz force-pushed the add_weak_dependency branch from d37835e to a82b7c7 Compare May 4, 2026 13:45
@aaannz

aaannz commented May 4, 2026

Copy link
Copy Markdown
Contributor Author

Please check also for filesystem dependency. When /var/lib/containers/storage/volumes/ is a mounted filesystem, it must be available before the uyuni-db is starting. I saw a logfile where uyuni-db reported "no such file or directory"

Added in latest commit.

Comment thread mgradm/shared/templates/pgsqlServiceTemplate.go
Comment thread mgradm/shared/templates/serviceTemplate.go
aaannz added 2 commits May 14, 2026 08:50
Add as a weak dependency so that server is not shut down in case
of database maintenance
@aaannz
aaannz force-pushed the add_weak_dependency branch from a82b7c7 to eb2d174 Compare May 14, 2026 06:51
@sonarqubecloud

Copy link
Copy Markdown

@aaannz
aaannz merged commit 52fb541 into uyuni-project:main May 14, 2026
12 checks passed
@aaannz
aaannz deleted the add_weak_dependency branch May 14, 2026 11:08
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