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

(MODULES-10308) Allow downgrades when using apt #457

Merged

Conversation

beechtom
Copy link
Contributor

This updates the install task to allow for downgrading the Puppet agent
when installing with apt. This aligns the behavior of the task with
other package handlers, such as rpm, that allow for downgrading the
Puppet agent version.

Part of puppetlabs/bolt#1208

This updates the install task to allow for downgrading the Puppet agent
when installing with `apt`. This aligns the behavior of the task with
other package handlers, such as `rpm`, that allow for downgrading the
Puppet agent version.
@beechtom beechtom changed the title (GH-1208) Allow downgrades when using apt (MODULES-10308) Allow downgrades when using apt Dec 10, 2019
@beechtom beechtom marked this pull request as ready for review December 10, 2019 00:44
@puppetcla
Copy link

CLA signed by all contributors.

@gimmyxd gimmyxd requested a review from a team December 10, 2019 08:11
@Dorin-Pleava
Copy link
Contributor

Tested on a debian 10 machine and it is working as expected.

@Dorin-Pleava Dorin-Pleava merged commit 965928b into puppetlabs:master Dec 10, 2019
@donoghuc
Copy link
Member

@beechtom I dont think this will work downgrading across platform versions (IE 6 to 5) even with the --allow-downgrades option. Also I think this should have acceptance tests so we get a sense of how it works across all the platforms we test on.

@gimmyxd
Copy link
Contributor

gimmyxd commented Dec 10, 2019

Tomorrow we are going to release a new version of this module.
Would be better to revert this PR for that release? @donoghuc @beechtom

@donoghuc
Copy link
Member

I dont think the change will hurt anything and will be fine to add in a release. I just dont think the functionality fully works as described.

@beechtom
Copy link
Contributor Author

beechtom commented Dec 10, 2019

@gimmyxd I would revert the PR for release, and I'll work on getting acceptance tests written.

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

Successfully merging this pull request may close these issues.

None yet

5 participants