Skip to content

WIP: dummy PR to test kube-proxy with nftables#17922

Closed
justinsb wants to merge 3 commits intokubernetes:masterfrom
justinsb:dummy_pr_for_nftables_testing
Closed

WIP: dummy PR to test kube-proxy with nftables#17922
justinsb wants to merge 3 commits intokubernetes:masterfrom
justinsb:dummy_pr_for_nftables_testing

Conversation

@justinsb
Copy link
Member

@justinsb justinsb commented Feb 3, 2026

No description provided.

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 3, 2026
@k8s-ci-robot k8s-ci-robot requested review from hakman and zetaab February 3, 2026 15:53
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign olemarkus for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Feb 3, 2026
@justinsb
Copy link
Member Author

justinsb commented Feb 3, 2026

/test pull-kops-aws-kindnet-rhel10arm64

Trying to establish a baseline: we do want to be sure that this test fails when we don't have #17920 (or whatever fix we figure out)

@justinsb
Copy link
Member Author

justinsb commented Feb 3, 2026

/test pull-kops-gce-ipalias-rhel10

@justinsb
Copy link
Member Author

justinsb commented Feb 3, 2026

So that's odd .... fails on AWS RHEL10 (good) but passes on GCE RHEL10 (unexpected).

I guess it is possible that the GCE version of rhel10 is different from the AWS version, specifically about whether or not the conntrack module is installed by default.

Added a line to nodeup to print the distribution, just so we can be sure of that.

/test pull-kops-gce-ipalias-rhel10

@justinsb justinsb force-pushed the dummy_pr_for_nftables_testing branch from e06d751 to 1afbfdf Compare February 3, 2026 16:56
@justinsb
Copy link
Member Author

justinsb commented Feb 3, 2026

/test pull-kops-gce-ipalias-rhel10

@justinsb
Copy link
Member Author

justinsb commented Feb 3, 2026

I0203 17:29:38.134697    4757 command.go:182] Detected distribution: {rpm rhel rhel10 10}

So it is rhel10

@justinsb
Copy link
Member Author

justinsb commented Feb 3, 2026

/test pull-kops-gce-ipalias-rhel10

/test pull-kops-aws-kindnet-rhel10arm64

Adding dumping of /proc/modules; it won't tell us why conntrack is loaded, but it will tell us that conntrack is loaded (or other differences if it is a different module)

@k8s-ci-robot
Copy link
Contributor

k8s-ci-robot commented Feb 3, 2026

@justinsb: 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-kops-aws-kindnet-rhel10arm64 e046231 link true /test pull-kops-aws-kindnet-rhel10arm64

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.

@justinsb
Copy link
Member Author

justinsb commented Feb 3, 2026

@justinsb
Copy link
Member Author

justinsb commented Feb 8, 2026

We merged #17920 , and I split out the diagnostic commits that would have been handy into #17936 and #17937

/close

@k8s-ci-robot
Copy link
Contributor

@justinsb: Closed this PR.

Details

In response to this:

We merged #17920 , and I split out the diagnostic commits that would have been handy into #17936 and #17937

/close

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.

@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 8, 2026
@k8s-ci-robot
Copy link
Contributor

PR needs rebase.

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.

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

Labels

area/nodeup cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants