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

test #1453 with kubevirt/kubevirtci#645 #1454

Closed
wants to merge 2 commits into from
Closed

test #1453 with kubevirt/kubevirtci#645 #1454

wants to merge 2 commits into from

Conversation

ormergi
Copy link
Contributor

@ormergi ormergi commented Jul 18, 2021 •
edited
Loading

@kubevirt-bot kubevirt-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. labels Jul 18, 2021
@kubevirt-bot
Copy link
Contributor

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

@kubevirt-bot kubevirt-bot requested review from stu-gott and vladikr July 18, 2021 16:32
@kubevirt-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign fgimenez after the PR has been reviewed.
You can assign the PR to them by writing /assign @fgimenez in a comment when ready.

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

@ormergi
Copy link
Contributor Author

ormergi commented Jul 18, 2021

/rehearse

@ormergi ormergi force-pushed the test-check-up-sriov-conformance branch from fb40787 to d268e14 Compare July 18, 2021 20:22
@ormergi
Copy link
Contributor Author

ormergi commented Jul 19, 2021

/rehearse

@ormergi ormergi force-pushed the test-check-up-sriov-conformance branch from d268e14 to 5399286 Compare July 19, 2021 08:15
@ormergi
Copy link
Contributor Author

ormergi commented Jul 19, 2021

/rehearse

@ormergi ormergi force-pushed the test-check-up-sriov-conformance branch from 5399286 to d75b0b7 Compare July 19, 2021 08:18
@ormergi
Copy link
Contributor Author

ormergi commented Jul 19, 2021

/rehearse

@ormergi ormergi force-pushed the test-check-up-sriov-conformance branch from d75b0b7 to 0baa44b Compare July 19, 2021 08:29
@ormergi
Copy link
Contributor Author

ormergi commented Jul 19, 2021

/rehearse

1 similar comment
@ormergi
Copy link
Contributor Author

ormergi commented Jul 19, 2021

/rehearse

@ormergi ormergi force-pushed the test-check-up-sriov-conformance branch from 0baa44b to f021687 Compare July 19, 2021 13:06
@ormergi
Copy link
Contributor Author

ormergi commented Jul 19, 2021

/rehearse

3 similar comments
@ormergi
Copy link
Contributor Author

ormergi commented Jul 19, 2021

/rehearse

@ormergi
Copy link
Contributor Author

ormergi commented Aug 5, 2021

/rehearse

@ormergi
Copy link
Contributor Author

ormergi commented Aug 5, 2021

/rehearse

@ormergi ormergi force-pushed the test-check-up-sriov-conformance branch from f021687 to cb068cb Compare August 5, 2021 21:59
@ormergi
Copy link
Contributor Author

ormergi commented Aug 5, 2021

/rehearse

1 similar comment
@ormergi
Copy link
Contributor Author

ormergi commented Aug 5, 2021

/rehearse

@ormergi ormergi force-pushed the test-check-up-sriov-conformance branch from cb068cb to 3806f35 Compare August 6, 2021 08:06
@ormergi
Copy link
Contributor Author

ormergi commented Aug 6, 2021

/rehearse

@kubevirt-bot
Copy link
Contributor

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

Test name Commit Details Rerun command
rehearsal-pull-project-infra-prow-deploy-test f021687 link /test pull-project-infra-prow-deploy-test
rehearsal-check-up-kind-1.19-sriov 3806f35 link /test check-up-kind-1.19-sriov

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.

ormergi added 2 commits August 9, 2021 13:09
@ormergi
Run Kubevirt conformance tests on check-up SR-IOV lane
7a206ce 
Running Kubevirt conformance tests will prevent bad changes
to get in.
Also we wont need to do initial testing with a secondary PR on
kubevirt/kubevirt repo in order to make sure the lase passes.

Use 'check-cluster-up.sh' script in order to deploy Kubevirt
and run Kubevirt conformance tests from nightly builds.
SONOBUOY_EXTRA_ARGS env var used by 'check-cluster-up.sh'
script and its content passed to sounuboy command.
Here we use it in order to pass E2E_FOCUS to the tests
suite in order to run only SRIOV tests.

Signed-off-by: Or Mergi <[email protected]>
@ormergi
Fetch kubevirt/kubevirtci#645 changes
deb3eb6 
Signed-off-by: Or Mergi <[email protected]>
@ormergi ormergi force-pushed the test-check-up-sriov-conformance branch from 3806f35 to deb3eb6 Compare August 11, 2021 17:10
@kubevirt-bot kubevirt-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 19, 2021
@kubevirt-bot
Copy link
Contributor

@ormergi: PR needs rebase.

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.

@ormergi
Copy link
Contributor Author

ormergi commented Oct 21, 2021

/close

@kubevirt-bot
Copy link
Contributor

@ormergi: Closed this PR.

In response to this:

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

@ormergi ormergi deleted the test-check-up-sriov-conformance branch October 21, 2021 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stu-gott stu-gott Awaiting requested review from stu-gott

@vladikr vladikr Awaiting requested review from vladikr

Assignees
No one assigned
Labels
dco-signoff: yes Indicates the PR's author has DCO signed all their commits. 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/S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ormergi @kubevirt-bot