Skip to content

Conversation

@pchila
Copy link
Collaborator

@pchila pchila commented Jan 19, 2022

Use helm chart for installing a keptn service.
Introduce pre-release and release workflows.

@pchila pchila force-pushed the use-helm-chart-for-release branch from 577ac79 to 10623c1 Compare January 20, 2022 11:04
@pchila pchila marked this pull request as ready for review January 20, 2022 11:04
@pchila pchila changed the title Use helm chart for release ci: Use helm chart for release Jan 20, 2022
Copy link
Contributor

@christian-kreuzberger-dtx christian-kreuzberger-dtx left a comment

Choose a reason for hiding this comment

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

Failed to run skaffold with a remote GKE cluster:

0.11.1-2-g10623c1-dirty: digest: sha256:1e3e449187fdc37ed91e38db2cc4a7df287a65523aaabddfaea790019f6bf458 size: 949
Tags used in deployment:
 - ghcr.io/keptnsandbox/keptn-service-template-go -> docker.io/christiankreuzbergerdtx/ghcr_io_keptnsandbox_keptn-service-template-go:0.11.1-2-g10623c1-dirty@sha256:1e3e449187fdc37ed91e38db2cc4a7df287a65523aaabddfaea790019f6bf458
Starting deploy...
Helm release keptn-service-template-go not installed. Installing...
Error: INSTALLATION FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
keptn-service-template-go:
- image.repository: Does not match pattern '^[a-z][a-z0-9-./]{2,100}$'

Copy link
Contributor

@christian-kreuzberger-dtx christian-kreuzberger-dtx left a comment

Choose a reason for hiding this comment

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

LGTM

@pchila pchila force-pushed the use-helm-chart-for-release branch from 4dc20a1 to 5fc92a7 Compare January 21, 2022 10:32
@github-actions
Copy link

The following Docker Images have been built:
ghcr.io/keptn-sandbox/keptn-service-template-go:0.11.2-dev-PR-92

@pchila pchila merged commit 9063430 into master Jan 21, 2022
@pchila pchila deleted the use-helm-chart-for-release branch January 21, 2022 10:39
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.

3 participants