All users were logged out of Bugzilla on October 13th, 2018

Proper deployment + cache-control headers



4 years ago
2 years ago


(Reporter: jonasfj, Unassigned)



Firefox Tracking Flags

(Not tracked)




4 years ago
In bug 1017155 we got SSL for which is done through cloudfront.
But some of our files really shouldn't have 24 hours delay.

For example /v1/telemetry.js, we need to update that when we move data files.

So we should specify cache-control header when uploading files to S3.
I'm okay with setting cache-control to 1 minute or less :)

We use cloudfront just to get the SSL support, not to get a CDN.

Also we need a smarter way to deploy this dashboard. I suggest that we use travis-ci.
Ie. create a travis.yml file with encrypted AWS credentials for an IAM user that only has access to the bucket.

This way, we can also do some preprocessing before uploading, if we should ever want to do that... Perhaps in the future we'll write telemetry.js using browserify.
Oh, and it'll enable to run some simple tests, I believe we already have some for telemtry.js

I think this might work:

But it doesn't seem to set cache-control headers, so we probably have to install aws-cli tools and do a custom deployment script:

Note, that we currently have something deploying to S3, running as a cron job.. .That should be killed when this is running. It lives on the dashboard aggregation node.
Ask jonasfj, if you forget where that node lives :)

Comment 1

2 years ago
:whd, is this 3-year-old bug still relevant to how tmo is deployed these days?
Flags: needinfo?(whd)

Comment 2

2 years ago
:rvitillo, :mreid and I replaced the cron s3 upload version of this with a github-pages based setup at some point between 3 years ago and now. We also no longer have a 24 hour delay from GH pages pushes to CDN propagation that I am aware of (I believe it is on the order of minutes). Closing this.
Last Resolved: 2 years ago
Flags: needinfo?(whd)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.