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

Add support for RedHat #26

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open

Add support for RedHat #26

wants to merge 32 commits into from

Conversation

michal800106
Copy link

No description provided.

zielonkam and others added 19 commits June 1, 2015 15:46
Change-Id: Ifad64c3fe8ed6f6a3e0a51b82a669d020033f147
This is custom version without pylint requirement

Change-Id: Icb500b0ac016e9bfdc7e0044f369b26488d14368
Change-Id: I537fe762da3cc4f6729a68471bf87c3e7ba9c03d
…plugin config files"

This reverts commit 73db978, reversing
changes made to 3ea7948.
Because monasca-setup handler was run at the end of a playbook, it
was overwriting custom 'process' plugin configuration. We had to make
ansible assemble configuration from default and provided custom
one. This solution is generic and will work also in case of other
plugins.

Change-Id: I590ba36e618ae82465e10bcdce6d28fb6262bcf5
Installation is only fired when there's postgres configuration in
monasca_checks defined.

Change-Id: I62e5c3bd38d526db0bb95295db7e43ee06c920ae
Change-Id: Iab4a509a68d6d4fca37244edb18cb4b21985bae2
Change-Id: Iddebdedb41fcb41d68265727d44156e5d4fcc647
Change-Id: If35efa1f079fddbd609071b44d692d5c731c1ba1
Change-Id: Ifad64c3fe8ed6f6a3e0a51b82a669d020033f147
- name: Create reconfigure script
template: dest="{{agent_reconfigure_script}}" src=monasca-reconfigure.j2 owner=root group=root mode=750
notify: run monasca-setup
- name: Create additional plugins config
Copy link
Contributor

Choose a reason for hiding this comment

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

There is some rather complicated reworking of the additional configuration to fix things up. Looking at it a bit the only problem seems to be that the '--overwrite' flag is specified in the monasca-reconfigure script and it shouldn't be in the case of any additional configuration being specified. Wouldn't fixing that solve the problems with additional config and leave the merging logic to monasca-setup?

Maciej Maciaszek and others added 8 commits September 4, 2015 09:22
Custom configuration won't be overwritten if you set
`monasca_agent_overwrite_config` to True.

Change-Id: I30391e73ce08db7ab08043fe221441eaf1f1c6e4
Change-Id: Ifc3e39763ad2563c7673a6ade7a6c6578268ec50
Change-Id: I30b8c82f1066190e8d566d56088dd70eed643d54
Change-Id: I640fac670e46820046c821534b65456c0eacde65
Remove config file fragments, prevent backup generation.
Moved the task which ensures that proper permissions are set to the
bottom so that all files generated by handlers have proper permissions.

Change-Id: Iad96ad35e58813e22a660dafaadc98ca1dd56fb8
- add environment variable for pip task

Change-Id: If64a4412f58e5395ac163890ed681a65a109eb7b
This prevents the problem with the process not starting if it can't
write to log files. The situation may occur if the agent is removed
along with the system user and then reinstalled.

Change-Id: I6a3dd48103829e3f1e0ab5841be1dd5a93446bbf
zawisz and others added 5 commits February 19, 2016 11:53
Change-Id: Ib29cac4e7dbab8049d6a1ee00a3f8916defdb6a1
Added installation of extra packages that are required for
monasca-agent.

Change-Id: I07777044eaffca8e5d1b9107ac6af1d5b8112c8c
Remove installing pyscopg2 from task and postgresql_dir from main.yml. 

Change-Id: I47cf45f607fb0856bc1b7c66a2a90ae5ff3f0d86
This patch installs pbr into virtual environment
to deal with user proxy setting.

Change-Id: I0d17ba56ad8b29d88302f7934ee1e5a304a7072c
The purpuse of this flush is to trigger monasca-setup
before the role sets proper attributes of files.
Monasca-setup manipulates configuration files and
may change their owner and group to root.

Change-Id: I5a4fae31b9ce0e7510a220f0041c6ddfded643b0
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.

7 participants