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.

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

[WIP] Add IPsec package with IPv6 fix #471

Closed

Conversation

markdgray
Copy link
Contributor

@markdgray markdgray commented Mar 18, 2021

DO NOT REVIEW - This PR is for testing

Signed-off-by: Mark Gray mark.d.gray@redhat.com

- What this PR does and why is it needed

- Special notes for reviewers

- How to verify it

- Description for the changelog

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 18, 2021
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: markdgray
To complete the pull request process, please assign danwinship after the PR has been reviewed.
You can assign the PR to them by writing /assign @danwinship 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

@markdgray
Copy link
Contributor Author

/test ci/prow/images

@openshift-ci-robot
Copy link
Contributor

@markdgray: The specified target(s) for /test were not found.
The following commands are available to trigger jobs:

  • /test e2e-aws-ovn
  • /test e2e-aws-ovn-windows
  • /test e2e-azure-ovn
  • /test e2e-gcp-ovn
  • /test e2e-gcp-ovn-upgrade
  • /test e2e-metal-ipi-ovn-dualstack
  • /test e2e-metal-ipi-ovn-ipv6
  • /test e2e-openstack-ovn
  • /test e2e-ovn-hybrid-step-registry
  • /test e2e-vsphere-ovn
  • /test e2e-vsphere-windows
  • /test images
  • /test okd-e2e-gcp-ovn
  • /test okd-images

Use /test all to run all jobs.

In response to this:

/test ci/prow/images

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.

@markdgray
Copy link
Contributor Author

/test images

1 similar comment
@markdgray
Copy link
Contributor Author

/test images

@yselkowitz
Copy link
Contributor

This cannot be merged in its current state, as it is not mulit-arch conformant. A mechanism already exists for pinning the version installed, so why is this necessary?

@markdgray
Copy link
Contributor Author

This cannot be merged in its current state, as it is not mulit-arch conformant. A mechanism already exists for pinning the version installed, so why is this necessary?

Hi @yselkowitz

Thanks for reviewing but this PR is only for testing. This is a scratch build RPM as the changes are not available in upstream OVS yet. I marked this PR as "draft" and "wip" to indicate that it was not ready for review. I will add some text to the PR to explain this to avoid further confusion.

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 26, 2021
@markdgray markdgray force-pushed the feat/ipsec_ipv6/sdn-1523 branch 2 times, most recently from 36a0436 to 28d10f5 Compare April 2, 2021 19:26
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 2, 2021
@markdgray
Copy link
Contributor Author

/test all

@markdgray
Copy link
Contributor Author

/retest

1 similar comment
@dcbw
Copy link
Contributor

dcbw commented Apr 5, 2021

/retest

@markdgray
Copy link
Contributor Author

Failures look unrelated

/retest

@markdgray
Copy link
Contributor Author

/retest

1 similar comment
@markdgray
Copy link
Contributor Author

/retest

@markdgray
Copy link
Contributor Author

/test all

@markdgray
Copy link
Contributor Author

Look unrelated

/retest

@dcbw
Copy link
Contributor

dcbw commented Apr 7, 2021

/retest
metal problems are container pull issues

@dcbw
Copy link
Contributor

dcbw commented Apr 7, 2021

/test all
another soak; windows, dual-stack, openstack are unrelated

@dcbw
Copy link
Contributor

dcbw commented Apr 7, 2021

/test e2e-aws-ovn
Failure waiting for pod annotations is unrelated to IPsec

@markdgray
Copy link
Contributor Author

/test e2e-metal-ipi-ovn-dualstack

Looks unrelated: dev-scripts does not finish deploying

@markdgray
Copy link
Contributor Author

/test e2e-gcp-ovn

Failures connecting to API server from, what looks to me to be, test runners. Seems unrelated.

@markdgray
Copy link
Contributor Author

/test e2e-aws-ovn-windows

Looks unrelated - waiting for node annotation

@markdgray
Copy link
Contributor Author

/test e2e-gcp-ovn

Cannot create service account - GCP IAM error

When we are happy that this passes all tests, we can tag this
release.

Signed-off-by: Mark Gray <mark.d.gray@redhat.com>
@markdgray
Copy link
Contributor Author

markdgray commented Apr 7, 2021

/test all

Rebased to latest master, as it looks like some changes have been introduced since I created this commit.

@markdgray
Copy link
Contributor Author

/test e2e-vsphere-ovn

Looks like some kind of privilege issue - unrelated.

@markdgray
Copy link
Contributor Author

/retest

@markdgray
Copy link
Contributor Author

Dualstack test failure is unrelated (Failing invariant - Only known images used by tests)
/test e2e-metal-ipi-ovn-dualstack

IPv6 failure is unrelated (Couldn't subscribe to Deployments: the server has received too many requests)
/test e2e-metal-ipi-ovn-ipv6

Azure failure is unrelated (KubePersistentVolumeErrors)
/test e2e-azure-ovn

vsphere windows is unrelated (TestWMCO/create/Node_Metadata failure)
/test e2e-vsphere-windows

AWS ovn windows is unrelated (TestWMCO/create/Node_Metadata failure)
/test e2e-aws-ovn-windows

vsphere is unrelated (KubePersistentVolumeErrors)
/test e2e-vsphere-ovn

openstack might be unrelated (lots of volumes errors)
/test e2e-openstack-ovn

gcp ovn upgrade (sandbox error)
/test e2e-gcp-ovn-upgrade

okd gcp ovn is suspicious (lots of tcp timeouts)
/test okd-e2e-gcp-ovn

@markdgray
Copy link
Contributor Author

Dualstack test failure is unrelated (Failing invariant - Only known images used by tests) +1
/test e2e-metal-ipi-ovn-dualstack

IPv6 failure is unrelated (Couldn't subscribe to Deployments: the server has received too many requests)

Azure failure is unrelated (KubePersistentVolumeErrors)

vsphere windows is unrelated (TestWMCO/create/Node_Metadata failure) +1
/test e2e-vsphere-windows

AWS ovn windows is unrelated (TestWMCO/create/Node_Metadata failure) +1
/test e2e-aws-ovn-windows

vsphere is unrelated (Pod openshift-vsphere-infra/mdns-publisher crashlooping and Pod openshift-marketplace/community-operators not healthy)
/test e2e-vsphere-ovn

openstack might be unrelated (lots of volumes errors)

gcp ovn upgrade is unrelated (sandbox error +1 and some pods (e.g. Prometheus) aren't getting created.)
/test e2e-gcp-ovn-upgrade

okd gcp ovn is unrelated(alert KubeAPIErrorBudgetBurn alerts, lifecycle manager panic, and Prometheus error)
/test okd-e2e-gcp-ovn

@markdgray
Copy link
Contributor Author

Dualstack test failure is unrelated (Failing invariant - Only known images used by tests +1 +1 and some "connection refused"s for oauth tests)
/test e2e-metal-ipi-ovn-dualstack

vsphere windows is unrelated (TestWMCO/create/Node_Metadata failure +1 and and failed to acquire lease: resources not found)
/test e2e-vsphere-windows

AWS ovn windows is unrelated (TestWMCO/create/Node_Metadata failure) +1 +1
/test e2e-aws-ovn-windows

vsphere is unrelated (Pod openshift-vsphere-infra/mdns-publisher crashlooping and Pod openshift-marketplace/community-operators not healthy and failed to acquire lease: resources not found)
/test e2e-vsphere-ovn

openstack might be unrelated (lots of volumes errors, test timeouts and readiness probe failures)
/test e2e-openstack-ovn

gcp ovn upgrade is unrelated (sandbox error +1 and some pods (e.g. Prometheus) aren't getting created and failed to acquire lease: resources not found)
/test e2e-gcp-ovn-upgrade

okd gcp ovn is unrelated(alert KubeAPIErrorBudgetBurn alerts, lifecycle manager panic, and Prometheus error)
(also seeing network policy errors which may be related but I think this is a known issue)
/test okd-e2e-gcp-ovn

@markdgray
Copy link
Contributor Author

Dualstack test failure is unrelated (Failing invariant - Only known images used by tests +1 +1 +1 and some "connection refused"s for oauth tests)
/test e2e-metal-ipi-ovn-dualstack

vsphere windows is unrelated (TestWMCO/create/Node_Metadata failure +1 and and failed to acquire lease: resources not found +1)
/test e2e-vsphere-windows

AWS ovn windows is unrelated (TestWMCO/create/Node_Metadata failure) +1 +1 +1
/test e2e-aws-ovn-windows

vsphere is unrelated (Pod openshift-vsphere-infra/mdns-publisher crashlooping and Pod openshift-marketplace/community-operators not healthy and failed to acquire lease: resources not found and marketplace pod crashlooping)
/test e2e-vsphere-ovn

openstack might be unrelated (lots of volumes errors, test timeouts and readiness probe failures, cluster does not deploy)
/test e2e-openstack-ovn

gcp ovn upgrade is unrelated (sandbox error +1 and some pods (e.g. Prometheus) aren't getting created and failed to acquire lease: resources not found +1)
/test e2e-gcp-ovn-upgrade

okd gcp ovn is unrelated(alert KubeAPIErrorBudgetBurn alerts, lifecycle manager panic, and Prometheus error)
(also seeing network policy errors which may be related but I think this is a known issue)
/test okd-e2e-gcp-ovn

@dcbw
Copy link
Contributor

dcbw commented Apr 8, 2021

celebdor/zeroconf#4 fixes the mdns-publisher crash

@dcbw
Copy link
Contributor

dcbw commented Apr 8, 2021

/override ci/prow/e2e-aws-ovn-windows
known issue with kubelet verison mismatch

@openshift-ci-robot
Copy link
Contributor

@dcbw: Overrode contexts on behalf of dcbw: ci/prow/e2e-aws-ovn-windows

In response to this:

/override ci/prow/e2e-aws-ovn-windows
known issue with kubelet verison mismatch

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.

@markdgray
Copy link
Contributor Author

This is still the soak test. It sounds like we are good to tag?

@markdgray
Copy link
Contributor Author

Full PR at #492

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Apr 8, 2021

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

Test name Commit Details Rerun command
ci/prow/e2e-openstack-ovn a3a1b2c link /test e2e-openstack-ovn
ci/prow/e2e-vsphere-windows a3a1b2c link /test e2e-vsphere-windows
ci/prow/e2e-metal-ipi-ovn-dualstack a3a1b2c link /test e2e-metal-ipi-ovn-dualstack
ci/prow/e2e-vsphere-ovn a3a1b2c link /test e2e-vsphere-ovn
ci/prow/okd-e2e-gcp-ovn a3a1b2c link /test okd-e2e-gcp-ovn
ci/prow/e2e-aws-ovn-windows a3a1b2c link /test e2e-aws-ovn-windows
ci/prow/e2e-gcp-ovn-upgrade a3a1b2c link /test e2e-gcp-ovn-upgrade

Full PR test history. Your PR dashboard.

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.

@markdgray markdgray closed this Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants