Commit Graph

17 Commits

Author SHA1 Message Date
Jonathan Herlin b67bd60df3 Cloudkitty role cleanup and config updates
The Cloudkitty role has been lacking some attention for a while
causing some of the configuration and examples outdated.

Now that Cloudkitty playbook and dashboard can be installed using
OSA without running any additional playbooks from this repository
i have removed those from here.

Change-Id: I0fe96c318273f0016d93cf043bda74feb11c63f8
2021-06-09 10:03:21 +02:00
Jonathan Rosser 2b54345586 Use ansible_facts[] instead of fact variables
See https://github.com/ansible/ansible/issues/73654

Change-Id: Ia088d1b4074e3bbcd3054f5be00b39429437750d
2021-03-16 08:01:56 +00:00
Dmitriy Rabotyagov 6fb1e18a97 Use systemd-journald instead of log files
This patch aims to migrate service from usage of regular syslog files
to journald. We also disable uwsgi logging, since it dublicates
requests that are logged by service itself.

Change-Id: I4cd3270ef873b6f595b151014e82c8b4b35f12aa
2019-07-30 17:16:19 +03:00
Zuul 8f626b8cff Merge "Correct typo in variable name" 2018-09-08 21:02:59 +00:00
Zuul efa4714aeb Merge "Trivial: Fix the pep8 warning" 2018-08-17 05:16:53 +00:00
wangqi 7f96645298 Trivial: Fix the pep8 warning
The yaml should start with "---"

Change-Id: Iaf43682075718e85c1e22f8d4ac28fd65dea0d17
2018-08-06 07:23:44 +00:00
Kevin Carter b78966cd09
Convert role to use a common systemd service role
This removes the systemd service templates and tasks from this role and
leverages a common systemd service role instead. This change removes a
lot of code duplication across all roles all without sacrificing features
or functionality. The intention of this change is to ensure uniformity and
reduce the maintenance burden on the community when sweeping changes are
needed.

The systemd journal would normally be populated with the standard out of
a service however with the use of uwsgi this is not actually happening
resulting in us only capturing the logs from the uwsgi process instead
of the service itself. This change implements journal logging in the
service config, which is part of OSLO logging.

OSLO logging docs found here: <https://docs.openstack.org/oslo.log/3.28.1/journal.html>

Change-Id: Ieff0119cb09e8971fab5b0393f023010a554edff
Signed-off-by: Kevin Carter <kevin.carter@rackspace.com>
2018-07-31 14:50:12 -05:00
Jesse Pretorius 1ab9470007 Cleanup playbook, vars and overrides
1. The venv tag override is unnecessary, given there is a default.
2. The install branch override is unnecessary, given there is a
   default. The constraint in defaults was set to use the
   'cloudkitty_requirements_git_install_branch' instead of this
   variable anyway. This has been corrected to the right var.
3. The galera address for role tests has a specific var for it.
4. The 'cloudkitty_requirements_git_install_branch' is a relic
   of the past, and has been removed.
5. The playbook is very dated and is using a lot of old mechanisms
   and unnecessary vars. It is updated to be normalised with the
   playbooks from the integrated build.

Change-Id: I6a8e8b3ec8b00790416648beb1c4ce0257406409
2018-07-31 08:31:52 +01:00
Jesse Pretorius 19f299854e Move MQ vhost/user creation into role
There is no record for why we implement the MQ vhost/user creation
outside of the role in the playbook, when we could do it inside the
role.

Implementing it inside the role allows us to reduce the quantity of
group_vars duplicated from the role, and allows us to better document
the required variables in the role. The delegation can still be done
as it is done in the playbook too.

In this patch we implement two new variables:
- cloudkitty_oslomsg_rpc_setup_host
- cloudkitty_oslomsg_notify_setup_host

These are used in the role to allow delegation of the MQ vhost/user
setup for each type to any host, but they default to using the first
member of the applicable oslomsg host group.

We also adjust some of the defaults to automatically inherit existing
vars set in group_vars form the integrated build so that we do not
need to do the wiring in the integrated build's group vars. We still
default them in the role too for independent role usage.

Finally, we remove the test mq setup tasks and clean up any unused
or unnecessary variables configured in tests.

Change-Id: Iac217b80df3da79a944185a90ba4c1ff5409945b
2018-07-31 08:31:19 +01:00
bhujay eee07a2ddb Correct typo in variable name
Change-Id: Ie7ea4baa6cc4032ec4986f8c3ffc07dd6db6ce61
2018-07-20 07:41:05 +05:30
Jean-Philippe Evrard cad8710a8d Fix usage of "|" for tests
With the more recent versions of ansible, we should now use
"is" instead of the "|" sign for the tests.

This should fix it.

Change-Id: I77c037ec01e192ec0723050c05e296d4ccbfb79e
2018-07-12 16:54:43 +02:00
zhulingjie 44037f0c1b Remove the unnecessary space
Change-Id: I5dd365ec21959c60df5f83ad98705f94633e3d8e
2018-07-11 23:18:19 -04:00
Jesse Pretorius 3fec5b02ec Move database creation into role
There is no record for why we implement the database creation outside
of the role in the playbook, when we could do it inside the role.

Implementing it inside the role allows us to reduce the quantity of
group_vars duplicated from the role, and allows us to better document
the required variables in the role. The delegation can still be done
as it is done in the playbook too.

In this patch we implement a new variable called 'cloudkitty_db_setup_host'
which is used in the role to allow delegation of the database setup
task to any host, but defaults to the first member of the galera_all
host group. We also document the variable cloudkitty_galera_address which
has been used for a long time, but never documented.

Change-Id: I7bb3f38556558ef14723361d3672a1d06f839b52
2018-06-12 20:40:05 +01:00
Jean-Philippe Evrard 13b51e5c26 Do not log passwords
This prevents data to be leaked into the callback plugin.

Change-Id: If71d5efc15c4867f877d147349fb3c5ae7b42e95
2018-04-11 13:54:02 +02:00
Pedro Perez be53f8eca8 update naming conventions for secret vars
Change-Id: I523b5f1ae56c1ff19ca6bd95e3e0e7f00e282c71
2016-11-18 16:42:48 +01:00
Kevin Carter b13646c716 Remove deprecated ansible_ssh_host variable
This changes 'ansible_ssh_host' to 'ansible_host'. The 'ansible_ssh_host'
variable has been deprecated as noted here: [0].

[0] - http://docs.ansible.com/ansible/intro_inventory.html#hosts-and-groups

Change-Id: I667cb70dd0fef80699cc9f6cb44091cddaaaa57b
Related-Bug: #1636606
Signed-off-by: Kevin Carter <kevin.carter@rackspace.com>
2016-10-25 19:41:55 +00:00
Michael Rice 6d6ca0c8fa Adding the extras and tests with updated readme
This change includes the bits needed to add to osa such
as the playbook and env.d file.
2016-05-19 22:21:55 -05:00