-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
kanidm: 1.1.0-beta.13 -> 1.1.0-rc.15 #264658
Conversation
c84d527
to
5d81702
Compare
@ofborg test kanidm |
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.
Looks good, thanks!
Test runs through (nix-build -A kanidm.tests
) and seems to work fine on my system.
5d81702
to
650e516
Compare
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.
Thanks, still LGTM
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
Are CI checks screwed up? Should I merge anyway? |
I hate to merge failing builds. I'm re-running these locally to test and see if I can reproduce the failure. |
Both x86_64 and aarch64 tests run successfully me. The errors from the ofborg failures seem to be:
|
Some hardening option on the systemd unit interferes with test runs on ofborg infra. This is not news, has happened on every update to kanidm before this one as well. I suspect it is |
Shall we merge? |
Yes, please. From what @mweinelt says, it’s definitely not a regression (and the test passes fine outside of ofborg). |
Description of changes
https://github.com/kanidm/kanidm/releases/tag/v1.1.0-rc.15
Currently failing a test, so not ready yet.Maintainers feel free to edit my PR if I'm not around.Failing test disabled upstream.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)