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

Run intermittent-tracker on servo-master1 #543

Closed
Manishearth opened this issue Nov 29, 2016 · 5 comments
Closed

Run intermittent-tracker on servo-master1 #543

Manishearth opened this issue Nov 29, 2016 · 5 comments

Comments

@Manishearth
Copy link
Member

@Manishearth Manishearth commented Nov 29, 2016

We should run https://github.com/servo/intermittent-tracker on servo-master1

(Playing around with it now to get it to work)

This needs github3.py==1.0.0a4 installed.

(Working out the other dependencies)

@jdm
Copy link
Member

@jdm jdm commented Nov 29, 2016

github3.py is only necessary for running sync.py. You can also run that locally, pipe the output to intermittents.json and scp the json file to the builder.

@jdm
Copy link
Member

@jdm jdm commented Nov 29, 2016

(Note that for best results the JSON file should be created right before enabling the webhook)

@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 29, 2016

I've got a version running with flask at http://build.servo.org/intermittent-tracker/query.py?name=private

(CGI seemed like a whole other world of pain, whereas we already use bottle in homu and it's the same principle here)

@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 29, 2016

servo/intermittent-tracker#1 has the flaskification.

The virtualenv needs flask installed. github3 doesn't get installed in non-root mode because of openssl stuff in the cryptography lib 🙁

We will need it eventually for a startup task to populate the db. We should also start verifying webhook secrets.

@aneeshusa Any idea how to daemonize this?

Manishearth added a commit to Manishearth/saltfs that referenced this issue Nov 30, 2016
Manishearth added a commit to Manishearth/saltfs that referenced this issue Nov 30, 2016
Manishearth added a commit to Manishearth/saltfs that referenced this issue Nov 30, 2016
Manishearth added a commit that referenced this issue Nov 30, 2016
Manishearth added a commit that referenced this issue Nov 30, 2016
Manishearth added a commit that referenced this issue Nov 30, 2016
Manishearth added a commit that referenced this issue Nov 30, 2016
bors-servo added a commit that referenced this issue Nov 30, 2016
Add persistent intermittent tracker

(fixes #543)

r? @larsbergstrom

cc @aneeshusa @jdm

Needs servo/intermittent-tracker#1

This is basically a copy of the homu setup, just tweaked to work with this.

This does not contain a job to set up intermittents.json on startup; but this can be added later.

Works running locally in vagrant.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/544)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Nov 30, 2016
Add persistent intermittent tracker

(fixes #543)

r? @larsbergstrom

cc @aneeshusa @jdm

Needs servo/intermittent-tracker#1

This is basically a copy of the homu setup, just tweaked to work with this.

This does not contain a job to set up intermittents.json on startup; but this can be added later.

Works running locally in vagrant.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/544)
<!-- Reviewable:end -->
@Manishearth
Copy link
Member Author

@Manishearth Manishearth commented Nov 30, 2016

I deployed a cherry-picked version of this (to avoid the android issue), cc @larsbergstrom

It works.

Coder206 added a commit to Coder206/saltfs that referenced this issue Apr 22, 2017
choudhary001 added a commit to choudhary001/saltfs that referenced this issue May 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants
You can’t perform that action at this time.