Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Install GPG when adding the repo on Debian systems #822

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

brigzzy
Copy link

@brigzzy brigzzy commented Sep 30, 2021

By default a Debian installation does not include GPG, but it's a requirement for the apt_key module.

https://docs.ansible.com/ansible/latest/collections/ansible/builtin/apt_key_module.html

This PR adds a play to install it, before apt_key is called, and lets the playbook work on a freshly provisioned Debian system.

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

5 similar comments
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

cla-checker-service bot commented Sep 30, 2021

❌ Author of the following commits did not sign a Contributor Agreement:
5cbfd84,

Please, read and sign the above mentioned agreement if you want to contribute to this project

@botelastic
Copy link

botelastic bot commented Jan 11, 2022

This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
To track this PR (even if closed), please open a corresponding issue if one does not already exist.

@ka2er
Copy link

ka2er commented Jan 26, 2022

need this fix. add to apply it manually. Can I help to get it merged ?

@botelastic botelastic bot removed the triage/stale label Jan 26, 2022
Copy link

@ka2er ka2er left a comment

Choose a reason for hiding this comment

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

tested it fix the issue on vanilla Debian

Copy link

@ka2er ka2er left a comment

Choose a reason for hiding this comment

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

reviewed

@ka2er
Copy link

ka2er commented Jan 26, 2022

@brigzzy could you sign contributor agreement in order to pass the CI/CD pipeline check ?

@brigzzy
Copy link
Author

brigzzy commented Jan 27, 2022

@ka2er Done!

@botelastic
Copy link

botelastic bot commented Jun 24, 2022

This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
To track this PR (even if closed), please open a corresponding issue if one does not already exist.

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

Successfully merging this pull request may close these issues.

3 participants