| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-10 | |||
| 08:20:36 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Add random sleep between retry calls to placement https://review.openstack.org/567506 | |
| 08:25:27 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | |
| 08:25:28 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider cadidates https://review.openstack.org/567150 | |
| 08:25:29 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix broken allocation candidates with traits https://review.openstack.org/567151 | |
| 08:25:30 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508 | |
| 08:51:12 | openstackgerrit | Martin Midolesov proposed openstack/nova master: vmware:Validation video ram against the max video ram allowed. https://review.openstack.org/564193 | |
| 08:57:28 | openstackgerrit | jichenjc proposed openstack/nova master: Avoid warning log when image not exist https://review.openstack.org/567522 | |
| 08:57:32 | guest___ | hi | |
| 08:57:39 | guest___ | can someone help me with GPU? | |
| 08:57:47 | guest___ | virtualization? | |
| 09:03:47 | openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201 | |
| 09:14:43 | moshele | stephenfin: hi | |
| 09:14:49 | stephenfin | moshele: o/ | |
| 09:15:07 | moshele | stephenfin: how are you? long time since we talked :) | |
| 09:15:21 | stephenfin | Can't complain. Keeping busy :) | |
| 09:16:34 | moshele | stephenfin: the nvme stuff was merged in cinder last night https://review.openstack.org/#/c/505556/ can you review the nova side https://review.openstack.org/#/c/482640/ ? | |
| 09:17:11 | stephenfin | I sure can | |
| 09:17:50 | moshele | stephenfin: thank :) are you going to summit ? | |
| 09:18:22 | stephenfin | Yup! Got a talk on NUMA fun (which I really need to finish). You? | |
| 09:19:16 | moshele | stephenfin: me too, so I will see you there. I need to catch up on nova project :) | |
| 09:24:13 | openstackgerrit | Tsuyoshi Nagata proposed openstack/nova master: nova improvement of maximum attach volumes more than 26 vols https://review.openstack.org/567472 | |
| 09:31:14 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table. https://review.openstack.org/560832 | |
| 09:31:15 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstacenGroup object and api models. https://review.openstack.org/563375 | |
| 09:31:16 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401 | |
| 09:31:17 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | |
| 09:44:09 | guest___ | hi guys | |
| 10:12:44 | openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | |
| 10:13:24 | openstackgerrit | Martin Midolesov proposed openstack/nova master: vmware:Validation video ram against the max video ram allowed. https://review.openstack.org/564193 | |
| 10:22:10 | openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | |
| 10:25:54 | openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077 | |
| 10:29:26 | openstackgerrit | Merged openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | |
| 10:52:20 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (2) https://review.openstack.org/567557 | |
| 11:05:37 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | |
| 11:11:11 | openstackgerrit | Chris Dent proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116 | |
| 11:55:23 | bhagyashri_s | cdent: Hi, I am trying to write the functional test case for change https://review.openstack.org/#/c/560459 . I have written functinal test for the change # link http://paste.openstack.org/show/720747/ in /nova/tests/functional/test_servers.py | |
| 11:56:58 | cdent | hi bhagyashri_s. is it not doing what you want? | |
| 12:02:04 | cdent | bhagyashri_s: for what you are trying to test, a functional test in test_servers is probably not what you want | |
| 12:03:09 | cdent | because what you're trying to test is that the virt driver is creating the right data (in the ProviderTree), not create resource providers | |
| 12:03:26 | cdent | bhagyashri_s: basically what Eric says in his comment | |
| 12:10:18 | bhagyashri_s | cdent: Firstly the change is regarding the shared resource provider thing . When user configure the shared resource provider then at that time the DISK_GB inventory should report from shared rp and not from the compute node rp | |
| 12:11:41 | bhagyashri_s | cdent: and for that libvirt driver update_provider_tree() removes the DISK_GB inventory from the cn rp if the shared rp is configure | |
| 12:12:38 | cdent | so in your test you should need to create a server. instead you just need to confirm that the correct resource providers and inventory are created | |
| 12:12:49 | cdent | sorry: shouldn't need! | |
| 12:14:47 | bhagyashri_s | cdent so in my functional test case i am trying to update by calling the resource tackers update_provider_tree which basically updates the inventory internally using def _update() method which calls the drivers update_provider_tree() method and in my test case how should i instantiate the LibvirtDriver | |
| 12:17:06 | openstackgerrit | Petersingh Anburaj proposed openstack/nova master: Making consistent used of GiB and MiB in Doc https://review.openstack.org/559985 | |
| 12:18:53 | cdent | bhagyashri_s It sounds like you are trying to test from too high a level. Maybe try: Set up the sharing resource provider and put it an aggregate. Then create a ProviderTree representing a compute node, put it in aggregate too, then pass that to the method on the virt driver and make sure that the provider tree that comes back doesn't have local disk. I'm not sure of the setup you'll need to do for that, without doing a | |
| 12:18:54 | cdent | more digging | |
| 12:19:19 | cdent | if/when efried comes back you might check with him, but the virt drivers are an area I've done very little work with | |
| 12:23:13 | openstackgerrit | Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802 | |
| 12:23:21 | jroll | someone wake up mriedem before I forget to tell him a thing :P | |
| 12:25:01 | cdent | destroy all irc | |
| 12:27:19 | efried | ō/ | |
| 12:27:20 | bhagyashri_s | cdent: ok ty | |
| 12:27:37 | efried | reading back... | |
| 12:33:44 | efried | cdent, bhagyashri_s: Yes we want a unit test, but we also want a functional test. The functional test should do just what your paste shows. When we talked about this a couple of days ago, I offered to help you debug it IF you posted your WIP test case to the change set so I could download it and be at the same baseline, rather than copy/pasting from pastebin and trying to figure out where to put it and how to run it. | |
| 12:37:54 | bhagyashri_s | efried: because of some environmental issue iam not able to push patches so just paste the code in paste.openstack.org | |
| 12:42:13 | bhagyashri_s | efried: once it will resolved i will push it, I catch the issue in my functional test case, as the change is in libvirt code so through the functional test case the driver is not loaded and that's why the inventory is not properly updated so my doubt is how i will initiate the driver so that it report the correct inventory if the shared resource provider is configured | |
| 12:43:40 | bhagyashri_s | efried: sorry for inconvenience i am not able to push the patch | |
| 12:43:41 | bauzas | FWIW, folks I'm on PTO till end of the week | |
| 12:43:52 | bauzas | call it May-hem | |
| 12:44:19 | efried | bhagyashri_s: okay, can you pastebin the whole file including imports and tell me the path where you put it? | |
| 12:47:06 | bauzas | efried: jaypipes: cdent: I'm on slides mode for my YVR talk but something came out of my mind. Do we plan to have quota resources using Placement resource classes ? | |
| 12:47:24 | bauzas | efried: jaypipes: cdent: for example, we now have VGPU RC | |
| 12:47:39 | bauzas | efried: jaypipes: cdent: I guess operators would love to get quotas for vGPUs | |
| 12:47:50 | bauzas | so, how to help them ? | |
| 12:47:55 | efried | I still don't know what a quota is | |
| 12:48:13 | bauzas | efried: it's old story | |
| 12:48:30 | bauzas | efried: https://docs.openstack.org/nova/pike/admin/quotas.html | |
| 12:48:53 | bhagyashri_s | efried: #link http://paste.openstack.org/show/720752/ path: nova/tests/functional/test_servers.py | |
| 12:50:50 | bauzas | efried: and https://docs.openstack.org/nova/latest/user/quotas.html | |
| 12:53:24 | openstackgerrit | Hironori Shiina proposed openstack/nova master: ironic: Support resize and cold migration https://review.openstack.org/500677 | |
| 13:05:26 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Rename parameters in call() of nova/image/glance.py https://review.openstack.org/508533 | |
| 13:17:17 | moshele | melwitt: hi, the cinder changes for nvmeof is merged in cinder so can you review the nova change https://review.openstack.org/#/c/482640/ | |
| 13:21:31 | efried | bauzas: So you're thinking about implementing quotas in placement, such that a user and/or project ID is allowed only a certain amount of a given resource class? | |
| 13:22:03 | bauzas | efried: well, yes | |
| 13:22:44 | efried | bauzas: It seems like a reasonable idea. I don't recall having heard of any plans along those lines, but that doesn't mean it's not on the radar for someone. | |
| 13:22:57 | efried | bauzas: queue it up for Denver? | |
| 13:23:13 | bauzas | efried: my point is about the fact we created a new resource class, VGPU | |
| 13:23:21 | bauzas | efried: now the scheduler verifies it | |
| 13:23:24 | jaypipes | bauzas: that's a good discussion. I'd like to have melwitt in it, though. | |
| 13:23:29 | efried | agree | |
| 13:23:30 | jaypipes | bauzas: shall we wait for her? | |
| 13:23:42 | bauzas | jaypipes: sure, but I'm in vacations atm :) | |
| 13:23:46 | bauzas | in the beach :) | |
| 13:24:07 | bauzas | so when she's here, not sure I'll be here too :p | |
| 13:24:12 | bauzas | but anyway | |
| 13:24:19 | bauzas | nothing really important atm | |
| 13:24:24 | bauzas | it's more for my talk | |
| 13:24:24 | jaypipes | bauzas: alex_xu was also working on quotas and placement things | |
| 13:24:40 | bauzas | gtk | |
| 13:24:53 | bauzas | AFAIK, we don't support custom quotas, right? | |
| 13:26:08 | jaypipes | bauzas: I'm not sure any more. | |
| 13:26:14 | bauzas | yup | |
| 13:26:27 | bauzas | it was possible previously | |
| 13:26:40 | bauzas | but when we provided the quotas for cells v2, then we stopped that | |
| 13:26:54 | bauzas | that's what I remember at least | |
| 13:31:10 | jaypipes | bauzas: right. I was under the impression that we (nova) would start using the placement API for various usage queries. but I don't think that's been done yet. | |
| 13:35:56 | jaypipes | Kevin_Zheng: lmao 'All of a sudden, everyone uses the word "granular"' | |
| 13:36:02 | jaypipes | efried: ^ | |
| 13:38:15 | mriedem | stephenfin: bauzas: can one of you take a look at this? https://review.openstack.org/#/c/566367/ needs a final +2 and then we need to backport it through to ocata | |