Skip to content

🌱 E2E: Use Kubernetes v1.36.1 in tests#3208

Open
nikParasyr wants to merge 1 commit into
kubernetes-sigs:mainfrom
nikParasyr:k8s_1.36
Open

🌱 E2E: Use Kubernetes v1.36.1 in tests#3208
nikParasyr wants to merge 1 commit into
kubernetes-sigs:mainfrom
nikParasyr:k8s_1.36

Conversation

@nikParasyr

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
E2E: Use Kubernetes v1.36.1 in tests

TODOs:

  • squashed commits
  • if necessary:
    • includes documentation
    • adds unit tests

@netlify

netlify Bot commented Jun 10, 2026

Copy link
Copy Markdown

Deploy Preview for kubernetes-sigs-cluster-api-openstack ready!

Name Link
🔨 Latest commit 559d2cd
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-sigs-cluster-api-openstack/deploys/6a31464fad2b490008a35c72
😎 Deploy Preview https://deploy-preview-3208--kubernetes-sigs-cluster-api-openstack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@k8s-triage-robot

Copy link
Copy Markdown

Unknown CLA label state. Rechecking for CLA labels.

Send feedback to sig-contributor-experience at kubernetes/community.

/check-cla
/easycla

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 10, 2026
@lentzi90

Copy link
Copy Markdown
Contributor

Not sure what the issue is. The changes looks good to me. Retrying, hoping it was just some random hickup.
/test pull-cluster-api-provider-openstack-e2e-full-test

@lentzi90

Copy link
Copy Markdown
Contributor

Failed in a different place. I am starting to wonder if we have some incompatibility. Maybe some older version of CAPI that we use does not like the newer version?
/test pull-cluster-api-provider-openstack-e2e-full-test

@nikParasyr

Copy link
Copy Markdown
Contributor Author

I am starting to wonder if we have some incompatibility. Maybe some older version of CAPI that we use does not like the newer version?

I'll try to look into this, probably tomorrow.
for now ill kick another run to have more input for tomorrow.

/test pull-cluster-api-provider-openstack-e2e-full-test

@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 12, 2026
@k8s-ci-robot

Copy link
Copy Markdown
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 mandre 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 size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jun 12, 2026
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jun 12, 2026
@nikParasyr

Copy link
Copy Markdown
Contributor Author

/test pull-cluster-api-provider-openstack-e2e-full-test

@nikParasyr

Copy link
Copy Markdown
Contributor Author

/test pull-cluster-api-provider-openstack-e2e-full-test

1 similar comment
@nikParasyr

Copy link
Copy Markdown
Contributor Author

/test pull-cluster-api-provider-openstack-e2e-full-test

@nikParasyr

Copy link
Copy Markdown
Contributor Author

/retest

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

@nikParasyr: 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-cluster-api-provider-openstack-e2e-full-test e2a6cbf link false /test pull-cluster-api-provider-openstack-e2e-full-test

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.

@nikParasyr

Copy link
Copy Markdown
Contributor Author

@lentzi90 I think the failure now is because capi v1.10 does not support k8s v1.36. Ill need to check this a bit and potentially update just the e2e to use v1.35 (i hope to have some time later this week).
In the meantime ill cherry pick my 2 findings to different PRs as i think they are legit and not v1.36 dependant and should be merged

@lentzi90

Copy link
Copy Markdown
Contributor

Sounds good! We can have a different version for the clusterctl upgrade test starting from CAPI v1.10. All the other tests should handle v1.36 then 🙂

@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 16, 2026
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

4 participants