Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

stable-2.4 | integration/k8s: Re-add DOCKER iptables chain #4633

Merged
merged 1 commit into from
May 9, 2022
Merged

stable-2.4 | integration/k8s: Re-add DOCKER iptables chain #4633

merged 1 commit into from
May 9, 2022

Conversation

Jakob-Naucke
Copy link
Member

in baremetal cleanup. Prior to saving iptables, all rules are removed,
but this is still required for the tracing test.

Fixes #4617
Signed-off-by: Jakob Naucke [email protected]

Backport of #4618

in baremetal cleanup. Prior to saving iptables, all rules are removed,
but this is still required for the tracing test.

Fixes #4617
Signed-off-by: Jakob Naucke <[email protected]>
@Jakob-Naucke Jakob-Naucke added architecture-specific Affects subset of architectures no-forward-port-needed Changed do not need to be applied to a newer branch / repository backport Change from a newer branch / repository area/ci labels Mar 28, 2022
@Jakob-Naucke Jakob-Naucke changed the title integration/k8s: Re-add DOCKER iptables chain stable-2.4 | integration/k8s: Re-add DOCKER iptables chain Mar 28, 2022
@snir911
Copy link
Member

snir911 commented Mar 28, 2022

/test

Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @Jakob-Naucke1

@Jakob-Naucke
Copy link
Member Author

/retest-s390x

@cmaf
Copy link

cmaf commented Mar 28, 2022

Adding DNM while jenkins-ci-s390x-ubuntu-20-04 is failing

@cmaf cmaf added the do-not-merge PR has problems or depends on another label Mar 28, 2022
@GabyCT
Copy link
Contributor

GabyCT commented Apr 12, 2022

ping @BbolroC should we close or you will work in this PR?

@BbolroC
Copy link
Member

BbolroC commented Apr 12, 2022

ping @BbolroC should we close or you will work in this PR?

Hi, unfortunately I cannot tell you about anything yet, because I am still learning the project itself. (and still half of my working time involved in other project) Maybe I can get back to you after going over an internal discussion for this or prioritizing the tasks Jakob has left (somewhere next week).

@jodh-intel
Copy link
Contributor

Hi @BbolroC - could you tal at the failing s390 test on this PR?

@BbolroC
Copy link
Member

BbolroC commented Apr 26, 2022

Hi @BbolroC - could you tal at the failing s390 test on this PR?

Hi, James. The failure was caused by the shortage of memories. which has been yesterday resolved. ATM, the PR test for s390x is excluded from the list. You can ignore it. I will reach Gaby today to get the test included again in the list.

@jodh-intel
Copy link
Contributor

@GabyCT, @dborquez - Do we have magic commands to re-trigger the *-devmapper CI's? If so, please could you add them to https://github.com/kata-containers/community/wiki/Controlling-the-CI.

@GabyCT
Copy link
Contributor

GabyCT commented Apr 27, 2022

@GabyCT, @dborquez - Do we have magic commands to re-trigger the *-devmapper CI's? If so, please could you add them to https://github.com/kata-containers/community/wiki/Controlling-the-CI.

/cc @fidencio who is the owner of that CI

@fidencio fidencio removed the do-not-merge PR has problems or depends on another label May 6, 2022
@fidencio
Copy link
Member

fidencio commented May 6, 2022

/test

@BbolroC
Copy link
Member

BbolroC commented May 6, 2022

/test-s390x

1 similar comment
@BbolroC
Copy link
Member

BbolroC commented May 6, 2022

/test-s390x

@BbolroC
Copy link
Member

BbolroC commented May 9, 2022

Could someone merge this?

@cmaf cmaf merged commit 71f21bb into kata-containers:stable-2.4 May 9, 2022
@Jakob-Naucke Jakob-Naucke deleted the backport-docker-chain branch May 27, 2022 22:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
architecture-specific Affects subset of architectures area/ci backport Change from a newer branch / repository no-forward-port-needed Changed do not need to be applied to a newer branch / repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants