opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/c/openstack/requirements/+/845082 | 05:43 |
---|---|---|
opendevreview | Dr. Jens Harbott proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/c/openstack/requirements/+/845082 | 06:28 |
opendevreview | Dr. Jens Harbott proposed openstack/requirements master: Add variable to requirements-check job https://review.opendev.org/c/openstack/requirements/+/845023 | 06:33 |
opendevreview | Matthew Thode proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/c/openstack/requirements/+/845082 | 15:12 |
prometheanfire | frickler: sorry, didn't notice you working the generate-constraints patch | 15:35 |
rosmaita | prometheanfire: frickler: i have a question about requirements for cinderlib | 15:36 |
rosmaita | https://opendev.org/openstack/cinderlib/src/branch/master/requirements.txt | 15:36 |
rosmaita | cinderlib is used by Ember-CSI to let you use the cinder drivers without running cinder | 15:37 |
rosmaita | but it depends on a bunch of stuff in cinder, obviously | 15:37 |
rosmaita | the problem is that we can't pass requirements check because cinder is not in global-requirements | 15:38 |
rosmaita | but if you install cinderlib from pypi, you absolutely need to install cinder too | 15:38 |
prometheanfire | rosmaita: sounds like cinder itself is used as a lib? | 15:38 |
rosmaita | yeah, basically | 15:38 |
rosmaita | i saw that Tony left a comment in global-requirements that normally openstack services don't go in there | 15:39 |
prometheanfire | well, 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 |
rosmaita | right, but we would most likely have to re-architect cinder | 15:40 |
prometheanfire | not saying it's easy, just what's 'right' | 15:42 |
prometheanfire | it's also require changing release types | 15:42 |
rosmaita | ? | 15:42 |
prometheanfire | iirc netron/horizon have a diferent release type because they are used as libs | 15:43 |
rosmaita | oh | 15:43 |
prometheanfire | a 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 |
prometheanfire | horizon is cycle-with-intermediary, neutron is cycle-with-rc, I think it should be with-intermediary though (standard for libs) | 15:46 |
rosmaita | well, we have that handled for cinderlib with CI | 15:46 |
rosmaita | i mean, we don't want just arbitrary people using cinder as a lib | 15:46 |
rosmaita | just cinderlib | 15:46 |
prometheanfire | right, that's the problem | 15:46 |
prometheanfire | didn't pip have this issue where they made some functions ultra-private or something | 15:47 |
rosmaita | well, we see cinderlib as defining the stable interface | 15:48 |
prometheanfire | right, but you are still using cinder itself as a lib | 15:52 |
rosmaita | or are we using it as a dynamic code repository for which cinderlib is the lib? | 15:53 |
frickler | prometheanfire: 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.txt | 16:26 |
frickler | likely in an etherpad? or do you have something like that already? | 16:27 |
prometheanfire | frickler: etherpad would be right | 16:45 |
opendevreview | Matthew Thode proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/c/openstack/requirements/+/845082 | 16:52 |
opendevreview | Merged openstack/requirements master: Updated from generate-constraints https://review.opendev.org/c/openstack/requirements/+/845082 | 20:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!