Review: Approve

+1, with some nits:

- add this check to Wedge protocol (same file), as it also relies on dmidecode
- the reason we don't declare dmidecode as a dependency of this script is that 
there isn't a package for all platforms MAAS deploys 
(https://packages.ubuntu.com/jammy/dmidecode). Given that, should we be relying 
on it? 
-- 
https://code.launchpad.net/~troyanov/maas/+git/maas/+merge/444076
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp

Reply via email to