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