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

Suggest next available size when creating an object store #264

Merged
merged 5 commits into from
Sep 26, 2022

Conversation

haardikdharma10
Copy link
Contributor

Description

Minimum size to create an Object Store is 500 and they should be created in multiples of 500 GB. If the user passes a size > 500, which is not a multiple, the CLI will suggest the next available size.

@DMajrekar
Copy link
Contributor

This is missing tests and docs

Copy link
Contributor

@SaptarshiSarkar12 SaptarshiSarkar12 left a comment

Choose a reason for hiding this comment

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

LGTM 👍. The changes can be merged.

@SaptarshiSarkar12
Copy link
Contributor

Also, please anyone add docs for it as mentioned by @DMajrekar

cmd/objectstore/objectstore_create.go Outdated Show resolved Hide resolved
utility/confirmation.go Outdated Show resolved Hide resolved
}

return nil
}

Choose a reason for hiding this comment

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

A couple of unit test would assure everything works fine

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have created an issue to add test suite for the CLI - #271
CLI docs for object store have been updated and merged in - #270

Copy link
Member

@alessandroargentieri alessandroargentieri left a comment

Choose a reason for hiding this comment

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

LGTM

cmd/objectstore/objectstore_create.go Outdated Show resolved Hide resolved
@haardikdharma10 haardikdharma10 merged commit c9968d1 into master Sep 26, 2022
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.

None yet

4 participants