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
kubernix: v0.2.0 #73385
kubernix: v0.2.0 #73385
Conversation
@GrahamcOfBorg build kubernix |
oh, I'm not 100% sure on how to scope tests in rust, I just noticed that all the |
sorry to bother @grahamc , but I'm not really sure how to scope |
also, it's hard to find documentation on the matter |
eac3f75
to
36bc5f5
Compare
Unfortunately its not that easily possible with the 0.2.0 release. I would suggest in changing this to the next version, where I would split up the independent tests that we can execute them here. |
@GrahamcOfBorg build kubernix |
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.
Fixing the tests in the next release sounds fine.
Implemented the review suggestions and rebased on top of the latest master. |
Signed-off-by: Sascha Grunert <sgrunert@suse.com>
@ofborg eval Looks like the bot got stuck; sorry about that. |
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, works.
Works against |
Motivation for this change
Add kubernix to nixpkgs.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)