-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
Permalink
Choose a base ref
{{ refName }}
default
Choose a head ref
{{ refName }}
default
Comparing changes
Choose two branches to see what’s changed or to start a new pull request.
If you need to, you can also or
learn more about diff comparisons.
Open a pull request
Create a new pull request by comparing changes across two branches. If you need to, you can also .
Learn more about diff comparisons here.
base repository: NixOS/nixpkgs
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 6845ebbff1c4
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
...
head repository: NixOS/nixpkgs
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: b4d9f14d3cc4
Could not load branches
Nothing to show
Loading
Could not load tags
Nothing to show
{{ refName }}
default
Loading
- 4 commits
- 2 files changed
- 1 contributor
Commits on Nov 23, 2018
-
Configuration menu - View commit details
-
Copy full SHA for d1ef00e - Browse repository at this point
Copy the full SHA d1ef00eView commit details -
Configuration menu - View commit details
-
Copy full SHA for b1032db - Browse repository at this point
Copy the full SHA b1032dbView commit details -
nixos/prometheus: require one alertmanager configuration parameter
This commit adds an assertion that checks that either `configFile` or `configuration` is configured for alertmanager. The alertmanager config can not be an empty attributeset. The check executed with `amtool` fails before the service even has the chance to start. We should probably not allow a broken alertmanager configuration anyway. This also introduces a test for alertmanager configuration that piggy backs on the existing prometheus tests.
Configuration menu - View commit details
-
Copy full SHA for 51c3082 - Browse repository at this point
Copy the full SHA 51c3082View commit details -
Merge pull request #49756 from andir/prometheus
nixos/prometheus: check alertmanager configuration
Configuration menu - View commit details
-
Copy full SHA for b4d9f14 - Browse repository at this point
Copy the full SHA b4d9f14View commit details
There are no files selected for viewing