Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

feat!: Keptn 0.18 compatibility #363

Merged

Conversation

christian-kreuzberger-dtx
Copy link
Contributor

@christian-kreuzberger-dtx christian-kreuzberger-dtx commented Aug 29, 2022

Unfortunately with the introduction of go-utils v0.18.0 (#361) we lost compatiblity with Keptn 0.17 and older versions.
This is a breaking change in Keptn, and we need to go forward with it.

Therefore, this PR

  • adds Keptn 0.18 to integration tests
  • removes older version of Keptn from integration tests
  • Added next release of prometheus-service to compatibility matrix
  • Added a variable for the GITEA_PROVISIONER_VERSION and set it to the latest release 0.1.1

Fixes #362

integration test run with 0.18.1 is green: https://github.com/keptn-contrib/prometheus-service/actions/runs/2946696911

@github-actions
Copy link
Contributor

github-actions bot commented Aug 29, 2022

Unit Test Results

12 tests   11 ✔️  0s ⏱️
  6 suites    1 💤
  1 files      0

Results for commit 39371f3.

♻️ This comment has been updated with latest results.

Signed-off-by: Christian Kreuzberger <christian.kreuzberger@dynatrace.com>
@christian-kreuzberger-dtx christian-kreuzberger-dtx force-pushed the patch/update-keptn-integration-tests branch from 9910097 to 39371f3 Compare August 29, 2022 07:20
@christian-kreuzberger-dtx christian-kreuzberger-dtx changed the title test: Added Keptn 0.17 and 0.18 for integration tests feat!: Keptn 0.18 compatibility Aug 29, 2022
@github-actions
Copy link
Contributor

The following Docker Images have been built:
keptncontrib/prometheus-service:0.8.6-dev-PR-363,keptncontrib/prometheus-service:0.8.6-dev-PR-363.202208290721

Copy link
Member

@Raffy23 Raffy23 left a comment

Choose a reason for hiding this comment

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

LGTM

@christian-kreuzberger-dtx christian-kreuzberger-dtx merged commit 1288de5 into master Aug 29, 2022
@christian-kreuzberger-dtx christian-kreuzberger-dtx deleted the patch/update-keptn-integration-tests branch August 29, 2022 07:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integration tests failed
2 participants