Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2020-present Open Networking Foundation |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 17 | //Package swupg provides the utilities for onu sw upgrade |
| 18 | package swupg |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 19 | |
| 20 | import ( |
| 21 | "context" |
| 22 | "encoding/hex" |
| 23 | "fmt" |
| 24 | "sync" |
| 25 | "time" |
| 26 | |
mpagenko | 836a1fd | 2021-11-01 16:12:42 +0000 | [diff] [blame] | 27 | "github.com/opencord/omci-lib-go/v2" |
| 28 | me "github.com/opencord/omci-lib-go/v2/generated" |
khenaidoo | 7d3c558 | 2021-08-11 18:09:44 -0400 | [diff] [blame] | 29 | "github.com/opencord/voltha-lib-go/v7/pkg/log" |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 30 | cmn "github.com/opencord/voltha-openonu-adapter-go/internal/pkg/common" |
khenaidoo | 7d3c558 | 2021-08-11 18:09:44 -0400 | [diff] [blame] | 31 | "github.com/opencord/voltha-protos/v5/go/voltha" |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 32 | ) |
| 33 | |
| 34 | //OnuImageStatus implements methods to get status info of onu images |
| 35 | type OnuImageStatus struct { |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 36 | deviceID string |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 37 | pDeviceHandler cmn.IdeviceHandler |
| 38 | pDevEntry cmn.IonuDeviceEntry |
| 39 | pOmciCC *cmn.OmciCC |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 40 | requestedAttributes me.AttributeValueMap |
| 41 | mutexWaitingForResp sync.RWMutex |
| 42 | waitingForResp bool |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 43 | respChannel chan cmn.Message |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 44 | mutexPLastTxMeInstance sync.RWMutex |
| 45 | pLastTxMeInstance *me.ManagedEntity |
Holger Hildebrandt | d930cb2 | 2022-06-17 09:24:50 +0000 | [diff] [blame] | 46 | isExtendedOmci bool |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 47 | } |
| 48 | |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 49 | const cResponse = "response: " |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 50 | |
| 51 | //NewOnuImageStatus creates a new instance of OnuImageStatus |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 52 | func NewOnuImageStatus(apDeviceHandler cmn.IdeviceHandler, apDevEntry cmn.IonuDeviceEntry) *OnuImageStatus { |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 53 | return &OnuImageStatus{ |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 54 | deviceID: apDeviceHandler.GetDeviceID(), |
| 55 | pDeviceHandler: apDeviceHandler, |
| 56 | pDevEntry: apDevEntry, |
| 57 | pOmciCC: apDevEntry.GetDevOmciCC(), |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 58 | requestedAttributes: make(me.AttributeValueMap), |
| 59 | waitingForResp: false, |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 60 | respChannel: make(chan cmn.Message), |
Holger Hildebrandt | d930cb2 | 2022-06-17 09:24:50 +0000 | [diff] [blame] | 61 | isExtendedOmci: apDevEntry.GetPersIsExtOmciSupported(), |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 62 | } |
| 63 | } |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 64 | |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 65 | // GetOnuImageStatus - TODO: add comment |
| 66 | func (oo *OnuImageStatus) GetOnuImageStatus(ctx context.Context) (*voltha.OnuImages, error) { |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 67 | |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 68 | if !oo.pDeviceHandler.IsReadyForOmciConfig() { |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 69 | logger.Errorw(ctx, "command rejected - improper device state", log.Fields{"device-id": oo.deviceID}) |
| 70 | return nil, fmt.Errorf("command-rejected-improper-device-state") |
| 71 | } |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 72 | if oo.pOmciCC == nil { |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 73 | logger.Errorw(ctx, "omciCC not ready to receive omci messages", log.Fields{"device-id": oo.deviceID}) |
| 74 | return nil, fmt.Errorf("omciCC-not-ready-to-receive-omci-messages") |
| 75 | } |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 76 | var images voltha.OnuImages |
| 77 | |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 78 | for i := cmn.FirstSwImageMeID; i <= cmn.SecondSwImageMeID; i++ { |
| 79 | logger.Debugw(ctx, "GetOnuImageStatus for image id", log.Fields{"image-id": i, "device-id": oo.deviceID}) |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 80 | |
| 81 | var image voltha.OnuImage |
| 82 | |
| 83 | // TODO: Since the summed length of the attributes exceeds the capacity of a single response, |
| 84 | // it is distributed on several requests here. It should be discussed whether, in the course of a refactoring, |
| 85 | // a global mechanism should be implemented that automates this distribution - which would entail quite some |
| 86 | // changes on the respective receiver sides. |
| 87 | |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 88 | oo.requestedAttributes = me.AttributeValueMap{me.SoftwareImage_Version: "", me.SoftwareImage_IsCommitted: 0, me.SoftwareImage_IsActive: 0, me.SoftwareImage_IsValid: 0} |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 89 | if err := oo.requestOnuImageAttributes(ctx, uint16(i), &image); err != nil { |
| 90 | logger.Errorw(ctx, err.Error(), log.Fields{"requestedAttributes": oo.requestedAttributes, "device-id": oo.deviceID}) |
| 91 | return nil, err |
| 92 | } |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 93 | oo.requestedAttributes = me.AttributeValueMap{me.SoftwareImage_ProductCode: ""} |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 94 | if err := oo.requestOnuImageAttributes(ctx, uint16(i), &image); err != nil { |
| 95 | logger.Errorw(ctx, err.Error(), log.Fields{"requestedAttributes": oo.requestedAttributes, "device-id": oo.deviceID}) |
| 96 | return nil, err |
| 97 | } |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 98 | oo.requestedAttributes = me.AttributeValueMap{me.SoftwareImage_ImageHash: 0} |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 99 | if err := oo.requestOnuImageAttributes(ctx, uint16(i), &image); err != nil { |
| 100 | logger.Errorw(ctx, err.Error(), log.Fields{"requestedAttributes": oo.requestedAttributes, "device-id": oo.deviceID}) |
| 101 | return nil, err |
| 102 | } |
| 103 | images.Items = append(images.Items, &image) |
| 104 | } |
| 105 | logger.Debugw(ctx, "images of the ONU", log.Fields{"images": images}) |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 106 | oo.updateOnuSwImagePersistentData(ctx) |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 107 | return &images, nil |
| 108 | } |
| 109 | |
| 110 | func (oo *OnuImageStatus) requestOnuImageAttributes(ctx context.Context, imageID uint16, image *voltha.OnuImage) error { |
| 111 | oo.mutexPLastTxMeInstance.Lock() |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 112 | meInstance, err := oo.pOmciCC.SendGetMe(log.WithSpanFromContext(context.TODO(), ctx), me.SoftwareImageClassID, |
Holger Hildebrandt | d930cb2 | 2022-06-17 09:24:50 +0000 | [diff] [blame] | 113 | imageID, oo.requestedAttributes, oo.pDeviceHandler.GetOmciTimeout(), true, oo.respChannel, oo.isExtendedOmci) |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 114 | if err != nil { |
| 115 | oo.mutexPLastTxMeInstance.Unlock() |
| 116 | logger.Errorw(ctx, "can't send omci request to get data for image id", log.Fields{"image-id": imageID, "device-id": oo.deviceID}) |
| 117 | return fmt.Errorf("can't-send-omci-request-to-get-data-for-image-id-%d", imageID) |
| 118 | } |
| 119 | oo.pLastTxMeInstance = meInstance |
| 120 | oo.mutexPLastTxMeInstance.Unlock() |
| 121 | |
| 122 | if err = oo.waitForGetOnuImageStatus(ctx, image); err != nil { |
| 123 | logger.Errorw(ctx, err.Error(), log.Fields{"device-id": oo.deviceID}) |
| 124 | return err |
| 125 | } |
| 126 | return nil |
| 127 | } |
| 128 | |
| 129 | func (oo *OnuImageStatus) waitForGetOnuImageStatus(ctx context.Context, image *voltha.OnuImage) error { |
| 130 | oo.setWaitingForResp(true) |
| 131 | select { |
| 132 | // maybe be also some outside cancel (but no context modeled for the moment ...) |
| 133 | case <-ctx.Done(): |
| 134 | logger.Errorw(ctx, "waitForGetOnuImageStatus context done", log.Fields{"device-id": oo.deviceID}) |
| 135 | oo.setWaitingForResp(false) |
| 136 | return fmt.Errorf("wait-for-image-status-context-done") |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 137 | case <-time.After(oo.pOmciCC.GetMaxOmciTimeoutWithRetries() * time.Second): |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 138 | logger.Errorw(ctx, "waitForGetOnuImageStatus timeout", log.Fields{"device-id": oo.deviceID}) |
| 139 | oo.setWaitingForResp(false) |
| 140 | return fmt.Errorf("wait-for-image-status-timeout") |
| 141 | case message, ok := <-oo.respChannel: |
| 142 | if !ok { |
| 143 | logger.Errorw(ctx, "waitForGetOnuImageStatus response error", log.Fields{"device-id": oo.deviceID}) |
| 144 | oo.setWaitingForResp(false) |
| 145 | return fmt.Errorf("wait-for-image-status-response-error") |
| 146 | } |
| 147 | switch message.Type { |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 148 | case cmn.OMCI: |
| 149 | msg, _ := message.Data.(cmn.OmciMessage) |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 150 | oo.setWaitingForResp(false) |
| 151 | return oo.processGetOnuImageStatusResp(ctx, msg, image) |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 152 | case cmn.TestMsg: |
| 153 | msg, _ := message.Data.(cmn.TestMessage) |
| 154 | if msg.TestMessageVal == cmn.AbortMessageProcessing { |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 155 | logger.Info(ctx, "waitForGetOnuImageStatus abort msg received", log.Fields{"device-id": oo.deviceID}) |
| 156 | oo.setWaitingForResp(false) |
| 157 | return fmt.Errorf("wait-for-image-status-abort-msg-received") |
| 158 | } |
| 159 | default: |
| 160 | logger.Errorw(ctx, "waitForGetOnuImageStatus wrong msg type received", log.Fields{"msgType": message.Type, "device-id": oo.deviceID}) |
| 161 | oo.setWaitingForResp(false) |
| 162 | return fmt.Errorf("wait-for-image-status-response-error") |
| 163 | } |
| 164 | } |
| 165 | logger.Errorw(ctx, "waitForGetOnuImageStatus processing error", log.Fields{"device-id": oo.deviceID}) |
| 166 | oo.setWaitingForResp(false) |
| 167 | return fmt.Errorf("wait-for-image-status-processing-error") |
| 168 | |
| 169 | } |
| 170 | |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 171 | func (oo *OnuImageStatus) processGetOnuImageStatusResp(ctx context.Context, msg cmn.OmciMessage, image *voltha.OnuImage) error { |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 172 | if msg.OmciMsg.MessageType != omci.GetResponseType { |
| 173 | logger.Errorw(ctx, "processGetOnuImageStatusResp wrong response type received", log.Fields{"respType": msg.OmciMsg.MessageType, "device-id": oo.deviceID}) |
| 174 | return fmt.Errorf("process-image-status-response-error") |
| 175 | } |
| 176 | msgLayer := (*msg.OmciPacket).Layer(omci.LayerTypeGetResponse) |
| 177 | if msgLayer == nil { |
| 178 | logger.Errorw(ctx, "processGetOnuImageStatusResp omci Msg layer not found", log.Fields{"device-id": oo.deviceID}) |
| 179 | return fmt.Errorf("process-image-status-response-error") |
| 180 | } |
| 181 | msgObj, msgOk := msgLayer.(*omci.GetResponse) |
| 182 | if !msgOk { |
| 183 | logger.Errorw(ctx, "processGetOnuImageStatusResp omci msgObj layer could not be found", log.Fields{"device-id": oo.deviceID}) |
| 184 | return fmt.Errorf("process-image-status-response-error") |
| 185 | } |
| 186 | oo.mutexPLastTxMeInstance.RLock() |
| 187 | if oo.pLastTxMeInstance != nil { |
| 188 | if msgObj.EntityClass == oo.pLastTxMeInstance.GetClassID() && |
| 189 | msgObj.EntityInstance == oo.pLastTxMeInstance.GetEntityID() { |
| 190 | oo.mutexPLastTxMeInstance.RUnlock() |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 191 | if err := oo.processAttributesReceived(ctx, msgObj, image); err != nil { |
| 192 | logger.Errorw(ctx, err.Error(), log.Fields{"device-id": oo.deviceID}) |
| 193 | return err |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 194 | } |
| 195 | return nil |
| 196 | } |
| 197 | oo.mutexPLastTxMeInstance.RUnlock() |
| 198 | logger.Errorw(ctx, "processGetOnuImageStatusResp wrong MeInstance received", log.Fields{"device-id": oo.deviceID}) |
| 199 | return fmt.Errorf("process-image-status-response-error") |
| 200 | } |
| 201 | oo.mutexPLastTxMeInstance.RUnlock() |
| 202 | logger.Errorw(ctx, "processGetOnuImageStatusResp pLastTxMeInstance is nil", log.Fields{"device-id": oo.deviceID}) |
| 203 | return fmt.Errorf("process-image-status-response-error") |
| 204 | } |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 205 | |
| 206 | func (oo *OnuImageStatus) processAttributesReceived(ctx context.Context, msgObj *omci.GetResponse, image *voltha.OnuImage) error { |
| 207 | meAttributes := msgObj.Attributes |
| 208 | logger.Debugw(ctx, "processAttributesReceived", log.Fields{"attributes": meAttributes, "device-id": oo.deviceID}) |
| 209 | |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 210 | if _, ok := oo.requestedAttributes[me.SoftwareImage_Version]; ok { |
Holger Hildebrandt | 94688c7 | 2021-12-17 12:13:02 +0000 | [diff] [blame] | 211 | if msgObj.Result != me.Success { |
Holger Hildebrandt | fdb4bba | 2022-03-10 12:12:59 +0000 | [diff] [blame] | 212 | logger.Errorw(ctx, "processAttributesReceived - retrieval of mandatory attributes not successful", |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 213 | log.Fields{"device-id": oo.deviceID}) |
Holger Hildebrandt | fdb4bba | 2022-03-10 12:12:59 +0000 | [diff] [blame] | 214 | return fmt.Errorf("retrieve-mandatory-attributes-not-successful") |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 215 | } |
Holger Hildebrandt | fdb4bba | 2022-03-10 12:12:59 +0000 | [diff] [blame] | 216 | if !oo.pDevEntry.HandleSwImageIndications(ctx, msgObj.EntityInstance, meAttributes) { |
| 217 | logger.Errorw(ctx, "processAttributesReceived - not all mandatory attributes present in SoftwareImage instance", log.Fields{"device-id": oo.deviceID}) |
| 218 | return fmt.Errorf("not-all-mandatory-attributes-present") |
| 219 | } |
Holger Hildebrandt | 94688c7 | 2021-12-17 12:13:02 +0000 | [diff] [blame] | 220 | } |
| 221 | for k := range oo.requestedAttributes { |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 222 | switch k { |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 223 | // mandatory attributes |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 224 | case me.SoftwareImage_IsCommitted: |
| 225 | if meAttributes[me.SoftwareImage_IsCommitted].(uint8) == cmn.SwIsCommitted { |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 226 | image.IsCommited = true |
| 227 | } else { |
| 228 | image.IsCommited = false |
| 229 | } |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 230 | case me.SoftwareImage_IsActive: |
| 231 | if meAttributes[me.SoftwareImage_IsActive].(uint8) == cmn.SwIsActive { |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 232 | image.IsActive = true |
| 233 | } else { |
| 234 | image.IsActive = false |
| 235 | } |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 236 | case me.SoftwareImage_IsValid: |
| 237 | if meAttributes[me.SoftwareImage_IsValid].(uint8) == cmn.SwIsValid { |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 238 | image.IsValid = true |
| 239 | } else { |
| 240 | image.IsValid = false |
| 241 | } |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 242 | case me.SoftwareImage_Version: |
| 243 | image.Version = cmn.TrimStringFromMeOctet(meAttributes[me.SoftwareImage_Version]) |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 244 | |
| 245 | // optional attributes |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 246 | case me.SoftwareImage_ProductCode: |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 247 | if msgObj.Result == me.Success { |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 248 | image.ProductCode = cmn.TrimStringFromMeOctet(meAttributes[me.SoftwareImage_ProductCode]) |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 249 | } else { |
| 250 | sResult := msgObj.Result.String() |
| 251 | logger.Infow(ctx, "processAttributesReceived - ProductCode", |
| 252 | log.Fields{"result": sResult, "unsupported attribute mask": msgObj.UnsupportedAttributeMask, "device-id": oo.deviceID}) |
| 253 | image.ProductCode = cResponse + sResult |
| 254 | } |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 255 | case me.SoftwareImage_ImageHash: |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 256 | if msgObj.Result == me.Success { |
Holger Hildebrandt | 3ac49bd | 2022-02-07 17:46:43 +0000 | [diff] [blame] | 257 | bytes, _ := me.InterfaceToOctets(meAttributes[me.SoftwareImage_ImageHash]) |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 258 | image.Hash = hex.EncodeToString(bytes) |
| 259 | } else { |
| 260 | sResult := msgObj.Result.String() |
| 261 | logger.Infow(ctx, "processAttributesReceived - ImageHash", |
| 262 | log.Fields{"result": sResult, "unsupported attribute mask": msgObj.UnsupportedAttributeMask, "device-id": oo.deviceID}) |
| 263 | image.Hash = cResponse + sResult |
| 264 | } |
| 265 | } |
| 266 | } |
| 267 | return nil |
| 268 | } |
| 269 | |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 270 | func (oo *OnuImageStatus) updateOnuSwImagePersistentData(ctx context.Context) { |
| 271 | |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 272 | activeImageVersion := oo.pDevEntry.GetActiveImageVersion(ctx) |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 273 | persActiveSwVersion := oo.pDevEntry.GetPersActiveSwVersion() |
| 274 | if persActiveSwVersion != activeImageVersion { |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 275 | logger.Infow(ctx, "Active SW version has been changed at ONU - update persistent data", |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 276 | log.Fields{"old version": persActiveSwVersion, |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 277 | "new version": activeImageVersion, "device-id": oo.deviceID}) |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 278 | oo.pDevEntry.SetPersActiveSwVersion(activeImageVersion) |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 279 | if err := oo.pDeviceHandler.StorePersistentData(ctx); err != nil { |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 280 | logger.Warnw(ctx, "store persistent data error - continue for now as there will be additional write attempts", |
| 281 | log.Fields{"device-id": oo.deviceID, "err": err}) |
| 282 | } |
| 283 | return |
| 284 | } |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 285 | } |
| 286 | |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 287 | func (oo *OnuImageStatus) setWaitingForResp(value bool) { |
| 288 | oo.mutexWaitingForResp.Lock() |
| 289 | oo.waitingForResp = value |
| 290 | oo.mutexWaitingForResp.Unlock() |
| 291 | } |
Holger Hildebrandt | 0501135 | 2021-06-15 09:40:24 +0000 | [diff] [blame] | 292 | |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 293 | func (oo *OnuImageStatus) isWaitingForResp() bool { |
| 294 | oo.mutexWaitingForResp.RLock() |
| 295 | value := oo.waitingForResp |
| 296 | oo.mutexWaitingForResp.RUnlock() |
| 297 | return value |
| 298 | } |
| 299 | |
| 300 | //CancelProcessing ensures that interrupted processing is canceled while waiting for a response |
| 301 | func (oo *OnuImageStatus) CancelProcessing(ctx context.Context) { |
Holger Hildebrandt | 12609a1 | 2022-03-25 13:23:25 +0000 | [diff] [blame] | 302 | logger.Debugw(ctx, "CancelProcessing entered", log.Fields{"device-id": oo.deviceID}) |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 303 | if oo.isWaitingForResp() { |
Holger Hildebrandt | 4b5e73f | 2021-08-19 06:51:21 +0000 | [diff] [blame] | 304 | abortMsg := cmn.Message{ |
| 305 | Type: cmn.TestMsg, |
| 306 | Data: cmn.TestMessage{ |
| 307 | TestMessageVal: cmn.AbortMessageProcessing, |
Holger Hildebrandt | fb402a6 | 2021-05-26 14:40:49 +0000 | [diff] [blame] | 308 | }, |
| 309 | } |
| 310 | oo.respChannel <- abortMsg |
| 311 | } |
| 312 | } |