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

fix: Use configuration-service for now #364

Closed
wants to merge 1 commit into from

Conversation

christian-kreuzberger-dtx
Copy link
Contributor

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

Apparently using "http://resource-service:8080" breaks backwards compatibility for now.
Introduced in: #361

This PR changes it back to configuration-service:8080.

Fixes #362 🤞
Integration Test Run: https://github.com/keptn-contrib/prometheus-service/actions/runs/2946474984

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

Unit Test Results

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

Results for commit 9141407.

@github-actions
Copy link
Contributor

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

@christian-kreuzberger-dtx
Copy link
Contributor Author

doesn't fix the problem; superseeded by #363

@christian-kreuzberger-dtx christian-kreuzberger-dtx deleted the patch/config-service branch August 29, 2022 07:23
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
1 participant