MetaCell / cloud-harness

Other
14 stars 5 forks source link

Add tag / build reference api #718

Closed filippomc closed 5 months ago

filippomc commented 6 months ago

Closes CH-47

Implemented solution: the build id (as git commit hash) is now extracted by harness-deployment and added to values.yaml. The common service exposes it and the tag. Added an example on samples microservice image

How to test this PR: start samples.DOMAIN and verify that is shows the tag and build id as in the screenshot.

Sanity checks:

Breaking changes (select one):

Possible deployment updates issues (select one):

Test coverage (select one):

Documentation (select one):

Nice to have (if relevant):

filippomc commented 5 months ago

@zsinnema the version of this api is representative of the deployment so it's the same for all applications. It is indeed implemented only for the common app, what the samples app does here is to display the value in the ui.

zsinnema commented 5 months ago

@zsinnema the version of this api is representative of the deployment so it's the same for all applications. It is indeed implemented only for the common app, what the samples app does here is to display the value in the ui.

what I meant @filippomc is that we don't need an application to show the version number. We can just show it in the keycloak login screen.....

filippomc commented 5 months ago

@zsinnema I think it can be a good idea to show the version in the keycloak login screen, but we are not customizing the keycloak theme on ch, which makes updates simpler. Also derived applications have their theme and might not want to show the version there.