Skip to content

Bug 2014710: Fix for Azure dns privateZone degrade e2e test#662

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
lmzuccarelli:fix-azure-dns-e2etest
Oct 15, 2021
Merged

Bug 2014710: Fix for Azure dns privateZone degrade e2e test#662
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
lmzuccarelli:fix-azure-dns-e2etest

Conversation

@lmzuccarelli
Copy link
Contributor

This fix ensures the e2e test for dns privateZone works for Azure , GCP and AWS.

The fix is really simple. Azure uses a "/" naming notation for privateZones, the original "error-" prefix did not cause a failure, the updated prefix injected is simply "/error" which causes the operator status to degrade to TRUE

The change was verified on Azure & AWS via cluster-bot

For Azure

GO111MODULE=on GOFLAGS=-mod=vendor go test -timeout 1h -count 1 -v -tags e2e -run "TestIngressStatus" ./test/e2e/                                                                                                       
I1015 12:27:32.540041 1235696 request.go:668] Waited for 1.046809151s due to client-side throttling, not priority and fairness, request: GET:https://api.ci-ln-zjdbpyt-002ac.ci.azure.devcluster.openshift.com:6443/apis/controlplane.operator.openshift.io/v1alpha1?timeout=32s
=== RUN   TestIngressStatus
--- PASS: TestIngressStatus (33.69s)
PASS
ok  	github.com/openshift/cluster-ingress-operator/test/e2e	37.117s

For AWS

GO111MODULE=on GOFLAGS=-mod=vendor go test -timeout 1h -count 1 -v -tags e2e -run "TestIngressStatus" ./test/e2e                                                                                       
I1015 14:04:16.269989    9695 request.go:668] Waited for 1.045602401s due to client-side throttling, not priority and fairness, request: GET:https://api.ci-ln-53wq70b-d5d6b.origin-ci-int-aws.dev.rhcloud.com:6443/apis/apiserver.openshift.io/v1?timeout=32s
=== RUN   TestIngressStatus
--- PASS: TestIngressStatus (35.77s)
PASS
ok  	github.com/openshift/cluster-ingress-operator/test/e2e	39.560s

@openshift-ci openshift-ci bot requested review from Miciah and rfredette October 15, 2021 13:16
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 15, 2021
@lmzuccarelli
Copy link
Contributor Author

/test e2e-aws

@Miciah
Copy link
Contributor

Miciah commented Oct 15, 2021

Thanks! We might as well check that the full test suite passes on Azure:
/test e2e-azure-operator

@Miciah
Copy link
Contributor

Miciah commented Oct 15, 2021

Tests look good. Thanks!
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 15, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 15, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lmzuccarelli, Miciah

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:
  • OWNERS [Miciah,lmzuccarelli]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot merged commit 42b2bc2 into openshift:master Oct 15, 2021
@Miciah
Copy link
Contributor

Miciah commented Oct 15, 2021

In retrospect, we should have a BZ linked to this PR so that we can backport the fix to the release-4.9, where the test was first introduced.

@Miciah
Copy link
Contributor

Miciah commented Oct 15, 2021

/retitle Bug 2014710: Fix for Azure dns privateZone degrade e2e test

@openshift-ci openshift-ci bot changed the title Fix for Azure dns privateZone degrade e2e test Bug 2014710: Fix for Azure dns privateZone degrade e2e test Oct 15, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 15, 2021

@lmzuccarelli: All pull requests linked via external trackers have merged:

Bugzilla bug 2014710 has been moved to the MODIFIED state.

Details

In response to this:

Bug 2014710: Fix for Azure dns privateZone degrade e2e test

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.

@Miciah
Copy link
Contributor

Miciah commented Oct 15, 2021

/cherry-pick release-4.9

@openshift-cherrypick-robot

@Miciah: new pull request created: #664

Details

In response to this:

/cherry-pick release-4.9

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

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants