*** coolsvap has joined #openstack-requirements | 02:56 | |
coolsvap | prometheanfire, the new requirements for monasca look good except for https://review.openstack.org/335470 and https://review.openstack.org/335455 | 03:08 |
---|---|---|
prometheanfire | coolsvap: ya, was going to look at them today, been busy | 03:15 |
prometheanfire | reroof, part of kitchen redo, woo | 03:15 |
coolsvap | prometheanfire, sure just wanted to let you know thought you are reviewing | 03:15 |
coolsvap | later | 03:15 |
prometheanfire | later | 03:17 |
openstackgerrit | Merged openstack/requirements: Updated from generate-constraints https://review.openstack.org/334227 | 03:48 |
*** prometheanfire has quit IRC | 06:38 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/335815 | 06:53 |
*** prometheanfire has joined #openstack-requirements | 06:55 | |
*** jpena|off is now known as jpena | 07:43 | |
*** coolsvap is now known as coolsvap_ | 08:02 | |
number80 | coolsvap_, prometheanfire: I think that https://review.openstack.org/#/c/335426/3 is also problematic | 09:06 |
number80 | we already have a CLI library | 09:07 |
number80 | (so conflicts w/ guidelines to accept new libraries) | 09:08 |
*** dolphm has quit IRC | 09:08 | |
*** dolphm has joined #openstack-requirements | 09:08 | |
number80 | and likely to be dropped as clients are being ported to openstackclient | 09:08 |
number80 | https://review.openstack.org/#/c/335462/2 has the same issue, conflicts with voluptuous as in purpose | 09:11 |
openstackgerrit | Joan Varvenne proposed openstack/requirements: Add scikit-learn, a library for machine learning. https://review.openstack.org/335469 | 09:42 |
*** toabctl has quit IRC | 10:44 | |
*** toabctl has joined #openstack-requirements | 10:46 | |
*** toabctl has quit IRC | 10:47 | |
*** toabctl has joined #openstack-requirements | 10:48 | |
*** jamespag` is now known as jamespge | 11:02 | |
*** jamespge is now known as jamespage | 11:02 | |
*** jpena is now known as jpena|lunch | 11:16 | |
coolsvap_ | number80, thanks i think in both cases we will need to analyse how much they depend on the libraries and what is the effort required to migrate | 11:25 |
coolsvap_ | number80, what is your view on https://review.openstack.org/#/c/335455/ | 11:26 |
number80 | coolsvap_: well, from docopt to argparse should be doable | 11:26 |
number80 | findspark needs a bit more of analysis | 11:27 |
number80 | it's not a very active project but should simple to package for vendors | 11:27 |
number80 | https://github.com/minrk/findspark | 11:27 |
number80 | it's likely not the pristine project though | 11:29 |
number80 | well, let's share my concerns | 11:34 |
number80 | *in the review | 11:34 |
coolsvap_ | number80, thanks! | 11:39 |
number80 | Not -1 as monasca folks could maintain it, it's trivial code | 11:42 |
*** coolsvap_ has quit IRC | 12:05 | |
*** jpena|lunch is now known as jpena | 12:46 | |
openstackgerrit | John L. Villalovos proposed openstack/requirements: Bump oslo.utils >= 3.14.0 https://review.openstack.org/335133 | 13:44 |
*** rbrndt_ has joined #openstack-requirements | 13:52 | |
prometheanfire | number80: thanks, that's why I haven't +2'd much of those yet, haven't had time to check them | 14:04 |
number80 | Yeah, I feel like Monasca people needs some help | 14:05 |
*** dims has quit IRC | 16:00 | |
*** dims has joined #openstack-requirements | 16:06 | |
openstackgerrit | Paul Belanger proposed openstack/requirements: Do not merge! Testing xenial wheels https://review.openstack.org/336104 | 16:16 |
*** jpena is now known as jpena|off | 17:08 | |
*** rbrndt_ has quit IRC | 18:17 | |
openstackgerrit | Merged openstack/requirements: Bump oslo.utils >= 3.14.0 https://review.openstack.org/335133 | 18:39 |
*** rbrndt_ has joined #openstack-requirements | 18:47 | |
openstackgerrit | Merged openstack/requirements: Add scipy, scientific tools for python. https://review.openstack.org/335459 | 18:59 |
openstackgerrit | Kaiyan Sheng proposed openstack/requirements: Add monasca-common to projects.txt https://review.openstack.org/334625 | 20:27 |
openstackgerrit | Kaiyan Sheng proposed openstack/requirements: Add ujson, an ultra fast JSON encoder and decoder https://review.openstack.org/335604 | 20:27 |
*** coreycb has quit IRC | 20:31 | |
*** coreycb has joined #openstack-requirements | 20:31 | |
*** jroll|dupe has joined #openstack-requirements | 21:44 | |
*** jroll|dupe has quit IRC | 21:44 | |
*** jroll|dupe has joined #openstack-requirements | 21:44 | |
*** rbrndt_ has quit IRC | 21:45 | |
*** jroll has quit IRC | 21:45 | |
*** jroll|dupe is now known as jroll | 21:45 | |
openstackgerrit | Merged openstack/requirements: Add python-tackerclient to global requirements https://review.openstack.org/334641 | 21:54 |
prometheanfire | tonyb: https://review.openstack.org/335604 | 22:27 |
prometheanfire | tonyb: what do you think? | 22:27 |
openstackgerrit | Ken'ichi Ohmichi proposed openstack/requirements: Bump minimum Tempest version https://review.openstack.org/335703 | 22:47 |
tonyb | prometheanfire: checking .... | 23:13 |
prometheanfire | thanks | 23:15 |
tonyb | prometheanfire: I'm with you. I think we need a little more reserach on how compatible ujson is. Can we add it and then transition users of simplejson? | 23:15 |
prometheanfire | exactly | 23:15 |
tonyb | but on the other hand monasca are *trying* to do the right thing and join in this part of our community at no small cosr to them so there is a balance between the technical and social costs ... | 23:16 |
* tonyb adds that to his "don | 23:16 | |
tonyb | t sleep until you have done .... " list | 23:16 |
prometheanfire | lol | 23:17 |
prometheanfire | I just commented asking them to ask on the ML about others migrating to it | 23:17 |
prometheanfire | anyway, dinner | 23:17 |
tonyb | Well simplejson is pure python where as ujson is pure C | 23:18 |
tonyb | so there will be resistance to that .... | 23:18 |
prometheanfire | ya | 23:21 |
prometheanfire | I saw that but din't mention it | 23:21 |
prometheanfire | personally, I feel that's fine | 23:21 |
tonyb | Hmm so we have json and simpljson which are basically the same thing. My vote is to add ujson and transition users of simplejson to the regular json | 23:22 |
tonyb | then people that want "fast" have an option and other can do without the C builddeps | 23:22 |
* prometheanfire shrugs, I don't have a prefrence, but just want to keep the sprawl down | 23:22 | |
tonyb | and then we add a todo list for the reuirements team to get simplejson out | 23:22 |
prometheanfire | afk | 23:23 |
prometheanfire | ya | 23:23 |
prometheanfire | +1 on that | 23:23 |
prometheanfire | really afk | 23:23 |
openstackgerrit | Merged openstack/requirements: Updated from generate-constraints https://review.openstack.org/335815 | 23:41 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!