Skip to content

Conversation

jonasbardino
Copy link
Contributor

@jonasbardino jonasbardino commented Sep 24, 2025

Expose new janitor service once it hits migrid next. Relies on the usual enable_X structure and is disabled by default in production until it has received more battle testing.

NB: CI will fail until migrid PR322 is merged.

@jonasbardino jonasbardino self-assigned this Sep 24, 2025
@jonasbardino jonasbardino added the enhancement New feature or request label Sep 24, 2025
Copy link
Contributor

@rasmunk rasmunk left a comment

Choose a reason for hiding this comment

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

Approved given that it can build the image in CI when the janitor feature is merged into migrid-sync

@jonasbardino jonasbardino marked this pull request as ready for review September 26, 2025 09:40
@jonasbardino
Copy link
Contributor Author

With migrid PR322+342 merged the CI build succeeds on rocky9. It still fails on rocky8 because the janitor is not in the old obsolete edge branch used in CI there. We are about to drop rocky8, too, so I'll proceed with the merge.

@jonasbardino jonasbardino merged commit 628653c into master Sep 26, 2025
2 of 6 checks passed
@jonasbardino jonasbardino deleted the add/expose-new-janitor-service-in-conf branch September 26, 2025 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants