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.
Ah yes, nice catch. Updated patch is available in the attachment.
This is the sad part part of doing copy paste :/
Dusty