Skip to content

[on-prem] Test - change keepalived VIP filtering syntax#2550

Closed
yboaron wants to merge 1 commit intoopenshift:masterfrom
yboaron:test_grep
Closed

[on-prem] Test - change keepalived VIP filtering syntax#2550
yboaron wants to merge 1 commit intoopenshift:masterfrom
yboaron:test_grep

Conversation

@yboaron
Copy link
Contributor

@yboaron yboaron commented Apr 28, 2021

Keepalived - Check CI gates when filtering VIP using grep with " instead of '

@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@yboaron
Copy link
Contributor Author

yboaron commented Apr 28, 2021

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 28, 2021
@yboaron
Copy link
Contributor Author

yboaron commented Apr 28, 2021

/test e2e-openstack
/test e2e-ovirt
/test e2e-vsphere

Also disable sigterm_handler for keepalived container
@yboaron
Copy link
Contributor Author

yboaron commented Apr 28, 2021

/test e2e-openstack
/test e2e-ovirt
/test e2e-vsphere

@stbenjam
Copy link
Member

/retest

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 29, 2021

@yboaron: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e-metal-ipi 136bdca link /test e2e-metal-ipi
ci/prow/e2e-gcp-op 136bdca link /test e2e-gcp-op
ci/prow/e2e-vsphere-upgrade 136bdca link /test e2e-vsphere-upgrade
ci/prow/e2e-ovn-step-registry 136bdca link /test e2e-ovn-step-registry
ci/prow/e2e-aws-serial 136bdca link /test e2e-aws-serial
ci/prow/e2e-aws-disruptive 136bdca link /test e2e-aws-disruptive
ci/prow/okd-e2e-aws 136bdca link /test okd-e2e-aws

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-ci
Copy link
Contributor

openshift-ci bot commented May 3, 2021

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

@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 3, 2021
@yboaron
Copy link
Contributor Author

yboaron commented May 3, 2021

/close

@openshift-ci-robot
Copy link
Contributor

@yboaron: Closed this PR.

Details

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.

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

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants