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
test: Add log artifact for ginkgo node e2e and tune default ginkgo flags #116057
Conversation
6228143
to
fa131d3
Compare
cc99e11
to
7b56374
Compare
7b56374
to
2ded12d
Compare
f2bd0be
to
af5c2fc
Compare
/pull pull-kubernetes-node-kubelet-serial-containerd |
/test pull-kubernetes-node-kubelet-serial-containerd |
af5c2fc
to
833eb30
Compare
d83e771
to
e001884
Compare
/kind cleanup |
/triage accepted |
/assign @xmcqueen |
@@ -359,7 +361,6 @@ func prepareGceImages() (*internalImageConfig, error) { | |||
metadata: getImageMetadata(metadata), | |||
kernelArguments: imageConfig.KernelArguments, | |||
machine: imageConfig.Machine, | |||
tests: imageConfig.Tests, |
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.
thank you for removing this. Did you check we removed all the instances of this?
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.
Yes, the only reference I could find is kubernetes/test-infra#26017 but it was deleted in that PR.
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
@xmcqueen sorry for stepping into your review. This change lgtm
LGTM label has been added. Git tree hash: 57bde7260b4b3bf73a8cba057e70a5f5e78f11d1
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bobbypage, SergeyKanzhelev The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This PR makes a few updates to the node e2e runner to make debugging the tests easier. Please review commit by commit.
The main changes are:
--no-color
to disable color as it makes examining the logs harder in prow and in text editors, and-v
to enable verbose logs, so even if tests pass we receive the test output. This is helpful to compare passing runs to failed runs.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: