Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-27
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
14:00:01 gibi mriedem: the fup is also up to date https://review.openstack.org/#/c/639159
14:00:09 mriedem ok
14:01:01 gibi I have to spend the rest of the afternoon in downstream land so let's communicate via the code reviews :/
14:01:13 mriedem fare thee well
14:01:32 gibi mriedem: thanks I will try :)
14:13:31 sean-k-mooney mriedem: o/
14:14:37 mriedem ~o~
14:14:46 sean-k-mooney mriedem: i asked this before you joined today but do you still want me to hold off on backporting https://review.openstack.org/#/q/topic:bug/1751923+(status:open+OR+status:merged) upstream? i will need to start backporting them downstream next week but i would prefer to do it upstream
14:15:56 mriedem sean-k-mooney: if you're going to do the backport work anyway, can you start it upstream and then cherry pick the upstream backports for your downstream work? knowing that the upstream backports might sit awhile to bake
14:16:26 mriedem i'm not comfortable landing that on stable before it's been released on master and someone like cern or vexxhost has run them yet

Earlier   Later