Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2018-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 | |
| 17 | package device |
| 18 | |
| 19 | import ( |
| 20 | "context" |
ssiddiqui | f076cb8 | 2021-04-23 10:47:04 +0530 | [diff] [blame] | 21 | |
yasin sapli | 5458a1c | 2021-06-14 22:24:38 +0000 | [diff] [blame] | 22 | "github.com/opencord/voltha-lib-go/v5/pkg/kafka" |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 23 | "github.com/opencord/voltha-protos/v4/go/common" |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 24 | |
| 25 | "github.com/gogo/protobuf/proto" |
| 26 | "github.com/golang/protobuf/ptypes" |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 27 | coreutils "github.com/opencord/voltha-go/rw_core/utils" |
yasin sapli | 5458a1c | 2021-06-14 22:24:38 +0000 | [diff] [blame] | 28 | "github.com/opencord/voltha-lib-go/v5/pkg/log" |
Maninder | dfadc98 | 2020-10-28 14:04:33 +0530 | [diff] [blame] | 29 | "github.com/opencord/voltha-protos/v4/go/voltha" |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 30 | "google.golang.org/grpc/codes" |
| 31 | "google.golang.org/grpc/status" |
| 32 | ) |
| 33 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 34 | func (agent *Agent) downloadImage(ctx context.Context, img *voltha.ImageDownload) (*common.OperationResp, error) { |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 35 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 36 | return nil, err |
| 37 | } |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 38 | logger.Debugw(ctx, "download-image", log.Fields{"device-id": agent.deviceID}) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 39 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 40 | if agent.device.Root { |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 41 | agent.requestQueue.RequestComplete() |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 42 | return nil, status.Errorf(codes.FailedPrecondition, "device-id:%s, is an OLT. Image update "+ |
| 43 | "not supported by VOLTHA. Use Device Manager or other means", agent.deviceID) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 44 | } |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 45 | |
| 46 | device := agent.cloneDeviceWithoutLock() |
| 47 | |
| 48 | if !agent.proceedWithRequest(device) { |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 49 | agent.requestQueue.RequestComplete() |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 50 | return nil, status.Errorf(codes.FailedPrecondition, "deviceId:%s, Cannot complete operation as device deletion is in progress or reconciling is in progress/failed.", |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 51 | agent.deviceID) |
| 52 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 53 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 54 | if device.ImageDownloads != nil { |
| 55 | for _, image := range device.ImageDownloads { |
| 56 | if image.DownloadState == voltha.ImageDownload_DOWNLOAD_REQUESTED { |
| 57 | return nil, status.Errorf(codes.FailedPrecondition, "device-id:%s, already downloading image:%s", |
| 58 | agent.deviceID, image.Name) |
| 59 | } |
| 60 | } |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 61 | } |
| 62 | |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 63 | // Save the image |
| 64 | clonedImg := proto.Clone(img).(*voltha.ImageDownload) |
| 65 | clonedImg.DownloadState = voltha.ImageDownload_DOWNLOAD_REQUESTED |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 66 | cloned := agent.cloneDeviceWithoutLock() |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 67 | _, index, err := getImage(img, device) |
| 68 | if err != nil { |
Andrey Pozolotin | 34dd63f | 2021-05-31 21:26:40 +0300 | [diff] [blame] | 69 | cloned.ImageDownloads = append(cloned.ImageDownloads, clonedImg) |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 70 | } else { |
| 71 | cloned.ImageDownloads[index] = clonedImg |
| 72 | } |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 73 | |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 74 | cloned.AdminState = voltha.AdminState_DOWNLOADING_IMAGE |
| 75 | if err := agent.updateDeviceAndReleaseLock(ctx, cloned); err != nil { |
| 76 | return nil, err |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 77 | } |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 78 | |
| 79 | // Send the request to the adapter |
Rohan Agrawal | cf12f20 | 2020-08-03 04:42:01 +0000 | [diff] [blame] | 80 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 81 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
| 82 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 83 | ch, err := agent.adapterProxy.DownloadImage(subCtx, cloned, clonedImg) |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 84 | if err != nil { |
| 85 | cancel() |
| 86 | return nil, err |
| 87 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 88 | go agent.waitForAdapterResponse(subCtx, cancel, "downloadImage", ch, agent.onImageSuccess, agent.onImageFailure) |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 89 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 90 | return &common.OperationResp{Code: voltha.OperationResp_OPERATION_SUCCESS}, nil |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 91 | } |
| 92 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 93 | // getImage is a helper method to figure out if an image is already registered |
| 94 | func getImage(img *voltha.ImageDownload, device *voltha.Device) (*voltha.ImageDownload, int, error) { |
| 95 | for pos, image := range device.ImageDownloads { |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 96 | if image.Id == img.Id && image.Name == img.Name { |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 97 | return image, pos, nil |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 98 | } |
| 99 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 100 | return nil, -1, status.Errorf(codes.FailedPrecondition, "device-id:%s, image-not-registered:%s", |
| 101 | device.Id, img.Name) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 102 | } |
| 103 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 104 | func (agent *Agent) cancelImageDownload(ctx context.Context, img *voltha.ImageDownload) (*common.OperationResp, error) { |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 105 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 106 | return nil, err |
| 107 | } |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 108 | logger.Debugw(ctx, "cancel-image-download", log.Fields{"device-id": agent.deviceID}) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 109 | |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 110 | cloned := agent.cloneDeviceWithoutLock() |
| 111 | |
| 112 | if !agent.proceedWithRequest(cloned) { |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 113 | agent.requestQueue.RequestComplete() |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 114 | return nil, status.Errorf(codes.FailedPrecondition, "deviceId:%s, Cannot complete operation as device deletion is in progress or reconciling is in progress/failed.", |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 115 | agent.deviceID) |
| 116 | } |
| 117 | |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 118 | // Update image download state |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 119 | _, index, err := getImage(img, cloned) |
| 120 | if err != nil { |
| 121 | agent.requestQueue.RequestComplete() |
| 122 | return nil, err |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 123 | } |
| 124 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 125 | cloned.ImageDownloads[index].DownloadState = voltha.ImageDownload_DOWNLOAD_CANCELLED |
| 126 | |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 127 | if cloned.AdminState != voltha.AdminState_DOWNLOADING_IMAGE { |
| 128 | agent.requestQueue.RequestComplete() |
| 129 | } else { |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 130 | // Set the device to Enabled |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 131 | cloned.AdminState = voltha.AdminState_ENABLED |
| 132 | if err := agent.updateDeviceAndReleaseLock(ctx, cloned); err != nil { |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 133 | return nil, err |
| 134 | } |
Rohan Agrawal | cf12f20 | 2020-08-03 04:42:01 +0000 | [diff] [blame] | 135 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 136 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
| 137 | |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 138 | ch, err := agent.adapterProxy.CancelImageDownload(subCtx, cloned, img) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 139 | if err != nil { |
| 140 | cancel() |
| 141 | return nil, err |
| 142 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 143 | go agent.waitForAdapterResponse(subCtx, cancel, "cancelImageDownload", ch, agent.onImageSuccess, |
| 144 | agent.onImageFailure) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 145 | } |
| 146 | return &voltha.OperationResp{Code: voltha.OperationResp_OPERATION_SUCCESS}, nil |
| 147 | } |
| 148 | |
| 149 | func (agent *Agent) activateImage(ctx context.Context, img *voltha.ImageDownload) (*voltha.OperationResp, error) { |
| 150 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 151 | return nil, err |
| 152 | } |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 153 | logger.Debugw(ctx, "activate-image", log.Fields{"device-id": agent.deviceID}) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 154 | |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 155 | cloned := agent.cloneDeviceWithoutLock() |
| 156 | |
| 157 | if !agent.proceedWithRequest(cloned) { |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 158 | agent.requestQueue.RequestComplete() |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 159 | return nil, status.Errorf(codes.FailedPrecondition, "deviceId:%s, Cannot complete operation as device deletion is in progress or reconcile is in progress/failed.", |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 160 | agent.deviceID) |
| 161 | } |
| 162 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 163 | // Update image download state |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 164 | image, index, err := getImage(img, cloned) |
| 165 | if err != nil { |
| 166 | agent.requestQueue.RequestComplete() |
| 167 | return nil, err |
| 168 | } |
| 169 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 170 | if image.DownloadState != voltha.ImageDownload_DOWNLOAD_SUCCEEDED { |
| 171 | agent.requestQueue.RequestComplete() |
| 172 | return nil, status.Errorf(codes.FailedPrecondition, "device-id:%s, device-has-not-downloaded-image:%s", agent.deviceID, img.Name) |
| 173 | } |
| 174 | |
| 175 | //TODO does this need to be removed ? |
| 176 | if cloned.AdminState == voltha.AdminState_DOWNLOADING_IMAGE { |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 177 | agent.requestQueue.RequestComplete() |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 178 | return nil, status.Errorf(codes.FailedPrecondition, "device-id:%s, device-in-downloading-state:%s", agent.deviceID, img.Name) |
| 179 | } |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 180 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 181 | // Save the image |
| 182 | cloned.ImageDownloads[index].ImageState = voltha.ImageDownload_IMAGE_ACTIVATING |
| 183 | |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 184 | cloned.AdminState = voltha.AdminState_DOWNLOADING_IMAGE |
| 185 | if err := agent.updateDeviceAndReleaseLock(ctx, cloned); err != nil { |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 186 | return nil, err |
| 187 | } |
| 188 | |
Rohan Agrawal | cf12f20 | 2020-08-03 04:42:01 +0000 | [diff] [blame] | 189 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 190 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
| 191 | |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 192 | ch, err := agent.adapterProxy.ActivateImageUpdate(subCtx, cloned, img) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 193 | if err != nil { |
| 194 | cancel() |
| 195 | return nil, err |
| 196 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 197 | go agent.waitForAdapterResponse(subCtx, cancel, "activateImageUpdate", ch, agent.onImageSuccess, agent.onFailure) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 198 | |
| 199 | // The status of the AdminState will be changed following the update_download_status response from the adapter |
| 200 | // The image name will also be removed from the device list |
| 201 | return &voltha.OperationResp{Code: voltha.OperationResp_OPERATION_SUCCESS}, nil |
| 202 | } |
| 203 | |
| 204 | func (agent *Agent) revertImage(ctx context.Context, img *voltha.ImageDownload) (*voltha.OperationResp, error) { |
| 205 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 206 | return nil, err |
| 207 | } |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 208 | logger.Debugw(ctx, "revert-image", log.Fields{"device-id": agent.deviceID}) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 209 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 210 | // Update image download state |
| 211 | cloned := agent.cloneDeviceWithoutLock() |
| 212 | _, index, err := getImage(img, cloned) |
| 213 | if err != nil { |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 214 | agent.requestQueue.RequestComplete() |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 215 | return nil, status.Errorf(codes.FailedPrecondition, "deviceId:%s, image-not-registered:%s", agent.deviceID, img.Name) |
| 216 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 217 | if cloned.AdminState != voltha.AdminState_ENABLED { |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 218 | agent.requestQueue.RequestComplete() |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 219 | return nil, status.Errorf(codes.FailedPrecondition, "deviceId:%s, device-not-enabled-state:%s", agent.deviceID, img.Name) |
| 220 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 221 | |
| 222 | cloned.ImageDownloads[index].ImageState = voltha.ImageDownload_IMAGE_REVERTING |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 223 | |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 224 | if err := agent.updateDeviceAndReleaseLock(ctx, cloned); err != nil { |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 225 | return nil, err |
| 226 | } |
| 227 | |
Rohan Agrawal | cf12f20 | 2020-08-03 04:42:01 +0000 | [diff] [blame] | 228 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 229 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
| 230 | |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 231 | ch, err := agent.adapterProxy.RevertImageUpdate(subCtx, cloned, img) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 232 | if err != nil { |
| 233 | cancel() |
| 234 | return nil, err |
| 235 | } |
| 236 | go agent.waitForAdapterResponse(subCtx, cancel, "revertImageUpdate", ch, agent.onSuccess, agent.onFailure) |
| 237 | |
| 238 | return &voltha.OperationResp{Code: voltha.OperationResp_OPERATION_SUCCESS}, nil |
| 239 | } |
| 240 | |
| 241 | func (agent *Agent) getImageDownloadStatus(ctx context.Context, img *voltha.ImageDownload) (*voltha.ImageDownload, error) { |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 242 | logger.Debugw(ctx, "get-image-download-status", log.Fields{"device-id": agent.deviceID}) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 243 | |
| 244 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 245 | return nil, err |
| 246 | } |
Kent Hagerman | cba2f30 | 2020-07-28 13:37:36 -0400 | [diff] [blame] | 247 | device := agent.getDeviceReadOnlyWithoutLock() |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 248 | ch, err := agent.adapterProxy.GetImageDownloadStatus(ctx, device, img) |
| 249 | agent.requestQueue.RequestComplete() |
| 250 | if err != nil { |
| 251 | return nil, err |
| 252 | } |
| 253 | // Wait for the adapter response |
| 254 | rpcResponse, ok := <-ch |
| 255 | if !ok { |
| 256 | return nil, status.Errorf(codes.Aborted, "channel-closed-device-id-%s", agent.deviceID) |
| 257 | } |
| 258 | if rpcResponse.Err != nil { |
| 259 | return nil, rpcResponse.Err |
| 260 | } |
| 261 | // Successful response |
| 262 | imgDownload := &voltha.ImageDownload{} |
| 263 | if err := ptypes.UnmarshalAny(rpcResponse.Reply, imgDownload); err != nil { |
| 264 | return nil, status.Errorf(codes.InvalidArgument, "%s", err.Error()) |
| 265 | } |
| 266 | return imgDownload, nil |
| 267 | } |
| 268 | |
| 269 | func (agent *Agent) updateImageDownload(ctx context.Context, img *voltha.ImageDownload) error { |
| 270 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 271 | return err |
| 272 | } |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 273 | logger.Debugw(ctx, "updating-image-download", log.Fields{"device-id": agent.deviceID, "img": img}) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 274 | |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 275 | cloned := agent.cloneDeviceWithoutLock() |
| 276 | |
| 277 | if !agent.proceedWithRequest(cloned) { |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 278 | agent.requestQueue.RequestComplete() |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 279 | return status.Errorf(codes.FailedPrecondition, "deviceId:%s, Cannot complete operation as device deletion is in progress or reconciling is in progress/failed.", |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 280 | agent.deviceID) |
| 281 | } |
| 282 | |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 283 | // Update the image as well as remove it if the download was cancelled |
| 284 | clonedImages := make([]*voltha.ImageDownload, len(cloned.ImageDownloads)) |
| 285 | for _, image := range cloned.ImageDownloads { |
| 286 | if image.Id == img.Id && image.Name == img.Name { |
| 287 | if image.DownloadState != voltha.ImageDownload_DOWNLOAD_CANCELLED { |
| 288 | clonedImages = append(clonedImages, img) |
| 289 | } |
| 290 | } |
| 291 | } |
| 292 | cloned.ImageDownloads = clonedImages |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 293 | |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 294 | // Set the Admin state to enabled if required |
| 295 | if (img.DownloadState != voltha.ImageDownload_DOWNLOAD_REQUESTED && |
| 296 | img.DownloadState != voltha.ImageDownload_DOWNLOAD_STARTED) || |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 297 | img.ImageState != voltha.ImageDownload_IMAGE_ACTIVATING { |
| 298 | cloned.AdminState = voltha.AdminState_ENABLED |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 299 | } |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 300 | return agent.updateDeviceAndReleaseLock(ctx, cloned) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 301 | } |
| 302 | |
| 303 | func (agent *Agent) getImageDownload(ctx context.Context, img *voltha.ImageDownload) (*voltha.ImageDownload, error) { |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 304 | logger.Debugw(ctx, "get-image-download", log.Fields{"device-id": agent.deviceID}) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 305 | |
Kent Hagerman | cba2f30 | 2020-07-28 13:37:36 -0400 | [diff] [blame] | 306 | device, err := agent.getDeviceReadOnly(ctx) |
| 307 | if err != nil { |
| 308 | return nil, status.Errorf(codes.Aborted, "%s", err) |
| 309 | } |
Kent Hagerman | f6db9f1 | 2020-07-22 17:16:19 -0400 | [diff] [blame] | 310 | for _, image := range device.ImageDownloads { |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 311 | if image.Id == img.Id && image.Name == img.Name { |
| 312 | return image, nil |
| 313 | } |
| 314 | } |
| 315 | return nil, status.Errorf(codes.NotFound, "image-not-found:%s", img.Name) |
| 316 | } |
| 317 | |
| 318 | func (agent *Agent) listImageDownloads(ctx context.Context, deviceID string) (*voltha.ImageDownloads, error) { |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 319 | logger.Debugw(ctx, "list-image-downloads", log.Fields{"device-id": agent.deviceID}) |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 320 | |
Kent Hagerman | cba2f30 | 2020-07-28 13:37:36 -0400 | [diff] [blame] | 321 | device, err := agent.getDeviceReadOnly(ctx) |
| 322 | if err != nil { |
| 323 | return nil, status.Errorf(codes.Aborted, "%s", err) |
| 324 | } |
| 325 | return &voltha.ImageDownloads{Items: device.ImageDownloads}, nil |
Mahir Gunyel | fa6ea27 | 2020-06-10 17:03:51 -0700 | [diff] [blame] | 326 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 327 | |
| 328 | // onImageFailure brings back the device to Enabled state and sets the image to image download_failed. |
| 329 | func (agent *Agent) onImageFailure(ctx context.Context, rpc string, response interface{}, reqArgs ...interface{}) { |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 330 | // original context has failed due to timeout , let's open a new one |
| 331 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
| 332 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 333 | defer cancel() |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 334 | |
| 335 | if err := agent.requestQueue.WaitForGreenLight(subCtx); err != nil { |
| 336 | logger.Errorw(subCtx, "can't obtain lock", log.Fields{"rpc": rpc, "device-id": agent.deviceID, "error": err, "args": reqArgs}) |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 337 | return |
| 338 | } |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 339 | |
| 340 | device := agent.getDeviceReadOnlyWithoutLock() |
| 341 | |
| 342 | if !agent.proceedWithRequest(device) { |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 343 | agent.requestQueue.RequestComplete() |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 344 | logger.Errorw(subCtx, "Cannot complete operation as device deletion is in progress or reconciling is in progress/failed.", |
Maninder | 581cf4b | 2021-06-16 22:42:07 +0530 | [diff] [blame] | 345 | log.Fields{"rpc": rpc, "device-id": agent.deviceID}) |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 346 | return |
| 347 | } |
| 348 | if res, ok := response.(error); ok { |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 349 | logger.Errorw(subCtx, "rpc-failed", log.Fields{"rpc": rpc, "device-id": agent.deviceID, "error": res, "args": reqArgs}) |
| 350 | cloned := agent.cloneDeviceWithoutLock() |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 351 | //TODO base this on IMAGE ID when created |
| 352 | var imageFailed *voltha.ImageDownload |
| 353 | var index int |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 354 | if cloned.ImageDownloads != nil { |
| 355 | for pos, image := range cloned.ImageDownloads { |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 356 | if image.DownloadState == voltha.ImageDownload_DOWNLOAD_REQUESTED || |
| 357 | image.ImageState == voltha.ImageDownload_IMAGE_ACTIVATING { |
| 358 | imageFailed = image |
| 359 | index = pos |
| 360 | } |
| 361 | } |
| 362 | } |
| 363 | |
| 364 | if imageFailed == nil { |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 365 | logger.Errorw(subCtx, "can't find image", log.Fields{"rpc": rpc, "device-id": agent.deviceID, "args": reqArgs}) |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 366 | return |
| 367 | } |
| 368 | |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 369 | //update image state on failure |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 370 | if imageFailed.DownloadState == voltha.ImageDownload_DOWNLOAD_REQUESTED { |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 371 | cloned.ImageDownloads[index].DownloadState = voltha.ImageDownload_DOWNLOAD_FAILED |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 372 | } else if imageFailed.ImageState == voltha.ImageDownload_IMAGE_ACTIVATING { |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 373 | cloned.ImageDownloads[index].ImageState = voltha.ImageDownload_IMAGE_INACTIVE |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 374 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 375 | //Enabled is the only state we can go back to. |
| 376 | cloned.AdminState = voltha.AdminState_ENABLED |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 377 | if err := agent.updateDeviceAndReleaseLock(subCtx, cloned); err != nil { |
| 378 | logger.Errorw(subCtx, "failed-enable-device-after-image-failure", |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 379 | log.Fields{"rpc": rpc, "device-id": agent.deviceID, "error": res, "args": reqArgs}) |
| 380 | } |
| 381 | } else { |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 382 | logger.Errorw(subCtx, "rpc-failed-invalid-error", log.Fields{"rpc": rpc, "device-id": agent.deviceID, "args": reqArgs}) |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 383 | return |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 384 | } |
| 385 | // TODO: Post failure message onto kafka |
| 386 | } |
| 387 | |
| 388 | // onImageSuccess brings back the device to Enabled state and sets the image to image download_failed. |
| 389 | func (agent *Agent) onImageSuccess(ctx context.Context, rpc string, response interface{}, reqArgs ...interface{}) { |
| 390 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
Himani Chawla | b4c2591 | 2020-11-12 17:16:38 +0530 | [diff] [blame] | 391 | logger.Errorw(ctx, "cannot-obtain-lock", log.Fields{"rpc": rpc, "device-id": agent.deviceID, "error": err, "args": reqArgs}) |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 392 | return |
| 393 | } |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 394 | |
| 395 | cloned := agent.cloneDeviceWithoutLock() |
| 396 | |
| 397 | if !agent.proceedWithRequest(cloned) { |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 398 | agent.requestQueue.RequestComplete() |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 399 | logger.Errorw(ctx, "Cannot complete operation as Device deletion is in progress or reconciling is in progress/failed.", |
Maninder | 581cf4b | 2021-06-16 22:42:07 +0530 | [diff] [blame] | 400 | log.Fields{"rpc": rpc, "device-id": agent.deviceID}) |
Maninder | 0aabf0c | 2021-03-17 14:55:14 +0530 | [diff] [blame] | 401 | return |
| 402 | } |
Matteo Scandolo | 03686da | 2021-03-09 13:16:55 -0800 | [diff] [blame] | 403 | logger.Infow(ctx, "rpc-successful", log.Fields{"rpc": rpc, "device-id": agent.deviceID, "response": response, "args": reqArgs}) |
Maninder | 2195ccc | 2021-06-23 20:23:01 +0530 | [diff] [blame] | 404 | |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 405 | //TODO base this on IMAGE ID when created |
| 406 | var imageSucceeded *voltha.ImageDownload |
| 407 | var index int |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 408 | if cloned.ImageDownloads != nil { |
| 409 | for pos, image := range cloned.ImageDownloads { |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 410 | if image.DownloadState == voltha.ImageDownload_DOWNLOAD_REQUESTED || |
| 411 | image.ImageState == voltha.ImageDownload_IMAGE_ACTIVATING { |
| 412 | imageSucceeded = image |
| 413 | index = pos |
| 414 | } |
| 415 | } |
| 416 | } |
| 417 | |
| 418 | if imageSucceeded == nil { |
| 419 | logger.Errorw(ctx, "can't find image", log.Fields{"rpc": rpc, "device-id": agent.deviceID, "args": reqArgs}) |
| 420 | return |
| 421 | } |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 422 | //update image state on success |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 423 | if imageSucceeded.DownloadState == voltha.ImageDownload_DOWNLOAD_REQUESTED { |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 424 | cloned.ImageDownloads[index].DownloadState = voltha.ImageDownload_DOWNLOAD_SUCCEEDED |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 425 | } else if imageSucceeded.ImageState == voltha.ImageDownload_IMAGE_ACTIVATING { |
Andrea Campanella | c05c4c4 | 2021-02-11 10:25:28 +0100 | [diff] [blame] | 426 | cloned.ImageDownloads[index].ImageState = voltha.ImageDownload_IMAGE_ACTIVE |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 427 | } |
Andrea Campanella | 025667e | 2021-01-14 11:50:07 +0100 | [diff] [blame] | 428 | //Enabled is the only state we can go back to. |
| 429 | cloned.AdminState = voltha.AdminState_ENABLED |
| 430 | if err := agent.updateDeviceAndReleaseLock(ctx, cloned); err != nil { |
| 431 | logger.Errorw(ctx, "failed-enable-device-after-image-download-success", |
| 432 | log.Fields{"rpc": rpc, "device-id": agent.deviceID, "response": response, "args": reqArgs}) |
| 433 | } |
| 434 | |
| 435 | } |
ssiddiqui | f076cb8 | 2021-04-23 10:47:04 +0530 | [diff] [blame] | 436 | |
| 437 | func (agent *Agent) downloadImageToDevice(ctx context.Context, request *voltha.DeviceImageDownloadRequest) (*voltha.DeviceImageResponse, error) { |
| 438 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 439 | return nil, err |
| 440 | } |
| 441 | |
| 442 | logger.Debugw(ctx, "download-image-to-device", log.Fields{"device-id": agent.deviceID}) |
| 443 | if agent.device.Root { |
| 444 | agent.requestQueue.RequestComplete() |
| 445 | return nil, status.Errorf(codes.FailedPrecondition, "device-id:%s, is an OLT. Image update "+ |
| 446 | "not supported by VOLTHA. Use Device Manager or other means", agent.deviceID) |
| 447 | } |
| 448 | |
| 449 | cloned := agent.cloneDeviceWithoutLock() |
| 450 | |
| 451 | // Send the request to the adapter |
| 452 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
| 453 | defer cancel() |
| 454 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
| 455 | |
| 456 | ch, err := agent.adapterProxy.DownloadImageToOnuDevice(subCtx, cloned, request) |
| 457 | agent.requestQueue.RequestComplete() |
| 458 | if err != nil { |
| 459 | return nil, err |
| 460 | } |
| 461 | |
| 462 | return agent.getDeviceImageResponseFromAdapter(ctx, ch) |
| 463 | } |
| 464 | |
| 465 | func (agent *Agent) getImageStatus(ctx context.Context, request *voltha.DeviceImageRequest) (*voltha.DeviceImageResponse, error) { |
| 466 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 467 | return nil, err |
| 468 | } |
| 469 | |
| 470 | cloned := agent.cloneDeviceWithoutLock() |
| 471 | |
| 472 | // Send the request to the adapter |
| 473 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
| 474 | defer cancel() |
| 475 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
| 476 | logger.Debugw(ctx, "get-image-status", log.Fields{"device-id": agent.deviceID}) |
| 477 | |
| 478 | ch, err := agent.adapterProxy.GetOnuImageStatus(subCtx, cloned, request) |
| 479 | agent.requestQueue.RequestComplete() |
| 480 | if err != nil { |
| 481 | return nil, err |
| 482 | } |
| 483 | |
| 484 | return agent.getDeviceImageResponseFromAdapter(subCtx, ch) |
| 485 | } |
| 486 | |
| 487 | func (agent *Agent) activateImageOnDevice(ctx context.Context, request *voltha.DeviceImageRequest) (*voltha.DeviceImageResponse, error) { |
| 488 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 489 | return nil, err |
| 490 | } |
| 491 | |
| 492 | cloned := agent.cloneDeviceWithoutLock() |
| 493 | |
| 494 | // Send the request to the adapter |
| 495 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
| 496 | defer cancel() |
| 497 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
| 498 | logger.Debugw(ctx, "activate-image-on-device", log.Fields{"device-id": agent.deviceID}) |
| 499 | |
| 500 | ch, err := agent.adapterProxy.ActivateOnuImage(subCtx, cloned, request) |
| 501 | agent.requestQueue.RequestComplete() |
| 502 | if err != nil { |
| 503 | return nil, err |
| 504 | } |
| 505 | |
| 506 | return agent.getDeviceImageResponseFromAdapter(subCtx, ch) |
| 507 | } |
| 508 | |
| 509 | func (agent *Agent) abortImageUpgradeToDevice(ctx context.Context, request *voltha.DeviceImageRequest) (*voltha.DeviceImageResponse, error) { |
| 510 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 511 | return nil, err |
| 512 | } |
| 513 | |
| 514 | cloned := agent.cloneDeviceWithoutLock() |
| 515 | |
| 516 | // Send the request to the adapter |
| 517 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
| 518 | defer cancel() |
| 519 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
| 520 | logger.Debugw(ctx, "abort-image-on-device", log.Fields{"device-id": agent.deviceID}) |
| 521 | |
| 522 | ch, err := agent.adapterProxy.AbortImageUpgrade(subCtx, cloned, request) |
| 523 | agent.requestQueue.RequestComplete() |
| 524 | if err != nil { |
| 525 | return nil, err |
| 526 | } |
| 527 | |
| 528 | return agent.getDeviceImageResponseFromAdapter(ctx, ch) |
| 529 | } |
| 530 | |
| 531 | func (agent *Agent) commitImage(ctx context.Context, request *voltha.DeviceImageRequest) (*voltha.DeviceImageResponse, error) { |
| 532 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 533 | return nil, err |
| 534 | } |
| 535 | |
| 536 | cloned := agent.cloneDeviceWithoutLock() |
| 537 | |
| 538 | // Send the request to the adapter |
| 539 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
| 540 | defer cancel() |
| 541 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
| 542 | logger.Debugw(ctx, "commit-image-on-device", log.Fields{"device-id": agent.deviceID}) |
| 543 | |
| 544 | ch, err := agent.adapterProxy.CommitImage(subCtx, cloned, request) |
| 545 | agent.requestQueue.RequestComplete() |
| 546 | if err != nil { |
| 547 | return nil, err |
| 548 | } |
| 549 | |
| 550 | return agent.getDeviceImageResponseFromAdapter(ctx, ch) |
| 551 | } |
| 552 | |
| 553 | func (agent *Agent) getOnuImages(ctx context.Context, id *common.ID) (*voltha.OnuImages, error) { |
| 554 | if err := agent.requestQueue.WaitForGreenLight(ctx); err != nil { |
| 555 | return nil, err |
| 556 | } |
| 557 | |
| 558 | cloned := agent.cloneDeviceWithoutLock() |
| 559 | |
| 560 | subCtx, cancel := context.WithTimeout(log.WithSpanFromContext(context.Background(), ctx), agent.defaultTimeout) |
| 561 | defer cancel() |
| 562 | subCtx = coreutils.WithRPCMetadataFromContext(subCtx, ctx) |
ssiddiqui | 47348f6 | 2021-05-20 20:41:15 +0530 | [diff] [blame] | 563 | logger.Debug(ctx, "get-onu-images") |
ssiddiqui | f076cb8 | 2021-04-23 10:47:04 +0530 | [diff] [blame] | 564 | |
| 565 | // Send the request to the adapter |
| 566 | ch, err := agent.adapterProxy.GetOnuImages(subCtx, cloned, id) |
| 567 | agent.requestQueue.RequestComplete() |
| 568 | if err != nil { |
| 569 | return nil, err |
| 570 | } |
| 571 | |
| 572 | //wait for adapter response |
| 573 | select { |
| 574 | case rpcResponse, ok := <-ch: |
| 575 | if !ok { |
| 576 | return nil, status.Errorf(codes.Aborted, "channel-closed-device-id-%s", agent.deviceID) |
| 577 | } else if rpcResponse.Err != nil { |
| 578 | // return error |
| 579 | return nil, status.Errorf(codes.Internal, "%s", rpcResponse.Err.Error()) |
| 580 | } else { |
| 581 | resp := &voltha.OnuImages{} |
| 582 | if err := ptypes.UnmarshalAny(rpcResponse.Reply, resp); err != nil { |
| 583 | return nil, status.Errorf(codes.Internal, "%s", err.Error()) |
| 584 | } |
| 585 | |
| 586 | return resp, nil |
| 587 | } |
| 588 | case <-ctx.Done(): |
| 589 | return nil, ctx.Err() |
| 590 | } |
| 591 | } |
| 592 | |
| 593 | func (agent *Agent) getDeviceImageResponseFromAdapter(ctx context.Context, ch chan *kafka.RpcResponse) (*voltha.DeviceImageResponse, error) { |
| 594 | //wait for adapter response |
| 595 | select { |
| 596 | case rpcResponse, ok := <-ch: |
| 597 | if !ok { |
| 598 | return nil, status.Errorf(codes.Aborted, "channel-closed-device-id-%s", agent.deviceID) |
| 599 | } else if rpcResponse.Err != nil { |
| 600 | // return error |
| 601 | return nil, status.Errorf(codes.Internal, "%s", rpcResponse.Err.Error()) |
| 602 | } else { |
| 603 | resp := &voltha.DeviceImageResponse{} |
| 604 | if err := ptypes.UnmarshalAny(rpcResponse.Reply, resp); err != nil { |
| 605 | return nil, status.Errorf(codes.Internal, "%s", err.Error()) |
| 606 | } |
| 607 | |
| 608 | if len(resp.DeviceImageStates) == 0 || resp.DeviceImageStates[0] == nil { |
| 609 | return nil, status.Errorf(codes.Internal, "invalid response from adapter") |
| 610 | } |
| 611 | |
| 612 | return resp, nil |
| 613 | } |
| 614 | case <-ctx.Done(): |
| 615 | return nil, ctx.Err() |
| 616 | } |
| 617 | } |