Fixed some minor issues

This commit is contained in:
David J. Allen 2023-10-05 13:57:52 -06:00
parent a03a54f35e
commit 4f839a5b61
2 changed files with 4 additions and 3 deletions

View file

@ -30,6 +30,7 @@ var updateCmd = &cobra.Command{
QueryParams: magellan.QueryParams{ QueryParams: magellan.QueryParams{
Drivers: []string{"redfish"}, Drivers: []string{"redfish"},
Preferred: "redfish", Preferred: "redfish",
Protocol: protocol,
Host: host, Host: host,
User: user, User: user,
Pass: pass, Pass: pass,
@ -70,7 +71,7 @@ func init() {
updateCmd.Flags().IntVar(&port, "port", 443, "set the BMC port") updateCmd.Flags().IntVar(&port, "port", 443, "set the BMC port")
updateCmd.Flags().StringVar(&user, "user", "", "set the BMC user") updateCmd.Flags().StringVar(&user, "user", "", "set the BMC user")
updateCmd.Flags().StringVar(&pass, "pass", "", "set the BMC password") updateCmd.Flags().StringVar(&pass, "pass", "", "set the BMC password")
updateCmd.Flags().StringVar(&transferProtocol, "protocol", "HTTP", "set the transfer protocol") updateCmd.Flags().StringVar(&transferProtocol, "transfer-protocol", "HTTP", "set the transfer protocol")
updateCmd.Flags().StringVar(&protocol, "protocol", "https", "set the Redfish protocol") updateCmd.Flags().StringVar(&protocol, "protocol", "https", "set the Redfish protocol")
updateCmd.Flags().StringVar(&firmwarePath, "firmware-path", "", "set the path to the firmware") updateCmd.Flags().StringVar(&firmwarePath, "firmware-path", "", "set the path to the firmware")
updateCmd.Flags().StringVar(&firmwareVersion, "firmware-version", "", "set the version of firmware to be installed") updateCmd.Flags().StringVar(&firmwareVersion, "firmware-version", "", "set the version of firmware to be installed")

View file

@ -126,7 +126,7 @@ func UpdateFirmware(client *bmclib.Client, l *log.Logger, q *UpdateParams) error
} }
func UpdateFirmwareRemote(q *UpdateParams) error { func UpdateFirmwareRemote(q *UpdateParams) error {
url := baseRedfishUrl(&q.QueryParams) + "/UpdateService/Actions/SimpleUpdate" url := baseRedfishUrl(&q.QueryParams) + "/redfish/v1/UpdateService/Actions/SimpleUpdate"
headers := map[string]string { headers := map[string]string {
"Content-Type": "application/json", "Content-Type": "application/json",
"cache-control": "no-cache", "cache-control": "no-cache",
@ -153,7 +153,7 @@ func UpdateFirmwareRemote(q *UpdateParams) error {
} }
func GetUpdateStatus(q *UpdateParams) error { func GetUpdateStatus(q *UpdateParams) error {
url := baseRedfishUrl(&q.QueryParams) + "/UpdateService" url := baseRedfishUrl(&q.QueryParams) + "/redfish/v1/UpdateService"
res, body, err := util.MakeRequest(url, "GET", nil, nil) res, body, err := util.MakeRequest(url, "GET", nil, nil)
if err != nil { if err != nil {
return fmt.Errorf("something went wrong: %v", err) return fmt.Errorf("something went wrong: %v", err)