Re: [PATCH] Add CDN testing cloudfront redirect for atomic repo

2019-01-04 Thread Sinny Kumari
On Fri, Jan 4, 2019 at 9:16 PM Mikolaj Izdebski wrote: > I've applied the patch and deployed it in production. > Thanks a lot Mikolaj! On Thu, Jan 3, 2019 at 4:55 PM Sinny Kumari wrote: > > > > > > > > On Thu, Jan 3, 2019 at 9:16 PM Dusty Mabe wrote: > >> > >> > >> > >> On 1/3/19 10:40 AM,

Re: [PATCH] Add CDN testing cloudfront redirect for atomic repo

2019-01-04 Thread Mikolaj Izdebski
I've applied the patch and deployed it in production. On Thu, Jan 3, 2019 at 4:55 PM Sinny Kumari wrote: > > > > On Thu, Jan 3, 2019 at 9:16 PM Dusty Mabe wrote: >> >> >> >> On 1/3/19 10:40 AM, Sinny Kumari wrote: >> > Hi, >> > >> > Currently, we are using two different cloudfront distributions

Re: [PATCH] Add CDN testing cloudfront redirect for atomic repo

2019-01-03 Thread Sinny Kumari
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 > > > -- http://sinny.io/ From 420894ee34e7561d134bc3713801c9ec3edde5c

Re: [PATCH] Add CDN testing cloudfront redirect for atomic repo

2019-01-03 Thread Dusty Mabe
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 >

[PATCH] Add CDN testing cloudfront redirect for atomic repo

2019-01-03 Thread Sinny Kumari
-- http://sinny.io/ From e42132ff2f5d5935cd2b0124091201d6a6cf2a9f Mon Sep 17 00:00:00 2001 From: Sinny Kumari Date: Thu, 3 Jan 2019 20:57:01 +0530 Subject: [PATCH] Add CDN testing cloudfront redirect for atomic repo Signed-off-by: Sinny Kumari --- roles/download/files/httpd