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

Plugin command cleanup #296

Merged
merged 1 commit into from
Oct 7, 2020
Merged

Plugin command cleanup #296

merged 1 commit into from
Oct 7, 2020

Conversation

nitisht
Copy link
Contributor

@nitisht nitisht commented Sep 17, 2020

Add tenant generate, tenant expand, and delete

Generate command allows production users an opportunity to
optionally add/edit any additional fields in the yaml
before the tenant actually gets created.

Also rename operator create command to
operator init, volume add and volume list to
expand and info.

@nitisht nitisht requested review from harshavardhana, ravindk89 and Praveenrajmani and removed request for harshavardhana September 17, 2020 12:18
Copy link
Contributor

@ravindk89 ravindk89 left a comment

Choose a reason for hiding this comment

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

A few thoughts, not necessarily blockers.

README.md Outdated Show resolved Hide resolved
kubectl-minio/README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
harshavardhana
harshavardhana previously approved these changes Sep 17, 2020
@harshavardhana
Copy link
Member

Also rename operator create command to
operator init.

Did you talk to @abperiasamy about this?

@nitisht
Copy link
Contributor Author

nitisht commented Sep 17, 2020

Also rename operator create command to
operator init.

Did you talk to @abperiasamy about this?

Yes these are his recommendations

@nitisht nitisht marked this pull request as draft September 18, 2020 11:26
@nitisht nitisht changed the title Change the tenant create command to tenant create-yaml Plugin command cleanup Sep 23, 2020
kubectl-minio/README.md Outdated Show resolved Hide resolved
@nitisht nitisht marked this pull request as ready for review October 1, 2020 16:24
Add tenant generate, tenant expand, and delete

Generate command allows production users an opportunity to
optionally add/edit any additional fields in the yaml
before the tenant actually gets created.

Also rename operator create command to
operator init, volume add and volume list to
expand and info.
Copy link
Contributor

@ravindk89 ravindk89 left a comment

Choose a reason for hiding this comment

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

I think this is a good quickstart foundation! I'll start working on the corresponding docs based on the structure here. Once we merge I'll begin testing to validate the 'final' product.

@harshavardhana harshavardhana merged commit e91a441 into minio:master Oct 7, 2020
@nitisht nitisht deleted the cleanup branch October 7, 2020 04:28
jmontleon pushed a commit to jmontleon/operator that referenced this pull request Jul 23, 2024
Signed-off-by: David Zager <dzager@redhat.com>
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.

3 participants