add e2e runner (go variant) #95
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gardener-robot-ci-1
added
reviewed/ok-to-test
OlegLoewen
force-pushed
the
e2etestv2
branch
2 times, most recently
from
fcbf067 to
77c29fe
Compare
OlegLoewen
changed the title
OlegLoewen
force-pushed
the
e2etestv2
branch
2 times, most recently
from
e75d150 to
d668a03
Compare
|
WIP status closed, ready for review. |
schrodit
suggested changes
Jun 25, 2019
schrodit
suggested changes
Jun 25, 2019
test/e2etest/util/util.go
Outdated
| return fallback | ||
| } | ||
|
|
||
| func RunCmd(command, execPath string) error { |
There was a problem hiding this comment.
maybe we should also let the stdout and stderr parameter customizable.
Then this functions could be reused for all commands
There was a problem hiding this comment.
Sure, let's discuss in person how the interface should look like
OlegLoewen
force-pushed
the
e2etestv2
branch
4 times, most recently
from
4dd2707 to
c134147
Compare
|
@OlegLoewen @dguendisch maybe we should move the e2e tests to a folder outside of |
|
Good idea. |
OlegLoewen
force-pushed
the
e2etestv2
branch
3 times, most recently
from
e56a8ed to
45d4743
Compare
…s and e2e.log files.
Use TM export path and kubeconfig path Reuse test-infra util functions Additional minor fixes
Check for errors for util.RunCmd calls
Add failed testcase names to test summary Fix "no Go files" error for cmd run add debug argument
…flakeAttempts, testcasegroup. Fix: setting up test utilities Fix: switching to antoher k8sRelese
Fix refactring issues
…ubectl is not preinstalled
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
add e2e runner (go variant)
Release note: