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

Modify how pg password is set in postgres pod #1540

Merged

Conversation

rooftopcellist
Copy link
Member

SUMMARY

Alternate approach for #1539

ISSUE TYPE
  • Bug, Docs Fix or other nominal change

@rooftopcellist rooftopcellist force-pushed the modify-pg-env-var-alternate branch 3 times, most recently from edcc7e6 to f8925a1 Compare August 29, 2023 01:06
@rooftopcellist rooftopcellist changed the title Modify pg env var alternate Modify how pg password is set in postgres pod Aug 29, 2023
…l characters

 - Certain characters like quotations would not be interpreted correctly
   if passed through bash. This avoids that issue.
 - Set env var directly on the Stateful Set

Signed-off-by: Christian M. Adams <chadams@redhat.com>
@rooftopcellist
Copy link
Member Author

I just tested a migration manually with this code and it was successful.

@rooftopcellist rooftopcellist merged commit 7012a6a into ansible:devel Aug 29, 2023
6 checks passed
rooftopcellist added a commit to rooftopcellist/awx-operator that referenced this pull request Aug 29, 2023
 - this aligns with the approach taken in the data migration logic
   here: ansible#1540
rooftopcellist added a commit to rooftopcellist/awx-operator that referenced this pull request Aug 29, 2023
 - this aligns with the approach taken in the data migration logic
   here: ansible#1540

Signed-off-by: Christian M. Adams <chadams@redhat.com>
rooftopcellist added a commit to ansible/galaxy-operator that referenced this pull request Jan 26, 2024
- Adds keep-alives for Upgrades, Backups, Restores and Migrations
- Modify how pg password is set in postgres pod - related: ansible/awx-operator#1540

Signed-off-by: Christian M. Adams <chadams@redhat.com>
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.

None yet

2 participants