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.

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

Bump kubevirtci #12677

Merged
merged 1 commit into from
Sep 18, 2024
Merged

Bump kubevirtci #12677

merged 1 commit into from
Sep 18, 2024

Conversation

kubevirt-bot
Copy link
Contributor

@kubevirt-bot kubevirt-bot commented Aug 27, 2024 •
edited
Loading

fc9be0a Fix prometheus port
eff1926 Allow opting out of frequent etcd flushes to storage
939e610 fix the busybox netcat not found issue in fedora-test-tooling for s390x.
4ad94f0 Label rook storage class
88a4f6b fix: Wait until istio cni files appear before copying them
4cc1018 Run ./hack/bump-cdi.sh
6149a01 Include manifests that are part of gocli when checking for prepull images
1b17b20 fix: Remove extra slash in file path and remove unneeded sed
b35649a Automatic bump of CentOS Stream to latest
374b5e9 bug: Move namespace and CRD files to names with higher alphabetical order to be created first
79bfd07 Run bazel run //robots/cmd/kubevirtci-bumper:kubevirtci-bumper -- -ensure-only-latest-three --k8s-provider-dir /home/prow/go/src/github.com/kubevirt/project-infra/../kubevirtci/cluster-provision/k8s --cluster-up-dir /home/prow/go/src/github.com/kubevirt/project-infra/../kubevirtci/cluster-up/cluster
ba145b2 bug: Fix KSM flag passing to the gocli
8501d22 Opts package
4f37d07 Automatic bump of CentOS Stream to latest

NONE

@kubevirt-bot kubevirt-bot added release-note-none Denotes a PR that doesn't merit a release note. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. sig/buildsystem Denotes an issue or PR that relates to changes in the build system. size/L labels Aug 27, 2024
@kubevirt-bot kubevirt-bot requested review from EdDev and enp0s3 August 27, 2024 00:02
@brianmcarey
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.29-sig-storage

@brianmcarey brianmcarey mentioned this pull request Aug 27, 2024
Merged
8 tasks
@brianmcarey
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.28-sig-network

@brianmcarey
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.29-sig-compute
/test pull-kubevirt-e2e-k8s-1.28-sig-operator

@kubevirt-bot kubevirt-bot force-pushed the bump-kubevirtci branch 5 times, most recently from 7b9c5fd to 0cca33c Compare August 29, 2024 12:05
@brianmcarey
Copy link
Member

/retest

@brianmcarey
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.29-sig-compute
/test pull-kubevirt-e2e-k8s-1.28-sig-operator

@brianmcarey
Copy link
Member

@dhiller there looks to be an issue with how the images have been built. There is another publish running at the moment - will try again after that.

@kubevirt-bot kubevirt-bot force-pushed the bump-kubevirtci branch 3 times, most recently from fb73274 to 3000434 Compare August 30, 2024 12:01
@brianmcarey
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.30-sig-network
/test pull-kubevirt-e2e-k8s-1.29-sig-network

@kubevirt-bot kubevirt-bot force-pushed the bump-kubevirtci branch 2 times, most recently from c5160a0 to 3f6f0ee Compare August 31, 2024 12:01
@aerosouund
Copy link
Member

aerosouund commented Aug 31, 2024 •
edited
Loading

@brianmcarey @dhiller
is it possible to go into the kubevirt testing infrastructure while a lane tests are running ? i want to inspect something
i tried running those istio vmi tests locally (i use ubuntu) and the tests that used istio with passt interface would fail with error

passt --one-off --socket /var/run/libvirt/qemu/run/passt/1-kubevirt-test-defaul-ua-default.socket --pid /var/run/libvirt/qemu/run/passt/1-kubevirt-test-defaul-ua-default-passt.pid --interface eth0 --log-file /var/run/kubevirt/passt.log --tcp-ports '~15000,~15001,~15004,~15006,~15008,~15009,~15020,~15021,~15053,~15090,1500,1501,22')
unexpected exit status 1: Failed to bind port 22 (Permission denied) for option '-t ~15000,~15001,~15004,~15006,~15008,~15009,~15020,~15021,~15053,~15090,1500,1501,22', exiting\n'

This is a problem with apparmor, and happens while binding to port 22 only

but apparently passt and ubuntu don't play nice with eachother according to #12333

and so i want to make sure if this is also whats happening here, as its showing similar behavior. all tests with passt are timing out because the vm remains scheduling

EDIT: I fixed this, not needed anymore

@kubevirt-bot kubevirt-bot force-pushed the bump-kubevirtci branch 5 times, most recently from eedae0a to 8810c07 Compare September 3, 2024 00:00
@brianmcarey
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.29-sig-performance

@brianmcarey
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.29-sig-monitoring

@xpivarc
Copy link
Member

xpivarc commented Sep 13, 2024

/hold
/test pull-kubevirt-e2e-k8s-1.29-sig-performance

@kubevirt-bot kubevirt-bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 13, 2024
@brianmcarey
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.31-sig-storage

@xpivarc
Copy link
Member

xpivarc commented Sep 17, 2024

/hold cancel
To refresh

@kubevirt-bot kubevirt-bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 17, 2024
@aerosouund
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.29-sig-performance

@aerosouund
Copy link
Member

/test pull-kubevirt-e2e-k8s-1.29-sig-performance

@aerosouund
Copy link
Member

aerosouund commented Sep 17, 2024 •
edited
Loading

@xpivarc @brianmcarey
I don't seem to have enough permissions to trigger the job, when i request it nothing happens and no run gets recorded

@brianmcarey
Copy link
Member

@xpivarc @brianmcarey I don't seem to have enough permissions to trigger the job, when i request it nothing happens and no run gets recorded

Its a large job so it can take time to get scheduled to the cluster - the jobs is triggered so we will just have to wait for it to get scheduled.

@kubevirt-bot
Bump kubevirtci
29c3acb 
[fc9be0a Fix prometheus port](kubevirt/kubevirtci#1272)
[eff1926 Allow opting out of frequent etcd flushes to storage](kubevirt/kubevirtci#1266)
[939e610 fix the busybox netcat not found issue in fedora-test-tooling for s390x.](kubevirt/kubevirtci#1268)
[4ad94f0 Label rook storage class](kubevirt/kubevirtci#1267)
[88a4f6b fix: Wait until istio cni files appear before copying them](kubevirt/kubevirtci#1262)
[4cc1018 Run ./hack/bump-cdi.sh](kubevirt/kubevirtci#1249)
[6149a01 Include manifests that are part of gocli when checking for prepull images](kubevirt/kubevirtci#1265)
[1b17b20 fix: Remove extra slash in file path and remove unneeded sed](kubevirt/kubevirtci#1258)
[b35649a Automatic bump of CentOS Stream to latest](kubevirt/kubevirtci#1257)
[374b5e9 bug: Move namespace and CRD files to names with higher alphabetical order to be created first](kubevirt/kubevirtci#1255)
[79bfd07 Run bazel run //robots/cmd/kubevirtci-bumper:kubevirtci-bumper -- -ensure-only-latest-three --k8s-provider-dir /home/prow/go/src/github.com/kubevirt/project-infra/../kubevirtci/cluster-provision/k8s --cluster-up-dir /home/prow/go/src/github.com/kubevirt/project-infra/../kubevirtci/cluster-up/cluster](kubevirt/kubevirtci#1256)
[ba145b2 bug: Fix KSM flag passing to the gocli](kubevirt/kubevirtci#1254)
[8501d22 Opts package](kubevirt/kubevirtci#1217)
[4f37d07 Automatic bump of CentOS Stream to latest](kubevirt/kubevirtci#1250)

```release-note
NONE
```

Signed-off-by: kubevirt-bot <[email protected]>
brianmcarey
brianmcarey reviewed Sep 18, 2024
View reviewed changes
Copy link
Member

@brianmcarey brianmcarey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Performance lane looks good after the last fix and all the latest e2e lanes are passing.

@kubevirt-bot kubevirt-bot added the lgtm Indicates that a PR is ready to be merged. label Sep 18, 2024
@brianmcarey
Copy link
Member

/cc @xpivarc

xpivarc
xpivarc reviewed Sep 18, 2024
View reviewed changes
Copy link
Member

@xpivarc xpivarc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@kubevirt-bot
Copy link
Contributor Author

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: xpivarc

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

The pull request process is described here

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

@kubevirt-bot kubevirt-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 18, 2024
@kubevirt-commenter-bot
Copy link

Required labels detected, running phase 2 presubmits:
/test pull-kubevirt-e2e-windows2016
/test pull-kubevirt-e2e-kind-1.27-vgpu
/test pull-kubevirt-e2e-kind-sriov
/test pull-kubevirt-e2e-k8s-1.30-ipv6-sig-network
/test pull-kubevirt-e2e-k8s-1.29-sig-network
/test pull-kubevirt-e2e-k8s-1.29-sig-storage
/test pull-kubevirt-e2e-k8s-1.29-sig-compute
/test pull-kubevirt-e2e-k8s-1.29-sig-operator
/test pull-kubevirt-e2e-k8s-1.30-sig-network
/test pull-kubevirt-e2e-k8s-1.30-sig-storage
/test pull-kubevirt-e2e-k8s-1.30-sig-compute
/test pull-kubevirt-e2e-k8s-1.30-sig-operator

@kubevirt-bot
Copy link
Contributor Author

kubevirt-bot commented Sep 18, 2024 •
edited
Loading

@kubevirt-bot: 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
pull-kubevirt-e2e-k8s-1.30-sig-compute-migrations d9914cd link true /test pull-kubevirt-e2e-k8s-1.30-sig-compute-migrations
pull-kubevirt-e2e-k8s-1.30-sig-compute-serial d9914cd link true /test pull-kubevirt-e2e-k8s-1.30-sig-compute-serial
pull-kubevirt-e2e-k8s-1.28-sig-network d9914cd link true /test pull-kubevirt-e2e-k8s-1.28-sig-network
pull-kubevirt-e2e-arm64 77af7fb link false /test pull-kubevirt-e2e-arm64

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.

@kubevirt-bot kubevirt-bot merged commit bd4fe0a into kubevirt:main Sep 18, 2024
36 of 38 checks passed
This was referenced Sep 26, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brianmcarey brianmcarey brianmcarey left review comments

@xpivarc xpivarc xpivarc left review comments

@EdDev EdDev Awaiting requested review from EdDev

@enp0s3 enp0s3 Awaiting requested review from enp0s3

Assignees

@brianmcarey brianmcarey

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. sig/buildsystem Denotes an issue or PR that relates to changes in the build system. size/L
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@kubevirt-bot @brianmcarey @aerosouund @akalenyu @xpivarc @kubevirt-commenter-bot