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 #427 backport][stable-1.5] ec2_key : Fix tests #778

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Apr 20, 2022

Depends-On: ansible/ansible-zuul-jobs#1498

This is a backport of PR #427 as merged into main (963a829).

SUMMARY

The Zuul nodes don't have OpenSSL installed on them, this breaks the generation of the fingerprints

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ec2_key

ADDITIONAL INFORMATION

https://dashboard.zuul.ansible.com/t/ansible/build/d79dcec2e3024558800bba5fd6917505/log/job-output.txt

"stderr": "/bin/sh: line 1: openssl: command not found"

ec2_key : Fix tests

SUMMARY
The Zuul nodes don't have OpenSSL installed on them, this breaks the generation of the fingerprints
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_key
ADDITIONAL INFORMATION
https://dashboard.zuul.ansible.com/t/ansible/build/d79dcec2e3024558800bba5fd6917505/log/job-output.txt
"stderr": "/bin/sh: line 1: openssl: command not found"

Depends-on: ansible-collections#460

Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
(cherry picked from commit 963a829)
@tremble tremble changed the title ec2_key : Fix tests (#427) [PR #427 backport][stable-1.5] ec2_key : Fix tests (#427) Apr 20, 2022
@tremble tremble changed the title [PR #427 backport][stable-1.5] ec2_key : Fix tests (#427) [PR #427 backport][stable-1.5] ec2_key : Fix tests Apr 20, 2022
@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration needs_triage tests tests labels Apr 20, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@tremble
Copy link
Contributor Author

tremble commented Apr 20, 2022

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@tremble tremble requested a review from alinabuzachis April 20, 2022 18:41
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Apr 21, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 0d1a0b4 into ansible-collections:stable-1.5 Apr 21, 2022
@tremble tremble deleted the backport/stable-1.5/pull-427 branch September 9, 2022 08:51
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
Ignore notifications for modules (wimnat)

SUMMARY
Add to ignore list of modules
ISSUE TYPE
N/A
COMPONENT NAME
N/A
ADDITIONAL INFORMATION
n/A

Reviewed-by: Mark Chappell <None>
Reviewed-by: None <None>
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 community_review integration tests/integration mergeit Merge the PR (SoftwareFactory) needs_triage tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants