-
Notifications
You must be signed in to change notification settings - Fork 1
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
Adds integration test #7
Conversation
751ddf0
to
a78acad
Compare
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.
Please use the k8s-test-harness
now that we have it.
a78acad
to
b7b0567
Compare
done |
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.
Looks good overall, just a comment on having a generic approach for the Rocks->helm setup.
b7b0567
to
33fcf55
Compare
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
96492fa
to
852e9f2
Compare
Whereabouts depends on Multus, which means that the integration test is deploying Multus as well.
852e9f2
to
302c5bd
Compare
Whereabouts depends on Multus, which means that the integration test is deploying Multus as well.
Depends-On: #8
Depends-On: canonical/k8s-test-harness#9
Depends-On: canonical/k8s-test-harness#10