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

[pulmap] replace bad google credentials file #3950

Merged
merged 5 commits into from
May 31, 2023
Merged

Conversation

hackartisan
Copy link
Member

@hackartisan hackartisan commented May 30, 2023

pulibrary/pulmap#1141 is still needed, but this solidifies the earlier quick fix done on these machines, which was lost during a recent playbook run.

closes pulibrary/pulmap#1159

This has been run in production.

@hackartisan hackartisan changed the title Replace bad pulmap google credentials file [pulmap] replace bad google credentials file May 30, 2023
@acozine
Copy link
Contributor

acozine commented May 31, 2023

The CI failures on this PR are related to Molecule itself, not to the change made to the pulmap role. The Python stacktrace led us to two possible issues:
ansible/molecule#2540
and
ansible/molecule#3903
It seems 3903 was the issue here, though I'm not sure why we didn't see this failure on other tests. We should keep this in mind as we do our upgrades to everything - the problem with ansible-compat has been fixed in the latest versions of molecule.

Related to #3722 and #3782.

@acozine acozine merged commit 0c56a45 into main May 31, 2023
@acozine acozine deleted the pulmap1159-bad-creds branch May 31, 2023 16:40
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.

Thumbnails are all broken in search results
2 participants