Wednesday, 2022-06-08

opendevreviewOpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints  https://review.opendev.org/c/openstack/requirements/+/84508205:43
opendevreviewDr. Jens Harbott proposed openstack/requirements master: Updated from generate-constraints  https://review.opendev.org/c/openstack/requirements/+/84508206:28
opendevreviewDr. Jens Harbott proposed openstack/requirements master: Add variable to requirements-check job  https://review.opendev.org/c/openstack/requirements/+/84502306:33
opendevreviewMatthew Thode proposed openstack/requirements master: Updated from generate-constraints  https://review.opendev.org/c/openstack/requirements/+/84508215:12
prometheanfirefrickler: sorry, didn't notice you working the generate-constraints patch15:35
rosmaitaprometheanfire: frickler: i have a question about requirements for cinderlib15:36
rosmaitahttps://opendev.org/openstack/cinderlib/src/branch/master/requirements.txt15:36
rosmaitacinderlib is used by Ember-CSI to let you use the cinder drivers without running cinder15:37
rosmaitabut it depends on a bunch of stuff in cinder, obviously15:37
rosmaitathe problem is that we can't pass requirements check because cinder is not in global-requirements15:38
rosmaitabut if you install cinderlib from pypi, you absolutely need to install cinder too15:38
prometheanfirerosmaita: sounds like cinder itself is used as a lib?15:38
rosmaitayeah, basically15:38
rosmaitai saw that Tony left a comment in global-requirements that normally openstack services don't go in there15:39
prometheanfirewell, the 'right' thing is to move the things you wish to use as a lib to cinderlib (like neutron-lib is supposed to do...)15:39
rosmaitaright, but we would most likely have to re-architect cinder15:40
prometheanfirenot saying it's easy, just what's 'right'15:42
prometheanfireit's also require changing release types15:42
rosmaita?15:42
prometheanfireiirc netron/horizon have a diferent release type because they are used as libs15:43
rosmaitaoh15:43
prometheanfirea problem with the service projects being used as libs is that that comes with the baggage of being a lib (stability and stuff)15:44
prometheanfirehorizon is cycle-with-intermediary, neutron is cycle-with-rc, I think it should be with-intermediary though (standard for libs)15:46
rosmaitawell, we have that handled for cinderlib with CI15:46
rosmaitai mean, we don't want just arbitrary people using cinder as a lib15:46
rosmaitajust cinderlib15:46
prometheanfireright, that's the problem15:46
prometheanfiredidn't pip have this issue where they made some functions ultra-private or something15:47
rosmaitawell, we see cinderlib as defining the stable interface15:48
prometheanfireright, but you are still using cinder itself as a lib15:52
rosmaitaor are we using it as a dynamic code repository for which cinderlib is the lib?15:53
fricklerprometheanfire: it's o.k., I was thinking to start generating a list of failures/bugs the relate to the list of pins you keep applying manually https://review.opendev.org/c/openstack/requirements/+/845082/2..3/upper-constraints.txt16:26
fricklerlikely in an etherpad? or do you have something like that already?16:27
prometheanfirefrickler: etherpad would be right16:45
opendevreviewMatthew Thode proposed openstack/requirements master: Updated from generate-constraints  https://review.opendev.org/c/openstack/requirements/+/84508216:52
opendevreviewMerged openstack/requirements master: Updated from generate-constraints  https://review.opendev.org/c/openstack/requirements/+/84508220:21

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!