Skip to content
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

Update nixpkgs to 23.11 #50

Merged
merged 1 commit into from
Jan 11, 2024
Merged

Update nixpkgs to 23.11 #50

merged 1 commit into from
Jan 11, 2024

Conversation

joinemm
Copy link
Collaborator

@joinemm joinemm commented Jan 5, 2024

23.05 has outdated grafana module which no longer works properly. Updating to 23.11 fixes the monitoring dashboard.

Requires verifying on all hosts that the update doesn't introduce breaking changes. Raising the stateVersion should be fine as we don't really have much state that could break.

monitoring-ficolo and binarycache-ficolo have been succesfully updated and show no recession.

Signed-off-by: Joonas Rautiola <joonas.rautiola@unikie.com>
@joinemm joinemm requested review from henrirosten, mkaapu, karim20230 and a team January 5, 2024 13:50
@karim20230
Copy link
Member

LGTM

@henrirosten
Copy link
Collaborator

Tested on ghafhydra-dev and I see some services on that host fail to start after this update.

However, that server is currently not used for anything other than demonstrating that we can setup hydra and x86 builder with nixos-anywhere. Moreover, it will be replaced with #35 sometime in the near future, so I will not block this PR due to that failure.

If we need to, we can later inspect the issue on ghafhydra-dev on 23.11. Until then (until someone has time to inspect the issue in detail) ghafhydra-dev will keep running on 23.05.

@mkaapu
Copy link
Contributor

mkaapu commented Jan 8, 2024

I think we need to test this also with Ficolo setup. We do not currently have bare metal server where we could safely test this, so I am going to deploy changes to our x86 Ficolo builder and test changes with remote building.

@joinemm joinemm merged commit 7ebc981 into main Jan 11, 2024
1 check passed
@henrirosten henrirosten deleted the nixos-23.11 branch January 16, 2024 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants