vmware-archive / salt-pack

Salt Package Builder
Apache License 2.0
55 stars 23 forks source link

Oxygen update naming of Redhat family supplied packages to distinguish as from SaltStack #496

Open dmurphy18 opened 6 years ago

dmurphy18 commented 6 years ago

The following email thread captures the discussion, edited for brevity:

David Murphy dmurphy@saltstack.com Jan 16 (8 days ago)

to nicole, Mike, Thomas, bcc: me Given I am touching the most of the dependencies for building Salt on Redhat 6 & 7 with the addition of Python 3, I thought this might be a good time to change the packaging names.

There is an issue already covering this that has been long standing and it would be good to finally get it addressed. https://github.com/saltstack/salt/issues/29132

Proposal is to add and additional 'ss' to any package that we provide, including 'ius' packages

Current name: python-cherrypy-5.6.0-1.el6.noarch.rpm python27-2.7.13-2.ius.el6.x86_64.rpm

Renamed name: python-cherrypy-5.6.0-1.ss.el6.noarch.rpm python27-2.7.13-2.ius.ss.el6.x86_64.rpm

This would only affect the name of the package and help in identification of source, that is provide by SaltStack.

Can you please provide comments and/or approval to proceed.

David Murphy Unix Engineer dmurphy@saltstack.com | www.saltstack.com

Nicole Thomas Jan 16 (8 days ago)

to me, Mike, Thomas Hi David,

That sounds good to me, but since I don't have much experience in this matter, I'd love to see if Mike or Tom have any other concerns about this approach.

Thanks, Nicole

David Murphy dmurphy@saltstack.com Jan 16 (8 days ago)

to Nicole, Mike, Thomas, bcc: me Just chatting with Daniel and he suggests instead of using 'ss' it would be better to use 'salt' and to add an additional 's' to the name similar to IUS.

For example: python35u-mod_wsgi-4.5.24-1.ius.el6

we could use similarly in bold: python35s-mod-wsgi-4.5.24-1.salt.el6

Thus altering the name slightly and showing the repo

Nicole Thomas Jan 16 (8 days ago)

to me, Mike, Thomas I like that better - it is more clear.

David Murphy dmurphy@saltstack.com 10:30 AM (35 minutes ago)

to Nicole, Mike, Thomas, bcc: me @Tom, @Mike

Any objections to doing the proposed above in the email thread ?

Thomas S Hatch 10:35 AM (30 minutes ago)

to me, Nicole, Mike I think that is the right approach. -- Thomas S. Hatch | Founder, CTO | SaltStack, Inc.

dmurphy18 commented 6 years ago

Blocked since yum update expects the package names to match and not on what the package provides, for example : python-chardet-2.2.1-1.el7.noarch.rpm pythons-chardet-2.2.1-1.el7.noarch.rpm python-chardet-2.2.1-1.salt.el7.noarch.rpm

Are considered as three different items, even when they provide support for the same item. And yum gets upset if they try to manipulate a file which one of the others controls.

Renaming everything is beyond belief, it would result in a tracking nightmare of CVE and updates. Note that using IUS as an example in Python2.7 support for Redhat 6 was fine for them since Python 2.7 is not provided by Redhat or EPEL. That is, IUS doesn't have any competing packages supplying Python 2.7, not the packages they provide compete with Redhat 7 or EPEL.

Need to consider a different approach

sagetherage commented 4 years ago

is this a valid issue @dmurphy18 ?