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