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

Enable trace jobs by default #2190

Merged
merged 1 commit into from Mar 28, 2019

Conversation

gravitystorm
Copy link
Collaborator

I think it's a good idea to use the trace jobs queue by default now, even while we wait on #2132.

@tomhughes
Copy link
Member

Animations is the only reason I hadn't gone ahead and thrown the switch...

I did do all the preparatory work on the chef side to be able to stand up a separate queue runner for this on machines that can see the GPX store.

@tomhughes tomhughes merged commit c44d689 into openstreetmap:master Mar 28, 2019
@tomhughes
Copy link
Member

I realised that this doesn't actually affect the production site because chef is forcing that to false anyway so we might we well merge this now.

I'm thinking I may switch the production site in the next few days as part of decommissioning spike-{01,02,03} and replacing them with spike-{06,07,08} given that spike-01 is currently running the import daemon.

@gravitystorm
Copy link
Collaborator Author

I realised that this doesn't actually affect the production site because chef is forcing that to false anyway so we might we well merge this now.

Yep, I wasn't intending to flick the switch for production yet, just to change the default for new deployments, and in particular change the default so that gpx importing works by default for developers.

I would like to see the animations fixed, but there's no progress on #2132 yet and I'm not sure what the way forward is on that one.

@mmd-osm
Copy link
Contributor

mmd-osm commented Mar 30, 2019

I would like to see the animations fixed, but there's no progress on #2132 yet and I'm not sure what the way forward is on that one.

No worries, I think we can get this thing working:

new_foo

@gravitystorm gravitystorm deleted the use-trace-job-queue branch June 13, 2019 07:44
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.

None yet

3 participants