search

foundObjects / pve-nag-buster

Persistent license nag removal for Proxmox VE 5.x+
GNU General Public License v2.0
841 stars 83 forks source link

Add functionality for multiple repos - fix for 8.0 #15

Open guffelman opened 1 year ago

guffelman commented 1 year ago

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

spacelama commented 1 year ago

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.

Yrlish commented 9 months ago

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