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
feature: add flashsystem external storage #9602
feature: add flashsystem external storage #9602
Conversation
Hi @shdn-ibm. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/assign |
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/models.ts
Outdated
Show resolved
Hide resolved
/ok=to-test |
.../packages/ceph-storage-plugin/src/components/create-storage-system/external-storage/index.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.
Thanks for PR. Please run yarn i18n
under the /frontend
dir.
This will create json for i18n
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/models.ts
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
/ok-to-test |
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
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.
tests are failing due to lint errors
Please fix them.
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/locales/ko/ceph-storage-plugin.json
Outdated
Show resolved
Hide resolved
frontend/packages/ceph-storage-plugin/locales/ja/ceph-storage-plugin.json
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.
Please squash commits into a single commit
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
...storage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/type.ts
Outdated
Show resolved
Hide resolved
...storage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/type.ts
Outdated
Show resolved
Hide resolved
...storage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/type.ts
Outdated
Show resolved
Hide resolved
...storage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/type.ts
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/models.ts
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
label: 'IBM Flash System', | ||
labelPlural: 'IBM Flash Systems', |
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.
In kind it says flashsystemcluster but in label it is flash system.
Is that a miss ? "IBM Flash System Cluster" ??
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.
Need to be resolved later. Not blocking PR
fix: lint error fix: lint check error fix: lint error fix: rename management address in secret fix: lint error fix: lint error
9e1e2b7
to
650451a
Compare
...orage-plugin/src/components/create-storage-system/external-storage/ibm-flashsystem/index.tsx
Outdated
Show resolved
Hide resolved
2595ef4
to
24967ca
Compare
fix: update translation file fix: lint error
24967ca
to
06c5084
Compare
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: afreen23, shdn-ibm 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 |
Add flashsystem as an exnternal storage to support ODF.
Snap of the page is shown as below