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

[PR #9383/482a90e8 backport][stable-10] add support for systemd creds encrypt/decrypt #9468

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Dec 29, 2024

This is a backport of PR #9383 as merged into main (482a90e).

SUMMARY

This PR adds systemd-creds encrypt/decrypt functionality.

ISSUE TYPE
  • New Module/Plugin Pull Request
COMPONENT NAME
  • systemd_creds_decrypt
  • systemd_creds_encrypt
ADDITIONAL INFORMATION

- Note that I haven't updated the version_added information.

  • Since systemd-creds encrypt encrypts the information using host specific settings, it needs to be run as well when testing systemd-creds decrypt.
  • The systemd-creds encrypt doesn't support (unencrypted plaintext) input credential files as the systemd tool does, only stdin as string.

* add support for systemd creds encrypt/decrypt

Signed-off-by: Thomas Sjögren <[email protected]>

* add __metaclass__

Signed-off-by: Thomas Sjögren <[email protected]>

* Python 2.7 issues

Signed-off-by: Thomas Sjögren <[email protected]>

* update version_added and ci test aliases

Signed-off-by: Thomas Sjögren <[email protected]>

* switch to container

Signed-off-by: Thomas Sjögren <[email protected]>

* run tests in docker as well

Signed-off-by: Thomas Sjögren <[email protected]>

* move tasks into tasks/

Signed-off-by: Thomas Sjögren <[email protected]>

* no need to call echo

Signed-off-by: Thomas Sjögren <[email protected]>

* lint and add become:

Signed-off-by: Thomas Sjögren <[email protected]>

* dont append a newline

Signed-off-by: Thomas Sjögren <[email protected]>

* don't clean newlines

Signed-off-by: Thomas Sjögren <[email protected]>

* only use module name

Signed-off-by: Thomas Sjögren <[email protected]>

* clean

Signed-off-by: Thomas Sjögren <[email protected]>

* change msg to value

Signed-off-by: Thomas Sjögren <[email protected]>

* add return values

Signed-off-by: Thomas Sjögren <[email protected]>

* update attributes and description

Signed-off-by: Thomas Sjögren <[email protected]>

* Update plugins/modules/systemd_creds_decrypt.py

Co-authored-by: Felix Fontein <[email protected]>

* set newline default

Signed-off-by: Thomas Sjögren <[email protected]>

* Update plugins/modules/systemd_creds_encrypt.py

Co-authored-by: Alexei Znamensky <[email protected]>

* Update plugins/modules/systemd_creds_encrypt.py

Co-authored-by: Alexei Znamensky <[email protected]>

* Update plugins/modules/systemd_creds_encrypt.py

Co-authored-by: Alexei Znamensky <[email protected]>

* update required and spelling

Signed-off-by: Thomas Sjögren <[email protected]>

* use single backslash

Signed-off-by: Thomas Sjögren <[email protected]>

---------

Signed-off-by: Thomas Sjögren <[email protected]>
Co-authored-by: Felix Fontein <[email protected]>
Co-authored-by: Alexei Znamensky <[email protected]>
(cherry picked from commit 482a90e)
@felixfontein felixfontein merged commit 4614047 into stable-10 Dec 29, 2024
129 checks passed
@felixfontein felixfontein deleted the patchback/backports/stable-10/482a90e8b4a66e1356c1b4771445edddfe1a026a/pr-9383 branch December 29, 2024 20:24
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.

2 participants