-
Notifications
You must be signed in to change notification settings - Fork 201
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
cleanup: use url from manifest.cue to download k8s #4443
Conversation
Pull Request Test Coverage Report for Build 9396018292Details
💛 - Coveralls |
e76d1cd
to
8378680
Compare
8378680
to
2df3789
Compare
2df3789
to
253b6f5
Compare
253b6f5
to
e048d27
Compare
if: always() | ||
with: | ||
files: | | ||
test-results/**/*.xml |
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 forgot does the glob pattern **
also looks in the current dir ? Since test-results/bats_shell.junit.xml
is not found in any subdirectory
assert_called rm -f /var/lib/man-db/auto-update | ||
} | ||
|
||
@test "failing test" { |
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'm guessing this is simply to test that the logic works and it's not tone committed ?
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.
maybe we want to create a separate PR for the new shell unit testing stuff?
otherwise I think we decided to move towards components.json and away from manifest.cue/json
closing for now, feel free to reopen |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Start getting all config information from manifest.json/cue instead of hardcoding it.
Which issue(s) this PR fixes:
Requirements:
Special notes for your reviewer:
Release note: