-
Notifications
You must be signed in to change notification settings - Fork 43
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
Update e2e test to run against machine.openshift.io #151
Update e2e test to run against machine.openshift.io #151
Conversation
/hold |
cf4ac72
to
0c88530
Compare
0c88530
to
c1359a7
Compare
c1359a7
to
6c53e4e
Compare
/test e2e-aws-operator |
6c53e4e
to
9c5177c
Compare
/lgtm |
test/e2e/main.go
Outdated
@@ -59,7 +59,7 @@ func runSuite() error { | |||
glog.Info("PASS: ExpectProviderAvailable") | |||
|
|||
glog.Info("RUN: ExpectOneClusterObject") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/ExpectOneClusterObject/ExpectNoClusterObject
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't need the cluster for machine.openshift.io https://github.com/openshift/cluster-api-provider-aws/pull/151/files#diff-1c49fed4c53a7b1e61a888973c253576R54
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Longer term we should have something more sophisticated so that we don't have to repeat all these string values.
9c5177c
to
9c829de
Compare
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ingvagabund The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Update e2e test to run against machine.openshift.io
Needs openshift/installer#1175