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

Add Nix Review Tools Reports #115

Merged
merged 1 commit into from
Mar 12, 2022
Merged

Add Nix Review Tools Reports #115

merged 1 commit into from
Mar 12, 2022

Conversation

malob
Copy link
Contributor

@malob malob commented Mar 11, 2022

No description provided.

@cyntheticfox
Copy link
Member

The information is certainly interesting, so I'm kinda surprised these kind of reports aren't already available on the NixOS monitoring website at https://monitoring.nixos.org/. Might be interesting to play around with hydra job statuses and frequencies in Apache Superset if I had the time...

The interface isn't super pretty, but as long as you're comitted to either keeping the list of jobsets up-to-date or putting in a webhook+action to do that, I think it's fine to add this as a resource. I might also recommend adding a link on the homepage to https://nixos.wiki/wiki/Nix_channels for those uninitiated on the jobset/nix-channel relationship and concepts.

@malob
Copy link
Contributor Author

malob commented Mar 11, 2022

The information is certainly interesting, so I'm kinda surprised these kind of reports aren't already available on the NixOS monitoring website at https://monitoring.nixos.org/. Might be interesting to play around with hydra job statuses and frequencies in Apache Superset if I had the time...

I'm surprised too!

I tried to find this kind of information, but didn't have any luck on that site or others. Given that folks post reports they generate themselves using nix-review-tools during the zero Hydra failures push before a new release (e.g., see ZERO Hydra Failures 21.11) seems unlikely that existing tools automate generating this info.

The interface isn't super pretty, but as long as you're comitted to either keeping the list of jobsets up-to-date or putting in a webhook+action to do that, I think it's fine to add this as a resource.

I agree the interface isn't super pretty (the site is just using the default Jekyll theme), but it does the job. I may get around to tweaking it at some point, but I don't have much experience with Jekyll, so it's a bit of a slog for me to change stuff like that. (Contributions always welcome 😛.)

I do plan on keeping the jobsets up to date, and making improvements as needed.

I might also recommend adding a link on the homepage to https://nixos.wiki/wiki/Nix_channels for those uninitiated on the jobset/nix-channel relationship and concepts.

Would be nice to have something like this, but having a look at that page it doesn't seem great for that. Each Hydra jobset does have a comment that usually indicates which branch/channel it's for, so maybe some text with a short explanation and a link to the nixos and nixpkgs Hydra projects would be a good idea. I'll think on it.

@cyntheticfox cyntheticfox merged commit f5c2834 into nix-community:main Mar 12, 2022
@malob malob deleted the add-nix-review-tools-reports branch March 13, 2022 02:53
@malob
Copy link
Contributor Author

malob commented Mar 15, 2022

FYI, I added some explanation of the jobset-channel correspondence and a bunch of other documentation to the site: https://malob.github.io/nix-review-tools-reports/about/

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.

2 participants