Skip to content

Scripts from pre-collections branch #16

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

elabpro
Copy link
Contributor

@elabpro elabpro commented Nov 12, 2024

Set of scripts for executing ansible playbooks.
These scripts were located in 'pre-collections' branch of ansible collection.
This ones are updated regarding new playbooks in 'main' branch of ansible collection

@elabpro elabpro requested a review from blinkov November 12, 2024 13:54
@@ -0,0 +1,3 @@
#! /bin/sh

ansible-playbook -b -f 20 ydb_platform.ydb.binaries_all
Copy link
Member

Choose a reason for hiding this comment

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

Why do you recommend these? I always run just ansible-playbook <playbook-name>, and it works fine. And for advanced people running this at a larger-than-default scale, it should be trivial to figure out -f.

Comment on lines +3 to +4
BACKUP_LABEL=`date '+%Y-%m-%d_%H-%M-%S'`
ansible-playbook -b -f 20 ydb_platform.ydb.update_config --extra-vars "ydb_config_backup=${BACKUP_LABEL}"
Copy link
Member

Choose a reason for hiding this comment

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

Let's move this logic inside the playbook?

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.

2 participants