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

Monitoring #62

Open
wants to merge 28 commits into
base: trunk
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
d6b3fa1
set up metrics exporter
intarga Feb 5, 2025
2153168
add initial ingestor metrics
intarga Feb 5, 2025
b0fdf24
add security group for metrics, fix bug with group vars dir name
intarga Feb 7, 2025
67479ad
ansible: fix typos
intarga Feb 7, 2025
ececf8d
ansible: replace keypairs to work around hashing bug
intarga Feb 7, 2025
069f4bf
register metrics at startup
intarga Feb 7, 2025
ed755b1
fix rebase dependency breakage
intarga Feb 7, 2025
4aa1bd8
ansible: rework security group task to be declarative
intarga Feb 7, 2025
c6dbe86
ansible: add node and postgres metrics exporters
intarga Feb 10, 2025
7a2b1b3
ansible: fix wrong user in postgres_exporter config
intarga Feb 11, 2025
f49716b
ansible: enable stat_statements collector in postgres_exporter
intarga Feb 11, 2025
07792b8
ingestion: add histogram metric for request latency
intarga Feb 11, 2025
4e0a559
ingestion: add kafka_failures metric
intarga Feb 12, 2025
99c56cd
ansible: idempotentize install_postgres.yml
intarga Feb 12, 2025
1c3282b
ansible: update ostack python deps
intarga Feb 14, 2025
c04850d
ansible: add missing region configs to teardown.yml
intarga Feb 14, 2025
5f886fc
ansible: bring back `replace` based pg conf changes where needed
intarga Feb 14, 2025
65b0207
ansible ensure config applied and pg running at end of install_postgr…
intarga Feb 14, 2025
dabb502
ansible: disable pg tasks that can't run on replicas
intarga Feb 14, 2025
2ca7164
ansible: improve service handling in pg role
intarga Feb 14, 2025
5734528
ansible: fix wrong boolean operators
intarga Feb 14, 2025
c8cb411
ansible: idempotentize the conf file changes by checking the file first
intarga Feb 14, 2025
6852d08
ansible: handle replication check failing on first run
intarga Feb 14, 2025
433f492
ansible: enable pg_stat_statements
intarga Feb 14, 2025
f2139aa
ingestion: set up tracing
intarga Feb 18, 2025
373c236
.github: add ansible lint to ci
intarga Feb 18, 2025
90c872f
.github: merge ci and ansible workflows, fix ansible-lint
intarga Feb 18, 2025
e45ee03
ansible: satisfy ansible-lint
intarga Feb 19, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 0 additions & 23 deletions .github/workflows/ansible.yml

This file was deleted.

26 changes: 24 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@ name: Continuous integration

on:
pull_request:
paths-ignore:
- 'ansible/**'
types:
- opened
- reopened
Expand Down Expand Up @@ -84,3 +82,27 @@ jobs:
- name: Run Go tests
working-directory: migrations
run: go test -v ./...

ansible-format:
name: Check yaml formatting of ansible files
runs-on: ubuntu-latest
container: ghcr.io/google/yamlfmt:latest
steps:
- uses: actions/checkout@v4

- name: Check yaml format
working-directory: ansible
run: yamlfmt -lint

ansible-lint:
name: Lint ansible files
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- name: Run ansible-lint
uses: ansible/ansible-lint@main
with:
setup_python: "true"
working_directory: "ansible"
requirements_file: "requirements.yml"
Comment on lines +103 to +108
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this be bundled together with the yamlfmt check? I originally made a separate workflow because changing the ansible files does not really affect the rust/go tests (and vice versa), but I don't fully get why it doesn't work properly (reading your commit message).

Loading
Loading