| *** tjones has quit IRC | 00:01 | |
| *** baoli has quit IRC | 00:02 | |
| *** zenoway has quit IRC | 00:03 | |
| *** thorst has quit IRC | 00:06 | |
| *** harlowja has joined #openstack-nova | 00:09 | |
| *** penick has joined #openstack-nova | 00:12 | |
| *** ducttape_ has quit IRC | 00:12 | |
| *** EinstCrazy has joined #openstack-nova | 00:13 | |
| *** pixelbeat_ has joined #openstack-nova | 00:14 | |
| *** Daisy has quit IRC | 00:14 | |
| *** tjones1 has quit IRC | 00:14 | |
| *** zz_dimtruck is now known as dimtruck | 00:15 | |
| *** Daisy has joined #openstack-nova | 00:15 | |
| *** takashin has joined #openstack-nova | 00:16 | |
| *** Jeffrey4l has joined #openstack-nova | 00:20 | |
| *** Daisy has quit IRC | 00:22 | |
| *** aginwala has quit IRC | 00:22 | |
| *** oomichi has joined #openstack-nova | 00:23 | |
| *** Daisy has joined #openstack-nova | 00:23 | |
| *** EinstCrazy has quit IRC | 00:24 | |
| *** thorst has joined #openstack-nova | 00:26 | |
| *** suro-patz has quit IRC | 00:26 | |
| *** dimtruck is now known as zz_dimtruck | 00:27 | |
| *** Daisy has quit IRC | 00:27 | |
| *** aginwala has joined #openstack-nova | 00:29 | |
| *** zz_dimtruck is now known as dimtruck | 00:29 | |
| *** dims has quit IRC | 00:29 | |
| *** thorst has quit IRC | 00:29 | |
| *** suro-patz has joined #openstack-nova | 00:30 | |
| *** thorst has joined #openstack-nova | 00:30 | |
| *** lixiaoy1 has quit IRC | 00:31 | |
| sdague | auggy: if you want to respin, that would be great | 00:32 |
|---|---|---|
| *** zenoway has joined #openstack-nova | 00:34 | |
| *** mrkz has joined #openstack-nova | 00:36 | |
| *** mnestratov has quit IRC | 00:37 | |
| *** zenoway has quit IRC | 00:38 | |
| *** thorst has quit IRC | 00:38 | |
| auggy | sdague: alrighty then | 00:39 |
| * auggy has been doing alittle branch cleanup ;) | 00:39 | |
| *** zhangjn has joined #openstack-nova | 00:41 | |
| openstackgerrit | Sean Dague proposed openstack/nova: test https://review.openstack.org/258737 | 00:42 |
| *** otter768 has joined #openstack-nova | 00:42 | |
| *** pratikmallya has joined #openstack-nova | 00:42 | |
| sdague | that follow up button is super interesting | 00:42 |
| *** dimtruck is now known as zz_dimtruck | 00:43 | |
| *** wyang has quit IRC | 00:43 | |
| *** Jeffrey4l has quit IRC | 00:46 | |
| *** otter768 has quit IRC | 00:47 | |
| *** rcernin has quit IRC | 00:50 | |
| auggy | sdague: i think we also need to do a reno release notes thing, which i'm working on | 00:52 |
| *** EinstCrazy has joined #openstack-nova | 00:53 | |
| *** ctrath has quit IRC | 00:54 | |
| *** angdraug has quit IRC | 00:55 | |
| *** diogogmt has quit IRC | 00:56 | |
| *** diegows has quit IRC | 00:58 | |
| openstackgerrit | Michael Still proposed openstack/nova: Remove the old object_compat wrapper. https://review.openstack.org/257686 | 01:01 |
| *** thorst has joined #openstack-nova | 01:02 | |
| *** dguitarbite has joined #openstack-nova | 01:02 | |
| *** suro-patz has quit IRC | 01:08 | |
| *** zhenq has left #openstack-nova | 01:08 | |
| *** zenoway has joined #openstack-nova | 01:10 | |
| *** hemna is now known as hemnafk | 01:11 | |
| *** thorst has quit IRC | 01:12 | |
| *** thorst has joined #openstack-nova | 01:13 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API https://review.openstack.org/248989 | 01:13 |
| *** ljxiash has joined #openstack-nova | 01:14 | |
| *** zenoway has quit IRC | 01:15 | |
| *** pixelbeat_ has quit IRC | 01:15 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Split-network-plane-for-live-migration https://review.openstack.org/245005 | 01:16 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Check 'destination_type' instead of 'source_type' in _check_and_transform_bdm https://review.openstack.org/252836 | 01:17 |
| *** tjones has joined #openstack-nova | 01:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add instance.save() when handling reboot in init instance https://review.openstack.org/257692 | 01:19 |
| *** thorst has quit IRC | 01:21 | |
| *** junjie has quit IRC | 01:23 | |
| *** junjie has joined #openstack-nova | 01:24 | |
| *** zhiyan has quit IRC | 01:25 | |
| *** zhiyan has joined #openstack-nova | 01:28 | |
| *** ssurana has quit IRC | 01:28 | |
| *** zhenguo has joined #openstack-nova | 01:38 | |
| *** zenoway has joined #openstack-nova | 01:45 | |
| *** mriedem has joined #openstack-nova | 01:48 | |
| *** signed8bit has quit IRC | 01:49 | |
| *** zenoway has quit IRC | 01:49 | |
| *** tjones has quit IRC | 01:57 | |
| *** mtanino has joined #openstack-nova | 01:59 | |
| *** mrkz has quit IRC | 02:03 | |
| *** amotoki has joined #openstack-nova | 02:03 | |
| *** ijuwang has joined #openstack-nova | 02:03 | |
| *** amotoki_ has joined #openstack-nova | 02:05 | |
| *** thorst has joined #openstack-nova | 02:08 | |
| *** amotoki has quit IRC | 02:09 | |
| *** thorst has quit IRC | 02:09 | |
| *** thorst has joined #openstack-nova | 02:09 | |
| *** browne has quit IRC | 02:12 | |
| *** ijuwang has quit IRC | 02:13 | |
| *** c00281451 is now known as chenzeng | 02:18 | |
| *** thorst has quit IRC | 02:18 | |
| *** yamahata has quit IRC | 02:19 | |
| *** baoli has joined #openstack-nova | 02:20 | |
| *** baoli has quit IRC | 02:22 | |
| *** baoli has joined #openstack-nova | 02:22 | |
| *** zenoway has joined #openstack-nova | 02:24 | |
| tangchen | Hi alex_xu, would you please help to take a look at this patch ? https://review.openstack.org/#/c/202605/ | 02:25 |
| tangchen | thanks. | 02:25 |
| alex_xu | tangchen: yeh, will try today | 02:26 |
| tangchen | oomichi: Hi, oomichi-san, thank you for reviewing the patch. And please also help to review another one, they are similar. https://review.openstack.org/#/c/234691/ Thx. | 02:26 |
| tangchen | alex_xu: Thx. | 02:26 |
| alex_xu | tangchen: np | 02:26 |
| *** Jeffrey4l has joined #openstack-nova | 02:28 | |
| *** zenoway has quit IRC | 02:28 | |
| oomichi | tangchen: I am waiting for base patch merger now | 02:33 |
| *** penick has quit IRC | 02:34 | |
| *** baoli has quit IRC | 02:35 | |
| tangchen | oomichi: I'm sorry, by base patch, do you mean this ? https://review.openstack.org/#/c/202605/ | 02:35 |
| *** baoli has joined #openstack-nova | 02:36 | |
| *** dguitarbite has quit IRC | 02:40 | |
| *** dguitarbite has joined #openstack-nova | 02:41 | |
| *** otter768 has joined #openstack-nova | 02:43 | |
| *** gongysh_ has joined #openstack-nova | 02:44 | |
| *** gcb has joined #openstack-nova | 02:44 | |
| *** diogogmt has joined #openstack-nova | 02:45 | |
| *** aginwala_ has joined #openstack-nova | 02:45 | |
| oomichi | tangchen: yeah, that is | 02:47 |
| *** ducttape_ has joined #openstack-nova | 02:48 | |
| *** aginwala has quit IRC | 02:48 | |
| *** otter768 has quit IRC | 02:48 | |
| *** jyuso1 has joined #openstack-nova | 02:52 | |
| *** gongysh_ has quit IRC | 02:53 | |
| tangchen | oomichi: Sure. Thx. :) | 02:54 |
| *** vilobhmm11 has quit IRC | 02:55 | |
| *** amotoki_ has quit IRC | 02:57 | |
| *** harlowja has quit IRC | 02:58 | |
| *** ducttape_ has quit IRC | 02:59 | |
| *** baoli has quit IRC | 03:00 | |
| *** mrkz has joined #openstack-nova | 03:02 | |
| *** apoorvad has quit IRC | 03:02 | |
| *** Jeffrey4l has quit IRC | 03:08 | |
| openstackgerrit | yuntongjin proposed openstack/nova: Add API /servers/migrations:index/show https://review.openstack.org/258771 | 03:10 |
| *** ducttape_ has joined #openstack-nova | 03:11 | |
| *** browne has joined #openstack-nova | 03:13 | |
| *** mriedem has quit IRC | 03:13 | |
| *** yonglihe has joined #openstack-nova | 03:14 | |
| *** thorst has joined #openstack-nova | 03:15 | |
| *** Kevin_Zheng has quit IRC | 03:19 | |
| *** Kevin_Zheng has joined #openstack-nova | 03:21 | |
| *** thorst has quit IRC | 03:23 | |
| *** unicell1 has quit IRC | 03:24 | |
| *** jamielennox is now known as jamielennox|away | 03:26 | |
| *** links has joined #openstack-nova | 03:28 | |
| *** zhangjn has quit IRC | 03:28 | |
| *** jamielennox|away is now known as jamielennox | 03:31 | |
| *** ducttape_ has quit IRC | 03:32 | |
| *** dave-mccowan has quit IRC | 03:34 | |
| *** Marga__ has quit IRC | 03:37 | |
| *** Marga_ has joined #openstack-nova | 03:37 | |
| *** pixelbeat_ has joined #openstack-nova | 03:42 | |
| *** aginwala_ has quit IRC | 03:44 | |
| *** zhangjn has joined #openstack-nova | 03:45 | |
| *** pixelbeat_ has quit IRC | 03:46 | |
| *** logan- has joined #openstack-nova | 03:47 | |
| *** amotoki has joined #openstack-nova | 03:47 | |
| *** amotoki has quit IRC | 03:50 | |
| *** ducttape_ has joined #openstack-nova | 03:55 | |
| *** amotoki has joined #openstack-nova | 03:55 | |
| *** suro-patz has joined #openstack-nova | 03:56 | |
| *** zhangjn has quit IRC | 03:56 | |
| *** amotoki_ has joined #openstack-nova | 04:02 | |
| *** suro-patz has quit IRC | 04:02 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add some validations for BDM https://review.openstack.org/258788 | 04:02 |
| *** ijuwang has joined #openstack-nova | 04:05 | |
| *** amotoki has quit IRC | 04:05 | |
| *** ijuwang has quit IRC | 04:06 | |
| *** dguitarbite has quit IRC | 04:08 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add some validations for BDM https://review.openstack.org/258788 | 04:11 |
| *** vilobhmm11 has joined #openstack-nova | 04:12 | |
| *** pratikmallya has quit IRC | 04:16 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: vNIC tagging https://review.openstack.org/234904 | 04:17 |
| *** vincent_hou has joined #openstack-nova | 04:17 | |
| vincent_hou | alex_xu: Hi dude. Do you know how the yaml under releasenotes folder is generated? | 04:18 |
| *** ljxiash has quit IRC | 04:20 | |
| *** ljxiash has joined #openstack-nova | 04:20 | |
| *** thorst has joined #openstack-nova | 04:20 | |
| *** zenoway has joined #openstack-nova | 04:22 | |
| *** ducttape_ has quit IRC | 04:23 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Ensure consistency of ENV_REQUEST_ID https://review.openstack.org/254529 | 04:24 |
| *** ljxiash has quit IRC | 04:25 | |
| *** ljxiash has joined #openstack-nova | 04:26 | |
| *** zenoway has quit IRC | 04:26 | |
| *** thorst has quit IRC | 04:28 | |
| auggy | vincent_hou: http://docs.openstack.org/developer/reno/usage.html#creating-new-release-notes | 04:30 |
| vincent_hou | auggy: Thx. | 04:31 |
| *** zhangjn has joined #openstack-nova | 04:31 | |
| auggy | vincent_hou: np :) | 04:31 |
| *** mriedem has joined #openstack-nova | 04:32 | |
| *** otter768 has joined #openstack-nova | 04:44 | |
| alex_xu | vincent_hou: yeh, just as auggy pointed | 04:46 |
| *** gongysh_ has joined #openstack-nova | 04:48 | |
| openstackgerrit | yuntongjin proposed openstack/nova: Add migration progress detail in DB https://review.openstack.org/258797 | 04:49 |
| vincent_hou | alex_xu: yep | 04:49 |
| *** otter768 has quit IRC | 04:49 | |
| *** Nakato has quit IRC | 04:51 | |
| *** Nakato has joined #openstack-nova | 04:51 | |
| *** gongysh_ has quit IRC | 04:52 | |
| *** gcb has quit IRC | 04:52 | |
| *** boris-42_ has quit IRC | 04:53 | |
| *** zenoway has joined #openstack-nova | 04:58 | |
| *** gongysh_ has joined #openstack-nova | 05:01 | |
| *** zenoway has quit IRC | 05:02 | |
| *** gus has quit IRC | 05:04 | |
| *** jwcroppe has joined #openstack-nova | 05:05 | |
| *** zhangjn has quit IRC | 05:08 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient: Add wrapper classes for return-request-id-to-caller https://review.openstack.org/258802 | 05:08 |
| *** mtreinish has quit IRC | 05:08 | |
| *** mtreinish has joined #openstack-nova | 05:17 | |
| *** signed8bit has joined #openstack-nova | 05:20 | |
| *** diogogmt has quit IRC | 05:21 | |
| *** thorst has joined #openstack-nova | 05:25 | |
| *** zhangjn has joined #openstack-nova | 05:29 | |
| *** mtanino has quit IRC | 05:30 | |
| *** yonglihe has quit IRC | 05:31 | |
| *** thorst has quit IRC | 05:33 | |
| *** diogogmt has joined #openstack-nova | 05:33 | |
| *** mrkz has quit IRC | 05:36 | |
| *** Thelo has joined #openstack-nova | 05:38 | |
| Thelo | Hi All, I made a change in os_brick and nova. How can I run the nova unit test ( tox -v -epy27) and take the change into the library into account? Besides, If I want to commit both, should I wait for the part in os_brick to be accepted? | 05:38 |
| *** gongysh_ has quit IRC | 05:43 | |
| *** gus has joined #openstack-nova | 05:49 | |
| *** vilobhmm11 has quit IRC | 05:50 | |
| *** vincent_hou has quit IRC | 05:51 | |
| *** ankit_ag has joined #openstack-nova | 05:51 | |
| *** vilobhmm11 has joined #openstack-nova | 05:51 | |
| *** moshele has joined #openstack-nova | 05:52 | |
| *** vilobhmm11 has quit IRC | 05:55 | |
| *** vilobhmm11 has joined #openstack-nova | 05:58 | |
| *** aginwala has joined #openstack-nova | 06:00 | |
| *** aginwala has quit IRC | 06:05 | |
| openstackgerrit | yuntongjin proposed openstack/nova: write live migration progress detail to DB in migration monitor https://review.openstack.org/258813 | 06:09 |
| *** asselin_ has quit IRC | 06:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/252197 | 06:11 |
| *** asselin_ has joined #openstack-nova | 06:11 | |
| *** davidlenwell has quit IRC | 06:12 | |
| *** davidlenwell has joined #openstack-nova | 06:14 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 06:16 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add support for API microversion 2.13 https://review.openstack.org/257703 | 06:25 |
| *** haomaiwang has joined #openstack-nova | 06:27 | |
| *** signed8bit_ZZZzz has quit IRC | 06:30 | |
| *** thorst has joined #openstack-nova | 06:31 | |
| *** pratikmallya has joined #openstack-nova | 06:32 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI: Fix VIF plug and unplug problem https://review.openstack.org/254022 | 06:37 |
| *** haomaiwang has quit IRC | 06:37 | |
| *** 5EXAAIFVW has joined #openstack-nova | 06:38 | |
| *** thorst has quit IRC | 06:38 | |
| *** Jeffrey4l has joined #openstack-nova | 06:40 | |
| *** otter768 has joined #openstack-nova | 06:45 | |
| *** suro-patz has joined #openstack-nova | 06:48 | |
| *** otter768 has quit IRC | 06:50 | |
| *** liusheng has quit IRC | 06:52 | |
| *** liusheng has joined #openstack-nova | 06:52 | |
| *** moshele has quit IRC | 06:52 | |
| tangchen | Hi, oomichi, alex_xu, I'm sorry the old problem again. Shall we agree on the instance action name, using triggerCrashDump or trigger_crash_dump ? | 06:55 |
| tangchen | oomichi, alex_xu: I personally prefer trigger_crash_dump because it is the same with API name. | 06:55 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Allow multiple volume attachments https://review.openstack.org/193135 | 06:55 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 06:55 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: obj: Allow multiple volume attachments https://review.openstack.org/193133 | 06:55 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: db: Allow multiple volume attachments https://review.openstack.org/153033 | 06:55 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 06:56 |
| alex_xu | tangchen: emm...I remember we said use 'trigger_crash_dump' last time | 06:56 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Pass host when call attach to Cinder https://review.openstack.org/256273 | 06:56 |
| tangchen | alex_xu: But oomichi gave me a -1 when I did so. So maybe we should make a clear decision again. | 06:56 |
| *** zenoway has joined #openstack-nova | 06:56 | |
| tangchen | oomichi: Hi, oomichi-san, are you around ? | 06:57 |
| *** ljxiash has quit IRC | 06:57 | |
| alex_xu | tangchen: oops, maybe we mis-understand oomichi last time | 06:57 |
| oomichi | tangchen: alex_xu: hello | 06:57 |
| *** ljxiash has joined #openstack-nova | 06:57 | |
| tangchen | Hi oomichi-san, I'm sorry maybe we should discuss about the instance action naming style once again. | 06:58 |
| *** garyk has quit IRC | 06:58 | |
| oomichi | tangchen: no problem, will catch the discussion | 06:58 |
| tangchen | oomichi, alex_xu: Alex and I would prefer the name to be trigger_crash_dump so that it is the same with the API name. | 06:58 |
| alex_xu | yup | 06:59 |
| alex_xu | that sounds good chocie for consistent | 06:59 |
| tangchen | oomichi, alex_xu: And from now on, I think new actions should be named in this style. Maybe also add comment into the source file. | 07:00 |
| alex_xu | tangchen: if we have agreement on that, the comment is really a good idea | 07:00 |
| *** zenoway has quit IRC | 07:00 | |
| *** 5EXAAIFVW has quit IRC | 07:01 | |
| *** ljxiash_ has joined #openstack-nova | 07:01 | |
| Thelo | Hi All, I made a change in os_brick and nova. How can I run the nova unit test ( tox -v -epy27) and take the change into the library into account? Besides, If I want to commit both, should I wait for the part in os_brick to be accepted? | 07:01 |
| *** haomaiwang has joined #openstack-nova | 07:01 | |
| oomichi | tangchen: alex_xu: action name in instance_actions.py are used in the recode only, not REST API layer, right? | 07:01 |
| oomichi | s/recode/record/ | 07:02 |
| *** ljxiash has quit IRC | 07:02 | |
| alex_xu | oomichi: we can get that from the api https://github.com/openstack/nova/blob/master/doc/api_samples/os-instance-actions/instance-action-get-resp.json#L3 | 07:02 |
| oomichi | alex_xu: ah, I misunderstood the point. yeah, it is nice to make names consistent as possible | 07:03 |
| oomichi | alex_xu: to REST API | 07:04 |
| alex_xu | oomichi: cool, thanks | 07:04 |
| oomichi | tangchen: sorry about my mess | 07:04 |
| *** rk4n has joined #openstack-nova | 07:04 | |
| alex_xu | tangchen: appreciate if you can add coment to say we use same naming style as rest api in the code | 07:04 |
| oomichi | tangchen: alex_xu: trigger_crash_dump is the best on instance_actions.py, right? | 07:05 |
| alex_xu | s/coment/comment/ | 07:05 |
| alex_xu | oomichi: yup | 07:05 |
| tangchen | oomichi, alex_xu: Sure, then I'll rename it to trigger_crash_dump. | 07:06 |
| tangchen | oomichi, alex_xu: Thank you for your patient. :) | 07:06 |
| alex_xu | tangchen: thank you for update also :) | 07:06 |
| tangchen | alex_xu: And BTW, the mox in the test file, the name mox is used everywhere... | 07:07 |
| tangchen | alex_xu: If we use mock instead, I need to send another patch to change them all, I think. | 07:07 |
| tangchen | alex_xu: do you mean that ? | 07:07 |
| alex_xu | tangchen: no, not for the name, it is for we should mock object like this https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute_api.py#L1495 | 07:09 |
| alex_xu | tangchen: you just need use mock in your new unittest, you needn't change other all | 07:09 |
| alex_xu | tangchen: we said we will use mock instead of mox in the new unittest | 07:09 |
| *** vilobhmm11 has quit IRC | 07:10 | |
| *** deepakcs has joined #openstack-nova | 07:10 | |
| alex_xu | tangchen: sorry, I have to go, but will be back after hour. let me know if you still have trouble with that. | 07:11 |
| tangchen | alex_xu: One sec for me to see that. | 07:11 |
| tangchen | alex_xu: Thx. :) | 07:11 |
| alex_xu | tangchen: np | 07:12 |
| *** rdopiera has joined #openstack-nova | 07:13 | |
| openstackgerrit | yuntongjin proposed openstack/python-novaclient: Add nova client server-migration-list and server-migration-show https://review.openstack.org/258841 | 07:14 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: Centralise floating ip options https://review.openstack.org/257087 | 07:23 |
| *** irenab has quit IRC | 07:24 | |
| *** haomaiwang has quit IRC | 07:26 | |
| *** tobe has joined #openstack-nova | 07:27 | |
| *** rk4n has quit IRC | 07:28 | |
| *** haomaiwang has joined #openstack-nova | 07:29 | |
| *** zenoway has joined #openstack-nova | 07:32 | |
| oomichi | bauzas: morning | 07:33 |
| *** jwcroppe has quit IRC | 07:35 | |
| *** thorst has joined #openstack-nova | 07:35 | |
| *** rha has joined #openstack-nova | 07:36 | |
| *** rha has quit IRC | 07:36 | |
| *** rha has joined #openstack-nova | 07:36 | |
| *** zenoway has quit IRC | 07:36 | |
| *** dedery has joined #openstack-nova | 07:37 | |
| *** rk4n has joined #openstack-nova | 07:39 | |
| *** ljxiash_ has quit IRC | 07:39 | |
| *** ljxiash has joined #openstack-nova | 07:40 | |
| *** ljxiash_ has joined #openstack-nova | 07:42 | |
| *** thorst has quit IRC | 07:43 | |
| *** ljxiash__ has joined #openstack-nova | 07:43 | |
| *** rk4n_ has joined #openstack-nova | 07:43 | |
| *** ljxiash has quit IRC | 07:44 | |
| *** rk4n has quit IRC | 07:45 | |
| *** jwcroppe has joined #openstack-nova | 07:46 | |
| *** dedery has quit IRC | 07:46 | |
| *** jwcroppe has quit IRC | 07:46 | |
| *** ljxiash_ has quit IRC | 07:46 | |
| *** jwcroppe has joined #openstack-nova | 07:46 | |
| *** ljxiash has joined #openstack-nova | 07:47 | |
| *** dedery has joined #openstack-nova | 07:48 | |
| *** ljxiash__ has quit IRC | 07:50 | |
| *** gjayavelu has joined #openstack-nova | 07:50 | |
| *** moshele has joined #openstack-nova | 07:51 | |
| *** jwcroppe has quit IRC | 07:51 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/nova: Fix multipath iSCSI encrypted volume attach failure https://review.openstack.org/196482 | 07:51 |
| *** ljxiash has quit IRC | 07:51 | |
| *** suro-patz has quit IRC | 07:52 | |
| *** cfriesen has quit IRC | 07:52 | |
| *** ljxiash has joined #openstack-nova | 07:52 | |
| *** rk4n has joined #openstack-nova | 07:53 | |
| *** rk4n_ has quit IRC | 07:55 | |
| *** ljxiash has quit IRC | 07:56 | |
| *** ljxiash has joined #openstack-nova | 07:57 | |
| *** rk4n_ has joined #openstack-nova | 07:58 | |
| *** rk4n has quit IRC | 08:00 | |
| *** takashin has left #openstack-nova | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** zenoway has joined #openstack-nova | 08:08 | |
| *** scheuran has joined #openstack-nova | 08:08 | |
| *** alexschm has joined #openstack-nova | 08:09 | |
| *** gjayavelu has quit IRC | 08:09 | |
| *** mjura has joined #openstack-nova | 08:11 | |
| *** mjura_ has joined #openstack-nova | 08:11 | |
| *** zenoway has quit IRC | 08:12 | |
| *** irenab has joined #openstack-nova | 08:12 | |
| *** unicell has joined #openstack-nova | 08:13 | |
| *** rk4n_ has quit IRC | 08:14 | |
| *** browne has quit IRC | 08:15 | |
| *** thorst has joined #openstack-nova | 08:16 | |
| *** lpetrut has joined #openstack-nova | 08:17 | |
| *** unicell1 has joined #openstack-nova | 08:17 | |
| *** unicell has quit IRC | 08:18 | |
| *** macsz has quit IRC | 08:22 | |
| *** thorst has quit IRC | 08:23 | |
| *** e0ne has joined #openstack-nova | 08:23 | |
| *** rcernin has joined #openstack-nova | 08:26 | |
| *** markus_z has joined #openstack-nova | 08:27 | |
| *** klkumar has joined #openstack-nova | 08:32 | |
| *** ianbrown has quit IRC | 08:36 | |
| bauzas | good morning Nova | 08:36 |
| bauzas | oomichi: morning | 08:36 |
| oomichi | bauzas: morning | 08:37 |
| *** ianbrown has joined #openstack-nova | 08:37 | |
| oomichi | bauzas: do you know filters "same_host" and "different_host" are enabled on the gate, or not? | 08:37 |
| *** rk4n has joined #openstack-nova | 08:38 | |
| bauzas | oomichi: nope, they aren't AFAIK | 08:38 |
| oomichi | bauzas: I am adding the corresponding test to Tempest, but the filters dont work | 08:38 |
| *** pratikmallya has quit IRC | 08:38 | |
| bauzas | oomichi: you can check if SameHostFilter and DifferentHostFilter are in the nova.conf for the gate | 08:38 |
| *** e0ne has quit IRC | 08:38 | |
| markus_z | The gate failure rates had a big increase 3 hours ago. Does anyone already know more? | 08:39 |
| *** alexschm_ has joined #openstack-nova | 08:39 | |
| bauzas | oomichi: tbh, I'd like to find a newcomer for helping us to have functional tests for the filters... | 08:39 |
| markus_z | http://grafana.openstack.org/dashboard/db/tempest-failure-rate | 08:39 |
| oomichi | bauzas: these filters should be on scheduler_default_filters ? | 08:39 |
| bauzas | oomichi: no, they aren't but lemme doublecheck | 08:39 |
| *** zenoway has joined #openstack-nova | 08:40 | |
| *** e0ne has joined #openstack-nova | 08:40 | |
| *** rcernin has quit IRC | 08:40 | |
| oomichi | bauzas: the tempest patch is https://review.openstack.org/#/c/257660/ | 08:40 |
| bauzas | oomichi: they aren't https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L56-L67 | 08:40 |
| *** rcernin has joined #openstack-nova | 08:41 | |
| *** alexsch__ has joined #openstack-nova | 08:41 | |
| oomichi | bauzas: thanks so much!! | 08:41 |
| oomichi | bauzas: I will post a patch for devstack for enabling them | 08:41 |
| oomichi | bauzas: just for the development | 08:41 |
| openstackgerrit | Oleg Bondarev proposed openstack/nova: Live migration: wait for vif-plugged event on pre live migration https://review.openstack.org/246910 | 08:42 |
| *** alexschm has quit IRC | 08:42 | |
| *** alexschm_ has quit IRC | 08:43 | |
| *** Marga_ has quit IRC | 08:45 | |
| openstackgerrit | wanghao proposed openstack/nova: Fix invalid Image&block device mapping combination https://review.openstack.org/171984 | 08:45 |
| *** astupnikov has joined #openstack-nova | 08:45 | |
| *** otter768 has joined #openstack-nova | 08:46 | |
| *** andrewbogott has quit IRC | 08:48 | |
| *** danpb has joined #openstack-nova | 08:48 | |
| oomichi | bauzas: one more question, is it acceptable to change https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L56-L67 directly instead of devstack change? | 08:49 |
| oomichi | bauzas: I guess no, because of default config change. right? | 08:49 |
| bauzas | oomichi: that's something we should discuss | 08:50 |
| bauzas | but given SameHost and DifferentHost are a bit legacy because of the server groups, I dunno | 08:50 |
| *** otter768 has quit IRC | 08:50 | |
| bauzas | I know that some deployers use them tho | 08:51 |
| oomichi | bauzas: yeah, and some deployers don't use. so if depending on the default config, something happens | 08:51 |
| bauzas | yup | 08:52 |
| bauzas | oomichi: that's why I'd like to functionally test all the filters and hints | 08:52 |
| oomichi | bauzas: without tempest? | 08:52 |
| *** e0ne has quit IRC | 08:53 | |
| openstackgerrit | Michal Pryc proposed openstack/nova: Implements proper UUID format for test_compute_cells and test_compute_utils. https://review.openstack.org/258878 | 08:53 |
| oomichi | johnthetubaguy: hello | 08:54 |
| oomichi | johnthetubaguy: I have a question about how to change default filters. | 08:55 |
| oomichi | johnthetubaguy: can we add SameHostFilter and DifferentHostFilter to https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L56-L67 ? | 08:55 |
| oomichi | johnthetubaguy: or should we change devstack for avoiding the default filter change on nova? | 08:55 |
| bauzas | oomichi: yup, I don't feel we need a full scenario like Tempest does | 08:56 |
| bauzas | oomichi: and I'd prefer that in-tree | 08:56 |
| *** patrickeast has quit IRC | 08:56 | |
| *** andrewbogott has joined #openstack-nova | 08:56 | |
| oomichi | bauzas: we can verify scheduler_hint works or not by actual openstack by Temepst | 08:57 |
| oomichi | bauzas: on the tempest test, we check the actual node of server which is created with scheduler_hint. | 08:57 |
| bauzas | oomichi: I guess you won't be able to attend the midcycle? | 08:57 |
| bauzas | oomichi: because that's something we could discuss there | 08:57 |
| bauzas | oomichi: I see | 08:58 |
| oomichi | bauzas: I cannot attend unfortunately. | 08:58 |
| bauzas | oomichi: well, it's an open point, I'm open to comments :) | 08:58 |
| oomichi | bauzas: because I will move to US next week. | 08:58 |
| bauzas | oomichi: no worries, I just feel we should somehow do that sooner or later | 08:58 |
| bauzas | oomichi: hah no worries | 08:59 |
| *** Kevin_Zheng has quit IRC | 08:59 | |
| openstackgerrit | Michal Pryc proposed openstack/nova: Implements proper UUID format for test_compute_cells and test_compute_utils. https://review.openstack.org/258878 | 08:59 |
| *** haomaiwang has quit IRC | 09:01 | |
| *** alexsch__ has quit IRC | 09:01 | |
| *** andrewbogott has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| *** matrohon has joined #openstack-nova | 09:01 | |
| oomichi | bauzas: I posted https://review.openstack.org/258882 on devstack | 09:02 |
| *** omnipresent has joined #openstack-nova | 09:03 | |
| *** alexschm has joined #openstack-nova | 09:03 | |
| bauzas | oomichi: I wonder if I wouldn't make sense to enable the flags for the specific Tempest builds? | 09:03 |
| *** briancurtin has quit IRC | 09:04 | |
| *** alexschm has quit IRC | 09:04 | |
| *** alexschm has joined #openstack-nova | 09:05 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI:Resolve Nova/Neutron race condition https://review.openstack.org/241127 | 09:05 |
| *** zenoway has quit IRC | 09:05 | |
| *** patrickeast has joined #openstack-nova | 09:06 | |
| *** lpetrut has quit IRC | 09:06 | |
| *** amotoki_ has quit IRC | 09:06 | |
| *** Kevin_Zheng has joined #openstack-nova | 09:07 | |
| *** dedery has quit IRC | 09:07 | |
| *** andrewbogott has joined #openstack-nova | 09:08 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add support for API microversion 2.13 https://review.openstack.org/257703 | 09:08 |
| *** derekh has joined #openstack-nova | 09:09 | |
| *** shuquan has joined #openstack-nova | 09:09 | |
| *** jistr has joined #openstack-nova | 09:09 | |
| *** ndipanov has joined #openstack-nova | 09:09 | |
| openstackgerrit | Rui Chen proposed openstack/python-novaclient: Fix Resource.__eq__ mismatch semantics of object equal https://review.openstack.org/251020 | 09:09 |
| *** shuquan has quit IRC | 09:11 | |
| *** briancurtin has joined #openstack-nova | 09:12 | |
| *** yassine__ has joined #openstack-nova | 09:13 | |
| *** sahid has joined #openstack-nova | 09:14 | |
| *** patrickeast has quit IRC | 09:15 | |
| *** sahid has quit IRC | 09:15 | |
| *** shuquan has joined #openstack-nova | 09:15 | |
| *** sahid has joined #openstack-nova | 09:16 | |
| *** andrewbogott has quit IRC | 09:17 | |
| *** briancurtin has quit IRC | 09:17 | |
| *** dedery has joined #openstack-nova | 09:17 | |
| *** Kevin_Zheng has quit IRC | 09:19 | |
| *** betherly has quit IRC | 09:19 | |
| *** thorst has joined #openstack-nova | 09:20 | |
| oomichi | bauzas: IIUC, devstack is for development/test and that is reason why devstack is under QA project | 09:22 |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add support for API microversion 2.13 https://review.openstack.org/257703 | 09:22 |
| oomichi | bauzas: so it is fine to change devstack for general tempest, not specific | 09:22 |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add support for API microversion 2.13 https://review.openstack.org/257703 | 09:24 |
| *** jlanoux has joined #openstack-nova | 09:25 | |
| *** zigo has quit IRC | 09:25 | |
| *** zigo_ has joined #openstack-nova | 09:26 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: centralize section: "keymgr" https://review.openstack.org/255044 | 09:27 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: config options: centralize section "keymgr" https://review.openstack.org/258893 | 09:27 |
| *** cooldharma06 has left #openstack-nova | 09:27 | |
| *** thorst has quit IRC | 09:27 | |
| *** salv-orlando has joined #openstack-nova | 09:28 | |
| *** shuquan has quit IRC | 09:28 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Do not update links for 'versions' tests https://review.openstack.org/257210 | 09:30 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: centeralized conf:compute/emphemeral_storage_encryption https://review.openstack.org/257176 | 09:31 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Allow to update 'v2.1' links in sample files https://review.openstack.org/257669 | 09:31 |
| openstackgerrit | Jianghua Wang proposed openstack/nova: xenapi: OVS agent updates the wrong port when using XenServer + Neutron https://review.openstack.org/242846 | 09:32 |
| gmann | alex_xu: johnthetubaguy : rebased for conflict. need +A - https://review.openstack.org/#/c/257210/3 https://review.openstack.org/#/c/257669/2 | 09:32 |
| *** amotoki has joined #openstack-nova | 09:33 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Prototype: get-valid-server-state https://review.openstack.org/258898 | 09:34 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 09:34 |
| *** Jeffrey4l has quit IRC | 09:34 | |
| *** zhangjn has quit IRC | 09:36 | |
| *** tobe has quit IRC | 09:38 | |
| *** betherly has joined #openstack-nova | 09:38 | |
| *** pratikmallya has joined #openstack-nova | 09:39 | |
| *** patrickeast has joined #openstack-nova | 09:39 | |
| *** shuquan has joined #openstack-nova | 09:40 | |
| *** haomaiwang has quit IRC | 09:40 | |
| *** tobe has joined #openstack-nova | 09:40 | |
| *** tobe has quit IRC | 09:41 | |
| *** tobe has joined #openstack-nova | 09:41 | |
| *** andrewbogott has joined #openstack-nova | 09:42 | |
| *** pratikmallya has quit IRC | 09:43 | |
| *** Kevin_Zheng has joined #openstack-nova | 09:43 | |
| *** amotoki has quit IRC | 09:44 | |
| *** briancurtin has joined #openstack-nova | 09:44 | |
| *** dedery has quit IRC | 09:45 | |
| *** dedery has joined #openstack-nova | 09:45 | |
| *** tobe has quit IRC | 09:46 | |
| *** openstackgerrit has quit IRC | 09:47 | |
| *** haomaiwang has joined #openstack-nova | 09:47 | |
| *** openstackgerrit has joined #openstack-nova | 09:47 | |
| openstackgerrit | Nan Zhang proposed openstack/nova: live-migration: Update instance availability_zone https://review.openstack.org/254041 | 09:50 |
| *** cdent has joined #openstack-nova | 09:51 | |
| *** rotbeard has joined #openstack-nova | 09:51 | |
| *** rk4n has quit IRC | 09:52 | |
| *** rk4n has joined #openstack-nova | 09:52 | |
| *** rk4n has quit IRC | 09:53 | |
| *** slunkad_ has joined #openstack-nova | 09:54 | |
| *** e0ne has joined #openstack-nova | 09:55 | |
| Thelo | Hi All, I made a change in os_brick and nova. How can I run the nova unit test ( tox -v -epy27) and take the change into the library into account? Besides, If I want to commit both, should I wait for the part in os_brick to be accepted? | 09:56 |
| *** rk4n has joined #openstack-nova | 09:57 | |
| *** shuquan has quit IRC | 09:57 | |
| *** aix has joined #openstack-nova | 09:57 | |
| *** sfinucan has joined #openstack-nova | 09:57 | |
| *** shuquan has joined #openstack-nova | 09:58 | |
| *** haomaiwang has quit IRC | 09:58 | |
| *** slunkad_ has quit IRC | 10:00 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.hardware' options https://review.openstack.org/256344 | 10:00 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 10:00 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 10:00 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt' conf options documentation https://review.openstack.org/256368 | 10:00 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 10:00 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.driver' options https://review.openstack.org/257956 | 10:00 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 10:00 |
| johnthetubaguy | oomichi: honestly, it is tempting just to add them into the default list. This is to test affinity I guess? | 10:00 |
| *** rk4n has quit IRC | 10:00 | |
| johnthetubaguy | oomichi: wow, moving to the US! thats a big move! | 10:00 |
| * johnthetubaguy is back from his training course | 10:01 | |
| * johnthetubaguy hopes the review day is going well for everyone! | 10:01 | |
| bauzas | oomichi: oh, you mean relocating ? | 10:02 |
| *** haomaiwa_ has joined #openstack-nova | 10:02 | |
| johnthetubaguy | gmann: +A | 10:03 |
| gmann | johnthetubaguy: Thanks. | 10:03 |
| *** alexschm has quit IRC | 10:03 | |
| migi | sdague, dansmith, I've addeed you as reviewers for my little changes: https://review.openstack.org/#/c/258878 and https://review.openstack.org/#/c/258530 | 10:04 |
| *** alexschm has joined #openstack-nova | 10:04 | |
| migi | sdague, dansmith, this is required for my next rev which goes to bigger modules, but prefer to get those first | 10:04 |
| *** e0ne has quit IRC | 10:05 | |
| * johnthetubaguy just found the setting to show intra line differences in the new gerrit, whoop, whoop | 10:05 | |
| bauzas | hopefully, the Gerrit review UI didn't really change from the previous "new UI" | 10:06 |
| *** haomaiw__ has joined #openstack-nova | 10:06 | |
| *** omnipresent has quit IRC | 10:06 | |
| sfinucan | bauzas: I was hoping the opposite :) The new UI is one of the worst web UIs I've ever laid eyes on | 10:07 |
| *** haomaiwa_ has quit IRC | 10:07 | |
| bauzas | johnthetubaguy: I guess we can use https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking for the review day (L12) ? | 10:07 |
| *** zenoway has joined #openstack-nova | 10:07 | |
| johnthetubaguy | bauzas: yup, sent a note on the ML | 10:07 |
| *** alexschm has quit IRC | 10:08 | |
| *** alexschm has joined #openstack-nova | 10:09 | |
| *** alexschm has quit IRC | 10:09 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise PCI options https://review.openstack.org/253732 | 10:09 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'pci' conf options documentation https://review.openstack.org/243335 | 10:09 |
| *** alexschm has joined #openstack-nova | 10:10 | |
| openstackgerrit | Michal Dubiel proposed openstack/nova: libvirt, vhostuser: Add support for OpenContrail vRouter https://review.openstack.org/241652 | 10:11 |
| *** zenoway has quit IRC | 10:11 | |
| gibi | hi! happy review day! I know that this is younger than 50 days but in the other hand it has a +2 already https://review.openstack.org/#/c/243562/ , so if you have time... :) | 10:14 |
| *** shuquan has quit IRC | 10:15 | |
| gibi | also the versioned notification work has 3 patches ready for review as the subteam already reviewed them https://review.openstack.org/#/c/247024/ | 10:15 |
| *** gszasz has joined #openstack-nova | 10:16 | |
| tojuvone | oomichi: Are you around? | 10:18 |
| *** armax has quit IRC | 10:20 | |
| *** shakamunyi has quit IRC | 10:20 | |
| *** unicell1 has quit IRC | 10:20 | |
| *** unicell has joined #openstack-nova | 10:20 | |
| *** shakamunyi has joined #openstack-nova | 10:21 | |
| *** superflyy has quit IRC | 10:21 | |
| *** barra204 has joined #openstack-nova | 10:21 | |
| *** zhangjn has joined #openstack-nova | 10:21 | |
| *** gjayavelu has joined #openstack-nova | 10:21 | |
| *** zhangjn has quit IRC | 10:22 | |
| *** armax has joined #openstack-nova | 10:23 | |
| *** moshele has quit IRC | 10:23 | |
| *** mnestratov has joined #openstack-nova | 10:24 | |
| *** haomaiw__ has quit IRC | 10:24 | |
| *** pixelbeat_ has joined #openstack-nova | 10:24 | |
| *** thorst has joined #openstack-nova | 10:26 | |
| *** gjayavelu has quit IRC | 10:26 | |
| *** cdent has quit IRC | 10:27 | |
| *** haomaiwang has joined #openstack-nova | 10:28 | |
| *** cdent has joined #openstack-nova | 10:29 | |
| *** mpavone has joined #openstack-nova | 10:30 | |
| *** zhangjn has joined #openstack-nova | 10:31 | |
| *** haomaiwang has quit IRC | 10:31 | |
| *** mpavone has quit IRC | 10:31 | |
| *** thorst has quit IRC | 10:32 | |
| *** haomaiwa_ has joined #openstack-nova | 10:33 | |
| *** diogogmt has quit IRC | 10:35 | |
| *** cdent has quit IRC | 10:39 | |
| *** ljxiash has quit IRC | 10:40 | |
| *** ljxiash has joined #openstack-nova | 10:41 | |
| *** cdent has joined #openstack-nova | 10:41 | |
| *** unicell has quit IRC | 10:42 | |
| *** unicell has joined #openstack-nova | 10:42 | |
| *** zenoway has joined #openstack-nova | 10:43 | |
| *** e0ne has joined #openstack-nova | 10:43 | |
| *** ptm_away is now known as PaulMurray | 10:46 | |
| *** otter768 has joined #openstack-nova | 10:47 | |
| *** zhangjn has quit IRC | 10:47 | |
| *** EinstCrazy has quit IRC | 10:48 | |
| *** zenoway has quit IRC | 10:48 | |
| *** yamahata has joined #openstack-nova | 10:48 | |
| *** rfolco_ has quit IRC | 10:48 | |
| *** rfolco has joined #openstack-nova | 10:50 | |
| *** moshele has joined #openstack-nova | 10:50 | |
| *** otter768 has quit IRC | 10:51 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 10:52 |
| *** ccesario has quit IRC | 10:57 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop https://review.openstack.org/174854 | 10:58 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: deprecate use_usb_tablet in favor of pointer_model https://review.openstack.org/176242 | 10:58 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce freeze filesystems https://review.openstack.org/249140 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: check graphic/serial consoles at pre_live_migration step https://review.openstack.org/254709 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce thaw filesystems https://review.openstack.org/249141 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: replace direct libvirt's call AbortJobBlock https://review.openstack.org/249138 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return a root block device https://review.openstack.org/249139 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "snapshot" to Guest's object https://review.openstack.org/249353 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce block_stats to return device stats https://review.openstack.org/251288 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "pause" to Guest's object https://review.openstack.org/249834 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: wrapper list_guests to Host's object https://review.openstack.org/249835 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "shutdown" to Guest's object https://review.openstack.org/249833 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce exceptions handler https://review.openstack.org/257991 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce get_job_info to Guest's object https://review.openstack.org/250740 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: configure migrate max downtime https://review.openstack.org/250739 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "abort_job" to Guest's object https://review.openstack.org/250738 | 10:59 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce "migrate" to Guest's object https://review.openstack.org/250365 | 11:00 |
| *** haomaiwa_ has quit IRC | 11:01 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add policy check to avoid using qga in realtime context https://review.openstack.org/247585 | 11:01 |
| openstackgerrit | sahid proposed openstack/nova: conductor: fix unbound local variable request_spec https://review.openstack.org/257344 | 11:01 |
| openstackgerrit | sahid proposed openstack/nova: conductor: initialize request_spec to handle expected exception https://review.openstack.org/258011 | 11:01 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add realtime support https://review.openstack.org/197569 | 11:01 |
| *** haomaiwang has joined #openstack-nova | 11:01 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: verify cpu bw policy capability for host https://review.openstack.org/224726 | 11:02 |
| *** klkumar has quit IRC | 11:04 | |
| *** deepakcs has quit IRC | 11:04 | |
| openstackgerrit | sonu proposed openstack/python-novaclient: Removes MANIFEST.in as it is not needed explicitely by PBR https://review.openstack.org/258953 | 11:05 |
| *** liusheng has quit IRC | 11:06 | |
| *** aysyd has joined #openstack-nova | 11:07 | |
| *** dims has joined #openstack-nova | 11:07 | |
| *** lucas-dinner is now known as lucasagomes | 11:07 | |
| *** zenoway has joined #openstack-nova | 11:07 | |
| *** liusheng has joined #openstack-nova | 11:08 | |
| *** salv-orlando has quit IRC | 11:08 | |
| *** salv-orlando has joined #openstack-nova | 11:11 | |
| pkoniszewski | hey, I have one question - when I'm adding new API to v2.1 do I also need to add this API to v2? It is completely new resource (new controller) | 11:11 |
| *** EinstCrazy has joined #openstack-nova | 11:11 | |
| *** zhangjn has joined #openstack-nova | 11:11 | |
| *** salv-orlando has quit IRC | 11:12 | |
| johnthetubaguy | pkoniszewski: we are not allowed to add anything to v2.0 any more | 11:12 |
| pkoniszewski | great to hear that | 11:12 |
| johnthetubaguy | pkoniszewski: v2.0 API is fronzen now | 11:12 |
| *** salv-orlando has joined #openstack-nova | 11:12 | |
| *** zenoway has quit IRC | 11:14 | |
| pkoniszewski | however, when I'm making this new API in v2.1 discoverable a v2 test fails (test_list_extensions(v2)) because there is a new item in extensions list :S | 11:14 |
| *** salv-orl_ has joined #openstack-nova | 11:15 | |
| * PaulMurray opens a review, takes in new view, closes eyes and smacks forehead on desk while crying "why, why, why......." | 11:15 | |
| andrearosa | lol | 11:16 |
| alex_xu | pkoniszewski: you can do something like this https://review.openstack.org/#/c/128940/95/nova/api/openstack/compute/extension_info.py | 11:16 |
| pkoniszewski | alex_xu: that should solve my problem! thanks a lot! :) | 11:17 |
| alex_xu | pkoniszewski: np | 11:17 |
| *** klkumar has joined #openstack-nova | 11:17 | |
| *** yamahata has quit IRC | 11:18 | |
| *** salv-orlando has quit IRC | 11:18 | |
| *** eglynn has joined #openstack-nova | 11:19 | |
| openstackgerrit | Tang Chen proposed openstack/nova: Add a compute API to trigger crash dump in instance. https://review.openstack.org/202605 | 11:23 |
| *** zenoway has joined #openstack-nova | 11:24 | |
| *** sfinucan has quit IRC | 11:28 | |
| *** sfinucan has joined #openstack-nova | 11:28 | |
| *** thorst has joined #openstack-nova | 11:31 | |
| artom | ndipanov, danpb, you think as part of the non-priority review day you can sneak a look at https://review.openstack.org/#/c/234904/ (vNIC tagging)? It's not ready to be merged yet as test coverage is anywhere between pathetic and non-existent, but the functionality is essentially all there and I'd like feedback that my approach isn't insane :) | 11:32 |
| artom | ^^ applies to anyone else participating in the review day as well :) | 11:32 |
| ndipanov | artom will do ! | 11:34 |
| artom | ndipanov, thanks :) | 11:34 |
| *** rk4n has joined #openstack-nova | 11:37 | |
| openstackgerrit | jichenjc proposed openstack/nova: Return empty object list instead [] https://review.openstack.org/258962 | 11:37 |
| *** jlanoux has quit IRC | 11:37 | |
| *** thorst has quit IRC | 11:38 | |
| johnthetubaguy | pkoniszewski: you will need to hide that extension from the v2.0 compatibility mode, that test is erroring out correctly | 11:39 |
| johnthetubaguy | pkoniszewski: generally we should not be adding an extensions, as such, thats now covered by the API version | 11:39 |
| *** betherly has quit IRC | 11:39 | |
| johnthetubaguy | pkoniszewski: ah, alex_xu pointed you there, cool | 11:39 |
| pkoniszewski | got it, alex_xu suggested how to solve that few lines above | 11:39 |
| pkoniszewski | :) | 11:40 |
| johnthetubaguy | PaulMurray: more features are always good right? ;) | 11:40 |
| cdent | features-- | 11:40 |
| PaulMurray | johnthetubaguy, I'm just starting to get the hang of it | 11:41 |
| *** claudiub has joined #openstack-nova | 11:41 | |
| PaulMurray | I like the list of related changes etc. | 11:41 |
| PaulMurray | took a while to realise they were the series and which order (up or down) | 11:42 |
| pkoniszewski | the list of conflicting patches is a great addition! | 11:42 |
| *** patrickeast has quit IRC | 11:42 | |
| PaulMurray | don't like hte fact I have to scroll down past the gate jobs | 11:42 |
| *** betherly has joined #openstack-nova | 11:44 | |
| *** andrewbogott has quit IRC | 11:45 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 11:46 |
| openstackgerrit | Rafael Folco proposed openstack/nova: libvirt: check min required qemu/kvm versions on ppc64/ppc64le https://review.openstack.org/226555 | 11:48 |
| markus_z | @all: in case you experience an issue with the check queue http://lists.openstack.org/pipermail/openstack-dev/2015-December/082576.html | 11:48 |
| *** klkumar has quit IRC | 11:52 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add way to pause VM during live migration https://review.openstack.org/245921 | 11:52 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add new DB API method to retrieve migration for instance https://review.openstack.org/257270 | 11:52 |
| *** Kevin_Zheng has quit IRC | 11:53 | |
| *** briancurtin has quit IRC | 11:53 | |
| *** betherly has quit IRC | 11:53 | |
| *** irenab has quit IRC | 11:56 | |
| *** Kevin_Zheng has joined #openstack-nova | 11:57 | |
| *** klkumar has joined #openstack-nova | 11:59 | |
| *** zhangjn has quit IRC | 11:59 | |
| *** rcernin has quit IRC | 12:00 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-nova | 12:01 | |
| *** lpetrut has joined #openstack-nova | 12:03 | |
| *** shuquan has joined #openstack-nova | 12:05 | |
| *** patrickeast has joined #openstack-nova | 12:05 | |
| *** smatzek has joined #openstack-nova | 12:07 | |
| *** Kevin_Zheng has quit IRC | 12:08 | |
| *** mjura_ has quit IRC | 12:09 | |
| *** zhangjn has joined #openstack-nova | 12:09 | |
| *** zhangjn has quit IRC | 12:10 | |
| moshele | johnthetubaguy: hi | 12:10 |
| *** zhangjn has joined #openstack-nova | 12:11 | |
| *** shuquan has quit IRC | 12:11 | |
| johnthetubaguy | moshele: hi | 12:11 |
| moshele | johnthetubaguy: we want to add another Job for mellanox CI to test macvtap see http://144.76.193.39/ci-artifacts/Nova-MACVTAP-ML2-Sriov_2/ | 12:12 |
| moshele | johnthetubaguy do we need nova core approval | 12:12 |
| johnthetubaguy | moshele: can't you test both macvtap and the other mode, in a single CI, using using alternative flavors? | 12:13 |
| *** rcernin has joined #openstack-nova | 12:15 | |
| *** patrickeast has quit IRC | 12:15 | |
| *** raildo-afk is now known as raildo | 12:16 | |
| *** smatzek has quit IRC | 12:16 | |
| johnthetubaguy | moshele: for commenting, I didn't think you needed core approval, but I could be wrong | 12:19 |
| *** shuquan has joined #openstack-nova | 12:21 | |
| moshele | johnthetubaguy | 12:21 |
| moshele | It a bit complicate to do it in one CI, I prefer to split it to 2 jobs | 12:21 |
| moshele | johnthetubaguy: who would know? | 12:21 |
| johnthetubaguy | moshele: infra, I guess, what do you need from us? some vote on a permission request patch? | 12:22 |
| johnthetubaguy | moshele: live-migrate job we are creating is testing all ceph, nfs, and local storage in one job. it feels like a good partern to fix, a single vote from "mellanox" stuff would be much easier longterm | 12:23 |
| moshele | johnthetubaguy: I think to review that the logs are according to your requirements | 12:23 |
| *** [1]Thelo has joined #openstack-nova | 12:23 | |
| moshele | johnthetubaguy: ok I will see if we can do that | 12:23 |
| *** Thelo has quit IRC | 12:25 | |
| *** [1]Thelo is now known as Thelo | 12:25 | |
| *** jaypipes has joined #openstack-nova | 12:26 | |
| *** Kevin_Zheng has joined #openstack-nova | 12:26 | |
| *** patrickeast has joined #openstack-nova | 12:27 | |
| *** shuquan has quit IRC | 12:27 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Use is_supported() to check microversion https://review.openstack.org/249116 | 12:27 |
| *** irenab has joined #openstack-nova | 12:28 | |
| *** briancurtin has joined #openstack-nova | 12:29 | |
| *** andrewbogott has joined #openstack-nova | 12:31 | |
| *** delattec has quit IRC | 12:34 | |
| *** sahid has quit IRC | 12:37 | |
| *** Marga_ has joined #openstack-nova | 12:39 | |
| *** smatzek has joined #openstack-nova | 12:39 | |
| *** betherly has joined #openstack-nova | 12:41 | |
| *** pratikmallya has joined #openstack-nova | 12:41 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 12:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 12:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 12:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 12:41 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 12:41 |
| openstackgerrit | Sean Dague proposed openstack/python-novaclient: Allow command line for virtual-interface-list https://review.openstack.org/254707 | 12:42 |
| *** ankit_ag has quit IRC | 12:43 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Add support for API microversion 2.13 https://review.openstack.org/257703 | 12:43 |
| *** Thelo_ has joined #openstack-nova | 12:43 | |
| *** amotoki has joined #openstack-nova | 12:43 | |
| *** ankit_ag has joined #openstack-nova | 12:43 | |
| *** haomaiwang has quit IRC | 12:43 | |
| *** Daisy has joined #openstack-nova | 12:44 | |
| *** amotoki has quit IRC | 12:46 | |
| *** pratikmallya has quit IRC | 12:46 | |
| *** s-moriya has quit IRC | 12:47 | |
| *** otter768 has joined #openstack-nova | 12:47 | |
| *** ducttape_ has joined #openstack-nova | 12:48 | |
| alexschm | johnthetubaguy: kaisers and me were wondering why the native AIO mode blueprint was not included in your review list | 12:52 |
| *** thorst has joined #openstack-nova | 12:52 | |
| alexschm | johnthetubaguy: is it because the change sets are not older than 50 days? | 12:52 |
| *** otter768 has quit IRC | 12:52 | |
| alexschm | johnthetubaguy: for reference, https://blueprints.launchpad.net/nova/+spec/libvirt-aio-mode | 12:53 |
| *** thorst_ has joined #openstack-nova | 12:54 | |
| *** baoli_ has joined #openstack-nova | 12:54 | |
| *** Marga_ has quit IRC | 12:56 | |
| *** thorst has quit IRC | 12:56 | |
| *** links has quit IRC | 12:58 | |
| *** jyuso1 has quit IRC | 13:01 | |
| *** gongysh has quit IRC | 13:02 | |
| *** cdelatte has joined #openstack-nova | 13:02 | |
| *** gongysh has joined #openstack-nova | 13:02 | |
| *** Daisy has quit IRC | 13:02 | |
| *** Daisy has joined #openstack-nova | 13:03 | |
| *** salv-orl_ has quit IRC | 13:04 | |
| johnthetubaguy | alexschm: basically, yes | 13:04 |
| *** lbeliveau has quit IRC | 13:04 | |
| johnthetubaguy | alexschm: we have things up for review that are still waiting from last release, so I feel we should look at those first | 13:05 |
| *** lbeliveau has joined #openstack-nova | 13:05 | |
| *** cdelatte has quit IRC | 13:06 | |
| alexschm | johnthetubaguy: ok I see, thank you | 13:06 |
| *** ducttape_ has quit IRC | 13:07 | |
| *** dslevin_ has joined #openstack-nova | 13:09 | |
| *** salv-orlando has joined #openstack-nova | 13:09 | |
| *** dave-mccowan has joined #openstack-nova | 13:09 | |
| *** zhangjn has quit IRC | 13:10 | |
| *** ducttape_ has joined #openstack-nova | 13:12 | |
| sdague | PaulMurray: for this whole stack - https://review.openstack.org/#/c/254115/ what's the in flight story? | 13:14 |
| sdague | I guess I'm trying to understand how we can make that change and not break things in the middle | 13:14 |
| openstackgerrit | Bob Ball proposed openstack/nova: XenAPI: Fix race in rotate_xen_guest_logs https://review.openstack.org/245898 | 13:16 |
| *** briancurtin has quit IRC | 13:17 | |
| *** briancurtin has joined #openstack-nova | 13:17 | |
| *** mwagner_lap has joined #openstack-nova | 13:18 | |
| *** dane-fichter has joined #openstack-nova | 13:20 | |
| openstackgerrit | huan proposed openstack/nova: XenAPI: Support neutron security group https://review.openstack.org/251271 | 13:20 |
| Thelo_ | Hi all, if i do two changes. One in os_brick the other one in nova, how can i run nova`s unit testsothat | 13:20 |
| Thelo_ | It takesmy change in os brick into account? | 13:21 |
| PaulMurray | sdague, before we had a compute node object supported_instance was passed around as a json string value in the compute node dict | 13:21 |
| *** rcernin has quit IRC | 13:21 | |
| PaulMurray | sdague, that was generated by the virt drivers | 13:21 |
| Thelo_ | Also, to commit sgould i wait for the change in os bricj to be accepted? | 13:21 |
| PaulMurray | now there is a compute node object it is held as a data structure in the object | 13:22 |
| PaulMurray | so the values returned by the virt drivers are immediately decoded before adding to the compute node object field | 13:22 |
| sdague | so the compute node object auto converts if it's in either format? which is why it's safe to change one at atime? | 13:22 |
| sdague | ok | 13:22 |
| PaulMurray | that's right - the last step gets rid of the auto conversion | 13:23 |
| *** briancurtin_ has joined #openstack-nova | 13:23 | |
| PaulMurray | stats is done in one step because only ironic provides a value | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.hardware' options https://review.openstack.org/256344 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.firewall' options https://review.openstack.org/257995 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.netutils' options https://review.openstack.org/258009 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'virt' conf options documentation https://review.openstack.org/256368 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.imagecache' options https://review.openstack.org/258984 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.images' options https://review.openstack.org/258004 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.disk' options https://review.openstack.org/258912 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.driver' options https://review.openstack.org/257956 | 13:23 |
| dane-fichter | Thelo_: you'd likely have to wait for a new release of os-brick which includes your change, and then update Nova's requirements in your change in Nova | 13:24 |
| PaulMurray | sdague, FWI - I was going to do it all in one patch, but the libvirt one gave me trouble so I split them out to make it easier to figure out why | 13:24 |
| sdague | yeh, that's fine | 13:24 |
| sdague | I just ran through the stack and +2ed it, seems pretty straight forward after the explanation | 13:25 |
| Thelo_ | dane-fitcher: thanks | 13:25 |
| PaulMurray | sdague, thanks | 13:25 |
| *** andrewbogott has quit IRC | 13:26 | |
| *** andrewbogott has joined #openstack-nova | 13:26 | |
| *** alraddarla has joined #openstack-nova | 13:26 | |
| Thelo_ | dane-fitcher: how do i update nova`s requirements? Iam not sure of what you mean. | 13:27 |
| dane-fichter | Thelo_: no problem. I bet there's a way around that locally, but in order to propose a change, that's what you'll have to do | 13:27 |
| dane-fichter | Thelo_: you change the version number in requirements.txt to be the correct version of os-brick | 13:28 |
| Thelo_ | Ok | 13:28 |
| Thelo_ | Okthanks | 13:28 |
| dane-fichter | Thelo_: glad I could help | 13:28 |
| *** betherly has quit IRC | 13:29 | |
| *** betherly has joined #openstack-nova | 13:29 | |
| *** patrickeast has quit IRC | 13:30 | |
| *** patrickeast has joined #openstack-nova | 13:30 | |
| *** pradk has joined #openstack-nova | 13:31 | |
| *** zhangjn has joined #openstack-nova | 13:32 | |
| *** betherly_ has joined #openstack-nova | 13:32 | |
| *** alexschm_ has joined #openstack-nova | 13:32 | |
| *** breitz has quit IRC | 13:32 | |
| *** breitz has joined #openstack-nova | 13:33 | |
| *** betherly has quit IRC | 13:33 | |
| *** betherly_ is now known as betherly | 13:33 | |
| *** links has joined #openstack-nova | 13:33 | |
| sdague | danpb: is this right - https://review.openstack.org/#/c/243117/2/nova/tests/unit/virt/libvirt/test_host.py (small libvirt unit test fix) | 13:33 |
| *** alexschm has quit IRC | 13:34 | |
| *** andrewbogott_ has joined #openstack-nova | 13:35 | |
| danpb | sdague: yes, that test case is supposed to be exercising has_version, the test case prior does has_min_version | 13:36 |
| danpb | looks like just copy+paste mistake | 13:36 |
| *** dedery_ has joined #openstack-nova | 13:36 | |
| *** ducttape_ has quit IRC | 13:37 | |
| *** dedery has quit IRC | 13:40 | |
| sdague | cool, thanks | 13:40 |
| *** zakora has joined #openstack-nova | 13:41 | |
| *** dedery_ has quit IRC | 13:41 | |
| *** betherly_ has joined #openstack-nova | 13:41 | |
| *** zhangjn has quit IRC | 13:42 | |
| openstackgerrit | Ryan Moore proposed openstack/nova: Add better help to rdp options https://review.openstack.org/253556 | 13:44 |
| openstackgerrit | Ryan Moore proposed openstack/nova: config options: centralize section: "rdp" https://review.openstack.org/252389 | 13:44 |
| *** patrickeast_ has joined #openstack-nova | 13:45 | |
| sdague | bauzas: are you ok with this now - https://review.openstack.org/#/c/180155/20 - it's reordering the default fitlers, but the new order does seem to be more sensible | 13:45 |
| *** Daisy_ has joined #openstack-nova | 13:47 | |
| PaulMurray | sdague, bauzas not sure about that one ^^ | 13:47 |
| sdague | PaulMurray: ok, what are your concerns with the order change? | 13:48 |
| *** Daisy has quit IRC | 13:48 | |
| PaulMurray | sdague, it really depends on what you have, but if you have AZs the AZ filter will always reduce number of hosts by 1/n where n is number of AZ | 13:49 |
| PaulMurray | Would we expect to see multiple AZ in a default setup? | 13:49 |
| PaulMurray | if no then it makes sense | 13:49 |
| edleafe | PaulMurray: and would the number of hosts per AZ be roughly equal? | 13:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: config options: Centralise 'virt.ironic' options https://review.openstack.org/259014 | 13:50 |
| PaulMurray | edleafe, compute filter removes disabled hosts - would you expect to have a large number of disabled hosts ? | 13:50 |
| PaulMurray | edleafe, but to answer your question, generally they are roughly equal | 13:51 |
| sdague | PaulMurray: so... just after the AZ filter would be better? | 13:51 |
| sdague | it still seems like getting it ahead for Ram / Disk makes sense | 13:51 |
| edleafe | PaulMurray: just wondering if people treat AZs like host aggs | 13:51 |
| PaulMurray | edleafe, we had 3 AZ per region - for reliability purposes | 13:52 |
| PaulMurray | edleafe, generally you would see the same types of thing in each | 13:52 |
| sdague | I'm going to assume any reasonably large deploy is going to need to tune things, so we don't need to be perfect here, however compute filter coming after ram filter seems weird | 13:52 |
| edleafe | sdague: but the compute filter doesn't come after the ram filter now | 13:53 |
| cdent | +1 to sdague | 13:53 |
| PaulMurray | sdague, generally the more things it will knock out the higher in the order | 13:53 |
| edleafe | you just swapped AZ and compute filters | 13:53 |
| sdague | edleafe: ummm, in the baremetal case | 13:53 |
| sdague | not in the base case | 13:54 |
| edleafe | doh! | 13:54 |
| johnthetubaguy | So if you cloud is normally full, RAM filter first makes sense, even if its not logical | 13:54 |
| sdague | johnthetubaguy: because the cost of loading every filter is the same? | 13:54 |
| johnthetubaguy | well, you normally have more full hosts than dead hosts | 13:54 |
| edleafe | sdague: yeah, that part seems more reasonable | 13:54 |
| johnthetubaguy | sdague: because you want the filter that removes the most hosts first, so the later filters loop through less hosts, assuming equal cost of running every filter (which I think is basically true now) | 13:55 |
| sdague | johnthetubaguy: ok, that was the question, I didn't realize that compute filter was as costly as ram filter | 13:55 |
| sdague | also, I kind of hate the filter model :) | 13:56 |
| johnthetubaguy | well, no filters do any DB queries, so you probably just want to reduce that down | 13:56 |
| edleafe | yeah, the filters don't hit the db anymore | 13:56 |
| edleafe | normally | 13:56 |
| johnthetubaguy | sdague: so did some up with a slightly different approach, but it drops the weighters actually: https://review.openstack.org/#/c/256323/ | 13:57 |
| *** dslevin_ has quit IRC | 13:57 | |
| johnthetubaguy | s/some/come/ | 13:57 |
| sdague | yeh, it still seems weird to me that we don't load this up into a numpy datastructure and do the compares there all at once | 13:58 |
| PaulMurray | sdague, I know, why don't you start a new scheduler :) | 13:58 |
| sdague | because I suspect that would fundamentally alter the performance for good here | 13:58 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move config options from nova/api directory https://review.openstack.org/258180 | 13:58 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move nova/opts.py to nova/conf/ https://review.openstack.org/255436 | 13:58 |
| *** sahid has joined #openstack-nova | 13:58 | |
| sdague | PaulMurray: :) | 13:58 |
| edleafe | sdague: that's just one of many proposed improvements :) | 13:59 |
| sdague | edleafe: ok, anyone actually hacking on that bit yet? | 13:59 |
| johnthetubaguy | sdague: possibly, honestly that bit of the scheduler is crazy fast, compared to the RPC call, when I last checked | 13:59 |
| sdague | johnthetubaguy: ok | 13:59 |
| edleafe | sdague: yeah, there are several out there - one was presented at the summit | 13:59 |
| sdague | edleafe: is it in gerrit? | 14:00 |
| johnthetubaguy | sdague: I have proposed this scheduler: https://review.openstack.org/#/c/256323/ but its all about the fact the scheduler is almost unsable in its current form, its so hard to configure | 14:00 |
| PaulMurray | sdague, in our experience anything that makes a big difference is worth paying attention to - everything else is noise | 14:00 |
| edleafe | sdague: hell no! | 14:00 |
| *** sahid has quit IRC | 14:00 | |
| *** sahid has joined #openstack-nova | 14:00 | |
| edleafe | sdague: they are largely single-company efforts. IBM has its own custom scheduler too | 14:01 |
| *** lykinsbd has joined #openstack-nova | 14:01 | |
| PaulMurray | sdague, to be honest if this makes a difference to anyone they would be setting these values themselves | 14:01 |
| sdague | PaulMurray: yeh, ok | 14:02 |
| sdague | ok, so the documentation part of that patch is something we should land | 14:02 |
| johnthetubaguy | FYI, I did upstream the one rackspace uses currently: https://github.com/openstack/nova/blob/master/nova/scheduler/caching_scheduler.py | 14:02 |
| sdague | which is the config set of values that we can agree on (personally I'm happy with anything) | 14:02 |
| edleafe | sdague: until the scheduler is independent (if ever), adding an incompatible scheduler is a non-starter. | 14:02 |
| *** rdopiera has quit IRC | 14:02 | |
| cdent | sdague: a numpy-based complex vector linear algebra driven scheduler is on my wish list | 14:03 |
| sdague | edleafe: it really wouldn't have to be incompatible, it would just be how you were doing the for looping for the filters | 14:03 |
| cdent | tensor-scheduler | 14:03 |
| sdague | but if what johnthetubaguy is saying is true, then it's sort of irrelevant most of the time, as the rpc is really the cost | 14:03 |
| sdague | but, back to the patch | 14:03 |
| sdague | https://review.openstack.org/#/c/180155/20/nova/conf/scheduler.py - are the current defaults fine? and we just change that part and land the docs? | 14:04 |
| johnthetubaguy | sdague: so I am thinking about when you use the caching_scheduler, at least thats what I remember seeing when I last dug into that, it works at the scale we need (peak of a couple of hundred builds per cell, per min) | 14:05 |
| johnthetubaguy | but yeah, I should stop distracting | 14:06 |
| *** dane-fichter has quit IRC | 14:06 | |
| johnthetubaguy | I am trying to get through all these old blueprints for the review day today | 14:06 |
| sdague | gotcha | 14:06 |
| * edleafe goes to make more coffee... | 14:08 | |
| sdague | fyi, played around with some new options in gerrit v2.11 for a compute review dashboard - https://goo.gl/7JpSFl | 14:08 |
| sdague | we now can label:Code-Review<=-1,nova-core | 14:09 |
| *** xyang1 has joined #openstack-nova | 14:09 | |
| sdague | and delta:<=10 (to show small patches) | 14:09 |
| johnthetubaguy | awesomeness | 14:09 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Allow saving empty pci_device_pools in ComputeNode object https://review.openstack.org/216049 | 14:10 |
| danpb | did they enable the arbitrary tagging extension in the end or not ? | 14:10 |
| *** dustins|out is now known as dustins | 14:10 | |
| *** haomaiwang has joined #openstack-nova | 14:11 | |
| *** Daisy_ has quit IRC | 14:11 | |
| *** rlrossit has joined #openstack-nova | 14:12 | |
| *** andrewbogott has quit IRC | 14:16 | |
| *** andrewbogott has joined #openstack-nova | 14:16 | |
| *** ctrath has joined #openstack-nova | 14:16 | |
| *** omnipresent has joined #openstack-nova | 14:17 | |
| *** lucasagomes is now known as lucas-hungry | 14:17 | |
| *** lykinsbd has quit IRC | 14:17 | |
| sdague | danpb: no, it's not doable without the notedb implementation, which is not yet finished | 14:18 |
| *** jlanoux has joined #openstack-nova | 14:19 | |
| *** zenoway has quit IRC | 14:20 | |
| sdague | so, if anyone wants to look at this patch series to allow glance to exist on a suburl in your cloud - https://review.openstack.org/#/c/254213/ (and up). It's pretty straight forward to review. And will let some of the API to real webserver work continue. | 14:20 |
| sahid | danpb: , any chance to re-get your ack on thoce r | 14:21 |
| sahid | reviews | 14:21 |
| *** markvoelker has quit IRC | 14:21 | |
| sahid | https://review.openstack.org/#/c/224726/ | 14:21 |
| sahid | and this serie https://review.openstack.org/#/c/257344/ | 14:21 |
| sahid | (already acked) | 14:21 |
| openstackgerrit | Ludovic Beliveau proposed openstack/nova: Support configuration of multiple pci_alias within an array https://review.openstack.org/242555 | 14:25 |
| *** sahid has quit IRC | 14:28 | |
| *** markvoelker has joined #openstack-nova | 14:29 | |
| andrearosa | workflow question: I have a nova fix which is blocked by a Cinder bug (no fix proposed for the bug yet), shall I abandon my change with a note or mark it as WIP, or it doesn't matter? | 14:30 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Cleanup needless code from oslo-incubator https://review.openstack.org/259044 | 14:31 |
| *** jcook has quit IRC | 14:32 | |
| snikitin | andrearosa: I think you should mark it as WIP | 14:33 |
| dansmith | agree | 14:33 |
| *** smatzek has quit IRC | 14:34 | |
| andrearosa | snikitin: dansmith thanks | 14:34 |
| *** mrkz has joined #openstack-nova | 14:35 | |
| *** ociuhandu has joined #openstack-nova | 14:35 | |
| *** jcook has joined #openstack-nova | 14:36 | |
| alex_xu | johnthetubaguy: danpb could you help on this https://review.openstack.org/255122, I know today is reviewday, but I'm afraid people geting vacation next week. hope this can merge then developer can get coding early, thanks in advance! | 14:38 |
| *** jlanoux has quit IRC | 14:40 | |
| *** alexschm_ has quit IRC | 14:40 | |
| *** pradk has quit IRC | 14:41 | |
| *** pradk has joined #openstack-nova | 14:41 | |
| *** thangp has joined #openstack-nova | 14:41 | |
| bauzas | sdague: PaulMurray: sorry I was away | 14:42 |
| *** alexschm has joined #openstack-nova | 14:43 | |
| bauzas | moving the ComputeNodeFilter to be the first sounds good | 14:43 |
| bauzas | because calling AZs for down hosts seems bad to me | 14:43 |
| bauzas | that said, I don't know if we need to modify that | 14:44 |
| sdague | ok, well we should decide if we are taking it, or want a change in it, and not just leave it flapping in the wind | 14:45 |
| bauzas | I'm not really opiniated for that | 14:47 |
| bauzas | that said, picking a patch for that makes me afraid to have a lot of patches coming in because operators would like to modify the default opt | 14:47 |
| edleafe | it falls under the "if it ain't broke" category IMO | 14:48 |
| bauzas | so, while I think to have the ComputeFilter as said first is good, but I'd be -1 for others | 14:48 |
| *** jlanoux has joined #openstack-nova | 14:48 | |
| *** eharney has joined #openstack-nova | 14:48 | |
| *** Thelo_ has quit IRC | 14:48 | |
| *** otter768 has joined #openstack-nova | 14:48 | |
| *** thangp has quit IRC | 14:49 | |
| sdague | bauzas: ok, well we need to make a statement either way, -1 is fine. It's ok to be opinionated about it. The lack of feedback is much worse than a -1 we don't want to do this. | 14:51 |
| *** alexschm has quit IRC | 14:51 | |
| *** otter768 has quit IRC | 14:53 | |
| *** betherly_ has quit IRC | 14:54 | |
| johnthetubaguy | better defaults are important, but only if there is a clearly better option for the better default | 14:54 |
| *** patrickeast_ has quit IRC | 14:54 | |
| johnthetubaguy | bauzas: whats the gerrit link on this one again? | 14:55 |
| *** alexschm has joined #openstack-nova | 14:56 | |
| johnthetubaguy | claudiub: is there any way we can get the CI to pass on this change? https://review.openstack.org/#/c/175479/ | 14:57 |
| edleafe | johnthetubaguy: https://review.openstack.org/#/c/180155/ | 14:58 |
| *** zhenq has joined #openstack-nova | 14:58 | |
| *** alexschm has quit IRC | 14:58 | |
| *** tjones has joined #openstack-nova | 14:58 | |
| *** ducttape_ has joined #openstack-nova | 14:58 | |
| *** alraddarla has quit IRC | 14:59 | |
| *** alexschm has joined #openstack-nova | 14:59 | |
| *** haomaiwang has quit IRC | 15:01 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Build timeouts task filters deleting instances https://review.openstack.org/240017 | 15:01 |
| *** briancurtin_ has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-nova | 15:01 | |
| *** dslev has joined #openstack-nova | 15:01 | |
| claudiub | johnthetubaguy: interesting. the hyper-v ci passed on the same patchset. | 15:01 |
| claudiub | johnthetubaguy: looking through the logs. | 15:02 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: enginefacade: 'migration' https://review.openstack.org/243496 | 15:03 |
| *** thangp has joined #openstack-nova | 15:03 | |
| *** andrewbogott_ has quit IRC | 15:03 | |
| *** rohit_ has joined #openstack-nova | 15:04 | |
| *** smatzek has joined #openstack-nova | 15:07 | |
| *** sahid has joined #openstack-nova | 15:08 | |
| PaulMurray | bauzas, on compute filter vs AZ filter.... if you have more disable hosts than you have hosts in an AZ your in a bad way. that is the only way putting compute filter ahead of AZ filter makes sense | 15:08 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Adds json sample for the versioned notifications https://review.openstack.org/248167 | 15:09 |
| PaulMurray | bauzas, but if you don't use AZs you don't need the filter | 15:10 |
| *** cfriesen has joined #openstack-nova | 15:10 | |
| openstackgerrit | Marian Horban proposed openstack/nova: Stop Libvirt event threads https://review.openstack.org/259066 | 15:10 |
| *** ccesario has joined #openstack-nova | 15:11 | |
| *** pratikmallya has joined #openstack-nova | 15:12 | |
| bauzas | johnthetubaguy: could you please review https://review.openstack.org/#/c/257344/ ? | 15:12 |
| bauzas | johnthetubaguy: I don't want to +W something just done by RH hguys | 15:12 |
| openstackgerrit | Marian Horban proposed openstack/nova: Stop Libvirt event threads https://review.openstack.org/259066 | 15:13 |
| *** lucas-hungry is now known as lucasagomes | 15:13 | |
| *** rgerganov_ has quit IRC | 15:13 | |
| *** hemna has joined #openstack-nova | 15:13 | |
| johnthetubaguy | bauzas: isn't that added a mox test, which is banned? | 15:14 |
| sfinucan | sdague: Could I get you have a look at a patch you -1d a while back? Pretty please? | 15:14 |
| *** pumaranikar has joined #openstack-nova | 15:14 | |
| bauzas | johnthetubaguy: the module is already using mox, and the added test is really a close one from the ohters | 15:15 |
| sfinucan | ndipanov: Got a moment? | 15:15 |
| bauzas | johnthetubaguy: asking for mock should seem to be a bit difficult to me just for that | 15:15 |
| johnthetubaguy | bauzas: but we will have to move all mox tests to mock to get py34 working, I am told | 15:15 |
| mriedem | scottda: hemna: should nova wait for os-detach to complete before trying to delete a volume? | 15:17 |
| openstackgerrit | Marian Horban proposed openstack/nova: Stop Libvirt event threads https://review.openstack.org/259066 | 15:18 |
| bauzas | johnthetubaguy: well, good point but the py34 job seems a bit unneeded IMHO | 15:18 |
| hemna | mriedem, yes because os-detach is basically a DB update, so it makes the volume state available | 15:18 |
| bauzas | johnthetubaguy: because it just leaves a lot of tets | 15:18 |
| bauzas | tests | 15:18 |
| johnthetubaguy | bauzas: I am talking about the end goal of us getting full py34 support, I don't care about the tests in them selves | 15:18 |
| mriedem | hemna: ok, nova doesn't wait today, it just gets the 202 and continues on | 15:19 |
| mriedem | hemna: this was regarding andrearosa's thread on force detach | 15:19 |
| hemna | mriedem ok I haven't seen that one | 15:19 |
| mriedem | hemna: and i'm assuming volume delete fails if the volume is attached | 15:20 |
| mriedem | so nova is really just racing here | 15:20 |
| hemna | yup | 15:20 |
| mriedem | and getting lucky | 15:20 |
| hemna | it should | 15:20 |
| mriedem | yeah | 15:20 |
| hemna | unless the volume state is available, delete should fail | 15:20 |
| mriedem | andrearosa: so that seems like a simple addition to _shutdown_instance in the compute manager - we could probably re-use the same timeout config option we use for waiting for volume attach to complete | 15:20 |
| andrearosa | mriedem: hello | 15:21 |
| * andrearosa reading back | 15:21 | |
| *** jlanoux has quit IRC | 15:21 | |
| claudiub | johnthetubaguy: it seems the ci failed due to some lost amqp messages. added a recheck on the patch. | 15:23 |
| andrearosa | mriedem: I've just replied to your thread I am going to revisit my change, | 15:23 |
| sdague | sfinucan: yuo need to be more specific, I have -1ed a lot of things | 15:23 |
| *** jlanoux has joined #openstack-nova | 15:24 | |
| sfinucan | sdague: https://review.openstack.org/#/c/240271/ | 15:24 |
| andrearosa | mriedem: as I workded on it months ago I just want to start from scratch and see if I messed something obvious. I actually worked on an initial patch put by wanghao | 15:24 |
| andrearosa | mriedem: are you ok with that? | 15:24 |
| ndipanov | sfinucan, sure | 15:24 |
| claudiub | johnthetubaguy: as for py3 compatibility, there are quite a few unit tests that I can't figure out why they fail. they are in nova.compute and they are related to volumes. apparently the volume's connection_info is None and it shouldn't be.. | 15:24 |
| andrearosa | mriedem: s/messed/missed even if I probably messed something :) | 15:25 |
| mriedem | andrearosa: that's fine, i'm just thinking nova probably needs to do a better job of the initial volume cleanup so we don't get into a state where we require hacks via nova-manage to cleanup busted state | 15:25 |
| *** haomaiwang has quit IRC | 15:25 | |
| mriedem | there is a clear race in the code i think where nova can be orphaning volumes | 15:25 |
| sdague | sfinucan: there are other oslo.db debug messages which are quite useful, like the retry ones | 15:26 |
| sdague | so I don't think blanket flipping that off is the right thing to do | 15:26 |
| mriedem | dansmith: when you get a chance, stable/kilo backport for leaking passwords https://review.openstack.org/#/c/249239/ | 15:26 |
| sfinucan | ndipanov: A quick one RE: CPU thread pinning | 15:26 |
| dansmith | mriedem: when you get a chance, I updated that patch you -1d in the live migration stuff | 15:26 |
| sfinucan | Deadline for feature freeze in January 21st. I'm gone from middle of next week until mid January. Anything I need to do before then? | 15:27 |
| andrearosa | mriedem: that's for sure. I am working at the moment to some other operations in the area betweem nova and cinder and I find things which seem not right. For example https://review.openstack.org/#/c/257275/2 or https://review.openstack.org/#/c/257853/ | 15:27 |
| mriedem | dansmith: i'll make it my xmas present for you | 15:27 |
| *** daemontool has joined #openstack-nova | 15:27 | |
| dansmith | heh | 15:27 |
| sfinucan | Also FYI if you see anything that needs changing while I'm out feel free to do so | 15:27 |
| ndipanov | johnthetubaguy, https://review.openstack.org/#/c/229574/ a gentle reminder :) | 15:27 |
| ndipanov | it's related to non priority feature | 15:27 |
| ndipanov | sfinucan, no I can baby sit it | 15:27 |
| ndipanov | thanks for letting me know | 15:28 |
| ndipanov | if some comments come up address them while you're here | 15:28 |
| sfinucan | ndipanov: awesome, thanks :) | 15:28 |
| sfinucan | sdague: | 15:28 |
| *** BobBall has quit IRC | 15:28 | |
| sfinucan | sdague: Hmmm, I think it's going to be all or nothing though, right? | 15:28 |
| andrearosa | ndipanov: I am interested in having your opinion here: https://review.openstack.org/#/c/257853/ | 15:28 |
| mriedem | dansmith: i'm going to need to cross check the libvirt migrate data object to the libvirt driver code for the fields, which will take a little bit | 15:28 |
| sdague | sfinucan: via that config option, yes | 15:28 |
| *** jwcroppe has joined #openstack-nova | 15:29 | |
| dansmith | mriedem: just as long as you don't start vacation until it lands, that's fine :) | 15:29 |
| sfinucan | sdague: Is there another way though? All the attempts I've made are variants of "turn the config option off" | 15:29 |
| mriedem | eee, i have about 15 minutes then | 15:29 |
| sdague | well, nothing is landing right now, the gate is stalled, infra looking into it | 15:30 |
| *** angdraug has joined #openstack-nova | 15:31 | |
| sdague | sfinucan: you can override python logging in nova-manage itself | 15:31 |
| dansmith | mriedem: better lower your standards then | 15:31 |
| sdague | the problem with setting it in nova config.py is that you just disabled it for all servers | 15:31 |
| sdague | by default | 15:31 |
| *** alexschm has quit IRC | 15:32 | |
| sfinucan | sdague: Yeah, it's quite extensive | 15:32 |
| sfinucan | However I didn't think oslo_db debug messages would be important enough to bother propogating up outside of development | 15:33 |
| *** raildo is now known as raildo-afk | 15:33 | |
| mriedem | is it just me or does new gerrit get the 'toggle ci' button screwed up? | 15:33 |
| sfinucan | I'm not an operator though, so I can't say for sure | 15:33 |
| sdague | sfinucan: if you turn debug=True you did want stuff like that, and honestly the retry stuff is a bit important because it exposes deadlocks | 15:33 |
| openstackgerrit | Cyril Roelandt proposed openstack/nova: Remove version hardcoding for 'create' method https://review.openstack.org/259078 | 15:33 |
| sdague | anyway, that fix is a really big hammer for a small ux issue on nova-manage. That ux thing should be done in nova-manage, not impacting everything else | 15:34 |
| *** jaosorior has joined #openstack-nova | 15:34 | |
| sdague | so I remain -1 | 15:34 |
| sfinucan | sdague: OK, so I should pursue something like I did patchset 2 https://review.openstack.org/#/c/240271/2/nova/cmd/manage.py | 15:34 |
| mriedem | dansmith: you didn't want to make any of those fields enums? | 15:35 |
| sfinucan | but one that retains logging levels for other services | 15:35 |
| mriedem | i had pointed out a few that could be enums | 15:35 |
| sdague | sfinucan: yes | 15:35 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova: Added new scheduler filter: AggregateTypeExtraSpecsAffinityFilter https://review.openstack.org/189279 | 15:35 |
| dansmith | mriedem: ah, yeah, I can do that, I forgot | 15:35 |
| mriedem | ok, don't do that yet. | 15:35 |
| dansmith | mriedem: but I can also do it in a later patch, if..you know.. you're in a hurry to go on vacay :P | 15:35 |
| *** mtanino has joined #openstack-nova | 15:36 | |
| sfinucan | sdague: OK, I'll try that and check back in if I get anywhere | 15:36 |
| sfinucan | Thanks for your time | 15:36 |
| *** alexschm has joined #openstack-nova | 15:36 | |
| *** alexschm has quit IRC | 15:38 | |
| *** raildo-afk is now known as raildo | 15:38 | |
| *** alexschm has joined #openstack-nova | 15:38 | |
| scottda | mriedem: hemna the call to os-detach also includes remove_export in the driver. So Perhaps that is why it is async | 15:39 |
| hemna | could be | 15:39 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: API: Adds instance live-resize https://review.openstack.org/184584 | 15:40 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: conductor: add live_resize task https://review.openstack.org/248579 | 15:40 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: compute: Adds instance live-resize https://review.openstack.org/248581 | 15:40 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: objects: Adds os_live_resize field to ImageMetaProps https://review.openstack.org/259082 | 15:40 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: db: Adds host_capabilities column in compute_nodes https://review.openstack.org/228279 | 15:43 |
| mriedem | dansmith: comments inline | 15:43 |
| mriedem | dansmith: otherwise it's looking pretty good | 15:43 |
| mriedem | haven't looked at the tests yet | 15:44 |
| *** snikitin2 has joined #openstack-nova | 15:44 | |
| *** mhorban has joined #openstack-nova | 15:45 | |
| *** links has quit IRC | 15:50 | |
| mriedem | alright, i'm out until monday, sniff you jerks later | 15:50 |
| *** ljxiash has quit IRC | 15:50 | |
| *** mriedem has quit IRC | 15:50 | |
| *** pratikmallya has quit IRC | 15:50 | |
| *** cfarquhar has quit IRC | 15:51 | |
| *** ljxiash has joined #openstack-nova | 15:51 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes Python 3 unit tests for nova.compute https://review.openstack.org/247742 | 15:53 |
| *** alexschm has quit IRC | 15:54 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove nova-cert from most functional tests https://review.openstack.org/258135 | 15:54 |
| *** alexschm has joined #openstack-nova | 15:55 | |
| *** david-lyle_ has joined #openstack-nova | 15:55 | |
| *** ljxiash has quit IRC | 15:55 | |
| *** belmoreira has joined #openstack-nova | 15:56 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes conductor py3 unit tests https://review.openstack.org/253989 | 15:56 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes network py3 unit tests https://review.openstack.org/253990 | 15:56 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Fixes cells py3 unit tests https://review.openstack.org/255824 | 15:57 |
| *** david-lyle_ has quit IRC | 15:57 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove use of _get_regexes in samples tests https://review.openstack.org/257439 | 15:57 |
| openstackgerrit | Sean Dague proposed openstack/nova: Use just ids in all request templates for flavors/images https://review.openstack.org/257438 | 15:57 |
| openstackgerrit | Sean Dague proposed openstack/nova: extract non instance methods https://review.openstack.org/256441 | 15:57 |
| *** thangp has quit IRC | 15:58 | |
| *** david-lyle has quit IRC | 15:59 | |
| mhorban | @sdague, @lxsli_festive: Hi guys! I want to ask about reloading configuration and review https://review.openstack.org/#/c/223668/4 . | 15:59 |
| mhorban | @sdague, @lxsli_festive: As I understood there should be two changes: 1. Add mutability to oslo.log config options, 2. Add service's code which react on reloading configuration. First step is done in review https://review.openstack.org/#/c/254821/. This review implements second step. ...Or my vision is wrong? | 16:00 |
| *** kmartin has quit IRC | 16:02 | |
| *** kmartin has joined #openstack-nova | 16:02 | |
| *** zenoway has joined #openstack-nova | 16:03 | |
| *** tjones has left #openstack-nova | 16:04 | |
| *** omnipresent has quit IRC | 16:04 | |
| *** david-lyle has joined #openstack-nova | 16:05 | |
| *** yassine__ has quit IRC | 16:06 | |
| *** yassine__ has joined #openstack-nova | 16:07 | |
| *** vishwana_ has joined #openstack-nova | 16:07 | |
| *** kylek3h has joined #openstack-nova | 16:08 | |
| *** vishwanathj has quit IRC | 16:09 | |
| snikitin2 | johnthetubaguy, alex_xu: Hi guys! Today is a code review day and I want to ask you to look at one very old patch called "instance tags". I'm pinging you because you are familiar with this patch more then others. Now it has -1 from Jenkins because of broken CI. I rechecked it but it stuck in zuul because of brocen CI :( I've been trying to merge it for about 2 years so please look at this patch. https://review.openstack.org/#/c/128940/ | 16:09 |
| *** yamahata has joined #openstack-nova | 16:10 | |
| *** boris-42_ has joined #openstack-nova | 16:12 | |
| *** cfarquhar has joined #openstack-nova | 16:14 | |
| *** cfarquhar has quit IRC | 16:14 | |
| *** cfarquhar has joined #openstack-nova | 16:14 | |
| bauzas | johnthetubaguy: mriedem: given that it seems there is a consensus of not providing new tests using mox, could we provide a code-review.rst change for that? | 16:16 |
| danpb | yeah we should do | 16:16 |
| bauzas | johnthetubaguy: mriedem: I was thinking of the previous consensus about adding new tests with mox if the module was using that | 16:17 |
| *** diogogmt has joined #openstack-nova | 16:17 | |
| johnthetubaguy | yep, I started to change my mind yesterday talking to sdague | 16:17 |
| johnthetubaguy | after discovering the issues we are seeing with mox3 right now | 16:17 |
| sdague | yeh, one of the places we're inadvertantly doing it is with the self.stubs.Set (which is used in a lot of tests that don't use mox otherwise) | 16:18 |
| *** ziggit has joined #openstack-nova | 16:18 | |
| sdague | I have a local patch that would create a self.patch in test.py that uses the MonkeyPatch fixture instead | 16:18 |
| moshele | danpb: if you have time can you please review https://review.openstack.org/#/c/227160/ is has already +2 from ndipanov | 16:19 |
| sdague | but I did want to get that idea acked by someone before pushing it out there | 16:19 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Adds json sample for the versioned notifications https://review.openstack.org/248167 | 16:20 |
| johnthetubaguy | sdague: that sounds worth trying | 16:20 |
| sdague | johnthetubaguy: so, should we put up a blueprint for tracking mox removal ? | 16:20 |
| johnthetubaguy | sdague: yeah, we could, or just keep it under py34 | 16:20 |
| sdague | ok, a number of the patches won't look like py34 patches, so it might confuse folks | 16:21 |
| johnthetubaguy | fair enough, seems a good split, I can approve that as a BP split | 16:21 |
| *** alexschm has quit IRC | 16:21 | |
| *** belmoreira has quit IRC | 16:22 | |
| *** ljxiash has joined #openstack-nova | 16:22 | |
| bauzas | we should at least write our new policy in the code-review.rst doc file I guess | 16:22 |
| bauzas | but I agree with you, folks | 16:22 |
| bauzas | py34 job just seems useless because the lots of unchecked tests, so if we can help that, it's cool | 16:22 |
| *** markus_z has quit IRC | 16:22 | |
| *** jwcroppe has quit IRC | 16:23 | |
| *** tonytan4ever has joined #openstack-nova | 16:23 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add way to pause VM during live migration https://review.openstack.org/245921 | 16:23 |
| sdague | johnthetubaguy: https://blueprints.launchpad.net/nova/+spec/remove-mox | 16:24 |
| bauzas | that sounds a low-hanging fruit for newcomers :) | 16:24 |
| bauzas | speaking of https://etherpad.openstack.org/p/nova-low-hanging-fruit | 16:24 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add way to pause VM during live migration https://review.openstack.org/245921 | 16:26 |
| johnthetubaguy | sdague: cool, approved | 16:26 |
| johnthetubaguy | bauzas: yeah, worth an add | 16:26 |
| *** pratikmallya has joined #openstack-nova | 16:26 | |
| *** ljxiash has quit IRC | 16:26 | |
| *** matrohon has quit IRC | 16:26 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add xenapi support for XenapiLiveMigrateData objects https://review.openstack.org/247853 | 16:27 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make libvirt driver return migrate data objects for source and dest checks https://review.openstack.org/247720 | 16:27 |
| openstackgerrit | Dan Smith proposed openstack/nova: Actually pass the migration data object down to the virt drivers https://review.openstack.org/248211 | 16:27 |
| *** pratikma_ has joined #openstack-nova | 16:27 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire https://review.openstack.org/251588 | 16:27 |
| openstackgerrit | Sean Dague proposed openstack/nova: introduce ``patch`` method to base test class https://review.openstack.org/259096 | 16:28 |
| sdague | johnthetubaguy: ^^ | 16:28 |
| andrearosa | sdague: +1000 on removing mox, thanks for that | 16:28 |
| *** pratikmallya has quit IRC | 16:30 | |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Remove version hardcoding for 'show' method https://review.openstack.org/228578 | 16:32 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Remove version hardcoding for 'download' method https://review.openstack.org/252341 | 16:32 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Remove version hardcoding for 'detail' method https://review.openstack.org/238309 | 16:32 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Bringing Glance v2 images to a uniform format https://review.openstack.org/231696 | 16:32 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Remove version hardcoding for 'delete' method https://review.openstack.org/252334 | 16:32 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Remove version hardcoding for 'create' method https://review.openstack.org/259097 | 16:32 |
| *** scheuran has quit IRC | 16:35 | |
| *** mjura has quit IRC | 16:36 | |
| *** browne has joined #openstack-nova | 16:38 | |
| *** aorourke has quit IRC | 16:38 | |
| *** aorourke has joined #openstack-nova | 16:38 | |
| *** aorourke has quit IRC | 16:39 | |
| *** aorourke has joined #openstack-nova | 16:40 | |
| openstackgerrit | Sean Dague proposed openstack/nova: introduce ``patch`` method to base test class https://review.openstack.org/259096 | 16:41 |
| sdague | oh pep8 unused imports | 16:41 |
| *** marcusvrn_ has joined #openstack-nova | 16:42 | |
| *** arunkant_ has joined #openstack-nova | 16:42 | |
| -openstackstatus- NOTICE: Zuul is moving in very slow motion since roughly 13:30 UTC; the Infra team is investigating. | 16:42 | |
| arunkant_ | Hi..Can someone please review nova keymgr bug fix..https://review.openstack.org/#/c/243322/ | 16:47 |
| openstackgerrit | esracelik proposed openstack/nova: Config options: centralize "configdrive" options https://review.openstack.org/254092 | 16:48 |
| openstackgerrit | Kaitlin Farr proposed openstack/nova-specs: Integrate Castellan for Key Management https://review.openstack.org/247561 | 16:48 |
| *** otter768 has joined #openstack-nova | 16:49 | |
| *** burgerk has joined #openstack-nova | 16:49 | |
| *** davideag_ has joined #openstack-nova | 16:50 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add xenapi support for XenapiLiveMigrateData objects https://review.openstack.org/247853 | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make libvirt driver return migrate data objects for source and dest checks https://review.openstack.org/247720 | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova: Actually pass the migration data object down to the virt drivers https://review.openstack.org/248211 | 16:50 |
| openstackgerrit | Dan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire https://review.openstack.org/251588 | 16:50 |
| sdague | dansmith: what's the story on removing rpc methods from nova-net that are grizzly era? | 16:51 |
| sdague | I noticed there has never been an rpc major bump there, so does that mean removes remain off the table? | 16:51 |
| dansmith | sdague: well, technically it needs a major bump | 16:51 |
| *** davideagnello has quit IRC | 16:51 | |
| *** zenoway has quit IRC | 16:52 | |
| dansmith | sdague: if you mark them now with a comment, we can plan to remove them after m3 | 16:52 |
| dansmith | er, bump them I mean | 16:52 |
| *** zenoway has joined #openstack-nova | 16:52 | |
| *** lpetrut has quit IRC | 16:52 | |
| *** yassine__ has quit IRC | 16:53 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused update_dhcp_hostfile_with_text method https://review.openstack.org/259105 | 16:54 |
| *** otter768 has quit IRC | 16:55 | |
| sdague | dansmith: ok - https://github.com/openstack/nova/blob/master/nova/network/floating_ips.py#L499 is what I was seeing for reference | 16:55 |
| sdague | it actually looks like there is a ton of stuff in there awaiting removal | 16:56 |
| *** davideag_ has quit IRC | 16:56 | |
| *** omnipresent has joined #openstack-nova | 16:57 | |
| *** rsquared has joined #openstack-nova | 16:57 | |
| *** claudiub has quit IRC | 16:58 | |
| rsquared | I've got a working cluster installed via packstack and I would like to add a new compute node. Is there a guide for how to do that? | 16:59 |
| dansmith | sdague: yep | 17:00 |
| dansmith | sdague: if we're going to do it, we should make it automatic like compute | 17:00 |
| *** changbl has quit IRC | 17:00 | |
| *** zenoway has quit IRC | 17:00 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused trigger_handler https://review.openstack.org/259108 | 17:01 |
| *** gszasz has quit IRC | 17:02 | |
| *** gszasz has joined #openstack-nova | 17:03 | |
| *** lykinsbd has joined #openstack-nova | 17:04 | |
| *** diogogmt has quit IRC | 17:04 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: PCI stats will also include device_type in it's pools for PFs https://review.openstack.org/249017 | 17:05 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Make pci_alias parsing logic not assume device_type is set https://review.openstack.org/259110 | 17:05 |
| ndipanov | vladikr, ping | 17:05 |
| *** eharney has quit IRC | 17:05 | |
| ndipanov | vladikr, see https://review.openstack.org/249017 | 17:06 |
| ndipanov | I updated one of your patches with some stuff I needed | 17:06 |
| ndipanov | it basically does less than your original patch though | 17:06 |
| ndipanov | so you may need to add one on top of that | 17:07 |
| ndipanov | I think this can safely be the bottom of the branch so maybe rebase on top of it? | 17:07 |
| ndipanov | anyway take a look and tell me what you think... <- vladikr | 17:07 |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: WIP: Change to use o.vo fixture for compat test https://review.openstack.org/259112 | 17:07 |
| *** diogogmt has joined #openstack-nova | 17:08 | |
| vladikr | ndipanov, sure, looking | 17:08 |
| *** eharney has joined #openstack-nova | 17:09 | |
| *** pkholkin has quit IRC | 17:09 | |
| *** zakora has quit IRC | 17:10 | |
| *** pixelbeat_ has quit IRC | 17:10 | |
| *** Marga_ has joined #openstack-nova | 17:11 | |
| *** apoorvad has joined #openstack-nova | 17:15 | |
| *** jwcroppe has joined #openstack-nova | 17:15 | |
| *** salv-orl_ has joined #openstack-nova | 17:15 | |
| *** snikitin2 has quit IRC | 17:16 | |
| *** smatzek has quit IRC | 17:16 | |
| *** smatzek has joined #openstack-nova | 17:17 | |
| *** salv-orlando has quit IRC | 17:18 | |
| *** Marga_ has quit IRC | 17:18 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Make pci_alias parsing logic not assume device_type is set https://review.openstack.org/259110 | 17:19 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: PCI stats will also include device_type in it's pools for PFs https://review.openstack.org/249017 | 17:19 |
| ndipanov | vladikr, fixed ^ | 17:19 |
| ndipanov | some things got added to the wrong commit | 17:19 |
| *** jistr has quit IRC | 17:19 | |
| *** changbl has joined #openstack-nova | 17:19 | |
| ndipanov | should make more sense now | 17:20 |
| *** unicell has quit IRC | 17:20 | |
| *** yamahata has quit IRC | 17:22 | |
| vladikr | ndipanov, I've missed your comment previously on ps4, here https://review.openstack.org/#/c/249017/4..6/nova/pci/request.py | 17:22 |
| *** bradjones has quit IRC | 17:23 | |
| ndipanov | vladikr, well that's explained in my new commit message - it;s not completely accurate, as you can add it to the whitelist and it will work | 17:23 |
| vladikr | ndipanov, I had 'device_type' working there I've just matched it to the spec , which was 'dev_type' only in the pcidevice object .. | 17:23 |
| ndipanov | vladikr, see my commit message - that way we can't be backwards compatible (I think) | 17:24 |
| *** bradjones has joined #openstack-nova | 17:24 | |
| *** bradjones has quit IRC | 17:24 | |
| *** bradjones has joined #openstack-nova | 17:24 | |
| vladikr | ndipanov, ok, reading, for aliases it would remain 'device_type' , but not sure about whitelist, as I didn't think if we can even add device_type there.. | 17:25 |
| vladikr | ndipanov, looking | 17:25 |
| ndipanov | vladikr, we can't as it breaks the "PF means also all of it's VFs" feature | 17:26 |
| *** jlanoux has quit IRC | 17:27 | |
| ndipanov | someone pointed it out on the spec too | 17:27 |
| vladikr | ndipanov, tbh, not sure what do you mean.. | 17:27 |
| ndipanov | vladikr, see the commit message :) I hope it's not too terse | 17:28 |
| vladikr | ndipanov, yeah :) reading | 17:28 |
| *** danpb has quit IRC | 17:29 | |
| *** Marga_ has joined #openstack-nova | 17:30 | |
| *** gjayavelu has joined #openstack-nova | 17:30 | |
| *** eharney has quit IRC | 17:32 | |
| *** rlrossit has quit IRC | 17:32 | |
| *** pixelbeat_ has joined #openstack-nova | 17:33 | |
| *** lpetrut has joined #openstack-nova | 17:33 | |
| vladikr | ndipanov, so that part "would only match a device that was whitelisted with 'device_type': 'NIC' as part of the whitelist entry like so:" is not right | 17:35 |
| ndipanov | orly | 17:35 |
| vladikr | yea | 17:35 |
| ndipanov | how else do you extend tags? | 17:36 |
| vladikr | sec | 17:36 |
| ndipanov | if you add it to the alias, it will end up in the RequestSpec.spec dict | 17:36 |
| ndipanov | and that has to match a tag of the pool | 17:37 |
| ndipanov | that's how physical_network hack works too <- vladikr | 17:38 |
| vladikr | yea, I have extended the pool_keys in stats with dev_type from what the host is exposing | 17:39 |
| vladikr | and RquestSpec.spec had dev_type in it , instead of the reported device_type | 17:40 |
| ndipanov | yeah | 17:40 |
| vladikr | so it matched | 17:40 |
| ndipanov | but we can't rename it really | 17:40 |
| vladikr | we didn't | 17:40 |
| vladikr | alias still has device_type | 17:40 |
| vladikr | but when we form the requestSpec.spec | 17:40 |
| vladikr | we have dev_type in it | 17:41 |
| vladikr | I have the same version, with the tests working | 17:41 |
| vladikr | I can upload it for you to see, if it still wont make sense we can got with anything else | 17:42 |
| vladikr | s/got/go | 17:42 |
| ndipanov | hmmm | 17:43 |
| ndipanov | that's because you add device_type to pool keys | 17:43 |
| ndipanov | but it's actually called dev_type on the object | 17:44 |
| vladikr | I've added device_type to pool keys | 17:45 |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver: Create Column Allocated https://review.openstack.org/242500 | 17:45 |
| *** dane-fichter has joined #openstack-nova | 17:45 | |
| ndipanov | right, but it will not get picked up from the device object itself | 17:45 |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota: Set default values to subproject https://review.openstack.org/242514 | 17:46 |
| ndipanov | since the actual attribute is called dev_type | 17:46 |
| vladikr | sorry , I have added dev_type tp pool keys | 17:46 |
| ndipanov | but how does dev_type end up in Request.spec then? | 17:46 |
| ndipanov | it's called device_type there | 17:47 |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver: Get Hierarchy info https://review.openstack.org/242568 | 17:47 |
| ndipanov | and we can't rename it - it'll break deployments potentially | 17:47 |
| *** ljxiash has joined #openstack-nova | 17:47 | |
| ndipanov | vladikr, ^ | 17:47 |
| vladikr | right, it's in the requests, when we form the spec, I have changed it from device_type tp dev_type | 17:47 |
| vladikr | I'll paste sec | 17:47 |
| ndipanov | that breaks potential folks who were specifying it in the whitelist | 17:48 |
| *** gjayavelu has quit IRC | 17:48 | |
| vladikr | https://review.openstack.org/#/c/249017/4/nova/pci/request.py | 17:48 |
| *** yamahata has joined #openstack-nova | 17:48 | |
| vladikr | ndipanov, line 118 | 17:48 |
| ndipanov | vladikr, yep - still maybe breaks people | 17:48 |
| *** asselin_ has quit IRC | 17:48 | |
| vladikr | ndipanov, how? they will still specify "device_type" in aliases | 17:49 |
| *** eharney has joined #openstack-nova | 17:49 | |
| *** ociuhandu has quit IRC | 17:49 | |
| vladikr | this is just how we form the spec | 17:49 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-novaclient: Validation for arguments of list command passed by "--fields" https://review.openstack.org/253734 | 17:49 |
| ndipanov | hmmm | 17:49 |
| *** gyee has joined #openstack-nova | 17:49 | |
| ndipanov | so what if they specified it in whitelist too | 17:49 |
| vladikr | ndipanov, I don't actually think they can, but it still doesn't break anything | 17:50 |
| mtanino | sdague: Hi, I updated this based on your comment. Is this fine? https://review.openstack.org/#/c/253734/ | 17:50 |
| ndipanov | they can | 17:50 |
| vladikr | still would be the same behaviour, as we don't touch the way aliases are created | 17:50 |
| sdague | mtanino: +2 | 17:50 |
| *** sahid has quit IRC | 17:51 | |
| mtanino | sdague: thank you for the confirmation. | 17:51 |
| ndipanov | tags (from the whitelist) override pool_keys from the object | 17:51 |
| *** tonytan4ever has quit IRC | 17:52 | |
| ndipanov | so you are right I think | 17:52 |
| vladikr | ndipanov, I will double check that in any case | 17:52 |
| ndipanov | ah no since the alias now says dev_type but pool will have device_type | 17:53 |
| *** e0ne has quit IRC | 17:53 | |
| ndipanov | and dev_type | 17:53 |
| ndipanov | which would not match what was in the alias provided dev_type (read from device_type) | 17:53 |
| vladikr | aliases will still have device_type | 17:53 |
| ndipanov | yes but it will match whatever is in dev_type | 17:54 |
| ndipanov | so if someone had a NIC there | 17:54 |
| ndipanov | it can't match the enum | 17:54 |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver: Create Column Allocated https://review.openstack.org/242500 | 17:54 |
| ndipanov | STANDARD = "type-PCI" | 17:54 |
| ndipanov | SRIOV_PF = "type-PF" | 17:54 |
| ndipanov | SRIOV_VF = "type-VF" | 17:54 |
| ndipanov | vladikr, ^^ | 17:54 |
| vladikr | ndipanov, right, as NIC is not even supported right now | 17:54 |
| vladikr | so we just need raise a correct exception | 17:55 |
| PaulMurray | Happy Christmas everyone - I'm disappearing off now - see you all after the new year | 17:55 |
| ndipanov | well no because people could have been doing "NIC" in whitelist and "NIC" in alias (unlikely but it would work) | 17:56 |
| ndipanov | probably an edge case... | 17:56 |
| *** asselin has joined #openstack-nova | 17:56 | |
| *** PaulMurray has quit IRC | 17:56 | |
| *** gjayavelu has joined #openstack-nova | 17:57 | |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota: Set default values to subproject https://review.openstack.org/242514 | 17:57 |
| ndipanov | vladikr, try that if you have a sec | 17:57 |
| vladikr | ndipanov, I think so far we have completely ignored device_type all together ... | 17:57 |
| vladikr | ndipanov, I'll try | 17:57 |
| *** gjayavelu has quit IRC | 17:58 | |
| *** suro-patz has joined #openstack-nova | 18:00 | |
| ndipanov | vladikr, well you can't ignore it if it's set on the alias | 18:00 |
| *** derekh has quit IRC | 18:00 | |
| ndipanov | and it was a valid thing to set | 18:00 |
| ndipanov | however unnecessary | 18:01 |
| ndipanov | let me just state at this point | 18:01 |
| ndipanov | that those whitelist are a horrible idea | 18:01 |
| *** eglynn has quit IRC | 18:02 | |
| *** eglynn has joined #openstack-nova | 18:02 | |
| ndipanov | vladikr, I'm off for today - talk tomorrow | 18:03 |
| *** ndipanov has quit IRC | 18:03 | |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver: Get Hierarchy info https://review.openstack.org/242568 | 18:03 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove use of _get_regexes in samples tests https://review.openstack.org/257439 | 18:03 |
| openstackgerrit | Sean Dague proposed openstack/nova: Use just ids in all request templates for flavors/images https://review.openstack.org/257438 | 18:03 |
| openstackgerrit | Sean Dague proposed openstack/nova: extract non instance methods https://review.openstack.org/256441 | 18:03 |
| *** pkholkin has joined #openstack-nova | 18:04 | |
| openstackgerrit | Samuel Matzek proposed openstack/nova: glance: secure https when getting client version https://review.openstack.org/254944 | 18:04 |
| *** penick has joined #openstack-nova | 18:07 | |
| *** unicell has joined #openstack-nova | 18:08 | |
| *** browne has quit IRC | 18:09 | |
| *** cdent has quit IRC | 18:10 | |
| *** davideagnello has joined #openstack-nova | 18:11 | |
| *** eharney has quit IRC | 18:11 | |
| *** pkholkin has quit IRC | 18:11 | |
| *** ssurana has joined #openstack-nova | 18:12 | |
| *** lpetrut has quit IRC | 18:12 | |
| *** vilobhmm11 has joined #openstack-nova | 18:13 | |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver https://review.openstack.org/242626 | 18:13 |
| *** klkumar has quit IRC | 18:14 | |
| *** moshele has quit IRC | 18:19 | |
| *** astupnikov has quit IRC | 18:19 | |
| *** garyk has joined #openstack-nova | 18:21 | |
| openstackgerrit | Samuel Matzek proposed openstack/nova: glance: secure https when getting client version https://review.openstack.org/254944 | 18:22 |
| openstackgerrit | Patrick East proposed openstack/nova: libvirt: add discard support for attached volumes https://review.openstack.org/205726 | 18:25 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add way to pause VM during live migration https://review.openstack.org/245921 | 18:26 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add new DB API method to retrieve migration for instance https://review.openstack.org/257270 | 18:26 |
| *** eharney has joined #openstack-nova | 18:26 | |
| *** harlowja has joined #openstack-nova | 18:27 | |
| *** ctrath has quit IRC | 18:29 | |
| openstackgerrit | Patrick East proposed openstack/nova: libvirt: add discard support for attached volumes https://review.openstack.org/205726 | 18:29 |
| *** tjones has joined #openstack-nova | 18:31 | |
| *** sfinucan has quit IRC | 18:32 | |
| dane-fichter | sdague: thanks! | 18:33 |
| *** edtubill has joined #openstack-nova | 18:34 | |
| *** rk4n has quit IRC | 18:34 | |
| *** omnipresent has quit IRC | 18:36 | |
| *** devananda has quit IRC | 18:36 | |
| *** Marga_ has quit IRC | 18:36 | |
| *** aginwala has joined #openstack-nova | 18:36 | |
| *** pixelbeat_ has quit IRC | 18:37 | |
| *** RichardRaseley has joined #openstack-nova | 18:41 | |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: VMware: configure limits during resize operation https://review.openstack.org/204219 | 18:41 |
| sdague | dane-fichter: np | 18:45 |
| *** e0ne has joined #openstack-nova | 18:46 | |
| *** gjayavelu has joined #openstack-nova | 18:47 | |
| sdague | dane-fichter: the patch above it got caught in the alembic break yesterday | 18:47 |
| sdague | so I rechecked that one | 18:47 |
| *** cdent has joined #openstack-nova | 18:48 | |
| sdague | dansmith: I was thinking about that missed lease release issue | 18:50 |
| *** otter768 has joined #openstack-nova | 18:50 | |
| sdague | I wonder if it's worth triggering the cleanup any time we add a new lease | 18:50 |
| dansmith | sdague: ...okay | 18:51 |
| dansmith | I don't think we should, | 18:51 |
| sdague | instead of a periodic heal, just heal on events | 18:51 |
| *** ljxiash has quit IRC | 18:51 | |
| sdague | dansmith: ok | 18:52 |
| dansmith | because the time is/should be related to the lease time | 18:52 |
| dansmith | depending on what you're doing, you may know it's safe to set that interval low, | 18:52 |
| dansmith | but in general it should be lease time, | 18:52 |
| dansmith | especially for something like ironic where you don't have as great an assurance that it's off | 18:52 |
| *** devananda has joined #openstack-nova | 18:52 | |
| dansmith | and even to some degree things like vmware where you're pretty far removed from the hypervisor | 18:53 |
| sdague | right, but the cleanup only happens if it's disassociated | 18:53 |
| dansmith | right, but you don't know that the thing is really off the network, that's the point | 18:53 |
| dansmith | disassociated is just a database thing, | 18:53 |
| *** Marga_ has joined #openstack-nova | 18:53 | |
| dansmith | not really directly tied to whether the driver has completely destroyed the instance | 18:54 |
| *** gyee has quit IRC | 18:54 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: Only attempt to inject files if the injection disk exists https://review.openstack.org/250872 | 18:54 |
| openstackgerrit | Matthew Booth proposed openstack/nova: WIP: libvirt storage pools initial step https://review.openstack.org/259148 | 18:54 |
| *** rk4n has joined #openstack-nova | 18:54 | |
| sdague | ok | 18:54 |
| sdague | well the current patch will at least make us consistent | 18:54 |
| dansmith | yeah | 18:54 |
| sdague | I guess the only question is whether we leak at a high enough rate under normal circumstances that people need to clean up faster | 18:55 |
| *** otter768 has quit IRC | 18:55 | |
| *** browne has joined #openstack-nova | 18:55 | |
| dansmith | well, we're not really leaking | 18:55 |
| dansmith | but yeah, | 18:55 |
| dansmith | and that's where the call comes to decide if you think it's safe to lower your periodic timer | 18:55 |
| sdague | yep, ok | 18:55 |
| *** eharney has quit IRC | 18:56 | |
| *** eharney has joined #openstack-nova | 18:56 | |
| *** aginwala has quit IRC | 18:56 | |
| *** Marga_ has quit IRC | 18:57 | |
| *** rk4n has quit IRC | 18:58 | |
| *** aginwala has joined #openstack-nova | 18:59 | |
| *** ducttape_ has quit IRC | 19:00 | |
| *** tjones has quit IRC | 19:00 | |
| *** burgerk has quit IRC | 19:01 | |
| *** tjones has joined #openstack-nova | 19:02 | |
| *** rk4n has joined #openstack-nova | 19:04 | |
| *** aginwala has quit IRC | 19:04 | |
| *** whenry_ has joined #openstack-nova | 19:05 | |
| *** smatzek has quit IRC | 19:05 | |
| *** jwcroppe has quit IRC | 19:06 | |
| *** rk4n has quit IRC | 19:07 | |
| *** pixelbeat_ has joined #openstack-nova | 19:07 | |
| *** jwcroppe has joined #openstack-nova | 19:09 | |
| *** aginwala has joined #openstack-nova | 19:14 | |
| *** mdrabe has joined #openstack-nova | 19:16 | |
| *** jaosorior has quit IRC | 19:16 | |
| *** aginwala has quit IRC | 19:17 | |
| *** aginwala has joined #openstack-nova | 19:18 | |
| *** ducttape_ has joined #openstack-nova | 19:21 | |
| *** rk4n has joined #openstack-nova | 19:22 | |
| *** dslev has quit IRC | 19:22 | |
| *** zul has quit IRC | 19:25 | |
| *** burt has joined #openstack-nova | 19:26 | |
| *** zenoway has joined #openstack-nova | 19:26 | |
| *** zul has joined #openstack-nova | 19:27 | |
| *** zenoway has quit IRC | 19:30 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Changed filter_by() to filter() during filtering instances in db API https://review.openstack.org/245677 | 19:31 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added db API layer to add instance tag-list filtering support https://review.openstack.org/248805 | 19:31 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/128940 | 19:31 |
| openstackgerrit | jichenjc proposed openstack/nova: Support mac and type for /servers/ips request https://review.openstack.org/257261 | 19:33 |
| *** e0ne has quit IRC | 19:34 | |
| *** ianbrown has quit IRC | 19:35 | |
| *** ianbrown has joined #openstack-nova | 19:35 | |
| jroll | dansmith: mind landing this to get it off my list? https://review.openstack.org/#/c/258687/ | 19:36 |
| dansmith | jroll: doneski | 19:37 |
| jroll | thanks | 19:37 |
| *** cdent has quit IRC | 19:40 | |
| *** rdopiera has joined #openstack-nova | 19:45 | |
| *** zenoway has joined #openstack-nova | 19:52 | |
| openstackgerrit | Sven Anderson proposed openstack/nova: Update the image size according to datastore https://review.openstack.org/258020 | 19:54 |
| openstackgerrit | Raildo Mascena proposed openstack/nova: Nested Quota Driver: Create Column Allocated https://review.openstack.org/242500 | 19:54 |
| *** smatzek has joined #openstack-nova | 19:55 | |
| *** ducttape_ has quit IRC | 19:55 | |
| *** zenoway has quit IRC | 19:57 | |
| *** lucasagomes is now known as lucas-dinner | 19:57 | |
| *** rk4n has quit IRC | 19:59 | |
| *** mnestratov has quit IRC | 20:00 | |
| *** ducttape_ has joined #openstack-nova | 20:00 | |
| *** rfolco has quit IRC | 20:00 | |
| *** salv-orl_ has quit IRC | 20:01 | |
| *** aysyd has quit IRC | 20:01 | |
| *** zenoway has joined #openstack-nova | 20:06 | |
| *** cdent has joined #openstack-nova | 20:06 | |
| *** gjayavelu has quit IRC | 20:08 | |
| *** toure is now known as toure|biab | 20:08 | |
| *** aginwala has quit IRC | 20:09 | |
| *** edtubill has quit IRC | 20:13 | |
| migi | dansmith, would appriciate if you could take a look at next 2 reviews from uuids | 20:17 |
| dansmith | migi: I will, I have them open in my queue | 20:17 |
| migi | dansmith, you have one in a time opportunity to give 2x - | 20:17 |
| migi | ok | 20:17 |
| migi | thanks | 20:17 |
| migi | :) | 20:17 |
| migi | no rush though.. | 20:18 |
| migi | learned how to stack one on top of another on top of another... :) | 20:18 |
| dansmith | cool | 20:19 |
| *** Marga_ has joined #openstack-nova | 20:22 | |
| *** tonytan4ever has joined #openstack-nova | 20:24 | |
| *** gszasz has quit IRC | 20:24 | |
| *** markvoelker has quit IRC | 20:24 | |
| *** diogogmt has quit IRC | 20:26 | |
| *** e0ne has joined #openstack-nova | 20:27 | |
| *** omnipresent has joined #openstack-nova | 20:27 | |
| *** aginwala has joined #openstack-nova | 20:28 | |
| *** aginwala has quit IRC | 20:29 | |
| *** matrohon has joined #openstack-nova | 20:30 | |
| *** zenoway has quit IRC | 20:30 | |
| *** tjones has quit IRC | 20:30 | |
| *** edtubill has joined #openstack-nova | 20:31 | |
| *** dane-fichter has quit IRC | 20:32 | |
| *** tjones has joined #openstack-nova | 20:33 | |
| *** pratikma_ is now known as pratikmallya | 20:35 | |
| *** jwcroppe has quit IRC | 20:35 | |
| openstackgerrit | Fawad Khaliq proposed openstack/nova: remove network information from IOvisor vif https://review.openstack.org/259176 | 20:38 |
| sdague | migi: you got links, I can check them out as well | 20:40 |
| *** smatzek has quit IRC | 20:40 | |
| migi | sdague: https://review.openstack.org/#/q/owner:%22Michal+Pryc%22+status:open | 20:40 |
| migi | there are 2 | 20:40 |
| *** jwcroppe has joined #openstack-nova | 20:41 | |
| sdague | looks good | 20:42 |
| openstackgerrit | Fawad Khaliq proposed openstack/nova: remove network information from IOvisor vif https://review.openstack.org/259176 | 20:42 |
| migi | sdague: thx | 20:43 |
| *** smatzek has joined #openstack-nova | 20:43 | |
| sdague | the ceph job is acting up again, are there outstanding patches to try to make that better? | 20:44 |
| *** zenoway has joined #openstack-nova | 20:45 | |
| vilobhmm11 | alaski : can you please have a look https://review.openstack.org/#/c/253306/ | 20:46 |
| vilobhmm11 | flavor classes | 20:46 |
| *** dslevin has quit IRC | 20:48 | |
| *** omnipresent has quit IRC | 20:48 | |
| *** dave-mccowan has quit IRC | 20:50 | |
| *** otter768 has joined #openstack-nova | 20:51 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused extends_name attribute https://review.openstack.org/259179 | 20:54 |
| tonyb | meeting in <10mins right? | 20:54 |
| sdague | yeh | 20:56 |
| dansmith | correction | 20:56 |
| dansmith | "short meeting in 10 minutes" | 20:56 |
| sdague | dansmith: you running it today? | 20:56 |
| dansmith | no | 20:56 |
| *** otter768 has quit IRC | 20:56 | |
| dansmith | but I want it to be short, and getting the terminology is an important step in the psychology | 20:56 |
| sdague | heh | 20:56 |
| sdague | who is running it, it's normally mriedem right? | 20:57 |
| *** daemontool has quit IRC | 20:57 | |
| dansmith | normally mikal | 20:57 |
| sdague | ah, ok | 20:57 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused methods from integrated_helpers test class https://review.openstack.org/259180 | 20:57 |
| sdague | dansmith: so, following up a statement ealier in the week - https://pypi.python.org/pypi/vulture | 20:58 |
| *** daemontool has joined #openstack-nova | 20:58 | |
| sdague | it's got a lot of false positives | 20:58 |
| *** smatzek has quit IRC | 20:58 | |
| dansmith | yeah | 20:58 |
| sdague | however, it makes for a reasonable list to actually inspect | 20:58 |
| dansmith | cool | 20:58 |
| sdague | it gets subclasses really wrong | 20:58 |
| sdague | but unused methods seem to be a reasonable hit rate | 20:59 |
| *** gjayavelu has joined #openstack-nova | 20:59 | |
| *** rsquared has quit IRC | 20:59 | |
| mikal | dansmith: nope, not me | 21:00 |
| dansmith | oh, do we get a pass this week? | 21:00 |
| dansmith | I'm tooootally down for that | 21:00 |
| mikal | dansmith: as in, I am unsure who johnthetubaguy has lined up to run it | 21:00 |
| *** aginwala has joined #openstack-nova | 21:01 | |
| *** raildo is now known as raildo-afk | 21:01 | |
| sdague | which meeting room is it normally in? | 21:01 |
| melwitt | maybe he forgot mriedem was going on vacation | 21:01 |
| mikal | sdague: Just openstack-meeting | 21:02 |
| tonyb | someone that can type better that I can should # startmeeting ;P | 21:02 |
| dansmith | is there anything important on the agenda? | 21:02 |
| sdague | ok, lets just do it quick | 21:02 |
| *** pixelbeat_ has quit IRC | 21:03 | |
| *** lykinsbd has quit IRC | 21:04 | |
| *** lykinsbd has joined #openstack-nova | 21:05 | |
| *** salv-orl_ has joined #openstack-nova | 21:06 | |
| openstackgerrit | Michael Still proposed openstack/nova: Remove the old object_compat wrapper. https://review.openstack.org/257686 | 21:06 |
| *** dslevin has joined #openstack-nova | 21:07 | |
| mikal | What does NOT_REGISTERED mean as a test result from jenkins? | 21:07 |
| mikal | It seems to count as a failure? | 21:07 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Allow multiple volume attachments https://review.openstack.org/193135 | 21:10 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 21:10 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: obj: Allow multiple volume attachments https://review.openstack.org/193133 | 21:10 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: db: Allow multiple volume attachments https://review.openstack.org/153033 | 21:10 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach https://review.openstack.org/153038 | 21:10 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: virt: Pass host when call attach to Cinder https://review.openstack.org/256273 | 21:10 |
| *** markus_z has joined #openstack-nova | 21:11 | |
| *** markus_z has left #openstack-nova | 21:11 | |
| dansmith | sdague: reasonable job | 21:11 |
| openstackgerrit | jichenjc proposed openstack/nova: Support mac and type for /servers/ips request https://review.openstack.org/257261 | 21:12 |
| dims | just under 12:00 mins :) | 21:12 |
| sdague | mikal: NOT_REGISTERED means that zuul asked jenkins to run a job by name | 21:12 |
| sdague | and jenkins was all "what you talking about willis" | 21:12 |
| *** ziggit has quit IRC | 21:12 | |
| sdague | because it's job definitions weren't up to date | 21:12 |
| cdent | sdague: I could probably convince myself that cross-projects and I were a good match, if other people thought so | 21:13 |
| mikal | sdague: was that expected fallout from the gerrit upgrade? | 21:13 |
| mikal | sdague: i.e. a recheck for the win? | 21:13 |
| *** dave-mccowan has joined #openstack-nova | 21:13 | |
| sdague | cdent: I think that would be cool | 21:13 |
| sdague | I'd just say circle with johnthetubaguy and raise your hand | 21:13 |
| sdague | mikal: I don't think it was expected | 21:14 |
| sdague | however it did seem to happen for about the first hour after the change window | 21:14 |
| mikal | Ok, so a recheck then. Done. | 21:14 |
| * cdent inquires with johnthetubaguy with a querying glance at jaypipes | 21:14 | |
| sdague | mikal: and the price of that answer, review my glance config fixes - https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:glance_image_config | 21:14 |
| *** aloga has quit IRC | 21:14 | |
| mikal | sdague: I can't just rage quit instead? | 21:14 |
| *** dave-mcc_ has joined #openstack-nova | 21:15 | |
| *** zenoway has quit IRC | 21:15 | |
| sdague | if that's the way you want to spend your "I saved your life" credit | 21:15 |
| mikal | Heh | 21:15 |
| mikal | Ok, I will review this thing post shower | 21:15 |
| *** zenoway has joined #openstack-nova | 21:15 | |
| mikal | And keep my "I saved your life" credit for another time | 21:16 |
| *** matrohon has quit IRC | 21:16 | |
| *** rdopiera has quit IRC | 21:16 | |
| *** dave-mccowan has quit IRC | 21:18 | |
| sdague | dansmith: the leased ips fix is in the gate finally | 21:19 |
| *** edleafe- has joined #openstack-nova | 21:19 | |
| * cdent fights the power | 21:23 | |
| *** aginwala has quit IRC | 21:24 | |
| *** krtaylor has quit IRC | 21:24 | |
| *** markvoelker has joined #openstack-nova | 21:25 | |
| *** edleafe- has quit IRC | 21:27 | |
| *** edleafe has quit IRC | 21:27 | |
| *** aginwala has joined #openstack-nova | 21:28 | |
| *** mdrabe has quit IRC | 21:28 | |
| *** edleafe has joined #openstack-nova | 21:29 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused pipeline_factory_v3 alias https://review.openstack.org/259185 | 21:29 |
| *** rotbeard has quit IRC | 21:29 | |
| *** gjayavelu has quit IRC | 21:29 | |
| *** markvoelker has quit IRC | 21:30 | |
| *** gjayavelu has joined #openstack-nova | 21:30 | |
| sdague | cdent: which power are you fighting? | 21:30 |
| cdent | I left a contrary opinion on the mission statement review | 21:30 |
| cdent | It could probably have been written much more shortly as "won't someone please think of the users" | 21:31 |
| *** omnipresent has joined #openstack-nova | 21:31 | |
| *** zenoway has quit IRC | 21:34 | |
| sdague | cdent: did you read the defcore review? | 21:34 |
| *** ziggit has joined #openstack-nova | 21:34 | |
| *** gjayavelu has quit IRC | 21:34 | |
| cdent | sdague: the one that was being reviewed during the tc meeting? | 21:35 |
| sdague | cdent: well, linked to | 21:36 |
| sdague | https://review.openstack.org/#/c/244782/ | 21:36 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove unused EXTENSION_DESERIALIZE_* constants https://review.openstack.org/259189 | 21:36 |
| *** browne has quit IRC | 21:36 | |
| cdent | sdague: no, not that one, just monty's thing in response | 21:37 |
| *** krtaylor has joined #openstack-nova | 21:37 | |
| sdague | ok, I'd say start with the defcore one | 21:38 |
| cdent | I've never disputed the argument there: I'm completely in agreement with what you and monty have stated so well. | 21:39 |
| sdague | oh, I'm reading your comments now, I agree with your comment | 21:39 |
| cdent | What I'm disagreeing with is _only_ the limitations on the breadth of nova | 21:39 |
| *** browne has joined #openstack-nova | 21:41 | |
| cdent | It's amusing to me that these defcore discussion involve people I've never heard of. | 21:42 |
| sdague | cdent: yeh, I'm with you | 21:42 |
| sdague | yeh, defcore is a charter of the board | 21:42 |
| sdague | so mostly not people you see in the technical community | 21:43 |
| openstackgerrit | Sean Dague proposed openstack/nova: remove vestigial XML_NS_V11 variable https://review.openstack.org/259191 | 21:43 |
| sdague | god there are some weird left over crud from the xml removal | 21:43 |
| cdent | sdague++ this is a good summary "feel like it's ok for the TC to declare that your cloud must be able to do X to be OpenStack, even if Nova can do X, Y, and Z." | 21:44 |
| cdent | You've used your verbal tick in your comment: honestly-as-a-prefix | 21:44 |
| cdent | Honestly, when I don't use the prefix "honestly" I'm not being honest. | 21:44 |
| cdent | ;) | 21:44 |
| sdague | heh | 21:45 |
| sdague | well communication includes emphasis :) | 21:46 |
| sdague | but, yes, I write like I talk | 21:46 |
| auggy | or "literally" that's another good prefix | 21:46 |
| cdent | It's probably a good thing: It makes you recognizable. | 21:46 |
| cdent | I think I became sensitive to it when I moved to the UK, where be use the phrase "to be honest" or "to be fair" as a prefix to sort of disclaim in a similar way to "I'm not a racist, but..." | 21:47 |
| cdent | s/where be/where people/ | 21:47 |
| tonyb | you're not sdague! sdague you have said "{so,honnestly,literally} ..." there :D | 21:47 |
| cdent | ha! | 21:48 |
| auggy | oh we're specifically referring to sdague prefixes! i just thought we were referring to prefixes in general | 21:48 |
| cdent | impostor! | 21:48 |
| *** zenoway has joined #openstack-nova | 21:52 | |
| *** omnipresent has quit IRC | 21:55 | |
| kfox1111 | nova-novnc service, if making ha, can that be active/active, or is it stateless and shoudl be active/passive? | 21:58 |
| kfox1111 | I'm trying to make all of the stuff we have that is active/active be on one set of nodes, where we should be able to drain and reboot nodes at whim, and keep the rest on a set of higher availability nodes. | 21:59 |
| *** lykinsbd has quit IRC | 21:59 | |
| sdague | heh | 21:59 |
| sdague | ok, time to call it a day folks. See yuo all tomorrow. | 22:02 |
| *** aginwala_ has joined #openstack-nova | 22:04 | |
| *** aginwala has quit IRC | 22:07 | |
| *** thorst_ has quit IRC | 22:07 | |
| *** ianbrown has quit IRC | 22:07 | |
| *** ianbrown has joined #openstack-nova | 22:08 | |
| *** gjayavelu has joined #openstack-nova | 22:08 | |
| *** rk4n has joined #openstack-nova | 22:10 | |
| *** omnipresent has joined #openstack-nova | 22:11 | |
| *** xyang1 has quit IRC | 22:12 | |
| *** dustins is now known as dustins|out | 22:12 | |
| *** gyee has joined #openstack-nova | 22:13 | |
| *** rk4n has quit IRC | 22:15 | |
| *** zenoway has quit IRC | 22:15 | |
| *** rk4n has joined #openstack-nova | 22:18 | |
| *** pumaranikar has quit IRC | 22:18 | |
| *** pumaranikar has joined #openstack-nova | 22:18 | |
| *** eharney has quit IRC | 22:18 | |
| *** aginwala_ has quit IRC | 22:21 | |
| *** aginwala has joined #openstack-nova | 22:22 | |
| *** rk4n has quit IRC | 22:22 | |
| *** rk4n has joined #openstack-nova | 22:22 | |
| *** [1]Thelo has joined #openstack-nova | 22:23 | |
| *** sturlry_ has joined #openstack-nova | 22:24 | |
| *** aginwala has quit IRC | 22:24 | |
| *** aginwala has joined #openstack-nova | 22:25 | |
| *** Thelo has quit IRC | 22:25 | |
| *** [1]Thelo is now known as Thelo | 22:25 | |
| *** rk4n has quit IRC | 22:27 | |
| *** aginwala has quit IRC | 22:27 | |
| *** aginwala has joined #openstack-nova | 22:27 | |
| *** pumaranikar has quit IRC | 22:29 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: Make project_id optional in v2.1 urls https://review.openstack.org/233076 | 22:31 |
| auggy | sdague: gnite! | 22:31 |
| *** alejandrito has joined #openstack-nova | 22:32 | |
| auggy | sdague: the microversion needed to be bumped to 2.15, a change landed for 2.14 | 22:32 |
| *** aginwala has quit IRC | 22:32 | |
| *** markvoelker has joined #openstack-nova | 22:33 | |
| *** aginwala has joined #openstack-nova | 22:33 | |
| *** dslevin has quit IRC | 22:35 | |
| *** rohit_ has quit IRC | 22:41 | |
| *** mwagner_lap has quit IRC | 22:42 | |
| *** dslevin has joined #openstack-nova | 22:43 | |
| *** rohit_ has joined #openstack-nova | 22:43 | |
| *** zenoway has joined #openstack-nova | 22:44 | |
| *** e0ne has quit IRC | 22:47 | |
| *** omnipresent has quit IRC | 22:47 | |
| vilobhmm11 | bauzas : ping | 22:48 |
| *** aloga has joined #openstack-nova | 22:49 | |
| *** gjayavelu has quit IRC | 22:49 | |
| *** moshele has joined #openstack-nova | 22:51 | |
| *** otter768 has joined #openstack-nova | 22:52 | |
| *** ducttape_ has quit IRC | 22:53 | |
| *** pradk has quit IRC | 22:53 | |
| *** jwcroppe has quit IRC | 22:53 | |
| *** jwcroppe has joined #openstack-nova | 22:54 | |
| *** omnipresent has joined #openstack-nova | 22:56 | |
| *** otter768 has quit IRC | 22:56 | |
| *** rk4n has joined #openstack-nova | 22:57 | |
| *** jwcroppe has quit IRC | 22:59 | |
| *** zenoway has quit IRC | 22:59 | |
| *** zhangjn has joined #openstack-nova | 23:03 | |
| *** aloga has quit IRC | 23:06 | |
| *** aloga has joined #openstack-nova | 23:08 | |
| *** edtubill has quit IRC | 23:08 | |
| *** zenoway has joined #openstack-nova | 23:11 | |
| mikal | sdague: you still around? | 23:13 |
| mikal | sdague: cause I like totally -1'ed your patch series | 23:13 |
| mikal | Cause I am poorly socialized | 23:13 |
| *** omnipresent has quit IRC | 23:15 | |
| *** ianbrown has quit IRC | 23:15 | |
| *** zenoway has quit IRC | 23:16 | |
| *** ianbrown has joined #openstack-nova | 23:16 | |
| *** pixelbeat_ has joined #openstack-nova | 23:18 | |
| *** aginwala has quit IRC | 23:21 | |
| *** Daisy has joined #openstack-nova | 23:23 | |
| *** zhangjn has quit IRC | 23:23 | |
| *** dave-mcc_ has quit IRC | 23:23 | |
| *** zenoway has joined #openstack-nova | 23:28 | |
| *** tonytan4ever has quit IRC | 23:28 | |
| *** aginwala has joined #openstack-nova | 23:29 | |
| *** zenoway has quit IRC | 23:33 | |
| alex_xu | mikal: morning, appreciate if you can help on this https://review.openstack.org/255122, the developer still wait for this correction, then start the coding. thanks in advance! | 23:34 |
| *** Daisy has quit IRC | 23:36 | |
| *** Daisy has joined #openstack-nova | 23:37 | |
| *** Daisy has quit IRC | 23:41 | |
| *** darrenc is now known as darrenc_afk | 23:44 | |
| *** erlon has quit IRC | 23:46 | |
| *** jaypipes has quit IRC | 23:47 | |
| *** moshele has quit IRC | 23:48 | |
| *** ziggit has quit IRC | 23:50 | |
| *** pratikmallya has quit IRC | 23:51 | |
| *** zenoway has joined #openstack-nova | 23:53 | |
| *** cdent has quit IRC | 23:53 | |
| *** zenoway has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!