Bug 236636 - sysutils/py-salt include dependencies
Summary: sysutils/py-salt include dependencies
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-03-19 10:14 UTC by Sven Ruediger
Modified: 2021-09-13 20:14 UTC (History)
4 users (show)

See Also:
bugzilla: maintainer-feedback? (christer.edwards)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sven Ruediger 2019-03-19 10:14:25 UTC 
What do you think about adding options to for additional dependencies? (and change transport technology buttons to type RADIO)

For example psutils and pip might be useful for many but not all users.


   │ ┌──────────────────────────────────────────────────────────────────────────┐ │
   │ │+[ ] AWS       Install dependencies required for AWS                      │ │
   │ │+[ ] AZURE     Install dependencies required for Azure                    │ │
   │ │+[ ] CLOUD     Install dependencies for cloud mgmt (curl, libcloud, reques│ │
   │ │+[ ] DB        Install dependencies for databases (Elasticsearch, Influx, │ │
   │ │+[x] SYSUTILS  Install support for sysutils (curl, pip, psutils)          │ │
   │ │────────────── Install dependencies for transport technology ─────────────│ │
   │ │+( ) RAET      Install dependencies required for RAET transport           │ │
   │ │+( ) TCP       Install dependencies required for TCP transport            │ │
   │ │+(*) ZEROMQ    Install dependencies required for ZeroMQ transport         │ │
   │ └──────────────────────────────────────────────────────────────────────────┘ │
   ├──────────────────────────────────────────────────────────────────────────────┤
   │                       <  OK  >            <Cancel>                           │


CLOUD:
net/py-libcloud (remove standard dependency)
www/py-requests
ftp/curl
security/ca_root_nss

AZURE:
net-mgmt/py-msrest
net-mgmt/py-msrestazure
devel/py-azure-mgmt-network
devel/py-azure-mgmt-resource
devel/py-azure-mgmt-dns
devel/py-azure-mgmt-compute

DB:
databases/py-sqlite3
databases/py-influxdb
textproc/py-elasticsearch
databases/py-mysqlclient

SYSUTILS (default):
devel/py-pip
sysutils/py-psutil
.... (anything else?)


Cheers,
Sven
Comment 1 Ben Woods freebsd_committer freebsd_triage 2020-03-06 23:14:15 UTC 
Hi Christer,

Any thoughts on this feature request?

Cheers,
Ben
Comment 2 Li-Wen Hsu freebsd_committer freebsd_triage 2020-06-17 17:18:59 UTC 
krion: are you interested in checking this?
Comment 3 Kirill Ponomarev freebsd_committer freebsd_triage 2020-06-17 17:27:56 UTC 
I agree to change radio for the transport, but I wouldn't add any sysutils like curl etc.
Comment 4 Sven Ruediger 2020-06-17 18:31:18 UTC 
Sorry for mixing up two request (radio button and new options).


For the sysutils option:
Salt uses many py-modules that aren't necessary for salt itself (just modules that can be executed on remote minion, like py-psutil, py-pip). It should be added by default (pkg), but making it configureable when using ports/poudriere could reduce a lot of (imho unnecessary) dependencies.

I agree, ftp/curl should be removed from options. Not sure for security/ca_root_nss as it could be used in https://docs.saltstack.com/en/latest/ref/modules/all/salt.modules.http.html