Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix pin prometheus grafana #301

Merged
merged 4 commits into from
Sep 14, 2023
Merged

Fix pin prometheus grafana #301

merged 4 commits into from
Sep 14, 2023

Conversation

i-chvets
Copy link
Contributor

@i-chvets i-chvets commented Sep 13, 2023

Description

Prometheus and Grafana updates on latest/edge require Juju > 3.0.3. Temporary pin to the revisions that will allow integration tests to deploy those charms.

More details are in:

Summary of changes

  • Pinned revisions of prometheus and grafana to avoid failure of integration tests. Refer to above issues for more details.

i-chvets and others added 4 commits March 28, 2023 09:30
* fix: add default value for pipelines
canonical/kfp-operators#159

Fixes #159 

Summary of changes:
- Added back default value for pipelines access.
- Added tests for default values of pipelines access and images for 1.7
  release.
- Modified code to check for mindspore image only not version.

---------

Co-authored-by: Andrew Scribner <[email protected]>
canonical/bundle-kubeflow#688
canonical/bundle-kubeflow#690

Summary of changes:
- Pinned revisions of prometheus and grafana to avoid failure of
  integration tests. Refer to above issues for more details.
Copy link
Contributor

@DnPlas DnPlas left a comment

Choose a reason for hiding this comment

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

Thanks @i-chvets

@i-chvets i-chvets merged commit e2758f6 into track/1.7 Sep 14, 2023
14 checks passed
@i-chvets i-chvets deleted the fix-pin-prometheus-grafana branch September 14, 2023 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants