Skip to content

Conversation

@afa-farkhod
Copy link

📝Summary:

  • .gitignore update to exclude user-specific files, ansible related files, sample files, system editor files
  • create playbooks directory and move main.yml and prometheus_config.yml playbook files inside to better organize the directory structure:
playbooks
|-- main.yml
`-- prometheus_config.yml
  • update README.md to include new path of main.yml playbook file and add some tuning in to some variables
  • create group_vars/all.yml: Moving port numbers from individual Ansible roles and tasks to a centralized group_vars/all.yml provides maintainability, reusability, and clarity
  • assign variables instead of port numbers based on group_vars/all.yml, changed files:
roles/alert_manager/files/alertmanager.service
roles/grafana/files/grafana.nginx.conf
roles/loki/files/loki.yml
roles/prometheus/files/prometheus.nginx.conf
roles/prometheus/files/prometheus.yml
roles/promtail/files/promtail.yml

@afa-farkhod
Copy link
Author

@husonghua Hi there, please have a look in the following PR, thanks 😊

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.

1 participant