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@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx 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@xxxxxxxxxxxxxxxxxxxxxxx