[Pkg-puppet-devel] puppet-agent_7.16.0-1_amd64.changes ACCEPTED into experimental, experimental

[Thomas Goirand]

Hi Jerome,

Thanks for caring about Puppet and for your quick reply.

On 5/16/22 14:35, Jerome Charaoui wrote:
> Le 2022-05-16 à 03 h 56, Thomas Goirand a écrit :
>> On 5/15/22 12:00, Debian FTP Masters wrote:
>>>     * Renamed service to "puppet-agent" with "puppet" alias
>>>     * debian/control:
>>>       * added new dependencies: ruby-concurrent and ruby-semantic-puppet
>>>       * do not build puppet-master or puppet-common packages
>>>       * build puppet -> puppet-agent transitional package
>>>       * add Breaks/Replaces older puppet package
>>
>>
>> Hi,
>>
>> We already did a transition puppet-agent -> puppet in the past, I 
>> don't think we want to do that work on the opposite direction a few 
>> release later.
>>
>> Jerome, could you please revert this, and keep the name puppet-agent 
>> for the binary? It's ok if you add a Provides: if you like though.
> 
> I think we should definitely keep the source package named 
> "puppet-agent" because this one only builds the agent whereas the old 
> "puppet" 5.5 source package built both agent and server components.

There's on consequence switching the source package name, and never the 
need to ask anyone about it, so I honestly don't care about this.

> As for the binary package I don't have a strong opinion either way, 
> though its true that keeping the name "puppet" for the binary of puppet 
> agent would at least avoid the need to update modules that manage the 
> agent packages on Debian & family.

I do (have a strong opinion). See the list of reverse dependency below. 
If you wish to make that change backward (ie: back from puppet to 
puppet-agent like it was 3 or 4 Debian releases ago), please also NMU 
changes to all of the below packages. That's more than 90 packages to 
change. All of that for almost no benefits (and no motivation given). If 
you do have a real reason to do it, I'm fine with discussing it in this 
list, but otherwise, I'm re-iterating: could you please reverse the 
binary package name change?

Cheers,

Thomas Goirand (zigo)

~ $ reverse-depends puppet -r testing
Reverse-Depends
* librarian-puppet
* puppet-master
* puppet-master-passenger
* puppet-module-aboe-chrony
* puppet-module-adrienthebo-filemapper
* puppet-module-alteholz-tdc
* puppet-module-antonlindstrom-powerdns
* puppet-module-aodh
* puppet-module-arioch-redis
* puppet-module-asciiduck-sssd
* puppet-module-barbican
* puppet-module-camptocamp-augeas
* puppet-module-camptocamp-kmod
* puppet-module-camptocamp-openssl
* puppet-module-camptocamp-postfix
* puppet-module-ceilometer
* puppet-module-ceph
* puppet-module-cinder
* puppet-module-cirrax-gitolite
* puppet-module-cloudkitty
* puppet-module-congress
* puppet-module-cristifalcas-etcd
* puppet-module-debian-archvsync
* puppet-module-deric-zookeeper
* puppet-module-designate
* puppet-module-duritong-sysctl
* puppet-module-etcddiscovery
* puppet-module-glance
* puppet-module-gnocchi
* puppet-module-heat
* puppet-module-heini-wait-for
* puppet-module-horizon
* puppet-module-icann-quagga
* puppet-module-icann-tea
* puppet-module-ironic
* puppet-module-joshuabaird-ipaclient
* puppet-module-keystone
* puppet-module-magnum
* puppet-module-manila
* puppet-module-michaeltchapman-galera
* puppet-module-mistral
* puppet-module-murano
* puppet-module-nanliu-staging
* puppet-module-neutron
* puppet-module-nova
* puppet-module-octavia
* puppet-module-openstack-extras
* puppet-module-openstacklib
* puppet-module-oslo
* puppet-module-ovn
* puppet-module-panko
* puppet-module-placement
* puppet-module-puppet-archive
* puppet-module-puppet-community-mcollective
* puppet-module-puppetlabs-apache
* puppet-module-puppetlabs-apt
* puppet-module-puppetlabs-concat
* puppet-module-puppetlabs-firewall
* puppet-module-puppetlabs-haproxy
* puppet-module-puppetlabs-inifile
* puppet-module-puppetlabs-mailalias-core
* puppet-module-puppetlabs-mongodb
* puppet-module-puppetlabs-mysql
* puppet-module-puppetlabs-ntp
* puppet-module-puppetlabs-postgresql
* puppet-module-puppetlabs-rabbitmq
* puppet-module-puppetlabs-rsync
* puppet-module-puppetlabs-stdlib
* puppet-module-puppetlabs-tftp
* puppet-module-puppetlabs-translate
* puppet-module-puppetlabs-vcsrepo
* puppet-module-puppetlabs-xinetd
* puppet-module-richardc-datacat
* puppet-module-rodjek-logrotate
* puppet-module-sahara
* puppet-module-saz-memcached
* puppet-module-saz-rsyslog
* puppet-module-saz-ssh
* puppet-module-sbitio-monit
* puppet-module-swift
* puppet-module-tempest
* puppet-module-theforeman-dns
* puppet-module-voxpupuli-alternatives
* puppet-module-voxpupuli-collectd
* puppet-module-voxpupuli-corosync
* puppet-module-voxpupuli-posix-acl
* puppet-module-voxpupuli-ssh-keygen
* puppet-module-vswitch
* puppet-strings
* ruby-puppet-syntax
* ruby-rspec-puppet
* ruby-rspec-puppet-facts