Sunday, 2018-03-25

*** odyssey4me has quit IRC00:17
*** odyssey4me has joined #openstack-requirements00:17
*** hongbin has joined #openstack-requirements03:52
*** hongbin has quit IRC04:41
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints  https://review.openstack.org/55599806:23
openstackgerritMatthew Thode proposed openstack/requirements master: Updated from generate-constraints  https://review.openstack.org/55599806:34
*** oanson has quit IRC10:26
openstackgerritAdit Sarfaty proposed openstack/requirements master: Bump tenacity to 4.4.0  https://review.openstack.org/55616410:31
*** oanson has joined #openstack-requirements10:31
openstackgerritgaryk proposed openstack/requirements master: Use the correct version for vmware-nsxlib  https://review.openstack.org/55617510:59
*** odyssey4me has quit IRC12:11
*** odyssey4me has joined #openstack-requirements12:11
pabelangerdhellmann: not to distract with +topic:requirements-stop-syncing but is openstack-tox-lower-constraints something all official projects will have to run?13:18
dhellmannpabelanger : no, it's optional.13:19
pabelangerack13:19
dhellmannthe requirements team wanted things set up so projects could adopt the job before they would agree to turning off the sync job13:19
pabelangerk, I wasn't sure if that job was intree with .zuul.yaml files, if projects would disable it for one reason or another. Causing issues with future releases13:20
dhellmannyeah, as far as I'm concerned it's entirely up to project teams to decide to support the idea of testing lower bounds13:21
dhellmannI have about 50 more patches to propose, and I'm hoping to finish that over the course of today while the check gate is relatively slow13:22
pabelangerwfm13:22
amotokidhellmann: lower-constainrts patches turn out django minimum version used by horizon rocky is not correct.17:41
amotokidhellmann: what is the recommended way to fix minimum version now?17:41
amotokican we update lower-constraints.txt in proposed patches or do we need to propose a minimum version bump to requirements repo?17:43
dhellmannamotoki : you will need to use Depends-On with https://review.openstack.org/555402 in order to change the lower bounds from something different than appears in the global requirements list, but then you can change both files at once20:06
dhellmannprometheanfire , dirk , tonyb : I think we are ready to move ahead with https://review.openstack.org/555402. Let me know what you think, please.20:06
amotokidhellmann: thanks. I saw your reply comment in horizon. it works well in the updated version :)20:07
dhellmannexcellent20:08
dhellmannthank you for taking over fixing that patch, amotoki :-)20:09
amotokidhellmann: I also will check l-c stuffs in neutron and horizon plugin repos later today20:10
dhellmannamotoki : ok, sounds good. For those, we may have problems until we figure out the right way to ensure that neutron or horizon are installed for the test jobs20:26
amotokidhellmann: yeah, most failures are caused by the recent installation change (tox_install.sh or installing from tarball). it seems better to wait mordred on this topic20:28
dhellmannI agree20:29
dhellmannprometheanfire : you might want to think about defining a tag like assert:tests-dependencies-lower-bounds so that teams that decide to participate in these new tests get credit in the form of "maturity points" for the project on the project navigator20:33
mordredamotoki: what happened?20:47
amotokimordred: some of lower-constraints patches are blocked by the recent upper-constraints/tox_install changes on horizon/neutron.20:49
amotokimordred: some repos have not caught up with the change. on the other hand, we don't have a consensus on the direction20:50
mordredgotcha. is there an example I should look at?20:50
mordredamotoki: I also owe people an email to the mailng list20:50
amotokimordred: one way is to drop tox_install.sh (or similar change) but I am not sure I should do that before getting a consensus20:53
amotokimordred: several patterns of failure https://review.openstack.org/#/q/topic:requirements-stop-syncing+(status:open+OR+status:merged)+project:%255Eopenstack/.*-dashboard+label:Verified%253C%253D-120:53
mordredamotoki: sweet, thanks20:55
mordredamotoki: and yes - I believe dropping tox_install is going to be the right choice - BUT - we do need to get consensus on that and make sure everyone understands what's going on20:55
mordredamotoki: I'm doing a conference this weekend, but I'll send out an email tomorrow on the plane20:56
amotokimordred: thanks for your time on a weekend. i think no hurries20:57
mordredamotoki: murano-dashboard and mistral-dashboard don't have tox_install - they just are listing tarballs in their tox.ini files20:59
mordredhttps://github.com/openstack/murano-dashboard/blob/master/tox.ini#L1921:00
amotokimordred: yes, it is a different pattern.21:00
amotokii would like to change them to follow the standard pattern once we get consensus21:00
mordred++21:01
amotokikarbor-dashboard has horizon tarball in test-reqs. this is another pattern :(21:02
prometheanfiredhellmann: good idea21:30
prometheanfiredhellmann: I'll review 555402 tonight or tomorrow21:30
prometheanfirestill working around the house21:30
openstackgerritMerged openstack/requirements master: Bump tenacity to 4.4.0  https://review.openstack.org/55616422:16
dhellmannprometheanfire : sure, I'm only working on this over the weekend because the gate queue is shallower and I thought I'd drop a mention so you saw it first thing monday. I have no expectation that you'd give up any weekend time.23:18
dhellmann mordred, amotoki : thanks, I think once we have a clear set of directions we can sort all of those patches out23:18

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!