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

[fix]getClusterPool: performing a type conversion on a nil. #309

Merged
merged 2 commits into from
Nov 22, 2023

Conversation

Songjf-ttk
Copy link
Contributor

@Songjf-ttk Songjf-ttk commented Sep 3, 2023

related: #308
fix curveadm is performing a type conversion on a nil.
Solution: Add poolset and diskType to the option choices. So, there won't be a nil.

fix curveadm is performing a type conversion on a nil.
Solution: Add poolset and diskType to the option choices.
Signed-off-by: sjf <[email protected]>
@Songjf-ttk Songjf-ttk changed the title [WIP][fix]getClusterPool: performing a type conversion on a nil. [fix]getClusterPool: performing a type conversion on a nil. Sep 18, 2023
@Songjf-ttk
Copy link
Contributor Author

@Wine93 @caoxianfei1 Can you help review it

@Songjf-ttk Songjf-ttk closed this Nov 12, 2023
@Wine93 Wine93 reopened this Nov 12, 2023
@Wine93
Copy link
Collaborator

Wine93 commented Nov 12, 2023

Hi, @Songjf-ttk , thank you for your contribute, we will review it, please don't close it :)

@Wine93 Wine93 self-requested a review November 12, 2023 15:21
@Songjf-ttk
Copy link
Contributor Author

Hi, @Songjf-ttk , thank you for your contribute, we will review it, please don't close it :)

Thank you, I will fix the CI errors and continue to follow up.

@caoxianfei1
Copy link
Contributor

caoxianfei1 commented Nov 21, 2023

@Songjf-ttk CI failed, please solve this problem. May be some wrong blank or tab.

Signed-off-by: caoxianfei1 <[email protected]>
@caoxianfei1 caoxianfei1 merged commit 8b2c6d8 into opencurve:develop Nov 22, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants