Were we aware that we essentially had the same issue with the
Packstack repository ?
Packstack uses branch names like "liberty" instead of
"stable/liberty", I just filed a bug for this [1]
[1]:
https://bugzilla.redhat.com/show_bug.cgi?id=1295503
David Moreau Simard
Senior Software Engineer | Openstack RDO
dmsimard = [irc, github, twitter]
On Mon, Dec 14, 2015 at 8:25 PM, Alan Pevec <apevec(a)gmail.com> wrote:
Hi all (but specifically Puppet folks in CC :)
we need to rebase openstack-puppet-modules[1] (short OPM) to Mitaka
while leaving Liberty on the stable branch asap.
OPM is a special case in Delorean, it is built from master-patches,
not master like other projects, because there are always some patches
required by packstack or rdom which are not or cannot be merged in
upstream openstack-puppet.
For Delorean Liberty rdoinfo was not forked, instead Delorean was
modified to try distro and source branch specified in projects.ini[2]
then fallback to default rpm-master and master respectively. This
works nicely for most of the projects but fallback is not active when
branch is specified explicitly in rdoinfo, like it is for OPM
special-case[3].
Alternative solutions are:
1. fork rdoinfo for Liberty
2. modify Delorean to support this special case
3. modify OPM repo to match working schema for other projects
AD 1. I'm -2, fork for just one special case is unjustified and
keeping everything else in sync would be wasteful.
AD 2. After quick poking at it, clear -2 from me.
AD 3. Rename branches in OPM repo like this:
current master -> upstream-master (verbatim copies of upstream
modules' master branches)
master-patches -> master (non-upstream patches rebased on top of
upstream-master)
current stable/liberty -> upstream-liberty (verbatim copies of
upstream modules' stable/liberty branches)
liberty-patches -> stable/liberty (non-upstream patches rebased on
top of upstream-liberty)
This would work immediately with current Delorean tooling, only
required change is to remove source-branch in rdoinfo for OPM,
and I hope OPM tooling could be modified easily to handle this change?
Cheers,
Alan
[1]
https://github.com/redhat-openstack/openstack-puppet-modules
[2]
https://github.com/redhat-openstack/delorean-instance/blob/2c182dd57c590c...
[3]
https://github.com/redhat-openstack/rdoinfo/blob/60e523481def987d6592f0dc...
_______________________________________________
Rdo-list mailing list
Rdo-list(a)redhat.com
https://www.redhat.com/mailman/listinfo/rdo-list
To unsubscribe: rdo-list-unsubscribe(a)redhat.com