Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: sumac upgrade #59

Merged
merged 1 commit into from
Dec 17, 2024
Merged

feat: sumac upgrade #59

merged 1 commit into from
Dec 17, 2024

Conversation

MoisesGSalas
Copy link
Member

@MoisesGSalas MoisesGSalas commented Oct 24, 2024

Keep in mind that a lot of the extra changes were part of this PR (#60)

setup.py Outdated Show resolved Hide resolved
@MoisesGSalas MoisesGSalas marked this pull request as ready for review December 17, 2024 18:01
@MoisesGSalas MoisesGSalas requested a review from a team as a code owner December 17, 2024 18:01
Copy link
Contributor

@magajh magajh left a comment

Choose a reason for hiding this comment

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

LGTM, just a small comment about a GA version

.github/workflows/ci.yml Outdated Show resolved Hide resolved
@MoisesGSalas MoisesGSalas force-pushed the sumac branch 2 times, most recently from db24a20 to 4d05c74 Compare December 17, 2024 20:23
Adjust the globing and abi rules in the apparmor profile:

Newer versions of ubuntu (>24.04) do not pin the AppArmor Policy feature
ABI which causes certain rules to not be enforced. We include an abi
rule to use the relatively common 3.0 policy whenever it's available in
the system, if it's not available we rely on the default fallback
behaviour. The 3.0 policy should be present on any system using
AppArmor>3.x (e.g. Ubuntu 22.04 or newer).

The globbing rules in the profile were adjusted to cover a larger range
of python versions and avoid creating new profiles for different
versions of python used by the sandbox environment.

To load the profile we need at least AppArmor 3.0, to avoid confusion in
the future we pin the alpine base image and define a proper tag for the
apparmorloader image.
@MoisesGSalas MoisesGSalas merged commit 8a9e678 into main Dec 17, 2024
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.

3 participants