-
Notifications
You must be signed in to change notification settings - Fork 104
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
Documentation and single test execution for kuttl tests #1595
Conversation
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
…he test folder now Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
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.
A naming nit but otherwise LGTM!
Co-authored-by: Aleksey Dukhovniy <adukhovniy@mesosphere.io> Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
10420f2
to
a8b1ef2
Compare
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
@@ -41,8 +37,7 @@ if docker build -f test/Dockerfile -t kudo-test .; then | |||
if docker run -e INTEGRATION_OUTPUT_JUNIT --net=host -it --rm -m 4g \ | |||
-v /var/run/docker.sock:/var/run/docker.sock \ | |||
-v "$(pwd)"/reports:/go/src/github.com/kudobuilder/kudo/reports \ | |||
-v "$(pwd)"/kind-logs:/tmp/kudo-e2e-test \ | |||
kudo-test bash -c "make $TARGET; ret=\$?; chmod a+r -R /tmp/kudo-e2e-test; exit \$ret" | |||
kudo-test bash -c "make $TARGET; ret=\$?; chmod a+r -R /go/src/github.com/kudobuilder/kudo/reports; exit \$ret" |
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.
👏
Good to know! The reason for the /tmp/kudo-e2e-test
was the requirement for absolute paths and not knowing where the reports would be written by default.
What this PR does / why we need it:
make TEST=plan-trigger e2e-test
kind-log
folders don't clutter the root directory