-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecate __package_update_index and __package_upgrade_all #7
Comments
So, in the light of this idea that every |
I updated #2.
One could argue that updating the local index and installing packages are separate processes and should thus be separate types. In any case, I think that it should be handled consistently for all package managers. |
Both types have been moved to cdist-museum (skonfig/museum#1). |
[__jitsi_meet_user] adds user validation and directory creation Closes skonfig#7 See merge request ungleich-public/cdist-contrib!35
…aster_explorer __find_exec: make explorer faster
This is a follow-up to #5.
After giving it some thought, I think we should move
__package_update_index
and__package_upgrade_all
to -museum, or at least -extra.Both types only support a subset (APT, APK, Pacman, YUM) of package managers supported by cdist types.
These types don't really fit well with the design of cdist.
cdist's job is managing configuration, updating local caches or packages isn't configuration.
Moreover, since cdist is agent-less, it can be run infrequently, possibly leaving systems in an unupgraded state for a long time.
Keeping a system up-to-date should be solved via some other means (e.g. unattended-upgrades on Debian, apk-cron on Alpine, a cron job).
As far as
__package_update_index
is concerned, it might be necessary to update local package indexes prior to installing new packages:__package_apt
: implements the refreshing logic itself.__package_apk
: requires refreshing the local cache.This could be handled in the type by adding
--cache-max-age
to the option list.__package_pacman
: requires refreshing the local cache.This could be handled in the type by adding
--refresh
to the option list.Then pacman will ask the mirror if it has a newer version of the file, and if so download it prior to installing the package.
The process is fairly quick.
__package_yum
: noop.The text was updated successfully, but these errors were encountered: