Skip to content

Commit 0b6d3cf

Browse files
Merge pull request #258 from jacobweinstock/implement-bmclib-update
Implement CRD update into bmclib client: ## Description <!--- Please describe what this PR is going to change --> The new CRD fields in the redfish provider options were not implemented in the client instantiation. ## Why is this needed <!--- Link to issue you have raised --> Fixes: # ## How Has This Been Tested? <!--- Please describe in detail how you tested your changes. --> <!--- Include details of your testing environment, and the tests you ran to --> <!--- see how your change affects other areas of the code, etc. --> ## How are existing users impacted? What migration steps/scripts do we need? <!--- Fixes a bug, unblocks installation, removes a component of the stack etc --> <!--- Requires a DB migration script, etc. --> ## Checklist: I have: - [ ] updated the documentation and/or roadmap (if required) - [ ] added unit or e2e tests - [ ] provided instructions on how to upgrade
2 parents adbecd0 + 31610c1 commit 0b6d3cf

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

controller/client.go

+6
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,12 @@ func (b BMCOptions) Translate(host string) []bmclib.Option {
6464
if b.Redfish.Port != 0 {
6565
o = append(o, bmclib.WithRedfishPort(strconv.Itoa(b.Redfish.Port)))
6666
}
67+
if b.Redfish.UseBasicAuth {
68+
o = append(o, bmclib.WithRedfishUseBasicAuth(true))
69+
}
70+
if b.Redfish.SystemName != "" {
71+
o = append(o, bmclib.WithRedfishSystemName(b.Redfish.SystemName))
72+
}
6773
}
6874

6975
// ipmitool options

0 commit comments

Comments
 (0)