Skip to content

Downgrade mysqld-exporter version#1487

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
vyzigold:downgrade_mysqld-exporter
Jun 24, 2025
Merged

Downgrade mysqld-exporter version#1487
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
vyzigold:downgrade_mysqld-exporter

Conversation

@vyzigold

@vyzigold vyzigold commented Jun 13, 2025

Copy link
Copy Markdown
Contributor

Downgrade the default mysqld-exporter image version to v0.15.1, because that's the last image buildable with go 1.21. The feature around the mysqld metrics is planed to release in FR3. I verified that the v0.15.1 image works for us with a job in telemetry-operator and a Depends-On. For the interested, the zuul build is here: https://softwarefactory-project.io/zuul/t/rdoproject.org/build/478c18a5708b443da0c46c3fe1ae3c31 .

@openshift-ci openshift-ci Bot requested review from frenzyfriday and stuggi June 13, 2025 10:04
vyzigold added a commit to vyzigold/telemetry-operator that referenced this pull request Jun 13, 2025
@vyzigold vyzigold changed the title [DNM] Try downgrading mysqld-exporter Downgrade mysqld-exporter version Jun 13, 2025
@vyzigold vyzigold force-pushed the downgrade_mysqld-exporter branch from f419ad7 to 21b7bdf Compare June 16, 2025 06:43

@stuggi stuggi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@stuggi

stuggi commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

/cherry-pick 18.0-fr3

@openshift-cherrypick-robot

Copy link
Copy Markdown

@stuggi: once the present PR merges, I will cherry-pick it on top of 18.0-fr3 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick 18.0-fr3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci

openshift-ci Bot commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprince, stuggi, vkmc, vyzigold

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@dprince

dprince commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

FR3 branch is now created, so when this lands please cherry-pick it there

Lower the version of the mysqld-exporter, which is the latest
buildable with go 1.21
@vyzigold vyzigold force-pushed the downgrade_mysqld-exporter branch from 21b7bdf to 2de3428 Compare June 23, 2025 15:19
@openshift-ci openshift-ci Bot removed the lgtm label Jun 23, 2025
@openshift-ci

openshift-ci Bot commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

New changes are detected. LGTM label has been removed.

@vyzigold

Copy link
Copy Markdown
Contributor Author

/retest

openshift-cherrypick-robot pushed a commit to openshift-cherrypick-robot/telemetry-operator that referenced this pull request Jun 23, 2025
@stuggi

stuggi commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

/retest

1 similar comment
@stuggi

stuggi commented Jun 24, 2025

Copy link
Copy Markdown
Contributor

/retest

@stuggi stuggi added the lgtm label Jun 24, 2025
@openshift-merge-bot openshift-merge-bot Bot merged commit e67fe4e into openstack-k8s-operators:main Jun 24, 2025
8 checks passed
@openshift-cherrypick-robot

Copy link
Copy Markdown

@stuggi: new pull request created: #1500

Details

In response to this:

/cherry-pick 18.0-fr3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

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.

5 participants