pve-nag-buster icon indicating copy to clipboard operation
pve-nag-buster copied to clipboard

Add functionality for multiple repos - fix for 8.0

Open guffelman opened this issue 1 year ago • 2 comments

I've added a new function disable_repo() that allows you to pass a repository in to disable. It will change the repository to "$REPO_BASE.disabled"

This should allow for multiple repositories to be disabled, which fixes the release for 8.0. They added a new enterprise repo in ceph.list.

Thanks! G

guffelman avatar Sep 08 '23 11:09 guffelman

I haven't tried running this yet, but I suspect install.sh also wants a change to enable a new /etc/apt/sources.list.d/ceph-no-subscription.list similar to how it creates the pve-no-subscription.list file. I don't understand why the repo is split up into two files install.sh and the embedded base64 version of pve-nag-buster.sh, the latter of which deletes the enterprise file and the former creating the new file, but in this case it hides the fact that the pull request isn't quite complete as far as PVE8 goes.

spacelama avatar Sep 16 '23 12:09 spacelama

I just forked the repo and added your changes to it. It works, I just ran it myself on a freshly installed PVE 8.1.4.

https://github.com/Yrlish/pve-nag-buster

Yrlish avatar Feb 06 '24 11:02 Yrlish