Skip to content

feat: use an "init service" to load the apparmor profile#63

Merged
MoisesGSalas merged 1 commit intomainfrom
mgs/use-init-container
Nov 25, 2025
Merged

feat: use an "init service" to load the apparmor profile#63
MoisesGSalas merged 1 commit intomainfrom
mgs/use-init-container

Conversation

@MoisesGSalas
Copy link
Member

This follows the same logic as the "permissions" service used by tutor core. The codejail-apparmor-loader service runs the command used previously by the init job.

It makes more sense to handling loading of the apparmor profile with an init service:

  • The profile is ephemeral, rebooting the host will require to load it again.
  • The profile is a dependency for the container to start. Things like database migrations, which are the main use case for init jobs, don't block the start of the main service container.

@MoisesGSalas MoisesGSalas force-pushed the mgs/use-init-container branch from fd8056e to 9a11838 Compare December 19, 2024 19:05
@MoisesGSalas MoisesGSalas marked this pull request as ready for review February 5, 2025 14:31
@MoisesGSalas MoisesGSalas requested a review from a team as a code owner February 5, 2025 14:32
@MoisesGSalas MoisesGSalas force-pushed the mgs/use-init-container branch from 9a11838 to 4bff93b Compare November 24, 2025 16:11
This follows the same logic as the "permissions" service used by tutor
core. The `codejail-apparmor-loader` service runs the command used
previously by the init job.

It makes more sense to handling loading of the apparmor profile with an
init service:

- The profile is ephemeral, rebooting the host will require to load it
  again.
- The profile is a dependency for the container to start. Things like
  database migrations, which are the main use case for init jobs, don't
  block the start of the main service container.
@MoisesGSalas MoisesGSalas force-pushed the mgs/use-init-container branch from 4bff93b to 80bd61a Compare November 24, 2025 17:14
@MoisesGSalas MoisesGSalas merged commit 7dc4b38 into main Nov 25, 2025
4 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.

2 participants