Skip to content

run tests against OKD and simplify e2e automation scripts #258

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

Merged
merged 1 commit into from
Nov 11, 2019

Conversation

phoracek
Copy link
Member

@phoracek phoracek commented Nov 8, 2019

Should go in after #257

Get rid of smart PROVIDER parsing and simply keep single shell script
per target.

Since we switched to kubevirtci's cluster-up, switch from OpenShift 3
to OKD 4 as part of this patch.

Signed-off-by: Petr Horacek [email protected]

@kubevirt-bot kubevirt-bot added dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/XL labels Nov 8, 2019
@phoracek
Copy link
Member Author

phoracek commented Nov 8, 2019

@qinqon @oshoval could you please take a look? Please note that only the second commit is part of this PR.

@phoracek phoracek force-pushed the simple_automation branch 3 times, most recently from 4317e8f to 9eeff84 Compare November 8, 2019 21:10
@phoracek
Copy link
Member Author

phoracek commented Nov 8, 2019

I have to disable Multus deployment checks on OKD (it is there by default)

@phoracek phoracek force-pushed the simple_automation branch 5 times, most recently from 5fae5eb to 83b8f22 Compare November 10, 2019 01:29
@phoracek phoracek changed the title simplify e2e automation run tests against OKD and simplify e2e automation scripts Nov 10, 2019
Copy link
Collaborator

@qinqon qinqon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Get rid of smart PROVIDER parsing and simply keep single shell script
per target.

Since we switched to kubevirtci's cluster-up, switch from OpenShift 3
to OKD 4 as part of this patch.

Signed-off-by: Petr Horacek <[email protected]>
@phoracek phoracek merged commit e9145a0 into master Nov 11, 2019
@phoracek phoracek deleted the simple_automation branch February 27, 2020 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants