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

Automated cherry pick of #6944: Reconcile host-network Pods after agent is #6970

Open
wants to merge 1 commit into
base: release-2.1
Choose a base branch
from

Conversation

wenyingd
Copy link
Contributor

@wenyingd wenyingd commented Feb 5, 2025

Cherry pick of #6944 on release-2.1.

#6944: Reconcile host-network Pods after agent is

For details on the cherry pick process, see the cherry pick requests page.

This change is to support reconciling the host-network Pods on Windows because
k8s expects to let CNI manage such Pods as long as they are not using
host-process containers.

Antrea received the CmdAdd request for such Pods when they were created, so
they should be included in the Pod reconcile list after agent is restarted.

Signed-off-by: Wenying Dong <[email protected]>
@wenyingd wenyingd added the kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release label Feb 5, 2025
@wenyingd wenyingd requested a review from antoninbas February 5, 2025 01:36
@wenyingd
Copy link
Contributor Author

wenyingd commented Feb 5, 2025

/test-all
/test-windows-all

@luolanzone
Copy link
Contributor

/test-windows-all

@luolanzone
Copy link
Contributor

@antoninbas could you take a look at this cherry-pick PR? Since the patch release 2.1.1 is still on-going, I suppose we can wait this one to merge and update the release log of v2.1.1?

@antoninbas
Copy link
Contributor

@luolanzone The cherry-pick looks good to me, but the Windows tests failed

@luolanzone
Copy link
Contributor

Hi @antoninbas , the failure is caused by a docker image pulling failure because of one domain is blocked in our CI env. Shuyang submitted a request to allow the domain yesterday, it's now moving forward, but we may need to wait for now. @XinShuYang can you rerun the CI in this PR when the env is recovered? Thanks.

@XinShuYang
Copy link
Contributor

@XinShuYang can you rerun the CI in this PR when the env is recovered? Thanks.

Sure, I will rerun the test once the request is approved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants