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

65/Setting puppetlabs/apt max version to 11.0.0 #73

Merged
merged 2 commits into from
Jan 28, 2025

Conversation

avitacco
Copy link
Contributor

I have tested this on Ubuntu, Debian, Alma, and Rocky and there are no issues.

This fixes #65

Notes for Reviewers

  • The commit history must be preserved - please use the rebase-merge or standard merge option instead of squash-merge
  • Sync up with the author before merging

I have tested this on Ubuntu, Debian, Alma, and Rocky and there are no
issues.
Copy link
Member

@bernd bernd left a comment

Choose a reason for hiding this comment

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

This requires us to bump the minimum required Puppet version. Can you please do that?

What's the convention for Puppet modules when bumping the required Puppet version? Should we do a major bump of the module?

I have tested this on Ubuntu, Debian, Alma, and Rocky and there are no
issues.
@avitacco
Copy link
Contributor Author

@bernd I have made the requested change to the metadata file, thanks for bringing that oversight to my attention.

To answer your other question; in the past, for modules I owned, I would bump the major version if I changed something that could make my module stop working on a system where it currently works. That is to say, if this were my module, I would bump the major version for this change.

@bernd
Copy link
Member

bernd commented Jan 28, 2025

@avitacco Thanks for the update! I agree, we should bump the major version. Thanks for your insights.

@bernd bernd merged commit 663d94f into Graylog2:main Jan 28, 2025
1 check passed
@avitacco avitacco deleted the dependency-version-bump branch January 28, 2025 13:29
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.

Please update compatiblitity of puppet-apt to atleast <10.0.0
2 participants