Skip to content

Commit

Permalink
fix: batchFirstFingerprints does not update device on node after v1.3…
Browse files Browse the repository at this point in the history
….5 (#15125)

* fix: update device in batch first footprint

* cl: add cl note

Co-authored-by: Seth Hoenig <shoenig@duck.com>
  • Loading branch information
vuuihc and shoenig committed Nov 3, 2022
1 parent cc036c0 commit ca06227
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .changelog/15125.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:bug
device: Fixed a bug where device plugins would not fingerprint on startup
```
1 change: 1 addition & 0 deletions client/node_updater.go
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ SEND_BATCH:
var devicesChanged bool
c.batchNodeUpdates.batchDevicesUpdates(func(devices []*structs.NodeDeviceResource) {
if c.updateNodeFromDevicesLocked(devices) {
newConfig.Node.NodeResources.Devices = devices
devicesChanged = true
}
})
Expand Down

0 comments on commit ca06227

Please sign in to comment.