On Fri, Mar 27, 2020 at 05:14:48PM -0700, Kevin Fenzi wrote:
> On Fri, Mar 27, 2020 at 12:06:22PM +0100, Pavel Raiskup wrote:
> > On Friday, March 27, 2020 11:34:28 AM CET Adrian Reber wrote:
> > > The global 'exclude' has '--exclude "*/repodata/*"' and you are using
> > > "${excludes[@]}" everywhere. In all three syncs. This looks like
> > > 'repodata/*' will never synced.
> > 
> > Good catch!  Thank you for the review.  I am attaching updated patch.
> > 
> > Pavel
> 
> Looks ok to me from a quick glance... lets give it a try. :) 
> 
> I think also we should remove the test releases one. It's currently
> failing the invalidate step (because it's repodata is not in the same
> place as the updates repos) and the only reason we would want say
> 32_Beta to be there is so we could point people to download isos from
> it. The repodata will never change. 

Accessing cloudfront I can now see the correct header:

cache-control: max-age=60

and the age header never gets larger than 60

I can see 'age: 59', but each request after that gives me an

x-cache: RefreshHit from cloudfront

From the header returned by cloudfront it seems we are now never seeing
repomd.xml files older than 1 minute.

I would like to turn on the cloudfront mirror again to see if COPR still
breaks. Any objections?

                Adrian
_______________________________________________
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org

Reply via email to