Skip to content

Use capzicicommunity register for calico node and add defender excluson for containerd tmpmount - #588

Open
marosset wants to merge 1 commit into
kubernetes-sigs:masterfrom
marosset:calico-node-fixes
Open

Use capzicicommunity register for calico node and add defender excluson for containerd tmpmount#588
marosset wants to merge 1 commit into
kubernetes-sigs:masterfrom
marosset:calico-node-fixes

Conversation

@marosset

Copy link
Copy Markdown
Contributor

This should fix the issues being see in https://testgrid.k8s.io/sig-release-master-informing#capz-windows-master&exclude-non-failed-tests= and https://prow.k8s.io/view/gs/kubernetes-ci-logs/logs/ci-kubernetes-e2e-capz-master-windows/2090774807804121088

I think the actual issue is the calico node image not mounting in sufficient time due to defender scanning but we should use the czapzicommunity pull-through azure container registery for all images coming form quay.io anyways for reliability

@kubernetes-prow kubernetes-prow Bot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Aug 21, 2026
@kubernetes-prow kubernetes-prow Bot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Aug 21, 2026
@kubernetes-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: marosset

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubernetes-prow kubernetes-prow Bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 21, 2026
@zylxjtu

zylxjtu commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

agree that we should unify the image registery to capzcommunity, but for the other change, the failures seems happening on linux node (control plane) instead of windows node? https://storage.googleapis.com/kubernetes-ci-logs/logs/ci-kubernetes-e2e-capz-master-windows/2090820362051784704/build-log.txt

@marosset

Copy link
Copy Markdown
Contributor Author

agree that we should unify the image registery to capzcommunity, but for the other change, the failures seems happening on linux node (control plane) instead of windows node? https://storage.googleapis.com/kubernetes-ci-logs/logs/ci-kubernetes-e2e-capz-master-windows/2090820362051784704/build-log.txt

yes, doh

…ion for containerd tmpmount

Signed-off-by: Mark Rossetti <marosset@microsoft.com>
@kubernetes-prow kubernetes-prow Bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 21, 2026
@kubernetes-prow

Copy link
Copy Markdown

@marosset: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-e2e-capz-windows-2025-extension cb69bf3 link false /test pull-e2e-capz-windows-2025-extension

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants