kubernetes / cloud-provider-openstack

Apache License 2.0
599 stars 597 forks source link

[cinder-csi-plugin] [manila-csi-plugin] Update Cinder and Manila CSI charts maintainers #2583

Closed dulek closed 2 months ago

dulek commented 2 months ago

What this PR does / why we need it:

The GitHub account @brtknr does not exist anymore and our chart linter complains about this. This commit adds me as the maintainer.

Which issue this PR fixes(if applicable): fixes #2582

Special notes for reviewers:

Release note:

Maintainer for Cinder and Manila CSI charts is updated.
dulek commented 2 months ago

/hold

It's a terrible idea to add me, we don't use Helm in our products, so this is looking for a better maintainer candidate.

jichenjc commented 2 months ago

/lgtm

mnasiadka commented 2 months ago

/lgtm

k8s-ci-robot commented 2 months ago

@mnasiadka: changing LGTM is restricted to collaborators

In response to [this](https://github.com/kubernetes/cloud-provider-openstack/pull/2583#issuecomment-2076540948): >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
dulek commented 2 months ago

/hold cancel

zetaab commented 2 months ago

/approve

k8s-ci-robot commented 2 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: zetaab

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: - ~~[OWNERS](https://github.com/kubernetes/cloud-provider-openstack/blob/master/OWNERS)~~ [zetaab] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
zetaab commented 2 months ago

/lgtm