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
(feat): Allow only object storage deployment #9563
Conversation
frontend/packages/ceph-storage-plugin/src/utils/create-storage-system.ts
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/create-storage-system/payloads.ts
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/create-storage-system/footer.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/src/components/create-storage-system/footer.tsx
Outdated
Show resolved
Hide resolved
.../create-storage-system/create-storage-system-steps/backing-storage-step/advanced-section.tsx
Outdated
Show resolved
Hide resolved
.../create-storage-system/create-storage-system-steps/backing-storage-step/advanced-section.tsx
Outdated
Show resolved
Hide resolved
d5fb834
to
12cc2c3
Compare
.../create-storage-system/create-storage-system-steps/backing-storage-step/advanced-section.tsx
Show resolved
Hide resolved
.../create-storage-system/create-storage-system-steps/backing-storage-step/advanced-section.tsx
Show resolved
Hide resolved
.../create-storage-system/create-storage-system-steps/backing-storage-step/advanced-section.tsx
Outdated
Show resolved
Hide resolved
.../create-storage-system/create-storage-system-steps/backing-storage-step/advanced-section.tsx
Outdated
Show resolved
Hide resolved
.../create-storage-system/create-storage-system-steps/backing-storage-step/advanced-section.tsx
Show resolved
Hide resolved
- added an advanced section which will allow support object storage deployment via MCG - implements https://issues.redhat.com/browse/ODFE-95 Signed-off-by: Afreen Rahman <afrahman@redhat.com>
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.
Looks pretty good .
Just a suggestion to use a Switch
for enabling MCG Only deployment than a checkbox. Will be more appropriate.
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: afreen23, 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 |
I will connect with UXD on that. Thanks for review @vbnrh |
}); | ||
}; | ||
const handleExpanadableToggling: ExpandableSectionProps['onToggle'] = (isExpanded: boolean) => { | ||
dispatch({ type: 'backingStorage/setIsAdvancedOpen', payload: isExpanded }); |
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.
I think it should be !isExpanded
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.
Not really. This is PF expandable section, the handler passes isExpanded
the state of expand - true/false .
If its expanded it will be true or false. We just leveraging this to set state of that in our wizard.
if its opened then it should be opened, even if the user navigates back and next.
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.
ack
Signed-off-by: Afreen Rahman afrahman@redhat.com