-
Notifications
You must be signed in to change notification settings - Fork 68
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
added health check in webserver docker container #486
Conversation
Codecov Report
@@ Coverage Diff @@
## main #486 +/- ##
==========================================
+ Coverage 66.30% 66.35% +0.04%
==========================================
Files 42 42
Lines 3642 3662 +20
==========================================
+ Hits 2415 2430 +15
- Misses 1054 1058 +4
- Partials 173 174 +1
Continue to review full report at Codecov.
|
airflow/docker.go
Outdated
func (d *DockerCompose) webserverHealthCheck() error { | ||
err := d.composeService.Events(context.Background(), d.projectName, api.EventsOptions{ | ||
Services: []string{config.CFG.WebserverContainerName.GetString()}, Consumer: func(event api.Event) error { | ||
if event.Status == "health_status: healthy" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can this status be a const string? followed by the exec_die as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make sense, will move it to a package level constant
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved to a package level constant
1a2639a
to
d145719
Compare
3016fb1
to
50a4876
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
馃殺
Description
Changes:
馃師 Issue(s)
Related astronomer/issues#3370
Related astronomer/issues#2997
馃И Functional Testing
馃摳 Screenshots
馃搵 Checklist
make test
before taking out of draft