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.

Sign up for GitHub

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

Jump to bottom

Deploy healthmonitor #151

Draft
wants to merge 56 commits into
base: master
Choose a base branch
from
Draft

Deploy healthmonitor #151

wants to merge 56 commits into from

Conversation

nurbal
Copy link
Collaborator

@nurbal nurbal commented Feb 17, 2025 •
edited
Loading

Goals of this PR:

  • Create HealthCheck classes to handle sanity checks and launch them with the Health Monitor
  • deploy HealthMonitor to production
  • output checks results to loki

depends on #139 and #150

(draft, work in progress)

nurbal and others added 30 commits October 11, 2024 18:04
@nurbal
Add loki settings
04d4970
@nurbal
config update : loki settings
862ef96
@nurbal
add loki logger to CLI main
a695a60
@nurbal
wip
940cf30
@nurbal
wip
321a9d3
@nurbal
remove some print
77449e4
@nurbal
fix strange syntax ??
149d213
@nurbal
lint
def75d5
@nurbal
fix test config
88f1dc3
@nurbal
wip
1a0a280
@nurbal
lint
5cc2451
@nurbal
move OTLP config to LoggingConfig
f261fad
@nurbal
Merge commit '67f000a6888c803f6169bab52b1978b3a264e7c7' into SARC-368…
7e54cf4 
…-loki-connect
@nurbal
update test `test_tracer_with_multiple_clusters_and_dates_and_prometh…
f2480c1 
…eus`
@nurbal
lint
e61bac9
@nurbal
fix config files
d06482d
@nurbal
update
1c20277
@nurbal
Merge commit '6cd17ec3689f5941ae59d4563858b546397543e8' into SARC-368…
6b2e8eb 
…-loki-connect

# Conflicts:
#	poetry.lock
@nurbal
Merge commit '85bd044a1315e48acece43b9007379f4a5aac582' into SARC-368…
af92b35 
…-loki-connect
@nurbal
update poetry.lock
b0b2be6
@nurbal
Merge commit '87ad1b11e95eb157fba66bd108bf5a7d99227cb7' into SARC-368…
478c896 
…-loki-connect
@nurbal
parametrise check_cluster_response() with cluster name (optional)
db431f8
@nurbal
added --once parameter to health check command
f430722
@nurbal
attempt to add a functionnal test to OTPL / Loki with a HTTPServer
8c6a480
@nurbal
update poetry.lock
c5977e5
@nurbal
Merge branch 'master' into SARC-368-loki-connect
1075a84
@nurbal
Merge commit '25649c7adcca7188a5db3fb12457da344f8a2e9b' into deploy_h…
777aa2d 
…ealthmonitor

# Conflicts:
#	sarc/cli/health/check.py
@nurbal
Add test_loki_logging_handler
3ab2c68
@nurbal
Merge branch 'master' into SARC-368-loki-connect
0a6c9a7
@nurbal
Merge branch 'master' into SARC-368-loki-connect
f27d6c6
nurbal added 26 commits January 30, 2025 15:35
@nurbal
Merge branch 'master' into deploy_healthmonitor
face417
@nurbal
fix check_cluster_response
744ae2d
@nurbal
add some sample checks functions
11dbb05
@nurbal
added --write optionnal parameter to sarc health check --once and…
9e7017c 
… `sarc health check --name`
@nurbal
wip
95cf2cd
@nurbal
some comments on test_loki_logging_handler
aea577e
@nurbal
fix imports position ni the file
ea53b3f
@nurbal
take verbose command-line parameter into account
f77ee66
@nurbal
black
6b569f8
@nurbal
update config file
38a5c9a
@nurbal
update sarc-dev.json
486472f
@nurbal
black
3b4afe1
@nurbal
fix logging level priority between command-line and config file
ab430fe
@nurbal
black
b6ca44f
@nurbal
ADD sanity check for jobs from users without @mila.quebec email address
939dd07 
FIX account matching error due to duplicate entries in drac_members CSV file
@nurbal
black
aefeb94
@nurbal
fix tests
aa3f1f3
@nurbal
fix lint
aba7973
@nurbal
Merge branch 'fix_account_matching' into deployed
6bb94b5
@nurbal
fix some return typings
d023399
@nurbal
lint
ad08c52
@nurbal
Merge branch 'deployed' into deploy_healthmonitor
6867b9f
@nurbal
Merge branch 'fix_account_matching' into deployed
9594441
@nurbal
Merge branch 'deployed' into deploy_healthmonitor
d69455d
@nurbal
add UsersInJobsCheck HealthCheck
a7793e6
@nurbal
set default logging level to WARNING (previously: DEBUG)
e626d7c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nurbal