Re: [PATCH] Add CDN testing cloudfront redirect for atomic repo
On 1/3/19 10:40 AM, Sinny Kumari wrote:
Hi,
Currently, we are using two different cloudfront distributions to CDN content for
/atomic/repo/objects/ and /atomic/repo/deltas/ . With the current setup we see
an issue https://github.com/ostreedev/ostree/issues/1541 . To overcome this
issue and provide faster delivery of ostree content, we have created a
cloudfront distribution where we CDN /atomic/repo/ .
This patch (available in attachment) adds some redirect rules to use new
cloudfront urls for testing purpose and see how much improvement we get in
content delivery. This patch shouldn't impact existing infra setup.
I'd like to highlight this is for testing purposes only and should have a short
life. We just wanted to make sure the "redirect penalty" that we experience
with
our prod redirects is also reflected in our testing so we can get more accurate
results.
Sinny,
One thing we might want to do is use https for the cloudfron URLs like we are for
the prod redirects. Otherwise LGTM.
Dusty