Skip to content

Conversation

zaibon
Copy link
Contributor

@zaibon zaibon commented Aug 17, 2023

this is required in GKE to be able to configure custom backend healthcheck on this service.

@migmartri
Copy link
Member

Thanks for the PR! The CI seems to be complaining, please take a look at the DCO (signed off) requirement, thanks!

Copy link
Member

@migmartri migmartri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

In order to package this change as a new Chart version, please update the version in the Chart.yaml. Bumping the patch portion will do the trick :)

Thanks!

this is required in GKE to be able to configure custom backend healthcheck on this service.

Signed-off-by: Christophe de Carvalho <christophe@archipelo.co>
@zaibon zaibon force-pushed the controlplane-service-annotations branch from 1b93c0e to 28d348f Compare August 20, 2023 17:52
Signed-off-by: Christophe de Carvalho <christophe@archipelo.co>
Copy link
Member

@migmartri migmartri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@migmartri migmartri merged commit a7bb2a5 into chainloop-dev:main Aug 21, 2023
@zaibon zaibon deleted the controlplane-service-annotations branch August 31, 2023 11:16
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.

2 participants