Skip to content

(CONT-792) - Add Puppet 8/Drop Puppet 6 #1414

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

Merged
merged 45 commits into from
Apr 21, 2023
Merged

Conversation

jordanbreen28
Copy link
Contributor

This PR adds support for Puppet 8 and Drops support for puppet 6.

@jordanbreen28 jordanbreen28 requested a review from a team as a code owner April 19, 2023 13:32
@jordanbreen28 jordanbreen28 self-assigned this Apr 19, 2023
@puppet-community-rangefinder
Copy link

postgresql::postgresql_password is a function

Breaking changes to this file WILL impact these 11 modules (exact match):
Breaking changes to this file MAY impact these 3 modules (near match):

postgresql_psql is a type

Breaking changes to this file WILL impact these 8 modules (exact match):
Breaking changes to this file MAY impact these 7 modules (near match):

postgresql_replication_slot is a type

Breaking changes to this file MAY impact these 1 modules (near match):

This module is declared in 70 of 580 indexed public Puppetfiles.


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.

@jordanbreen28 jordanbreen28 force-pushed the CONT-792-add_puppet8_support branch from bbcbed1 to c77755b Compare April 19, 2023 15:51
@jordanbreen28 jordanbreen28 force-pushed the CONT-792-add_puppet8_support branch 4 times, most recently from 29d2f51 to a470e8d Compare April 20, 2023 15:20
@jordanbreen28 jordanbreen28 force-pushed the CONT-792-add_puppet8_support branch from a470e8d to a098797 Compare April 20, 2023 15:22
Copy link
Contributor

@LukasAud LukasAud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LukasAud LukasAud merged commit b80b8aa into main Apr 21, 2023
@LukasAud LukasAud deleted the CONT-792-add_puppet8_support branch April 21, 2023 10:16
TargetRubyVersion: '2.5'
ExtraDetails: true
DisplayStyleGuide: true
TargetRubyVersion: '2.7'
Copy link
Collaborator

@alexjfisher alexjfisher Apr 22, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Puppetserver 7 is only jruby 9.3 and therefore only compatible with ruby 2.6. If you try to fix any of the rubocop_todo.yml contents you will likely break this module for all users (since Puppet 8 isn't even out yet)

EDIT: Lots of cops were corrected, but I can't tell if any of them here do enforce 2.7+ features though. Big risk IMO though.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexjfisher thanks for being thorough and bringing this to our attention.. You have prompted an important conversation in the team.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants