-
Notifications
You must be signed in to change notification settings - Fork 18
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
Closes: #371 - invalid lineliness probe for web process in latest release of app #372
Conversation
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 you please fix a space between comments which fail our CI.
charts/nautobot/Chart.yaml
Outdated
appVersion: "2.0.5" | ||
version: "2.0.5" | ||
appVersion: "2.1.9" | ||
version: "2.0.6" |
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.
Do we want to go to 2.1.0
? Some changes are not just patches. Not breaking, but still:
We may need to add the changes here: helm-charts/charts/nautobot/Chart.yaml Lines 32 to 48 in af603ff
|
…ease of app
Fixes: #371
Update liveness probe to use health check management command instead of unauthenticated httpGet to
/api/