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

Improvements for kubect-minio plugin #1176

Merged
merged 1 commit into from
Jun 21, 2022

Conversation

Alevsk
Copy link
Contributor

@Alevsk Alevsk commented Jun 21, 2022

  • Use secure GenerateCredentials() function for tenants created via the
    krew plugin
  • Use configuration secret instead of credsSecret for tenants created
    via the krew plugin

Signed-off-by: Lenin Alevski alevsk.8772@gmail.com

@Alevsk Alevsk self-assigned this Jun 21, 2022
@Alevsk Alevsk force-pushed the kubectl-plugin-tenant-configuration branch from 3911df1 to ad8792b Compare June 21, 2022 02:48
dvaldivia
dvaldivia previously approved these changes Jun 21, 2022
pkg/apis/minio.min.io/v2/utils.go Outdated Show resolved Hide resolved
pkg/apis/minio.min.io/v2/utils.go Outdated Show resolved Hide resolved
pkg/apis/minio.min.io/v2/utils.go Outdated Show resolved Hide resolved
@Alevsk Alevsk force-pushed the kubectl-plugin-tenant-configuration branch from ad8792b to 7cbdfe7 Compare June 21, 2022 06:18
harshavardhana
harshavardhana previously approved these changes Jun 21, 2022
dvaldivia
dvaldivia previously approved these changes Jun 21, 2022
pkg/apis/minio.min.io/v2/utils.go Outdated Show resolved Hide resolved
pkg/apis/minio.min.io/v2/utils.go Outdated Show resolved Hide resolved
@harshavardhana harshavardhana dismissed stale reviews from dvaldivia and themself via ce75445 June 21, 2022 12:30
- Use secure GenerateCredentials() function for tenants created via the
  krew plugin
- Use configuration secret instead of credsSecret for tenants created
  via the krew plugin

Signed-off-by: Lenin Alevski <alevsk.8772@gmail.com>
@Alevsk Alevsk force-pushed the kubectl-plugin-tenant-configuration branch from b250897 to 091078c Compare June 21, 2022 19:09
@dvaldivia dvaldivia merged commit b2dccd7 into minio:master Jun 21, 2022
@Alevsk Alevsk deleted the kubectl-plugin-tenant-configuration branch June 21, 2022 20:06
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.

4 participants