Skip to content

k8s 1.25.0 #27526

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.

Sign up for GitHub

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

Jump to bottom
Merged
openshift-merge-robot merged 22 commits into from
Nov 7, 2022
Merged

k8s 1.25.0 #27526

openshift-merge-robot merged 22 commits into from
Nov 7, 2022

Conversation

@soltysh
Copy link
Contributor

@soltysh soltysh commented Nov 4, 2022

No description provided.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 4, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 4, 2022

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@soltysh soltysh marked this pull request as ready for review November 4, 2022 17:47
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 4, 2022
@soltysh
Copy link
Contributor Author

soltysh commented Nov 4, 2022

/test all

@soltysh soltysh closed this Nov 4, 2022
@soltysh soltysh reopened this Nov 4, 2022
@bparees
Copy link
Contributor

bparees commented Nov 4, 2022

/payload 4.13 nightly blocking
/payload 4.13 ci blocking

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 4, 2022

@bparees: An error was encountered. No known errors were detected, please see the full error message for details.

Full error message. could not resolve jobs for 4.13 nightly blocking: failed to request latest release: server responded with 400: error: unknown release stream specified: 4.13.0-0.nightly

Please contact an administrator to resolve this issue.

@openshift-ci openshift-ci bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. e2e-images-update Related to images used by e2e tests vendor-update Touching vendor dir or related files labels Nov 4, 2022
@bparees bparees mentioned this pull request Nov 4, 2022
Closed
@soltysh
Copy link
Contributor Author

soltysh commented Nov 4, 2022

/retest-required

bparees and others added 14 commits November 7, 2022 13:21
@bparees @soltysh
Switch imports to ginkgo v2
98d9c70
@soltysh
Switch to go-ldap/v3
cd37ef2 
Following changes from openshift/library-go#1378
@soltysh
Add newlines to SimultaneousPodIPController
1bc22fc
@bparees @soltysh
Adapt to v2 by migrating use of extensions/table package
67f6c37
@bparees @soltysh
Adapt to v2 by migrating use of config.GinkgoConfig to ginkgo.GinkgoC…
0a9037b 
…onfiguration()
@soltysh
Adopt to v2 by migrating from CurrentGinkgoTestDescription() to Curre…
184bbf0 
…ntSpecReport()
@bparees @soltysh
Remove dead code - ExecuteTest
ebfb46a 
ExecuteTest was never executed anywhere, so not to confuse users how we
run tests this is being removed.
@soltysh
Use rhel-8-release-golang-1.19-openshift-4.12 builder
01f33a4
@soltysh
Update dependencies
06e50a1
@soltysh
React to k8s 1.25 changes
8d2a59e
@soltysh
Update ETCD storage test alpha versions
bbf9305
@soltysh
Update test images
e813df2
@soltysh
Add registry.k8s.io/pause:3.8
896bcad 
registry.k8s.io/pause:3.8 has problems with uncompressed windows
layers which are not accepted by quay.io
@soltysh
ginkgo v2 adaptation
93d4ec9
@soltysh
Drop [Suite:openshift/registry/serial] from Image signature worklow t…
44f5015 
…ests
@soltysh
Update gofmt with golang 1.19
227a778
@soltysh
Fix TestBackendSampler_checkConnection unit test
88740cc
@soltysh
make update
ba3feb4
@soltysh
Switch from waiting for running pod to wait for a completed pod in im…
bc2fbb1 
…age tests

The WaitForPod* methods has changed in kubernetes/kubernetes#109704
currently they are very strict about their expected state, so a quick
run, like in most cases in these tests where we just print usage and
complete won't likely catch running pod, so instead it's best to wait
for it to finish.
@soltysh
Do not panic when test annotation is not found
2f48719
@soltysh
Only invoke BuildTree one time
f017528
@soltysh
Update ginkgo dependency
44ca1e5
@soltysh soltysh force-pushed the rebase-1.25.0-master branch from c57ce8a to 44ca1e5 Compare November 7, 2022 12:23
@stbenjam
Copy link
Member

stbenjam commented Nov 7, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 7, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 7, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: soltysh, stbenjam

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

@soltysh soltysh mentioned this pull request Nov 7, 2022
Merged
@soltysh
Copy link
Contributor Author

soltysh commented Nov 7, 2022

/override ci/prow/e2e-gcp-image-ecosystem
this is known to fail on 2 tests which are disabled in #27529

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 7, 2022

@soltysh: /override requires failed status contexts, check run or a prowjob name to operate on.
The following unknown contexts/checkruns were given:

  • ci/prow/e2e-gcp-image-ecosystem

Only the following failed contexts/checkruns were expected:

  • ci/prow/e2e-agnostic-ovn-cmd
  • ci/prow/e2e-aws-csi
  • ci/prow/e2e-aws-ovn-cgroupsv2
  • ci/prow/e2e-aws-ovn-fips
  • ci/prow/e2e-aws-ovn-image-registry
  • ci/prow/e2e-aws-ovn-serial
  • ci/prow/e2e-aws-ovn-single-node
  • ci/prow/e2e-aws-ovn-single-node-serial
  • ci/prow/e2e-aws-ovn-single-node-upgrade
  • ci/prow/e2e-aws-ovn-upgrade
  • ci/prow/e2e-gcp-csi
  • ci/prow/e2e-gcp-ovn
  • ci/prow/e2e-gcp-ovn-builds
  • ci/prow/e2e-gcp-ovn-image-ecosystem
  • ci/prow/e2e-gcp-ovn-rt-upgrade
  • ci/prow/e2e-gcp-ovn-upgrade
  • ci/prow/e2e-metal-ipi-ovn-ipv6
  • ci/prow/e2e-metal-ipi-sdn
  • ci/prow/e2e-openstack-ovn
  • ci/prow/images
  • ci/prow/lint
  • ci/prow/unit
  • ci/prow/verify
  • ci/prow/verify-deps
  • pull-ci-openshift-origin-master-e2e-agnostic-ovn-cmd
  • pull-ci-openshift-origin-master-e2e-aws-csi
  • pull-ci-openshift-origin-master-e2e-aws-ovn-cgroupsv2
  • pull-ci-openshift-origin-master-e2e-aws-ovn-fips
  • pull-ci-openshift-origin-master-e2e-aws-ovn-image-registry
  • pull-ci-openshift-origin-master-e2e-aws-ovn-serial
  • pull-ci-openshift-origin-master-e2e-aws-ovn-single-node
  • pull-ci-openshift-origin-master-e2e-aws-ovn-single-node-serial
  • pull-ci-openshift-origin-master-e2e-aws-ovn-single-node-upgrade
  • pull-ci-openshift-origin-master-e2e-aws-ovn-upgrade
  • pull-ci-openshift-origin-master-e2e-gcp-csi
  • pull-ci-openshift-origin-master-e2e-gcp-ovn
  • pull-ci-openshift-origin-master-e2e-gcp-ovn-builds
  • pull-ci-openshift-origin-master-e2e-gcp-ovn-image-ecosystem
  • pull-ci-openshift-origin-master-e2e-gcp-ovn-rt-upgrade
  • pull-ci-openshift-origin-master-e2e-gcp-ovn-upgrade
  • pull-ci-openshift-origin-master-e2e-metal-ipi-ovn-ipv6
  • pull-ci-openshift-origin-master-e2e-metal-ipi-sdn
  • pull-ci-openshift-origin-master-e2e-openstack-ovn
  • pull-ci-openshift-origin-master-images
  • pull-ci-openshift-origin-master-lint
  • pull-ci-openshift-origin-master-unit
  • pull-ci-openshift-origin-master-verify
  • pull-ci-openshift-origin-master-verify-deps
  • tide

If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.

Details

In response to this:

/override ci/prow/e2e-gcp-image-ecosystem
this is known to fail on 2 tests which are disabled in #27529

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/test-infra repository.

@soltysh
Copy link
Contributor Author

soltysh commented Nov 7, 2022

/override ci/prow/e2e-gcp-ovn-image-ecosystem
this is known to fail on 2 tests which are disabled in #27529

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 7, 2022

@soltysh: Overrode contexts on behalf of soltysh: ci/prow/e2e-gcp-ovn-image-ecosystem

Details

In response to this:

/override ci/prow/e2e-gcp-ovn-image-ecosystem
this is known to fail on 2 tests which are disabled in #27529

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/test-infra repository.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 7, 2022

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

Test name Commit Details Required Rerun command
ci/prow/e2e-gcp-builds c57ce8a link true /test e2e-gcp-builds
ci/prow/e2e-gcp-image-ecosystem c57ce8a link true /test e2e-gcp-image-ecosystem
ci/prow/e2e-metal-ipi-ovn-ipv6 44ca1e5 link false /test e2e-metal-ipi-ovn-ipv6
ci/prow/e2e-aws-ovn-upgrade 44ca1e5 link false /test e2e-aws-ovn-upgrade
ci/prow/e2e-aws-ovn-single-node-upgrade 44ca1e5 link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-aws-ovn-single-node-serial 44ca1e5 link false /test e2e-aws-ovn-single-node-serial
ci/prow/e2e-openstack-ovn 44ca1e5 link false /test e2e-openstack-ovn

Full PR test history. Your PR dashboard.

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/test-infra repository. I understand the commands that are listed here.

@openshift-merge-robot openshift-merge-robot merged commit c7563a1 into openshift:master Nov 7, 2022
@soltysh soltysh deleted the rebase-1.25.0-master branch November 8, 2022 11:45
@bertinatto bertinatto mentioned this pull request Nov 8, 2022
Merged
@hasbro17 hasbro17 mentioned this pull request Nov 8, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adambkaplan adambkaplan Awaiting requested review from adambkaplan

@awgreene awgreene Awaiting requested review from awgreene

Assignees

@stbenjam stbenjam

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. e2e-images-update Related to images used by e2e tests lgtm Indicates that a PR is ready to be merged. vendor-update Touching vendor dir or related files

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@soltysh @bparees @stbenjam @openshift-merge-robot