Skip to content

/ servo

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

Jump to bottom

Add notification when EC2 is not spinning up #8131

Closed
larsbergstrom opened this issue Oct 21, 2015 · 2 comments
Closed

Add notification when EC2 is not spinning up #8131

larsbergstrom opened this issue Oct 21, 2015 · 2 comments
Labels
A-infrastructure

Comments

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Oct 21, 2015

According to:
http://docs.buildbot.net/latest/manual/cfg-buildslaves-ec2.html#configure-the-master-with-an-ec2latentbuildslave

We should be able to add some notification stuffs to:
https://github.com/servo/saltfs/blob/master/buildbot/master/master.cfg#L57

An e-mail address seems... awkward, though. Any ideas on how to maybe have it ping #servo instead?

cc @metajack @edunham
@jdm
Copy link
Member

@jdm jdm commented Oct 21, 2015

crowbot could log into a mail account every N minutes and report new failures!
@edunham
Copy link
Contributor

@edunham edunham commented Oct 21, 2015

It's pretty easy to make a Buildbot notify a given IRC channel on preset status events:

https://github.com/rust-lang/rust-buildbot/blob/master/master/master.cfg#L1791

However, the notifications can get kind of spammy. Do you think the benefits of keeping #servo a mostly-humans environment would outweigh the inconvenience of having everyone who cares join a #servo-bots channel?
@edunham edunham mentioned this issue Oct 21, 2015
Merged
bors-servo pushed a commit to servo/saltfs that referenced this issue Oct 21, 2015
bors-servo
Auto merge of #130 - edunham:ircnotify, r=larsbergstrom
Loading status checks…
ee3a3b3 
Notify in #servo-bots

Might resolve servo/servo#8131

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/130)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-infrastructure
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

Notify in #servo-bots
3 participants
@jdm @larsbergstrom @edunham
You can’t perform that action at this time.