-
Notifications
You must be signed in to change notification settings - Fork 39
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
Compatibility with newer Ansible versions (tested 2.17.5) #411
base: master
Are you sure you want to change the base?
Changes from all commits
1f7bcdf
2254cff
1a5600a
db22456
f7aec20
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,24 +1,4 @@ | ||
--- | ||
- name: deploy local Scylla on the Manager node | ||
import_role: | ||
name: "{{ role_path }}/../ansible-scylla-node" | ||
vars: | ||
# TODO how to use scylla_manager_db_vars to be passed on as role params? | ||
install_only: True | ||
scylla_manager_enabled: false | ||
scylla_version: 'latest' | ||
scylla_edition: "{{ scylla_manager_db_vars.scylla_edition|default('oss') }}" | ||
elrepo_kernel: false | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why did you remove this variable? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I get the impression that this manager setup wants the most basic local ScyllaDB setup as possible. Ideally, there shouldn't be much to pass in terms of variables in such a case. This variable it already happens to be false by default in the scylla-node role. Since it's not doing anything, I removed it. I.e. I think we should align the scylla-node role to have the most 'basic' defaults as possible. Minimal hassle to get an out-of-the-box working experience. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I've placed the variable back, and we can address such variables and defaults in a separate PR. |
||
scylla_repo_keyserver: "{{ scylla_manager_db_vars.scylla_repo_keyserver|default('') }}" | ||
scylla_repo_keys: "{{ scylla_manager_db_vars.scylla_repo_keys|default([]) }}" | ||
scylla_dependencies: "{{ scylla_manager_db_vars.scylla_dependencies|default([]) }}" | ||
scylla_ssl: | ||
internode: | ||
enabled: false | ||
client: | ||
enabled: false | ||
|
||
|
||
- name: install Scylla Manager | ||
include_tasks: "{{ ansible_os_family }}.yml" | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -60,8 +60,4 @@ | |
state: restarted | ||
enabled: yes | ||
become: true | ||
when: manager_agent_config_change.changed and start_scylla_service is defined and start_scylla_service|bool | ||
ignore_errors: true | ||
#TODO: remove ignore_errors when ansible is bumped to 2.10.4 or 2.9.16 as per https://github.com/ansible/ansible/issues/71528 | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is one of the changes that may break things with older Ansible There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is the only such change AFAICT |
||
|
||
when: manager_agent_config_change.changed and start_scylla_service is defined and start_scylla_service|bool |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
#!/usr/bin/env python3 | ||
#!/usr/bin/python3 | ||
# -*- coding: utf-8 -*- | ||
# | ||
# Copyright (C) 2021 ScyllaDB | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
#!/usr/bin/env python3 | ||
#!/usr/bin/python3 | ||
# -*- coding: utf-8 -*- | ||
# | ||
# Copyright (C) 2021 ScyllaDB | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
#!/usr/bin/env python3 | ||
#!/usr/bin/python3 | ||
# -*- coding: utf-8 -*- | ||
# | ||
# Copyright (C) 2021 ScyllaDB | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
#!/usr/bin/env python3 | ||
#!/usr/bin/python3 | ||
# -*- coding: utf-8 -*- | ||
# | ||
# Copyright (C) 2021 ScyllaDB | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
#!/usr/bin/env python3 | ||
#!/usr/bin/python3 | ||
# -*- coding: utf-8 -*- | ||
# | ||
# Copyright (C) 2021 ScyllaDB | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, this is how you do it these days, huh?
Interesting...