On Wed, Mar 14, 2018 at 12:03:11PM +0100, Sandro Bonazzola wrote:
> 2018-03-06 17:40 GMT+01:00 Brian Stinson <brian@bstinson.com>:
>
> > -----BEGIN PGP SIGNED MESSAGE-----
> > Hash: SHA1
> >
> > On Mar 06 00:11, Brian Stinson wrote:
> > > Hi Folks,
> > >
> > > Now that we have SIG Content in the /altarch directories for CentOS 7 on
> > the
> > > mirrors, we need to come with some requirements and recommendations for
> > dealing
> > > with the different locations.
> > >
> > > tl;dr The current thinking is to add the following snippet in the %post
> > of
> > > centos-release and release to all of the arches. This will set a yum var
> > to the
> > > correct content location when a new system gets installed:
> > >
> > >     `uname -m | grep -q 'x86_64'  && echo 'centos'
> > >/etc/yum/vars/contentdir || echo 'altarch' >/etc/yum/vars/contentdir`
> > >
> > > Here's the start of a proposal from the SIG perspective
> > >
> > > ## Requirements:
> > >
> > > - The same noarch binary RPM *must* be distributable to the Extras
> > >   repo for each architecture
> > >
> > > - The centos-release-<SIG> packages *must* require centos-release
> > >
> > >
> > > ## New Recommendations for creating a centos-release file:
> > >
> > > - A yum variable called 'contentdir' will be populated with the toplevel
> > >   directory for SIGs to use
> > >
> > > 1.) Modify baseurls in the SIG repo files to look like the following
> > (using
> > > openstack-queens as an example):
> > >
> > >
> > >  [centos-openstack-queens]
> > >  name=CentOS-7 - OpenStack queens
> > >  baseurl=http://mirror.centos.org/$contentdir/cloud/$
> > basearch/openstack-queens/
> > >  gpgcheck=1
> > >  enabled=1
> > >  gpgkey=file:///etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-SIG-Cloud
> > >  exclude=sip,PyQt4
> > >
> > >
> > > ## Discussion
> > >
> > > I'd like to hear thoughts, additions, and subtractions here before we
> > update
> > > the SIGGuide and rework centos-release.
> > >
> > > Cheers!
> > >
> > > --
> > > Brian Stinson
> > > _______________________________________________
> > > CentOS-devel mailing list
> > > CentOS-devel@centos.org
> > > https://lists.centos.org/mailman/listinfo/centos-devel
> >
> > To be clear,
> >
> > This is a PROPOSAL at this stage, we haven't actually made any changes
> > to centos-release yet. At this point if you're actively doing work
> > relying on the new variable, it will break!
> >
> > Also thank you to the folks up-thread who pointed out that there was a
> > typo in the repo entry. mirror.centos.org should use http://
> >
>
>
> Any update on the acceptance of the proposal?

This looks good to me. For the Gluster repos in the Storage SIG we'll
follow this approach as soon as the packages are synced to the mirror
(just filed bug 14631) and a new centos-release with the 'contentdir'
variable is available.

Thanks,
Niels