-
Notifications
You must be signed in to change notification settings - Fork 88
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
Add e2e test setup for operator #1534
Add e2e test setup for operator #1534
Conversation
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
c137373
to
7182dd0
Compare
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
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.
I have to add a build tag to the e2e test files to make sure we are not executing them when running go test ./...
f82ca92
to
5706d58
Compare
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Test failure is related to this issue: |
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.
I do think we need to make a documentation run through this repo. In the open source world, undocumented code is not well received…
Let me spend some time today/this week on adding better documentation to this repo. |
Result of fdb-kubernetes-operator-pr on Linux CentOS 7
|
Description
This change adds the operator related e2e tests into the operator repository and allows users to run those e2e tests.
Type of change
Please select one of the options below.
Discussion
I removed the old tests, since they never ran automatically.
Testing
Documentation
I will add more documentation once we move forward with the refactoring.
Follow-up