Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-28
08:21:36 bauzas jianghuaw_: (first valid config)
08:29:47 openstackgerrit Johannes Kulik proposed openstack/nova master: Fix spelling mistake of HTTPNotFound exception https://review.openstack.org/557187
08:32:29 jianghuaw_ bauzas, Do you mean we can't create different type vGPUs on different pgpu belong to a group?
08:33:07 bauzas jianghuaw_: a group is multiple pGPUs, right?
08:33:12 jianghuaw_ yes.
08:33:18 jianghuaw_ with same type
08:33:20 bauzas jianghuaw_: so, you can have one type per pGPU
08:33:38 bauzas jianghuaw_: no, not the same type, you just said there are multiple types per group
08:34:55 jianghuaw_ I mean a group can be multiple PGPUs which belong to the same PGPU type e.g. all are "Tesla M60"
08:37:06 jianghuaw_ Each PGPU can support multiple vGPU types; as we make group as the unit to expose vGPUs, we have to restrict each group having only one vgpu type enabled.
08:41:01 jianghuaw_ bauzas, with the new configure option, we can't enable multiple types in each group either.
08:43:46 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/534384
09:02:03 kashyap sean-k-mooney[m]: ^ Can you re-ACK it, please
09:02:46 kashyap (Since I moved the 'choices' stuff into config, I just removed the uppercase/lowercase check in driver.py; small mechanical change.)
09:16:34 kashyap imacdonn: stvnoyes1 Follow-up from last night, please update the versions of libvirt, QEMU, libguestfs for Oracle Linux here: https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix
09:17:34 openstackgerrit garyk proposed openstack/nova master: VMware: add log message for VIF info details https://review.openstack.org/557256
09:18:49 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
09:18:49 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
09:18:50 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
09:18:50 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
09:18:51 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
09:25:21 bauzas jianghuaw_: anyway, I don't want to rathole on Xen specifics
09:25:40 bauzas jianghuaw_: if you feel you don't need that spec, all fair, I'll make that conf opt libvirt-specific
09:28:01 openstackgerrit Bhagyashri Shewale proposed openstack/nova-specs master: Disallow rotation parameter 0 for 'createBackup' API https://review.openstack.org/511825
09:28:32 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove Ec2 object https://review.openstack.org/557150
09:31:02 openstackgerrit Jie Li proposed openstack/nova-specs master: Support volume-backed server rescue https://review.openstack.org/532410
09:32:54 jianghuaw_ bauzas, thanks.
09:54:17 openstackgerrit Bhagyashri Shewale proposed openstack/nova-specs master: Disallow rotation parameter 0 for 'createBackup' API https://review.openstack.org/511825
09:56:34 jianghuaw_ bauzas, In the queens vGPU spec, we have display heads as consumable resource. Now we will make it as a trait.
09:56:35 openstackgerrit jichenjc proposed openstack/nova master: Move update_task_state out of try/except https://review.openstack.org/557152
09:57:00 jianghuaw_ Should we just update the queens spec or should make a new spec in Rocky?
10:43:00 openstackgerrit Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775
11:03:28 openstackgerrit Claudiu Belu proposed openstack/nova master: WIP: replace spec with autospec https://review.openstack.org/557299
11:22:02 openstackgerrit Claudiu Belu proposed openstack/nova master: WIP: replace spec with autospec https://review.openstack.org/557299
11:38:20 priya__ Hi team, my code breaks with "flavor not found" issue. It is seen that instance_types table is empty. The same code works in Mitaka. Is there any issue/upgrade for pike?
11:55:45 Kevin_Zheng priya__ seems related with cells v2 structure,
11:58:11 Kevin_Zheng pike reads api_db for flavor, mitaka reads nova db, so it will work with mitaka but cannot work with pike
11:58:19 Kevin_Zheng should perform data migration
12:00:06 priya__ Kevin_Zheng flavor = db.flavor_get(context, inst_type_id) is the call which breaks. how do i perform data migration?
12:02:26 Kevin_Zheng priya__ how did you install your nova?
12:02:44 Kevin_Zheng with source code or packages?
12:03:20 Kevin_Zheng or devstack
12:03:32 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Speed things up and document them https://review.openstack.org/534382
12:03:32 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Make everything work with Python 3 https://review.openstack.org/556894
12:03:33 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Remove 'tools/releasenotes_tox.sh' https://review.openstack.org/534383
12:03:44 Kevin_Zheng another thing is that the default flavors are removed after mitaka
12:03:55 Kevin_Zheng so you have to create your own flavor before use
12:25:23 priya__ Kevin_Zheng through package
12:25:49 Kevin_Zheng Then I guess you should create your own flavor before use
12:26:07 Kevin_Zheng because there will be no default flavors after mitaka
12:28:05 priya__ Kevin_Zheng yes i have created a flavor, and that shows up in nova_api,flavors table. But it is not updated in instance_types.
12:28:35 priya__ Kevin_Zheng flavor = db.flavor_get(context, inst_type_id) refers to instance_types table and my code breaks here..i dont understand why
12:30:09 Kevin_Zheng where is this code?
12:31:00 priya__ this is my custom code
12:31:07 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Allow to specify granular CPU feature flags https://review.openstack.org/534384
12:31:22 kashyap alex_xu_: Thanks for the eagle eyes; I think '20' is the lucky number :-)
12:31:35 Kevin_Zheng OK then you access the wrong db
12:31:57 Kevin_Zheng because in Pike the flavor is not in that table anymore
12:32:17 Kevin_Zheng it moved to flavors in api_db
12:32:32 Kevin_Zheng you should query that
12:32:57 alex_xu_ kashyap: hah
12:33:22 priya__ Kevin_Zheng how do i change the database to access? because db.flavor_get is in nova api code
12:33:35 kashyap alex_xu_: Thank you!
12:33:52 Kevin_Zheng but you said it is your cutom code?
12:35:30 priya__ Kevin_Zheng The call is in my custom code, howevr the db.flavor_get method def refers to nova/db/api.py(1579)flavor_get()
12:37:06 Kevin_Zheng which nova api did your code call?
12:38:06 openstackgerrit Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487
12:40:47 Kevin_Zheng priya__ sorry I have to go
12:41:17 priya__ Kevin_Zheng thank you for the help.
12:41:40 priya__ Kevin_Zheng i have used the db api
12:41:51 gibi stephenfin: I left some questions in the numa aware switches spec https://review.openstack.org/#/c/541290
12:50:46 cdent jaypipes, stephenfin: quick test coverage win in placement one +2 already: https://review.openstack.org/#/c/513264/
12:50:58 jaypipes cdent: cool.
12:52:33 cdent thanks jaypipes
12:58:04 efried mikal: I do not accept your admonishment. I may have been mistaken or talked out of the more substantive issues, but taken at face value, those would have justified further work on the patch.
12:58:14 efried mikal: Thank you for the updates. Hugs!
13:00:40 jaypipes efried: so...
13:01:20 jaypipes efried: on https://review.openstack.org/#/c/557187/ I'm torn
13:01:40 jaypipes efried: we don't have a good set of unit tests for the placement HTTP handlers
13:01:56 cdent (on purpose)
13:02:13 jaypipes efried: and asking Johannes to create one from scratch is a bit much for a one-byte patch
13:02:32 jaypipes efried: I would just as much say let's merge the thing
13:02:53 efried That's fair, I didn't realize we didn't have a framework in place that would make it trivial (I didn't look).
13:03:13 jaypipes efried: yeah, we don't have unit tests for this stuff really.
13:03:34 jaypipes efried: that said, we could ask for a gabbit to be added that would force this code path
13:03:35 cdent Just so it is clear, the lack of unit tests is supposed to reflect that anything that ought to have a unit test is in a different file
13:03:39 jaypipes I think..
13:03:51 cdent jaypipes: it won't work without multiple processes
13:04:03 efried But in any case, not in this patch.
13:04:10 jaypipes cdent: right.
13:04:11 efried jaypipes, cdent: I flipped to +1.
13:04:11 cdent delicately controlled multiple processes
13:04:29 efried thanks for setting me straight there.
13:05:17 jaypipes efried: like I said, I'm a bit torn, but I think I land on the "just merge this" side.
13:05:39 efried jaypipes: I'm fine with that, especially given it's a clearly correct fix to a clear mistake in the code.
13:05:50 efried If writing the test were trivial, that would be one thing.
13:07:05 jaypipes efried: right, agreed.
13:09:59 openstackgerrit Surya Seetharaman proposed openstack/nova master: Allow scheduling only to enabled cells (Filter Scheduler) https://review.openstack.org/550527
13:10:00 openstackgerrit Surya Seetharaman proposed openstack/nova master: Update the cells FAQs and scheduler maintenance docs. https://review.openstack.org/556459
13:10:00 openstackgerrit Surya Seetharaman proposed openstack/nova master: Add --enable and --disable options to nova-manage update_cell https://review.openstack.org/555416

Earlier   Later