Thursday, 2015-07-09

*** browne has joined #openstack-nova00:02
*** SourabhP has quit IRC00:04
*** tongli has quit IRC00:05
*** tongli has joined #openstack-nova00:06
*** tongli has quit IRC00:10
*** jecarey has quit IRC00:12
*** Piet has joined #openstack-nova00:14
*** irenab has joined #openstack-nova00:17
*** ssurana has joined #openstack-nova00:17
*** thorst has quit IRC00:18
*** yamahata has joined #openstack-nova00:18
*** irenab has quit IRC00:21
*** dims has quit IRC00:21
*** tongli has joined #openstack-nova00:22
*** yingjun has joined #openstack-nova00: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|away00:29
*** diogogmt has joined #openstack-nova00:29
mriedemijw_: link?00:29
mriedemandrewbogott: 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 blueprints00:30
andrewbogottmriedem: I’m planning to work on that tonight, I’ll ping you when I have something.00:30
*** ijw_ has quit IRC00:33
*** jwcroppe has quit IRC00:35
mriedemandrewbogott: ok, i added the review to the agenda anyway in case we have time00:36
andrewbogottthx00:36
*** jamielennox|away is now known as jamielennox00:39
andrewbogottmriedem: Here’s the outline:  https://blueprints.launchpad.net/nova/+spec/instance-id-in-metadata00:40
andrewbogottShall I explain use cases on the whiteboard or is it better to make a wiki page?00:40
*** r-daneel has quit IRC00:40
*** thorst has joined #openstack-nova00:43
andrewbogottjohnthetubaguy: same question?00:45
*** otter768 has joined #openstack-nova00:46
*** Jeffrey4l has quit IRC00:49
*** Jeffrey4l has joined #openstack-nova00:50
*** tsekiyam_ has joined #openstack-nova00:50
mriedemandrewbogott: just put it in the description or whiteboard00:50
andrewbogott‘k00:51
*** otter768 has quit IRC00:51
*** Piet has quit IRC00:51
*** mtanino has quit IRC00:53
*** tsekiyama has quit IRC00:54
*** tsekiyam_ has quit IRC00:55
*** Piet has joined #openstack-nova00:57
*** rdo has quit IRC00:58
mriedemjohnthetubaguy: this is complete https://blueprints.launchpad.net/nova/+spec/allow-ip6-search-for-non-admin - the changes are all merged00:58
mriedemjust need to update the bp00:58
*** piyanai has joined #openstack-nova01:00
andrewbogottmriedem: 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-nova01:00
*** thorst has quit IRC01:00
openstackgerritAndrew Bogott proposed openstack/nova: Include project_id in instance metadata.  https://review.openstack.org/19718501:01
mriedemandrewbogott: don't think so01:02
andrewbogottgreat :)01:02
mriedemit's a straight forward change, the question is process for us really01:02
mriedemandrewbogott: remember to link your code chagne into the bp with the blueprint ref link in your commit message :)01:03
mriedemso the change is listed in the whiteboard of the bp01:03
andrewbogottah, right01:03
*** asettle has quit IRC01:04
openstackgerritAndrew Bogott proposed openstack/nova: Include project_id in instance metadata.  https://review.openstack.org/19718501:04
*** rdo has joined #openstack-nova01:05
*** jwcroppe has joined #openstack-nova01:06
*** eglynn has joined #openstack-nova01:07
*** jgriffit1 has quit IRC01:10
*** amotoki has joined #openstack-nova01:10
*** jgriffit1 has joined #openstack-nova01:10
*** ssurana has quit IRC01:11
*** oomichi has joined #openstack-nova01:11
*** LukeHinds has quit IRC01:12
*** Jeffrey4l has quit IRC01:14
*** pixelbeat has joined #openstack-nova01:14
*** jwcroppe has quit IRC01:17
*** yingjun has quit IRC01:19
*** yingjun has joined #openstack-nova01:19
*** armax has quit IRC01:21
*** arnaud___ has joined #openstack-nova01:21
*** jgriffit1 has quit IRC01:26
*** arnaud___ has quit IRC01:26
*** bdr7 has joined #openstack-nova01:27
*** tjones1 has quit IRC01:27
*** shuquan has joined #openstack-nova01:27
*** tpeoples has joined #openstack-nova01:28
*** shuquan_ has joined #openstack-nova01:31
*** asettle has joined #openstack-nova01:32
*** shuquan has quit IRC01:32
*** shuquan_ is now known as shuquan01:32
*** lyanchih_ has joined #openstack-nova01:34
*** jasondotstar has joined #openstack-nova01:35
*** mriedem has quit IRC01:42
*** bitblt has quit IRC01:45
*** vilobhmm has quit IRC01:45
*** oomichi has quit IRC01:46
*** Sukhdev has quit IRC01:48
*** tongli has quit IRC01:48
anishcan 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 while01:55
*** unicell1 has quit IRC02:00
*** baoli has joined #openstack-nova02:02
*** baoli_ has joined #openstack-nova02:04
*** davideagnello has quit IRC02:05
*** baoli has quit IRC02:07
anishlyanchih_: thanks :)02:08
*** shuquan has quit IRC02:08
*** yamahata has quit IRC02:10
*** Sukhdev has joined #openstack-nova02:11
*** mutoulbj has joined #openstack-nova02:17
*** shuquan has joined #openstack-nova02:17
*** vilobhmm has joined #openstack-nova02:19
*** Sukhdev has quit IRC02:20
*** achanda has quit IRC02:21
*** achanda has joined #openstack-nova02:21
*** achanda has quit IRC02:22
*** arnaud___ has joined #openstack-nova02:22
*** annashen has joined #openstack-nova02:26
*** arnaud___ has quit IRC02:26
*** gyee has quit IRC02:28
*** Jeffrey4l has joined #openstack-nova02:29
*** mutoulbj_ has joined #openstack-nova02:33
*** vilobhmm has quit IRC02:33
*** vilobhmm has joined #openstack-nova02:33
*** r-daneel has joined #openstack-nova02:35
*** lyanchih_ has quit IRC02:36
*** mutoulbj has quit IRC02:36
*** annashen has quit IRC02:42
*** annashen has joined #openstack-nova02:42
*** annashen has quit IRC02:43
*** otter768 has joined #openstack-nova02:47
*** patrickeast has quit IRC02:49
*** otter768 has quit IRC02:52
*** hakimo has joined #openstack-nova02:52
*** hakimo_ has quit IRC02:54
*** ssurana has joined #openstack-nova02:54
*** r-daneel has quit IRC02:55
openstackgerritJeffrey Zhang proposed openstack/nova: use graduated oslo.policy  https://review.openstack.org/19806502:56
*** lyanchih has joined #openstack-nova02:57
*** VW_ has joined #openstack-nova02:57
openstackgerritAndrew Bogott proposed openstack/nova: Include project_id in instance metadata.  https://review.openstack.org/19718502:59
*** shyu has joined #openstack-nova03:02
*** humble_ has joined #openstack-nova03:02
*** VW_ has quit IRC03:04
*** VW_ has joined #openstack-nova03:05
*** wanghua has quit IRC03:06
*** bdr7 has quit IRC03:06
*** bdr7 has joined #openstack-nova03:07
*** xyang1 has quit IRC03:09
*** VW_ has quit IRC03:10
*** andrewbogott is now known as andrewbogott_afk03:14
*** armax has joined #openstack-nova03:16
*** jdandrea has quit IRC03:16
*** annashen has joined #openstack-nova03:17
*** davideagnello has joined #openstack-nova03:19
*** jamielennox is now known as jamielennox|away03:22
*** achanda has joined #openstack-nova03:26
*** bdr7 has quit IRC03:30
*** jamielennox|away is now known as jamielennox03:31
*** dave-mccowan has quit IRC03:33
*** piyanai has quit IRC03:34
*** garyk has quit IRC03:34
*** achanda has quit IRC03:34
*** piyanai has joined #openstack-nova03:34
*** Kennan2 has joined #openstack-nova03:35
*** Kennan has quit IRC03:36
*** shuquan has quit IRC03:37
*** amotoki has quit IRC03:37
*** shuquan has joined #openstack-nova03:37
*** jasondotstar has quit IRC03:37
*** diogogmt has quit IRC03:40
*** shuquan has quit IRC03:44
*** dims_ has joined #openstack-nova03:46
*** dims_ has quit IRC03:46
*** dims_ has joined #openstack-nova03:46
*** piyanai has quit IRC03:47
*** Kevin_Zheng_ has joined #openstack-nova03:48
*** yingjun has quit IRC03:51
*** kiran-r has joined #openstack-nova03:51
*** arnaud___ has joined #openstack-nova03:51
*** amotoki has joined #openstack-nova03:51
*** pixelbeat has quit IRC03:51
*** jwcroppe has joined #openstack-nova03:54
*** achanda has joined #openstack-nova04:00
*** Marga_ has joined #openstack-nova04:02
*** vishwanathj has quit IRC04:02
*** Marga__ has joined #openstack-nova04:02
*** vishwanathj has joined #openstack-nova04:03
*** lyanchih has quit IRC04:05
*** piyanai has joined #openstack-nova04:05
*** Jeffrey4l has quit IRC04:05
*** achanda has quit IRC04:06
*** Marga_ has quit IRC04:07
*** vishwana_ has joined #openstack-nova04:07
*** markvoelker has quit IRC04:08
*** vishwanathj has quit IRC04:09
*** dims_ has quit IRC04:14
*** dims has joined #openstack-nova04:15
*** amotoki has quit IRC04:17
*** dims has quit IRC04:20
*** baoli_ has quit IRC04:22
*** amotoki has joined #openstack-nova04:23
*** achanda has joined #openstack-nova04:24
*** kiran-r has quit IRC04:25
*** vilobhmm has quit IRC04:30
*** tojuvone has joined #openstack-nova04:30
*** shyama has joined #openstack-nova04:31
*** amotoki has quit IRC04:33
*** vilobhmm has joined #openstack-nova04:34
*** piyanai has quit IRC04:37
*** eglynn has quit IRC04:39
*** garyk has joined #openstack-nova04:41
*** amotoki has joined #openstack-nova04:43
*** wanghua has joined #openstack-nova04:43
*** humble_ has quit IRC04:46
*** mutoulbj_ has quit IRC04:46
*** otter768 has joined #openstack-nova04:48
*** annashen has quit IRC04:50
*** tpeoples has quit IRC04:51
*** unicell has joined #openstack-nova04:52
*** mutoulbj has joined #openstack-nova04:52
*** otter768 has quit IRC04:53
*** ildikov has quit IRC04:55
*** _gryf|afk has quit IRC04:58
*** dims has joined #openstack-nova05:01
*** dims_ has joined #openstack-nova05:02
*** _gryf has joined #openstack-nova05:03
_gryfo/05:03
*** arnaud___ has quit IRC05:04
*** dims has quit IRC05:06
*** shyama has quit IRC05:08
*** shyama has joined #openstack-nova05:09
*** mutoulbj has quit IRC05:11
*** arnaud___ has joined #openstack-nova05:14
*** dims_ has quit IRC05:14
*** mutoulbj has joined #openstack-nova05:15
*** nkrinner has joined #openstack-nova05:16
*** humble_ has joined #openstack-nova05:20
*** wanghua has quit IRC05:24
*** Sukhdev has joined #openstack-nova05:25
*** irenab has joined #openstack-nova05:29
*** tojuvone has quit IRC05:29
*** tojuvone has joined #openstack-nova05:29
*** SourabhP has joined #openstack-nova05:31
*** _gryf has quit IRC05:35
*** _gryf has joined #openstack-nova05:36
*** lyanchih has joined #openstack-nova05:39
*** Murali has joined #openstack-nova05:40
*** arnaud___ has quit IRC05:42
*** arnaud___ has joined #openstack-nova05:43
*** Nikolay_St has quit IRC05:43
*** ildikov has joined #openstack-nova05:44
*** fawadkhaliq has joined #openstack-nova05:45
*** deepthi has joined #openstack-nova05:45
*** sudipto has joined #openstack-nova05:48
*** vilobhmm has quit IRC05:49
*** ig0r_ has joined #openstack-nova05:51
*** _gryf has quit IRC05:55
*** ig0r__ has quit IRC05:55
*** _gryf has joined #openstack-nova06:00
*** markvoelker has joined #openstack-nova06:00
*** yingjun has joined #openstack-nova06:01
openstackgerritSergey Nikitin proposed openstack/nova: Added method exists to the Tag object  https://review.openstack.org/17810306:01
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex  https://review.openstack.org/19721306:02
*** Kennan2 has quit IRC06:02
*** Kennan has joined #openstack-nova06:03
*** moshele has joined #openstack-nova06:03
*** yingjun has quit IRC06:03
*** markvoelker has quit IRC06:04
*** unicell1 has joined #openstack-nova06:06
*** davidlenwell has quit IRC06:06
*** exploreshaifali has joined #openstack-nova06:06
*** davidlenwell has joined #openstack-nova06:08
*** unicell has quit IRC06:08
*** afazekas has joined #openstack-nova06:08
*** yingjun has joined #openstack-nova06:08
*** aix has joined #openstack-nova06:09
*** pradipta has joined #openstack-nova06:09
*** ajayaa has joined #openstack-nova06:12
*** arnaud___ has quit IRC06:13
*** dims has joined #openstack-nova06:15
*** sahid has joined #openstack-nova06:17
*** kiran-r has joined #openstack-nova06:19
*** dims has quit IRC06:20
*** josecastroleon has quit IRC06:22
*** jichen has joined #openstack-nova06:25
*** annashen_ has joined #openstack-nova06:27
*** cfriesen has quit IRC06:29
*** radek__ has joined #openstack-nova06:31
*** moshele has quit IRC06:32
*** scheuran has joined #openstack-nova06:35
*** hakimo has quit IRC06:40
*** yingjun has quit IRC06:43
*** annashen_ has quit IRC06:45
*** bfic has joined #openstack-nova06:46
*** yingjun has joined #openstack-nova06:46
*** _kiran_ has joined #openstack-nova06:48
*** otter768 has joined #openstack-nova06:49
*** kiran-r has quit IRC06:51
*** achanda has quit IRC06:52
*** rdopiera has joined #openstack-nova06:53
*** otter768 has quit IRC06:54
*** pbandzi has joined #openstack-nova06:54
*** wanghua has joined #openstack-nova06:56
*** yamahata has joined #openstack-nova06:57
*** nihilifer has joined #openstack-nova06:58
*** humble_ has quit IRC06:58
*** humble_ has joined #openstack-nova06:59
*** armax has quit IRC07:00
*** wanghua has quit IRC07:03
*** davideagnello has quit IRC07:04
*** _kiran_ has quit IRC07:08
*** Piet has quit IRC07:10
*** Longgeek has quit IRC07:10
*** asettle has quit IRC07:10
*** lpetrut has joined #openstack-nova07:13
*** belmoreira has joined #openstack-nova07:15
*** ssurana has quit IRC07:16
*** ociuhandu has joined #openstack-nova07:16
*** SourabhP has quit IRC07:17
*** apuimedo has joined #openstack-nova07:18
*** ssurana has joined #openstack-nova07:18
*** tobe has joined #openstack-nova07:18
*** lpetrut has quit IRC07:19
*** belmoreira has quit IRC07:21
*** lyanchih has quit IRC07:21
*** moshele has joined #openstack-nova07:22
*** ssurana has quit IRC07:22
*** Sukhdev has quit IRC07:25
*** lyanchih_ has joined #openstack-nova07:26
*** mrda is now known as mrda-away07:26
*** belmoreira has joined #openstack-nova07:28
*** karimb has joined #openstack-nova07:29
*** jlanoux has joined #openstack-nova07:29
*** jwcroppe has quit IRC07:29
*** ferest has joined #openstack-nova07:30
openstackgerritSergey Vilgelm proposed openstack/nova: Switch to the oslo_utils.fileutils  https://review.openstack.org/19957507:32
*** pmurray has joined #openstack-nova07:34
*** pmurray is now known as Guest1838807:34
*** exploreshaifali has quit IRC07:34
*** Nikolay_St has joined #openstack-nova07:35
*** PaulMurray has quit IRC07:35
openstackgerritheijlong proposed openstack/nova: Add uuid into SecurityGroup database model  https://review.openstack.org/19992807:37
*** kumar has joined #openstack-nova07:37
*** lyanchih_ has left #openstack-nova07:37
*** kun_huang has joined #openstack-nova07:41
kun_huangguys, is ta reasonable to add fping functionality in shell.py?07:42
*** lyanchih has joined #openstack-nova07:42
*** Sukhdev has joined #openstack-nova07:46
*** amotoki_ has joined #openstack-nova07:47
*** amotoki has quit IRC07:49
*** Longgeek has joined #openstack-nova07:50
*** kumar has quit IRC07:51
*** yamahata has quit IRC07:51
*** bkopilov has quit IRC07:52
*** LukeHinds has joined #openstack-nova07:59
*** exploreshaifali has joined #openstack-nova08:00
*** humble_ has quit IRC08:00
*** humble_ has joined #openstack-nova08:01
*** markvoelker has joined #openstack-nova08:01
*** Sukhdev has quit IRC08:02
openstackgerritMoshe Levi proposed openstack/nova: Enhance PCI passthrough whitelist to support regex  https://review.openstack.org/19948808:02
*** lucas-dinner has joined #openstack-nova08:02
*** lucas-dinner has quit IRC08:03
*** lucasagomes has joined #openstack-nova08:03
*** bkopilov has joined #openstack-nova08:04
*** jasondotstar has joined #openstack-nova08:04
*** lpetrut has joined #openstack-nova08:05
*** markvoelker has quit IRC08:06
*** jistr has joined #openstack-nova08:09
*** bkopilov has quit IRC08:12
*** ferest has quit IRC08:13
*** lpetrut has left #openstack-nova08:14
*** whitebyte has joined #openstack-nova08:15
*** dims has joined #openstack-nova08:16
*** shyama_ has joined #openstack-nova08:19
*** shyama has quit IRC08:20
*** shyama_ is now known as shyama08:20
openstackgerritAbhishek Kekane proposed openstack/nova: POC: Improve performance of Unshelve api  https://review.openstack.org/18487108:20
*** dims has quit IRC08:21
*** ndipanov has joined #openstack-nova08:21
openstackgerritsahid proposed openstack/nova: libvirt: add os_admin_user to use with set admin password  https://review.openstack.org/18592108:21
openstackgerritsahid proposed openstack/nova: libvirt: enhance libvirt to set admin password  https://review.openstack.org/18591008:21
openstackgerritsahid proposed openstack/nova: libvirt: rework quiesce to not share "sensitive" informations  https://review.openstack.org/19662008:21
*** abhishekk has joined #openstack-nova08:21
*** bkopilov has joined #openstack-nova08:24
openstackgerritjichenjc proposed openstack/nova: Check flavor type before add tenant access  https://review.openstack.org/12433808:24
*** Longgeek has quit IRC08:25
*** browne has quit IRC08:25
openstackgerritsahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop  https://review.openstack.org/17485408:26
openstackgerritsahid proposed openstack/nova: libvirt: deprecate use_usb_tablet in favor of pointer_model  https://review.openstack.org/17624208:26
sahidndipanov: 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 it08:27
*** whitebyte has quit IRC08:28
*** jasondotstar has quit IRC08:29
ndipanovyes I will do it sahid08:29
ndipanovthis is just moving direct calls to domain.smth() into a class right?08:30
*** whitebyte has joined #openstack-nova08:30
sahidndipanov: yes, just encapsulation08:30
sahidthen we introduce new class here https://review.openstack.org/#/c/190210 for block device08:31
sahidand again... encapsulation08:31
*** yassine_ has joined #openstack-nova08:31
*** Longgeek has joined #openstack-nova08:34
*** wwriverrat has quit IRC08:36
*** whitebyte has quit IRC08:37
*** wwriverrat has joined #openstack-nova08:37
ndipanovok sahid let me fix one of my patches and I will jump on this08:39
johnthetubaguyandrewbogott_afk: mriedem is correct, best to add it to the meeting agenda08:40
sahidgreat thank you ndipanov08:41
*** wwriverrat has quit IRC08:44
*** bkopilov has quit IRC08:44
*** wanghua has joined #openstack-nova08:46
*** ssurana has joined #openstack-nova08:48
bauzasmorning Nova08:48
*** kiran-r has joined #openstack-nova08:49
bauzasjohnthetubaguy: FYI, I had a discussion with alaski and dansmith last night08:49
bauzasjohnthetubaguy: about the cells v1 features08:49
johnthetubaguybauzas: interesting, whats the news?08:49
bauzasjohnthetubaguy: tbc, we're saying that cells v1 is stable08:49
*** kun_huang has left #openstack-nova08:50
*** otter768 has joined #openstack-nova08:50
bauzasjohnthetubaguy: meaning that new features should be discussed and -1 unless it's a very nit08:50
*** humble_ has quit IRC08:50
bauzasjohnthetubaguy: like alaski's change for adding a new filter using the existing properties08:50
johnthetubaguybauzas: so block all cells v1 features, basically?08:50
bauzasjohnthetubaguy: yeah, but we can leave some room for quickwins08:51
bauzasjohnthetubaguy: anyway, we're post-freeze, so that doesn't really mean we will have new features08:52
*** ankit has joined #openstack-nova08:52
bauzasjohnthetubaguy: but that's the consensus08:52
*** ssurana has quit IRC08:52
johnthetubaguybauzas: so I thought that was the consensus before, so I am glad that is the consensus now08:52
bauzasjohnthetubaguy: https://review.openstack.org/#/c/192875/ is an exception08:52
johnthetubaguybauzas: we should write that down I guess08:52
bauzasjohnthetubaguy: totally agreed, that was my point on yesterdayt08:53
bauzasjohnthetubaguy: it left me 2 thoughts08:53
bauzasjohnthetubaguy: 1/ we need to write down the stable policy for cells v108:53
*** jasondotstar has joined #openstack-nova08:53
bauzasjohnthetubaguy: 2/ I'm not very comfortable having the filters not functionally tested08:53
bauzasjohnthetubaguy: in particular since I'm working on the Spec objectification which modifies all the filters08:54
bauzasjohnthetubaguy: (see my comments on the above change)08:54
*** otter768 has quit IRC08:54
openstackgerritAlex Xu proposed openstack/nova: Remove useless db call instance_get_all_hung_in_rebooting  https://review.openstack.org/19997008:54
openstackgerritAlex Xu proposed openstack/nova: Remove useless db call instance_get_all_by_host_and_not_type  https://review.openstack.org/19997108:54
openstackgerritAlex Xu proposed openstack/nova: Remove db layer hard-code permission checks for reservation_expire  https://review.openstack.org/19997208:54
johnthetubaguybauzas: yeah, we need the cells thing written down08:54
bauzasjohnthetubaguy: so, about 1/, I'll raise the point to the next cells meeting on Wed08:55
johnthetubaguybauzas: unit tests should be fine for the filters really, most are not functionally tested08:55
bauzasjohnthetubaguy: well, by looking at that, I'm not very comfortable with not having in-tree functional tests for filters08:55
bauzasjohnthetubaguy: not speaking about Tempest tests08:55
bauzasjohnthetubaguy: just functional tests in the nova treee08:55
bauzastree even08:56
bauzasjohnthetubaguy: at least for the defaulted filters08:56
bauzasjohnthetubaguy: and possibly to all of them08:56
bauzasjohnthetubaguy: so the thing is, I need to think about it a little more08:57
bauzasjohnthetubaguy: for 1/, I'll draft something soon08:57
*** haomaiwang has joined #openstack-nova08:57
bauzas(just adding that in my big TODO list :)08:57
*** haomaiw__ has quit IRC09:00
*** bkopilov has joined #openstack-nova09:00
johnthetubaguybauzas: yeah, its worth a try09:01
johnthetubaguybauzas: let me link to a horrible unit test I added, that might help...09:01
bauzasjohnthetubaguy: sure, any thought is nice09:01
bauzasjohnthetubaguy: maybe I'm overthinking about functionally testing the filters, but maybe not :)09:02
johnthetubaguybauzas: https://github.com/openstack/nova/blob/master/nova/tests/unit/scheduler/test_caching_scheduler.py#L15709:02
*** amotoki_ has quit IRC09:02
bauzasjohnthetubaguy: oh that one, yeak I know it :)09:02
bauzasjohnthetubaguy: since I modified it for the Spec BP09:02
johnthetubaguybauzas: so that is actually a full test of the filters and wights09:02
bauzasjohnthetubaguy: yeah, with a default coverage09:03
johnthetubaguybauzas: sends in data they manipulate, and can even benchmark and profile the performance09:03
bauzasjohnthetubaguy: I mean, default coverage is already done by Tempest, right ?09:03
bauzasjohnthetubaguy: my wonders are more about the corner cases09:03
johnthetubaguybauzas: I guess I mean, thats the framework you need to test those corner cases09:04
bauzasjohnthetubaguy: I see09:04
*** davideagnello has joined #openstack-nova09:04
bauzasjohnthetubaguy: mmm, it deserves a bit more thinking and probably a discussion at the subteam meeting :)09:04
johnthetubaguyI suspect there is a nicer way, but it does the trick09:04
bauzasjohnthetubaguy: agreed09:04
johnthetubaguybasically it just mocks out all the db calls09:05
bauzasI see09:06
bauzasokay, I'll leave that out for now, just modify the cells devref and see what the other guys think09:06
bauzasthat reminds me...09:06
johnthetubaguymore testing is good, but I think that should cover it OK09:06
johnthetubaguyso I have a similar unit test (in an internal branch) that does the same for the cells scheduler09:07
bauzasn0ano: 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 morning09:07
bauzasjohnthetubaguy: correct09:07
johnthetubaguybauzas: is that Bastille day or something?09:08
bauzasjohnthetubaguy: I'm not very worried by any complexity for functionally testing, I'm more concerned about the necessity :)09:08
bauzasjohnthetubaguy: yup, July 14th and our French office is asking us to take July 13th as well09:09
johnthetubaguyah, gotcha09:09
*** davideagnello has quit IRC09:09
bauzastime for seeing the sea :)09:09
openstackgerritNikola Dipanov proposed openstack/nova: Allow for migration object to be passed to _move_claim  https://review.openstack.org/19967909:10
openstackgerritNikola Dipanov proposed openstack/nova: rt: generalize claim code to be useful for other move actions  https://review.openstack.org/19713809:10
openstackgerritNikola Dipanov proposed openstack/nova: rt: move filtering of mygration by type lower in the call stack  https://review.openstack.org/19967809:10
openstackgerritNikola Dipanov proposed openstack/nova: Define migration states and transitions and use when changing status  https://review.openstack.org/19766909:10
openstackgerritNikola Dipanov proposed openstack/nova: Introduce a simple state machine module  https://review.openstack.org/19766809:10
ndipanovsahid, if you want to review something in return ^09:10
johnthetubaguybauzas: good plan!09:10
bauzasjohnthetubaguy: :)09:10
sahidndipanov: sure09:11
bauzasndipanov: on my stars list09:12
ndipanovbauzas, many thanks ;) ;)09:12
*** gszasz has joined #openstack-nova09:13
*** nihilifer has quit IRC09:13
*** deepakcs has joined #openstack-nova09:13
*** wwriverrat has joined #openstack-nova09:14
*** park_heijlong has quit IRC09:14
*** coolsvap|away is now known as coolsvap09:15
deepakcshello all09:16
deepakcswrt: https://review.openstack.org/#/c/192736/8/nova/virt/libvirt/driver.py ...09:18
*** lpetrut has joined #openstack-nova09:19
deepakcsdoes 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 IRC09:21
*** fawadkhaliq has quit IRC09:22
*** lpetrut1 has joined #openstack-nova09:27
*** marzif_ has joined #openstack-nova09:29
*** lpetrut has quit IRC09:29
*** yingjun has quit IRC09:32
*** tdurakov has quit IRC09:33
*** derekh has joined #openstack-nova09:36
*** shuquan has joined #openstack-nova09:37
*** rotbeard has joined #openstack-nova09:47
*** marzif_ has quit IRC09:47
*** coolsvap is now known as coolsvap|away09:48
*** yassine_ has quit IRC09:52
openstackgerritJohn Garbutt proposed openstack/nova: devref: discuss current state of error handling  https://review.openstack.org/17863509:54
*** nihilifer has joined #openstack-nova09:55
kashyapdeepakcs: Why not go the route of what Eric suggests for now?  Fix it for files, and post a subsequent patch for network?09:57
openstackgerritJohn Garbutt proposed openstack/nova: docs: clear between current vs future plans  https://review.openstack.org/19225910:00
*** shuquan has quit IRC10:01
*** markvoelker has joined #openstack-nova10:02
openstackgerritsahid proposed openstack/nova: libvirt: add cpu schedular priority config  https://review.openstack.org/19756710:03
openstackgerritsahid proposed openstack/nova: libvirt: rework membacking config to support future features  https://review.openstack.org/19756610:03
openstackgerritsahid proposed openstack/nova: virt: add contraint to handle realtime policy  https://review.openstack.org/19756810:03
openstackgerritsahid proposed openstack/nova: libvirt: add realtime support  https://review.openstack.org/19756910:03
openstackgerritJohn Garbutt proposed openstack/nova: docs: clear between current vs future plans  https://review.openstack.org/19225910:04
*** radek__ has quit IRC10:05
*** markvoelker has quit IRC10:06
openstackgerritJohn Garbutt proposed openstack/nova: docs: add the scheduler evolution plans  https://review.openstack.org/19226010:06
johnthetubaguybauzas: 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,cm10:07
deepakcskashyap: 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 ideal10:07
*** radek__ has joined #openstack-nova10:07
*** coolsvap|away is now known as coolsvap10:11
openstackgerritAbhishek Kekane proposed openstack/nova-specs: Improve performance of UnShelve API  https://review.openstack.org/13538710:12
*** dims has joined #openstack-nova10:17
*** dims has quit IRC10:21
*** mutoulbj has quit IRC10:22
*** fawadkhaliq has joined #openstack-nova10:22
*** fawadkhaliq has quit IRC10:23
*** fawadkhaliq has joined #openstack-nova10:23
*** coolsvap is now known as coolsvap|away10:25
*** pradipta has quit IRC10:27
openstackgerritJohn Garbutt proposed openstack/nova: Add a summary of the architecture evolution work  https://review.openstack.org/20000110:30
*** humble_ has joined #openstack-nova10:31
*** LukeHinds has quit IRC10:32
*** wanghua has quit IRC10:34
*** andreykurilin__ has joined #openstack-nova10:44
*** IanGovett has joined #openstack-nova10:45
*** wanghua has joined #openstack-nova10:47
*** bkopilov has quit IRC10:49
*** humble_ has quit IRC10:49
*** otter768 has joined #openstack-nova10:51
*** jlanoux has quit IRC10:53
*** otter768 has quit IRC10:55
*** Longgeek has quit IRC10:59
*** bkopilov has joined #openstack-nova11:00
*** pczesno has joined #openstack-nova11:00
*** deepthi has quit IRC11:00
*** pixelbeat has joined #openstack-nova11:01
*** yassine_ has joined #openstack-nova11:03
*** davideagnello has joined #openstack-nova11:05
*** piyanai has joined #openstack-nova11:06
*** fawadkhaliq has quit IRC11:06
*** mutoulbj has joined #openstack-nova11:07
*** fawadkhaliq has joined #openstack-nova11:08
*** davideagnello has quit IRC11:10
*** dprince has joined #openstack-nova11:13
*** Longgeek has joined #openstack-nova11:14
*** mutoulbj has quit IRC11:17
*** wanghua has quit IRC11:17
bauzasjohnthetubaguy: ack, will try to do this today11:18
*** mutoulbj has joined #openstack-nova11:18
*** wanghua has joined #openstack-nova11:19
*** thorst has joined #openstack-nova11:20
*** coolsvap|away is now known as coolsvap11:22
*** markvoelker has joined #openstack-nova11:33
*** dave-mccowan has joined #openstack-nova11:34
openstackgerritAlex Xu proposed openstack/nova: Remove useless db call instance_get_all_by_host_and_not_type  https://review.openstack.org/19997111:34
openstackgerritAlex Xu proposed openstack/nova: Remove useless db call instance_get_all_hung_in_rebooting  https://review.openstack.org/19997011:34
openstackgerritAlex Xu proposed openstack/nova: Remove db layer hard-code permission checks for reservation_expire  https://review.openstack.org/19997211:34
*** jlanoux has joined #openstack-nova11:34
openstackgerritIvan Mironov proposed openstack/nova: libvirt: Minimal qemu version for discard support should depend on image backend.  https://review.openstack.org/19567211:37
*** markvoelker has quit IRC11:37
haypohi. 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
haypoi'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 serie11:40
openstackgerritTomi Juvonen proposed openstack/nova-specs: Get valid server state  https://review.openstack.org/19224611:40
openstackgerritJohn Garbutt proposed openstack/nova: Add a summary of the architecture evolution work  https://review.openstack.org/20000111:41
ndipanovhaypo, queued up11:42
openstackgerritJohn Garbutt proposed openstack/nova: Add a summary of the architecture evolution work  https://review.openstack.org/20000111:42
johnthetubaguyawe, I was so close to the magic change number :(11:43
johnthetubaguyat least I get the one about some space journey a long time in the future11:43
* johnthetubaguy goes for lunch11:43
*** mutoulbj has quit IRC11:44
pkholkindhellmann: 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-nova11:44
*** jaypipes has joined #openstack-nova11:45
*** igordcard_ has quit IRC11:46
*** tobe has quit IRC11:48
*** zhenguo has quit IRC11:50
*** lucasagomes is now known as lucas-hungry11:51
sahidndipanov: yep good catch ! https://review.openstack.org/#/c/190214/9/nova/virt/libvirt/guest.py,cm11:53
ndipanovsahid, now just get another person to +A this and a lot of those can go in11:55
ndipanovhttps://review.openstack.org/#/c/196991/11:55
sahidndipanov: yes cool11:56
sahidactually this one already gave two +2 but wanted to address the doctstring related comment11:57
sahidit should be easy to find a guy :)11:57
ndipanovsahid, meh11:57
ndipanovimho don't bother11:57
*** tojuvone has quit IRC11:57
sdaguesahid: +A on that base patch, that is pretty straight forward11:58
sahidsdague: cool thank you :)11:58
ndipanovsdague, 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 IRC12:02
sdagueno, it's part of a cross project spec12:03
sdaguendipanov: http://specs.openstack.org/openstack/openstack-specs/specs/no-downward-sql-migration.html12:03
ndipanovsdague, thanks12:03
sdagueit merged... a few months ago12:03
ndipanovI see e2e846f12:05
ndipanovand it was even you who did it hehe12:05
ndipanovI just picked you to ask at random as in "he might know " :)12:05
openstackgerritlyanchih proposed openstack/nova: Detaching and then attaching a volume to a paused instance will failed  https://review.openstack.org/20003212:06
*** markvoelker has joined #openstack-nova12:12
*** annegentle has joined #openstack-nova12:15
garykany chance to beg for a few reviews …12:16
garykbeg beg beg12:16
luvumm 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 incorrected12:17
luvas "nova"12:17
*** gabor-t has joined #openstack-nova12:17
*** dims has joined #openstack-nova12:17
garykluv: 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 scheduler12:19
sdague:)12:19
openstackgerritMaxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo  https://review.openstack.org/18431112:21
*** bjornar has quit IRC12:21
nihiliferbauzas: are you here?12:22
luvumm, that didnt help :(12:22
*** dims has quit IRC12:22
*** fawadkhaliq has quit IRC12:22
luv2015-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
luvbut i guess that might not be relevant12:23
luvumm 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 IRC12:27
luvhttp://paste.openstack.org/show/358430/12:28
*** yingjun has joined #openstack-nova12:28
*** andreykurilin has quit IRC12:30
openstackgerritBence Romsics proposed openstack/nova: Do not block instance boot on net without subnet  https://review.openstack.org/20004612:30
luvok, i wil restart nova-api :)12:30
luvfixed, yaay; garyk thanks12:31
*** aarefiev has quit IRC12:31
*** lyanchih has quit IRC12:34
garykluv: 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-nova12:35
*** gabor-t has quit IRC12:36
*** edmondsw has joined #openstack-nova12:37
*** annegentle has quit IRC12:38
*** jasondotstar has quit IRC12:38
*** ajayaa has quit IRC12:38
*** dims has joined #openstack-nova12:40
garykmdbooth: 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
openstackLaunchpad bug 1462957 in OpenStack Compute (nova) "VMware driver cannot report non-contiguous resources to the scheduler" [Undecided,New]12:41
*** piyanai has quit IRC12:41
garykmdbooth: i am not even sure how ceph reports stuff like this - but that is another issue12:41
*** mutoulbj has joined #openstack-nova12:41
openstackgerritJohn Garbutt proposed openstack/nova: docs: clear between current vs future plans  https://review.openstack.org/19225912:43
openstackgerritJohn Garbutt proposed openstack/nova: docs: add a placeholder link to mentoring docs  https://review.openstack.org/20005312:43
*** yingjun has quit IRC12:45
*** fawadkhaliq has joined #openstack-nova12:45
*** mutoulbj has quit IRC12:46
*** mutoulbj has joined #openstack-nova12:46
*** lucas-hungry is now known as lucasagomes12:46
*** andreykurilin__ has quit IRC12:47
*** Longgeek has quit IRC12:49
*** vladikr has joined #openstack-nova12:49
*** pserebryakov has joined #openstack-nova12:49
*** VW_ has joined #openstack-nova12:50
*** otter768 has joined #openstack-nova12:51
openstackgerritJohn Garbutt proposed openstack/nova: Add a summary of the architecture evolution work  https://review.openstack.org/20000112:51
openstackgerritJohn Garbutt proposed openstack/nova: docs: clear between current vs future plans  https://review.openstack.org/19225912:55
*** wanghua has quit IRC12:56
*** amotoki has joined #openstack-nova12:56
*** eglynn has joined #openstack-nova12:56
*** wanghua has joined #openstack-nova12:56
*** otter768 has quit IRC12:56
*** jaypipes has quit IRC12:56
*** andreykurilin has joined #openstack-nova12:57
*** jasondotstar has joined #openstack-nova12:58
*** aarefiev has joined #openstack-nova12:59
*** Marga__ has quit IRC13:00
*** yingjun has joined #openstack-nova13:00
*** ivasev has joined #openstack-nova13:01
*** mdrabe has joined #openstack-nova13:01
mdboothgaryk: My angle is that it's a deficiency in the scheduler13:01
*** ZZelle has quit IRC13:01
*** fawadkhaliq has quit IRC13:01
*** VW_ has quit IRC13:02
*** ZZelle has joined #openstack-nova13:02
*** VW_ has joined #openstack-nova13:02
*** diogogmt has joined #openstack-nova13:02
edleafebauzas: reading back, ack that you won't be able to make next scheduler meeting13:03
edleafebauzas: np, we'll assign everything to you :)13:03
*** rlrossit has joined #openstack-nova13:05
*** Marga_ has joined #openstack-nova13:05
*** ivasev has quit IRC13:05
*** davideagnello has joined #openstack-nova13:06
*** VW_ has quit IRC13:07
*** Marga_ has quit IRC13:07
*** Marga_ has joined #openstack-nova13:07
*** bfic has quit IRC13:08
*** zhenguo has joined #openstack-nova13:09
garykmdbooth: i concur13:09
*** amotoki has quit IRC13:09
garykmdbooth: 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 cracks13:10
*** jaypipes has joined #openstack-nova13:10
garykthe same issues exist with any shared storage system and we'll slowly but sure be bitten by this.13:10
*** davideagnello has quit IRC13:10
mdboothI don't have a fully joined-up understanding of exactly how that data is used13:10
mdboothBut it's certainly not confined to storage13:10
mdboothThe same goes for both memory and cpu13:11
garykjohnthetubaguy: please see the comments i have added to the various ref you have posted - typos and other minor stuff13:11
johnthetubaguygaryk: yes, thank you for those, I have addressed some of those already13:11
mdboothI'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 resources13:12
openstackgerritJohn Garbutt proposed openstack/nova: docs: add a placeholder link to mentoring docs  https://review.openstack.org/20005313:12
*** mriedem has joined #openstack-nova13:12
mdboothI'd be interested to know what the scheduler does with numa, and to know if we could do something similar for the whole hypervisor13:12
dimsjaypipes: 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 tests13:14
*** ivasev has joined #openstack-nova13:15
mdboothjaypipes: This review called to tell me it was feeling sad and unloved: https://review.openstack.org/#/c/141115/13:16
*** artom has joined #openstack-nova13:16
*** amotoki has joined #openstack-nova13:16
*** ijuwang has joined #openstack-nova13:17
*** Longgeek has joined #openstack-nova13:17
jaypipesdims: done.13:17
garykdims: around? https://review.openstack.org/19868913:21
*** piyanai has joined #openstack-nova13:22
*** amotoki has quit IRC13:28
*** amotoki_ has joined #openstack-nova13:28
*** VW_ has joined #openstack-nova13:28
*** edmondsw has quit IRC13:28
*** eharney has joined #openstack-nova13:28
jaypipesmdbooth: done.13:29
*** deepakcs has quit IRC13:29
mdboothjaypipes: Thanks!13:29
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 1  https://review.openstack.org/15256913:30
*** diogogmt has quit IRC13:31
*** piyanai has quit IRC13:33
*** ajayaa has joined #openstack-nova13:33
mriedemjaypipes: 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
jaypipesmriedem: ewww. ok :)13:35
*** dboik_ has quit IRC13:36
mdboothjaypipes: I'm nervous about fixing those typos because I'll lose your +2 :) If I fix them could you re-bless?13:37
jaypipesmdbooth: of course.13:37
mdboothThanks, coming up13:37
*** sudipto has joined #openstack-nova13:38
mriedemso, the last time this got a review was 1/26: https://review.openstack.org/#/c/69047/ - it's been posted since 4/28/1413:38
ndipanovjaypipes, bauzas https://review.openstack.org/#/c/197138/ <- see my comment pls13:38
mriedemi'm giving my first plug for reviews this release to prep for the rare non-priority feature review mating ritual tomorrow13:39
*** lucas__ has joined #openstack-nova13:39
ndipanovmriedem, gross13:39
mriedemndipanov: it's not gross, it's beautiful and deadly13:39
mriedemi expect someone to be eaten when it's over13:39
openstackgerritMatthew Booth proposed openstack/nova: Implement compare-and-swap for instance update  https://review.openstack.org/14111513:40
ndipanovmriedem, watched too many nature shows lately? :)13:41
mriedemndipanov: ok, how about, rare orbital docking sequence?13:42
*** andrewbogott_afk is now known as andrewbogott13:42
*** _gryf is now known as _gryf|afk13:42
*** irenab has quit IRC13:42
*** jdandrea has joined #openstack-nova13:43
johnthetubaguymriedem: I noticed you have the oldest patch on that review day list I created13:44
mriedemjohnthetubaguy: yeah, hence the plug ^13:44
johnthetubaguyFor 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-tracking13:46
johnthetubaguysome of the dates a tad misleading, but its a start13:46
*** tpeoples has joined #openstack-nova13:47
*** maishsk has joined #openstack-nova13:48
mriedemmnestratov: is someone working on replying to the comments here? https://review.openstack.org/#/c/182257/13:48
maishskGood afternoon #nova13:48
*** gabor-t has joined #openstack-nova13:48
maishskAre 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
mriedemmaishsk: sec13:49
mriedemmaishsk: did you go here http://docs.openstack.org/ and see the link for 'High Availability Guide'?13:49
maishskhttp://docs.openstack.org/high-availability-guide/content/ha-aa-controllers.html13:49
mriedemhttp://docs.openstack.org/high-availability-guide/content/_api_services.html13:50
mriedemthe haproxy specific stuff is here http://docs.openstack.org/high-availability-guide/content/ha-aa-haproxy.html13:50
maishskmriedem: I just do not see anything there for Nova.13:50
johnthetubaguynova_compute_api_cluster?13:51
maishskThat is API generic - nothing for Nova13:51
bauzasndipanov: ack13:51
maishskOr is it this - http://docs.openstack.org/high-availability-guide/content/ha-aa-haproxy.html13:51
maishskYeah.. Thats it13:52
mriedemyes13:52
maishskI knew it was there somewhere :)13:53
maishskThanks13:53
*** pbandzi has left #openstack-nova13:53
*** xyang1 has joined #openstack-nova13:55
*** dboik has joined #openstack-nova13:56
mriedemrlrossit: did you see that your patch blew up in grenade partial n-cpu while you were out celebrating?13:57
mriedemi see you did13:58
rlrossitmriedem: yeah, and I put something on the review saying I don't think that's the cause13:58
rlrossitbut I most definitely could be wrong13:58
mriedemhrm13:58
*** kiran-r has quit IRC13:58
*** amotoki has joined #openstack-nova13:58
*** lyanchih has joined #openstack-nova13:59
rlrossitme blowing that up was jenkins's way of saying "Happy Birthday Ryan"13:59
mriedemwould be nice if we had some logging in NovaObjectSerializer._process_object to see what it was processing when it's doing the infinite recursion13:59
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 2  https://review.openstack.org/16740813:59
*** karimb has quit IRC13:59
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Adds support for x509 certificates as keypairs  https://review.openstack.org/13645813:59
johnthetubaguyabhishekk 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 meeting13:59
*** mtanino has joined #openstack-nova13:59
*** ctrath has joined #openstack-nova13:59
rlrossitmriedem: like see what child it's trying to serialize/deserialize?14:00
*** pserebryakov has quit IRC14:00
mriedemrlrossit: sure, just something to point at what objects isn't looping on14:00
*** shyu has quit IRC14:00
*** amotoki_ has quit IRC14:01
rlrossitdidn'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-nova14:02
*** amotoki has quit IRC14:03
*** claudiub has joined #openstack-nova14:03
*** Kennan2 has joined #openstack-nova14:03
dansmith+1 for some tracing14:03
*** Kennan has quit IRC14:04
dansmithI was going to add some of this to conductor if we ever get a TRACE log level back14:04
dansmithbecause it would be very verbose14:04
*** READ10 has joined #openstack-nova14:04
*** mtanino has quit IRC14:08
*** yamahata has joined #openstack-nova14:09
*** edmondsw has joined #openstack-nova14:09
*** yingjun has quit IRC14:10
*** lyanchih has quit IRC14:10
*** lyanchih has joined #openstack-nova14:11
*** eglynn has quit IRC14:11
*** browne has joined #openstack-nova14:11
*** annegentle has joined #openstack-nova14:12
*** mtanino has joined #openstack-nova14:13
*** jwcroppe has joined #openstack-nova14:15
*** SourabhP has joined #openstack-nova14:18
*** whenry_ has joined #openstack-nova14:18
*** piyanai has joined #openstack-nova14:20
SourabhPjohnthetubaguy: ping14:21
edleafeSourabhP: he's running the Nova meeting right now14:23
openstackgerritgaryk proposed openstack/nova: libvirt: improve logging in the driver.py code  https://review.openstack.org/20012314:23
SourabhPedleafe: thanks14:23
*** piyanai has quit IRC14:23
*** ildikov has quit IRC14:26
*** cfriesen has joined #openstack-nova14:27
*** lucas__ has quit IRC14:27
*** lucas__ has joined #openstack-nova14:28
*** mdrabe has quit IRC14:28
*** amotoki has joined #openstack-nova14:29
*** piyanai has joined #openstack-nova14:30
*** pkholkin has quit IRC14:30
snikitindansmith: 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 pr14:32
snikitinimary 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_migrati14:32
snikitinons.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-nova14:32
dansmithsnikitin: in the meeting now, we can chat in a bit14:32
snikitindansmith: ok14:33
kashyapankit: There is a meeting in-progress in #openstack-meeting.  Your request is better suited in this channel.14:33
*** pkholkin has joined #openstack-nova14:34
*** bdr7 has joined #openstack-nova14:34
*** jwcroppe has quit IRC14:34
*** irenab has joined #openstack-nova14:35
*** jwcroppe has joined #openstack-nova14:35
*** ijuwang has quit IRC14:35
ankitkashyap: Thanks, I have added my review request in nova meeting :)14:36
*** nihilifer has quit IRC14:39
*** Murali has quit IRC14:39
*** thangp has joined #openstack-nova14:39
*** yamahata has quit IRC14:40
*** jecarey has joined #openstack-nova14:41
*** yamahata has joined #openstack-nova14:41
openstackgerritJeffrey Zhang proposed openstack/nova: use graduated oslo.policy  https://review.openstack.org/19806514:41
*** ijuwang has joined #openstack-nova14:42
dansmithsnikitin: 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 IRC14:42
SourabhPjohnthetubaguy: hello14:43
mriedemdansmith: it came up in the spec review14:43
mriedemdansmith: because it's tied to an instance which is soft deletable14:43
*** ankit has quit IRC14:43
openstackgerritPavel Kholkin proposed openstack/nova: libvirt: optimize multipath call to identify IQN  https://review.openstack.org/16373314:43
mriedemso 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 back14:43
dansmithmriedem: so don't delete the tags?14:43
*** mnestratov|2 has joined #openstack-nova14:43
mriedemthe tags have basically an fkey to the instance14:44
mriedemeven though we don't do cascading deletes14:44
johnthetubaguySourabhP: hi14:44
dansmithright... I don't get the problem14:44
jaypipesdansmith: what mriedem said..14:44
tpatiljohnthetubaguy: Hi14:44
*** dims has quit IRC14:45
johnthetubaguydansmith: is that the soft delete issues with tags?14:45
mriedemi guess it just feels wrong to me to leave tags hanging out tied to an instance that's deleted14:45
dansmithyeah14:45
dansmithmriedem: we leave sysmeta and other things for the same reason14:45
*** whenry_ has quit IRC14:45
johnthetubaguyseems consistent with those, thats true14:45
dansmithseems like a thing we clean up in some DB purge tool,14:45
dansmithbut making tags soft-deletable just because instance is seems like a backwards step to me14:45
SourabhPjohnthetubaguy: 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
mdboothjaypipes: All the Jenkins +1 goodness you've come to expect without the unsightly typos: https://review.openstack.org/#/c/141115/14:45
mriedemi was going to say, at what point do we delete the tags - a script to do that would help14:45
johnthetubaguyyeah, thats basically what we do for the other stuff14:45
tpatiljohnthetubaguy: I want to talk with you about Improve performance of UnShelve API specs14:46
johnthetubaguySourabhP: 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 place14:46
*** mnestratov has quit IRC14:46
johnthetubaguytpatil: yes, I think we should really now focus on getting this merged in the spec backlog14:47
*** mdrabe has joined #openstack-nova14:47
mriedemdansmith: jaypipes: snikitin: i'd be up for a revision to the spec if we put something in there about handling the delete case14:47
tpatiljohnthetubaguy:So Is this spec still in consideration for Liberty cycle?14:47
dansmithmriedem: we don't have a thing to do the actual cleanup of instance, sysmeta, etc, etc AFAIK14:48
johnthetubaguytpatil: I don't think it is at this point really14:48
dansmithso making the tags implementor do that doesn't seem quite fair14:48
*** bdr7 has quit IRC14:48
dansmithmriedem: but I agree we need that thing.. I've been asked for it a couple times in the last month14:48
mriedemtpatil: is there code up for any of the unshelve api stuff?14:48
johnthetubaguytpatil: we have way too much in liberty already, I don't think we would ever get the code reviewed in time14:48
tpatiljohnthetubaguy: yes14:48
*** bdr7 has joined #openstack-nova14:48
mriedemtpatil: i also had questions in the spec this morning14:48
mriedemabout the actual need for a config option14:48
jaypipesmdbooth: done.14:49
johnthetubaguytpatil: 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 opens14:49
mdboothjaypipes: Thanks14:50
jaypipesnp14:50
mriedemctrath: no more soft delete is a tihng you could help with if you're looking for work14:50
mriedemseems to have stalled14:50
*** dims has joined #openstack-nova14:50
SourabhPjohnthetubaguy: 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
ctrathmriedem: sounds good.. throw something my way14:50
tpatilmriedem: IMO, If shelve instance is working for ceph/sheeepdog today then it will continue to work in the same way14:50
mriedemhttps://review.openstack.org/#/c/184645/14:50
mriedemctrath: ^14:50
johnthetubaguySourabhP: at this point, no we extended the exception process more than I wanted to, but that has finished now I am afraid14:51
mriedemSourabhP: we've already approved at least one or two new vif types for liberty14:51
*** burt has joined #openstack-nova14:51
mriedemSourabhP: did you re-propose any bp for your new type/14:51
mriedem?14:51
mriedemhttps://bugs.launchpad.net/nova/+bug/1423308 isn't a bug, it's a feature add14:51
openstackLaunchpad 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
ndipanovmriedem, hey man what up with this https://review.openstack.org/#/c/139688/14:51
mriedemso you need a bp14:51
johnthetubaguySourabhP: yeah, looking at your patch, we don't have a blueprint for this14:51
johnthetubaguyany yeah, we sure need a blueprint for it14:51
mriedemndipanov: ?14:52
*** Jeffrey4l has joined #openstack-nova14:52
ndipanovthe code is ready I think - no need to pull the "process" card14:52
SourabhPoh ok, maybe I waited too long and hoped that os-vif lib to happen this cycle14:52
mriedemndipanov: there is lots of code ready for lots of things that are already approved before the deadlines14:52
*** otter768 has joined #openstack-nova14:52
johnthetubaguymriedem: +114:52
ndipanovtrue14:52
ndipanovI mean whatever14:53
tpatiljohnthetubaguy: 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 day14:53
sahidthe spec got the 2 +2 before the deadline14:53
*** armax has joined #openstack-nova14:53
sahidjohnthetubaguy: mriedem ^14:53
johnthetubaguywe 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 point14:53
ndipanovI just think the whole </3 for nfv is a contentious for the wrong reasons14:53
ndipanovand it seeps through in things like this14:53
*** marzif_ has joined #openstack-nova14:53
mriedemhonestly it's not really about nfv14:53
johnthetubaguytpatil: tomorrow is for code reviews I am afraid, and a blueprint has to be approved to get a priority attached14:54
mriedemthe spec was restored on 6/22 which is really late14:54
sahidwe have a feature ready spec accepted by 2 core before the deadline14:54
tpatiljohnthetubaguy: ok14:54
SourabhPjohnthetubaguy: mriedem: thanks for the clarification!14:54
*** tsekiyama has joined #openstack-nova14:55
sahidunfortunatly mikal was not here to reapproved it after the rebase and before the deadline14:55
johnthetubaguySourabhP: reach out to me at the beginning of M, and we can make sure things are sorted14:55
SourabhPjohnthetubaguy: will do14:55
*** bdr7 has quit IRC14:55
johnthetubaguysahid: 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 them14:56
*** bdr7 has joined #openstack-nova14:56
mriedemsahid: the spec freeze for L was 6/25 https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule14:56
mriedemsahid: i see mikal +2'ed on 6/2214:56
johnthetubaguySourabhP: 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_Schedule14:56
mriedemthere weren't 2 +2s14:56
ndipanovmriedem, late for what? it's like 2 small patches... I mean whatever14:56
ndipanovit's kinda silly to pull process on this because it's a good example of why blanket process is not good for everything14:57
mriedemndipanov: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/libvirt-real-time,n,z14:57
*** agireud has joined #openstack-nova14:57
*** otter768 has quit IRC14:57
mriedemit's a few hundred LOC after all of the deadlines14:57
mriedemand we already have a full load14:58
mriedemand this isn't a priority14:58
SourabhPjohnthetubaguy: sure thanks, will keep a closer eye on the deadlines from now on14:58
dansmithbauzas: did you see my question(s) on the migrate destinations patch?14:58
johnthetubaguymriedem: +114:58
*** Jeffrey4l has quit IRC14:58
dansmithbauzas: happy to iterate quick on that if you want to take a look14:58
bauzasdansmith: not yet,14:58
*** annegentle has quit IRC14:58
bauzasdansmith: but going there14:58
ndipanovmriedem, yes all of that is true and it's still not very justified to block it14:59
ndipanovbut whatever14:59
bauzasdansmith: oooh, seems I was unclear14:59
ndipanovit gets in and we find bugs and fix them14:59
ndipanovit doesn't and we don't for 6 more months15:00
bauzasdansmith: that's not the API which will call the scheduler, but rather the conductor15:00
*** achanda has joined #openstack-nova15:00
bauzasdansmith: see L26115:00
bauzasdansmith: so there is no blocking call15:00
bauzasfrom the API15:00
mriedemndipanov: i'm sure red hat has a process for carrying it out of tree and fixing the bugs there before its reproposed for M15:00
bauzasdansmith: I was just planning to add a call where the conductor is already checkign that15:01
bauzasdansmith: but I'll modify the spec for being more clear15:01
dansmithbauzas: hang on let me pull it back up15:01
johnthetubaguyndipanov: 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 no15:01
ndipanovmriedem, I mean that's exactly the wrong way to go about this15:01
ndipanovyes - let's drop it15:02
ndipanovnot a huge deal15:02
bauzasdansmith: ack15:03
*** sudipto has quit IRC15:04
dansmithbauzas: okay, I see that, but conductor will then have to make a blocking call to scheduler, yes?15:04
dansmithbauzas: my point is, we can't return Conflict to he API caller without making a blocking call to scheduler, even if we jump through conductor15:04
ndipanovbut - most comments on it are some spelling sillyness afaict, and the code is up... so I don't know15:06
*** achanda has quit IRC15:06
dansmithFWIW,15:06
dansmithI think that these are docs read by lots of people and they're easy to spellcheck15:06
dansmithmissing a deadline because of spelling nits sucks (although that's not what happened here), but ffs.. Just spellcheck it15:07
*** davideagnello has joined #openstack-nova15:07
*** changbl has quit IRC15:07
*** browne has quit IRC15:07
ndipanovdansmith, fwiw I do think you are right - it's a low bar15:07
dansmiththat's my only comment... SPELL SHIT RIGHT15:08
dansmith:)15:08
scheuranmriedem: 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 object15:08
scheuranmriedem: I must admit, I have no clue! The input comes from an neutron api response15:08
scheuranmriedem: so it's being parsed in some other nova layer15:09
*** diogogmt has joined #openstack-nova15:09
*** coolsvap is now known as coolsvap|away15:09
scheuranmriedem: would you recommend to go with six.text_type in any case?15:09
*** gabor-t has quit IRC15:10
*** dave-mccowan has quit IRC15:10
*** davideagnello has quit IRC15:11
ndipanovmriedem, on a slightly less contentious note: https://review.openstack.org/#/c/18063715:14
ndipanovare you ok with the two other patches on there?15:14
*** wanghua has quit IRC15:14
*** sambetts has quit IRC15:15
*** ildikov has joined #openstack-nova15:16
mriedemndipanov: haven't gotten back to them yet after +2 on the bottom change15:16
*** ildikov is now known as ildikov_afk15:16
*** rdopiera has quit IRC15:16
ndipanovkk15:16
*** sambetts has joined #openstack-nova15:17
ndipanovthanks for reviews on those - it'd be really good to have someone else on core who is somewhat familiar with this stuff15:17
mriedemscheuran: so i doubt that vif_details.get(network_model.VIF_DETAILS_VLAN) would have unicode in it15:17
ndipanovwithout having to load a ton of context every tim15:17
ndipanove15:17
mriedemndipanov: well, that's why i hammer you on comment blocks in the code :)15:17
mriedemb/c i promise i'll forget15:18
*** yingjun has joined #openstack-nova15:18
*** yingjun has quit IRC15:18
mriedemscheuran: i'm just prickly when it comes to seeing str() on anytihng b/c of prevoius issues with i18n15:18
mriedemscheuran: six.text_type would be safest, so might not hurt to just use that15:18
scheuranmriedem: I see. I just got rid of it using return '%s.%s' % (src_dev, vlan)15:19
mriedemgaryk: maybe you know, can the vlan info in vif details ever have unicode?15:19
*** tonytan4ever has joined #openstack-nova15:19
scheuranmriedem: this is also how others do it in the same class15:19
mriedemhttps://review.openstack.org/#/c/182283/8/nova/virt/libvirt/vif.py15:19
mriedemscheuran: ok, cool15:19
ndipanovI'm just self conscious about my failure to comprehend how articles work in English so I avoid comments15:19
mriedemndipanov: oh come on, you're a master wordsmith when it comes to speaking 'merican15:20
ndipanovfor some reason whenever i make a break to think - I add a 'the' or 'a' since I don't have a natural feel for them15:20
dansmithI in no way believe this15:21
mriedemsecond, i call bullshit15:21
garykmriedem: as far as i understand they should not.15:21
ndipanovthe true a story15:21
dansmithI'm pretty sure ndipanov talks english betterrer than me15:21
mriedemme fail english? unpossible!15:21
scheuranmriedem: there was also the exception that I introduced and you where asking if the user could recover from it15:21
garyki speak a gooder english15:21
mriedemscheuran: yeah15:22
scheuranmriedem: the answer is no! This is either a bug in the neutron code or a missconfiguration of the neutron code15:22
ndipanovit's the gooder english garyk15:22
garykheh15:22
scheuranmriedem: so I wonder if such a exception is required at all..15:22
mriedemscheuran: well, i think that's what i was looking for (not code bugs, but misconfigs)15:22
mriedemscheuran: is there something in neutron config that would impact this?15:22
*** haypo has quit IRC15:22
*** dave-mccowan has joined #openstack-nova15:22
*** vishwana_ has quit IRC15:23
mriedemscheuran: 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_details15:23
scheuranmriedem: yes, so these two values are stored i a neutron config file, and will be added to the vif information by the ml2 plugin15:23
mriedemelse we get NoneType and a bug is obvious15:23
mriedemscheuran: ah,15:23
scheuranmriedem: but checking for that should be done in neutron I guess15:23
*** vishwanathj has joined #openstack-nova15:23
*** jamielennox is now known as jamielennox|away15:23
*** SourabhP has quit IRC15:23
mriedemscheuran: 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
mriedemi just don't think the error given would be helpful to anyone today15:24
mriedemwhich is why i ask for hints on how to recover15:24
scheuranscheuran: yes, that's right15:25
*** bkopilov has quit IRC15:25
garykscheuran: i am with mriedem here. better to be safe than sorry. today neutron may do it and tomorrow it may not15:26
*** bkopilov has joined #openstack-nova15:26
nagyz_mriedem, any opinion on https://bugs.launchpad.net/nova/+bug/1457517 ?15:26
openstackLaunchpad 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-nova15:27
mriedemnagyz_: i'd ask ndipanov15:27
*** jgriffith has left #openstack-nova15:27
ndipanovoh I know this15:27
ndipanovit's a bug15:28
scheuranmriedem: 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 problem15:28
ndipanovmdbooth, worked on this15:29
ndipanovnagyz_, ^15:29
ndipanovI think the solution is not that simple15:29
mriedemscheuran: well, the admin is the one that gets the support ticket and sees the logs right?15:29
mriedemscheuran: the user just sees that networking failed15:29
mriedemand opens the support ticket15:29
mriedemndipanov: nagyz_: yeah, there was some dependent RT change that looked like it was languishing15:30
mriedemand i'm not very familiar with the RT or scheduler filters stuff here15:30
ndipanovmriedem, no I don't this has to do with RT15:30
ndipanovthe thing is15:30
ndipanovwe grab metadata from volume15:30
pkholkindhellmann: are you here?15:30
*** dims has quit IRC15:30
nagyz_in my head it's a simple fix: if we're booting from volume then ignore the flavor's disk size15:30
ndipanovand then we do a check against it like if it was an image15:30
ndipanovnagyz_, yes that is one way to do it15:31
pkholkindhellmann: 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
ndipanovbut then15:31
ndipanovdo we really want to sidestep that check15:31
mdboothI did15:31
nagyz_ndipanov, in case from booting from volume? yes15:31
nagyz_ndipanov, that only makes sense for ephemeral checks IIRC15:31
nagyz_s/IIRC/IMHO15:31
mdboothnagyz_: So, I believe the correct fix is to set the image size to zero when booting from a volume15:31
ndipanovbecause 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 do15:32
mdboothMy patch set it to the actual size of the volume15:32
*** jamielennox|away is now known as jamielennox15:32
ndipanovso it's not as clear cut as that15:32
mdboothWhereas zero seems to be interpreted as 'ignore this'15:32
*** dims has joined #openstack-nova15:32
*** dims has quit IRC15:32
ndipanovmdbooth, so that would have the same result15:32
ndipanovbut the question is do we want to do it15:32
ndipanovif we agree that - yes15:32
mdboothThe potential issue with that is that it's an undocumented dict of badness, which is passed to several places15:32
ndipanovyes that too15:32
*** dims has joined #openstack-nova15:33
ndipanovnagyz_, does that make some sense?15:33
*** igordcard_ has joined #openstack-nova15: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-nova15:34
*** nihilifer has quit IRC15:34
ndipanovnagyz_, yes15:34
ndipanovthat is a good point too15:34
ndipanovmdbooth, ^15:34
mriedemmdbooth: we're working on the undocumented dict of badness15:34
mdboothmriedem: :)15:35
mriedemmdbooth: see https://review.openstack.org/#/c/192651/ per documentation as a start15:35
*** nihilifer has joined #openstack-nova15:35
mriedemmdbooth: 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 are15:35
*** bdr768 has joined #openstack-nova15:35
*** bdr768 is now known as jgriffith15:35
*** shyama has quit IRC15:35
*** diogogmt has quit IRC15:35
mdboothnagyz_: Absolutely, which is exactly what it does15:35
mdboothThe only issue there is accounting for image size15:36
dansmithisn't "The Undocumented Dict of Badness" the new Indiana Jones movie?15:36
mdboothHowever, you still want to sanity check the flavour's cpu and memory against the values expected by the original image15:36
*** bkopilov has quit IRC15:36
mriedembig ol' bag of dicts (tm)15:36
mriedemthat's the name of my band15:36
mdboothlol15:37
*** piyanai has quit IRC15:37
mriedemi'll always remember the time mrodden flushed when i said 'bag of dicts' too loud in a crowded restaurant once15:37
ndipanovdansmith, ahaha15:37
*** diogogmt has joined #openstack-nova15:37
scheuranmriedem: 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 totally15:38
mriedemscheuran: 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 checked15:38
mriedemscheuran: else, we need something that does validation in neutron15:38
mriedemand logs the misconfig error there before it gets to nova (passing the bad vif_details that is)15:39
mriedemscheuran: just because neutron and the vif stuff has always sucked doesn't mean it needs to continue to suck so bad15:39
mriedemfrom a serviceability pov15:39
jaypipessnikitin, 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 e15:40
jaypipesver hit master for tags, which means no user has ever tagged an instance in production.15:40
*** agireud has quit IRC15:40
scheuranmriedem: ok thx. So I will updated the exception text and see what others comment on it15:40
mriedemjaypipes: well, i was under the assumption that if i delete the instance i'm also deleting the tags,15:41
mriedemdansmith: pointed out just don't delete the tags,15:41
mriedemwhich opens up the question of when do we cleanup the tags?15:41
mriedemfor deleted instances i mean15:41
mriedemthat could be a task, it could be a db cleanup command15:42
dansmithwe don't currently delete instance tags when we delete an instance, AFAICT15:42
dansmithmriedem: that's a larger question of cleanup that affects many other things, not just tags15:42
*** piyanai has joined #openstack-nova15:42
tpatilmriedem: I have replied to your comments on PS https://review.openstack.org/#/c/135387/15. Please check15:42
jaypipesmriedem, 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-nova15:43
*** ctrath has quit IRC15:43
*** jcoufal has quit IRC15:43
*** ctrath has joined #openstack-nova15:43
dansmithjaypipes: 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 them15:43
jaypipesmriedem, 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
dansmithokay so how about this:15:44
dansmithtags has been dragging on for decades15:44
jaypipesdansmith: 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
n0anojaypipes, 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 box15:45
dansmithjaypipes: 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 IRC15:45
jaypipesn0ano: that sounds like hell. :)15:45
dansmithn0ano: you know where that laptop sits on your desk? jaypipes has a pug sitting there.15:45
*** ctrath has joined #openstack-nova15:46
n0anodansmith, mine's behind a nat'd firewall, good luck getting to it15:46
jaypipesdansmith: 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
jaypipesdansmith: the way it is now, tags are a special unicorn since they don't inherit from oslo.db.sqlalchemy.SoftDeleteMixin15:47
n0anojaypipes, yeah, it's not worth the effort to fight my windows centric IT and this way I can the bulk of my work on Linux15:47
dansmithjaypipes: I don't think I understand that at all15:47
dansmithjaypipes: 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
jaypipesdansmith: 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
mriedemi tend to agree with liking consistency and fewer unicorns15:48
dansmithsigh, this is crazy15:48
*** belmoreira has quit IRC15:48
* n0ano intel has recruiters - who knew15:48
mriedemdansmith: would you be opposed to a nova-manage db purge-tags command?15:49
mroddensort 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
jaypipesdansmith: 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
mriedemeven though it's not universal?15:49
dansmithjaypipes: it's not the same thing as a metadata entry though15:49
dansmithI'd just really like to have a high-bw conversation about it, whenever that happens15:50
mriedemi'm game for that15:50
mriedemi have a headset that works!15:50
dansmithbefore I agree to a special case of breaking our rules15:50
mriedemmrodden: chime in on https://review.openstack.org/#/c/184645/ then15:50
mriedemmrodden: and/or https://review.openstack.org/#/c/137669/15:50
*** whenry_ has joined #openstack-nova15:51
dansmithmriedem: oh, missed your purge-tags note15:52
*** bkopilov has joined #openstack-nova15:52
jaypipesdansmith: 3pm EST is the earliest I can do.15:52
dansmithmriedem: I'd rather that than the proposed schema change, yeah, even if it's just temporary15:52
dansmithjaypipes: does it have to be today?15:52
jaypipesdansmith: no, of course not.15:52
dansmithjaypipes: I mean if today is bad, we can do tomorrow15:52
tpatilmriedem: We have uploaded code for improving unshelving instance performance https://review.openstack.org/#/c/184871/, workflow is set to -2 as specs are not approved15:52
jaypipesdansmith: tomorrow would be much beter.15:52
dansmithjaypipes: assuming snikitin is okay with waiting a day15:53
jaypipesdansmith: yes, that's fine.15:53
dansmithokay15:53
jaypipesok, will see you guys in a few hours.15:53
*** jaypipes has quit IRC15:53
*** gszasz has quit IRC15:53
mriedemcool, now let's come up with the *real* plan15:54
mriedemhehehe15:54
dansmithsnikitin: sorry :/15:54
*** coolsvap|away is now known as coolsvap15:54
mriedemwe're going to rename tags as pugs and remove the limit of 50 per resource15:54
*** pask81 has joined #openstack-nova15:54
*** jlanoux has quit IRC15:55
*** yassine_ has quit IRC15:55
dansmithmriedem: so, resource_id is a primary key15:55
dansmither, wait15:55
dansmithso is tag15:55
mroddenas 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
mroddenso i am not sure about that BP15:56
dansmithmrodden: we should provide the audit trail in a different way15:56
mroddeni will admit that its totally not consistent in nova lol15:56
dansmiththe current way is terrible15:56
*** moshele has quit IRC15:56
mroddendansmith, yeah i think that would be preferred15:56
mroddenand TBH things like ceilometer aren't really ... super useful for that today15:57
*** shyama has joined #openstack-nova15:58
dansmithmriedem: 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
dansmithmriedem: and then we just delete instance, sysmeta, metadata, tags, etc15:58
*** ctrath has quit IRC15:58
*** claudiub has quit IRC15:59
dansmithmriedem: like, I don't think it's much more complicated to do the more useful thing15:59
*** ctrath has joined #openstack-nova15:59
dansmithalso,15:59
bauzasdansmith: okay I see the problem, thanks for your thoughts15:59
dansmithwe never really delete instances, and require people to manually purge today15:59
*** davideagnello has joined #openstack-nova15:59
dansmithso I don't see any reason why they can't just delete tags based on instances they delete16:00
dansmithsoft-deleted or not16:00
dansmithreally doesn't matter, IMHO16:00
dansmithbauzas: cool16:00
bauzasdansmith: 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 problem16:00
*** davideagnello has quit IRC16:00
dansmithbauzas: I dunno about vm_state, but in general yes16:01
mriedemdansmith: "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
mriedemor as part of a tool?16:01
dansmithbauzas: when we create the migration record, we could set the status=scheduling and then status=conflict16:01
*** browne has joined #openstack-nova16:01
dansmithmriedem: I mean we don't actually delete anything today, not even instances16:01
bauzasdansmith: sounds good to me16:01
dansmithmriedem: so not deleting tags is the same. People have to delete their soft-deleted instances manually from the DB to reclaim the space16:01
bauzasdansmith: any stuff I could look at for the migration record ?16:01
mriedemdansmith: right16:01
*** pask81 has quit IRC16:02
dansmithbauzas: just look for objects.Migration usage.. it's only in a few places16:02
bauzasdansmith: sounds good16:02
bauzasdansmith: okay, will change the spec16:02
dansmithmriedem: 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
mriedemdansmith: yeah16:03
*** angdraug has joined #openstack-nova16:03
*** bitblt has joined #openstack-nova16:03
mriedemi 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 idea16:03
dansmithyes16:04
mriedembut yeah, if we have a tool that just hard deletes instance-related thingies that normal instance-delete doesn't do, then sure16:04
mriedemsince we can lump tags in with the rest16:04
dansmithyeah, that would be best, IMHO16:04
mriedemit's at least a manual step for a not very hard thing until we can put some automation around archive and no more soft delete16:05
dansmithyeah16:05
*** dims_ has joined #openstack-nova16:05
mriedemctrath: make it so16:05
dansmithit will avoid a schema migration _and_ fill a need that people currently have16:05
*** lucas__ has quit IRC16:05
rlrossitmriedem: 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 IRC16:05
dansmithrlrossit: yeah, put it in front of this one16:06
mriedemrlrossit: i'd say yes, throw that up before your change16:06
dansmithrlrossit: did you see my comment from yesterday?16:06
openstackgerritMatt Riedemann proposed openstack/nova: Set autodoc_index_modules=True so tox -e docs builds module docs again  https://review.openstack.org/19929716:06
rlrossitdansmith: 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
openstackgerritjichenjc proposed openstack/nova: Add direct power_on function in compute layer  https://review.openstack.org/20016516:07
dansmithrlrossit: and I think it is16:07
rlrossitdansmith: well if I had to gamble on who's most likely right, I would bet on you...16:08
dansmithrlrossit: 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 it16:08
dansmithrlrossit: if I'm wrong it'll be that much sweeter for you, right? :D16:08
bauzasdansmith: sounds good for using the Migration.status field, as I can see in the conductor16:08
rlrossitbut in the case of lists, the subobject is literally always just 'object' with the object it holds... that never changes in any version16:08
dansmithbauzas: cool16:08
bauzasdansmith: I guess we have the API resource for seeing all migrations right?16:08
dansmithrlrossit: sure, but you don't know it's recursing on a list16:09
bauzasdansmith: I remember something in the API, I need to look at the official docs16:09
openstackgerritAndreas Scheuring proposed openstack/nova: Add macvtap as new virtual interface (vif) type to Nova's libvirt driver  https://review.openstack.org/18228316:09
*** dims has quit IRC16:09
rlrossitdansmith: ...this function is only on the ListObjectBase right?16:09
dansmithbauzas: it's admin-only, but yes16:09
bauzasdansmith: my point being, I want to make sure that the user knows it failed16:09
dansmithgah ETOOMANYTHREADS16:09
bauzasdansmith: okay, that's good then, since evacuate and live-migrate are admin too16:10
bauzasah ah16:10
dansmithbauzas: ah, right16:10
rlrossitlol16:10
bauzasobjects.DanSmith.lock()16:10
dansmithbauzas: but yeah, for these two, those users can see the migrations16:10
rlrossitgotta speed up that context switching16:10
dansmithbauzas: and we want to make that api exposed to the user, we just need to clean up some things first16:10
dansmithbauzas: so that they can see that an admin moved their thing16:10
bauzasdansmith: agreed, and all of that is just RBAC eh :)16:10
dansmithbauzas: yeah16:11
bauzasdansmith: so any operator can just change the policy16: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
bauzasdansmith: ok, thanks for your help16:11
dansmithbauzas: I think we have to make the policy finer grained before we can, but yes that's the idea16:11
bauzasdansmith: objects.DanSmith.free_lock()16:11
*** exploreshaifali has joined #openstack-nova16:11
bauzasor semaphore-116:11
rlrossitobjects.DanSmith.lock() :)16:11
*** fawadkhaliq has joined #openstack-nova16:11
dansmithrlrossit: 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 it16:12
ctrathmriedem: 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 tags16:12
dansmithrlrossit: but this is just overridden on the mixin16:12
dansmithrlrossit: I dunno, I'd do the logging thing and see how it goes16:12
dansmithrlrossit: did my recheck poke the bug again?16:12
*** rotbeard has quit IRC16:12
mdboothnagyz_: I believe the correct solution is to set 'size' to 0 when pulling image metadata from a volume16:13
rlrossitdansmith: it looks to still be problematic if you're talking about the recheck from yesterday afternoon16:13
nagyz_mdbooth, is there patch/review for that at the moment?16:13
mdboothnagyz_: No16:13
nagyz_ok so that's something I can feel free to patch when I get home? :)16:14
dansmithrlrossit: yeah16:14
*** scheuran has quit IRC16:14
dansmithrlrossit: you didn't hit it before and then twice in a row after rebasing on mriedem's thing16: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
mdboothIt should have the effect of ignoring image size, whilst still sanity checking other values16:14
dansmithrlrossit: surely seems suspcious16:14
mdboothThere is a unit test, and it tests that the value is correctly broken :)16:14
rlrossitdansmith: you saying it's time to point fingers?16:14
nagyz_mdbooth, lol16:14
nagyz_mdbooth, ok once I get home I'll see what I can do.16:15
dansmithrlrossit: well, after you left I blamed all this on mriedem anyway16:15
dansmithrlrossit: so yeah, you have a point on the list thing, BUT, if it's not in the list of children then it's broken anyway16:15
dansmithrlrossit: your break will just ignore the impossibility and cause failures elsewhere like we're seeing16:15
rlrossitdansmith: agree on that one16:15
dansmithrlrossit: so making it explode there (or by deleting the thing) will make it easier to spot I think16:16
rlrossitI should add a comment saying you should never get here :P those are always fun16:16
dansmithwell, it's true16:16
mriedemthere are some comments like that in nova-network16:16
dansmithraise Exception("derp") or something16:16
mriedem"this should never happen, but..."16:16
rlrossitraise Exception("it's mriedem's fault")16:16
dansmithues16:16
*** Marga_ has quit IRC16:17
dansmithrlrossit: 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 days16:17
dansmithmriedem: amirite?16:17
*** haypo has joined #openstack-nova16:17
*** haypo has joined #openstack-nova16:17
*** tjones1 has joined #openstack-nova16:17
mriedemunderstanding objects is importante?16:17
dansmithmuy importante16:18
mriedemeverything is important to everyone at all times16:18
mriedemfin16:18
dansmithalright well I retract my gracious comment then16:18
*** piyanai has left #openstack-nova16:18
rlrossitoh by the way dansmith... I have 4 cupcakes left over from yesterday in my freezer just FYI16:18
mriedemwell ndipanov said i needed to understand the bdm stuff16:18
dansmith\o/16:18
rlrossitask mriedem and ctrath, they'll probably tell you how amazing they were16:19
rlrossitI can save one for you if you want it!16:19
dansmithoregon just legalized extra-special baked goods, so that's a tall order!16:19
mriedemas long as i never have to understand the resource tracker or scheduler stuff, i think i'll be happy16:19
ctrathrlrossit: they were good.  Did you make them?16:19
mriedemdansmith: medicinal or recreational?16:19
dansmithmriedem: recreational16:19
*** afazekas has quit IRC16:19
mriedemooo, beat MN then16:19
*** xgsa has quit IRC16:19
mriedemwe're only medicinal as of last week16:19
*** hemnafk is now known as hemna16:19
rlrossitmriedem: I think I'm coming down with cataracts...16:20
dansmithwe've been medicinal so long that they hand you medicinal cards on the way into the state now16:20
rlrossitctrath: I baked them, my mom frosted them, and I put on the sprinkles16:20
dansmithrlrossit: are you twelve?16:20
mriedemhaha16:20
ctrathlol16:20
rlrossitdansmith: almost16:20
mriedem11.5, dick16:20
dansmith.........16:21
*** jamielennox is now known as jamielennox|away16:21
dansmithmoving on16:21
mriedemrlrossit: just keep them for a couple of weks16:21
mriedemi'm sure they'll still be good16:21
* rlrossit is unsure if mriedem is being sarcastic16:21
rlrossitfreezers seriously can keep those bad boys for a long time16:21
*** boris-42 has quit IRC16:22
rlrossitdansmith: 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 IRC16:22
dansmithrlrossit: 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 fine16:23
bauzasjohnthetubaguy: 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
johnthetubaguybauzas: 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 hope16:27
bauzasjohnthetubaguy: I was thinking of calling launchpadlib to get all the changes related to the blueprint and mark them to the dash creator16:27
bauzasjohnthetubaguy: 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.py16:28
*** jamielennox|away is now known as jamielennox16:29
bauzasjohnthetubaguy: that can be a nice way to get a list of changes to add to the gerrit dash creator as a entrypoint16:29
johnthetubaguybauzas: sorry, I am just not going to get chance to do that today16:29
bauzasjohnthetubaguy: same goes for me16:29
bauzasjohnthetubaguy: so perhaps we can try to see how to help with that later one16:30
bauzaslater on, even16:30
johnthetubaguybauzas: no halm in trying it for a later one16:30
*** derekh has quit IRC16:30
johnthetubaguybauzas: I am really just holding out for the gerrit user tags16:30
bauzasjohnthetubaguy: ack16:31
bauzasjohnthetubaguy: well, if all people were correcting marking the topic names16:31
bauzas...16:31
bauzasjohnthetubaguy: it wouldn't need that above ^16:31
johnthetubaguybauzas: we have a list of patches for tomorrow, that should do us16:31
bauzasjohnthetubaguy: it could be a single Gerrit query with the list of topic names being the list of bps16:31
*** nic has joined #openstack-nova16:32
bauzasjohnthetubaguy: yup, for sure16:32
bauzasjohnthetubaguy: but since I'll be on PTO, I was looking how to help :)16:32
bauzasasynchronously :)16:32
johnthetubaguybauzas: build a nice openstack logo shaped sand castle, maybe ;)16:33
*** kiran-r has joined #openstack-nova16:33
bauzasjohnthetubaguy: eeeeeeh, sounds a good idea !16:34
*** annegentle has joined #openstack-nova16:35
*** mgarza_ has joined #openstack-nova16:35
openstackgerritSylvain Bauza proposed openstack/nova-specs: Proposes to check destination on migrations  https://review.openstack.org/17922416:35
*** coolsvap is now known as coolsvap|away16:36
*** ndipanov has quit IRC16:36
*** jistr has quit IRC16:37
*** angdraug has quit IRC16:37
*** annashen has joined #openstack-nova16:38
*** unicell has joined #openstack-nova16:40
*** Nikolay_St has quit IRC16:41
*** jgriffith1 has joined #openstack-nova16:42
openstackgerritTracy Jones proposed openstack/nova: Cleanup of compute api reboot method  https://review.openstack.org/12742716:42
*** jgriffith has quit IRC16:42
*** jgriffith1 is now known as jgriffith16:43
*** lucas__ has joined #openstack-nova16:44
*** bitblt has quit IRC16:45
*** lucas___ has joined #openstack-nova16:46
*** annegentle has quit IRC16:46
*** lucas___ has quit IRC16:47
*** lucas___ has joined #openstack-nova16:47
*** lucas__ has quit IRC16:48
*** annegentle has joined #openstack-nova16:50
*** garyk has quit IRC16:50
mriedemdansmith: you want i should start writing a backlog spec for nova-manage db purge-deleted-instances?16:51
*** lucas___ has quit IRC16:51
*** lucasagomes is now known as lucas-beer16:53
*** otter768 has joined #openstack-nova16:53
*** andrewbogott is now known as andrewbogott_afk16:55
melwittmriedem: 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
mriedemmelwitt: yeah16:55
mriedemmelwitt: that and everything related to instances here http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/instance.py#n4316:56
*** garyk has joined #openstack-nova16:56
mriedemplus instance_extra i think16:57
mriedemctrath: ^16:57
melwittmriedem: okay16:57
*** lucas__ has joined #openstack-nova16:57
mriedemmelwitt: is there already a proposal for that?16:58
*** otter768 has quit IRC16:58
mriedemoutside of jay's archive thing and lxsli's no more soft delete?16:58
melwittmriedem: not that I know of16:58
mriedemmelwitt: this would be temporary until those other things land16:58
mriedemwe shall call it, 'nova-manage db shit-or-get-off-the-pot'16:58
melwittlol16:59
*** whenry_ has quit IRC16:59
*** busterswt has joined #openstack-nova17:02
*** lucas__ has quit IRC17:02
*** SourabhP has joined #openstack-nova17:03
dansmithmriedem: on another call :(17:04
*** Sukhdev has joined #openstack-nova17:05
*** afazekas has joined #openstack-nova17:06
johnthetubaguymriedem: 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 conversation17:06
mriedemjohnthetubaguy: yeah that's the plan17:07
johnthetubaguythe main smarts that are quite specific to use, is that it throttles the delete rate based on how far the DB read slaves are lagging17:07
mriedemhopefully can start that this afternoon17:07
johnthetubaguymriedem: sweet17:07
johnthetubaguys/to use/to use/17:07
johnthetubaguyarg17:07
mriedemha17:07
johnthetubaguys/use/us/17:07
johnthetubaguyso thats a sign I need to go cook tea, and stop attempting to write docs17:08
*** coolsvap|away is now known as coolsvap17:08
johnthetubaguyit will not go well17:08
mriedemyeah i figured i'd just start the 'typey typey' as mikal would say we others can point out the specifics17:08
johnthetubaguy:)17:08
mriedems/we/so/17:08
melwittcool17:09
*** davideagnello has joined #openstack-nova17:10
dims_mriedem: equivalent of yada yada :)17:11
openstackgerritJoel Coffman proposed openstack/nova: libvirt: Add logging for dm-crypt error conditions  https://review.openstack.org/19877417:12
openstackgerritJoel Coffman proposed openstack/nova: libvirt: Replace stubs with mocks for test_dmcrypt  https://review.openstack.org/19875217:12
openstackgerritJoel Coffman proposed openstack/nova: libvirt: Disconnect dm-crypt on instance suspend/stop  https://review.openstack.org/14148517:12
openstackgerritJoel Coffman proposed openstack/nova: libvirt: Add unit tests for methods  https://review.openstack.org/19888017:12
*** afazekas has quit IRC17:12
*** nic has quit IRC17:12
*** nic has joined #openstack-nova17:13
openstackgerritRoman Dobosz proposed openstack/nova: Introducing new forced_down field for a Service object  https://review.openstack.org/18528017:14
openstackgerritRoman Dobosz proposed openstack/nova: New nova API call to mark nova-compute down  https://review.openstack.org/18408617:14
openstackgerritRoman Dobosz proposed openstack/nova: Service group drivers forced_down flag utilization  https://review.openstack.org/19706317:14
mriedem'i went to see my ex, yada yada, now we're getting back together'17:14
mriedemdims_: like that? ^17:14
dims_:)17:15
*** whenry has joined #openstack-nova17:15
*** lucas__ has joined #openstack-nova17:16
*** annegentle has quit IRC17:16
*** salv-orl_ has quit IRC17:20
*** boris-42 has joined #openstack-nova17:20
*** achanda has joined #openstack-nova17:20
*** salv-orlando has joined #openstack-nova17:22
*** yamahata has quit IRC17:22
*** beagles is now known as beagles_lunch17:24
*** afazekas has joined #openstack-nova17:24
*** garyk has quit IRC17:26
*** Marga_ has joined #openstack-nova17:27
*** ssurana has joined #openstack-nova17:27
openstackgerritJohn Garbutt proposed openstack/nova: Add a summary of the architecture evolution work  https://review.openstack.org/20000117:28
*** nihilifer has quit IRC17:29
*** Marga_ has quit IRC17:30
*** Marga_ has joined #openstack-nova17:30
*** lucas__ has quit IRC17:32
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py  https://review.openstack.org/18796817:33
*** vilobhmm has joined #openstack-nova17:33
*** dims_ has quit IRC17:35
*** dims has joined #openstack-nova17:37
*** raildo has quit IRC17:37
*** raildo has joined #openstack-nova17:37
rlrossitmriedem: dansmith: is forcing 3 merge conflicts on me per day your way of hazing the new guy?17:40
*** raildo has quit IRC17:40
*** raildo has joined #openstack-nova17:41
*** annegentle has joined #openstack-nova17:41
mriedemrlrossit: i'm done for awhile17:41
*** tpeoples has quit IRC17:42
*** raildo has quit IRC17:42
*** tpeoples has joined #openstack-nova17:42
*** lucas__ has joined #openstack-nova17:44
*** SourabhP has quit IRC17:47
*** tpeoples has quit IRC17:47
*** raildo_ has joined #openstack-nova17:49
*** lucas__ has quit IRC17:51
openstackgerritRyan Rossiter proposed openstack/nova: Change List objects to use obj_relationships  https://review.openstack.org/19873017:53
*** bdr768 has joined #openstack-nova17:54
*** lucas__ has joined #openstack-nova17:54
*** salv-orlando has quit IRC17:55
*** shyama has quit IRC17:56
*** tpeoples has joined #openstack-nova17:57
*** agireud has quit IRC17:59
*** yamahata has joined #openstack-nova17:59
*** annegentle has quit IRC18:00
*** Mike_D_laptop has joined #openstack-nova18:01
*** jdurgin has quit IRC18:02
*** jgriffith has quit IRC18:03
*** bdr768 is now known as jgriffith18:03
*** markmcclain has quit IRC18:03
*** davidlenwell has quit IRC18:03
*** jgriffith is now known as Guest7774718:04
*** afazekas has quit IRC18:05
*** davidlenwell has joined #openstack-nova18:05
openstackgerritJosh Gachnang proposed openstack/nova: Build Neutron network data for metadata service  https://review.openstack.org/10264918:06
*** markmcclain has joined #openstack-nova18:06
*** Guest77747 has quit IRC18:07
*** bdr768 has joined #openstack-nova18:08
*** kiran-r has quit IRC18:09
*** dprince has quit IRC18:09
*** bdr768 has quit IRC18:09
*** dprince has joined #openstack-nova18:10
*** agireud has joined #openstack-nova18:10
*** dprince has quit IRC18:10
*** dprince has joined #openstack-nova18:10
*** jgriffit1 has joined #openstack-nova18:10
*** SourabhP has joined #openstack-nova18:13
*** jgriffit1 is now known as jdg18:13
*** lpetrut1 has quit IRC18:17
*** Sukhdev has quit IRC18:19
*** annegentle has joined #openstack-nova18:19
*** mutoulbj has quit IRC18:19
*** annegentle has quit IRC18:20
*** jdurgin has joined #openstack-nova18:21
*** dims has quit IRC18:21
*** abhishekk has quit IRC18:21
*** jecarey has quit IRC18:23
*** markvoelker has quit IRC18:24
*** tpeoples has quit IRC18:24
*** markvoelker has joined #openstack-nova18:24
*** Marga_ has quit IRC18:24
*** jdg is now known as jgriffith18:24
*** afazekas has joined #openstack-nova18:25
*** jgriffith is now known as Guest4364318: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
mriedemmelwitt: dansmith: i'm having a hard time remembering the api to restore a deleted instance18:26
*** Guest43643 is now known as jgriffith18:27
dansmithmriedem: you mean a soft_deleted one or a soft_deleted one?18:27
dansmiththere are two18:27
*** jaypipes has joined #openstack-nova18:27
*** anoopjb has joined #openstack-nova18:27
mriedem?18:27
dansmithtwo types of soft_delete18:27
mriedemrestore a soft deleted instance18:27
mriedemthat's not archived in shadow tables18:27
mriedembrb18:27
openstackgerritJay Pipes proposed openstack/nova: Expose Neutron network data in metadata service  https://review.openstack.org/15309718:28
dansmithso, if you set instance_reclaim to nonzero,18:28
dansmiththen deleting an instance doesn't delete it, it just sets vm_state=SOFT_DELETED18:29
dansmithyou can restore that kind and we've done no actual deleted=1 in the database18:29
dansmithif 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, AFAIK18:29
dansmithanother reason soft_deleted is BS18:29
jaypipesJoshNang: hey, https://review.openstack.org/#/c/152703/ needs a rebase when you get a chance...18:29
anoopjbserver = 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 release18:30
jaypipesdansmith, mriedem: wow. I'm away for >2 hours and you're still talking about soft delete :)18:30
jaypipesanoopjb: please use the openstack-dev mailing list for usage questions. thanks!18:30
*** irenab has quit IRC18:31
*** patrickeast has joined #openstack-nova18:31
*** jdurgin1 has joined #openstack-nova18:31
openstackgerritsajeesh cs proposed openstack/nova: Two roles ,namely project_admin and cloud_admin are created to control the quota operations.  https://review.openstack.org/18252218:32
melwittmriedem: 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 of18:33
melwittdansmith: 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 example18:34
dansmithmelwitt: +118:34
*** jdurgin has quit IRC18:34
*** _gryf|afk is now known as _gryf18:35
*** jecarey has joined #openstack-nova18:35
openstackgerritsajeesh cs proposed openstack/nova: Nested Quota : cloud_admin and project_admin creation  https://review.openstack.org/18252218:35
*** beagles_lunch is now known as beagles18:37
alaskimelwitt: 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 delete18:37
alaskiit just wouldn't be easily discoverable18:37
melwittalaski: I see18:37
*** anoopjb has quit IRC18:38
melwittalaski: 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 that18:38
*** lucas__ has quit IRC18:40
JoshNangjaypipes: kk i'll do that right now18:40
alaskimelwitt: 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 well18:40
*** andrewbogott_afk is now known as andrewbogott18:41
melwittalaski: 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 purposes18:42
*** claudiub has joined #openstack-nova18:42
*** afazekas has quit IRC18:43
alaskimelwitt: 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 them18:47
*** lpetrut has joined #openstack-nova18:47
mriedemdansmith: ok, are you thinking that a purge command would check CONF.reclaim_instance_interval then?18:49
melwittalaski: that would be cool to do the latter. I'll check about it :)18:49
mriedemdansmith: i was thinking it's just checking for instances.deleted != 018:49
openstackgerritNicolas Simonds proposed openstack/nova: libvirt: Split out resize_image logic from create_image  https://review.openstack.org/18785718:49
dansmithmriedem: no18:49
*** raildo_ has quit IRC18:50
dansmithmriedem: that's for the vm_states.SOFT_DELETED thing, which is entirely different18:50
mriedemright18:50
mriedemok18:50
*** lpetrut has quit IRC18:50
*** Sukhdev has joined #openstack-nova18:50
*** lucas__ has joined #openstack-nova18:51
*** ajayaa has quit IRC18:51
*** otter768 has joined #openstack-nova18:54
*** claudiub|2 has joined #openstack-nova18:54
*** claudiub has quit IRC18:56
*** lyanchih has quit IRC18:57
*** otter768 has quit IRC18:59
*** jcoufal has joined #openstack-nova19:00
*** lpetrut has joined #openstack-nova19:00
*** changbl has joined #openstack-nova19:00
nagyz_mriedem, how do you want to proceed with the nova part of https://bugs.launchpad.net/cinder/+bug/1463525 ?19:01
openstackLaunchpad bug 1463525 in Cinder "There is no volume encryption metadata for rbd-backed volumes" [Undecided,Triaged]19:01
mriedemnagyz_: 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_yes19:01
nagyz_if that's the fix. :-)19:01
*** thangp has quit IRC19:01
nagyz_I've been playing around with it for a while now19:01
mriedemnagyz_: i think people would appreciate encrypted volume support with ceph backends yes19:01
nagyz_right. I should have a review up early next week19:02
nagyz_people have been bugging me about it as well internally to publish it19: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
vilobhmmalaski : ping19:03
alaskivilobhmm: hi19:03
vilobhmmi 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 state19:04
*** coolsvap is now known as coolsvap|away19:04
*** thangp has joined #openstack-nova19:04
vilobhmmhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L137519:05
*** Marga_ has joined #openstack-nova19:05
vilobhmmwhen the "power_state" is checked it should return "STOPPED"19:05
vilobhmmSHUTDOWN19:05
vilobhmmso should we have a check here beforehand uploading snap to glance that checks for if we have enough available space19:06
vilobhmmalaski : ^^19:06
andrewbogottIs 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 IRC19:07
andrewbogottoh, nevermind, it’s right here in the comment19:07
vilobhmmor the VM/instance should not be brought into "SHUTDOWN" power_state before verifying if we have enough available space19:07
vilobhmmalaski : ^^19:07
alaskivilobhmm: 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 preferable19:07
mriedemnagyz_: yeah you could reword the bug title as needed19:08
vilobhmmalaski : makes sense19:08
mriedemnagyz_: adding the support borders on a specless blueprint possibly, but i guess it depends on how involved the change is19:08
*** ociuhandu has quit IRC19:08
vilobhmmdansmith : ping…have some queries regarding libvirt driver19:09
sdagueso... we bollocks up the metadata server entirely?19:09
dansmithvilobhmm: I'm heads down on something important, maybe someone else can help you19:09
sdagueI 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-nova19:10
vilobhmmdansmith : alrite19:10
sdagueand 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 consistently19:10
*** jgriffith has quit IRC19:10
vilobhmmmriedem : ^^19:10
*** jgriffith has joined #openstack-nova19:11
*** annashen has quit IRC19:11
mriedemkashyap: meet vilobhmm19:12
*** ajayaa has joined #openstack-nova19:12
mriedemsdague: i haven't seen any related changes https://github.com/openstack/nova/commits/master/nova/api/metadata19:12
vilobhmmkashyap : 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 state19:13
vilobhmmso 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 space19:13
*** markvoelker has quit IRC19:15
*** markvoelker has joined #openstack-nova19:15
*** lucas__ has quit IRC19:15
*** lucas__ has joined #openstack-nova19:16
*** READ10 has quit IRC19:17
sdaguemriedem: 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_29019:17
sdagueit's not finding any nodes19:18
*** vladikr has quit IRC19:18
mriedemsdague: 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
sdaguemriedem: that's resolved now19:20
*** mutoulbj has joined #openstack-nova19:20
mriedemnagyz_: i'm pretty sure my usual disclaimer applies19:21
nagyz_which is? :-)19:21
mriedemwhich is, i don't recognize any previously made statements19:21
nagyz_hah!19:21
*** lpetrut has quit IRC19:22
*** lpetrut has joined #openstack-nova19:24
*** mutoulbj has quit IRC19:25
*** lpetrut has quit IRC19:25
openstackgerritNicolas Simonds proposed openstack/nova: libvirt: Split out resize_image logic from create_image  https://review.openstack.org/18785719:26
openstackgerritNicolas Simonds proposed openstack/nova: Fix/implement revert-resize for RBD-backed images  https://review.openstack.org/18739519:26
sdaguemriedem: 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
mriedemsdague: yeah19:27
sdagueI was reading the scheduler log, and trying to figure out why there was only ever 1 host19:28
*** radek__ has quit IRC19:28
rlrossitmriedem: 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_81419:29
sdaguealso, since when did we start reporting every possible qemu arch in our systems?19:29
mriedemsdague: 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 IRC19:30
sdaguethat's the contents of my "supported_instances" column in my local env19:30
mriedemrlrossit: http://logs.openstack.org/30/198730/6/check/check-grenade-dsvm-partial-ncpu/0263871/logs/old/screen-n-cpu.txt.gz?level=TRACE19:30
mriedemsdague: that's probably from the the arch enum in the image_meta object now19:30
mriedemi bet19:30
sdaguecould be, it seems, kind of silly :)19:31
mriedemsdague: open a bug19:31
mriedemrlrossit: i don't actually see the same infinite recursion error as yesterday19:31
mriedembut there is a gmr dump, which is bad19:31
rlrossitmriedem: I noticed the same19:31
rlrossitgmr?19:31
dansmithrlrossit: I thought you were going to add an exception to make the explody bit real obvious19:32
mriedemrlrossit: guru meditation report19:32
*** jecarey has joined #openstack-nova19:32
rlrossitdansmith: I was going to do that in a separate patch I thought19:32
sdaguemriedem: we are dumping a gmr on any failure now19:32
mriedemrlrossit: http://docs.openstack.org/developer/nova/devref/gmr.html19:32
dansmithrlrossit: the logging in a separate patch19:32
mriedemsdague: ok, cool19:32
mriedemrlrossit: i'm checking to see if there is something else more vanilla in the failed job19:32
dansmithmriedem: given that it changed behavior, I'd say that's a pretty good indication that that line was the culprit19:32
sdagueso that's not an indication of direct badness, just an attempt to give first failure capture19:32
mriedemdansmith: agreed19:32
mriedemsdague: yeah, ack, plus, nice19:33
mriedem:)19:33
dansmithmriedem: 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 relationships19:33
mriedems/mriedem/rlrossit/19:33
dansmithyeah19:33
dansmithsorry19:33
rlrossithrm this sounds like a good unit test...19:33
mriedemrlrossit: 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_73719:33
*** radek__ has joined #openstack-nova19:33
mriedemrlrossit: so,19:34
openstackgerritsajeesh cs proposed openstack/nova: Nested Quota Driver: Create column allocated in Quota table  https://review.openstack.org/15132719:34
mriedemrlrossit: yes, we need a unit test since unit tests didn't catch that typo before on ps519:34
mriedemrlrossit: 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 fixed19:34
mriedemdansmith: ^ wootness19:34
mriedemhttp://status.openstack.org/elastic-recheck/index.html#147309819:34
rlrossitso this is one is for realz needing a recheck now?19:35
*** annashen has joined #openstack-nova19:35
*** sambetts has quit IRC19:35
mriedemjust rechecked it19:35
mriedemrlrossit: so, add a unit test for the terrible horrible break in ps5 and then i'm +219:37
rlrossitwoot19:37
openstackgerritMatt Riedemann proposed openstack/nova-specs: Spec: add nova-manage db purge-deleted-instances  https://review.openstack.org/20022419:38
*** sambetts has joined #openstack-nova19:38
mriedemalso, booya ^19:38
mriedemjaypipes: melwitt: dansmith: ctrath: ^19:38
*** Marga_ has quit IRC19:38
*** Nikolay_St has joined #openstack-nova19:38
dansmithrlrossit: so you owe me two cupcakes now I think19:38
dansmithif we're counting it's probably like 42, but..19:39
*** Marga_ has joined #openstack-nova19:39
dansmithI expect an endless stream of cupcakes whilst in MN19:39
mriedemha19:39
mriedemjust get a piece of bunny cake here when you're in town http://restaurant.canadianhonker.com/19:40
mriedemit's right down the street from your hotel19:40
mriedemi suggest breakfast there every morning19:40
dansmithnot sure I can go to a place called "canadian honker"19:40
*** dave-mccowan has quit IRC19:40
mriedemyou have to19:41
rlrossitbetter start liking geese if you're coming to Rochester dansmith19:41
mriedemyeah, we have an infatuation with geese19:41
rlrossitthey aren't as bad this year19:41
dansmithunless your cupcakes are iced with goose poo, I think I'll keep my current feelings for geese where they are19:41
mriedemdon't forget that they hiss at you and don't move out of the way when you're walking19:41
rlrossittry biking... I've almost crashed on multiple occasions19:42
mriedemplus, how would goose shit-frosted cupcakes change your opinion on geese?19:42
dansmithnot sure. sounded funnier in my head19:42
rlrossitdansmith: here's how your trip to rochester will be except s/donuts/cupcakes https://youtu.be/X3ZcZ2h4Ths?t=1m50s19:42
sdaguemriedem: i just thought you folks put marshmallow on everything19:43
mtreinishmriedem, dansmith: http://www.nytimes.com/2015/06/15/world/europe/geese-exterminator-of-the-netherlands-enrages-animal-rights-activists.html19:43
mriedemsdague: no, that's casserole in south carolina19:43
mriedemrussellb: ^19:43
*** jichen has joined #openstack-nova19:43
mtreinishdo you guys need someone like that in rochester?19:43
*** zhenguo has quit IRC19:43
*** lucas__ has quit IRC19:43
mriedemno19:43
dansmithlots of marshmallow-augmented dishes in the more superior carolina too19:43
rlrossitmtreinish: oh my goodness the picture of the goose running towards the camera with its mouth open is making me laugh uncontrollably19:45
melwittwhat kind of casserole? like mashed sweet potato with marshmallow on top?19:45
mriedemmelwitt: that's a thanksgiving thing most places19:45
mriedemmelwitt: i believe the other was something actually like a dinner thing, but with mallow on top19:45
melwittmriedem: oh, wow19:46
rlrossitexcept in minnesota you need to substitute the word "casserole" with "hotdish"19:46
mriedemright19:46
mriedemi was omitting that19:46
dansmithmelwitt: there was a carrot salad my mamma always made me with marshmallows in it19:46
mriedem'here dan, we know you hate vegetables, so we threw some marshmallows on it, now eat it and shaddup'19:47
melwittlol19:47
rlrossitmy parents made squash with marshmallows and butter on top. everything except the squash tasted great19:47
dansmith(except I love vegetables)19:47
mriedemreally?19:47
dansmithyeah19:47
dansmithno scary ones19:47
dansmithbut normal ones19:47
mriedemit's like a paradox19:47
* melwitt makes notes19:47
mriedemdefine a scary vegetable?19:47
dansmithwhite ones19:47
dansmithpink ones19:47
*** achanda has quit IRC19:48
mriedemwe can all agree that cauliflower is disgusting and only belongs in a teenage wrestler's ear19:48
mriedemnext19:48
*** Mike_D_laptop has quit IRC19:48
rlrossitI can't think of a pink vegetable19:48
openstackgerritDean Daskalantonakis proposed openstack/nova: Move existing V2 to legacy-v2  https://review.openstack.org/19372519:48
melwittradish? watermelon19:48
mriedemradish19:48
melwittoh wait, that's a fruit19:48
rlrossitradish is red in my opinion...19:48
mriedembeats19:49
mriedemso anything that can live through the russian winter19:49
rlrossitbeats still exist?19:49
dansmithmriedem: right19:49
dansmithwell, for pink I was thinking eggplant flesh19:49
*** lucas___ has joined #openstack-nova19:49
mriedemeggplant19:49
rlrossitbleck19:49
mriedemthere are special white eggplants19:49
mriedembut that falls in the white category19:49
*** Longgeek has quit IRC19:49
dansmithyep19:49
mriedemi can see you've put a lot of thought into this19:50
mriedemkudos19:50
rlrossitso wait are potatoes white or brown?19:50
rlrossitor pink?19:50
rlrossitthey come in like every color of the rainbow19:50
dansmithrlrossit: they're yummy19:50
*** lucas___ has quit IRC19:50
dansmithrlrossit: also, damn you for finding a flaw in my otherwise perfect system19:50
mriedempotatoes are the ACLU of the vegetable world19:50
beaglesbeets19:50
beaglesnot beats19:50
rlrossitpotatoes, sweet corn, carrots, and lettuce are like the only veggies I can eat19:50
*** annashen has quit IRC19:51
dansmithrlrossit: that's a good assortment, yeah19:51
dansmithI like cucumbers19:51
dansmithand peppers19:51
*** Nikolay_St has quit IRC19:51
dansmithand tomatoes19:51
rlrossitbeagles: I thought we were eating fancy headphones19:51
dansmithrlrossit: "fancy"19:51
beagles:)19:51
sdaguenow I get to scare dansmith with vegetables - https://www.flickr.com/photos/7545231@N03/5815763752/19:51
*** lucas___ has joined #openstack-nova19:51
melwittexpensive headphones19:51
dansmithrlrossit: you just said that to the wrong couple of guys19:51
rlrossitlet's start an argument about whether tomatoes are a fruit or a vegetable19:51
*** annashen has joined #openstack-nova19:51
*** VW_ has quit IRC19:51
dansmithsdague: gross19:51
mriedemsdague: another one bites the dust https://www.flickr.com/photos/7545231@N03/5813900340/in/photostream/19:51
rlrossitoh I hate them19:51
rlrossit(them being beats)19:52
rlrossitto me fancy == expensive19:52
dansmithmassively overpriced19:52
*** VW_ has joined #openstack-nova19:52
dansmithcompletely-terrible-but-still-overly-expensive19:52
mriedemi raise a pinky finger while drinking if i have dr dre's headphones on19:52
rlrossitwhile you're drinking your fancy box wine?19:53
mriedemfranzia19:53
mriedemnothing but the best19:53
rlrossitmriedem: this reminds me of the conversation we had earlier about you being surprised I showed up today by 8...19:54
rlrossitduring 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 want19:54
*** VW__ has joined #openstack-nova19:55
*** VW_ has quit IRC19:56
*** annashen has quit IRC19:56
ctrathrlrossit: do you hibernate all winter?19:57
rlrossitsomething like that ctrath19:58
mriedemdansmith: looks like you missed one in https://review.openstack.org/#/c/196912/19:58
*** ssurana has quit IRC19:59
*** lucas___ has quit IRC20:00
*** irenab has joined #openstack-nova20:01
*** irenab has quit IRC20:01
rlrossitmriedem: 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-nova20:02
mriedemrlrossit: toggle ci button20:02
mriedemthen go to jenkins results from ps520:02
mriedemi have a comment in there with the link to the failure in ps520:02
*** tjones1 has quit IRC20:02
rlrossitwow that button really knows how to hide20:03
openstackgerritMarian Horban proposed openstack/nova: Switch to oslo.service library  https://review.openstack.org/19290020:03
melwittI'm not sure what I have set up but past CI always shows for me20:03
*** annegentle has joined #openstack-nova20:04
*** andrewbogott is now known as andrewbogott_afk20:05
russellbmriedem: mmm, marshmallows20:06
*** dave-mccowan has joined #openstack-nova20:08
*** achanda has joined #openstack-nova20:15
*** packet has joined #openstack-nova20:16
rlrossitdansmith: found out why it's blowing up!20:17
rlrossitdansmith: *someone* decided to start InstanceList's versions at 1.1 https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L117420:18
*** salv-orlando has joined #openstack-nova20:19
rlrossitwait a minute InstanceList was actually doing something before it was version 1.1, but it didn't get child_versions until 1.120:21
*** Nikolay_St has joined #openstack-nova20:23
*** _gryf is now known as _gryf|afk20:23
*** ijuwang has quit IRC20:24
*** ijuwang has joined #openstack-nova20:24
*** jwcroppe has quit IRC20:25
*** lpetrut has joined #openstack-nova20:25
*** jcoufal has quit IRC20:26
*** jecarey has quit IRC20:26
*** ssurana has quit IRC20:27
*** jichen has quit IRC20:29
openstackgerritMarian Horban proposed openstack/nova: Switch to oslo.service library  https://review.openstack.org/19290020:31
*** annashen has joined #openstack-nova20:31
*** tonytan4ever has quit IRC20:31
*** jwcroppe has joined #openstack-nova20:31
*** lpetrut has quit IRC20:34
*** tonytan4ever has joined #openstack-nova20:35
*** achanda has quit IRC20:36
*** karimb has joined #openstack-nova20:38
*** r-daneel has joined #openstack-nova20:39
*** Nikolay_St has quit IRC20:39
*** karimb has quit IRC20:39
*** karimb has joined #openstack-nova20:39
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Switch to oslo.service library  https://review.openstack.org/19290020:40
*** bustersw_ has joined #openstack-nova20:47
*** bustersw_ has quit IRC20:48
*** achanda has joined #openstack-nova20:49
*** r-daneel has quit IRC20:49
*** tjones1 has joined #openstack-nova20:49
*** Sukhdev has quit IRC20:50
*** Longgeek has joined #openstack-nova20:50
*** xgsa has joined #openstack-nova20:51
*** busterswt has quit IRC20:52
*** jaypipes has quit IRC20:53
*** dprince has quit IRC20:54
*** tjones1 has quit IRC20:54
*** lucas__ has joined #openstack-nova20:55
*** otter768 has joined #openstack-nova20:55
*** Longgeek has quit IRC20:55
*** annegentle has quit IRC20:56
*** xgsa1 has joined #openstack-nova20:57
*** ssurana has joined #openstack-nova20:58
*** xgsa has quit IRC20:58
openstackgerritDean Daskalantonakis proposed openstack/nova: Move existing V2 to legacy-v2  https://review.openstack.org/19372520:59
*** tjones1 has joined #openstack-nova20:59
*** otter768 has quit IRC21:00
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Switch to oslo.service library  https://review.openstack.org/19290021:01
*** patrickeast_ has joined #openstack-nova21:01
*** rlrossit has left #openstack-nova21:01
*** packet has quit IRC21:01
*** lucas__ has quit IRC21:03
*** patrickeast has quit IRC21:03
*** patrickeast_ is now known as patrickeast21:03
*** ajayaa has quit IRC21:05
*** andrewbogott_afk is now known as andrewbogott21:05
*** dims has quit IRC21:05
*** tjones1 has quit IRC21:05
*** tpeoples has joined #openstack-nova21:06
*** tjones1 has joined #openstack-nova21:07
*** dims has joined #openstack-nova21:07
*** davideagnello has quit IRC21:08
*** davideagnello has joined #openstack-nova21:09
*** lucas__ has joined #openstack-nova21:10
*** thorst has quit IRC21:10
*** lucas___ has joined #openstack-nova21:11
openstackgerritMatt Riedemann proposed openstack/nova: Add libvirt-python to test-requirements.txt to enforce min version  https://review.openstack.org/18370621:12
*** lucas__ has quit IRC21:14
*** whenry has quit IRC21:14
*** exploreshaifali has quit IRC21:15
*** lucas___ has quit IRC21:15
*** apuimedo has quit IRC21:18
*** lucas__ has joined #openstack-nova21:18
*** jwcroppe has quit IRC21:18
*** cbader has quit IRC21:18
*** jwcroppe_ has joined #openstack-nova21:18
*** jwcroppe has joined #openstack-nova21:21
*** lucas__ has quit IRC21:22
*** jwcroppe_ has quit IRC21:24
*** annashen has quit IRC21:26
*** annashen has joined #openstack-nova21:27
*** browne has quit IRC21:29
mgagnesorrison: Let me know if you have questions regarding my comment on https://review.openstack.org/#/c/184155/21:29
openstackgerritMatt Riedemann proposed openstack/nova: Handle KeyError when volume encryption is not supported  https://review.openstack.org/19383021:29
mgagnesorrison: I'll try to partially forward port my patch to kilo and see how it goes21:30
*** lucas__ has joined #openstack-nova21:30
openstackgerritJosh Gachnang proposed openstack/nova: Add support for provider networks in network_data.json  https://review.openstack.org/15270321:31
*** lpetrut has joined #openstack-nova21:34
*** VW_ has joined #openstack-nova21:35
*** VW_ has quit IRC21:35
*** lucas__ has quit IRC21:35
*** tonytan4ever has quit IRC21:35
*** VW_ has joined #openstack-nova21:35
*** slagle has quit IRC21:36
andrewbogottmriedem: ‘more details’ like… what is metadata?  What is a project_id?  Seems very straightforward to me.21:36
mriedemandrewbogott: like why are you adding the thing21:36
mriedembasically a summary of what you have in the blueprint21:36
*** VW__ has quit IRC21:37
mriedemgit log means i don't have to check external links for information about a commit21:37
*** jamielennox is now known as jamielennox|away21:37
openstackgerritAndrew Bogott proposed openstack/nova: Include project_id in instance metadata.  https://review.openstack.org/19718521:38
mriedemandrewbogott: better, maybe a typo in there21:39
andrewbogottah, so I see21:40
*** haypo has left #openstack-nova21:40
*** radek__ has quit IRC21:40
*** agireud has quit IRC21:41
*** jwcroppe_ has joined #openstack-nova21:41
*** jecarey has joined #openstack-nova21:41
*** jwcroppe has quit IRC21:42
openstackgerritAndrew Bogott proposed openstack/nova: Include project_id in instance metadata.  https://review.openstack.org/19718521:43
*** dboik has quit IRC21:43
mriedem+221:43
*** liusheng has quit IRC21:44
openstackgerritVictor Stinner proposed openstack/nova: Fix Python 3 issues in nova.utils and nova.tests  https://review.openstack.org/19798221:45
openstackgerritVictor Stinner proposed openstack/nova: Port crypto to Python 3  https://review.openstack.org/19774821:45
andrewbogottbah, hang on, one more change...21:45
*** liusheng has joined #openstack-nova21:45
*** dave-mcc_ has joined #openstack-nova21:45
*** mgoddard has quit IRC21:45
openstackgerritAndrew Bogott proposed openstack/nova: Include project_id in instance metadata.  https://review.openstack.org/19718521:45
*** jecarey has quit IRC21:46
*** jwcroppe has joined #openstack-nova21:46
andrewbogottok, /now/ I think it’s right.  thanks mriedem21:47
mriedemyup, +2 again21:47
*** jwcroppe_ has quit IRC21:47
*** jamielennox|away is now known as jamielennox21:48
*** chaochin has quit IRC21:48
*** xgsa1 has quit IRC21:49
*** dave-mccowan has quit IRC21:49
*** xgsa has joined #openstack-nova21:49
*** mgoddard has joined #openstack-nova21:49
openstackgerritMatt Riedemann proposed openstack/nova: objects: add missing enum values to DiskBus field  https://review.openstack.org/20028021:49
*** Sukhdev has joined #openstack-nova21:51
mriedemdansmith: so i'm not actually sure how to handle a compat shim in here with the enum stuff ^21:51
*** Longgeek has joined #openstack-nova21:51
*** chaochin has joined #openstack-nova21:52
*** nelsnelson has joined #openstack-nova21:54
*** dboik has joined #openstack-nova21:56
*** masayukig has quit IRC21:56
*** thangp has quit IRC21:56
mriedemanything less than version 1.2 would explode if trying to create DiskBus fields with lxc or uml21:57
*** dboik has quit IRC21:57
*** masayukig_ has quit IRC21:57
*** masayukig has joined #openstack-nova21:57
*** VW_ has quit IRC21:58
*** VW_ has joined #openstack-nova21:58
*** VW__ has joined #openstack-nova21:59
*** amotoki has quit IRC22:02
*** changbl has quit IRC22:02
*** dims has quit IRC22:03
*** VW_ has quit IRC22:03
*** dims has joined #openstack-nova22:03
*** oomichi has joined #openstack-nova22:05
*** coolsvap|away is now known as coolsvap22:05
*** jgriffith is now known as Guest8583122:06
*** nic has quit IRC22:10
*** Longgeek has quit IRC22:10
*** mriedem is now known as mriedem_away22:10
*** mdrabe has quit IRC22:12
*** jgriffith has joined #openstack-nova22:14
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 2  https://review.openstack.org/16740822:16
*** sahid has quit IRC22:16
*** edmondsw has quit IRC22:18
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Adds support for x509 certificates as keypairs  https://review.openstack.org/13645822:19
*** Marga_ has quit IRC22:19
*** pixelbeat has joined #openstack-nova22:19
*** lucas-beer has quit IRC22:20
cfriesenanyone 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-nova22:22
*** lpetrut has quit IRC22:24
*** jwcroppe has quit IRC22:24
*** jasondotstar has quit IRC22:25
*** browne has joined #openstack-nova22:25
*** sorrison_laptop has quit IRC22:25
*** ivasev has quit IRC22:32
*** jwcroppe has joined #openstack-nova22:36
*** eharney has quit IRC22:37
*** jwcroppe_ has quit IRC22:38
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Switch to oslo.service library  https://review.openstack.org/19290022:40
* alex_xu begin non priority feature review day~22:41
*** jasondotstar has joined #openstack-nova22:43
*** fawadkhaliq has quit IRC22:43
*** annashen has quit IRC22:43
*** annashen has joined #openstack-nova22:44
*** tpeoples has quit IRC22:45
openstackgerritTracy Jones proposed openstack/nova: Cleanup of compute api reboot method  https://review.openstack.org/12742722:48
*** andreykurilin__ has joined #openstack-nova22:48
*** annashen has quit IRC22:49
*** annashen has joined #openstack-nova22:51
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py  https://review.openstack.org/18796822:52
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Switch to oslo.reports  https://review.openstack.org/19555822:54
*** otter768 has joined #openstack-nova22:56
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Switch to oslo.reports  https://review.openstack.org/19555822:57
*** apuimedo has joined #openstack-nova22:57
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Switch to oslo.reports  https://review.openstack.org/19555822:57
*** diogogmt has quit IRC22:58
openstackgerritTracy Jones proposed openstack/nova: Cleanup of compute api reboot method  https://review.openstack.org/12742722:58
*** otter768 has quit IRC23:01
*** amotoki has joined #openstack-nova23:02
*** tongli has quit IRC23:04
*** ctrath has quit IRC23:05
*** amotoki has quit IRC23:07
*** hemna is now known as hemnafk23:11
*** mtanino_ has joined #openstack-nova23:11
*** mtanino has quit IRC23:11
*** coolsvap is now known as coolsvap|afk23:14
*** tpatil has quit IRC23:17
*** salv-orlando has quit IRC23:18
claudiub|2mikal: 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-nova23:21
*** yjiang5 has joined #openstack-nova23:23
*** mutoulbj has joined #openstack-nova23:24
*** mutoulbj has quit IRC23:28
*** tongli has joined #openstack-nova23:29
*** tpeoples has joined #openstack-nova23:34
*** tongli has quit IRC23:34
*** annashen has quit IRC23:34
*** tongli has joined #openstack-nova23:34
*** mtanino_ has quit IRC23:36
*** patrickeast has quit IRC23:37
*** mtanino has joined #openstack-nova23:37
*** mgarza_ has quit IRC23:37
*** Longgeek has joined #openstack-nova23:38
*** openstack has joined #openstack-nova23:39
*** VW__ has quit IRC23:41
*** nelsnelson has quit IRC23:42
*** VW_ has quit IRC23:43
openstackgerritTracy Jones proposed openstack/nova: WIP - convert direct db calls to objects  https://review.openstack.org/19485123:44
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Raise exception when API does't support microversions  https://review.openstack.org/18881623:49
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: discover supported versions to choose latest one  https://review.openstack.org/18428123:49
*** achanda has quit IRC23:49
*** Sukhdev has quit IRC23:51
*** Longgeek has quit IRC23:52
*** Sukhdev has joined #openstack-nova23:52
*** IanGovett has quit IRC23:53
*** tongli has quit IRC23:53
dansmithcburgess: around by chance?23:53
*** tongli has joined #openstack-nova23:54
*** tongli has quit IRC23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!