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
Fix gitlab #30939
Fix gitlab #30939
Conversation
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.
The content looks good, but I'd like to see an explanation (in the commit message) which problem the last commit fixes.
Can you maybe try running the |
This line previously produced an error and an empty secret file.
@bjornfor updated, thanks |
Yeah as I said I updated the gitlab test (needs a lot of updating), but the gitlab service doesn't start in 30 min in qemu... bundle runs for a long time. I'll check back if I get it up. |
Do the VM tests pass --enable-kvm to QEMU? (Speed up.) |
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.
The fixes are correct and valid. Thanks!
I have tried making the test work again a few times, but currently gitlab is doing too much I/O for 9pfs to cope, I'd be really happy if someone fixes that but I can't see us blocking a merge on that.. |
Motivation for this change
There were several mistakes in the setup script, including hardcoded values and secret generation that didn't work.
I'm trying to get the gitlab nixos test running again but RoR is very slow in qemu. So I'm submitting these in the mean time.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)