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

Enhance deployment workflows with monitoring and update health probes for workers #531

Merged

Conversation

tjementum
Copy link
Member

@tjementum tjementum commented Jul 8, 2024

Summary & Motivation

Add logic to deployment workflows to monitor deployments and fail workflows if activation does not result in a healthy running container. This ensures that any issues during deployment are promptly identified and addressed.

Change health probes for worker containers to use TCP instead of HTTP, improving reliability and accuracy in monitoring the health of worker containers.

Update deployment logic to ensure that any changes to _deploy-container.yml trigger the deployment of self-contained systems.

Checklist

  • I have added a Label to the pull-request
  • I have added tests, and done manual regression tests
  • I have updated the documentation, if necessary

@tjementum tjementum added the Enhancement New feature or request label Jul 8, 2024
@tjementum tjementum self-assigned this Jul 8, 2024
@tjementum tjementum requested a review from a team as a code owner July 8, 2024 12:26
@tjementum tjementum linked an issue Jul 8, 2024 that may be closed by this pull request
@tjementum tjementum force-pushed the monitory-container-health-before-succeeding-deployments branch from 3b94004 to 9b1e33d Compare July 8, 2024 12:31
@tjementum tjementum force-pushed the monitory-container-health-before-succeeding-deployments branch 2 times, most recently from ceb1a37 to 584a50f Compare July 8, 2024 12:50
Copy link

sonarqubecloud bot commented Jul 8, 2024

@tjementum tjementum merged commit 8fd14d0 into main Jul 8, 2024
16 checks passed
@tjementum tjementum deleted the monitory-container-health-before-succeeding-deployments branch July 8, 2024 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Configure HTTPS health probes
1 participant