While re-imaging worker nodes as part of the ServiceOps vlan migration work {T421711}, we realized that the `sre.hosts.reimage` cookbook does not support drain and cordon logic needed for k8s worker nodes (intentional by design), since we do worker node reimages in batches using the `sre.k8s.roll-reimage-nodes`cookbook in [0]
As such, we could either implement support `--move-vlan` in `sre.k8s.roll-reimage-nodes` or decide on some workaround if we choose not to introduce necessarily interactive changes (out of cookbook homer diffs for example) to the current non interactive workflow in `sre.k8s.roll-reimage-nodes`.
We could also consider an automated workflow for vlan migrations on k8s worker nodes separate to the `sre.k8s.roll-reimage-nodes` cookbook altogether.
A temporary workaround may be to call the `pool-depool-node` cookbook [1], reimage with `sre.hosts.reimage` + `--move-vlan`, then pool again with `pool-depool-node`.
[0] https://gerrit.wikimedia.org/r/plugins/gitiles/operations/cookbooks/+/refs/heads/master/cookbooks/sre/k8s/roll-reimage-nodes.py
[1] https://gerrit.wikimedia.org/r/plugins/gitiles/operations/cookbooks/+/refs/heads/master/cookbooks/sre/k8s/pool-depool-node.py