Skip to content

Do not stop container on healthcheck failure - #774

Merged
mcalmer merged 2 commits into
uyuni-project:mainfrom
aaannz:temp_disable_healtcheck_stop
Apr 29, 2026
Merged

Do not stop container on healthcheck failure#774
mcalmer merged 2 commits into
uyuni-project:mainfrom
aaannz:temp_disable_healtcheck_stop

Conversation

@aaannz

@aaannz aaannz commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

What does this PR change?

This PR temporarily disable stopping container on health check failure until correct/agreed upon solution is found and implemented.

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/30492

  • 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 force-pushed the temp_disable_healtcheck_stop branch from 0a46664 to 102c04d Compare April 29, 2026 08:27
This adds 15s grace period so container has a chance to start
and after that stopped container means failure
@aaannz
aaannz requested a review from cbosdo April 29, 2026 11:26
@sonarqubecloud

Copy link
Copy Markdown

@mcalmer
mcalmer merged commit 731c3cf into uyuni-project:main Apr 29, 2026
12 checks passed
@aaannz
aaannz deleted the temp_disable_healtcheck_stop branch April 29, 2026 12:20
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.

4 participants