-
Notifications
You must be signed in to change notification settings - Fork 605
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
Adds support for creating single namespace store bucket class #8266
Adds support for creating single namespace store bucket class #8266
Conversation
b320f67
to
698cada
Compare
698cada
to
b320f67
Compare
b320f67
to
f9cd2a5
Compare
f9cd2a5
to
b320f67
Compare
/test ceph-storage-plugin |
b320f67
to
9f83e66
Compare
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.scss
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.scss
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/review-utils.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/general-page.tsx
Outdated
Show resolved
Hide resolved
...kages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/namespace-policy-page.tsx
Outdated
Show resolved
Hide resolved
...end/packages/ceph-storage-plugin/src/components/namespace-store/namespace-store-dropdown.tsx
Outdated
Show resolved
Hide resolved
...end/packages/ceph-storage-plugin/src/components/namespace-store/namespace-store-dropdown.tsx
Outdated
Show resolved
Hide resolved
...end/packages/ceph-storage-plugin/src/components/namespace-store/namespace-store-dropdown.tsx
Outdated
Show resolved
Hide resolved
...end/packages/ceph-storage-plugin/src/components/namespace-store/namespace-store-dropdown.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/constants/providers.ts
Outdated
Show resolved
Hide resolved
9f83e66
to
97c7a0c
Compare
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/shared.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/shared.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/shared.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you check and fix all such occurrences
Backing Store -> BackingStore
Namespace Store -> NamespaceStore
Bucket Class -> BucketClass
- use sentence case for form labels
frontend/packages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/review-page.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/review-page.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/review-page.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/review-page.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/review-page.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/review-page.tsx
Outdated
Show resolved
Hide resolved
97c7a0c
to
b64094b
Compare
cdd06ec
to
4d8b552
Compare
7ef8e32
to
cbb2d78
Compare
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/general-page.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/utils/bucket-class.ts
Outdated
Show resolved
Hide resolved
df6f4a3
to
258a430
Compare
frontend/packages/noobaa-storage-plugin/src/constants/providers.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/backingstore-table.tsx
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.scss
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.scss
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.scss
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/namespace-store/create-namespace-store.tsx
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/constants/bucket-class.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/constants/bucket-class.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/constants/bucket-class.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/utils/bucket-class.ts
Outdated
Show resolved
Hide resolved
258a430
to
5edbdbd
Compare
frontend/packages/ceph-storage-plugin/src/utils/bucket-class.ts
Outdated
Show resolved
Hide resolved
b81b108
to
5a7dded
Compare
frontend/packages/ceph-storage-plugin/src/components/bucket-class/wizard-pages/general-page.tsx
Outdated
Show resolved
Hide resolved
...end/packages/ceph-storage-plugin/src/components/namespace-store/namespace-store-dropdown.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/constants/bucket-class.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/constants/bucket-class.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/bucket-class/create-bc.tsx
Show resolved
Hide resolved
Signed-off-by: Vineet Badrinath <vbadrina@redhat.com>
5a7dded
to
31f0d34
Compare
/test ceph-storage-plugin |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bipuladh, vbnrh 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 |
Depends on #8223. Please review the commit for single-namespace store bucket class
Issue Link: https://issues.redhat.com/browse/RHSTOR-1653
Designs: https://marvelapp.com/prototype/fa616je/screen/74336152
How to test
Use OCS image: quay.io/vbnrh/ocs-registry:nooba-namespace-store-2
along with this PR
All screenshots taken on an ultra wide monitor hence may seem stretched out