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

fix: Update repo to work with cstar cluster #32

Draft
wants to merge 80 commits into
base: main
Choose a base branch
from

Conversation

lucaconsalvi
Copy link
Contributor

@lucaconsalvi lucaconsalvi commented Aug 20, 2024

All the PR changes:

Mod github workflows files
- build deploy form branch
- post merge
- postman test
- pr validation
- scan
- sonar cloud coverage

Add dockefile multistage and delete the native-micro. This new dockefile provides the way to build also the app jar instead of making only the image build. That makes possibile to build the app and run it in an image locally directly building it through docker.

Change the application.properties to get the values from the pod config map, previously mounted by the helm chart.

Change the postman test to work with the new URL ( CSTAR ) and not anymore by the container app.

To make liveness and readiness probe work:
Run:

mvn quarkus:add-extension -Dextensions='smallrye-health' (to add this dependency)
mvn depcheck:generate (to update the sha)
mvn clean install -U (to check that the repo build correctly)
The endpoint exposed by this dependency to check read and live are:

/q/health/live
/q/health/ready
( https://quarkus.io/guides/smallrye-health )

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copy link

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