Earlier  
Posted Nick Remark
#openstack-nova - 2019-02-27
06:59:32 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Detach/Attach root volume API changes https://review.openstack.org/623981
07:19:48 openstackgerrit Yongli He proposed openstack/nova master: Adds the server group info into show server detail API. https://review.openstack.org/621474
07:30:02 openstackgerrit Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.openstack.org/621476
08:25:51 openstackgerrit Sagar Waghmare proposed openstack/nova master: db sync prints stack-trace on invalid version https://review.openstack.org/639591
08:35:29 openstackgerrit Boxiang Zhu proposed openstack/nova master: Convert to raw format into rbd volume https://review.openstack.org/638080
08:46:53 openstackgerrit garyk proposed openstack/nova master: Better handle live migration abort https://review.openstack.org/635440
09:02:34 openstackgerrit Boxiang Zhu proposed openstack/nova master: Convert to raw format into rbd volume https://review.openstack.org/638080
09:20:32 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP status: Ensure templated connection URLs are rendered before use https://review.openstack.org/639607
09:24:40 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fup for the bandwidth series https://review.openstack.org/639159
09:24:40 openstackgerrit Balazs Gibizer proposed openstack/nova master: Handle placement error during re-schedule https://review.openstack.org/639608
09:32:18 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Detach/Attach root volume API changes https://review.openstack.org/623981
09:40:41 aspiers can anyone help me understand why this failed? http://logs.openstack.org/77/638677/2/gate/grenade-py3/e4fb83b/
11:25:04 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Detach/Attach root volume API changes https://review.openstack.org/623981
11:47:59 bauzas gibi: hola
11:48:32 bauzas gibi: do you know where we set VCPU inventories for https://review.openstack.org/#/c/631559/ ?
11:49:03 bauzas FWIW, we have a VCPU inventory that only accepts two VCPUs :)
11:49:12 bauzas (Pdb) compute_inventory
11:49:12 bauzas {u'VCPU': {u'allocation_ratio': 16.0, u'total': 2, u'reserved': 0, u'step_size': 1, u'min_unit': 1, u'max_unit': 2}, u'MEMORY_MB': {u'allocation_ratio': 1.5, u'total': 4096, u'reserved': 512, u'step_size': 1, u'min_unit': 1, u'max_unit': 4096}, u'DISK_GB': {u'allocation_ratio': 1.0, u'total': 128, u'reserved': 0, u'step_size': 1, u'min_unit': 1, u'max_unit': 128}}
11:50:13 bauzas oh my bad
11:52:07 sean-k-mooney bauzas: technically the vcpu inventory will accept 32 cpu allocation but each allcoation can only contin 2
11:52:28 bauzas yup, so that's not the problem I have then
11:52:42 bauzas super weird
11:52:55 sean-k-mooney what is the problem you are having?
11:53:02 bauzas placement returns me no alloc candidates but inventories are okay
11:53:15 sean-k-mooney is it a timeing issue?
11:53:38 sean-k-mooney and have we talked about this like 3 times in the past and we keep forgetting
11:54:45 bauzas sean-k-mooney: nope, I don't think it's a race
11:54:49 bauzas sean-k-mooney: http://paste.openstack.org/show/746416/
11:55:17 sean-k-mooney bauzas: ok i was wondering if it was the downstream issue you had been looking at on and off for the last few months
11:55:53 bauzas nope, working on the reshape series
11:56:20 bauzas http://paste.openstack.org/show/746418/
11:57:34 sean-k-mooney im not sure what im looking at.
11:58:15 bauzas sean-k-mooney: I modified the functest and now i have a NoValidHost http://paste.openstack.org/show/746419/
11:59:06 bauzas that's probably a PEBKAC
11:59:33 bauzas oh wait, it's the memory...
11:59:54 bauzas we ask for 2048GB of RAM in the flavor
12:00:08 bauzas we have 4096 * 1.5 - 512
12:00:23 sean-k-mooney GB i think you mean MB
12:00:40 sean-k-mooney * i hope you mean MB
12:00:41 bauzas whoops indeed
12:00:55 bauzas anyway, that's why I get a placement thingy
12:01:04 bauzas I need to create more room or ask for less
12:01:39 bauzas but then the question remains, where do we set inventories from the functional tests ?
12:01:55 sean-k-mooney ubutnu/fedroa and centos will work with 256mb and cirros will work with 64mb
12:02:43 sean-k-mooney its using the libvirt fake driver
12:02:51 sean-k-mooney so its in the driver code
12:03:56 bauzas bingo, found it
12:04:09 bauzas HostInfo()
12:04:20 sean-k-mooney ya that sound about right
12:05:58 bauzas yay, it works when changing kB_mem attribute \o/
12:27:38 aspiers can anyone give me some hints on how to debug failing grenade jobs?
12:31:19 openstackgerrit Sylvain Bauza proposed openstack/nova master: Use the correct mdev allocated from the pGPU https://review.openstack.org/636591
12:31:19 openstackgerrit Sylvain Bauza proposed openstack/nova master: libvirt: implement reshaper for vgpu https://review.openstack.org/599208
12:31:20 openstackgerrit Sylvain Bauza proposed openstack/nova master: Add functional test for libvirt vgpu reshape https://review.openstack.org/631559
12:31:20 openstackgerrit Sylvain Bauza proposed openstack/nova master: FUP: docs nit https://review.openstack.org/639647
12:32:46 bauzas efried: ^
12:34:41 gibi bauzas: this is where the 2 VCPU is coming from fakelibvirt.HostInfo
12:35:04 aspiers ah, logs/grenade.sh.txt.gz is the logfile I was looking for
12:35:09 gibi bauzas: ohh I see your figured it out
12:47:59 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421
12:48:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add remove_resources_from_instance_allocation to report client https://review.openstack.org/639653
12:50:00 jaypipes aspiers: have a friend jump on them.
12:50:20 aspiers :)
12:58:27 sean-k-mooney how do people feel about backporting https://review.openstack.org/#/q/topic:bug/1751923+(status:open+OR+status:merged)
12:59:46 sean-k-mooney mriedemn asked me to hold off for a while a when it merged but i have a downstream customer asking for this so im wondering if i should backport upstream or not?
13:01:18 sean-k-mooney i wont get around to starting the backport untill next week but input would be welcome.
13:03:13 tssurya stephenfin: I had a small doubt here: https://review.openstack.org/#/c/634600/9 regarding the doc samples; could you confirm ?
13:09:41 openstackgerrit Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310
13:13:31 sean-k-mooney tssurya: assuming you are correct that means we are missing a gate job to build the api samples
13:13:49 tssurya sean-k-mooney: yea that's what I think so too
13:14:33 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311
13:14:33 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543
13:14:34 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support server create with ports having resource request https://review.openstack.org/636360
13:14:34 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor _heal_allocations_for_instance to make place for port healing https://review.openstack.org/637953
13:14:35 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor _heal_allocations_for_instance (2) https://review.openstack.org/637954
13:14:35 openstackgerrit Balazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations https://review.openstack.org/637955
13:14:36 openstackgerrit Balazs Gibizer proposed openstack/nova master: cache neutron ports in heal allocation https://review.openstack.org/638207
13:15:14 sean-k-mooney i have never actully built the api-sampels personally so i dont know the answer to your question but ya my takeaway from that question is we shoudl fix them if need and add a gate job in the same patch to prevent future regressions
13:16:34 tssurya sean-k-mooney: yea thanks
13:21:38 artom tssurya, I think the stuff under doc/ is generated automatically, don't remember how though, it's been forever since I played with that..
13:22:13 artom Hrmm, although Matt had to add them manually in his path here https://review.openstack.org/#/c/631948/
13:22:22 artom OK, I have no idea, ignore me
13:25:45 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add remove_resources_from_instance_allocation to report client https://review.openstack.org/639653
13:25:46 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove port allocation during detach https://review.openstack.org/622421
13:25:46 openstackgerrit Balazs Gibizer proposed openstack/nova master: Record requester in the InstancePCIRequest https://review.openstack.org/625310
13:25:47 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add pf_interface_name tag to passthrough_whitelist https://review.openstack.org/625311
13:25:47 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ensure that bandwidth and VF are from the same PF https://review.openstack.org/623543
13:25:48 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support server create with ports having resource request https://review.openstack.org/636360
13:25:48 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor _heal_allocations_for_instance to make place for port healing https://review.openstack.org/637953
13:25:49 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor _heal_allocations_for_instance (2) https://review.openstack.org/637954
13:25:49 openstackgerrit Balazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations https://review.openstack.org/637955
13:25:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: cache neutron ports in heal allocation https://review.openstack.org/638207
13:40:24 tssurya artom: oh he added them manually ?
13:41:02 tssurya I was actually trying to review this: https://review.openstack.org/#/c/621474/26 and some things seemed off when added manually like the test samples and doc samples didn't match
13:41:35 tssurya anyways thanks artom I am confused too at this point
13:46:38 mriedem tssurya: from your question in https://review.openstack.org/#/c/634600/ it sounds like a bug if you want to report one and push a fix
13:54:14 tssurya mriedem: ack will do
13:54:52 mriedem also, nice review on https://review.openstack.org/#/c/621474/
13:55:18 tssurya :)
13:58:39 gibi mriedem: fixed your comments in https://review.openstack.org/#/c/622421 and split out the report client change to a separate patch

Earlier   Later