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

fix: remove microservice hosts from alerts #98

Merged
merged 1 commit into from
Sep 6, 2024
Merged

Conversation

DecFox
Copy link
Contributor

@DecFox DecFox commented Sep 6, 2024

This diff removes the microservice hosts from the prometheus alerts. Part of #93

Copy link

github-actions bot commented Sep 6, 2024

Ansible Run Output 🤖

Ansible Playbook Recap 🔍



Ansible playbook output 📖success

Show Execution

$ ansible-playbook playbook.yml --check --diff -i ../tf/modules/ansible_inventory/inventories/inventory-dev.ini
[WARNING]: provided hosts list is empty, only localhost is available. Note that
the implicit localhost does not match 'all'
[WARNING]: Could not match supplied host pattern, ignoring: clickhouse_servers
[WARNING]: Could not match supplied host pattern, ignoring: monitoring.ooni.org
[WARNING]: Could not match supplied host pattern, ignoring: data.ooni.org
[WARNING]: Could not match supplied host pattern, ignoring: openvpn-
server1.ooni.io

PLAY [ClickHouse servers] ******************************************************
skipping: no hosts matched

PLAY [Update monitoring config] ************************************************
skipping: no hosts matched

PLAY [Deploy data.ooni.org host] ***********************************************
skipping: no hosts matched

PLAY [Setup OpenVPN server] ****************************************************
skipping: no hosts matched

PLAY RECAP *********************************************************************

Pusher @DecFox
Action pull_request
Working Directory
Workflow .github/workflows/check_ansible.yml
Last updated Fri, 06 Sep 2024 11:45:22 GMT

@hellais
Copy link
Member

hellais commented Sep 6, 2024

LGTM

@hellais hellais merged commit 8edde89 into main Sep 6, 2024
2 checks passed
@hellais hellais deleted the fix/monitoring branch September 6, 2024 13:27
hellais added a commit that referenced this pull request Oct 1, 2024
* 'main' of github.com:ooni/devops:
  feat: add clickhouse proxy instance (#100)
  fix: remove microservice hosts from alerts (#98)
  refactor: remove individual load balancers from services (#97)
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