NE-1324: E2E test for External DNS Operator Shared VPC#42894
Conversation
|
@gcs278: This pull request references NE-1324 which is a valid jira issue. DetailsIn response to this:
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. |
|
Skipping CI for Draft Pull Request. |
|
@gcs278, Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
dd7dfeb to
b9a7e19
Compare
|
/pj-rehearse |
|
@gcs278, Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
8dd224f to
f88a50d
Compare
|
/pj-rehearse pull-ci-openshift-external-dns-operator-main-e2e-aws-shared-vpc-phz-operator |
f88a50d to
c6af72d
Compare
c6af72d to
e992ef2
Compare
|
/pj-rehearse pull-ci-openshift-external-dns-operator-main-e2e-aws-shared-vpc-phz-operator |
|
/pj-rehearse |
e992ef2 to
e314e21
Compare
|
@gcs278: This pull request references NE-1324 which is a valid jira issue. DetailsIn response to this:
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. |
|
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/42894/rehearse-42894-pull-ci-openshift-external-dns-operator-main-e2e-aws-shared-vpc-phz-operator/1697644313543970816 failed due a syntax error in ci-operator/step-registry/optional-operators/ci/aws/shared-vpc-phz/optional-operators-ci-aws-shared-vpc-phz-workflow.yaml which didn't install the optional External-DNS operator Fixed. /pj-rehearse |
|
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/42894/rehearse-42894-pull-ci-openshift-external-dns-operator-main-e2e-aws-shared-vpc-phz-operator/1697670677244416000 failed with Shared VPC setup failure. Will dive into this next week, but in the meantime, let's try again: /pj-rehearse |
|
/test ci/rehearse/openshift/external-dns-operator/main/e2e-aws-shared-vpc-phz-operator |
|
@gcs278: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
DetailsIn response to this:
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. |
|
Success. Just need to write the E2E test for testing new API field |
|
Retesting to check for flakes: |
|
@gcs278: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
DetailsIn response to this:
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. |
|
@alebedev87 fixed rebase. Could I get another LGTM? |
|
/lgtm |
be357c8 to
536a077
Compare
This introduces a new test job for the external-dns-operator repo called e2e-aws-shared-vpc-phz-operator as well as a new workflow optional-operators-ci-aws-shared-vpc-phz. This new workflow installs and configures cluster that is using a shared Route 53 private hosted zone with another AWS account and then installs an optional operator. The new job runs the new workflow with parameters for installing the external-dns-operator. It runs the operator's E2E test suite after configuring the cluster with a shared Route 53 and the external-dns-operator.
536a077 to
fca5cc3
Compare
|
/pj-rehearse ack |
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/pj-rehearse ack |
|
@gcs278: This pull request references NE-1324 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.15.0" version, but no target version was set. DetailsIn response to this:
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. |
|
/approve |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alebedev87, gcs278, smg247 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@gcs278: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
This introduces a new test job for the external-dns-operator repo called
e2e-aws-shared-vpc-phz-operatoras well as a new workflowoptional-operators-ci-aws-shared-vpc-phz.The new workflow
optional-operators-ci-aws-shared-vpc-phzis a combination of the existing workflowsopenshift-e2e-aws-shared-vpc-phzandoptional-operators-ci-aws. This new workflow installs and configures cluster that is using a Shared Route 53 Private Hosted Zone with another AWS account and then installs an optional operator (external-dns-operator in this case).The job
e2e-aws-shared-vpc-phz-operatorruns the workflowoptional-operators-ci-aws-shared-vpc-phzwith parameters for installing the external-dns-operator. It runs the operator's E2E test suite after configuring the cluster with a shared Route 53 and the external-dns-operator.See openshift/external-dns-operator#195 for the implementation and E2E test openshift/external-dns-operator#198 that is the motivation for this new E2E test.