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

Remove tests checking that no apparmor profiles are complaining #7308

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

legoktm
Copy link
Member

@legoktm legoktm commented Oct 29, 2024

Status

Ready for review

Description of Changes

Ubuntu noble now has some transmission-related profiles in complain mode. We don't really care about this, so just remove the test. Ideally distros will continue to add more apparmor profiles, even if they're complain-only to start.

Testing

How should the reviewer test this PR?

  • visual review

Deployment

Any special considerations for deployment? n/a

Ubuntu noble now has some transmission-related profiles in complain
mode. We don't really care about this, so just remove the test. Ideally
distros will continue to add more apparmor profiles, even if they're
complain-only to start.
@legoktm legoktm added the noble Ubuntu Noble related work label Oct 29, 2024
@legoktm legoktm requested a review from a team as a code owner October 29, 2024 17:44
@zenmonkeykstop
Copy link
Contributor

Is transmission actually being installed tho? In that case we should be removing said package, not disabling tests that get caught up on it.

@legoktm
Copy link
Member Author

legoktm commented Nov 4, 2024 via email

@legoktm
Copy link
Member Author

legoktm commented Nov 6, 2024

Here's all the profiles the base apparmor package installs: https://gist.github.com/legoktm/c15059f372eab85754829b6984237c67. Since it's in the base package, there's nothing we can really do and I don't think overriding/disabling them would be worth it.

Copy link
Contributor

@zenmonkeykstop zenmonkeykstop left a comment

Choose a reason for hiding this comment

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

LGTM based on visual review, discussion.

@zenmonkeykstop zenmonkeykstop added this pull request to the merge queue Nov 8, 2024
Merged via the queue into develop with commit a670288 Nov 8, 2024
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
noble Ubuntu Noble related work
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants