-
Notifications
You must be signed in to change notification settings - Fork 118
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
Virtual packages are depricated. #128
Comments
@absmith82 The problem here is this formula is still using the old-style |
I definitely understand why Salt deprecate support of "virtual packages" but form the user point of view it brings new inconvenience. Even if we have Although this issue definitely can be solved as I mention above, we just need to populate Debian-10:
pgsql:
pkgs:
- postgresql-client-common-11
- postgresql-client-11
Debian-9:
pgsql:
pkgs:
- postgresql-client-common-9.6
- postgresql-client-9.6
# etc... After this new issue will arise - it's became mandatory to explicitly define version if user want to use not "OS default" package version, maybe this is not a problem at all, depending on the point of view. |
The issue is that regardless of the fact that is was an easier system, I agree I wish it were back, all current supported and future versions of salt will not support keeping the virtual packages. |
Do you have any other (probably better) solution then what a proposed in my previous message? |
When running zabbix.pgsql.conf.sls on ubuntu 18.04 I get this error.
These packages should be placed into the map.jinja file so that we can install the correct package for the environment.
it looks like the correct client for Ubuntu depends on the version of Ubuntu, so this may be hard to maintain, but currently 16.04 uses postgresql-client-9.5, 18.04 and 18.10 use postgresql-client-10, and 19.04 uses postgresql-client-11
I can see how this could be a constant and daunting task as each release of different OSes could have a different client, however if this is not going to be supported moving forward it needs to be fixed.
The text was updated successfully, but these errors were encountered: