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

add support to take backups from PostgreSQL databases #3

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

pmereddy
Copy link

Description:
Created an ansible playbook to take backup of PostgreSQL databases and pull them to the local server

Usage:

  1. Review and update the list of databases to backup in vars.yml
  2. Update hosts field in pgsql_bkp.yml file to point to the ansible section that points to the PostgreSQL database host
  3. If you want to pull the backupsto your local server, set fetch_local to true in vars.yml
  4. ansible-playbook -u root -k ./pgsql_bkup.yml -i <ansible hosts file generated using utilities/nodes.py>

Copy link
Contributor

@Chaffelson Chaffelson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Please don't have spaces in file or directory names, it makes WIndows users suffer
  • Please add a readme to the directory with the instructions for usage.
  • Please add a section to the top level Readme pointing to this toolkit and its purpose

@pmereddy
Copy link
Author

@Chaffelson : I have made the changes you have requested.

@Chaffelson
Copy link
Contributor

@pmereddy can you please rebase this and I'll merge it.

Nishant-K-Raj and others added 10 commits November 2, 2022 09:18
* CDP PvC DS Pre-Check

* renamed version_check.py to preinstall_check.py to better reflect actual usage
Updated ReadMe.md
Refactored entirety of preinstall_check.py reducing overall SLOC by 30%

Signed-off-by: Daniel Chaffelson <[email protected]>

* Correctly quote spaces in cluster names
correctly write cells to xlsxwriter
Add basic logging support
Add passwordless SSH check at start of script
Simplify ECS check sequencing logic

Signed-off-by: Daniel Chaffelson <[email protected]>

* Added dns check for provided hostnames to fail early
Reordered top level check sequence to make it easy to comment checks you want to skip
Moved SSH errors check to static value at top of Handler class

Signed-off-by: Daniel Chaffelson <[email protected]>

Co-authored-by: Daniel Chaffelson <[email protected]>
Co-authored-by: @Nishant-K-Raj
* Shantanu HDP CHanges

* Update README.md

Signed-off-by: sgope <[email protected]>

* Create requirements.txt

Signed-off-by: sgope <[email protected]>

* Update README.md

Signed-off-by: sgope <[email protected]>

* fixing merge comments

Co-authored-by: Shantanu Gope <[email protected]>
Co-authored-by: sgope <[email protected]>
* Corrected functions for uninstall_cm.sh

* Refresh the notebook directory to share only current notebooks.
* Add flag for WXM log collection

* Fix README
@Chaffelson
Copy link
Contributor

@pmereddy Can you please fix your commit tree here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants