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.

Sign up for GitHub

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

Jump to bottom

Revert "Add apparmor support" #3993

Merged
merged 1 commit into from
Oct 25, 2023
Merged

Revert "Add apparmor support" #3993

merged 1 commit into from
Oct 25, 2023

Conversation

mythri-garaga
Copy link
Contributor

@mythri-garaga mythri-garaga commented Oct 25, 2023 •
edited
Loading

Summary

ECS Agent v1.78.0 is failing to start on Ubuntu platforms due to recently added AppArmor support - #3941

Oct 24 17:29:52 ip-10-232-231-104 audit[226910]: AVC apparmor="DENIED" operation="create" profile="ecs-default" pid=226910 comm="agent" family="netlink" sock_type="dgram" protocol=15 requested_mask="create" denied_mask="create"
Oct 24 17:29:52 ip-10-232-231-104 kernel: audit: type=1400 audit(1698193792.304:1451): apparmor="DENIED" operation="create" profile="ecs-default" pid=226910 comm="agent" family="netlink" sock_type="dgram" protocol=15 requested_mask="create" denied_mask="create"

This PR reverts the commits 96633a3bd22e0882ec185f16de48a715bdb4f887 and d20ceaa5179d2456666780f256a8c69e34ecb96b to fix the issue.

Implementation details

Testing

New tests cover the changes:

Description for the changelog

Does this PR include breaking model changes? If so, Have you added transformation functions?

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mythri-garaga
Revert "Add apparmor support"
afebb62 
This reverts commit 96633a3.

Revert "wip"

This reverts commit d20ceaa.
@mythri-garaga mythri-garaga requested a review from a team as a code owner October 25, 2023 23:36
@mythri-garaga mythri-garaga merged commit 8f6ad52 into aws:dev Oct 25, 2023
@singholt singholt mentioned this pull request Oct 26, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SreeeS SreeeS SreeeS approved these changes

@singholt singholt singholt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mythri-garaga @singholt @SreeeS @amazon-ecs-bot