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

issues 449,448 remove_cluster.yml improvements #450

Merged
merged 2 commits into from
Aug 24, 2023
Merged

issues 449,448 remove_cluster.yml improvements #450

merged 2 commits into from
Aug 24, 2023

Conversation

rrrru
Copy link
Contributor

@rrrru rrrru commented Aug 23, 2023

@vitabaks vars -> default(false), blocks for dry, remove systemd file cuz consul role do not restart service if exist
#449

@rrrru rrrru changed the title issues 449,448 improvements issues 449,448 remove_cluster.yml improvements Aug 23, 2023
@rrrru
Copy link
Contributor Author

rrrru commented Aug 24, 2023

@vitabaks I set 'ignore_errors' because the task crashes when restarted.

@vitabaks
Copy link
Owner

@rrrru thanks!

I see that you decided not to add the task 'consul kv delete', why?

@rrrru
Copy link
Contributor Author

rrrru commented Aug 24, 2023

@rrrru thanks!

I see that you decided not to add the task 'consul kv delete', why?

At first I had wrong idea what the problem was.
It turned out that the problem was in the scope to which the task was distributed (postgre_cluster). So I added another group for conusl and specified consul_instances in hosts.

@vitabaks vitabaks merged commit c79a315 into vitabaks:master Aug 24, 2023
18 checks passed
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