Skip to content

YDBDOCS-668: add a playbook to show node logs #5

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

Closed
wants to merge 2 commits into from

Conversation

blinkov
Copy link
Member

@blinkov blinkov commented May 6, 2024

No description provided.

@blinkov blinkov requested a review from artgromov May 6, 2024 05:26
@blinkov blinkov changed the title Add a playbook to show node logs YDBDOCS-668: add a playbook to show node logs May 6, 2024
@@ -0,0 +1,28 @@
---
Copy link
Member

@artgromov artgromov May 6, 2024

Choose a reason for hiding this comment

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

You should put this playbook here https://github.com/ydb-platform/ydb-ansible/tree/main/playbooks
to avoid duplicate code and to be able to use it with any project.

Copy link
Member Author

Choose a reason for hiding this comment

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

@artgromov then why do we have setup_playbook.yaml copy-pasted to all examples? How people are supposed to discover the playbooks in that folder?

Copy link
Member

@artgromov artgromov May 7, 2024

Choose a reason for hiding this comment

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

@artgromov then why do we have setup_playbook.yaml copy-pasted to all examples?

I dont know, they seem excessive to me.

How people are supposed to discover the playbooks in that folder?

Only by reading collection documentation, which is absent for now.

That is the usual way for ansible collections. By making people copy-pasting example playbooks we are loosing maintainability for user-deployed systems. If we want to guarantee service continuity while using our playbooks, we should distribute them with ydb_platform collection and should heavily use various asserts.

Copy link
Member Author

@blinkov blinkov May 8, 2024

Choose a reason for hiding this comment

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

@artgromov ok, so it sounds like a plan:

  1. Here's a copy of this PR in that other repo: YDBDOCS-668: add a playbook to show node logs ydb-ansible#19
  2. We'll need to update the documentation so all users end up with Ansible 2.11+ (currently, I got 2.10 by following them on Ubuntu 22.04, and the steps below don't work)
  3. Recommend people to use playbooks from our collection via https://docs.ansible.com/ansible/latest/collections_guide/collections_using_playbooks.html#using-a-playbook-from-a-collection (and move setup_playbook.yaml there as well)
  4. Document other available playbooks.

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy of setup_playbook.yaml ydb-platform/ydb-ansible#20

Copy link
Member Author

Choose a reason for hiding this comment

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

Improvement of instructions to lower the chance of people accidentally ending up with an old Ansible version like I did: https://github.com/ydb-platform/ydb/pull/4394/files

@artgromov
Copy link
Member

Discarding this PR in favor of ydb-platform/ydb-ansible#19.

Playbook from the collection could be invoked using command:

ansible-playbook -i <your-inventory> ydb_platform.ydb.logs

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