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

Add support for modern keyrings #1128

Merged
merged 36 commits into from
Oct 27, 2023
Merged

Add support for modern keyrings #1128

merged 36 commits into from
Oct 27, 2023

Conversation

praj1001
Copy link

@praj1001 praj1001 commented Sep 12, 2023

Summary

Add apt::keyring defined type which creates modern-style keyrings. Just a PR with resolved comments and fixed parallel specs on PR #1105

Additional Context

Related Issues (if any)

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

Courtesy Pull Requests (PRs)

We extend our gratitude to the original authors who have paved the way for us in the realm of open-source development. Special thanks to @jorhett for taking the initiative and demonstrating the handling of modern apt GPG keyrings in PR #1105. Additionally, we would like to express our appreciation to @jps-help for their contributions in PR #1120, specifically in the area of keyring management.

Your efforts are highly valued and greatly appreciated. Keep up the great work, and let's continue to drive innovation in the open-source community! Cheers!"

@CLAassistant
Copy link

CLAassistant commented Sep 12, 2023

CLA assistant check
All committers have signed the CLA.

@kenyon
Copy link

kenyon commented Sep 12, 2023

Can you provide a better summary and title for this pull request?

Includes #1105.

@praj1001
Copy link
Author

Can you provide a better summary and title for this pull request?

Includes #1105.

Sure @kenyon

manifests/source.pp Outdated Show resolved Hide resolved
manifests/keyring.pp Outdated Show resolved Hide resolved
manifests/init.pp Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@smortex smortex left a comment

Choose a reason for hiding this comment

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

I did a first quick look without testing and apart some style issues, that looks great. This is really something that is needed and am looking forward for 👍

manifests/init.pp Outdated Show resolved Hide resolved
manifests/source.pp Outdated Show resolved Hide resolved
manifests/source.pp Outdated Show resolved Hide resolved
manifests/keyring.pp Outdated Show resolved Hide resolved
manifests/keyring.pp Outdated Show resolved Hide resolved
manifests/source.pp Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
manifests/init.pp Outdated Show resolved Hide resolved
@praj1001 praj1001 force-pushed the modern_debian_keyrings branch 6 times, most recently from 49c5de7 to fa62402 Compare September 28, 2023 09:39
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
manifests/source.pp Outdated Show resolved Hide resolved
manifests/source.pp Outdated Show resolved Hide resolved
manifests/source.pp Outdated Show resolved Hide resolved
manifests/source.pp Outdated Show resolved Hide resolved
manifests/source.pp Outdated Show resolved Hide resolved
@kenyon
Copy link

kenyon commented Sep 29, 2023

I think a PR title like "Add support for modern keyrings" would read better in the changelog.

@Ramesh7 Ramesh7 changed the title Modern debian keyrings Add support for modern keyrings Sep 29, 2023
jorhett and others added 5 commits September 29, 2023 10:15
Co-authored-by: Kenyon Ralph <kenyon@kenyonralph.com>
Co-authored-by: Kenyon Ralph <kenyon@kenyonralph.com>
Co-authored-by: Kenyon Ralph <kenyon@kenyonralph.com>
Co-authored-by: Kenyon Ralph <kenyon@kenyonralph.com>
kenyon added a commit to kenyon/puppetlabs-postgresql that referenced this pull request Jan 3, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in /etc/apt/keyrings and add a signed-by
option to the sources.list.d entry.
kenyon added a commit to kenyon/puppetlabs-docker that referenced this pull request Jan 3, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in `/etc/apt/keyrings` and add a `signed-by`
option to the `sources.list.d` entry.
kenyon added a commit to kenyon/puppetlabs-postgresql that referenced this pull request Jan 7, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in /etc/apt/keyrings and add a signed-by
option to the sources.list.d entry.
kenyon added a commit to kenyon/puppetlabs-docker that referenced this pull request Jan 7, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in `/etc/apt/keyrings` and add a `signed-by`
option to the `sources.list.d` entry.
smortex added a commit to voxpupuli/puppet-virtualbox that referenced this pull request Jan 8, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
smortex added a commit to voxpupuli/puppet-virtualbox that referenced this pull request Jan 8, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
smortex added a commit to ccin2p3/puppet-syslog_ng that referenced this pull request Jan 8, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
smortex added a commit to ccin2p3/puppet-syslog_ng that referenced this pull request Jan 8, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
smortex added a commit to choria-io/puppet-choria that referenced this pull request Jan 8, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
kenyon added a commit to kenyon/puppetlabs-postgresql that referenced this pull request Jan 11, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in /etc/apt/keyrings and add a signed-by
option to the sources.list.d entry.
smortex added a commit to ccin2p3/puppet-syslog_ng that referenced this pull request Jan 11, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
smortex added a commit to ccin2p3/puppet-syslog_ng that referenced this pull request Jan 11, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
smortex added a commit to choria-io/puppet-choria that referenced this pull request Jan 11, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
kenyon added a commit to kenyon/puppetlabs-postgresql that referenced this pull request Jan 17, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in /etc/apt/keyrings and add a signed-by
option to the sources.list.d entry.
smortex added a commit to ccin2p3/puppet-syslog_ng that referenced this pull request Feb 13, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
smortex added a commit to ccin2p3/puppet-syslog_ng that referenced this pull request Feb 13, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
smortex added a commit to ccin2p3/puppet-syslog_ng that referenced this pull request Feb 13, 2024
This makes use of puppetlabs/puppetlabs-apt#1128 to store the public key
in `/etc/apt/keyrings` and add a `signed-by` option to the
`sources.list.d` entry.
kenyon added a commit to kenyon/puppetlabs-postgresql that referenced this pull request Apr 4, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in /etc/apt/keyrings and add a signed-by
option to the sources.list.d entry.
kenyon added a commit to kenyon/puppetlabs-docker that referenced this pull request Apr 16, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in `/etc/apt/keyrings` and add a `signed-by`
option to the `sources.list.d` entry.
kali-brandwatch pushed a commit to brandwatch/puppetlabs-apt that referenced this pull request May 10, 2024
kenyon added a commit to kenyon/puppetlabs-docker that referenced this pull request Jun 7, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in `/etc/apt/keyrings` and add a `signed-by`
option to the `sources.list.d` entry.
kenyon added a commit to kenyon/puppetlabs-docker that referenced this pull request Sep 27, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in `/etc/apt/keyrings` and add a `signed-by`
option to the `sources.list.d` entry.
vaol pushed a commit to vaol/puppetlabs-postgresql that referenced this pull request Oct 13, 2024
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in /etc/apt/keyrings and add a signed-by
option to the sources.list.d entry.
kenyon added a commit to kenyon/puppetlabs-docker that referenced this pull request Jan 10, 2025
This makes use of puppetlabs/puppetlabs-apt#1128
to store the public key in `/etc/apt/keyrings` and add a `signed-by`
option to the `sources.list.d` entry.
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.

8 participants