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

Fixes for CI/CD pipeline, Dockerfile #334

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

Conversation

mihirsamdarshi
Copy link
Collaborator

Dockerfile fixes

Added a dedicated health check endpoint for the NGINX server in the Dockerfile. This is located at /health and only accessible to local IP addresses. This is necessary for load balancing health checks in various environments.

CI/CD pipeline fixes

  • paths should have been paths-ignore
  • The deployment workflow automatically bumps the version, pushes the version bumped package.json file to the branch that it came from, builds the docker image, and pushes it to either the dev or prod repositories.

@mihirsamdarshi mihirsamdarshi requested a review from jimmyzhen May 4, 2024 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant