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
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ 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
