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

onepassword_raw - Add missing parameter to doc string #5511

Merged

Conversation

samdoran
Copy link
Contributor

@samdoran samdoran commented Nov 8, 2022

SUMMARY

Fixes #5506

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

plugins/lookup/onepassword_raw.py

ADDITIONAL INFORMATION

This might fail sanity tests since the version_added is 6.0.0 which is already release. Though I'm curious how this passed sanity tests originally...

@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added bug This issue/PR relates to a bug lookup lookup plugin plugins plugin (any type) labels Nov 8, 2022
@felixfontein
Copy link
Collaborator

This might fail sanity tests since the version_added is 6.0.0 which is already release. Though I'm curious how this passed sanity tests originally...

No worry, ansible/ansible#69291 never got finished, so it won't fail :)

@felixfontein felixfontein added the check-before-release PR will be looked at again shortly before release and merged if possible. label Nov 8, 2022
@github-actions
Copy link

github-actions bot commented Nov 8, 2022

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@samdoran samdoran force-pushed the issue/5506-1p_raw-v1-failure branch from 9a0e3fd to 58ae13e Compare November 8, 2022 22:11
@samdoran
Copy link
Contributor Author

samdoran commented Nov 8, 2022

Thanks for all your help @felixfontein!

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Nov 9, 2022
@felixfontein felixfontein merged commit c604cc5 into ansible-collections:main Nov 9, 2022
@patchback
Copy link

patchback bot commented Nov 9, 2022

Backport to stable-6: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-6/c604cc5ba901574697b99bd31236d99e0e6cd1a3/pr-5511

Backported as #5518

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Nov 9, 2022
* onepassword_raw - Add missing parameter to doc string

* Remove redundant mention of default value

* Update changelogs/fragments/5506-onepassword_raw-missing-param.yml

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit c604cc5)
@felixfontein
Copy link
Collaborator

@samdoran thanks for this one as well!

felixfontein pushed a commit that referenced this pull request Nov 9, 2022
* onepassword_raw - Add missing parameter to doc string

* Remove redundant mention of default value

* Update changelogs/fragments/5506-onepassword_raw-missing-param.yml

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit c604cc5)

Co-authored-by: Sam Doran <sdoran@redhat.com>
@samdoran samdoran deleted the issue/5506-1p_raw-v1-failure branch November 9, 2022 18:50
russoz pushed a commit to russoz-ansible/community.general that referenced this pull request Nov 10, 2022
…tions#5511)

* onepassword_raw - Add missing parameter to doc string

* Remove redundant mention of default value

* Update changelogs/fragments/5506-onepassword_raw-missing-param.yml

Co-authored-by: Felix Fontein <felix@fontein.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug lookup lookup plugin plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

onepassword_raw; V2 (general 6.0.0); domain was not defined in configuration.
3 participants