bowlofeggs added a new comment to an issue you are following:
``
The first option has an additional negative in that it makes Bodhi more complicated to
deploy. It's important to keep in mind that Fedora is not the only user of Bodhi (I
know of at least 3 other deployments), and so we shouldn't hard code behaviors into it
because Fedora wants configuration to be a particular way. Like any other service, it
should expect to find the config files it needs inside /etc/, and the concerns for
managing those configs should be separate from the concerns of Bodhi. There are many ways
to do config management, and we shouldn't make Bodhi be opinionated about how configs
are managed. I'm pretty strongly opposed to the first option for these reasons.
I think second option (storing the configs in the infra Ansible repo) is the most
sensible. We shouldn't let the fact that Pungi and Bodhi are separate programs confuse
us. The Pungi config makes sense to think of as being a Bodhi config, since we think of
Bodhi as being the thing that creates the update repos. It happens to do it via Pungi, but
that's an implementation detail and so the configuration for making update repos
should live with the rest of Bodhi's configuration. Bodhi's mash configs are
stored in the infra ansible repo today, and it makes the most sense to keep them in there
because that's the place where we configure Bodhi.
For an analogous relationship between programs, consider libvirt and qemu. It wouldn't
be sensible for libvirt to git clone qemu configs. Qemu configs are managed alongside it
(and even by it, if so chosen, which is analogous to Martin's Bodhi PR that has Bodhi
dynamically creating Pungi configs for module "mashing"), because in the end
libvirt users don't care that qemu is how the VMs are made - they think of libvirt as
doing it.
P.S. It might make sense to move this ticket to the infra repository, since Bodhi mash
configs are managed in the infra repo today.
``
To reply, visit the link below or just reply to this email
https://pagure.io/releng/issue/6971