-
Notifications
You must be signed in to change notification settings - Fork 16.9k
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: alexppg If they are not already assigned, you can assign the PR to them by writing The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Hi @alexppg. Thanks for your PR. I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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/test-infra repository. |
I'm not sure why it says that I fail the DCO. I've signed with my main e-mail, but it wants me to set the corporate mail. |
@alexppg the DCO checks that the signing email corresponds to the email in the commit itself.
|
Signed-off-by: Àlex Pérez-Pujol <alexperezpujol@disroot.org> Signed-off-by: Alex Perez-Pujol <alexperezpujol@disroot.org>
Signed-off-by: Àlex Pérez-Pujol <alexperezpujol@disroot.org> Signed-off-by: Alex Perez-Pujol <alexperezpujol@disroot.org>
Signed-off-by: Àlex Pérez-Pujol <alexperezpujol@disroot.org> Signed-off-by: Alex Perez-Pujol <alexperezpujol@disroot.org>
Signed-off-by: Alex Perez-Pujol <alexperezpujol@disroot.org>
Thanks @vsliouniaev, I've change it. I believe there's nothing to be done on my side until somebody reviews it, the lint that fails won't pass until some maintainer is asigned. |
Since you are submitting this chart to the repository, wouldn't it make sense that you should add yourself as the maintainer? |
Signed-off-by: Alex Perez-Pujol <alexperezpujol@disroot.org>
No problem @mvasilenko, but I believe I can't add myself like that and make the tests pass. In a prior commit I added myself to the maintainers section but this check says I'm not validated as a maintainer, and I don't know how it validates it. |
Signed-off-by: Alex Perez-Pujol <alexperezpujol@disroot.org>
ping @viglesiasce @foxish |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions. |
Just to keep this open as the bot is trying to close this. Is there anything still to be done here? |
@ntavares Somebody needs to revise it, as far as I know I don't have to do anything. |
hey @alexppg Have you considered hosting this chart and adding it to helm hub? |
I'd like to that, but right now I'm not able. I'll try to convince my company to do it, but for now is no possible. It's a problem @maorfr ? |
hey @alexppg, thanks for your reply! it is an effort problem. chart maintainers are over loaded, so reviewing a new chart usually takes a long time. with that said, the way forward is not this repository, but rather helm hub! :) Here are some things to consider if you are exploring hosting your own charts repository (copied from a similar PR): I think that you totally should host your own charts! This means that when users want to install your chart, instead of installing them from stable, they will install it from (for example) mycompany-charts: $ helm install --name my-release mycompany-charts/mychart In order for people to find out about your charts repo, you should add it to helm hub: Here are some instructions about how to add your charts repo to helm hub: Hope that answered all your questions! :) |
Thanks @maorfr ! I'll sure like to do that, because of all the reasons you mentioned. But as I said, I don't think this will be possible in a short time, so I'd like to maintain this chart here. Sorry and thanks for your kind explanation! |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions. |
This issue is being automatically closed due to inactivity. |
Signed-off-by: Àlex Pérez-Pujol alexperezpujol@disroot.org
What this PR does / why we need it:
It adds the prometheus-es-exporter to the stable chart repository.
Special notes for your reviewer:
None.
Checklist