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

DE2222 and IQN update. #205

Merged
merged 5 commits into from
Jun 14, 2024
Merged

DE2222 and IQN update. #205

merged 5 commits into from
Jun 14, 2024

Conversation

ron-saito
Copy link
Contributor

No description provided.

@ron-saito ron-saito added the bug Something isn't working label Jun 13, 2024
@ron-saito ron-saito requested a review from a team as a code owner June 13, 2024 22:33
@ron-saito ron-saito requested review from dbozzato81 and mchuang3 June 13, 2024 22:34
@@ -508,6 +508,7 @@ func resourceMetalHostCreate(d *schema.ResourceData, meta interface{}) (err erro
Pending: []string{
string(rest.HOSTSTATE_NEW),
string(rest.HOSTSTATE_IMAGING_PREP),
string(rest.HOSTSTATE_ISCSI_ATTACHING),
Copy link
Contributor

@dbozzato81 dbozzato81 Jun 14, 2024

Choose a reason for hiding this comment

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

In line 514 is

string(rest.HOSTSTATE_ATTACHING)

only for FC?

Copy link
Contributor

Choose a reason for hiding this comment

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

In line 514 is

string(rest.HOSTSTATE_ATTACHING)

only for FC?

Yes, sort of, it's for all but iSCSI. For now, it's only FC.

Copy link
Contributor

@mchuang3 mchuang3 left a comment

Choose a reason for hiding this comment

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

Please also add reference to quake PR for the state changes, and US54740 for the IQN update in the PR text for the records.

@@ -508,6 +508,7 @@ func resourceMetalHostCreate(d *schema.ResourceData, meta interface{}) (err erro
Pending: []string{
string(rest.HOSTSTATE_NEW),
string(rest.HOSTSTATE_IMAGING_PREP),
string(rest.HOSTSTATE_ISCSI_ATTACHING),
Copy link
Contributor

Choose a reason for hiding this comment

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

In line 514 is

string(rest.HOSTSTATE_ATTACHING)

only for FC?

Yes, sort of, it's for all but iSCSI. For now, it's only FC.

internal/resources/resource_host.go Show resolved Hide resolved
@ron-saito ron-saito requested a review from mchuang3 June 14, 2024 13:11
@ron-saito ron-saito merged commit 0bd2325 into master Jun 14, 2024
2 checks passed
@ron-saito ron-saito deleted the cytoe/DE22222 branch June 14, 2024 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants