*** browne has joined #openstack-nova | 00:02 | |
*** SourabhP has quit IRC | 00:04 | |
*** tongli has quit IRC | 00:05 | |
*** tongli has joined #openstack-nova | 00:06 | |
*** tongli has quit IRC | 00:10 | |
*** jecarey has quit IRC | 00:12 | |
*** Piet has joined #openstack-nova | 00:14 | |
*** irenab has joined #openstack-nova | 00:17 | |
*** ssurana has joined #openstack-nova | 00:17 | |
*** thorst has quit IRC | 00:18 | |
*** yamahata has joined #openstack-nova | 00:18 | |
*** irenab has quit IRC | 00:21 | |
*** dims has quit IRC | 00:21 | |
*** tongli has joined #openstack-nova | 00:22 | |
*** yingjun has joined #openstack-nova | 00:24 | |
ijw_ | Anyone know why the VIF binding blueprint got rejected? | 00:26 |
---|---|---|
ijw_ | The comment on it amounts to 'I'll explain later' | 00:26 |
*** jamielennox is now known as jamielennox|away | 00:29 | |
*** diogogmt has joined #openstack-nova | 00:29 | |
mriedem | ijw_: link? | 00:29 |
mriedem | andrewbogott: re: https://review.openstack.org/#/c/197185/ - did you get a blueprint posted? i was going to add that to the nova meeting agenda for tomorrow when talking about specless blueprints | 00:30 |
andrewbogott | mriedem: I’m planning to work on that tonight, I’ll ping you when I have something. | 00:30 |
*** ijw_ has quit IRC | 00:33 | |
*** jwcroppe has quit IRC | 00:35 | |
mriedem | andrewbogott: ok, i added the review to the agenda anyway in case we have time | 00:36 |
andrewbogott | thx | 00:36 |
*** jamielennox|away is now known as jamielennox | 00:39 | |
andrewbogott | mriedem: Here’s the outline: https://blueprints.launchpad.net/nova/+spec/instance-id-in-metadata | 00:40 |
andrewbogott | Shall I explain use cases on the whiteboard or is it better to make a wiki page? | 00:40 |
*** r-daneel has quit IRC | 00:40 | |
*** thorst has joined #openstack-nova | 00:43 | |
andrewbogott | johnthetubaguy: same question? | 00:45 |
*** otter768 has joined #openstack-nova | 00:46 | |
*** Jeffrey4l has quit IRC | 00:49 | |
*** Jeffrey4l has joined #openstack-nova | 00:50 | |
*** tsekiyam_ has joined #openstack-nova | 00:50 | |
mriedem | andrewbogott: just put it in the description or whiteboard | 00:50 |
andrewbogott | ‘k | 00:51 |
*** otter768 has quit IRC | 00:51 | |
*** Piet has quit IRC | 00:51 | |
*** mtanino has quit IRC | 00:53 | |
*** tsekiyama has quit IRC | 00:54 | |
*** tsekiyam_ has quit IRC | 00:55 | |
*** Piet has joined #openstack-nova | 00:57 | |
*** rdo has quit IRC | 00:58 | |
mriedem | johnthetubaguy: this is complete https://blueprints.launchpad.net/nova/+spec/allow-ip6-search-for-non-admin - the changes are all merged | 00:58 |
mriedem | just need to update the bp | 00:58 |
*** piyanai has joined #openstack-nova | 01:00 | |
andrewbogott | mriedem: ok, all done (although I’m going to submit a new patch version in a moment.) Is there any need for me to attend the nova meeting to state my case? | 01:00 |
*** jgriffit1 has joined #openstack-nova | 01:00 | |
*** thorst has quit IRC | 01:00 | |
openstackgerrit | Andrew Bogott proposed openstack/nova: Include project_id in instance metadata. https://review.openstack.org/197185 | 01:01 |
mriedem | andrewbogott: don't think so | 01:02 |
andrewbogott | great :) | 01:02 |
mriedem | it's a straight forward change, the question is process for us really | 01:02 |
mriedem | andrewbogott: remember to link your code chagne into the bp with the blueprint ref link in your commit message :) | 01:03 |
mriedem | so the change is listed in the whiteboard of the bp | 01:03 |
andrewbogott | ah, right | 01:03 |
*** asettle has quit IRC | 01:04 | |
openstackgerrit | Andrew Bogott proposed openstack/nova: Include project_id in instance metadata. https://review.openstack.org/197185 | 01:04 |
*** rdo has joined #openstack-nova | 01:05 | |
*** jwcroppe has joined #openstack-nova | 01:06 | |
*** eglynn has joined #openstack-nova | 01:07 | |
*** jgriffit1 has quit IRC | 01:10 | |
*** amotoki has joined #openstack-nova | 01:10 | |
*** jgriffit1 has joined #openstack-nova | 01:10 | |
*** ssurana has quit IRC | 01:11 | |
*** oomichi has joined #openstack-nova | 01:11 | |
*** LukeHinds has quit IRC | 01:12 | |
*** Jeffrey4l has quit IRC | 01:14 | |
*** pixelbeat has joined #openstack-nova | 01:14 | |
*** jwcroppe has quit IRC | 01:17 | |
*** yingjun has quit IRC | 01:19 | |
*** yingjun has joined #openstack-nova | 01:19 | |
*** armax has quit IRC | 01:21 | |
*** arnaud___ has joined #openstack-nova | 01:21 | |
*** jgriffit1 has quit IRC | 01:26 | |
*** arnaud___ has quit IRC | 01:26 | |
*** bdr7 has joined #openstack-nova | 01:27 | |
*** tjones1 has quit IRC | 01:27 | |
*** shuquan has joined #openstack-nova | 01:27 | |
*** tpeoples has joined #openstack-nova | 01:28 | |
*** shuquan_ has joined #openstack-nova | 01:31 | |
*** asettle has joined #openstack-nova | 01:32 | |
*** shuquan has quit IRC | 01:32 | |
*** shuquan_ is now known as shuquan | 01:32 | |
*** lyanchih_ has joined #openstack-nova | 01:34 | |
*** jasondotstar has joined #openstack-nova | 01:35 | |
*** mriedem has quit IRC | 01:42 | |
*** bitblt has quit IRC | 01:45 | |
*** vilobhmm has quit IRC | 01:45 | |
*** oomichi has quit IRC | 01:46 | |
*** Sukhdev has quit IRC | 01:48 | |
*** tongli has quit IRC | 01:48 | |
anish | can any of the core reviewers take a look at this bugfix https://review.openstack.org/#/c/193414/ , it's been up for review for a while | 01:55 |
*** unicell1 has quit IRC | 02:00 | |
*** baoli has joined #openstack-nova | 02:02 | |
*** baoli_ has joined #openstack-nova | 02:04 | |
*** davideagnello has quit IRC | 02:05 | |
*** baoli has quit IRC | 02:07 | |
anish | lyanchih_: thanks :) | 02:08 |
*** shuquan has quit IRC | 02:08 | |
*** yamahata has quit IRC | 02:10 | |
*** Sukhdev has joined #openstack-nova | 02:11 | |
*** mutoulbj has joined #openstack-nova | 02:17 | |
*** shuquan has joined #openstack-nova | 02:17 | |
*** vilobhmm has joined #openstack-nova | 02:19 | |
*** Sukhdev has quit IRC | 02:20 | |
*** achanda has quit IRC | 02:21 | |
*** achanda has joined #openstack-nova | 02:21 | |
*** achanda has quit IRC | 02:22 | |
*** arnaud___ has joined #openstack-nova | 02:22 | |
*** annashen has joined #openstack-nova | 02:26 | |
*** arnaud___ has quit IRC | 02:26 | |
*** gyee has quit IRC | 02:28 | |
*** Jeffrey4l has joined #openstack-nova | 02:29 | |
*** mutoulbj_ has joined #openstack-nova | 02:33 | |
*** vilobhmm has quit IRC | 02:33 | |
*** vilobhmm has joined #openstack-nova | 02:33 | |
*** r-daneel has joined #openstack-nova | 02:35 | |
*** lyanchih_ has quit IRC | 02:36 | |
*** mutoulbj has quit IRC | 02:36 | |
*** annashen has quit IRC | 02:42 | |
*** annashen has joined #openstack-nova | 02:42 | |
*** annashen has quit IRC | 02:43 | |
*** otter768 has joined #openstack-nova | 02:47 | |
*** patrickeast has quit IRC | 02:49 | |
*** otter768 has quit IRC | 02:52 | |
*** hakimo has joined #openstack-nova | 02:52 | |
*** hakimo_ has quit IRC | 02:54 | |
*** ssurana has joined #openstack-nova | 02:54 | |
*** r-daneel has quit IRC | 02:55 | |
openstackgerrit | Jeffrey Zhang proposed openstack/nova: use graduated oslo.policy https://review.openstack.org/198065 | 02:56 |
*** lyanchih has joined #openstack-nova | 02:57 | |
*** VW_ has joined #openstack-nova | 02:57 | |
openstackgerrit | Andrew Bogott proposed openstack/nova: Include project_id in instance metadata. https://review.openstack.org/197185 | 02:59 |
*** shyu has joined #openstack-nova | 03:02 | |
*** humble_ has joined #openstack-nova | 03:02 | |
*** VW_ has quit IRC | 03:04 | |
*** VW_ has joined #openstack-nova | 03:05 | |
*** wanghua has quit IRC | 03:06 | |
*** bdr7 has quit IRC | 03:06 | |
*** bdr7 has joined #openstack-nova | 03:07 | |
*** xyang1 has quit IRC | 03:09 | |
*** VW_ has quit IRC | 03:10 | |
*** andrewbogott is now known as andrewbogott_afk | 03:14 | |
*** armax has joined #openstack-nova | 03:16 | |
*** jdandrea has quit IRC | 03:16 | |
*** annashen has joined #openstack-nova | 03:17 | |
*** davideagnello has joined #openstack-nova | 03:19 | |
*** jamielennox is now known as jamielennox|away | 03:22 | |
*** achanda has joined #openstack-nova | 03:26 | |
*** bdr7 has quit IRC | 03:30 | |
*** jamielennox|away is now known as jamielennox | 03:31 | |
*** dave-mccowan has quit IRC | 03:33 | |
*** piyanai has quit IRC | 03:34 | |
*** garyk has quit IRC | 03:34 | |
*** achanda has quit IRC | 03:34 | |
*** piyanai has joined #openstack-nova | 03:34 | |
*** Kennan2 has joined #openstack-nova | 03:35 | |
*** Kennan has quit IRC | 03:36 | |
*** shuquan has quit IRC | 03:37 | |
*** amotoki has quit IRC | 03:37 | |
*** shuquan has joined #openstack-nova | 03:37 | |
*** jasondotstar has quit IRC | 03:37 | |
*** diogogmt has quit IRC | 03:40 | |
*** shuquan has quit IRC | 03:44 | |
*** dims_ has joined #openstack-nova | 03:46 | |
*** dims_ has quit IRC | 03:46 | |
*** dims_ has joined #openstack-nova | 03:46 | |
*** piyanai has quit IRC | 03:47 | |
*** Kevin_Zheng_ has joined #openstack-nova | 03:48 | |
*** yingjun has quit IRC | 03:51 | |
*** kiran-r has joined #openstack-nova | 03:51 | |
*** arnaud___ has joined #openstack-nova | 03:51 | |
*** amotoki has joined #openstack-nova | 03:51 | |
*** pixelbeat has quit IRC | 03:51 | |
*** jwcroppe has joined #openstack-nova | 03:54 | |
*** achanda has joined #openstack-nova | 04:00 | |
*** Marga_ has joined #openstack-nova | 04:02 | |
*** vishwanathj has quit IRC | 04:02 | |
*** Marga__ has joined #openstack-nova | 04:02 | |
*** vishwanathj has joined #openstack-nova | 04:03 | |
*** lyanchih has quit IRC | 04:05 | |
*** piyanai has joined #openstack-nova | 04:05 | |
*** Jeffrey4l has quit IRC | 04:05 | |
*** achanda has quit IRC | 04:06 | |
*** Marga_ has quit IRC | 04:07 | |
*** vishwana_ has joined #openstack-nova | 04:07 | |
*** markvoelker has quit IRC | 04:08 | |
*** vishwanathj has quit IRC | 04:09 | |
*** dims_ has quit IRC | 04:14 | |
*** dims has joined #openstack-nova | 04:15 | |
*** amotoki has quit IRC | 04:17 | |
*** dims has quit IRC | 04:20 | |
*** baoli_ has quit IRC | 04:22 | |
*** amotoki has joined #openstack-nova | 04:23 | |
*** achanda has joined #openstack-nova | 04:24 | |
*** kiran-r has quit IRC | 04:25 | |
*** vilobhmm has quit IRC | 04:30 | |
*** tojuvone has joined #openstack-nova | 04:30 | |
*** shyama has joined #openstack-nova | 04:31 | |
*** amotoki has quit IRC | 04:33 | |
*** vilobhmm has joined #openstack-nova | 04:34 | |
*** piyanai has quit IRC | 04:37 | |
*** eglynn has quit IRC | 04:39 | |
*** garyk has joined #openstack-nova | 04:41 | |
*** amotoki has joined #openstack-nova | 04:43 | |
*** wanghua has joined #openstack-nova | 04:43 | |
*** humble_ has quit IRC | 04:46 | |
*** mutoulbj_ has quit IRC | 04:46 | |
*** otter768 has joined #openstack-nova | 04:48 | |
*** annashen has quit IRC | 04:50 | |
*** tpeoples has quit IRC | 04:51 | |
*** unicell has joined #openstack-nova | 04:52 | |
*** mutoulbj has joined #openstack-nova | 04:52 | |
*** otter768 has quit IRC | 04:53 | |
*** ildikov has quit IRC | 04:55 | |
*** _gryf|afk has quit IRC | 04:58 | |
*** dims has joined #openstack-nova | 05:01 | |
*** dims_ has joined #openstack-nova | 05:02 | |
*** _gryf has joined #openstack-nova | 05:03 | |
_gryf | o/ | 05:03 |
*** arnaud___ has quit IRC | 05:04 | |
*** dims has quit IRC | 05:06 | |
*** shyama has quit IRC | 05:08 | |
*** shyama has joined #openstack-nova | 05:09 | |
*** mutoulbj has quit IRC | 05:11 | |
*** arnaud___ has joined #openstack-nova | 05:14 | |
*** dims_ has quit IRC | 05:14 | |
*** mutoulbj has joined #openstack-nova | 05:15 | |
*** nkrinner has joined #openstack-nova | 05:16 | |
*** humble_ has joined #openstack-nova | 05:20 | |
*** wanghua has quit IRC | 05:24 | |
*** Sukhdev has joined #openstack-nova | 05:25 | |
*** irenab has joined #openstack-nova | 05:29 | |
*** tojuvone has quit IRC | 05:29 | |
*** tojuvone has joined #openstack-nova | 05:29 | |
*** SourabhP has joined #openstack-nova | 05:31 | |
*** _gryf has quit IRC | 05:35 | |
*** _gryf has joined #openstack-nova | 05:36 | |
*** lyanchih has joined #openstack-nova | 05:39 | |
*** Murali has joined #openstack-nova | 05:40 | |
*** arnaud___ has quit IRC | 05:42 | |
*** arnaud___ has joined #openstack-nova | 05:43 | |
*** Nikolay_St has quit IRC | 05:43 | |
*** ildikov has joined #openstack-nova | 05:44 | |
*** fawadkhaliq has joined #openstack-nova | 05:45 | |
*** deepthi has joined #openstack-nova | 05:45 | |
*** sudipto has joined #openstack-nova | 05:48 | |
*** vilobhmm has quit IRC | 05:49 | |
*** ig0r_ has joined #openstack-nova | 05:51 | |
*** _gryf has quit IRC | 05:55 | |
*** ig0r__ has quit IRC | 05:55 | |
*** _gryf has joined #openstack-nova | 06:00 | |
*** markvoelker has joined #openstack-nova | 06:00 | |
*** yingjun has joined #openstack-nova | 06:01 | |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Added method exists to the Tag object https://review.openstack.org/178103 | 06:01 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/197213 | 06:02 |
*** Kennan2 has quit IRC | 06:02 | |
*** Kennan has joined #openstack-nova | 06:03 | |
*** moshele has joined #openstack-nova | 06:03 | |
*** yingjun has quit IRC | 06:03 | |
*** markvoelker has quit IRC | 06:04 | |
*** unicell1 has joined #openstack-nova | 06:06 | |
*** davidlenwell has quit IRC | 06:06 | |
*** exploreshaifali has joined #openstack-nova | 06:06 | |
*** davidlenwell has joined #openstack-nova | 06:08 | |
*** unicell has quit IRC | 06:08 | |
*** afazekas has joined #openstack-nova | 06:08 | |
*** yingjun has joined #openstack-nova | 06:08 | |
*** aix has joined #openstack-nova | 06:09 | |
*** pradipta has joined #openstack-nova | 06:09 | |
*** ajayaa has joined #openstack-nova | 06:12 | |
*** arnaud___ has quit IRC | 06:13 | |
*** dims has joined #openstack-nova | 06:15 | |
*** sahid has joined #openstack-nova | 06:17 | |
*** kiran-r has joined #openstack-nova | 06:19 | |
*** dims has quit IRC | 06:20 | |
*** josecastroleon has quit IRC | 06:22 | |
*** jichen has joined #openstack-nova | 06:25 | |
*** annashen_ has joined #openstack-nova | 06:27 | |
*** cfriesen has quit IRC | 06:29 | |
*** radek__ has joined #openstack-nova | 06:31 | |
*** moshele has quit IRC | 06:32 | |
*** scheuran has joined #openstack-nova | 06:35 | |
*** hakimo has quit IRC | 06:40 | |
*** yingjun has quit IRC | 06:43 | |
*** annashen_ has quit IRC | 06:45 | |
*** bfic has joined #openstack-nova | 06:46 | |
*** yingjun has joined #openstack-nova | 06:46 | |
*** _kiran_ has joined #openstack-nova | 06:48 | |
*** otter768 has joined #openstack-nova | 06:49 | |
*** kiran-r has quit IRC | 06:51 | |
*** achanda has quit IRC | 06:52 | |
*** rdopiera has joined #openstack-nova | 06:53 | |
*** otter768 has quit IRC | 06:54 | |
*** pbandzi has joined #openstack-nova | 06:54 | |
*** wanghua has joined #openstack-nova | 06:56 | |
*** yamahata has joined #openstack-nova | 06:57 | |
*** nihilifer has joined #openstack-nova | 06:58 | |
*** humble_ has quit IRC | 06:58 | |
*** humble_ has joined #openstack-nova | 06:59 | |
*** armax has quit IRC | 07:00 | |
*** wanghua has quit IRC | 07:03 | |
*** davideagnello has quit IRC | 07:04 | |
*** _kiran_ has quit IRC | 07:08 | |
*** Piet has quit IRC | 07:10 | |
*** Longgeek has quit IRC | 07:10 | |
*** asettle has quit IRC | 07:10 | |
*** lpetrut has joined #openstack-nova | 07:13 | |
*** belmoreira has joined #openstack-nova | 07:15 | |
*** ssurana has quit IRC | 07:16 | |
*** ociuhandu has joined #openstack-nova | 07:16 | |
*** SourabhP has quit IRC | 07:17 | |
*** apuimedo has joined #openstack-nova | 07:18 | |
*** ssurana has joined #openstack-nova | 07:18 | |
*** tobe has joined #openstack-nova | 07:18 | |
*** lpetrut has quit IRC | 07:19 | |
*** belmoreira has quit IRC | 07:21 | |
*** lyanchih has quit IRC | 07:21 | |
*** moshele has joined #openstack-nova | 07:22 | |
*** ssurana has quit IRC | 07:22 | |
*** Sukhdev has quit IRC | 07:25 | |
*** lyanchih_ has joined #openstack-nova | 07:26 | |
*** mrda is now known as mrda-away | 07:26 | |
*** belmoreira has joined #openstack-nova | 07:28 | |
*** karimb has joined #openstack-nova | 07:29 | |
*** jlanoux has joined #openstack-nova | 07:29 | |
*** jwcroppe has quit IRC | 07:29 | |
*** ferest has joined #openstack-nova | 07:30 | |
openstackgerrit | Sergey Vilgelm proposed openstack/nova: Switch to the oslo_utils.fileutils https://review.openstack.org/199575 | 07:32 |
*** pmurray has joined #openstack-nova | 07:34 | |
*** pmurray is now known as Guest18388 | 07:34 | |
*** exploreshaifali has quit IRC | 07:34 | |
*** Nikolay_St has joined #openstack-nova | 07:35 | |
*** PaulMurray has quit IRC | 07:35 | |
openstackgerrit | heijlong proposed openstack/nova: Add uuid into SecurityGroup database model https://review.openstack.org/199928 | 07:37 |
*** kumar has joined #openstack-nova | 07:37 | |
*** lyanchih_ has left #openstack-nova | 07:37 | |
*** kun_huang has joined #openstack-nova | 07:41 | |
kun_huang | guys, is ta reasonable to add fping functionality in shell.py? | 07:42 |
*** lyanchih has joined #openstack-nova | 07:42 | |
*** Sukhdev has joined #openstack-nova | 07:46 | |
*** amotoki_ has joined #openstack-nova | 07:47 | |
*** amotoki has quit IRC | 07:49 | |
*** Longgeek has joined #openstack-nova | 07:50 | |
*** kumar has quit IRC | 07:51 | |
*** yamahata has quit IRC | 07:51 | |
*** bkopilov has quit IRC | 07:52 | |
*** LukeHinds has joined #openstack-nova | 07:59 | |
*** exploreshaifali has joined #openstack-nova | 08:00 | |
*** humble_ has quit IRC | 08:00 | |
*** humble_ has joined #openstack-nova | 08:01 | |
*** markvoelker has joined #openstack-nova | 08:01 | |
*** Sukhdev has quit IRC | 08:02 | |
openstackgerrit | Moshe Levi proposed openstack/nova: Enhance PCI passthrough whitelist to support regex https://review.openstack.org/199488 | 08:02 |
*** lucas-dinner has joined #openstack-nova | 08:02 | |
*** lucas-dinner has quit IRC | 08:03 | |
*** lucasagomes has joined #openstack-nova | 08:03 | |
*** bkopilov has joined #openstack-nova | 08:04 | |
*** jasondotstar has joined #openstack-nova | 08:04 | |
*** lpetrut has joined #openstack-nova | 08:05 | |
*** markvoelker has quit IRC | 08:06 | |
*** jistr has joined #openstack-nova | 08:09 | |
*** bkopilov has quit IRC | 08:12 | |
*** ferest has quit IRC | 08:13 | |
*** lpetrut has left #openstack-nova | 08:14 | |
*** whitebyte has joined #openstack-nova | 08:15 | |
*** dims has joined #openstack-nova | 08:16 | |
*** shyama_ has joined #openstack-nova | 08:19 | |
*** shyama has quit IRC | 08:20 | |
*** shyama_ is now known as shyama | 08:20 | |
openstackgerrit | Abhishek Kekane proposed openstack/nova: POC: Improve performance of Unshelve api https://review.openstack.org/184871 | 08:20 |
*** dims has quit IRC | 08:21 | |
*** ndipanov has joined #openstack-nova | 08:21 | |
openstackgerrit | sahid proposed openstack/nova: libvirt: add os_admin_user to use with set admin password https://review.openstack.org/185921 | 08:21 |
openstackgerrit | sahid proposed openstack/nova: libvirt: enhance libvirt to set admin password https://review.openstack.org/185910 | 08:21 |
openstackgerrit | sahid proposed openstack/nova: libvirt: rework quiesce to not share "sensitive" informations https://review.openstack.org/196620 | 08:21 |
*** abhishekk has joined #openstack-nova | 08:21 | |
*** bkopilov has joined #openstack-nova | 08:24 | |
openstackgerrit | jichenjc proposed openstack/nova: Check flavor type before add tenant access https://review.openstack.org/124338 | 08:24 |
*** Longgeek has quit IRC | 08:25 | |
*** browne has quit IRC | 08:25 | |
openstackgerrit | sahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop https://review.openstack.org/174854 | 08:26 |
openstackgerrit | sahid proposed openstack/nova: libvirt: deprecate use_usb_tablet in favor of pointer_model https://review.openstack.org/176242 | 08:26 |
sahid | ndipanov: hello, any chance to get review on this serie https://review.openstack.org/#/c/196991/ - most of the patches has already be +2ed by Jay and since the serie touch several part of thedriver it's difficult to maintain it | 08:27 |
*** whitebyte has quit IRC | 08:28 | |
*** jasondotstar has quit IRC | 08:29 | |
ndipanov | yes I will do it sahid | 08:29 |
ndipanov | this is just moving direct calls to domain.smth() into a class right? | 08:30 |
*** whitebyte has joined #openstack-nova | 08:30 | |
sahid | ndipanov: yes, just encapsulation | 08:30 |
sahid | then we introduce new class here https://review.openstack.org/#/c/190210 for block device | 08:31 |
sahid | and again... encapsulation | 08:31 |
*** yassine_ has joined #openstack-nova | 08:31 | |
*** Longgeek has joined #openstack-nova | 08:34 | |
*** wwriverrat has quit IRC | 08:36 | |
*** whitebyte has quit IRC | 08:37 | |
*** wwriverrat has joined #openstack-nova | 08:37 | |
ndipanov | ok sahid let me fix one of my patches and I will jump on this | 08:39 |
johnthetubaguy | andrewbogott_afk: mriedem is correct, best to add it to the meeting agenda | 08:40 |
sahid | great thank you ndipanov | 08:41 |
*** wwriverrat has quit IRC | 08:44 | |
*** bkopilov has quit IRC | 08:44 | |
*** wanghua has joined #openstack-nova | 08:46 | |
*** ssurana has joined #openstack-nova | 08:48 | |
bauzas | morning Nova | 08:48 |
*** kiran-r has joined #openstack-nova | 08:49 | |
bauzas | johnthetubaguy: FYI, I had a discussion with alaski and dansmith last night | 08:49 |
bauzas | johnthetubaguy: about the cells v1 features | 08:49 |
johnthetubaguy | bauzas: interesting, whats the news? | 08:49 |
bauzas | johnthetubaguy: tbc, we're saying that cells v1 is stable | 08:49 |
*** kun_huang has left #openstack-nova | 08:50 | |
*** otter768 has joined #openstack-nova | 08:50 | |
bauzas | johnthetubaguy: meaning that new features should be discussed and -1 unless it's a very nit | 08:50 |
*** humble_ has quit IRC | 08:50 | |
bauzas | johnthetubaguy: like alaski's change for adding a new filter using the existing properties | 08:50 |
johnthetubaguy | bauzas: so block all cells v1 features, basically? | 08:50 |
bauzas | johnthetubaguy: yeah, but we can leave some room for quickwins | 08:51 |
bauzas | johnthetubaguy: anyway, we're post-freeze, so that doesn't really mean we will have new features | 08:52 |
*** ankit has joined #openstack-nova | 08:52 | |
bauzas | johnthetubaguy: but that's the consensus | 08:52 |
*** ssurana has quit IRC | 08:52 | |
johnthetubaguy | bauzas: so I thought that was the consensus before, so I am glad that is the consensus now | 08:52 |
bauzas | johnthetubaguy: https://review.openstack.org/#/c/192875/ is an exception | 08:52 |
johnthetubaguy | bauzas: we should write that down I guess | 08:52 |
bauzas | johnthetubaguy: totally agreed, that was my point on yesterdayt | 08:53 |
bauzas | johnthetubaguy: it left me 2 thoughts | 08:53 |
bauzas | johnthetubaguy: 1/ we need to write down the stable policy for cells v1 | 08:53 |
*** jasondotstar has joined #openstack-nova | 08:53 | |
bauzas | johnthetubaguy: 2/ I'm not very comfortable having the filters not functionally tested | 08:53 |
bauzas | johnthetubaguy: in particular since I'm working on the Spec objectification which modifies all the filters | 08:54 |
bauzas | johnthetubaguy: (see my comments on the above change) | 08:54 |
*** otter768 has quit IRC | 08:54 | |
openstackgerrit | Alex Xu proposed openstack/nova: Remove useless db call instance_get_all_hung_in_rebooting https://review.openstack.org/199970 | 08:54 |
openstackgerrit | Alex Xu proposed openstack/nova: Remove useless db call instance_get_all_by_host_and_not_type https://review.openstack.org/199971 | 08:54 |
openstackgerrit | Alex Xu proposed openstack/nova: Remove db layer hard-code permission checks for reservation_expire https://review.openstack.org/199972 | 08:54 |
johnthetubaguy | bauzas: yeah, we need the cells thing written down | 08:54 |
bauzas | johnthetubaguy: so, about 1/, I'll raise the point to the next cells meeting on Wed | 08:55 |
johnthetubaguy | bauzas: unit tests should be fine for the filters really, most are not functionally tested | 08:55 |
bauzas | johnthetubaguy: well, by looking at that, I'm not very comfortable with not having in-tree functional tests for filters | 08:55 |
bauzas | johnthetubaguy: not speaking about Tempest tests | 08:55 |
bauzas | johnthetubaguy: just functional tests in the nova treee | 08:55 |
bauzas | tree even | 08:56 |
bauzas | johnthetubaguy: at least for the defaulted filters | 08:56 |
bauzas | johnthetubaguy: and possibly to all of them | 08:56 |
bauzas | johnthetubaguy: so the thing is, I need to think about it a little more | 08:57 |
bauzas | johnthetubaguy: for 1/, I'll draft something soon | 08:57 |
*** haomaiwang has joined #openstack-nova | 08:57 | |
bauzas | (just adding that in my big TODO list :) | 08:57 |
*** haomaiw__ has quit IRC | 09:00 | |
*** bkopilov has joined #openstack-nova | 09:00 | |
johnthetubaguy | bauzas: yeah, its worth a try | 09:01 |
johnthetubaguy | bauzas: let me link to a horrible unit test I added, that might help... | 09:01 |
bauzas | johnthetubaguy: sure, any thought is nice | 09:01 |
bauzas | johnthetubaguy: maybe I'm overthinking about functionally testing the filters, but maybe not :) | 09:02 |
johnthetubaguy | bauzas: https://github.com/openstack/nova/blob/master/nova/tests/unit/scheduler/test_caching_scheduler.py#L157 | 09:02 |
*** amotoki_ has quit IRC | 09:02 | |
bauzas | johnthetubaguy: oh that one, yeak I know it :) | 09:02 |
bauzas | johnthetubaguy: since I modified it for the Spec BP | 09:02 |
johnthetubaguy | bauzas: so that is actually a full test of the filters and wights | 09:02 |
bauzas | johnthetubaguy: yeah, with a default coverage | 09:03 |
johnthetubaguy | bauzas: sends in data they manipulate, and can even benchmark and profile the performance | 09:03 |
bauzas | johnthetubaguy: I mean, default coverage is already done by Tempest, right ? | 09:03 |
bauzas | johnthetubaguy: my wonders are more about the corner cases | 09:03 |
johnthetubaguy | bauzas: I guess I mean, thats the framework you need to test those corner cases | 09:04 |
bauzas | johnthetubaguy: I see | 09:04 |
*** davideagnello has joined #openstack-nova | 09:04 | |
bauzas | johnthetubaguy: mmm, it deserves a bit more thinking and probably a discussion at the subteam meeting :) | 09:04 |
johnthetubaguy | I suspect there is a nicer way, but it does the trick | 09:04 |
bauzas | johnthetubaguy: agreed | 09:04 |
johnthetubaguy | basically it just mocks out all the db calls | 09:05 |
bauzas | I see | 09:06 |
bauzas | okay, I'll leave that out for now, just modify the cells devref and see what the other guys think | 09:06 |
bauzas | that reminds me... | 09:06 |
johnthetubaguy | more testing is good, but I think that should cover it OK | 09:06 |
johnthetubaguy | so I have a similar unit test (in an internal branch) that does the same for the cells scheduler | 09:07 |
bauzas | n0ano: edleafe: lxsli: jaypipes: I won't be able to attend next scheduler meeting, as I'm French, I'm usually taking some days off from tonight to Wed morning | 09:07 |
bauzas | johnthetubaguy: correct | 09:07 |
johnthetubaguy | bauzas: is that Bastille day or something? | 09:08 |
bauzas | johnthetubaguy: I'm not very worried by any complexity for functionally testing, I'm more concerned about the necessity :) | 09:08 |
bauzas | johnthetubaguy: yup, July 14th and our French office is asking us to take July 13th as well | 09:09 |
johnthetubaguy | ah, gotcha | 09:09 |
*** davideagnello has quit IRC | 09:09 | |
bauzas | time for seeing the sea :) | 09:09 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: Allow for migration object to be passed to _move_claim https://review.openstack.org/199679 | 09:10 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: rt: generalize claim code to be useful for other move actions https://review.openstack.org/197138 | 09:10 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: rt: move filtering of mygration by type lower in the call stack https://review.openstack.org/199678 | 09:10 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: Define migration states and transitions and use when changing status https://review.openstack.org/197669 | 09:10 |
openstackgerrit | Nikola Dipanov proposed openstack/nova: Introduce a simple state machine module https://review.openstack.org/197668 | 09:10 |
ndipanov | sahid, if you want to review something in return ^ | 09:10 |
johnthetubaguy | bauzas: good plan! | 09:10 |
bauzas | johnthetubaguy: :) | 09:10 |
sahid | ndipanov: sure | 09:11 |
bauzas | ndipanov: on my stars list | 09:12 |
ndipanov | bauzas, many thanks ;) ;) | 09:12 |
*** gszasz has joined #openstack-nova | 09:13 | |
*** nihilifer has quit IRC | 09:13 | |
*** deepakcs has joined #openstack-nova | 09:13 | |
*** wwriverrat has joined #openstack-nova | 09:14 | |
*** park_heijlong has quit IRC | 09:14 | |
*** coolsvap|away is now known as coolsvap | 09:15 | |
deepakcs | hello all | 09:16 |
deepakcs | wrt: https://review.openstack.org/#/c/192736/8/nova/virt/libvirt/driver.py ... | 09:18 |
*** lpetrut has joined #openstack-nova | 09:19 | |
deepakcs | does anyone know whats the right way to form the fully qualified network disk path (gluster://server:port/volume/image) in libvirt/driver.py which will work with qemu-img rebase cmd ? | 09:19 |
*** exploreshaifali has quit IRC | 09:21 | |
*** fawadkhaliq has quit IRC | 09:22 | |
*** lpetrut1 has joined #openstack-nova | 09:27 | |
*** marzif_ has joined #openstack-nova | 09:29 | |
*** lpetrut has quit IRC | 09:29 | |
*** yingjun has quit IRC | 09:32 | |
*** tdurakov has quit IRC | 09:33 | |
*** derekh has joined #openstack-nova | 09:36 | |
*** shuquan has joined #openstack-nova | 09:37 | |
*** rotbeard has joined #openstack-nova | 09:47 | |
*** marzif_ has quit IRC | 09:47 | |
*** coolsvap is now known as coolsvap|away | 09:48 | |
*** yassine_ has quit IRC | 09:52 | |
openstackgerrit | John Garbutt proposed openstack/nova: devref: discuss current state of error handling https://review.openstack.org/178635 | 09:54 |
*** nihilifer has joined #openstack-nova | 09:55 | |
kashyap | deepakcs: Why not go the route of what Eric suggests for now? Fix it for files, and post a subsequent patch for network? | 09:57 |
openstackgerrit | John Garbutt proposed openstack/nova: docs: clear between current vs future plans https://review.openstack.org/192259 | 10:00 |
*** shuquan has quit IRC | 10:01 | |
*** markvoelker has joined #openstack-nova | 10:02 | |
openstackgerrit | sahid proposed openstack/nova: libvirt: add cpu schedular priority config https://review.openstack.org/197567 | 10:03 |
openstackgerrit | sahid proposed openstack/nova: libvirt: rework membacking config to support future features https://review.openstack.org/197566 | 10:03 |
openstackgerrit | sahid proposed openstack/nova: virt: add contraint to handle realtime policy https://review.openstack.org/197568 | 10:03 |
openstackgerrit | sahid proposed openstack/nova: libvirt: add realtime support https://review.openstack.org/197569 | 10:03 |
openstackgerrit | John Garbutt proposed openstack/nova: docs: clear between current vs future plans https://review.openstack.org/192259 | 10:04 |
*** radek__ has quit IRC | 10:05 | |
*** markvoelker has quit IRC | 10:06 | |
openstackgerrit | John Garbutt proposed openstack/nova: docs: add the scheduler evolution plans https://review.openstack.org/192260 | 10:06 |
johnthetubaguy | bauzas: I just got this one rebased, if you want to take a look: https://review.openstack.org/#/c/192260/1/doc/source/scheduler_evolution.rst,cm | 10:07 |
deepakcs | kashyap: yes we can, check out the followup comment I made. But i wanted to see if we can fix it for both, which would have been ideal | 10:07 |
*** radek__ has joined #openstack-nova | 10:07 | |
*** coolsvap|away is now known as coolsvap | 10:11 | |
openstackgerrit | Abhishek Kekane proposed openstack/nova-specs: Improve performance of UnShelve API https://review.openstack.org/135387 | 10:12 |
*** dims has joined #openstack-nova | 10:17 | |
*** dims has quit IRC | 10:21 | |
*** mutoulbj has quit IRC | 10:22 | |
*** fawadkhaliq has joined #openstack-nova | 10:22 | |
*** fawadkhaliq has quit IRC | 10:23 | |
*** fawadkhaliq has joined #openstack-nova | 10:23 | |
*** coolsvap is now known as coolsvap|away | 10:25 | |
*** pradipta has quit IRC | 10:27 | |
openstackgerrit | John Garbutt proposed openstack/nova: Add a summary of the architecture evolution work https://review.openstack.org/200001 | 10:30 |
*** humble_ has joined #openstack-nova | 10:31 | |
*** LukeHinds has quit IRC | 10:32 | |
*** wanghua has quit IRC | 10:34 | |
*** andreykurilin__ has joined #openstack-nova | 10:44 | |
*** IanGovett has joined #openstack-nova | 10:45 | |
*** wanghua has joined #openstack-nova | 10:47 | |
*** bkopilov has quit IRC | 10:49 | |
*** humble_ has quit IRC | 10:49 | |
*** otter768 has joined #openstack-nova | 10:51 | |
*** jlanoux has quit IRC | 10:53 | |
*** otter768 has quit IRC | 10:55 | |
*** Longgeek has quit IRC | 10:59 | |
*** bkopilov has joined #openstack-nova | 11:00 | |
*** pczesno has joined #openstack-nova | 11:00 | |
*** deepthi has quit IRC | 11:00 | |
*** pixelbeat has joined #openstack-nova | 11:01 | |
*** yassine_ has joined #openstack-nova | 11:03 | |
*** davideagnello has joined #openstack-nova | 11:05 | |
*** piyanai has joined #openstack-nova | 11:06 | |
*** fawadkhaliq has quit IRC | 11:06 | |
*** mutoulbj has joined #openstack-nova | 11:07 | |
*** fawadkhaliq has joined #openstack-nova | 11:08 | |
*** davideagnello has quit IRC | 11:10 | |
*** dprince has joined #openstack-nova | 11:13 | |
*** Longgeek has joined #openstack-nova | 11:14 | |
*** mutoulbj has quit IRC | 11:17 | |
*** wanghua has quit IRC | 11:17 | |
bauzas | johnthetubaguy: ack, will try to do this today | 11:18 |
*** mutoulbj has joined #openstack-nova | 11:18 | |
*** wanghua has joined #openstack-nova | 11:19 | |
*** thorst has joined #openstack-nova | 11:20 | |
*** coolsvap|away is now known as coolsvap | 11:22 | |
*** markvoelker has joined #openstack-nova | 11:33 | |
*** dave-mccowan has joined #openstack-nova | 11:34 | |
openstackgerrit | Alex Xu proposed openstack/nova: Remove useless db call instance_get_all_by_host_and_not_type https://review.openstack.org/199971 | 11:34 |
openstackgerrit | Alex Xu proposed openstack/nova: Remove useless db call instance_get_all_hung_in_rebooting https://review.openstack.org/199970 | 11:34 |
openstackgerrit | Alex Xu proposed openstack/nova: Remove db layer hard-code permission checks for reservation_expire https://review.openstack.org/199972 | 11:34 |
*** jlanoux has joined #openstack-nova | 11:34 | |
openstackgerrit | Ivan Mironov proposed openstack/nova: libvirt: Minimal qemu version for discard support should depend on image backend. https://review.openstack.org/195672 | 11:37 |
*** markvoelker has quit IRC | 11:37 | |
haypo | hi. is there someone available to review my patch "Port crypto to Python 3"? it already got a +2 from sdague, and +1 from bauzas (hi!). https://review.openstack.org/#/c/197748/ | 11:39 |
haypo | i'm asking because "Fix Python 3 issues in nova.utils and nova.tests" has been approved https://review.openstack.org/#/c/197982/ but it depends on the crypto patch, and i would like to submit new patches based on this patch serie | 11:40 |
openstackgerrit | Tomi Juvonen proposed openstack/nova-specs: Get valid server state https://review.openstack.org/192246 | 11:40 |
openstackgerrit | John Garbutt proposed openstack/nova: Add a summary of the architecture evolution work https://review.openstack.org/200001 | 11:41 |
ndipanov | haypo, queued up | 11:42 |
openstackgerrit | John Garbutt proposed openstack/nova: Add a summary of the architecture evolution work https://review.openstack.org/200001 | 11:42 |
johnthetubaguy | awe, I was so close to the magic change number :( | 11:43 |
johnthetubaguy | at least I get the one about some space journey a long time in the future | 11:43 |
* johnthetubaguy goes for lunch | 11:43 | |
*** mutoulbj has quit IRC | 11:44 | |
pkholkin | dhellmann: Hello Doug, could you please remove -2 from this patch https://review.openstack.org/#/c/173834/ ? I think my backport needs it https://review.openstack.org/#/c/199569/ | 11:44 |
*** exploreshaifali has joined #openstack-nova | 11:44 | |
*** jaypipes has joined #openstack-nova | 11:45 | |
*** igordcard_ has quit IRC | 11:46 | |
*** tobe has quit IRC | 11:48 | |
*** zhenguo has quit IRC | 11:50 | |
*** lucasagomes is now known as lucas-hungry | 11:51 | |
sahid | ndipanov: yep good catch ! https://review.openstack.org/#/c/190214/9/nova/virt/libvirt/guest.py,cm | 11:53 |
ndipanov | sahid, now just get another person to +A this and a lot of those can go in | 11:55 |
ndipanov | https://review.openstack.org/#/c/196991/ | 11:55 |
sahid | ndipanov: yes cool | 11:56 |
sahid | actually this one already gave two +2 but wanted to address the doctstring related comment | 11:57 |
sahid | it should be easy to find a guy :) | 11:57 |
ndipanov | sahid, meh | 11:57 |
ndipanov | imho don't bother | 11:57 |
*** tojuvone has quit IRC | 11:57 | |
sdague | sahid: +A on that base patch, that is pretty straight forward | 11:58 |
sahid | sdague: cool thank you :) | 11:58 |
ndipanov | sdague, do you remember when we stopped asking for downgrade migrations? that's part of the whole online migration story rite? | 12:02 |
*** sudipto has quit IRC | 12:02 | |
sdague | no, it's part of a cross project spec | 12:03 |
sdague | ndipanov: http://specs.openstack.org/openstack/openstack-specs/specs/no-downward-sql-migration.html | 12:03 |
ndipanov | sdague, thanks | 12:03 |
sdague | it merged... a few months ago | 12:03 |
ndipanov | I see e2e846f | 12:05 |
ndipanov | and it was even you who did it hehe | 12:05 |
ndipanov | I just picked you to ask at random as in "he might know " :) | 12:05 |
openstackgerrit | lyanchih proposed openstack/nova: Detaching and then attaching a volume to a paused instance will failed https://review.openstack.org/200032 | 12:06 |
*** markvoelker has joined #openstack-nova | 12:12 | |
*** annegentle has joined #openstack-nova | 12:15 | |
garyk | any chance to beg for a few reviews … | 12:16 |
garyk | beg beg beg | 12:16 |
luv | umm this is strange, if i set availability zone when starting a new instance - it is respected by nova-scheduler and i can see in debug log it filtered down nodes the instances can run on correctly - but the availability zone reported for the instance is incorrected | 12:17 |
luv | as "nova" | 12:17 |
*** gabor-t has joined #openstack-nova | 12:17 | |
*** dims has joined #openstack-nova | 12:17 | |
garyk | luv: try and restert your nova-compute - i have seen this problem too and restarting the service helps. seems like a bug and the data does not get to the scheduler | 12:19 |
sdague | :) | 12:19 |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 12:21 |
*** bjornar has quit IRC | 12:21 | |
nihilifer | bauzas: are you here? | 12:22 |
luv | umm, that didnt help :( | 12:22 |
*** dims has quit IRC | 12:22 | |
*** fawadkhaliq has quit IRC | 12:22 | |
luv | 2015-07-09 14:20:31.261 13004 WARNING nova.compute.manager [req-3cbbb5f6-72f2-4865-850e-a563abb35f82 - - - - -] While synchronizing instance power states, found 4 instances in the database and 3 instances on the hypervisor. | 12:23 |
luv | but i guess that might not be relevant | 12:23 |
luv | umm it looks properly f*cked up actually - in the ui (both nova client and horizon) I can see "Availability Zone" as "nova" but select availability_zone from instances where uuid="c90f6654-50dd-4b2d-9a52-3a5744fffda5"; returns "lab12" | 12:27 |
*** exploreshaifali has quit IRC | 12:27 | |
luv | http://paste.openstack.org/show/358430/ | 12:28 |
*** yingjun has joined #openstack-nova | 12:28 | |
*** andreykurilin has quit IRC | 12:30 | |
openstackgerrit | Bence Romsics proposed openstack/nova: Do not block instance boot on net without subnet https://review.openstack.org/200046 | 12:30 |
luv | ok, i wil restart nova-api :) | 12:30 |
luv | fixed, yaay; garyk thanks | 12:31 |
*** aarefiev has quit IRC | 12:31 | |
*** lyanchih has quit IRC | 12:34 | |
garyk | luv: np. if you have the time maybe it is worthwhile to open a bug so it can be addressed … | 12:35 |
*** bjornar has joined #openstack-nova | 12:35 | |
*** gabor-t has quit IRC | 12:36 | |
*** edmondsw has joined #openstack-nova | 12:37 | |
*** annegentle has quit IRC | 12:38 | |
*** jasondotstar has quit IRC | 12:38 | |
*** ajayaa has quit IRC | 12:38 | |
*** dims has joined #openstack-nova | 12:40 | |
garyk | mdbooth: is your sabbatical over? btw we have tried to address the issue you mentioned in https://bugs.launchpad.net/nova/+bug/1462957 on a number of occasions and each time shot down. I think that even you were against solutions proposed in the past... | 12:41 |
openstack | Launchpad bug 1462957 in OpenStack Compute (nova) "VMware driver cannot report non-contiguous resources to the scheduler" [Undecided,New] | 12:41 |
*** piyanai has quit IRC | 12:41 | |
garyk | mdbooth: i am not even sure how ceph reports stuff like this - but that is another issue | 12:41 |
*** mutoulbj has joined #openstack-nova | 12:41 | |
openstackgerrit | John Garbutt proposed openstack/nova: docs: clear between current vs future plans https://review.openstack.org/192259 | 12:43 |
openstackgerrit | John Garbutt proposed openstack/nova: docs: add a placeholder link to mentoring docs https://review.openstack.org/200053 | 12:43 |
*** yingjun has quit IRC | 12:45 | |
*** fawadkhaliq has joined #openstack-nova | 12:45 | |
*** mutoulbj has quit IRC | 12:46 | |
*** mutoulbj has joined #openstack-nova | 12:46 | |
*** lucas-hungry is now known as lucasagomes | 12:46 | |
*** andreykurilin__ has quit IRC | 12:47 | |
*** Longgeek has quit IRC | 12:49 | |
*** vladikr has joined #openstack-nova | 12:49 | |
*** pserebryakov has joined #openstack-nova | 12:49 | |
*** VW_ has joined #openstack-nova | 12:50 | |
*** otter768 has joined #openstack-nova | 12:51 | |
openstackgerrit | John Garbutt proposed openstack/nova: Add a summary of the architecture evolution work https://review.openstack.org/200001 | 12:51 |
openstackgerrit | John Garbutt proposed openstack/nova: docs: clear between current vs future plans https://review.openstack.org/192259 | 12:55 |
*** wanghua has quit IRC | 12:56 | |
*** amotoki has joined #openstack-nova | 12:56 | |
*** eglynn has joined #openstack-nova | 12:56 | |
*** wanghua has joined #openstack-nova | 12:56 | |
*** otter768 has quit IRC | 12:56 | |
*** jaypipes has quit IRC | 12:56 | |
*** andreykurilin has joined #openstack-nova | 12:57 | |
*** jasondotstar has joined #openstack-nova | 12:58 | |
*** aarefiev has joined #openstack-nova | 12:59 | |
*** Marga__ has quit IRC | 13:00 | |
*** yingjun has joined #openstack-nova | 13:00 | |
*** ivasev has joined #openstack-nova | 13:01 | |
*** mdrabe has joined #openstack-nova | 13:01 | |
mdbooth | garyk: My angle is that it's a deficiency in the scheduler | 13:01 |
*** ZZelle has quit IRC | 13:01 | |
*** fawadkhaliq has quit IRC | 13:01 | |
*** VW_ has quit IRC | 13:02 | |
*** ZZelle has joined #openstack-nova | 13:02 | |
*** VW_ has joined #openstack-nova | 13:02 | |
*** diogogmt has joined #openstack-nova | 13:02 | |
edleafe | bauzas: reading back, ack that you won't be able to make next scheduler meeting | 13:03 |
edleafe | bauzas: np, we'll assign everything to you :) | 13:03 |
*** rlrossit has joined #openstack-nova | 13:05 | |
*** Marga_ has joined #openstack-nova | 13:05 | |
*** ivasev has quit IRC | 13:05 | |
*** davideagnello has joined #openstack-nova | 13:06 | |
*** VW_ has quit IRC | 13:07 | |
*** Marga_ has quit IRC | 13:07 | |
*** Marga_ has joined #openstack-nova | 13:07 | |
*** bfic has quit IRC | 13:08 | |
*** zhenguo has joined #openstack-nova | 13:09 | |
garyk | mdbooth: i concur | 13:09 |
*** amotoki has quit IRC | 13:09 | |
garyk | mdbooth: when dan was writing the data that is passed to the scheduler i wanted him to add the max contiguos disk. but that either went ignored or fell between the cracks | 13:10 |
*** jaypipes has joined #openstack-nova | 13:10 | |
garyk | the same issues exist with any shared storage system and we'll slowly but sure be bitten by this. | 13:10 |
*** davideagnello has quit IRC | 13:10 | |
mdbooth | I don't have a fully joined-up understanding of exactly how that data is used | 13:10 |
mdbooth | But it's certainly not confined to storage | 13:10 |
mdbooth | The same goes for both memory and cpu | 13:11 |
garyk | johnthetubaguy: please see the comments i have added to the various ref you have posted - typos and other minor stuff | 13:11 |
johnthetubaguy | garyk: yes, thank you for those, I have addressed some of those already | 13:11 |
mdbooth | I've never got around to looking in detail, but I can't help but feel that any solution which exposes numa topology to the scheduler would be solving a very similar problem to non-contigious hypervisor resources | 13:12 |
openstackgerrit | John Garbutt proposed openstack/nova: docs: add a placeholder link to mentoring docs https://review.openstack.org/200053 | 13:12 |
*** mriedem has joined #openstack-nova | 13:12 | |
mdbooth | I'd be interested to know what the scheduler does with numa, and to know if we could do something similar for the whole hypervisor | 13:12 |
dims | jaypipes: you had looked at this one earlier and i reworked it with your suggestions, can you please bless it when you get a chance? (https://review.openstack.org/#/c/187785/) - py3 tests | 13:14 |
*** ivasev has joined #openstack-nova | 13:15 | |
mdbooth | jaypipes: This review called to tell me it was feeling sad and unloved: https://review.openstack.org/#/c/141115/ | 13:16 |
*** artom has joined #openstack-nova | 13:16 | |
*** amotoki has joined #openstack-nova | 13:16 | |
*** ijuwang has joined #openstack-nova | 13:17 | |
*** Longgeek has joined #openstack-nova | 13:17 | |
jaypipes | dims: done. | 13:17 |
garyk | dims: around? https://review.openstack.org/198689 | 13:21 |
*** piyanai has joined #openstack-nova | 13:22 | |
*** amotoki has quit IRC | 13:28 | |
*** amotoki_ has joined #openstack-nova | 13:28 | |
*** VW_ has joined #openstack-nova | 13:28 | |
*** edmondsw has quit IRC | 13:28 | |
*** eharney has joined #openstack-nova | 13:28 | |
jaypipes | mdbooth: done. | 13:29 |
*** deepakcs has quit IRC | 13:29 | |
mdbooth | jaypipes: Thanks! | 13:29 |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 1 https://review.openstack.org/152569 | 13:30 |
*** diogogmt has quit IRC | 13:31 | |
*** piyanai has quit IRC | 13:33 | |
*** ajayaa has joined #openstack-nova | 13:33 | |
mriedem | jaypipes: i added you to this spec since (1) it will never go away and (2) it's very glance-y https://review.openstack.org/#/c/135387/ | 13:33 |
jaypipes | mriedem: ewww. ok :) | 13:35 |
*** dboik_ has quit IRC | 13:36 | |
mdbooth | jaypipes: I'm nervous about fixing those typos because I'll lose your +2 :) If I fix them could you re-bless? | 13:37 |
jaypipes | mdbooth: of course. | 13:37 |
mdbooth | Thanks, coming up | 13:37 |
*** sudipto has joined #openstack-nova | 13:38 | |
mriedem | so, the last time this got a review was 1/26: https://review.openstack.org/#/c/69047/ - it's been posted since 4/28/14 | 13:38 |
ndipanov | jaypipes, bauzas https://review.openstack.org/#/c/197138/ <- see my comment pls | 13:38 |
mriedem | i'm giving my first plug for reviews this release to prep for the rare non-priority feature review mating ritual tomorrow | 13:39 |
*** lucas__ has joined #openstack-nova | 13:39 | |
ndipanov | mriedem, gross | 13:39 |
mriedem | ndipanov: it's not gross, it's beautiful and deadly | 13:39 |
mriedem | i expect someone to be eaten when it's over | 13:39 |
openstackgerrit | Matthew Booth proposed openstack/nova: Implement compare-and-swap for instance update https://review.openstack.org/141115 | 13:40 |
ndipanov | mriedem, watched too many nature shows lately? :) | 13:41 |
mriedem | ndipanov: ok, how about, rare orbital docking sequence? | 13:42 |
*** andrewbogott_afk is now known as andrewbogott | 13:42 | |
*** _gryf is now known as _gryf|afk | 13:42 | |
*** irenab has quit IRC | 13:42 | |
*** jdandrea has joined #openstack-nova | 13:43 | |
johnthetubaguy | mriedem: I noticed you have the oldest patch on that review day list I created | 13:44 |
mriedem | johnthetubaguy: yeah, hence the plug ^ | 13:44 |
johnthetubaguy | For the curious, I created a list of blueprint patches that have been waiting more than 50 days since the patch was uploaded in the usual etherpad: https://etherpad.openstack.org/p/liberty-nova-priorities-tracking | 13:46 |
johnthetubaguy | some of the dates a tad misleading, but its a start | 13:46 |
*** tpeoples has joined #openstack-nova | 13:47 | |
*** maishsk has joined #openstack-nova | 13:48 | |
mriedem | mnestratov: is someone working on replying to the comments here? https://review.openstack.org/#/c/182257/ | 13:48 |
maishsk | Good afternoon #nova | 13:48 |
*** gabor-t has joined #openstack-nova | 13:48 | |
maishsk | Are my eyes deceiving me - or is there no documentation on how to set up nova-api services under HAproxy in a redundant way? | 13:48 |
mriedem | maishsk: sec | 13:49 |
mriedem | maishsk: did you go here http://docs.openstack.org/ and see the link for 'High Availability Guide'? | 13:49 |
maishsk | http://docs.openstack.org/high-availability-guide/content/ha-aa-controllers.html | 13:49 |
mriedem | http://docs.openstack.org/high-availability-guide/content/_api_services.html | 13:50 |
mriedem | the haproxy specific stuff is here http://docs.openstack.org/high-availability-guide/content/ha-aa-haproxy.html | 13:50 |
maishsk | mriedem: I just do not see anything there for Nova. | 13:50 |
johnthetubaguy | nova_compute_api_cluster? | 13:51 |
maishsk | That is API generic - nothing for Nova | 13:51 |
bauzas | ndipanov: ack | 13:51 |
maishsk | Or is it this - http://docs.openstack.org/high-availability-guide/content/ha-aa-haproxy.html | 13:51 |
maishsk | Yeah.. Thats it | 13:52 |
mriedem | yes | 13:52 |
maishsk | I knew it was there somewhere :) | 13:53 |
maishsk | Thanks | 13:53 |
*** pbandzi has left #openstack-nova | 13:53 | |
*** xyang1 has joined #openstack-nova | 13:55 | |
*** dboik has joined #openstack-nova | 13:56 | |
mriedem | rlrossit: did you see that your patch blew up in grenade partial n-cpu while you were out celebrating? | 13:57 |
mriedem | i see you did | 13:58 |
rlrossit | mriedem: yeah, and I put something on the review saying I don't think that's the cause | 13:58 |
rlrossit | but I most definitely could be wrong | 13:58 |
mriedem | hrm | 13:58 |
*** kiran-r has quit IRC | 13:58 | |
*** amotoki has joined #openstack-nova | 13:58 | |
*** lyanchih has joined #openstack-nova | 13:59 | |
rlrossit | me blowing that up was jenkins's way of saying "Happy Birthday Ryan" | 13:59 |
mriedem | would be nice if we had some logging in NovaObjectSerializer._process_object to see what it was processing when it's doing the infinite recursion | 13:59 |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 2 https://review.openstack.org/167408 | 13:59 |
*** karimb has quit IRC | 13:59 | |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Adds support for x509 certificates as keypairs https://review.openstack.org/136458 | 13:59 |
johnthetubaguy | abhishekk mikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes gilliard garyk edleafe dims moshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov sc68cal akuriata artom jlvillal mnestratov kashyap aloga rgeragnov bauzas its time for the nova meeting | 13:59 |
*** mtanino has joined #openstack-nova | 13:59 | |
*** ctrath has joined #openstack-nova | 13:59 | |
rlrossit | mriedem: like see what child it's trying to serialize/deserialize? | 14:00 |
*** pserebryakov has quit IRC | 14:00 | |
mriedem | rlrossit: sure, just something to point at what objects isn't looping on | 14:00 |
*** shyu has quit IRC | 14:00 | |
*** amotoki_ has quit IRC | 14:01 | |
rlrossit | didn't dansmith say something about how conductor will keep trying to redeserialize an object when it can't figure out subobject versioning? (or it'll freak out or something) | 14:01 |
*** maishsk has left #openstack-nova | 14:02 | |
*** amotoki has quit IRC | 14:03 | |
*** claudiub has joined #openstack-nova | 14:03 | |
*** Kennan2 has joined #openstack-nova | 14:03 | |
dansmith | +1 for some tracing | 14:03 |
*** Kennan has quit IRC | 14:04 | |
dansmith | I was going to add some of this to conductor if we ever get a TRACE log level back | 14:04 |
dansmith | because it would be very verbose | 14:04 |
*** READ10 has joined #openstack-nova | 14:04 | |
*** mtanino has quit IRC | 14:08 | |
*** yamahata has joined #openstack-nova | 14:09 | |
*** edmondsw has joined #openstack-nova | 14:09 | |
*** yingjun has quit IRC | 14:10 | |
*** lyanchih has quit IRC | 14:10 | |
*** lyanchih has joined #openstack-nova | 14:11 | |
*** eglynn has quit IRC | 14:11 | |
*** browne has joined #openstack-nova | 14:11 | |
*** annegentle has joined #openstack-nova | 14:12 | |
*** mtanino has joined #openstack-nova | 14:13 | |
*** jwcroppe has joined #openstack-nova | 14:15 | |
*** SourabhP has joined #openstack-nova | 14:18 | |
*** whenry_ has joined #openstack-nova | 14:18 | |
*** piyanai has joined #openstack-nova | 14:20 | |
SourabhP | johnthetubaguy: ping | 14:21 |
edleafe | SourabhP: he's running the Nova meeting right now | 14:23 |
openstackgerrit | garyk proposed openstack/nova: libvirt: improve logging in the driver.py code https://review.openstack.org/200123 | 14:23 |
SourabhP | edleafe: thanks | 14:23 |
*** piyanai has quit IRC | 14:23 | |
*** ildikov has quit IRC | 14:26 | |
*** cfriesen has joined #openstack-nova | 14:27 | |
*** lucas__ has quit IRC | 14:27 | |
*** lucas__ has joined #openstack-nova | 14:28 | |
*** mdrabe has quit IRC | 14:28 | |
*** amotoki has joined #openstack-nova | 14:29 | |
*** piyanai has joined #openstack-nova | 14:30 | |
*** pkholkin has quit IRC | 14:30 | |
snikitin | dansmith: Hi Dan! I'm trying to implement instance tag spec. https://review.openstack.org/#/c/177112/15 Some part of code was merged in 2014. including Tag db model here https://review.openstack.org/#/c/97168/50/nova/db/sqlalchemy/models.py This model has only two fields 'tag' and 'resource_id' In the last spec review we decided to make tags soft-deletable. For that I need to create 'deleted', 'deleted_at' and 'id' columns. 'id' column must be a pr | 14:32 |
snikitin | imary key. But now primary key columns are 'tag' and 'resource_tag'. So I have to remove primary_key from 'tag' and 'resource_id' columns and create new primary column 'id'. But I can't do it because of your patch 'Block subtractive operations in migrations for Kilo and beyond'. I spoke with jaypipes and the solution we found is to add my new migration into your exception list. https://review.openstack.org/#/c/197349/4/nova/tests/unit/db/test_migrati | 14:32 |
snikitin | ons.py I know that you don't want to add anything into exception list, but now tags table is used nowhere, because API module, supported this table is on review now. No production code has even created a tag yet. So what do you think? Can I add my migration in exception list and drop/alter tags table? | 14:32 |
*** tpatil has joined #openstack-nova | 14:32 | |
dansmith | snikitin: in the meeting now, we can chat in a bit | 14:32 |
snikitin | dansmith: ok | 14:33 |
kashyap | ankit: There is a meeting in-progress in #openstack-meeting. Your request is better suited in this channel. | 14:33 |
*** pkholkin has joined #openstack-nova | 14:34 | |
*** bdr7 has joined #openstack-nova | 14:34 | |
*** jwcroppe has quit IRC | 14:34 | |
*** irenab has joined #openstack-nova | 14:35 | |
*** jwcroppe has joined #openstack-nova | 14:35 | |
*** ijuwang has quit IRC | 14:35 | |
ankit | kashyap: Thanks, I have added my review request in nova meeting :) | 14:36 |
*** nihilifer has quit IRC | 14:39 | |
*** Murali has quit IRC | 14:39 | |
*** thangp has joined #openstack-nova | 14:39 | |
*** yamahata has quit IRC | 14:40 | |
*** jecarey has joined #openstack-nova | 14:41 | |
*** yamahata has joined #openstack-nova | 14:41 | |
openstackgerrit | Jeffrey Zhang proposed openstack/nova: use graduated oslo.policy https://review.openstack.org/198065 | 14:41 |
*** ijuwang has joined #openstack-nova | 14:42 | |
dansmith | snikitin: why do we want tags to be soft deletable? I'm shocked to hear jaypipes being in favor of any new soft-deletable things :) | 14:42 |
*** nkrinner has quit IRC | 14:42 | |
SourabhP | johnthetubaguy: hello | 14:43 |
mriedem | dansmith: it came up in the spec review | 14:43 |
mriedem | dansmith: because it's tied to an instance which is soft deletable | 14:43 |
*** ankit has quit IRC | 14:43 | |
openstackgerrit | Pavel Kholkin proposed openstack/nova: libvirt: optimize multipath call to identify IQN https://review.openstack.org/163733 | 14:43 |
mriedem | so if you have tags on an instance and delete the instance, the tags are gone, but if you restore that instance you can't get the tags back | 14:43 |
dansmith | mriedem: so don't delete the tags? | 14:43 |
*** mnestratov|2 has joined #openstack-nova | 14:43 | |
mriedem | the tags have basically an fkey to the instance | 14:44 |
mriedem | even though we don't do cascading deletes | 14:44 |
johnthetubaguy | SourabhP: hi | 14:44 |
dansmith | right... I don't get the problem | 14:44 |
jaypipes | dansmith: what mriedem said.. | 14:44 |
tpatil | johnthetubaguy: Hi | 14:44 |
*** dims has quit IRC | 14:45 | |
johnthetubaguy | dansmith: is that the soft delete issues with tags? | 14:45 |
mriedem | i guess it just feels wrong to me to leave tags hanging out tied to an instance that's deleted | 14:45 |
dansmith | yeah | 14:45 |
dansmith | mriedem: we leave sysmeta and other things for the same reason | 14:45 |
*** whenry_ has quit IRC | 14:45 | |
johnthetubaguy | seems consistent with those, thats true | 14:45 |
dansmith | seems like a thing we clean up in some DB purge tool, | 14:45 |
dansmith | but making tags soft-deletable just because instance is seems like a backwards step to me | 14:45 |
SourabhP | johnthetubaguy: I have a question regarding adding a new vif driver in nova. Is this activity still doable in this cycle, or I need to wait for the os-vif library work to commence? | 14:45 |
mdbooth | jaypipes: All the Jenkins +1 goodness you've come to expect without the unsightly typos: https://review.openstack.org/#/c/141115/ | 14:45 |
mriedem | i was going to say, at what point do we delete the tags - a script to do that would help | 14:45 |
johnthetubaguy | yeah, thats basically what we do for the other stuff | 14:45 |
tpatil | johnthetubaguy: I want to talk with you about Improve performance of UnShelve API specs | 14:46 |
johnthetubaguy | SourabhP: we did allow it this cycle, but we are closed to new specs I am afraid, it should be easier next cycle though, once we have the os-vif lib in place | 14:46 |
*** mnestratov has quit IRC | 14:46 | |
johnthetubaguy | tpatil: yes, I think we should really now focus on getting this merged in the spec backlog | 14:47 |
*** mdrabe has joined #openstack-nova | 14:47 | |
mriedem | dansmith: jaypipes: snikitin: i'd be up for a revision to the spec if we put something in there about handling the delete case | 14:47 |
tpatil | johnthetubaguy:So Is this spec still in consideration for Liberty cycle? | 14:47 |
dansmith | mriedem: we don't have a thing to do the actual cleanup of instance, sysmeta, etc, etc AFAIK | 14:48 |
johnthetubaguy | tpatil: I don't think it is at this point really | 14:48 |
dansmith | so making the tags implementor do that doesn't seem quite fair | 14:48 |
*** bdr7 has quit IRC | 14:48 | |
dansmith | mriedem: but I agree we need that thing.. I've been asked for it a couple times in the last month | 14:48 |
mriedem | tpatil: is there code up for any of the unshelve api stuff? | 14:48 |
johnthetubaguy | tpatil: we have way too much in liberty already, I don't think we would ever get the code reviewed in time | 14:48 |
tpatil | johnthetubaguy: yes | 14:48 |
*** bdr7 has joined #openstack-nova | 14:48 | |
mriedem | tpatil: i also had questions in the spec this morning | 14:48 |
mriedem | about the actual need for a config option | 14:48 |
jaypipes | mdbooth: done. | 14:49 |
johnthetubaguy | tpatil: I think we should try to get agreement on the spec as soon as we can though, so its ready to go as soon as M opens | 14:49 |
mdbooth | jaypipes: Thanks | 14:50 |
jaypipes | np | 14:50 |
mriedem | ctrath: no more soft delete is a tihng you could help with if you're looking for work | 14:50 |
mriedem | seems to have stalled | 14:50 |
*** dims has joined #openstack-nova | 14:50 | |
SourabhP | johnthetubaguy: any chance of an exception for the vif driver? I was actually waiting to see if os-vif lib work takes place this cycle. My patch from last cycle: https://review.openstack.org/#/c/157616/ | 14:50 |
ctrath | mriedem: sounds good.. throw something my way | 14:50 |
tpatil | mriedem: IMO, If shelve instance is working for ceph/sheeepdog today then it will continue to work in the same way | 14:50 |
mriedem | https://review.openstack.org/#/c/184645/ | 14:50 |
mriedem | ctrath: ^ | 14:50 |
johnthetubaguy | SourabhP: at this point, no we extended the exception process more than I wanted to, but that has finished now I am afraid | 14:51 |
mriedem | SourabhP: we've already approved at least one or two new vif types for liberty | 14:51 |
*** burt has joined #openstack-nova | 14:51 | |
mriedem | SourabhP: did you re-propose any bp for your new type/ | 14:51 |
mriedem | ? | 14:51 |
mriedem | https://bugs.launchpad.net/nova/+bug/1423308 isn't a bug, it's a feature add | 14:51 |
openstack | Launchpad bug 1423308 in OpenStack Compute (nova) "Cisco Nexus1000V to support using 'vemctl' for binding ports" [Wishlist,In progress] - Assigned to Sourabh Patwardhan (sopatwar) | 14:51 |
ndipanov | mriedem, hey man what up with this https://review.openstack.org/#/c/139688/ | 14:51 |
mriedem | so you need a bp | 14:51 |
johnthetubaguy | SourabhP: yeah, looking at your patch, we don't have a blueprint for this | 14:51 |
johnthetubaguy | any yeah, we sure need a blueprint for it | 14:51 |
mriedem | ndipanov: ? | 14:52 |
*** Jeffrey4l has joined #openstack-nova | 14:52 | |
ndipanov | the code is ready I think - no need to pull the "process" card | 14:52 |
SourabhP | oh ok, maybe I waited too long and hoped that os-vif lib to happen this cycle | 14:52 |
mriedem | ndipanov: there is lots of code ready for lots of things that are already approved before the deadlines | 14:52 |
*** otter768 has joined #openstack-nova | 14:52 | |
johnthetubaguy | mriedem: +1 | 14:52 |
ndipanov | true | 14:52 |
ndipanov | I mean whatever | 14:53 |
tpatil | johnthetubaguy: getting approval on specs is more important. Can you please set the priority of specs to "Low" so that people can review our specs in tomorrow's review bash day | 14:53 |
sahid | the spec got the 2 +2 before the deadline | 14:53 |
*** armax has joined #openstack-nova | 14:53 | |
sahid | johnthetubaguy: mriedem ^ | 14:53 |
johnthetubaguy | we have more code up that we can possibly review, we have to be honest to people, and tell them its just not going to happen, as some point | 14:53 |
ndipanov | I just think the whole </3 for nfv is a contentious for the wrong reasons | 14:53 |
ndipanov | and it seeps through in things like this | 14:53 |
*** marzif_ has joined #openstack-nova | 14:53 | |
mriedem | honestly it's not really about nfv | 14:53 |
johnthetubaguy | tpatil: tomorrow is for code reviews I am afraid, and a blueprint has to be approved to get a priority attached | 14:54 |
mriedem | the spec was restored on 6/22 which is really late | 14:54 |
sahid | we have a feature ready spec accepted by 2 core before the deadline | 14:54 |
tpatil | johnthetubaguy: ok | 14:54 |
SourabhP | johnthetubaguy: mriedem: thanks for the clarification! | 14:54 |
*** tsekiyama has joined #openstack-nova | 14:55 | |
sahid | unfortunatly mikal was not here to reapproved it after the rebase and before the deadline | 14:55 |
johnthetubaguy | SourabhP: reach out to me at the beginning of M, and we can make sure things are sorted | 14:55 |
SourabhP | johnthetubaguy: will do | 14:55 |
*** bdr7 has quit IRC | 14:55 | |
johnthetubaguy | sahid: it was on the exception list, but didn't get any votes, so it didn't get the exception, just like the other specs that had the same thing happen to them | 14:56 |
*** bdr7 has joined #openstack-nova | 14:56 | |
mriedem | sahid: the spec freeze for L was 6/25 https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule | 14:56 |
mriedem | sahid: i see mikal +2'ed on 6/22 | 14:56 |
johnthetubaguy | SourabhP: thanks, sorry for all the delays, you dropped between the cracks there, working on documenting this all more here: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule | 14:56 |
mriedem | there weren't 2 +2s | 14:56 |
ndipanov | mriedem, late for what? it's like 2 small patches... I mean whatever | 14:56 |
ndipanov | it's kinda silly to pull process on this because it's a good example of why blanket process is not good for everything | 14:57 |
mriedem | ndipanov: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/libvirt-real-time,n,z | 14:57 |
*** agireud has joined #openstack-nova | 14:57 | |
*** otter768 has quit IRC | 14:57 | |
mriedem | it's a few hundred LOC after all of the deadlines | 14:57 |
mriedem | and we already have a full load | 14:58 |
mriedem | and this isn't a priority | 14:58 |
SourabhP | johnthetubaguy: sure thanks, will keep a closer eye on the deadlines from now on | 14:58 |
dansmith | bauzas: did you see my question(s) on the migrate destinations patch? | 14:58 |
johnthetubaguy | mriedem: +1 | 14:58 |
*** Jeffrey4l has quit IRC | 14:58 | |
dansmith | bauzas: happy to iterate quick on that if you want to take a look | 14:58 |
bauzas | dansmith: not yet, | 14:58 |
*** annegentle has quit IRC | 14:58 | |
bauzas | dansmith: but going there | 14:58 |
ndipanov | mriedem, yes all of that is true and it's still not very justified to block it | 14:59 |
ndipanov | but whatever | 14:59 |
bauzas | dansmith: oooh, seems I was unclear | 14:59 |
ndipanov | it gets in and we find bugs and fix them | 14:59 |
ndipanov | it doesn't and we don't for 6 more months | 15:00 |
bauzas | dansmith: that's not the API which will call the scheduler, but rather the conductor | 15:00 |
*** achanda has joined #openstack-nova | 15:00 | |
bauzas | dansmith: see L261 | 15:00 |
bauzas | dansmith: so there is no blocking call | 15:00 |
bauzas | from the API | 15:00 |
mriedem | ndipanov: i'm sure red hat has a process for carrying it out of tree and fixing the bugs there before its reproposed for M | 15:00 |
bauzas | dansmith: I was just planning to add a call where the conductor is already checkign that | 15:01 |
bauzas | dansmith: but I'll modify the spec for being more clear | 15:01 |
dansmith | bauzas: hang on let me pull it back up | 15:01 |
johnthetubaguy | ndipanov: specs opened for liberty in april time or so, this one was restored a few days before the spec freeze, it got more review than the process allowed for already, there is a point you have to say no | 15:01 |
ndipanov | mriedem, I mean that's exactly the wrong way to go about this | 15:01 |
ndipanov | yes - let's drop it | 15:02 |
ndipanov | not a huge deal | 15:02 |
bauzas | dansmith: ack | 15:03 |
*** sudipto has quit IRC | 15:04 | |
dansmith | bauzas: okay, I see that, but conductor will then have to make a blocking call to scheduler, yes? | 15:04 |
dansmith | bauzas: my point is, we can't return Conflict to he API caller without making a blocking call to scheduler, even if we jump through conductor | 15:04 |
ndipanov | but - most comments on it are some spelling sillyness afaict, and the code is up... so I don't know | 15:06 |
*** achanda has quit IRC | 15:06 | |
dansmith | FWIW, | 15:06 |
dansmith | I think that these are docs read by lots of people and they're easy to spellcheck | 15:06 |
dansmith | missing a deadline because of spelling nits sucks (although that's not what happened here), but ffs.. Just spellcheck it | 15:07 |
*** davideagnello has joined #openstack-nova | 15:07 | |
*** changbl has quit IRC | 15:07 | |
*** browne has quit IRC | 15:07 | |
ndipanov | dansmith, fwiw I do think you are right - it's a low bar | 15:07 |
dansmith | that's my only comment... SPELL SHIT RIGHT | 15:08 |
dansmith | :) | 15:08 |
scheuran | mriedem: Matt, in the review https://review.openstack.org/#/c/182283/8/nova/virt/libvirt/vif.py you asked if vlan could ever be a unicode object | 15:08 |
scheuran | mriedem: I must admit, I have no clue! The input comes from an neutron api response | 15:08 |
scheuran | mriedem: so it's being parsed in some other nova layer | 15:09 |
*** diogogmt has joined #openstack-nova | 15:09 | |
*** coolsvap is now known as coolsvap|away | 15:09 | |
scheuran | mriedem: would you recommend to go with six.text_type in any case? | 15:09 |
*** gabor-t has quit IRC | 15:10 | |
*** dave-mccowan has quit IRC | 15:10 | |
*** davideagnello has quit IRC | 15:11 | |
ndipanov | mriedem, on a slightly less contentious note: https://review.openstack.org/#/c/180637 | 15:14 |
ndipanov | are you ok with the two other patches on there? | 15:14 |
*** wanghua has quit IRC | 15:14 | |
*** sambetts has quit IRC | 15:15 | |
*** ildikov has joined #openstack-nova | 15:16 | |
mriedem | ndipanov: haven't gotten back to them yet after +2 on the bottom change | 15:16 |
*** ildikov is now known as ildikov_afk | 15:16 | |
*** rdopiera has quit IRC | 15:16 | |
ndipanov | kk | 15:16 |
*** sambetts has joined #openstack-nova | 15:17 | |
ndipanov | thanks for reviews on those - it'd be really good to have someone else on core who is somewhat familiar with this stuff | 15:17 |
mriedem | scheuran: so i doubt that vif_details.get(network_model.VIF_DETAILS_VLAN) would have unicode in it | 15:17 |
ndipanov | without having to load a ton of context every tim | 15:17 |
ndipanov | e | 15:17 |
mriedem | ndipanov: well, that's why i hammer you on comment blocks in the code :) | 15:17 |
mriedem | b/c i promise i'll forget | 15:18 |
*** yingjun has joined #openstack-nova | 15:18 | |
*** yingjun has quit IRC | 15:18 | |
mriedem | scheuran: i'm just prickly when it comes to seeing str() on anytihng b/c of prevoius issues with i18n | 15:18 |
mriedem | scheuran: six.text_type would be safest, so might not hurt to just use that | 15:18 |
scheuran | mriedem: I see. I just got rid of it using return '%s.%s' % (src_dev, vlan) | 15:19 |
mriedem | garyk: maybe you know, can the vlan info in vif details ever have unicode? | 15:19 |
*** tonytan4ever has joined #openstack-nova | 15:19 | |
scheuran | mriedem: this is also how others do it in the same class | 15:19 |
mriedem | https://review.openstack.org/#/c/182283/8/nova/virt/libvirt/vif.py | 15:19 |
mriedem | scheuran: ok, cool | 15:19 |
ndipanov | I'm just self conscious about my failure to comprehend how articles work in English so I avoid comments | 15:19 |
mriedem | ndipanov: oh come on, you're a master wordsmith when it comes to speaking 'merican | 15:20 |
ndipanov | for some reason whenever i make a break to think - I add a 'the' or 'a' since I don't have a natural feel for them | 15:20 |
dansmith | I in no way believe this | 15:21 |
mriedem | second, i call bullshit | 15:21 |
garyk | mriedem: as far as i understand they should not. | 15:21 |
ndipanov | the true a story | 15:21 |
dansmith | I'm pretty sure ndipanov talks english betterrer than me | 15:21 |
mriedem | me fail english? unpossible! | 15:21 |
scheuran | mriedem: there was also the exception that I introduced and you where asking if the user could recover from it | 15:21 |
garyk | i speak a gooder english | 15:21 |
mriedem | scheuran: yeah | 15:22 |
scheuran | mriedem: the answer is no! This is either a bug in the neutron code or a missconfiguration of the neutron code | 15:22 |
ndipanov | it's the gooder english garyk | 15:22 |
garyk | heh | 15:22 |
scheuran | mriedem: so I wonder if such a exception is required at all.. | 15:22 |
mriedem | scheuran: well, i think that's what i was looking for (not code bugs, but misconfigs) | 15:22 |
mriedem | scheuran: is there something in neutron config that would impact this? | 15:22 |
*** haypo has quit IRC | 15:22 | |
*** dave-mccowan has joined #openstack-nova | 15:22 | |
*** vishwana_ has quit IRC | 15:23 | |
mriedem | scheuran: if not, and the only way it'd happen is some regression bug in the neutron code, then i'd just assume those values wil lbe in the vif_details | 15:23 |
scheuran | mriedem: yes, so these two values are stored i a neutron config file, and will be added to the vif information by the ml2 plugin | 15:23 |
mriedem | else we get NoneType and a bug is obvious | 15:23 |
mriedem | scheuran: ah, | 15:23 |
scheuran | mriedem: but checking for that should be done in neutron I guess | 15:23 |
*** vishwanathj has joined #openstack-nova | 15:23 | |
*** jamielennox is now known as jamielennox|away | 15:23 | |
*** SourabhP has quit IRC | 15:23 | |
mriedem | scheuran: well, my point is, if there is a chance for a user error here with configs, i'd point that out in the exception message, saying 'check your xyz config' | 15:23 |
mriedem | i just don't think the error given would be helpful to anyone today | 15:24 |
mriedem | which is why i ask for hints on how to recover | 15:24 |
scheuran | scheuran: yes, that's right | 15:25 |
*** bkopilov has quit IRC | 15:25 | |
garyk | scheuran: i am with mriedem here. better to be safe than sorry. today neutron may do it and tomorrow it may not | 15:26 |
*** bkopilov has joined #openstack-nova | 15:26 | |
nagyz_ | mriedem, any opinion on https://bugs.launchpad.net/nova/+bug/1457517 ? | 15:26 |
openstack | Launchpad bug 1457517 in OpenStack Compute (nova) "Unable to boot from volume when flavor disk too small" [Undecided,New] | 15:26 |
nagyz_ | I guess I've missed some discussion around this as it used to work in I, and now it's broken in K :) | 15:27 |
*** tongli has joined #openstack-nova | 15:27 | |
mriedem | nagyz_: i'd ask ndipanov | 15:27 |
*** jgriffith has left #openstack-nova | 15:27 | |
ndipanov | oh I know this | 15:27 |
ndipanov | it's a bug | 15:28 |
scheuran | mriedem: What about getting rid of this exception at all? Cause this is nothing the Openstack user can recover from. Only the Openstack admin could solve that problem | 15:28 |
ndipanov | mdbooth, worked on this | 15:29 |
ndipanov | nagyz_, ^ | 15:29 |
ndipanov | I think the solution is not that simple | 15:29 |
mriedem | scheuran: well, the admin is the one that gets the support ticket and sees the logs right? | 15:29 |
mriedem | scheuran: the user just sees that networking failed | 15:29 |
mriedem | and opens the support ticket | 15:29 |
mriedem | ndipanov: nagyz_: yeah, there was some dependent RT change that looked like it was languishing | 15:30 |
mriedem | and i'm not very familiar with the RT or scheduler filters stuff here | 15:30 |
ndipanov | mriedem, no I don't this has to do with RT | 15:30 |
ndipanov | the thing is | 15:30 |
ndipanov | we grab metadata from volume | 15:30 |
pkholkin | dhellmann: are you here? | 15:30 |
*** dims has quit IRC | 15:30 | |
nagyz_ | in my head it's a simple fix: if we're booting from volume then ignore the flavor's disk size | 15:30 |
ndipanov | and then we do a check against it like if it was an image | 15:30 |
ndipanov | nagyz_, yes that is one way to do it | 15:31 |
pkholkin | dhellmann: could you please remove -2 from this patch https://review.openstack.org/#/c/173834/ ? I think my backport needs it https://review.openstack.org/#/c/199569/ | 15:31 |
ndipanov | but then | 15:31 |
ndipanov | do we really want to sidestep that check | 15:31 |
mdbooth | I did | 15:31 |
nagyz_ | ndipanov, in case from booting from volume? yes | 15:31 |
nagyz_ | ndipanov, that only makes sense for ephemeral checks IIRC | 15:31 |
nagyz_ | s/IIRC/IMHO | 15:31 |
mdbooth | nagyz_: So, I believe the correct fix is to set the image size to zero when booting from a volume | 15:31 |
ndipanov | because it's not only about the size of the disk but maybe about where you want a flavor to go based on io it can do | 15:32 |
mdbooth | My patch set it to the actual size of the volume | 15:32 |
*** jamielennox|away is now known as jamielennox | 15:32 | |
ndipanov | so it's not as clear cut as that | 15:32 |
mdbooth | Whereas zero seems to be interpreted as 'ignore this' | 15:32 |
*** dims has joined #openstack-nova | 15:32 | |
*** dims has quit IRC | 15:32 | |
ndipanov | mdbooth, so that would have the same result | 15:32 |
ndipanov | but the question is do we want to do it | 15:32 |
ndipanov | if we agree that - yes | 15:32 |
mdbooth | The potential issue with that is that it's an undocumented dict of badness, which is passed to several places | 15:32 |
ndipanov | yes that too | 15:32 |
*** dims has joined #openstack-nova | 15:33 | |
ndipanov | nagyz_, does that make some sense? | 15:33 |
*** igordcard_ has joined #openstack-nova | 15:33 | |
nagyz_ | I have to be honest I'm not seeing the issue here - if the user choose the provision a volume beforehand than nova should just use the metadata from the volume, no? since the argument is around scheduler input? | 15:34 |
*** nihilifer has joined #openstack-nova | 15:34 | |
*** nihilifer has quit IRC | 15:34 | |
ndipanov | nagyz_, yes | 15:34 |
ndipanov | that is a good point too | 15:34 |
ndipanov | mdbooth, ^ | 15:34 |
mriedem | mdbooth: we're working on the undocumented dict of badness | 15:34 |
mdbooth | mriedem: :) | 15:35 |
mriedem | mdbooth: see https://review.openstack.org/#/c/192651/ per documentation as a start | 15:35 |
*** nihilifer has joined #openstack-nova | 15:35 | |
mriedem | mdbooth: but agree i'd like to see more of this modeled and documented in the code since it's very confusing how many pitfalls there are | 15:35 |
*** bdr768 has joined #openstack-nova | 15:35 | |
*** bdr768 is now known as jgriffith | 15:35 | |
*** shyama has quit IRC | 15:35 | |
*** diogogmt has quit IRC | 15:35 | |
mdbooth | nagyz_: Absolutely, which is exactly what it does | 15:35 |
mdbooth | The only issue there is accounting for image size | 15:36 |
dansmith | isn't "The Undocumented Dict of Badness" the new Indiana Jones movie? | 15:36 |
mdbooth | However, you still want to sanity check the flavour's cpu and memory against the values expected by the original image | 15:36 |
*** bkopilov has quit IRC | 15:36 | |
mriedem | big ol' bag of dicts (tm) | 15:36 |
mriedem | that's the name of my band | 15:36 |
mdbooth | lol | 15:37 |
*** piyanai has quit IRC | 15:37 | |
mriedem | i'll always remember the time mrodden flushed when i said 'bag of dicts' too loud in a crowded restaurant once | 15:37 |
ndipanov | dansmith, ahaha | 15:37 |
*** diogogmt has joined #openstack-nova | 15:37 | |
scheuran | mriedem: yes you're right. But config missconfiguration should be handled in neutron before. Most of the other vif types also do not check anything... so I wonder if I just should join them and get rid of the exception totally | 15:38 |
mriedem | scheuran: regardless, if there is the chance for misconfiguration and nova is the consumer and sees it, it can point it out that the config needs to be checked | 15:38 |
mriedem | scheuran: else, we need something that does validation in neutron | 15:38 |
mriedem | and logs the misconfig error there before it gets to nova (passing the bad vif_details that is) | 15:39 |
mriedem | scheuran: just because neutron and the vif stuff has always sucked doesn't mean it needs to continue to suck so bad | 15:39 |
mriedem | from a serviceability pov | 15:39 |
jaypipes | snikitin, dansmith, mriedem: OK, so with the server tag soft-delete thing... mriedem, I thought you were adamant about making tags soft-deleteable with the instance object until we figure out a long-term archival solution? Based on that, I told snikitin that it should be doable to add an exception to dansmith's recent test of schema migrations for the schema migration that would be needed to make tags soft-deleteable, since no REST API part has e | 15:40 |
jaypipes | ver hit master for tags, which means no user has ever tagged an instance in production. | 15:40 |
*** agireud has quit IRC | 15:40 | |
scheuran | mriedem: ok thx. So I will updated the exception text and see what others comment on it | 15:40 |
mriedem | jaypipes: well, i was under the assumption that if i delete the instance i'm also deleting the tags, | 15:41 |
mriedem | dansmith: pointed out just don't delete the tags, | 15:41 |
mriedem | which opens up the question of when do we cleanup the tags? | 15:41 |
mriedem | for deleted instances i mean | 15:41 |
mriedem | that could be a task, it could be a db cleanup command | 15:42 |
dansmith | we don't currently delete instance tags when we delete an instance, AFAICT | 15:42 |
dansmith | mriedem: that's a larger question of cleanup that affects many other things, not just tags | 15:42 |
*** piyanai has joined #openstack-nova | 15:42 | |
tpatil | mriedem: I have replied to your comments on PS https://review.openstack.org/#/c/135387/15. Please check | 15:42 |
jaypipes | mriedem, dansmith: you are both correct, of course. I just figured it would be easier to make tags consistent with the other instance-related objects (even though tags can be related to more than just instances). | 15:43 |
*** agireud has joined #openstack-nova | 15:43 | |
*** ctrath has quit IRC | 15:43 | |
*** jcoufal has quit IRC | 15:43 | |
*** ctrath has joined #openstack-nova | 15:43 | |
dansmith | jaypipes: well, IMHO, we shouldn't just add new soft-deletable things if there's no good reason to do so if we're trying to get rid of them | 15:43 |
jaypipes | mriedem, dansmith, snikitin: warning: I will need to leave my comfy linux quarters and boot into Windoze in order to use friggin webex shortly. | 15:44 |
dansmith | okay so how about this: | 15:44 |
dansmith | tags has been dragging on for decades | 15:44 |
jaypipes | dansmith: so, I would agree with you, but I think that a single archival solution that cleans the soft-delete stuff up once and for all is an easier thing to do than make things internally inconsistent. | 15:44 |
n0ano | jaypipes, do what I do, windows laptop permanent running but closed on the desk and then run rdesktop from your real linux machine into the windows box | 15:45 |
dansmith | jaypipes: what does that have to do with this decision though? We either soft delete or don't do anything, but either way, the rows are still there and need to be cleaned up right? | 15:45 |
*** ctrath has quit IRC | 15:45 | |
jaypipes | n0ano: that sounds like hell. :) | 15:45 |
dansmith | n0ano: you know where that laptop sits on your desk? jaypipes has a pug sitting there. | 15:45 |
*** ctrath has joined #openstack-nova | 15:46 | |
n0ano | dansmith, mine's behind a nat'd firewall, good luck getting to it | 15:46 |
jaypipes | dansmith: if we correct the tags to be soft-deletable, then they will behave identiaclly to other things like instance_extra. that's what I'd like to see, then we work long-term on just removing the soft-delete crap in favor of an arhcival framework. | 15:46 |
jaypipes | dansmith: the way it is now, tags are a special unicorn since they don't inherit from oslo.db.sqlalchemy.SoftDeleteMixin | 15:47 |
n0ano | jaypipes, yeah, it's not worth the effort to fight my windows centric IT and this way I can the bulk of my work on Linux | 15:47 |
dansmith | jaypipes: I don't think I understand that at all | 15:47 |
dansmith | jaypipes: maybe we do a hangout with the few of us when you return from windows purgatory? | 15:47 |
* jaypipes takes a note of n0ano's comment just in case any more Intel recruiters come a knockin. | 15:47 | |
jaypipes | dansmith: unfortunately, I won't be back for a while (couple hours) and when I do return, I need to prepare for yet another webex meeting from 5-6pm. :( | 15:48 |
mriedem | i tend to agree with liking consistency and fewer unicorns | 15:48 |
dansmith | sigh, this is crazy | 15:48 |
*** belmoreira has quit IRC | 15:48 | |
* n0ano intel has recruiters - who knew | 15:48 | |
mriedem | dansmith: would you be opposed to a nova-manage db purge-tags command? | 15:49 |
mrodden | sort of related to the softdelete thing... as an operator, would be nice to have some kind of official shadow tabling/archival setup vs having to set up a cronjob to do the nova-manage archive_db thing... | 15:49 |
jaypipes | dansmith: I'm just trying to make things consistent. I recognize I was the original one who asked snikitin to not make tags soft-deleteable, but mriedem has convinced me that being consistent with the other models is important. | 15:49 |
mriedem | even though it's not universal? | 15:49 |
dansmith | jaypipes: it's not the same thing as a metadata entry though | 15:49 |
dansmith | I'd just really like to have a high-bw conversation about it, whenever that happens | 15:50 |
mriedem | i'm game for that | 15:50 |
mriedem | i have a headset that works! | 15:50 |
dansmith | before I agree to a special case of breaking our rules | 15:50 |
mriedem | mrodden: chime in on https://review.openstack.org/#/c/184645/ then | 15:50 |
mriedem | mrodden: and/or https://review.openstack.org/#/c/137669/ | 15:50 |
*** whenry_ has joined #openstack-nova | 15:51 | |
dansmith | mriedem: oh, missed your purge-tags note | 15:52 |
*** bkopilov has joined #openstack-nova | 15:52 | |
jaypipes | dansmith: 3pm EST is the earliest I can do. | 15:52 |
dansmith | mriedem: I'd rather that than the proposed schema change, yeah, even if it's just temporary | 15:52 |
dansmith | jaypipes: does it have to be today? | 15:52 |
jaypipes | dansmith: no, of course not. | 15:52 |
dansmith | jaypipes: I mean if today is bad, we can do tomorrow | 15:52 |
tpatil | mriedem: We have uploaded code for improving unshelving instance performance https://review.openstack.org/#/c/184871/, workflow is set to -2 as specs are not approved | 15:52 |
jaypipes | dansmith: tomorrow would be much beter. | 15:52 |
dansmith | jaypipes: assuming snikitin is okay with waiting a day | 15:53 |
jaypipes | dansmith: yes, that's fine. | 15:53 |
dansmith | okay | 15:53 |
jaypipes | ok, will see you guys in a few hours. | 15:53 |
*** jaypipes has quit IRC | 15:53 | |
*** gszasz has quit IRC | 15:53 | |
mriedem | cool, now let's come up with the *real* plan | 15:54 |
mriedem | hehehe | 15:54 |
dansmith | snikitin: sorry :/ | 15:54 |
*** coolsvap|away is now known as coolsvap | 15:54 | |
mriedem | we're going to rename tags as pugs and remove the limit of 50 per resource | 15:54 |
*** pask81 has joined #openstack-nova | 15:54 | |
*** jlanoux has quit IRC | 15:55 | |
*** yassine_ has quit IRC | 15:55 | |
dansmith | mriedem: so, resource_id is a primary key | 15:55 |
dansmith | er, wait | 15:55 |
dansmith | so is tag | 15:55 |
mrodden | as an operator, i really like the softdelete thing because i get an audit-like trail when i need to see what happened to a users instances... | 15:56 |
mrodden | so i am not sure about that BP | 15:56 |
dansmith | mrodden: we should provide the audit trail in a different way | 15:56 |
mrodden | i will admit that its totally not consistent in nova lol | 15:56 |
dansmith | the current way is terrible | 15:56 |
*** moshele has quit IRC | 15:56 | |
mrodden | dansmith, yeah i think that would be preferred | 15:56 |
mrodden | and TBH things like ceilometer aren't really ... super useful for that today | 15:57 |
*** shyama has joined #openstack-nova | 15:58 | |
dansmith | mriedem: so, what if we do the purge thing, but just do it for instance and children, and put a big warning on it that says "this deletes the archive stuff. It's just a tool to help you, but not a proper archival thing" | 15:58 |
dansmith | mriedem: and then we just delete instance, sysmeta, metadata, tags, etc | 15:58 |
*** ctrath has quit IRC | 15:58 | |
*** claudiub has quit IRC | 15:59 | |
dansmith | mriedem: like, I don't think it's much more complicated to do the more useful thing | 15:59 |
*** ctrath has joined #openstack-nova | 15:59 | |
dansmith | also, | 15:59 |
bauzas | dansmith: okay I see the problem, thanks for your thoughts | 15:59 |
dansmith | we never really delete instances, and require people to manually purge today | 15:59 |
*** davideagnello has joined #openstack-nova | 15:59 | |
dansmith | so I don't see any reason why they can't just delete tags based on instances they delete | 16:00 |
dansmith | soft-deleted or not | 16:00 |
dansmith | really doesn't matter, IMHO | 16:00 |
dansmith | bauzas: cool | 16:00 |
bauzas | dansmith: so, we should rather async that by accepting on the API then still 'cast' the conductor and only change the vm state then if a problem | 16:00 |
*** davideagnello has quit IRC | 16:00 | |
dansmith | bauzas: I dunno about vm_state, but in general yes | 16:01 |
mriedem | dansmith: "so I don't see any reason why they can't just delete tags based on instances they delete" - does that mean delete the associated tags when we delete the instance? | 16:01 |
mriedem | or as part of a tool? | 16:01 |
dansmith | bauzas: when we create the migration record, we could set the status=scheduling and then status=conflict | 16:01 |
*** browne has joined #openstack-nova | 16:01 | |
dansmith | mriedem: I mean we don't actually delete anything today, not even instances | 16:01 |
bauzas | dansmith: sounds good to me | 16:01 |
dansmith | mriedem: so not deleting tags is the same. People have to delete their soft-deleted instances manually from the DB to reclaim the space | 16:01 |
bauzas | dansmith: any stuff I could look at for the migration record ? | 16:01 |
mriedem | dansmith: right | 16:01 |
*** pask81 has quit IRC | 16:02 | |
dansmith | bauzas: just look for objects.Migration usage.. it's only in a few places | 16:02 |
bauzas | dansmith: sounds good | 16:02 |
bauzas | dansmith: okay, will change the spec | 16:02 |
dansmith | mriedem: so if they have to do that, they might as well write their delete query to take out the associated tags in the process, right? | 16:02 |
bauzas | (trying to do that before going on PTO for the next 5 days, eh) | 16:02 |
mriedem | dansmith: yeah | 16:03 |
*** angdraug has joined #openstack-nova | 16:03 | |
*** bitblt has joined #openstack-nova | 16:03 | |
mriedem | i was with you at "and then we just delete instance, sysmeta, metadata, tags, etc" - then "so I don't see any reason why they can't just delete tags based on instances they delete" confused me - but i think you were just backing up the previous idea | 16:03 |
dansmith | yes | 16:04 |
mriedem | but yeah, if we have a tool that just hard deletes instance-related thingies that normal instance-delete doesn't do, then sure | 16:04 |
mriedem | since we can lump tags in with the rest | 16:04 |
dansmith | yeah, that would be best, IMHO | 16:04 |
mriedem | it's at least a manual step for a not very hard thing until we can put some automation around archive and no more soft delete | 16:05 |
dansmith | yeah | 16:05 |
*** dims_ has joined #openstack-nova | 16:05 | |
mriedem | ctrath: make it so | 16:05 |
dansmith | it will avoid a schema migration _and_ fill a need that people currently have | 16:05 |
*** lucas__ has quit IRC | 16:05 | |
rlrossit | mriedem: dansmith: do you think I should put up a patch for logging what goes on in the object serializer so we can see what the heck is going on with this patch? | 16:05 |
*** unicell1 has quit IRC | 16:05 | |
dansmith | rlrossit: yeah, put it in front of this one | 16:06 |
mriedem | rlrossit: i'd say yes, throw that up before your change | 16:06 |
dansmith | rlrossit: did you see my comment from yesterday? | 16:06 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Set autodoc_index_modules=True so tox -e docs builds module docs again https://review.openstack.org/199297 | 16:06 |
rlrossit | dansmith: if that's the one where you ask if we should remove the subobject, I replied to that one because I don't think that's the root of the problem (though I could be wrong) | 16:07 |
openstackgerrit | jichenjc proposed openstack/nova: Add direct power_on function in compute layer https://review.openstack.org/200165 | 16:07 |
dansmith | rlrossit: and I think it is | 16:07 |
rlrossit | dansmith: well if I had to gamble on who's most likely right, I would bet on you... | 16:08 |
dansmith | rlrossit: if the subobject was added in some version, then the older node won't know about that object, so if it's there, it's just going to trigger it to ask to backport it | 16:08 |
dansmith | rlrossit: if I'm wrong it'll be that much sweeter for you, right? :D | 16:08 |
bauzas | dansmith: sounds good for using the Migration.status field, as I can see in the conductor | 16:08 |
rlrossit | but in the case of lists, the subobject is literally always just 'object' with the object it holds... that never changes in any version | 16:08 |
dansmith | bauzas: cool | 16:08 |
bauzas | dansmith: I guess we have the API resource for seeing all migrations right? | 16:08 |
dansmith | rlrossit: sure, but you don't know it's recursing on a list | 16:09 |
bauzas | dansmith: I remember something in the API, I need to look at the official docs | 16:09 |
openstackgerrit | Andreas Scheuring proposed openstack/nova: Add macvtap as new virtual interface (vif) type to Nova's libvirt driver https://review.openstack.org/182283 | 16:09 |
*** dims has quit IRC | 16:09 | |
rlrossit | dansmith: ...this function is only on the ListObjectBase right? | 16:09 |
dansmith | bauzas: it's admin-only, but yes | 16:09 |
bauzas | dansmith: my point being, I want to make sure that the user knows it failed | 16:09 |
dansmith | gah ETOOMANYTHREADS | 16:09 |
bauzas | dansmith: okay, that's good then, since evacuate and live-migrate are admin too | 16:10 |
bauzas | ah ah | 16:10 |
dansmith | bauzas: ah, right | 16:10 |
rlrossit | lol | 16:10 |
bauzas | objects.DanSmith.lock() | 16:10 |
dansmith | bauzas: but yeah, for these two, those users can see the migrations | 16:10 |
rlrossit | gotta speed up that context switching | 16:10 |
dansmith | bauzas: and we want to make that api exposed to the user, we just need to clean up some things first | 16:10 |
dansmith | bauzas: so that they can see that an admin moved their thing | 16:10 |
bauzas | dansmith: agreed, and all of that is just RBAC eh :) | 16:10 |
dansmith | bauzas: yeah | 16:11 |
bauzas | dansmith: so any operator can just change the policy | 16:11 |
nagyz_ | mdbooth, sorry I had to drop off in a meeting and looking back I don't see any resolutions :) was there any? | 16:11 |
bauzas | dansmith: ok, thanks for your help | 16:11 |
dansmith | bauzas: I think we have to make the policy finer grained before we can, but yes that's the idea | 16:11 |
bauzas | dansmith: objects.DanSmith.free_lock() | 16:11 |
*** exploreshaifali has joined #openstack-nova | 16:11 | |
bauzas | or semaphore-1 | 16:11 |
rlrossit | objects.DanSmith.lock() :) | 16:11 |
*** fawadkhaliq has joined #openstack-nova | 16:11 | |
dansmith | rlrossit: okay, so I think the deal is that you're now slipping through a list to an object, and then not dropping a thing from it | 16:12 |
ctrath | mriedem: alright. I'll have to do some research of the code in order to completely understand what your guys are referring to when speaking of tags | 16:12 |
dansmith | rlrossit: but this is just overridden on the mixin | 16:12 |
dansmith | rlrossit: I dunno, I'd do the logging thing and see how it goes | 16:12 |
dansmith | rlrossit: did my recheck poke the bug again? | 16:12 |
*** rotbeard has quit IRC | 16:12 | |
mdbooth | nagyz_: I believe the correct solution is to set 'size' to 0 when pulling image metadata from a volume | 16:13 |
rlrossit | dansmith: it looks to still be problematic if you're talking about the recheck from yesterday afternoon | 16:13 |
nagyz_ | mdbooth, is there patch/review for that at the moment? | 16:13 |
mdbooth | nagyz_: No | 16:13 |
nagyz_ | ok so that's something I can feel free to patch when I get home? :) | 16:14 |
dansmith | rlrossit: yeah | 16:14 |
*** scheuran has quit IRC | 16:14 | |
dansmith | rlrossit: you didn't hit it before and then twice in a row after rebasing on mriedem's thing | 16:14 |
nagyz_ | and I guess there's no unit test either at the moment to test this otherwise it wouldn't be broken? | 16:14 |
mdbooth | It should have the effect of ignoring image size, whilst still sanity checking other values | 16:14 |
dansmith | rlrossit: surely seems suspcious | 16:14 |
mdbooth | There is a unit test, and it tests that the value is correctly broken :) | 16:14 |
rlrossit | dansmith: you saying it's time to point fingers? | 16:14 |
nagyz_ | mdbooth, lol | 16:14 |
nagyz_ | mdbooth, ok once I get home I'll see what I can do. | 16:15 |
dansmith | rlrossit: well, after you left I blamed all this on mriedem anyway | 16:15 |
dansmith | rlrossit: so yeah, you have a point on the list thing, BUT, if it's not in the list of children then it's broken anyway | 16:15 |
dansmith | rlrossit: your break will just ignore the impossibility and cause failures elsewhere like we're seeing | 16:15 |
rlrossit | dansmith: agree on that one | 16:15 |
dansmith | rlrossit: so making it explode there (or by deleting the thing) will make it easier to spot I think | 16:16 |
rlrossit | I should add a comment saying you should never get here :P those are always fun | 16:16 |
dansmith | well, it's true | 16:16 |
mriedem | there are some comments like that in nova-network | 16:16 |
dansmith | raise Exception("derp") or something | 16:16 |
mriedem | "this should never happen, but..." | 16:16 |
rlrossit | raise Exception("it's mriedem's fault") | 16:16 |
dansmith | ues | 16:16 |
*** Marga_ has quit IRC | 16:17 | |
dansmith | rlrossit: have I said it's really good to have you building context on this stuff? Too few people understand these details and they're pretty core to nova these days | 16:17 |
dansmith | mriedem: amirite? | 16:17 |
*** haypo has joined #openstack-nova | 16:17 | |
*** haypo has joined #openstack-nova | 16:17 | |
*** tjones1 has joined #openstack-nova | 16:17 | |
mriedem | understanding objects is importante? | 16:17 |
dansmith | muy importante | 16:18 |
mriedem | everything is important to everyone at all times | 16:18 |
mriedem | fin | 16:18 |
dansmith | alright well I retract my gracious comment then | 16:18 |
*** piyanai has left #openstack-nova | 16:18 | |
rlrossit | oh by the way dansmith... I have 4 cupcakes left over from yesterday in my freezer just FYI | 16:18 |
mriedem | well ndipanov said i needed to understand the bdm stuff | 16:18 |
dansmith | \o/ | 16:18 |
rlrossit | ask mriedem and ctrath, they'll probably tell you how amazing they were | 16:19 |
rlrossit | I can save one for you if you want it! | 16:19 |
dansmith | oregon just legalized extra-special baked goods, so that's a tall order! | 16:19 |
mriedem | as long as i never have to understand the resource tracker or scheduler stuff, i think i'll be happy | 16:19 |
ctrath | rlrossit: they were good. Did you make them? | 16:19 |
mriedem | dansmith: medicinal or recreational? | 16:19 |
dansmith | mriedem: recreational | 16:19 |
*** afazekas has quit IRC | 16:19 | |
mriedem | ooo, beat MN then | 16:19 |
*** xgsa has quit IRC | 16:19 | |
mriedem | we're only medicinal as of last week | 16:19 |
*** hemnafk is now known as hemna | 16:19 | |
rlrossit | mriedem: I think I'm coming down with cataracts... | 16:20 |
dansmith | we've been medicinal so long that they hand you medicinal cards on the way into the state now | 16:20 |
rlrossit | ctrath: I baked them, my mom frosted them, and I put on the sprinkles | 16:20 |
dansmith | rlrossit: are you twelve? | 16:20 |
mriedem | haha | 16:20 |
ctrath | lol | 16:20 |
rlrossit | dansmith: almost | 16:20 |
mriedem | 11.5, dick | 16:20 |
dansmith | ......... | 16:21 |
*** jamielennox is now known as jamielennox|away | 16:21 | |
dansmith | moving on | 16:21 |
mriedem | rlrossit: just keep them for a couple of weks | 16:21 |
mriedem | i'm sure they'll still be good | 16:21 |
* rlrossit is unsure if mriedem is being sarcastic | 16:21 | |
rlrossit | freezers seriously can keep those bad boys for a long time | 16:21 |
*** boris-42 has quit IRC | 16:22 | |
rlrossit | dansmith: so should I make the change that you suggested, upload it, see what jenkins says, and while that's going put up a change for the logging? | 16:22 |
*** jichen has quit IRC | 16:22 | |
dansmith | rlrossit: that's fine I guess. I'd put the logging thing in front, but you can probably do the first change quicker and get a full run on it in the meantime, so that's fine | 16:23 |
bauzas | johnthetubaguy: had no real time to help with the nova dash for tomorrow, but I guess you know sdague's https://github.com/stackforge/gerrit-dash-creator ? | 16:25 |
johnthetubaguy | bauzas: no worries, I am trying to get some stuff lined up so we can discuss it at the midcyle, so focusing on that for the rest of the day, we should be OK with just the etherpad I hope | 16:27 |
bauzas | johnthetubaguy: I was thinking of calling launchpadlib to get all the changes related to the blueprint and mark them to the dash creator | 16:27 |
bauzas | johnthetubaguy: see my work for the NFV workgroup here for getting all the changes from a BP whiteboard https://github.com/sbauza/gerrit-dashboard-nfv/blob/master/scrapnfving/sketching.py | 16:28 |
*** jamielennox|away is now known as jamielennox | 16:29 | |
bauzas | johnthetubaguy: that can be a nice way to get a list of changes to add to the gerrit dash creator as a entrypoint | 16:29 |
johnthetubaguy | bauzas: sorry, I am just not going to get chance to do that today | 16:29 |
bauzas | johnthetubaguy: same goes for me | 16:29 |
bauzas | johnthetubaguy: so perhaps we can try to see how to help with that later one | 16:30 |
bauzas | later on, even | 16:30 |
johnthetubaguy | bauzas: no halm in trying it for a later one | 16:30 |
*** derekh has quit IRC | 16:30 | |
johnthetubaguy | bauzas: I am really just holding out for the gerrit user tags | 16:30 |
bauzas | johnthetubaguy: ack | 16:31 |
bauzas | johnthetubaguy: well, if all people were correcting marking the topic names | 16:31 |
bauzas | ... | 16:31 |
bauzas | johnthetubaguy: it wouldn't need that above ^ | 16:31 |
johnthetubaguy | bauzas: we have a list of patches for tomorrow, that should do us | 16:31 |
bauzas | johnthetubaguy: it could be a single Gerrit query with the list of topic names being the list of bps | 16:31 |
*** nic has joined #openstack-nova | 16:32 | |
bauzas | johnthetubaguy: yup, for sure | 16:32 |
bauzas | johnthetubaguy: but since I'll be on PTO, I was looking how to help :) | 16:32 |
bauzas | asynchronously :) | 16:32 |
johnthetubaguy | bauzas: build a nice openstack logo shaped sand castle, maybe ;) | 16:33 |
*** kiran-r has joined #openstack-nova | 16:33 | |
bauzas | johnthetubaguy: eeeeeeh, sounds a good idea ! | 16:34 |
*** annegentle has joined #openstack-nova | 16:35 | |
*** mgarza_ has joined #openstack-nova | 16:35 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova-specs: Proposes to check destination on migrations https://review.openstack.org/179224 | 16:35 |
*** coolsvap is now known as coolsvap|away | 16:36 | |
*** ndipanov has quit IRC | 16:36 | |
*** jistr has quit IRC | 16:37 | |
*** angdraug has quit IRC | 16:37 | |
*** annashen has joined #openstack-nova | 16:38 | |
*** unicell has joined #openstack-nova | 16:40 | |
*** Nikolay_St has quit IRC | 16:41 | |
*** jgriffith1 has joined #openstack-nova | 16:42 | |
openstackgerrit | Tracy Jones proposed openstack/nova: Cleanup of compute api reboot method https://review.openstack.org/127427 | 16:42 |
*** jgriffith has quit IRC | 16:42 | |
*** jgriffith1 is now known as jgriffith | 16:43 | |
*** lucas__ has joined #openstack-nova | 16:44 | |
*** bitblt has quit IRC | 16:45 | |
*** lucas___ has joined #openstack-nova | 16:46 | |
*** annegentle has quit IRC | 16:46 | |
*** lucas___ has quit IRC | 16:47 | |
*** lucas___ has joined #openstack-nova | 16:47 | |
*** lucas__ has quit IRC | 16:48 | |
*** annegentle has joined #openstack-nova | 16:50 | |
*** garyk has quit IRC | 16:50 | |
mriedem | dansmith: you want i should start writing a backlog spec for nova-manage db purge-deleted-instances? | 16:51 |
*** lucas___ has quit IRC | 16:51 | |
*** lucasagomes is now known as lucas-beer | 16:53 | |
*** otter768 has joined #openstack-nova | 16:53 | |
*** andrewbogott is now known as andrewbogott_afk | 16:55 | |
melwitt | mriedem: is that to get an instances targeted thing that does a real delete as opposed to the 'nova-manage db archive_deleted_rows' which moves all deleted to shadow tables? | 16:55 |
mriedem | melwitt: yeah | 16:55 |
mriedem | melwitt: that and everything related to instances here http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/instance.py#n43 | 16:56 |
*** garyk has joined #openstack-nova | 16:56 | |
mriedem | plus instance_extra i think | 16:57 |
mriedem | ctrath: ^ | 16:57 |
melwitt | mriedem: okay | 16:57 |
*** lucas__ has joined #openstack-nova | 16:57 | |
mriedem | melwitt: is there already a proposal for that? | 16:58 |
*** otter768 has quit IRC | 16:58 | |
mriedem | outside of jay's archive thing and lxsli's no more soft delete? | 16:58 |
melwitt | mriedem: not that I know of | 16:58 |
mriedem | melwitt: this would be temporary until those other things land | 16:58 |
mriedem | we shall call it, 'nova-manage db shit-or-get-off-the-pot' | 16:58 |
melwitt | lol | 16:59 |
*** whenry_ has quit IRC | 16:59 | |
*** busterswt has joined #openstack-nova | 17:02 | |
*** lucas__ has quit IRC | 17:02 | |
*** SourabhP has joined #openstack-nova | 17:03 | |
dansmith | mriedem: on another call :( | 17:04 |
*** Sukhdev has joined #openstack-nova | 17:05 | |
*** afazekas has joined #openstack-nova | 17:06 | |
johnthetubaguy | mriedem: melwitt: we have some code to do that internally, that might be useful, just hasn't been upstreamed yet, a backlog spec would help force that conversation | 17:06 |
mriedem | johnthetubaguy: yeah that's the plan | 17:07 |
johnthetubaguy | the main smarts that are quite specific to use, is that it throttles the delete rate based on how far the DB read slaves are lagging | 17:07 |
mriedem | hopefully can start that this afternoon | 17:07 |
johnthetubaguy | mriedem: sweet | 17:07 |
johnthetubaguy | s/to use/to use/ | 17:07 |
johnthetubaguy | arg | 17:07 |
mriedem | ha | 17:07 |
johnthetubaguy | s/use/us/ | 17:07 |
johnthetubaguy | so thats a sign I need to go cook tea, and stop attempting to write docs | 17:08 |
*** coolsvap|away is now known as coolsvap | 17:08 | |
johnthetubaguy | it will not go well | 17:08 |
mriedem | yeah i figured i'd just start the 'typey typey' as mikal would say we others can point out the specifics | 17:08 |
johnthetubaguy | :) | 17:08 |
mriedem | s/we/so/ | 17:08 |
melwitt | cool | 17:09 |
*** davideagnello has joined #openstack-nova | 17:10 | |
dims_ | mriedem: equivalent of yada yada :) | 17:11 |
openstackgerrit | Joel Coffman proposed openstack/nova: libvirt: Add logging for dm-crypt error conditions https://review.openstack.org/198774 | 17:12 |
openstackgerrit | Joel Coffman proposed openstack/nova: libvirt: Replace stubs with mocks for test_dmcrypt https://review.openstack.org/198752 | 17:12 |
openstackgerrit | Joel Coffman proposed openstack/nova: libvirt: Disconnect dm-crypt on instance suspend/stop https://review.openstack.org/141485 | 17:12 |
openstackgerrit | Joel Coffman proposed openstack/nova: libvirt: Add unit tests for methods https://review.openstack.org/198880 | 17:12 |
*** afazekas has quit IRC | 17:12 | |
*** nic has quit IRC | 17:12 | |
*** nic has joined #openstack-nova | 17:13 | |
openstackgerrit | Roman Dobosz proposed openstack/nova: Introducing new forced_down field for a Service object https://review.openstack.org/185280 | 17:14 |
openstackgerrit | Roman Dobosz proposed openstack/nova: New nova API call to mark nova-compute down https://review.openstack.org/184086 | 17:14 |
openstackgerrit | Roman Dobosz proposed openstack/nova: Service group drivers forced_down flag utilization https://review.openstack.org/197063 | 17:14 |
mriedem | 'i went to see my ex, yada yada, now we're getting back together' | 17:14 |
mriedem | dims_: like that? ^ | 17:14 |
dims_ | :) | 17:15 |
*** whenry has joined #openstack-nova | 17:15 | |
*** lucas__ has joined #openstack-nova | 17:16 | |
*** annegentle has quit IRC | 17:16 | |
*** salv-orl_ has quit IRC | 17:20 | |
*** boris-42 has joined #openstack-nova | 17:20 | |
*** achanda has joined #openstack-nova | 17:20 | |
*** salv-orlando has joined #openstack-nova | 17:22 | |
*** yamahata has quit IRC | 17:22 | |
*** beagles is now known as beagles_lunch | 17:24 | |
*** afazekas has joined #openstack-nova | 17:24 | |
*** garyk has quit IRC | 17:26 | |
*** Marga_ has joined #openstack-nova | 17:27 | |
*** ssurana has joined #openstack-nova | 17:27 | |
openstackgerrit | John Garbutt proposed openstack/nova: Add a summary of the architecture evolution work https://review.openstack.org/200001 | 17:28 |
*** nihilifer has quit IRC | 17:29 | |
*** Marga_ has quit IRC | 17:30 | |
*** Marga_ has joined #openstack-nova | 17:30 | |
*** lucas__ has quit IRC | 17:32 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 17:33 |
*** vilobhmm has joined #openstack-nova | 17:33 | |
*** dims_ has quit IRC | 17:35 | |
*** dims has joined #openstack-nova | 17:37 | |
*** raildo has quit IRC | 17:37 | |
*** raildo has joined #openstack-nova | 17:37 | |
rlrossit | mriedem: dansmith: is forcing 3 merge conflicts on me per day your way of hazing the new guy? | 17:40 |
*** raildo has quit IRC | 17:40 | |
*** raildo has joined #openstack-nova | 17:41 | |
*** annegentle has joined #openstack-nova | 17:41 | |
mriedem | rlrossit: i'm done for awhile | 17:41 |
*** tpeoples has quit IRC | 17:42 | |
*** raildo has quit IRC | 17:42 | |
*** tpeoples has joined #openstack-nova | 17:42 | |
*** lucas__ has joined #openstack-nova | 17:44 | |
*** SourabhP has quit IRC | 17:47 | |
*** tpeoples has quit IRC | 17:47 | |
*** raildo_ has joined #openstack-nova | 17:49 | |
*** lucas__ has quit IRC | 17:51 | |
openstackgerrit | Ryan Rossiter proposed openstack/nova: Change List objects to use obj_relationships https://review.openstack.org/198730 | 17:53 |
*** bdr768 has joined #openstack-nova | 17:54 | |
*** lucas__ has joined #openstack-nova | 17:54 | |
*** salv-orlando has quit IRC | 17:55 | |
*** shyama has quit IRC | 17:56 | |
*** tpeoples has joined #openstack-nova | 17:57 | |
*** agireud has quit IRC | 17:59 | |
*** yamahata has joined #openstack-nova | 17:59 | |
*** annegentle has quit IRC | 18:00 | |
*** Mike_D_laptop has joined #openstack-nova | 18:01 | |
*** jdurgin has quit IRC | 18:02 | |
*** jgriffith has quit IRC | 18:03 | |
*** bdr768 is now known as jgriffith | 18:03 | |
*** markmcclain has quit IRC | 18:03 | |
*** davidlenwell has quit IRC | 18:03 | |
*** jgriffith is now known as Guest77747 | 18:04 | |
*** afazekas has quit IRC | 18:05 | |
*** davidlenwell has joined #openstack-nova | 18:05 | |
openstackgerrit | Josh Gachnang proposed openstack/nova: Build Neutron network data for metadata service https://review.openstack.org/102649 | 18:06 |
*** markmcclain has joined #openstack-nova | 18:06 | |
*** Guest77747 has quit IRC | 18:07 | |
*** bdr768 has joined #openstack-nova | 18:08 | |
*** kiran-r has quit IRC | 18:09 | |
*** dprince has quit IRC | 18:09 | |
*** bdr768 has quit IRC | 18:09 | |
*** dprince has joined #openstack-nova | 18:10 | |
*** agireud has joined #openstack-nova | 18:10 | |
*** dprince has quit IRC | 18:10 | |
*** dprince has joined #openstack-nova | 18:10 | |
*** jgriffit1 has joined #openstack-nova | 18:10 | |
*** SourabhP has joined #openstack-nova | 18:13 | |
*** jgriffit1 is now known as jdg | 18:13 | |
*** lpetrut1 has quit IRC | 18:17 | |
*** Sukhdev has quit IRC | 18:19 | |
*** annegentle has joined #openstack-nova | 18:19 | |
*** mutoulbj has quit IRC | 18:19 | |
*** annegentle has quit IRC | 18:20 | |
*** jdurgin has joined #openstack-nova | 18:21 | |
*** dims has quit IRC | 18:21 | |
*** abhishekk has quit IRC | 18:21 | |
*** jecarey has quit IRC | 18:23 | |
*** markvoelker has quit IRC | 18:24 | |
*** tpeoples has quit IRC | 18:24 | |
*** markvoelker has joined #openstack-nova | 18:24 | |
*** Marga_ has quit IRC | 18:24 | |
*** jdg is now known as jgriffith | 18:24 | |
*** afazekas has joined #openstack-nova | 18:25 | |
*** jgriffith is now known as Guest43643 | 18:25 | |
nagyz_ | I always wondered but haven't tried: if I have multiple commits and do a git review, will that open up a review for each? or how do people push in like 3-4 reviews at once? | 18:26 |
mriedem | melwitt: dansmith: i'm having a hard time remembering the api to restore a deleted instance | 18:26 |
*** Guest43643 is now known as jgriffith | 18:27 | |
dansmith | mriedem: you mean a soft_deleted one or a soft_deleted one? | 18:27 |
dansmith | there are two | 18:27 |
*** jaypipes has joined #openstack-nova | 18:27 | |
*** anoopjb has joined #openstack-nova | 18:27 | |
mriedem | ? | 18:27 |
dansmith | two types of soft_delete | 18:27 |
mriedem | restore a soft deleted instance | 18:27 |
mriedem | that's not archived in shadow tables | 18:27 |
mriedem | brb | 18:27 |
openstackgerrit | Jay Pipes proposed openstack/nova: Expose Neutron network data in metadata service https://review.openstack.org/153097 | 18:28 |
dansmith | so, if you set instance_reclaim to nonzero, | 18:28 |
dansmith | then deleting an instance doesn't delete it, it just sets vm_state=SOFT_DELETED | 18:29 |
dansmith | you can restore that kind and we've done no actual deleted=1 in the database | 18:29 |
dansmith | if you don't have that set and you do a delete, then we set deleted=1 in the database and you can't restore it without hacking the DB, AFAIK | 18:29 |
dansmith | another reason soft_deleted is BS | 18:29 |
jaypipes | JoshNang: hey, https://review.openstack.org/#/c/152703/ needs a rebase when you get a chance... | 18:29 |
anoopjb | server = nova.servers.create(name = "api_cirros_test", image = image.id, flavor = flavor.id ) how to specify network id if there is multiple networks? This is V1_1 novaclient of havana release | 18:30 |
jaypipes | dansmith, mriedem: wow. I'm away for >2 hours and you're still talking about soft delete :) | 18:30 |
jaypipes | anoopjb: please use the openstack-dev mailing list for usage questions. thanks! | 18:30 |
*** irenab has quit IRC | 18:31 | |
*** patrickeast has joined #openstack-nova | 18:31 | |
*** jdurgin1 has joined #openstack-nova | 18:31 | |
openstackgerrit | sajeesh cs proposed openstack/nova: Two roles ,namely project_admin and cloud_admin are created to control the quota operations. https://review.openstack.org/182522 | 18:32 |
melwitt | mriedem: what dansmith said. if you mean soft delete as in, normal delete that sets deleted=1, that's not restorable via any api I'm aware of | 18:33 |
melwitt | dansmith: yet another reason soft_deleted is BS is that instance_actions can't read deleted instances either, so you can't even get the benefit of seeing what user deleted an instance, for example | 18:34 |
dansmith | melwitt: +1 | 18:34 |
*** jdurgin has quit IRC | 18:34 | |
*** _gryf|afk is now known as _gryf | 18:35 | |
*** jecarey has joined #openstack-nova | 18:35 | |
openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota : cloud_admin and project_admin creation https://review.openstack.org/182522 | 18:35 |
*** beagles_lunch is now known as beagles | 18:37 | |
alaski | melwitt: that's why tasks were designed to not live under the instance resource. the reason instance_actions is that way is because it lives under the servers/<uuid>/ resource. though I would be fine with letting it be visible even after instance delete | 18:37 |
alaski | it just wouldn't be easily discoverable | 18:37 |
melwitt | alaski: I see | 18:37 |
*** anoopjb has quit IRC | 18:38 | |
melwitt | alaski: lately we've been having the situation of users wanting to know who deleted an instance and when, and I don't know a good way to do that | 18:38 |
*** lucas__ has quit IRC | 18:40 | |
JoshNang | jaypipes: kk i'll do that right now | 18:40 |
alaski | melwitt: if that use case involves them still having the uuid it's possible that we could find a way to expose the instance_action. but collecting notifications and finding a way to expose that feed works as well | 18:40 |
*** andrewbogott_afk is now known as andrewbogott | 18:41 | |
melwitt | alaski: yeah, that was my thought was collecting notifications is "the way" to do it. we don't yet :( but we're going to since I think that's the right way to gather info for historical purposes | 18:42 |
*** claudiub has joined #openstack-nova | 18:42 | |
*** afazekas has quit IRC | 18:43 | |
alaski | melwitt: as an event feed they're much better than instance_actions. but a lot depends on the consumer. for a public cloud a lot of work needs to go into making them visible to users, in a private cloud it might be okay to set up something like stacktach and let users search them | 18:47 |
*** lpetrut has joined #openstack-nova | 18:47 | |
mriedem | dansmith: ok, are you thinking that a purge command would check CONF.reclaim_instance_interval then? | 18:49 |
melwitt | alaski: that would be cool to do the latter. I'll check about it :) | 18:49 |
mriedem | dansmith: i was thinking it's just checking for instances.deleted != 0 | 18:49 |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Split out resize_image logic from create_image https://review.openstack.org/187857 | 18:49 |
dansmith | mriedem: no | 18:49 |
*** raildo_ has quit IRC | 18:50 | |
dansmith | mriedem: that's for the vm_states.SOFT_DELETED thing, which is entirely different | 18:50 |
mriedem | right | 18:50 |
mriedem | ok | 18:50 |
*** lpetrut has quit IRC | 18:50 | |
*** Sukhdev has joined #openstack-nova | 18:50 | |
*** lucas__ has joined #openstack-nova | 18:51 | |
*** ajayaa has quit IRC | 18:51 | |
*** otter768 has joined #openstack-nova | 18:54 | |
*** claudiub|2 has joined #openstack-nova | 18:54 | |
*** claudiub has quit IRC | 18:56 | |
*** lyanchih has quit IRC | 18:57 | |
*** otter768 has quit IRC | 18:59 | |
*** jcoufal has joined #openstack-nova | 19:00 | |
*** lpetrut has joined #openstack-nova | 19:00 | |
*** changbl has joined #openstack-nova | 19:00 | |
nagyz_ | mriedem, how do you want to proceed with the nova part of https://bugs.launchpad.net/cinder/+bug/1463525 ? | 19:01 |
openstack | Launchpad bug 1463525 in Cinder "There is no volume encryption metadata for rbd-backed volumes" [Undecided,Triaged] | 19:01 |
mriedem | nagyz_: i thought you were cooking up things for that? | 19:01 |
nagyz_ | I mean would actually adding the rbd encryption support would be the fix? | 19:01 |
nagyz_ | yes | 19:01 |
nagyz_ | if that's the fix. :-) | 19:01 |
*** thangp has quit IRC | 19:01 | |
nagyz_ | I've been playing around with it for a while now | 19:01 |
mriedem | nagyz_: i think people would appreciate encrypted volume support with ceph backends yes | 19:01 |
nagyz_ | right. I should have a review up early next week | 19:02 |
nagyz_ | people have been bugging me about it as well internally to publish it | 19:02 |
nagyz_ | just the bug's title is misleading then from this point of view - it should read "there is no volume encryption for rbd-backed volumes", right? | 19:02 |
nagyz_ | not just metadata :-) | 19:02 |
vilobhmm | alaski : ping | 19:03 |
alaski | vilobhmm: hi | 19:03 |
vilobhmm | i am running into an issue where if i try to create an instance snap locally (on the hypervisor itself and the hypervisor does not have enough available space) then the instance that was being snapshotted is brought into un-usable state | 19:04 |
*** coolsvap is now known as coolsvap|away | 19:04 | |
*** thangp has joined #openstack-nova | 19:04 | |
vilobhmm | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1375 | 19:05 |
*** Marga_ has joined #openstack-nova | 19:05 | |
vilobhmm | when the "power_state" is checked it should return "STOPPED" | 19:05 |
vilobhmm | SHUTDOWN | 19:05 |
vilobhmm | so should we have a check here beforehand uploading snap to glance that checks for if we have enough available space | 19:06 |
vilobhmm | alaski : ^^ | 19:06 |
andrewbogott | Is it possible for me to explicitly trigger a recheck by “DB Datasets CI”? It looks to me like ‘recheck’ only kicked Jenkins, not the others. | 19:06 |
*** pixelbeat has quit IRC | 19:07 | |
andrewbogott | oh, nevermind, it’s right here in the comment | 19:07 |
vilobhmm | or the VM/instance should not be brought into "SHUTDOWN" power_state before verifying if we have enough available space | 19:07 |
vilobhmm | alaski : ^^ | 19:07 |
alaski | vilobhmm: I'm not really familiar with the libvirt driver so there's likely a better person to ask. but if you can check the available space before doing anything that seems preferable | 19:07 |
mriedem | nagyz_: yeah you could reword the bug title as needed | 19:08 |
vilobhmm | alaski : makes sense | 19:08 |
mriedem | nagyz_: adding the support borders on a specless blueprint possibly, but i guess it depends on how involved the change is | 19:08 |
*** ociuhandu has quit IRC | 19:08 | |
vilobhmm | dansmith : ping…have some queries regarding libvirt driver | 19:09 |
sdague | so... we bollocks up the metadata server entirely? | 19:09 |
dansmith | vilobhmm: I'm heads down on something important, maybe someone else can help you | 19:09 |
sdague | I was trying to look into the 35 base line failures of the multinode jobs (now that the subnode booting issue got resolved in infra) | 19:10 |
*** dims has joined #openstack-nova | 19:10 | |
vilobhmm | dansmith : alrite | 19:10 |
sdague | and http://logs.openstack.org/91/199091/1/experimental/check-tempest-dsvm-multinode-full/c57e568/logs/screen-n-api.txt.gz?level=TRACE#_2015-07-09_12_05_32_854 it seems like we fail metadata server connects pretty consistently | 19:10 |
*** jgriffith has quit IRC | 19:10 | |
vilobhmm | mriedem : ^^ | 19:10 |
*** jgriffith has joined #openstack-nova | 19:11 | |
*** annashen has quit IRC | 19:11 | |
mriedem | kashyap: meet vilobhmm | 19:12 |
*** ajayaa has joined #openstack-nova | 19:12 | |
mriedem | sdague: i haven't seen any related changes https://github.com/openstack/nova/commits/master/nova/api/metadata | 19:12 |
vilobhmm | kashyap : ping …hope mriedem meant that you might be able to help so asking you i am running into an issue where if i try to create an instance snap locally (on the hypervisor itself and the hypervisor does not have enough available space) then the instance that was being snapshotted is brought into un-usable state | 19:13 |
vilobhmm | so should we have a check here beforehand uploading snap to glance that checks for if we have enough available space OR the VM/instance should not be brought into "SHUTDOWN" power_state before verifying if we have enough available space | 19:13 |
*** markvoelker has quit IRC | 19:15 | |
*** markvoelker has joined #openstack-nova | 19:15 | |
*** lucas__ has quit IRC | 19:15 | |
*** lucas__ has joined #openstack-nova | 19:16 | |
*** READ10 has quit IRC | 19:17 | |
sdague | mriedem: actually, it looks like it's a scheduler issue - http://logs.openstack.org/91/199091/1/experimental/check-tempest-dsvm-multinode-full/c57e568/logs/screen-n-sch.txt.gz#_2015-07-09_11_39_32_290 | 19:17 |
sdague | it's not finding any nodes | 19:18 |
*** vladikr has quit IRC | 19:18 | |
mriedem | sdague: yeah, there was some multinode failure thing that jogo was poking on a couple of weeks ago and he said it turned out to be some infra issue with the hostnames or something? they were the same so nova didn't know which to send it to? | 19:19 |
nagyz_ | mriedem, didn't you just agree the other day that we'll chalk it up as a bugfix instead of a bp? :) | 19:19 |
sdague | mriedem: that's resolved now | 19:20 |
*** mutoulbj has joined #openstack-nova | 19:20 | |
mriedem | nagyz_: i'm pretty sure my usual disclaimer applies | 19:21 |
nagyz_ | which is? :-) | 19:21 |
mriedem | which is, i don't recognize any previously made statements | 19:21 |
nagyz_ | hah! | 19:21 |
*** lpetrut has quit IRC | 19:22 | |
*** lpetrut has joined #openstack-nova | 19:24 | |
*** mutoulbj has quit IRC | 19:25 | |
*** lpetrut has quit IRC | 19:25 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: libvirt: Split out resize_image logic from create_image https://review.openstack.org/187857 | 19:26 |
openstackgerrit | Nicolas Simonds proposed openstack/nova: Fix/implement revert-resize for RBD-backed images https://review.openstack.org/187395 | 19:26 |
sdague | mriedem: oh... I think the resolution now means the hostnames are all "ubuntu". Which works fine in single node, not so much in multinode. | 19:27 |
mriedem | sdague: yeah | 19:27 |
sdague | I was reading the scheduler log, and trying to figure out why there was only ever 1 host | 19:28 |
*** radek__ has quit IRC | 19:28 | |
rlrossit | mriedem: dansmith: looks like check-grenade-dsvm-partial-ncpu failed on me again in https://review.openstack.org/#/c/198730/5. Kinda clueless as to where I should be looking in these logs to find where something went wrong, but I did find http://logs.openstack.org/30/198730/6/check/check-grenade-dsvm-partial-ncpu/0263871/logs/grenade.sh.txt.gz#_2015-07-09_18_45_39_814 | 19:29 |
sdague | also, since when did we start reporting every possible qemu arch in our systems? | 19:29 |
mriedem | sdague: link? | 19:29 |
sdague | [["alpha", "qemu", "hvm"], ["armv7l", "qemu", "hvm"], ["aarch64", "qemu", "hvm"], ["cris", "qemu", "hvm"], ["i686", "qemu", "hvm"], ["i686", "kvm", "hvm"], ["lm32", "qemu", "hvm"], ["m68k", "qemu", "hvm"], ["microblaze", "qemu", "hvm"], ["microblazeel", "qemu", "hvm"], ["mips", "qemu", "hvm"], ["mipsel", "qemu", "hvm"], ["mips64", "qemu", "hvm"], ["mips64el", "qemu", "hvm"], ["ppc", "qemu", "hvm"], ["ppc64", "qemu", "hvm"], ["ppc64le", "qemu", | 19:30 |
sdague | "hvm"], ["ppcemb", "qemu", "hvm"], ["s390x", "qemu", "hvm"], ["sh4", "qemu", "hvm"], ["sh4eb", "qemu", "hvm"], ["sparc", "qemu", "hvm"], ["sparc64", "qemu", "hvm"], ["unicore32", "qemu", "hvm"], ["x86_64", "qemu", "hvm"], ["x86_64", "kvm", "hvm"], ["xtensa", "qemu", "hvm"], ["xtensaeb", "qemu", "hvm"]] | 19:30 |
*** jecarey has quit IRC | 19:30 | |
sdague | that's the contents of my "supported_instances" column in my local env | 19:30 |
mriedem | rlrossit: http://logs.openstack.org/30/198730/6/check/check-grenade-dsvm-partial-ncpu/0263871/logs/old/screen-n-cpu.txt.gz?level=TRACE | 19:30 |
mriedem | sdague: that's probably from the the arch enum in the image_meta object now | 19:30 |
mriedem | i bet | 19:30 |
sdague | could be, it seems, kind of silly :) | 19:31 |
mriedem | sdague: open a bug | 19:31 |
mriedem | rlrossit: i don't actually see the same infinite recursion error as yesterday | 19:31 |
mriedem | but there is a gmr dump, which is bad | 19:31 |
rlrossit | mriedem: I noticed the same | 19:31 |
rlrossit | gmr? | 19:31 |
dansmith | rlrossit: I thought you were going to add an exception to make the explody bit real obvious | 19:32 |
mriedem | rlrossit: guru meditation report | 19:32 |
*** jecarey has joined #openstack-nova | 19:32 | |
rlrossit | dansmith: I was going to do that in a separate patch I thought | 19:32 |
sdague | mriedem: we are dumping a gmr on any failure now | 19:32 |
mriedem | rlrossit: http://docs.openstack.org/developer/nova/devref/gmr.html | 19:32 |
dansmith | rlrossit: the logging in a separate patch | 19:32 |
mriedem | sdague: ok, cool | 19:32 |
mriedem | rlrossit: i'm checking to see if there is something else more vanilla in the failed job | 19:32 |
dansmith | mriedem: given that it changed behavior, I'd say that's a pretty good indication that that line was the culprit | 19:32 |
sdague | so that's not an indication of direct badness, just an attempt to give first failure capture | 19:32 |
mriedem | dansmith: agreed | 19:32 |
mriedem | sdague: yeah, ack, plus, nice | 19:33 |
mriedem | :) | 19:33 |
dansmith | mriedem: if you explode there and say what object and version you're looking for then you can probably figure out which thing is missing an entry in their relationships | 19:33 |
mriedem | s/mriedem/rlrossit/ | 19:33 |
dansmith | yeah | 19:33 |
dansmith | sorry | 19:33 |
rlrossit | hrm this sounds like a good unit test... | 19:33 |
mriedem | rlrossit: http://logs.openstack.org/30/198730/6/check/check-grenade-dsvm-partial-ncpu/0263871/logs/grenade.sh.txt.gz#_2015-07-09_18_45_44_737 | 19:33 |
*** radek__ has joined #openstack-nova | 19:33 | |
mriedem | rlrossit: so, | 19:34 |
openstackgerrit | sajeesh cs proposed openstack/nova: Nested Quota Driver: Create column allocated in Quota table https://review.openstack.org/151327 | 19:34 |
mriedem | rlrossit: yes, we need a unit test since unit tests didn't catch that typo before on ps5 | 19:34 |
mriedem | rlrossit: and http://logs.openstack.org/30/198730/6/check/check-grenade-dsvm-partial-ncpu/0263871/logs/grenade.sh.txt.gz#_2015-07-09_18_45_44_737 is a known thing in the gate today for grenade jobs, so good news is, your issue is fixed | 19:34 |
mriedem | dansmith: ^ wootness | 19:34 |
mriedem | http://status.openstack.org/elastic-recheck/index.html#1473098 | 19:34 |
rlrossit | so this is one is for realz needing a recheck now? | 19:35 |
*** annashen has joined #openstack-nova | 19:35 | |
*** sambetts has quit IRC | 19:35 | |
mriedem | just rechecked it | 19:35 |
mriedem | rlrossit: so, add a unit test for the terrible horrible break in ps5 and then i'm +2 | 19:37 |
rlrossit | woot | 19:37 |
openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Spec: add nova-manage db purge-deleted-instances https://review.openstack.org/200224 | 19:38 |
*** sambetts has joined #openstack-nova | 19:38 | |
mriedem | also, booya ^ | 19:38 |
mriedem | jaypipes: melwitt: dansmith: ctrath: ^ | 19:38 |
*** Marga_ has quit IRC | 19:38 | |
*** Nikolay_St has joined #openstack-nova | 19:38 | |
dansmith | rlrossit: so you owe me two cupcakes now I think | 19:38 |
dansmith | if we're counting it's probably like 42, but.. | 19:39 |
*** Marga_ has joined #openstack-nova | 19:39 | |
dansmith | I expect an endless stream of cupcakes whilst in MN | 19:39 |
mriedem | ha | 19:39 |
mriedem | just get a piece of bunny cake here when you're in town http://restaurant.canadianhonker.com/ | 19:40 |
mriedem | it's right down the street from your hotel | 19:40 |
mriedem | i suggest breakfast there every morning | 19:40 |
dansmith | not sure I can go to a place called "canadian honker" | 19:40 |
*** dave-mccowan has quit IRC | 19:40 | |
mriedem | you have to | 19:41 |
rlrossit | better start liking geese if you're coming to Rochester dansmith | 19:41 |
mriedem | yeah, we have an infatuation with geese | 19:41 |
rlrossit | they aren't as bad this year | 19:41 |
dansmith | unless your cupcakes are iced with goose poo, I think I'll keep my current feelings for geese where they are | 19:41 |
mriedem | don't forget that they hiss at you and don't move out of the way when you're walking | 19:41 |
rlrossit | try biking... I've almost crashed on multiple occasions | 19:42 |
mriedem | plus, how would goose shit-frosted cupcakes change your opinion on geese? | 19:42 |
dansmith | not sure. sounded funnier in my head | 19:42 |
rlrossit | dansmith: here's how your trip to rochester will be except s/donuts/cupcakes https://youtu.be/X3ZcZ2h4Ths?t=1m50s | 19:42 |
sdague | mriedem: i just thought you folks put marshmallow on everything | 19:43 |
mtreinish | mriedem, dansmith: http://www.nytimes.com/2015/06/15/world/europe/geese-exterminator-of-the-netherlands-enrages-animal-rights-activists.html | 19:43 |
mriedem | sdague: no, that's casserole in south carolina | 19:43 |
mriedem | russellb: ^ | 19:43 |
*** jichen has joined #openstack-nova | 19:43 | |
mtreinish | do you guys need someone like that in rochester? | 19:43 |
*** zhenguo has quit IRC | 19:43 | |
*** lucas__ has quit IRC | 19:43 | |
mriedem | no | 19:43 |
dansmith | lots of marshmallow-augmented dishes in the more superior carolina too | 19:43 |
rlrossit | mtreinish: oh my goodness the picture of the goose running towards the camera with its mouth open is making me laugh uncontrollably | 19:45 |
melwitt | what kind of casserole? like mashed sweet potato with marshmallow on top? | 19:45 |
mriedem | melwitt: that's a thanksgiving thing most places | 19:45 |
mriedem | melwitt: i believe the other was something actually like a dinner thing, but with mallow on top | 19:45 |
melwitt | mriedem: oh, wow | 19:46 |
rlrossit | except in minnesota you need to substitute the word "casserole" with "hotdish" | 19:46 |
mriedem | right | 19:46 |
mriedem | i was omitting that | 19:46 |
dansmith | melwitt: there was a carrot salad my mamma always made me with marshmallows in it | 19:46 |
mriedem | 'here dan, we know you hate vegetables, so we threw some marshmallows on it, now eat it and shaddup' | 19:47 |
melwitt | lol | 19:47 |
rlrossit | my parents made squash with marshmallows and butter on top. everything except the squash tasted great | 19:47 |
dansmith | (except I love vegetables) | 19:47 |
mriedem | really? | 19:47 |
dansmith | yeah | 19:47 |
dansmith | no scary ones | 19:47 |
dansmith | but normal ones | 19:47 |
mriedem | it's like a paradox | 19:47 |
* melwitt makes notes | 19:47 | |
mriedem | define a scary vegetable? | 19:47 |
dansmith | white ones | 19:47 |
dansmith | pink ones | 19:47 |
*** achanda has quit IRC | 19:48 | |
mriedem | we can all agree that cauliflower is disgusting and only belongs in a teenage wrestler's ear | 19:48 |
mriedem | next | 19:48 |
*** Mike_D_laptop has quit IRC | 19:48 | |
rlrossit | I can't think of a pink vegetable | 19:48 |
openstackgerrit | Dean Daskalantonakis proposed openstack/nova: Move existing V2 to legacy-v2 https://review.openstack.org/193725 | 19:48 |
melwitt | radish? watermelon | 19:48 |
mriedem | radish | 19:48 |
melwitt | oh wait, that's a fruit | 19:48 |
rlrossit | radish is red in my opinion... | 19:48 |
mriedem | beats | 19:49 |
mriedem | so anything that can live through the russian winter | 19:49 |
rlrossit | beats still exist? | 19:49 |
dansmith | mriedem: right | 19:49 |
dansmith | well, for pink I was thinking eggplant flesh | 19:49 |
*** lucas___ has joined #openstack-nova | 19:49 | |
mriedem | eggplant | 19:49 |
rlrossit | bleck | 19:49 |
mriedem | there are special white eggplants | 19:49 |
mriedem | but that falls in the white category | 19:49 |
*** Longgeek has quit IRC | 19:49 | |
dansmith | yep | 19:49 |
mriedem | i can see you've put a lot of thought into this | 19:50 |
mriedem | kudos | 19:50 |
rlrossit | so wait are potatoes white or brown? | 19:50 |
rlrossit | or pink? | 19:50 |
rlrossit | they come in like every color of the rainbow | 19:50 |
dansmith | rlrossit: they're yummy | 19:50 |
*** lucas___ has quit IRC | 19:50 | |
dansmith | rlrossit: also, damn you for finding a flaw in my otherwise perfect system | 19:50 |
mriedem | potatoes are the ACLU of the vegetable world | 19:50 |
beagles | beets | 19:50 |
beagles | not beats | 19:50 |
rlrossit | potatoes, sweet corn, carrots, and lettuce are like the only veggies I can eat | 19:50 |
*** annashen has quit IRC | 19:51 | |
dansmith | rlrossit: that's a good assortment, yeah | 19:51 |
dansmith | I like cucumbers | 19:51 |
dansmith | and peppers | 19:51 |
*** Nikolay_St has quit IRC | 19:51 | |
dansmith | and tomatoes | 19:51 |
rlrossit | beagles: I thought we were eating fancy headphones | 19:51 |
dansmith | rlrossit: "fancy" | 19:51 |
beagles | :) | 19:51 |
sdague | now I get to scare dansmith with vegetables - https://www.flickr.com/photos/7545231@N03/5815763752/ | 19:51 |
*** lucas___ has joined #openstack-nova | 19:51 | |
melwitt | expensive headphones | 19:51 |
dansmith | rlrossit: you just said that to the wrong couple of guys | 19:51 |
rlrossit | let's start an argument about whether tomatoes are a fruit or a vegetable | 19:51 |
*** annashen has joined #openstack-nova | 19:51 | |
*** VW_ has quit IRC | 19:51 | |
dansmith | sdague: gross | 19:51 |
mriedem | sdague: another one bites the dust https://www.flickr.com/photos/7545231@N03/5813900340/in/photostream/ | 19:51 |
rlrossit | oh I hate them | 19:51 |
rlrossit | (them being beats) | 19:52 |
rlrossit | to me fancy == expensive | 19:52 |
dansmith | massively overpriced | 19:52 |
*** VW_ has joined #openstack-nova | 19:52 | |
dansmith | completely-terrible-but-still-overly-expensive | 19:52 |
mriedem | i raise a pinky finger while drinking if i have dr dre's headphones on | 19:52 |
rlrossit | while you're drinking your fancy box wine? | 19:53 |
mriedem | franzia | 19:53 |
mriedem | nothing but the best | 19:53 |
rlrossit | mriedem: this reminds me of the conversation we had earlier about you being surprised I showed up today by 8... | 19:54 |
rlrossit | during the summer I find a way to consistently run on 5 hours of sleep a night so I can stay up as late as I want | 19:54 |
*** VW__ has joined #openstack-nova | 19:55 | |
*** VW_ has quit IRC | 19:56 | |
*** annashen has quit IRC | 19:56 | |
ctrath | rlrossit: do you hibernate all winter? | 19:57 |
rlrossit | something like that ctrath | 19:58 |
mriedem | dansmith: looks like you missed one in https://review.openstack.org/#/c/196912/ | 19:58 |
*** ssurana has quit IRC | 19:59 | |
*** lucas___ has quit IRC | 20:00 | |
*** irenab has joined #openstack-nova | 20:01 | |
*** irenab has quit IRC | 20:01 | |
rlrossit | mriedem: how do I access past patch set's logs to see what was getting hosed with that infinite recursion? | 20:01 |
*** ssurana has joined #openstack-nova | 20:02 | |
mriedem | rlrossit: toggle ci button | 20:02 |
mriedem | then go to jenkins results from ps5 | 20:02 |
mriedem | i have a comment in there with the link to the failure in ps5 | 20:02 |
*** tjones1 has quit IRC | 20:02 | |
rlrossit | wow that button really knows how to hide | 20:03 |
openstackgerrit | Marian Horban proposed openstack/nova: Switch to oslo.service library https://review.openstack.org/192900 | 20:03 |
melwitt | I'm not sure what I have set up but past CI always shows for me | 20:03 |
*** annegentle has joined #openstack-nova | 20:04 | |
*** andrewbogott is now known as andrewbogott_afk | 20:05 | |
russellb | mriedem: mmm, marshmallows | 20:06 |
*** dave-mccowan has joined #openstack-nova | 20:08 | |
*** achanda has joined #openstack-nova | 20:15 | |
*** packet has joined #openstack-nova | 20:16 | |
rlrossit | dansmith: found out why it's blowing up! | 20:17 |
rlrossit | dansmith: *someone* decided to start InstanceList's versions at 1.1 https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L1174 | 20:18 |
*** salv-orlando has joined #openstack-nova | 20:19 | |
rlrossit | wait a minute InstanceList was actually doing something before it was version 1.1, but it didn't get child_versions until 1.1 | 20:21 |
*** Nikolay_St has joined #openstack-nova | 20:23 | |
*** _gryf is now known as _gryf|afk | 20:23 | |
*** ijuwang has quit IRC | 20:24 | |
*** ijuwang has joined #openstack-nova | 20:24 | |
*** jwcroppe has quit IRC | 20:25 | |
*** lpetrut has joined #openstack-nova | 20:25 | |
*** jcoufal has quit IRC | 20:26 | |
*** jecarey has quit IRC | 20:26 | |
*** ssurana has quit IRC | 20:27 | |
*** jichen has quit IRC | 20:29 | |
openstackgerrit | Marian Horban proposed openstack/nova: Switch to oslo.service library https://review.openstack.org/192900 | 20:31 |
*** annashen has joined #openstack-nova | 20:31 | |
*** tonytan4ever has quit IRC | 20:31 | |
*** jwcroppe has joined #openstack-nova | 20:31 | |
*** lpetrut has quit IRC | 20:34 | |
*** tonytan4ever has joined #openstack-nova | 20:35 | |
*** achanda has quit IRC | 20:36 | |
*** karimb has joined #openstack-nova | 20:38 | |
*** r-daneel has joined #openstack-nova | 20:39 | |
*** Nikolay_St has quit IRC | 20:39 | |
*** karimb has quit IRC | 20:39 | |
*** karimb has joined #openstack-nova | 20:39 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Switch to oslo.service library https://review.openstack.org/192900 | 20:40 |
*** bustersw_ has joined #openstack-nova | 20:47 | |
*** bustersw_ has quit IRC | 20:48 | |
*** achanda has joined #openstack-nova | 20:49 | |
*** r-daneel has quit IRC | 20:49 | |
*** tjones1 has joined #openstack-nova | 20:49 | |
*** Sukhdev has quit IRC | 20:50 | |
*** Longgeek has joined #openstack-nova | 20:50 | |
*** xgsa has joined #openstack-nova | 20:51 | |
*** busterswt has quit IRC | 20:52 | |
*** jaypipes has quit IRC | 20:53 | |
*** dprince has quit IRC | 20:54 | |
*** tjones1 has quit IRC | 20:54 | |
*** lucas__ has joined #openstack-nova | 20:55 | |
*** otter768 has joined #openstack-nova | 20:55 | |
*** Longgeek has quit IRC | 20:55 | |
*** annegentle has quit IRC | 20:56 | |
*** xgsa1 has joined #openstack-nova | 20:57 | |
*** ssurana has joined #openstack-nova | 20:58 | |
*** xgsa has quit IRC | 20:58 | |
openstackgerrit | Dean Daskalantonakis proposed openstack/nova: Move existing V2 to legacy-v2 https://review.openstack.org/193725 | 20:59 |
*** tjones1 has joined #openstack-nova | 20:59 | |
*** otter768 has quit IRC | 21:00 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Switch to oslo.service library https://review.openstack.org/192900 | 21:01 |
*** patrickeast_ has joined #openstack-nova | 21:01 | |
*** rlrossit has left #openstack-nova | 21:01 | |
*** packet has quit IRC | 21:01 | |
*** lucas__ has quit IRC | 21:03 | |
*** patrickeast has quit IRC | 21:03 | |
*** patrickeast_ is now known as patrickeast | 21:03 | |
*** ajayaa has quit IRC | 21:05 | |
*** andrewbogott_afk is now known as andrewbogott | 21:05 | |
*** dims has quit IRC | 21:05 | |
*** tjones1 has quit IRC | 21:05 | |
*** tpeoples has joined #openstack-nova | 21:06 | |
*** tjones1 has joined #openstack-nova | 21:07 | |
*** dims has joined #openstack-nova | 21:07 | |
*** davideagnello has quit IRC | 21:08 | |
*** davideagnello has joined #openstack-nova | 21:09 | |
*** lucas__ has joined #openstack-nova | 21:10 | |
*** thorst has quit IRC | 21:10 | |
*** lucas___ has joined #openstack-nova | 21:11 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add libvirt-python to test-requirements.txt to enforce min version https://review.openstack.org/183706 | 21:12 |
*** lucas__ has quit IRC | 21:14 | |
*** whenry has quit IRC | 21:14 | |
*** exploreshaifali has quit IRC | 21:15 | |
*** lucas___ has quit IRC | 21:15 | |
*** apuimedo has quit IRC | 21:18 | |
*** lucas__ has joined #openstack-nova | 21:18 | |
*** jwcroppe has quit IRC | 21:18 | |
*** cbader has quit IRC | 21:18 | |
*** jwcroppe_ has joined #openstack-nova | 21:18 | |
*** jwcroppe has joined #openstack-nova | 21:21 | |
*** lucas__ has quit IRC | 21:22 | |
*** jwcroppe_ has quit IRC | 21:24 | |
*** annashen has quit IRC | 21:26 | |
*** annashen has joined #openstack-nova | 21:27 | |
*** browne has quit IRC | 21:29 | |
mgagne | sorrison: Let me know if you have questions regarding my comment on https://review.openstack.org/#/c/184155/ | 21:29 |
openstackgerrit | Matt Riedemann proposed openstack/nova: Handle KeyError when volume encryption is not supported https://review.openstack.org/193830 | 21:29 |
mgagne | sorrison: I'll try to partially forward port my patch to kilo and see how it goes | 21:30 |
*** lucas__ has joined #openstack-nova | 21:30 | |
openstackgerrit | Josh Gachnang proposed openstack/nova: Add support for provider networks in network_data.json https://review.openstack.org/152703 | 21:31 |
*** lpetrut has joined #openstack-nova | 21:34 | |
*** VW_ has joined #openstack-nova | 21:35 | |
*** VW_ has quit IRC | 21:35 | |
*** lucas__ has quit IRC | 21:35 | |
*** tonytan4ever has quit IRC | 21:35 | |
*** VW_ has joined #openstack-nova | 21:35 | |
*** slagle has quit IRC | 21:36 | |
andrewbogott | mriedem: ‘more details’ like… what is metadata? What is a project_id? Seems very straightforward to me. | 21:36 |
mriedem | andrewbogott: like why are you adding the thing | 21:36 |
mriedem | basically a summary of what you have in the blueprint | 21:36 |
*** VW__ has quit IRC | 21:37 | |
mriedem | git log means i don't have to check external links for information about a commit | 21:37 |
*** jamielennox is now known as jamielennox|away | 21:37 | |
openstackgerrit | Andrew Bogott proposed openstack/nova: Include project_id in instance metadata. https://review.openstack.org/197185 | 21:38 |
mriedem | andrewbogott: better, maybe a typo in there | 21:39 |
andrewbogott | ah, so I see | 21:40 |
*** haypo has left #openstack-nova | 21:40 | |
*** radek__ has quit IRC | 21:40 | |
*** agireud has quit IRC | 21:41 | |
*** jwcroppe_ has joined #openstack-nova | 21:41 | |
*** jecarey has joined #openstack-nova | 21:41 | |
*** jwcroppe has quit IRC | 21:42 | |
openstackgerrit | Andrew Bogott proposed openstack/nova: Include project_id in instance metadata. https://review.openstack.org/197185 | 21:43 |
*** dboik has quit IRC | 21:43 | |
mriedem | +2 | 21:43 |
*** liusheng has quit IRC | 21:44 | |
openstackgerrit | Victor Stinner proposed openstack/nova: Fix Python 3 issues in nova.utils and nova.tests https://review.openstack.org/197982 | 21:45 |
openstackgerrit | Victor Stinner proposed openstack/nova: Port crypto to Python 3 https://review.openstack.org/197748 | 21:45 |
andrewbogott | bah, hang on, one more change... | 21:45 |
*** liusheng has joined #openstack-nova | 21:45 | |
*** dave-mcc_ has joined #openstack-nova | 21:45 | |
*** mgoddard has quit IRC | 21:45 | |
openstackgerrit | Andrew Bogott proposed openstack/nova: Include project_id in instance metadata. https://review.openstack.org/197185 | 21:45 |
*** jecarey has quit IRC | 21:46 | |
*** jwcroppe has joined #openstack-nova | 21:46 | |
andrewbogott | ok, /now/ I think it’s right. thanks mriedem | 21:47 |
mriedem | yup, +2 again | 21:47 |
*** jwcroppe_ has quit IRC | 21:47 | |
*** jamielennox|away is now known as jamielennox | 21:48 | |
*** chaochin has quit IRC | 21:48 | |
*** xgsa1 has quit IRC | 21:49 | |
*** dave-mccowan has quit IRC | 21:49 | |
*** xgsa has joined #openstack-nova | 21:49 | |
*** mgoddard has joined #openstack-nova | 21:49 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: objects: add missing enum values to DiskBus field https://review.openstack.org/200280 | 21:49 |
*** Sukhdev has joined #openstack-nova | 21:51 | |
mriedem | dansmith: so i'm not actually sure how to handle a compat shim in here with the enum stuff ^ | 21:51 |
*** Longgeek has joined #openstack-nova | 21:51 | |
*** chaochin has joined #openstack-nova | 21:52 | |
*** nelsnelson has joined #openstack-nova | 21:54 | |
*** dboik has joined #openstack-nova | 21:56 | |
*** masayukig has quit IRC | 21:56 | |
*** thangp has quit IRC | 21:56 | |
mriedem | anything less than version 1.2 would explode if trying to create DiskBus fields with lxc or uml | 21:57 |
*** dboik has quit IRC | 21:57 | |
*** masayukig_ has quit IRC | 21:57 | |
*** masayukig has joined #openstack-nova | 21:57 | |
*** VW_ has quit IRC | 21:58 | |
*** VW_ has joined #openstack-nova | 21:58 | |
*** VW__ has joined #openstack-nova | 21:59 | |
*** amotoki has quit IRC | 22:02 | |
*** changbl has quit IRC | 22:02 | |
*** dims has quit IRC | 22:03 | |
*** VW_ has quit IRC | 22:03 | |
*** dims has joined #openstack-nova | 22:03 | |
*** oomichi has joined #openstack-nova | 22:05 | |
*** coolsvap|away is now known as coolsvap | 22:05 | |
*** jgriffith is now known as Guest85831 | 22:06 | |
*** nic has quit IRC | 22:10 | |
*** Longgeek has quit IRC | 22:10 | |
*** mriedem is now known as mriedem_away | 22:10 | |
*** mdrabe has quit IRC | 22:12 | |
*** jgriffith has joined #openstack-nova | 22:14 | |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 2 https://review.openstack.org/167408 | 22:16 |
*** sahid has quit IRC | 22:16 | |
*** edmondsw has quit IRC | 22:18 | |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Adds support for x509 certificates as keypairs https://review.openstack.org/136458 | 22:19 |
*** Marga_ has quit IRC | 22:19 | |
*** pixelbeat has joined #openstack-nova | 22:19 | |
*** lucas-beer has quit IRC | 22:20 | |
cfriesen | anyone on here familiar with PCI and NUMA in nova? Based on nova.tests.unit.compute.test_claims.ClaimTestCase.test_numa_topology_with_pci_no_numa_info it's okay for a PCI device to have no NUMA information, and I'm wondering when that would actually happen in real life. | 22:20 |
*** jwcroppe_ has joined #openstack-nova | 22:22 | |
*** lpetrut has quit IRC | 22:24 | |
*** jwcroppe has quit IRC | 22:24 | |
*** jasondotstar has quit IRC | 22:25 | |
*** browne has joined #openstack-nova | 22:25 | |
*** sorrison_laptop has quit IRC | 22:25 | |
*** ivasev has quit IRC | 22:32 | |
*** jwcroppe has joined #openstack-nova | 22:36 | |
*** eharney has quit IRC | 22:37 | |
*** jwcroppe_ has quit IRC | 22:38 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Switch to oslo.service library https://review.openstack.org/192900 | 22:40 |
* alex_xu begin non priority feature review day~ | 22:41 | |
*** jasondotstar has joined #openstack-nova | 22:43 | |
*** fawadkhaliq has quit IRC | 22:43 | |
*** annashen has quit IRC | 22:43 | |
*** annashen has joined #openstack-nova | 22:44 | |
*** tpeoples has quit IRC | 22:45 | |
openstackgerrit | Tracy Jones proposed openstack/nova: Cleanup of compute api reboot method https://review.openstack.org/127427 | 22:48 |
*** andreykurilin__ has joined #openstack-nova | 22:48 | |
*** annashen has quit IRC | 22:49 | |
*** annashen has joined #openstack-nova | 22:51 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 22:52 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Switch to oslo.reports https://review.openstack.org/195558 | 22:54 |
*** otter768 has joined #openstack-nova | 22:56 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Switch to oslo.reports https://review.openstack.org/195558 | 22:57 |
*** apuimedo has joined #openstack-nova | 22:57 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Switch to oslo.reports https://review.openstack.org/195558 | 22:57 |
*** diogogmt has quit IRC | 22:58 | |
openstackgerrit | Tracy Jones proposed openstack/nova: Cleanup of compute api reboot method https://review.openstack.org/127427 | 22:58 |
*** otter768 has quit IRC | 23:01 | |
*** amotoki has joined #openstack-nova | 23:02 | |
*** tongli has quit IRC | 23:04 | |
*** ctrath has quit IRC | 23:05 | |
*** amotoki has quit IRC | 23:07 | |
*** hemna is now known as hemnafk | 23:11 | |
*** mtanino_ has joined #openstack-nova | 23:11 | |
*** mtanino has quit IRC | 23:11 | |
*** coolsvap is now known as coolsvap|afk | 23:14 | |
*** tpatil has quit IRC | 23:17 | |
*** salv-orlando has quit IRC | 23:18 | |
claudiub|2 | mikal: Hello. If you could take a look at the hyper-v uefi secure boot, that would be great. It already has a +2. https://review.openstack.org/#/c/190997/ | 23:21 |
*** zhenguo has joined #openstack-nova | 23:21 | |
*** yjiang5 has joined #openstack-nova | 23:23 | |
*** mutoulbj has joined #openstack-nova | 23:24 | |
*** mutoulbj has quit IRC | 23:28 | |
*** tongli has joined #openstack-nova | 23:29 | |
*** tpeoples has joined #openstack-nova | 23:34 | |
*** tongli has quit IRC | 23:34 | |
*** annashen has quit IRC | 23:34 | |
*** tongli has joined #openstack-nova | 23:34 | |
*** mtanino_ has quit IRC | 23:36 | |
*** patrickeast has quit IRC | 23:37 | |
*** mtanino has joined #openstack-nova | 23:37 | |
*** mgarza_ has quit IRC | 23:37 | |
*** Longgeek has joined #openstack-nova | 23:38 | |
*** openstack has joined #openstack-nova | 23:39 | |
*** VW__ has quit IRC | 23:41 | |
*** nelsnelson has quit IRC | 23:42 | |
*** VW_ has quit IRC | 23:43 | |
openstackgerrit | Tracy Jones proposed openstack/nova: WIP - convert direct db calls to objects https://review.openstack.org/194851 | 23:44 |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Raise exception when API does't support microversions https://review.openstack.org/188816 | 23:49 |
openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: discover supported versions to choose latest one https://review.openstack.org/184281 | 23:49 |
*** achanda has quit IRC | 23:49 | |
*** Sukhdev has quit IRC | 23:51 | |
*** Longgeek has quit IRC | 23:52 | |
*** Sukhdev has joined #openstack-nova | 23:52 | |
*** IanGovett has quit IRC | 23:53 | |
*** tongli has quit IRC | 23:53 | |
dansmith | cburgess: around by chance? | 23:53 |
*** tongli has joined #openstack-nova | 23:54 | |
*** tongli has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!