Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-27
00:19:23 melwitt for this, it was about whether we can make the InstanceMapping.user_id field non-nullable. and I think because of the aforementioned things, we can't
00:19:48 melwitt no user_id on BuildRequest, and user_id not guaranteed to be set on RequestSpec
00:19:49 mriedem yonglihe: i replied in https://review.openstack.org/#/c/621474/ and i'm ok with the server_groups as a list since that seems to be the consensus
00:20:38 sean-k-mooney melwitt: if this change lands in stien you could make it non nullable in train
00:21:22 sean-k-mooney the only donwside to that would be for FFU you would have to run the online migration. although i assume this could be done as an offline migration too
00:21:47 melwitt sean-k-mooney: yeah. I don't understand whether it needs to be a major object version bump or not, but I can ask about that next cycle. because it would be good to make it non-nullable whenever we can
00:22:27 melwitt yes, offline (nova-manage) migration is the one that does the cell database reads
00:23:26 melwitt we're going to have a blocker migration in T also, to ensure all migrated, and then be able to remove transitionary online/active migration code
00:23:43 sean-k-mooney ok if its an offline data migration that is executed centrally on the controller that is not too much of a headache for ffu
00:23:57 melwitt right
00:24:52 sean-k-mooney thanks for +1'ing the os-vif releae patch by the way.
00:25:17 sean-k-mooney hopefully it will get merged soon but ill follow up tomorrow if not
00:25:46 sean-k-mooney night o/
00:29:36 melwitt np, gnight
00:44:24 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Fix changes-before values in an instance action test https://review.openstack.org/638849
00:46:54 openstackgerrit Adam Spiers proposed openstack/nova master: Convert driver supported capabilities to compute node provider traits https://review.openstack.org/538498
01:01:58 openstackgerrit Artom Lifshitz proposed openstack/nova master: RPC changes to prepare for NUMA live migration https://review.openstack.org/634605
01:01:58 openstackgerrit Artom Lifshitz proposed openstack/nova master: Make the use of the CastAsCall fixture configurable https://review.openstack.org/639428
01:01:59 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP needs more tests] Full NUMA live migration support https://review.openstack.org/634606
01:08:19 jaypipes melwitt: hmm, how user_id isn't guaranteed on request spec or build request is very odd to me...
01:08:40 jaypipes one would think both of those would by their very nature have a user associated with them. :)
01:09:05 melwitt jaypipes: build request doesn't even have a user_id field (wah wah). but request spec waits until conductor to set project_id and user_id
01:10:10 melwitt and yeah, I agree. I think it's because user_id wasn't "required" when they were first added as tables
01:10:31 melwitt there's definitely a user associated with them. just not recorded so well
01:21:59 openstackgerrit Ghanshyam Mann proposed openstack/nova stable/queens: DNM: For testing multinode and tempest-slow job https://review.openstack.org/623992
01:47:46 openstackgerrit Merged openstack/nova master: Mention SR-IOV cold migration limitation in admin docs https://review.openstack.org/603909
01:51:09 yonglihe mriedem, got that, working on it now, thanks.
02:03:58 openstackgerrit melanie witt proposed openstack/nova master: Add online data migration for populating user_id https://review.openstack.org/633351
02:03:58 openstackgerrit melanie witt proposed openstack/nova master: Add get_counts() to InstanceMappingList https://review.openstack.org/638072
02:03:59 openstackgerrit melanie witt proposed openstack/nova master: WIP Count instances from mappings and cores/ram from placement https://review.openstack.org/638073
02:03:59 openstackgerrit melanie witt proposed openstack/nova master: Use instance mappings to count server group members https://review.openstack.org/638324
03:53:29 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add compute service support for attach/detach root volume https://review.openstack.org/614750
04:21:06 openstackgerrit Merged openstack/python-novaclient master: Fix changes-before values in an instance action test https://review.openstack.org/638849
05:13:22 openstackgerrit Merged openstack/nova master: Avoid BadRequest error log on volume attachment https://review.openstack.org/581453
05:37:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove duplicate cleanup in functional tests https://review.openstack.org/636996
06:11:08 openstackgerrit Yongli He proposed openstack/nova master: Adds the server group info into show server detail API. https://review.openstack.org/621474
06:16:23 openstackgerrit Yongli He proposed openstack/nova master: Adds the server group info into show server detail API. https://review.openstack.org/621474
06:45:33 openstackgerrit Boxiang Zhu proposed openstack/nova master: Convert to raw format into rbd volume https://review.openstack.org/638080
06:59:30 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
06:59:31 openstackgerrit Michael Still proposed openstack/nova master: Move final bridge commands to privsep. https://review.openstack.org/639580
06:59:31 openstackgerrit Michael Still proposed openstack/nova master: Cleanup the _execute shim in nova/network. https://review.openstack.org/639581
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

Earlier   Later