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

fix: Use distributor 0.18.0 for Keptn 0.18.0 compatibility #5

Merged
merged 1 commit into from
Aug 29, 2022

Conversation

christian-kreuzberger-dtx
Copy link
Contributor

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

During testing I noticed that jmeter-service does not work with Keptn 0.18, and I got the following error message:

time="2022-08-29T09:54:22Z" level=info msg="Loading jmeter/jmeter.conf.yaml for sockshop.dev.carts"
time="2022-08-29T09:54:22Z" level=info msg="error when trying to load jmeter/jmeter.conf.yaml file for service carts on stage dev or project-level sockshop"
time="2022-08-29T09:54:22Z" level=error msg="Unable to run JMeter tests: error when trying to load jmeter/jmeter.conf.yaml file for service carts on stage dev or project-level sockshop"

This was most likely caused by using an old version of keptn/distributor. Updating fixes it:

image

Signed-off-by: Christian Kreuzberger <christian.kreuzberger@dynatrace.com>
@github-actions
Copy link
Contributor

Unit Test Results

20 tests   20 ✔️  1m 0s ⏱️
  1 suites    0 💤
  1 files      0

Results for commit 0b5580b.

@github-actions
Copy link
Contributor

The following Docker Images have been built:
ghcr.io/keptn-contrib/jmeter-service:0.18.1-dev-PR-5,ghcr.io/keptn-contrib/jmeter-service:0.18.1-dev-PR-5.202208291055

@christian-kreuzberger-dtx christian-kreuzberger-dtx marked this pull request as ready for review August 29, 2022 11:16
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 deleted the patch/keptn-018-compat branch August 29, 2022 11:40
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.

2 participants