-
Notifications
You must be signed in to change notification settings - Fork 57
Description
Hi, I was just made aware of your interesting project and instantly glanced at what it plugs in via API and what it potentially modifies.
I noticed you added an upgrade option to node_cmd.
This might break PVE install as them tracking all dependencies correctly is not guaranteed - it happened before. It needs to be full-upgrade (formerly dist-upgrade) instead.
On a separate note, if you were planning to keep rolling your patched backend Perl module forward, you might like my PoC (split off another project) which can do this for you, i.e. you could just define new node_cmd, provide old and new blocks of code and unless Proxmox change that very lines, it will keep preserving your change on Proxmox upgrades (where a regular patch would fail).
Enough self-promotion, if you were interested in using it as dependency, feel free to file me an issue there - I am leaving it experimental for now (for the person who first requested it), but the approach has been well tested by now in patching front-end. You are, of course, also free to fork it, license is AGPL though.
Cheers and all the best with this nice project!