| *** mriedem has quit IRC | 00:13 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: Re-propose numa-aware-live-migration spec https://review.openstack.org/599587 | 00:36 |
|---|---|---|
| *** tetsuro has joined #openstack-placement | 00:58 | |
| *** tetsuro has quit IRC | 04:05 | |
| *** tetsuro has joined #openstack-placement | 05:55 | |
| *** tetsuro has quit IRC | 06:40 | |
| *** e0ne has joined #openstack-placement | 07:10 | |
| *** tetsuro has joined #openstack-placement | 07:54 | |
| *** tetsuro has quit IRC | 08:14 | |
| *** tetsuro has joined #openstack-placement | 08:16 | |
| *** helenafm has joined #openstack-placement | 08:17 | |
| *** dims has quit IRC | 08:32 | |
| *** dims has joined #openstack-placement | 08:33 | |
| *** ttsiouts has joined #openstack-placement | 08:34 | |
| *** ttsiouts has quit IRC | 08:36 | |
| *** ttsiouts has joined #openstack-placement | 08:37 | |
| *** ttsiouts has quit IRC | 08:41 | |
| *** tetsuro has quit IRC | 09:08 | |
| *** ttsiouts has joined #openstack-placement | 09:09 | |
| *** tssurya has joined #openstack-placement | 09:27 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Per aggregate scheduling weight (spec) https://review.openstack.org/599308 | 09:38 |
| *** cdent has joined #openstack-placement | 09:39 | |
| cdent | moin | 09:42 |
| *** ttsiouts has quit IRC | 10:05 | |
| *** ttsiouts has joined #openstack-placement | 10:05 | |
| *** ttsiouts has quit IRC | 10:08 | |
| *** ttsiouts has joined #openstack-placement | 10:08 | |
| gibi | cdent: PS25 from nova and PS3 from https://review.openstack.org/#/c/620617/ passed more than 50 full nova functional run | 10:32 |
| cdent | gibi: I got 77 overnight without 620617 | 10:33 |
| gibi | seems your last change in https://review.openstack.org/#/c/619121 helped | 10:33 |
| cdent | yeah | 10:33 |
| cdent | I suspect that was the keystone | 10:33 |
| gibi | OK, then I can drop 620617 | 10:33 |
| cdent | we may wish to consider 620617 as a separate thing if we decide run_once is too inscrutable | 10:34 |
| gibi | cdent: I think I will not push for 620617 until another problem hits the same area | 10:35 |
| cdent | I think we can be sure of plenty problems to come of all sorts :) | 10:35 |
| gibi | :) | 10:35 |
| * gibi runs out for food | 10:36 | |
| cdent | I think you were right yesterday: whatever the root cause has been, we've improved a lot of code trying to get it right | 10:36 |
| cdent | now all we need are some reviewers | 10:43 |
| *** ttsiouts has quit IRC | 11:24 | |
| *** ttsiouts has joined #openstack-placement | 11:24 | |
| *** ttsiouts has quit IRC | 11:32 | |
| *** purplerbot has quit IRC | 11:32 | |
| *** purplerbot has joined #openstack-placement | 11:33 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Allow placement to start without a config file https://review.openstack.org/619049 | 11:42 |
| *** ttsiouts has joined #openstack-placement | 11:58 | |
| *** takashin has quit IRC | 12:25 | |
| *** ttsiouts has quit IRC | 12:30 | |
| *** ttsiouts has joined #openstack-placement | 12:31 | |
| *** ttsiouts has quit IRC | 12:32 | |
| *** ttsiouts has joined #openstack-placement | 12:32 | |
| *** Nel1x has joined #openstack-placement | 12:38 | |
| *** rubasov has quit IRC | 12:41 | |
| *** rubasov has joined #openstack-placement | 12:43 | |
| cdent | jaypipes: in the realm of tiny containers, can you look at https://review.openstack.org/#/c/619049/ | 12:46 |
| jaypipes | cdent: soon as I'm finished with my second and final ML response of the day. :P | 12:53 |
| *** e0ne has quit IRC | 12:53 | |
| cdent | jaypipes: if the second one as enjoyable as the first... | 12:54 |
| *** takashin has joined #openstack-placement | 13:00 | |
| jaypipes | cdent: not sure. see for yourself. ;) | 13:01 |
| * cdent reads | 13:02 | |
| cdent | yup | 13:04 |
| jaypipes | cdent: small request on https://review.openstack.org/#/c/619049. | 13:05 |
| cdent | cool, thanks | 13:05 |
| jaypipes | cdent: fix'r up and feel free to +W away | 13:05 |
| jaypipes | since giblet was +2 already | 13:05 |
| jaypipes | cdent: also, question... | 13:08 |
| jaypipes | cdent: are we requiring that version of oslo.cfg in our requirements already? | 13:08 |
| jaypipes | cdent: easier to ask then look it up :) | 13:08 |
| cdent | ah good catch | 13:09 |
| cdent | well, actually | 13:09 |
| cdent | the use of env and the use of no config is orthogonal | 13:09 |
| cdent | but probably worth updating the reqs anyway | 13:09 |
| cdent | I'll fix both things, and expand the commit message a bit | 13:10 |
| jaypipes | cdent: danke | 13:10 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Allow placement to start without a config file https://review.openstack.org/619049 | 13:29 |
| cdent | gibi, jaypipes ^ updated to be a bit more correct, getting requirements as well | 13:29 |
| gibi | cdent: looking... | 13:29 |
| gibi | cdent: +2 | 13:30 |
| cdent | thanks | 13:30 |
| *** e0ne has joined #openstack-placement | 13:31 | |
| jaypipes | cdent: +W | 13:34 |
| cdent | yay! | 13:34 |
| *** ttsiouts has quit IRC | 13:37 | |
| gibi | nova meeting in two minutes on #openstack-meeting | 13:58 |
| *** mriedem has joined #openstack-placement | 14:29 | |
| *** ttsiouts has joined #openstack-placement | 14:30 | |
| *** ttsiouts has quit IRC | 14:47 | |
| *** ttsiouts has joined #openstack-placement | 14:48 | |
| *** ttsiouts has quit IRC | 14:48 | |
| *** ttsiouts has joined #openstack-placement | 14:49 | |
| *** takashin has left #openstack-placement | 15:03 | |
| cdent | gibi, efried, edleafe, mriedem: I just spent some time perusing coverage results for facebook and found that there are a few methods in objects/resource_provider that have zero coverage (neither unit nor functional). Should I make a) one bug, b) multiple bugs, c) make an email. I don't have time to just fix them right now but don't want it forgotten | 15:24 |
| efried | cdent: facebook? | 15:25 |
| * cdent guesses efried had a good breakfast | 15:25 | |
| efried | I have no idea what you're talking about. | 15:26 |
| edleafe | cdent: I would prefer a single bug with a listing of the problematic methods | 15:27 |
| efried | but a) sounds good to me | 15:27 |
| efried | yeah, what edleafe said. | 15:27 |
| cdent | wfm, on it | 15:27 |
| efried | and/or a link to the coverage results | 15:27 |
| edleafe | wait - are we on StoryBoard? | 15:28 |
| * edleafe can never remember | 15:28 | |
| cdent | not yet | 15:28 |
| cdent | will link to some results | 15:29 |
| mriedem | single bug | 15:29 |
| *** mriedem is now known as mriedem_afk | 15:29 | |
| edleafe | this seems like a perfect fit for the stories/tasks model of SB | 15:30 |
| cdent | yes | 15:32 |
| *** ttsiouts has quit IRC | 15:32 | |
| *** ttsiouts has joined #openstack-placement | 15:33 | |
| cdent | there we go https://bugs.launchpad.net/nova/+bug/1805858 | 15:34 |
| openstack | Launchpad bug 1805858 in OpenStack Compute (nova) "placement/objects/resource_provider.py missing test coverage for several methods" [Medium,Triaged] | 15:34 |
| *** ttsiouts has quit IRC | 15:37 | |
| *** ttsiouts has joined #openstack-placement | 15:52 | |
| *** mriedem_afk is now known as mriedem | 16:22 | |
| *** e0ne has quit IRC | 16:26 | |
| *** rubasov has quit IRC | 16:28 | |
| *** e0ne has joined #openstack-placement | 16:38 | |
| *** e0ne has quit IRC | 16:48 | |
| *** ttsiouts has quit IRC | 16:51 | |
| *** ttsiouts has joined #openstack-placement | 16:52 | |
| *** ttsiouts has quit IRC | 16:56 | |
| openstackgerrit | Jack Ding proposed openstack/nova-specs master: Select cpu model from a list of cpu models https://review.openstack.org/620959 | 16:58 |
| *** helenafm has quit IRC | 17:06 | |
| guilhermesp | hello! I'm from openstack-ansible team and we are building our role to manage the installation of placement https://review.openstack.org/#/c/618820/ | 18:34 |
| guilhermesp | there are some valuable information that you guys could provide tho | 18:34 |
| guilhermesp | we appreciate any support from you | 18:35 |
| guilhermesp | there are some things that we are still add to that PR, for example, we are not sure which dependencies placement would need https://review.openstack.org/#/c/618820/35/vars/redhat-7.yml | 18:36 |
| efried | cdent: ^^ | 18:37 |
| efried | and mriedem ^^ | 18:37 |
| guilhermesp | also which port placement runs http://logs.openstack.org/20/618820/35/check/openstack-ansible-functional-centos-7/ec63f00/ara-report/result/65e95372-f3ff-4aff-853e-be25b379d6a4/ that we didn't set yet | 18:37 |
| cdent | guilhermesp: ideally it would run on :80/placement or :443/placement but if that is not an option, whatever you like, just make sure it gets set properly in the service catalog | 18:39 |
| cdent | guilhermesp: I'll give that stuff a fresh review tomorrow (off to dinner in a moment) but generally I think you're being a bit of pioneer here. Some of that stuff you'll need to just try and see what works | 18:40 |
| guilhermesp | thanks cdent ! Yeah, our role already deals with service catalog | 18:41 |
| guilhermesp | it's been challenging but i think we are doing a good progress | 18:41 |
| cdent | great to here | 18:41 |
| cdent | and hear too! | 18:41 |
| guilhermesp | and als, I think we can make things easy communicating the process of developing via irc channel | 18:41 |
| guilhermesp | also, is that the placement package https://github.com/openstack/placement/blob/4a2a26bf78c404ce962353f938d3582d901d1de6/setup.cfg#L2 ? | 18:42 |
| cdent | yes, that's its name on pypi | 18:43 |
| cdent | people thought just "placement" was too generic | 18:44 |
| guilhermesp | that's good. We assume for now that there're no dependencies so, I will just make the correction of the placement port on the PR and see what happens next :) | 18:45 |
| mnaser | guilhermesp: i remember this now but we're actually using some generic port for placement in os_nova .. we should probably use that same one | 18:52 |
| guilhermesp | I will take a look at our os_nova role mnaser ! | 18:53 |
| *** e0ne has joined #openstack-placement | 20:20 | |
| *** e0ne has quit IRC | 20:30 | |
| openstackgerrit | Merged openstack/placement master: Allow placement to start without a config file https://review.openstack.org/619049 | 20:33 |
| *** cdent has quit IRC | 21:40 | |
| dansmith | am I missing something or is https://developer.openstack.org/api-ref/placement/?expanded=update-resource-provider-aggregates-detail#update-resource-provider-aggregates very wrong? | 21:59 |
| dansmith | the first request and request example don't line up, amirite/ | 21:59 |
| *** mriedem is now known as mriedem_afk | 22:33 | |
| efried | dansmith: looks aight to me, I think. Is the concern that the list is named 'aggregates' in the table, but has no name in the sample? | 22:42 |
| efried | The alternative would be what, to leave that table cell blank? Not sure that's less confusing. | 22:42 |
| dansmith | maybe I'm not used to reading api-refs but yeah that looks really confusing to me | 22:43 |
| dansmith | like, the example below it looks like I would expect, | 22:43 |
| dansmith | yet the tables look basically the same | 22:43 |
| dansmith | and we have some single-key dict bodies in places which would look like the second example, but identical table to the first | 22:44 |
| dansmith | I would maybe change "aggregates" to "<body>" or something I guess | 22:44 |
| dansmith | anyway, we figured out our issue by just looking at a working client and seeing that it matched the example not the table, but... | 22:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!