Skip to content

Commit e031b5e

Browse files
jardonca-scribner
andauthored
Add prometheus scraping and grafana support to charmed katib-controller operator (#1839)
* Add prometheus scraping and grafana support to charmed operator * Upgrade black version to 22.3.0 to fix issue with click dependency * fix: unpin `black`, fix formatting errors * fix: minor refactor of prometheus integration Revert to defaults for relation names and paths, where appropriate. * fix: apply operator linting checks only to source code * chore: point katib-db to charm in charmhub * fix: remove unneeded handling of prometheus relation event * feat: Add template dashboard and alert rules These are not working properly. When connecting to grafana, the dashboard shows up but does not populate properly with data. The data source appears wrong * fix: handle leader-elected events Without this, upgrade-charm does not work. * fix: correctly template the sample grafana dashboard * fix: remove placeholder grafana/prometheus files * fix: bump wait time to avoid flaky test failure Co-authored-by: Andrew Scribner <[email protected]>
1 parent c62bacd commit e031b5e

File tree

9 files changed

+3890
-7
lines changed

9 files changed

+3890
-7
lines changed

.github/workflows/test-charmed-katib.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,10 @@ jobs:
2121
sudo pip3 install black flake8
2222
2323
- name: Check black
24-
run: black --check operators
24+
run: black --check operators/*/src
2525

2626
- name: Check flake8
27-
run: cd operators && flake8
27+
run: cd operators && flake8 ./katib*/src
2828

2929
build:
3030
name: Test
@@ -73,7 +73,7 @@ jobs:
7373
sg microk8s -c 'juju bootstrap microk8s uk8s'
7474
juju add-model kubeflow
7575
juju bundle deploy --build --destructive-mode --serial
76-
juju wait -wvt 300
76+
juju wait -wvt 600
7777
7878
- name: Test Katib
7979
run: kubectl apply -f examples/v1beta1/hp-tuning/random.yaml

operators/bundle.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
bundle: kubernetes
22
applications:
33
katib-controller: { charm: katib-controller, scale: 1 }
4-
katib-db: { charm: cs:~charmed-osm/mariadb-k8s, scale: 1, options: { database: katib } }
4+
katib-db: { charm: charmed-osm-mariadb-k8s, scale: 1, options: { database: katib } }
55
katib-db-manager: { charm: katib-db-manager, scale: 1 }
66
katib-ui: { charm: katib-ui, scale: 1 }
77
relations:

0 commit comments

Comments
 (0)