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.

Sign up for GitHub

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

Jump to bottom

Test hydra notify #674

Closed
wants to merge 14 commits into from
Closed

Test hydra notify #674

wants to merge 14 commits into from
+410 −161

Conversation

craigem
Copy link
Contributor

@craigem craigem commented Aug 26, 2019

This PR exists to build Hydra from master, specifically to test hydra-notify.

cleverca22 and others added 14 commits September 10, 2018 09:50
@cleverca22
fix hydra build on nixpkgs master
186137e 
copied from NixOS/nixpkgs@c501958
@cleverca22
allow using a shorter context and increase hydra-notify debug
1c76ad3
@cleverca22
also add CatalystXRoleApplicator
1317417 
NixOS/nixpkgs@f7fe612
@cleverca22
re-sync nix-prefetch-git to the copy in nixpkgs
69bb5a9 
this fixes the ability to prefetch things with submodules
@cleverca22
change the output API of nix-prefetch-git back to what GitInput.pm …

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
GPG key ID: 4AEE18F83AFDEB23
Expired
Verified
Learn about vigilant mode
2f39355 
…expects
@cleverca22
redo the notify events between eval and queue-runner
73ca325
@cleverca22
improve the error messages when invalid declarative jobsets are defined
7568b89
@deepfire
nix-prefetch-git: untangle deepClone from submodule fetching
6d5d50c
@rvl @disassembler
[DEVOPS-1028] GithubStatus.pm: normalize job names for bors-ng branches
a3887f5
@deepfire @disassembler
[CGKIELE-758] Mantis hydra's project need the submodules
861853e
@deepfire @disassembler
[CGKIELE-758] Cut the odd '((null))' in the submodule parser
3b41fb9
@cleverca22 @disassembler
[DEVOPS-1126] throttle github status calls to remain under api rateli…
5fb456c 
…mits

(cherry picked from commit 7916c6b)
@edolstra @disassembler
Fix building against nix master
5f30a10 
(cherry picked from commit 8d26144)
@craigem
Merge remote-tracking branch 'upstream/master' into test_hydra-notify
adcce49
@grahamc
Copy link
Member

grahamc commented Apr 14, 2021

Since this PR was submitted hydra-notify was changed to use postgresql events, and #912 was merged. There was also #878 which tests some of the behavior too. I'm going to close it, but if there is somethin specific here that is worth applying, maybe they could be a new PR?

@grahamc grahamc closed this Apr 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@craigem @grahamc @cleverca22 @deepfire @rvl @edolstra