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

[prometheus-ipmi-exporter] Add ipmi-exporter chart #3740

Merged
merged 13 commits into from
Sep 27, 2023

Conversation

lexfrei
Copy link
Contributor

@lexfrei lexfrei commented Aug 30, 2023

What this PR does / why we need it

  • Adds ipmi-exporter initial chart

Which issue this PR fixes

Special notes for your reviewer

сс @bitfehler

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

Aleksey Sviridkin added 2 commits August 30, 2023 14:41
Signed-off-by: Aleksey Sviridkin <[email protected]>
Signed-off-by: Aleksey Sviridkin <[email protected]>
@lexfrei lexfrei requested a review from a team as a code owner August 30, 2023 11:51
@lexfrei
Copy link
Contributor Author

lexfrei commented Sep 11, 2023

@zanhsieh do we need to do something to merge this PR?

@bitfehler
Copy link

Just wanted to say thanks and that I will link to this in the ipmi exporter README once it landed! Cheers!

@zanhsieh
Copy link
Contributor

Maybe ask @monotek to see if he can merge it. I can't merge it.

@adyekjaer
Copy link

@zeritti Any chance this can be merged ?

@zeritti
Copy link
Member

zeritti commented Sep 27, 2023

@zeritti Any chance this can be merged ?

I am sorry, I cannot help. An approval by @prometheus-community/helm-charts-admins is required.

@monotek monotek merged commit 0665981 into prometheus-community:main Sep 27, 2023
4 checks passed
Matiasmct pushed a commit to giffgaff/prometheus-charts-backup that referenced this pull request Mar 20, 2024
…ity#3740)

* add ipmi-exporter

Signed-off-by: Aleksey Sviridkin <[email protected]>

* switch to "official" image

Signed-off-by: Aleksey Sviridkin <[email protected]>

* add maintainer, keywords and home link

Signed-off-by: Aleksey Sviridkin <[email protected]>

* Update CODEOWNERS

Signed-off-by: MH <[email protected]>

* fix linting

Signed-off-by: Aleksey Sviridkin <[email protected]>

* Update charts/prometheus-ipmi-exporter/templates/deployment.yaml

Signed-off-by: MH <[email protected]>

---------

Signed-off-by: Aleksey Sviridkin <[email protected]>
Signed-off-by: MH <[email protected]>
Co-authored-by: MH <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Running in Kubernetes
6 participants