Skip to content

(MODULES-6923) Migrate from puppet-staging to puppet-archive #1060

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

Closed
wants to merge 1 commit into from

Conversation

sbadia
Copy link

@sbadia sbadia commented Mar 27, 2018

The voxpupuli puppet-staging module is deprecated since ±1y, please see puppet-staging deprecation notice, we recomand to migrate to puppet-archive module (also maintained by voxpupuli team).

Staging module is only used for the mysql tunner script.

@@ -3,52 +3,20 @@
$ensure = 'present',
$version = 'v1.3.0',
$source = undef,
$environment = undef, # environment for staging::file
$environment = undef, # UNUSED (puppetlabs-mysql 5.3.0)
Copy link
Contributor

Choose a reason for hiding this comment

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

It's already a breaking change to add a new dependency, so we can just go ahead and remove this parameter.

Copy link
Author

Choose a reason for hiding this comment

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

ack, thanks @hunner for the review !

@sbadia sbadia force-pushed the migrate-to-archive branch from f30bec5 to 7dbf303 Compare March 30, 2018 14:17
@sbadia
Copy link
Author

sbadia commented Aug 19, 2018

Hi here,

Oh, the major version 6.0.0 was released without this simple change :-/
(puppet-staging is deprecated since a while…)

Please see voxpupuli/puppet-staging#162

@david22swan
Copy link
Member

david22swan commented Nov 5, 2018

@sbadia Apologies on the wait. If you could rebase you changes I would be happy to take a look at your PR.

@sbadia
Copy link
Author

sbadia commented Nov 6, 2018

Hi @david22swan !

No problem :-)

Finally MOD-6923 was fixed by @tphoney in the PR-1115.

I close this one,

Cheers,

@sbadia sbadia closed this Nov 6, 2018
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