On Sat, Mar 28, 2020 at 11:04:51AM +0100, Adrian Reber wrote: > 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?
None here, +1 kevin
signature.asc
Description: PGP signature
_______________________________________________ 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