Skip to content
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

(MAINT) Revert hardening changes #599

Merged
merged 3 commits into from Dec 13, 2022
Merged

Conversation

chelnak
Copy link
Contributor

@chelnak chelnak commented Dec 12, 2022

This PR cleans up a few discrepancies left over from the hardening work.

Along with the work from @deric , the work here should help towards returning main back to a green state.

@puppet-community-rangefinder
Copy link

kubernetes::kube_addons is a class

that may have no external impact to Forge modules.

kubernetes::kubeadm_init is a type

that may have no external impact to Forge modules.

kubernetes::kubeadm_join is a type

that may have no external impact to Forge modules.

kubernetes::packages is a class

that may have no external impact to Forge modules.

kubernetes::service is a class

that may have no external impact to Forge modules.

kubernetes::wait_for_default_sa is a type

that may have no external impact to Forge modules.

This module is declared in 0 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.

@chelnak chelnak self-assigned this Dec 12, 2022
@chelnak chelnak added the bug label Dec 12, 2022
@chelnak chelnak marked this pull request as ready for review December 12, 2022 11:00
@chelnak chelnak requested a review from a team as a code owner December 12, 2022 11:00
@chelnak chelnak force-pushed the maint-revert_hardening_changes branch 2 times, most recently from 5b4828a to e11b332 Compare December 12, 2022 11:27
@chelnak chelnak force-pushed the maint-revert_hardening_changes branch from e11b332 to cb74fc4 Compare December 12, 2022 12:28
This commit builds on the work that @deric submitted in a previous pr.

In a few other manifests there were variables set that added no value.
They have been reduced back to literal entries in the exec resources.

This commit applies those changes to this PR to keep things consistent.
@chelnak chelnak force-pushed the maint-revert_hardening_changes branch from d6078cd to fe6c101 Compare December 12, 2022 12:43
Copy link
Contributor

@GSPatton GSPatton left a comment

Choose a reason for hiding this comment

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

everything looks good

@GSPatton GSPatton merged commit d7b84b9 into main Dec 13, 2022
@GSPatton GSPatton deleted the maint-revert_hardening_changes branch December 13, 2022 09:44
@deric deric mentioned this pull request Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants