-
Notifications
You must be signed in to change notification settings - Fork 613
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(MODULES-8553) Further cleanup for package tag issues #1055
Conversation
postgresql::client is a classBreaking changes to this file WILL impact these modules (exact match):
Breaking changes to this file MAY impact these modules (near match): Breaking changes to this file MAY impact these modules (near match):
that may have no external impact to Forge modules. postgresql::lib::java is a classBreaking changes to this file MAY impact these modules (near match): that may have no external impact to Forge modules. postgresql::repo::yum_postgresql_org is a classthat may have no external impact to Forge modules. postgresql::server::contrib is a classBreaking changes to this file WILL impact these modules (exact match):
postgresql::server::extension is a typeBreaking changes to this file WILL impact these modules (exact match):
Breaking changes to this file MAY impact these modules (near match):
that may have no external impact to Forge modules. postgresql::server::plperl is a classthat may have no external impact to Forge modules. postgresql::server::plpython is a classthat may have no external impact to Forge modules. postgresql::server::postgis is a classBreaking changes to this file WILL impact these modules (exact match):
These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report. Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! The only thing I found was that lib/python.pp and lib/perl.pp are missing tags on their packages. The package names defined in params.pp appear to line up with what the Postgres repos provide. For example:
https://download.postgresql.org/pub/repos/yum/9.6/redhat/rhel-7.6-x86_64/
37c4414
to
facd625
Compare
|
Thanks for the review @Sharpie , that's it up to date with those two manifests as well :) Let me know if this is ready to go and I'll get it merged and a release of the back of it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Update all this modules tagged packages to use 'puppetlabs-postgresql' instead of just 'postgresql' which is ambiguous and causing dependancy issues.