Skip to content

Conversation

aanm
Copy link
Member

@aanm aanm commented May 23, 2022

No description provided.

aanm added 2 commits May 23, 2022 21:43
This reverts commit cd2a8eb.

Signed-off-by: André Martins <andre@cilium.io>
Commit e70a0fe modified the wrong Dockerfile. Although the main
issue was fixed by commit cd2a8eb, the change performed in this
commit is the correct one.

[ upstream commit 67f74ff ]

Fixes: e70a0fe ("images/cilium: remove cilium group from Dockerfile")
Reported-by: Joe Stringer <joe@cilium.io>
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm requested a review from joestringer May 23, 2022 19:45
@aanm aanm requested a review from a team as a code owner May 23, 2022 19:45
@maintainer-s-little-helper maintainer-s-little-helper bot added the kind/backports This PR provides functionality previously merged into master. label May 23, 2022
@aanm
Copy link
Member Author

aanm commented May 23, 2022

/test-backport-1.9

Job 'Cilium-PR-Runtime-4.9' failed:

Click to show.

Test Name

RuntimePolicies Tests Egress To World

Failure Output

FAIL: Unable to delete all policies

If it is a flake and a GitHub issue doesn't already exist to track it, comment /mlh new-flake Cilium-PR-Runtime-4.9 so I can create one.

Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Thanks!

@aanm
Copy link
Member Author

aanm commented May 23, 2022

/test-1.17-5.4

@aanm aanm merged commit fd5d932 into cilium:v1.9 May 24, 2022
@aanm aanm deleted the pr/fix-dockerfile-group branch May 24, 2022 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants