Thursday, 2016-01-07

mriedemjroll: yikes, so that's just a compat shim right?00:02
*** zenoway has joined #openstack-nova00:02
mriedemnvm, just a partial revert w/o the misleading py26 note00:04
jrollmriedem: it's a compat shim that someone thought was 2.6 and dropped, and I put it back00:04
*** baoli has quit IRC00:04
jrollyeah00:04
mriedemyeah, it did say py26 before00:04
mriedemthat's why it was removed00:04
jrollright00:04
mriedemyeah let me see what our change history looks like for a release00:04
jrollI was wondering if that changed py2.5 -> py2.6 and if that was new for 2.600:04
mriedemwe were waiting for some changes before the break to do a major version00:04
jrollcool, no rush but sooner is better :)00:05
mriedemoh snap, i guess someone did a 3.0.0 release00:05
*** zenoway has quit IRC00:07
*** dims_ has quit IRC00:07
mriedemok, i'll request a 3.1.100:07
jrollthanks much00:08
mriedemactually 3.2.0, but semantics00:08
mriedemthere was a reqs bump fecf83300:08
*** lihg has joined #openstack-nova00:10
*** lihg_ has joined #openstack-nova00:10
*** ducttape_ has quit IRC00:11
oomichimriedem: hi, can you take a look at a liberty patch (https://review.openstack.org/#/c/263567/) ?00:12
*** jwcroppe has quit IRC00:13
*** nic1 has quit IRC00:14
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out  https://review.openstack.org/26429700:15
*** otter768 has joined #openstack-nova00:16
*** renatoarmani has joined #openstack-nova00:16
*** sacharya has joined #openstack-nova00:16
*** Sukhdev has joined #openstack-nova00:17
*** sacharya has quit IRC00:21
*** otter768 has quit IRC00:21
*** pratikmallya has quit IRC00:22
*** baoli has joined #openstack-nova00:25
*** doug-fish has quit IRC00:26
*** haomaiwa_ has joined #openstack-nova00:33
*** dedery has quit IRC00:34
*** suro-patz has quit IRC00:35
*** haomaiwa_ has quit IRC00:38
*** zhenq has quit IRC00:38
*** zenoway has joined #openstack-nova00:38
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out  https://review.openstack.org/26429700:40
*** otter768 has joined #openstack-nova00:41
mriedemoomichi: yeah in a bit00:42
*** jwcroppe has joined #openstack-nova00:42
*** zhangjn has joined #openstack-nova00:43
*** zenoway has quit IRC00:43
oomichimriedem: thanks. btw the release schedule of stable branch seems to be changed since liberty. so when is nova's liberty release?00:43
mriedemoomichi: as soon as we can get through the open stable/liberty reviews00:44
mriedemhttps://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/liberty00:44
mriedemor at least the things we want to get into the release00:45
mriedemi want to get this in https://review.openstack.org/#/c/258695/ since it fixes a gate issue00:45
oomichimriedem: I see, thanks for your explanation :)00:46
*** bklei has joined #openstack-nova00:53
*** zhangjn has quit IRC00:55
*** dims has joined #openstack-nova00:56
*** baoli has quit IRC00:57
mriedemoomichi: +W00:57
*** thorst has joined #openstack-nova00:58
oomichimriedem: thanks again :)00:59
*** zz_dimtruck is now known as dimtruck00:59
*** jaypipes has quit IRC00:59
mriedemjohnthetubaguy: garyk: claudiub: dansmith: mikal: i'm trying to go through the stable/liberty reviews to prep for a 12.0.1 release, there are a few with +2s here https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/liberty01:01
mriedemwould appreciate reviews this week to wittle the list down to what we think we should release01:02
*** zenoway has joined #openstack-nova01:02
claudiubwill do in the morning, it's 3 am atm here. :)01:03
*** eil397 has quit IRC01:03
*** thorst_ has joined #openstack-nova01:04
mriedemclaudiub: yeah, np, just an fyi01:05
mriedemclaudiub: do you by chance live in the part of romania that used to be transylvania? :)01:06
*** zhangjn has joined #openstack-nova01:06
*** zenoway has quit IRC01:06
claudiubmriedem: well, romania is split in several larger regions, one of which is still transylvania :)01:06
*** thorst has quit IRC01:06
mriedemso do you live in that one?01:07
claudiubmriedem: but no, I would hibernate until spring if I were there. :)01:07
mriedemoh, ok01:07
*** EinstCrazy has joined #openstack-nova01:07
mriedemi was just trying to put together the 3am thing...01:07
*** Jeffrey4l has joined #openstack-nova01:07
claudiubwell, I am a bit of an night owl, but not a transforming bat. :P01:08
*** renatoarmani has quit IRC01:09
*** bklei has quit IRC01:09
*** ducttape_ has joined #openstack-nova01:12
*** zhangyufei has joined #openstack-nova01:12
*** boris-42 has quit IRC01:13
*** tjones has quit IRC01:15
*** ducttape_ has quit IRC01:17
*** bklei has joined #openstack-nova01:20
*** Sukhdev has quit IRC01:25
mriedemgaryk: can we abandon this https://review.openstack.org/#/c/238208/ to cleanup the stable/liberty review queue?01:26
openstackgerritMatt Riedemann proposed openstack/nova: Convert nova.tests.unit.image.fake.stub_out_image_service to use stub_out  https://review.openstack.org/26439301:29
openstackgerritZhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API  https://review.openstack.org/24898901:31
*** angdraug has quit IRC01:32
*** angdraug has joined #openstack-nova01:33
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out  https://review.openstack.org/26429701:35
*** asselin has quit IRC01:36
*** angdraug has quit IRC01:36
*** zenoway has joined #openstack-nova01:38
*** Sukhdev has joined #openstack-nova01:38
*** zenoway has quit IRC01:42
*** bklei has quit IRC01:43
*** Sukhdev has quit IRC01:43
*** mtanino has quit IRC01:47
openstackgerritGiridhar Jayavelu proposed openstack/nova: pci: avoid parsing whitelist repeatedly  https://review.openstack.org/26451001:50
openstackgerritZhenyu Zheng proposed openstack/nova: config options: Centralise floating ip options  https://review.openstack.org/25708701:53
*** dimtruck is now known as zz_dimtruck01:54
*** cdlhh has joined #openstack-nova01:57
*** lyanchih has joined #openstack-nova01:57
*** ssurana has quit IRC02:00
*** zhangyufei has quit IRC02:02
*** ljxiash has joined #openstack-nova02:04
*** zhangyufei has joined #openstack-nova02:05
*** davideagnello has quit IRC02:07
*** mriedem has quit IRC02:13
*** s-moriya has quit IRC02:13
*** zenoway has joined #openstack-nova02:13
*** Guest80202 has joined #openstack-nova02:17
*** Guest80202 has quit IRC02:17
*** zenoway has quit IRC02:18
*** zhangjn has quit IRC02:19
*** ducttape_ has joined #openstack-nova02:20
*** cdlhh has quit IRC02:20
*** cdlhh has joined #openstack-nova02:21
*** cdlhh has joined #openstack-nova02:21
*** bklei has joined #openstack-nova02:23
*** salv-orl_ has quit IRC02:23
*** salv-orlando has joined #openstack-nova02:23
*** ljxiash has quit IRC02:23
*** ljxiash has joined #openstack-nova02:24
*** zhangyufei has quit IRC02:24
*** haomaiwang has joined #openstack-nova02:26
*** claudiub has quit IRC02:27
*** fawadkhaliq has joined #openstack-nova02:28
*** haomaiw__ has joined #openstack-nova02:29
openstackgerrithuan proposed openstack/nova: XenAPI: Fix VIF plug and unplug problem  https://review.openstack.org/25402202:30
*** haomaiwang has quit IRC02:31
*** houming has joined #openstack-nova02:33
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out  https://review.openstack.org/26429702:33
*** szhsong has joined #openstack-nova02:34
*** lyanchih has quit IRC02:35
oomichialex_xu: morning, can we push https://review.openstack.org/#/c/259319 now? you are drafter of the spec, so you are familer with this feature ;)02:36
*** zhangjn has joined #openstack-nova02:38
*** cdlhh has quit IRC02:38
alex_xuoomichi: ah, thanks for the review, I will check that. :)02:38
*** zhangyufei has joined #openstack-nova02:39
alex_xuoomichi: there are dependence patches also02:39
alex_xuprobably I need choice a time go over all of them02:39
alex_xus/choose/choice02:39
oomichialex_xu: microversions patches are lined up on the queue and that is easiest I feel02:39
alex_xuoomichi: yup :)02:40
oomichialex_xu: I already +2 for the other microversion patches, but they will be need to be rebased due to version conflict(2.15)02:40
alex_xuoomichi: yea, I noticed those patches. after the contributors rebase them, I will try to review first.02:41
alex_xuoomichi: btw, there are some microversion doc write up https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/complete-todo-in-api-concept-doc :)02:42
*** baoli has joined #openstack-nova02:42
openstackgerrithuan proposed openstack/nova: XenAPI:Resolve Nova/Neutron race condition  https://review.openstack.org/24112702:43
oomichialex_xu: ok, will check them :)02:45
*** baoli has quit IRC02:46
*** baoli has joined #openstack-nova02:47
*** lyanchih has joined #openstack-nova02:48
*** Fdaisuke has joined #openstack-nova02:48
*** baoli has quit IRC02:48
*** tjones has joined #openstack-nova02:49
*** qianli has quit IRC02:52
*** qianli has joined #openstack-nova02:53
*** baoli has joined #openstack-nova02:54
*** baoli has quit IRC02:54
*** baoli has joined #openstack-nova02:55
alex_xuoomichi: thanks :)02:56
*** bklei has quit IRC02:58
*** dims has quit IRC02:58
*** mragupat has joined #openstack-nova02:58
*** bklei has joined #openstack-nova02:59
openstackgerritTang Chen proposed openstack/nova: Add a REST API to trigger crash dump in an instance.  https://review.openstack.org/20261702:59
*** haomaiw__ has quit IRC03:01
*** lyanchih has quit IRC03:01
*** bklei has quit IRC03:01
*** haomaiwang has joined #openstack-nova03:01
*** bklei has joined #openstack-nova03:02
*** andreykurilin__ has quit IRC03:02
*** boris-42 has joined #openstack-nova03:03
*** bklei_ has joined #openstack-nova03:03
*** vladikr has joined #openstack-nova03:05
*** vilobhmm11 has quit IRC03:05
*** bklei has quit IRC03:06
*** lihg has quit IRC03:06
*** lihg_ has quit IRC03:06
*** thorst_ has quit IRC03:09
*** thorst has joined #openstack-nova03:09
*** links has joined #openstack-nova03:10
*** links has quit IRC03:10
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out  https://review.openstack.org/26429703:11
openstackgerritRushi Agrawal proposed openstack/nova: [WIP]Remove mox completely from nova/tests/unit/objects/test_instance.py  https://review.openstack.org/26429203:11
*** baoli has quit IRC03:12
*** edmondsw has quit IRC03:12
*** links has joined #openstack-nova03:12
openstackgerritZhenyu Zheng proposed openstack/nova: config options: Centralise floating ip options  https://review.openstack.org/25708703:14
*** sacharya has joined #openstack-nova03:14
*** unicell1 has quit IRC03:16
*** thorst has quit IRC03:18
*** bklei_ has quit IRC03:21
*** bklei has joined #openstack-nova03:22
*** sacharya has quit IRC03:23
*** salv-orlando has quit IRC03:26
*** salv-orlando has joined #openstack-nova03:27
*** bklei has quit IRC03:27
*** sacharya has joined #openstack-nova03:34
*** yamahata has quit IRC03:36
*** lihg_ has joined #openstack-nova03:36
*** lihg has joined #openstack-nova03:36
*** Jeffrey4l has quit IRC03:37
*** ljxiash has quit IRC03:38
*** ljxiash has joined #openstack-nova03:39
*** zhangjn has quit IRC03:40
*** sacharya has quit IRC03:40
*** lyanchih has joined #openstack-nova03:42
*** ljxiash has quit IRC03:43
*** vishwanathj has joined #openstack-nova03:49
*** mdrabe has joined #openstack-nova03:49
*** ducttape_ has quit IRC03:49
*** fawadkhaliq has quit IRC03:49
*** vishwana_ has quit IRC03:51
*** zhangyufei has quit IRC03:52
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out  https://review.openstack.org/26429703:54
*** gongysh_ has quit IRC03:57
*** zz_dimtruck is now known as dimtruck03:58
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-nova04:01
*** sacharya has joined #openstack-nova04:02
*** thorst has joined #openstack-nova04:02
*** unicell has joined #openstack-nova04:05
*** lyanchih has quit IRC04:08
*** thorst has quit IRC04:08
*** mragupat has quit IRC04:08
*** thorst has joined #openstack-nova04:08
*** salv-orl_ has joined #openstack-nova04:09
*** gongysh_ has joined #openstack-nova04:10
*** salv-orlando has quit IRC04:11
*** ljxiash has joined #openstack-nova04:11
*** tjones has quit IRC04:12
*** vilobhmm11 has joined #openstack-nova04:13
*** shyama has joined #openstack-nova04:16
*** thorst has quit IRC04:17
*** ljxiash has quit IRC04:17
*** bklei has joined #openstack-nova04:18
*** Jeffrey4l has joined #openstack-nova04:19
*** coolsvap|away is now known as coolsvap04:21
*** bklei has quit IRC04:23
*** lykinsbd has joined #openstack-nova04:24
*** lykinsbd has quit IRC04:26
*** otter768 has quit IRC04:28
*** vilobhmm11 has quit IRC04:31
*** ljxiash has joined #openstack-nova04:35
*** ljxiash_ has joined #openstack-nova04:43
*** ljxiash has quit IRC04:44
*** jaosorior has joined #openstack-nova04:45
*** lihg_ has quit IRC04:46
*** lihg has quit IRC04:46
*** annegentle has joined #openstack-nova04:49
*** annegentle has quit IRC04:49
*** dave-mccowan has quit IRC04:50
*** gjayavelu has joined #openstack-nova04:51
*** shyama has quit IRC04:55
*** sudipto has joined #openstack-nova04:56
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/26444104:56
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements  https://review.openstack.org/26440504:57
*** tjones has joined #openstack-nova04:58
*** shyama has joined #openstack-nova04:58
*** haomaiwang has quit IRC05:01
*** haomaiwa_ has joined #openstack-nova05:01
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out  https://review.openstack.org/26429705:09
*** lyanchih has joined #openstack-nova05:11
*** smoriya_ has joined #openstack-nova05:13
*** dimtruck is now known as zz_dimtruck05:14
*** EinstCrazy has quit IRC05:14
*** kevinbenton has quit IRC05:14
*** kevinbenton has joined #openstack-nova05:14
*** EinstCrazy has joined #openstack-nova05:14
*** thorst has joined #openstack-nova05:15
*** liusheng has quit IRC05:16
*** gongysh_ has quit IRC05:16
*** liusheng has joined #openstack-nova05:17
*** zhangyufei has joined #openstack-nova05:18
*** lyanchih has quit IRC05:18
*** EinstCra_ has joined #openstack-nova05:20
*** fawadkhaliq has joined #openstack-nova05:20
*** mragupat has joined #openstack-nova05:21
*** DinaBelova has quit IRC05:21
*** thorst has quit IRC05:21
*** DinaBelova has joined #openstack-nova05:22
*** shyama has quit IRC05:23
*** zhangjn has joined #openstack-nova05:23
*** EinstCrazy has quit IRC05:23
*** baoli has joined #openstack-nova05:23
*** baoli has quit IRC05:28
*** kevinbenton has quit IRC05:29
*** zhangjn_ has joined #openstack-nova05:34
*** gongysh_ has joined #openstack-nova05:35
*** zhangjn has quit IRC05:36
*** zhangjn_ has quit IRC05:38
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out  https://review.openstack.org/26429705:40
*** chhavi has joined #openstack-nova05:41
*** mdrabe has quit IRC05:45
openstackgerritSudipta Biswas proposed openstack/nova: libvirt: Add ppc to supported arch for NUMA  https://review.openstack.org/24576505:45
*** zhangjn has joined #openstack-nova05:45
*** chhavi has quit IRC05:46
*** zhangjn has quit IRC05:51
*** moshele has joined #openstack-nova05:54
*** zhangjn has joined #openstack-nova05:54
*** zhangjn has quit IRC05:55
*** lyanchih has joined #openstack-nova05:58
*** tjones has quit IRC06:00
*** zhangjn has joined #openstack-nova06:00
*** haomaiwa_ has quit IRC06:01
*** zhangjn has quit IRC06:01
*** 7JTABHHZB has joined #openstack-nova06:01
*** zhangjn has joined #openstack-nova06:02
*** shyama has joined #openstack-nova06:04
*** bklei has joined #openstack-nova06:06
*** achinm has joined #openstack-nova06:07
openstackgerritlyanchih proposed openstack/nova: libvirt - Flavor's quota instance resource not observed for non-ephemeral storage  https://review.openstack.org/20101906:07
*** chhavi has joined #openstack-nova06:08
*** salv-orl_ has quit IRC06:08
*** salv-orlando has joined #openstack-nova06:08
*** sacharya has quit IRC06:08
*** bklei has quit IRC06:10
*** markvoelker has quit IRC06:11
*** kotra03 has joined #openstack-nova06:14
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata  https://review.openstack.org/25219706:14
*** vilobhmm11 has joined #openstack-nova06:15
*** thorst has joined #openstack-nova06:20
*** kotra03 has quit IRC06:22
*** chhavi has quit IRC06:22
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out  https://review.openstack.org/26429706:23
*** zhangjn has quit IRC06:25
*** thorst has quit IRC06:26
*** otter768 has joined #openstack-nova06:29
openstackgerritVladik Romanovsky proposed openstack/nova: pci: do not filter out any SRIOV Physical Functions  https://review.openstack.org/24901606:30
openstackgerritVladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment  https://review.openstack.org/24901806:30
openstackgerritVladik Romanovsky proposed openstack/nova: pci: adding support to specify a device_type in pci requests  https://review.openstack.org/25925406:30
openstackgerritVladik Romanovsky proposed openstack/nova: objects: adding a parent_addr field to the PciDevice object  https://review.openstack.org/24901506:30
*** sacharya has joined #openstack-nova06:30
*** mragupat has quit IRC06:30
*** garyk has quit IRC06:32
*** garyk has joined #openstack-nova06:33
*** otter768 has quit IRC06:33
*** nkrinner has joined #openstack-nova06:35
*** zhangjn has joined #openstack-nova06:35
*** zhangjn has quit IRC06:36
*** _diana_1 has quit IRC06:36
*** zhangjn has joined #openstack-nova06:37
*** agireud has quit IRC06:38
*** PsionTheory has quit IRC06:38
*** sacharya has quit IRC06:38
*** ankit_ag has joined #openstack-nova06:38
*** agireud has joined #openstack-nova06:44
*** zhangjn has quit IRC06:44
*** tongli has quit IRC06:51
*** unicell1 has joined #openstack-nova06:51
*** unicell has quit IRC06:52
*** irenab has quit IRC06:54
*** mdbooth has quit IRC06:54
*** mdbooth has joined #openstack-nova06:55
*** coolsvap is now known as coolsvap|away06:56
*** mjura has joined #openstack-nova06:58
*** tongli has joined #openstack-nova06:58
*** 7JTABHHZB has quit IRC07:01
*** vilobhmm11 has quit IRC07:02
openstackgerritJianghua Wang proposed openstack/nova: xenapi: OVS agent updates the wrong port when using XenServer + Neutron  https://review.openstack.org/24284607:02
*** zhangjn has joined #openstack-nova07:04
*** qianli has quit IRC07:06
*** qianli has joined #openstack-nova07:07
*** haomaiwang has joined #openstack-nova07:07
*** rotbeard has joined #openstack-nova07:09
*** ankit_ag has quit IRC07:10
*** EinstCrazy has joined #openstack-nova07:11
*** EinstCra_ has quit IRC07:11
*** markvoelker has joined #openstack-nova07:11
openstackgerritesracelik proposed openstack/nova: Config options: centralize section "hyperv"  https://review.openstack.org/25512407:12
*** paul-carlton2 has joined #openstack-nova07:12
*** ankit_ag has joined #openstack-nova07:15
*** markvoelker has quit IRC07:16
*** kevinbenton has joined #openstack-nova07:19
*** kevinbenton has quit IRC07:20
*** kevinbenton has joined #openstack-nova07:20
*** irenab has joined #openstack-nova07:20
*** vilobhmm11 has joined #openstack-nova07:25
*** thorst has joined #openstack-nova07:25
*** kevinbenton has quit IRC07:26
*** vilobhmm11 has quit IRC07:26
*** belmoreira has joined #openstack-nova07:26
*** EinstCrazy has quit IRC07:29
*** kevinbenton has joined #openstack-nova07:29
*** EinstCrazy has joined #openstack-nova07:29
*** thorst has quit IRC07:32
*** chhavi has joined #openstack-nova07:33
*** sacharya has joined #openstack-nova07:38
*** vilobhmm11 has joined #openstack-nova07:38
*** sudipto has quit IRC07:39
openstackgerritTang Chen proposed openstack/nova: Add a REST API to trigger crash dump in an instance.  https://review.openstack.org/20261707:41
*** coolsvap|away is now known as coolsvap07:41
openstackgerritsandeep nandal proposed openstack/nova: Replaced range() and xrange() by six.moves.range()  https://review.openstack.org/26465207:42
*** sahid has joined #openstack-nova07:43
*** sacharya has quit IRC07:43
*** ljxiash_ has quit IRC07:44
*** deepakcs has joined #openstack-nova07:46
*** ljxiash has joined #openstack-nova07:49
*** lpetrut has joined #openstack-nova07:50
*** bklei has joined #openstack-nova07:54
*** rdopiera has joined #openstack-nova07:57
*** liverpooler has joined #openstack-nova07:58
*** bklei has quit IRC07:58
*** Jeffrey4l has quit IRC07:58
*** fawadkhaliq has quit IRC08:00
*** otter768 has joined #openstack-nova08:00
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-nova08:01
*** mpavone has joined #openstack-nova08:01
*** otter768_ has joined #openstack-nova08:02
*** zhangjn has quit IRC08:03
*** salv-orlando has quit IRC08:04
*** otter76__ has joined #openstack-nova08:04
*** otter768 has quit IRC08:04
*** salv-orlando has joined #openstack-nova08:04
*** mpavone has quit IRC08:05
*** zhangjn has joined #openstack-nova08:05
*** otter76__ has quit IRC08:06
*** otter768 has joined #openstack-nova08:06
*** paul-carlton2 has quit IRC08:06
*** otter768_ has quit IRC08:06
*** matrohon has joined #openstack-nova08:07
*** otter768_ has joined #openstack-nova08:08
*** otter768 has quit IRC08:08
*** otter768 has joined #openstack-nova08:10
*** otter768_ has quit IRC08:10
*** Jeffrey4l has joined #openstack-nova08:11
*** otter768_ has joined #openstack-nova08:12
*** otter768_ has quit IRC08:13
*** otter768_ has joined #openstack-nova08:14
*** otter768 has quit IRC08:14
*** otter768_ has quit IRC08:15
*** otter768 has joined #openstack-nova08:15
*** esracelik has joined #openstack-nova08:17
*** otter768_ has joined #openstack-nova08:17
*** otter768 has quit IRC08:17
*** otter768_ has quit IRC08:19
*** otter768 has joined #openstack-nova08:19
*** paul-carlton2 has joined #openstack-nova08:20
*** zakora has joined #openstack-nova08:21
*** jichen has joined #openstack-nova08:21
*** blaisebool has joined #openstack-nova08:21
*** otter768_ has joined #openstack-nova08:21
*** otter768 has quit IRC08:21
*** cfriesen__ has joined #openstack-nova08:22
*** otter768_ has quit IRC08:23
*** otter768 has joined #openstack-nova08:23
*** zakora has quit IRC08:23
*** otter768_ has joined #openstack-nova08:25
*** zakora has joined #openstack-nova08:25
*** otter76__ has joined #openstack-nova08:27
*** zakora has quit IRC08:28
*** otter768 has quit IRC08:28
*** otter768 has joined #openstack-nova08:29
*** zakora has joined #openstack-nova08:29
*** salv-orlando has quit IRC08:30
*** salv-orlando has joined #openstack-nova08:30
*** otter768_ has quit IRC08:30
*** otter768_ has joined #openstack-nova08:31
*** otter76__ has quit IRC08:31
*** thorst has joined #openstack-nova08:32
*** otter76__ has joined #openstack-nova08:33
*** otter768 has quit IRC08:33
*** browne has quit IRC08:34
*** otter768 has joined #openstack-nova08:34
*** otter76__ has quit IRC08:35
*** otter768_ has quit IRC08:36
openstackgerritlyanchih proposed openstack/nova: Listing volumes does not provide a link to next element  https://review.openstack.org/23799908:36
*** otter768_ has joined #openstack-nova08:36
*** otter768 has quit IRC08:37
*** thorst has quit IRC08:37
*** otter768_ has quit IRC08:38
*** otter768 has joined #openstack-nova08:38
*** markus_z has joined #openstack-nova08:40
*** otter768 has quit IRC08:40
*** otter768 has joined #openstack-nova08:40
*** otter768_ has joined #openstack-nova08:42
*** otter76__ has joined #openstack-nova08:44
*** otter768 has quit IRC08:44
*** jlanoux has joined #openstack-nova08:46
*** otter76__ has quit IRC08:46
*** zhihaisong has joined #openstack-nova08:46
*** otter768 has joined #openstack-nova08:46
*** vilobhmm11 has quit IRC08:46
*** otter768_ has quit IRC08:47
*** zhihais has joined #openstack-nova08:47
*** mkoderer_cloud has joined #openstack-nova08:48
*** otter768_ has joined #openstack-nova08:48
*** otter768 has quit IRC08:48
*** ijw has quit IRC08:49
*** szhsong has quit IRC08:49
*** otter768 has joined #openstack-nova08:50
*** zhihaisong has quit IRC08:51
*** zakora has quit IRC08:51
*** otter76__ has joined #openstack-nova08:52
*** otter768 has quit IRC08:52
*** otter768_ has quit IRC08:52
*** zakora has joined #openstack-nova08:53
*** otter76__ has quit IRC08:53
*** otter768 has joined #openstack-nova08:53
*** otter768_ has joined #openstack-nova08:56
openstackgerritlyanchih proposed openstack/nova: libvirt - Flavor's quota instance resource not observed for non-ephemeral storage  https://review.openstack.org/20101908:56
*** paul-carlton2 has quit IRC08:57
*** chhavi has quit IRC08:57
*** otter768 has quit IRC08:57
*** otter76__ has joined #openstack-nova08:58
*** markus_z has quit IRC08:58
*** otter768 has joined #openstack-nova08:59
*** otter76__ has quit IRC08:59
*** haomaiwang has quit IRC09:01
*** otter768_ has quit IRC09:01
*** otter768_ has joined #openstack-nova09:01
*** haomaiwang has joined #openstack-nova09:01
openstackgerritStephen Finucane proposed openstack/nova: Better document the osapi_max_limit config option  https://review.openstack.org/23321209:02
*** markus_z has joined #openstack-nova09:02
*** sfinucan has joined #openstack-nova09:03
*** otter76__ has joined #openstack-nova09:03
*** otter768_ has quit IRC09:03
*** otter768 has quit IRC09:04
bauzasgood morning Nova09:05
*** otter768 has joined #openstack-nova09:05
*** otter76__ has quit IRC09:05
* johnthetubaguy says "good mooorning" in his best British policeman voice09:06
*** lpetrut has quit IRC09:06
*** otter768_ has joined #openstack-nova09:07
*** haomaiwang has quit IRC09:08
*** otter76__ has joined #openstack-nova09:09
*** otter768_ has quit IRC09:09
*** haomaiwang has joined #openstack-nova09:09
*** otter768 has quit IRC09:10
*** jogo has quit IRC09:10
*** otter768 has joined #openstack-nova09:11
*** otter76__ has quit IRC09:11
*** markvoelker has joined #openstack-nova09:12
*** otter768 has quit IRC09:13
*** otter768_ has joined #openstack-nova09:13
*** otter768 has joined #openstack-nova09:14
*** otter768_ has quit IRC09:15
johnthetubaguylxsli: did we get that patch merged OK now?09:15
*** klkumar has joined #openstack-nova09:16
*** otter768 has quit IRC09:16
*** otter768 has joined #openstack-nova09:17
*** markvoelker has quit IRC09:17
*** jaosorior has quit IRC09:17
*** otter768 has quit IRC09:18
*** otter768 has joined #openstack-nova09:18
*** otter768_ has joined #openstack-nova09:20
*** otter768 has quit IRC09:20
*** yassine__ has joined #openstack-nova09:20
*** otter768_ has quit IRC09:22
*** otter768 has joined #openstack-nova09:22
*** otter768 has quit IRC09:24
*** otter768_ has joined #openstack-nova09:24
*** browne has joined #openstack-nova09:25
*** otter768_ has quit IRC09:25
*** otter768 has joined #openstack-nova09:26
*** browne has quit IRC09:28
*** otter768_ has joined #openstack-nova09:28
*** otter768 has quit IRC09:28
*** rk4n has joined #openstack-nova09:28
openstackgerritMichal Pryc proposed openstack/nova: Implements proper UUID format for compute/test_stats*  https://review.openstack.org/26468509:29
*** otter768 has joined #openstack-nova09:30
*** otter768_ has quit IRC09:30
bauzasjohnthetubaguy: heh09:30
bauzassorry, I'm actually super busy with http://snowcamp.io/2016/fr/09:30
*** rk4n has quit IRC09:30
bauzasdamn09:31
johnthetubaguybauzas: ah, cool beans09:31
openstackgerritsahid proposed openstack/nova: hardware: check whether realtime capable in API  https://review.openstack.org/26416509:31
openstackgerritsahid proposed openstack/nova: libvirt: add policy check to avoid using qga in realtime context  https://review.openstack.org/24758509:31
openstackgerritsahid proposed openstack/nova: conductor: initialize request_spec to handle expected exception  https://review.openstack.org/25801109:31
bauzasjust 3 days before our midcycle...09:31
openstackgerritsahid proposed openstack/nova: libvirt: add realtime support  https://review.openstack.org/19756909:31
*** oomichi has quit IRC09:32
*** otter768_ has joined #openstack-nova09:32
*** jaosorior has joined #openstack-nova09:32
*** daemontool has joined #openstack-nova09:32
*** otter76__ has joined #openstack-nova09:34
*** otter768 has quit IRC09:34
*** derekh has joined #openstack-nova09:35
*** otter768 has joined #openstack-nova09:35
*** haomaiwang has quit IRC09:35
*** otter76__ has quit IRC09:35
*** thorst has joined #openstack-nova09:36
*** otter768_ has quit IRC09:36
*** jogo has joined #openstack-nova09:37
*** otter768_ has joined #openstack-nova09:37
swamireddyHI09:38
gjayavelujohnthetubaguy: Hi, when you get chance can you please take a look at https://review.openstack.org/#/c/204219/?09:38
swamireddyI want to move a VM from one compute node to other node. Can I use EVACUATE command?09:38
gjayavelujohnthetubaguy: you expressed concerns about release notes and code merge status.  were you expecting the release notes done by m-1?09:39
*** otter768_ has quit IRC09:39
*** otter76__ has joined #openstack-nova09:39
johnthetubaguyswamireddy: we have a new doc on that, one sec09:39
*** sacharya has joined #openstack-nova09:40
johnthetubaguyswamireddy: in summary its probably a bad idea, but here is why: http://developer.openstack.org/api-guide/compute/server_concepts.html#moving-servers09:40
*** otter768 has quit IRC09:40
johnthetubaguygjayavelu: just looking at your patch, not sure I get your question, we should add the release notes as we go, and m-1 is released already09:41
*** jistr has joined #openstack-nova09:41
*** otter76__ has quit IRC09:41
*** otter768 has joined #openstack-nova09:41
*** yamahata has joined #openstack-nova09:41
gjayavelujohnthetubaguy: sure. it was added as a last patch in that set09:42
*** thorst has quit IRC09:42
johnthetubaguygjayavelu: ah, I see what you mean now, I was thinking that patch was merged for some reason, no idea why...09:42
gjayavelujohnthetubaguy: oh ok.09:42
johnthetubaguygjayavelu: yeah, last patch of the series is the best way to do it09:42
*** bklei has joined #openstack-nova09:42
*** zhangjn has quit IRC09:42
*** otter768_ has joined #openstack-nova09:43
johnthetubaguygjayavelu: I am not sure it has the dependency relationship I was expecting, the release notes patch that is, but it should be merged after the feature, so that all looks good09:43
*** otter768 has quit IRC09:43
openstackgerritBalazs Gibizer proposed openstack/nova: api: add soft-affinity policies for server groups  https://review.openstack.org/14752009:44
openstackgerritBalazs Gibizer proposed openstack/nova: scheduler: fill RequestSpec.instance_group.members  https://review.openstack.org/14827709:44
openstackgerritBalazs Gibizer proposed openstack/nova: scheduler: add soft-(anti-)affinity weighers  https://review.openstack.org/14751609:44
gjayavelujohnthetubaguy: let me check with garyk and update09:44
*** sacharya has quit IRC09:44
*** rajalakshmig has joined #openstack-nova09:44
*** paul-carlton has quit IRC09:45
*** haomaiwa_ has joined #openstack-nova09:45
*** otter768 has joined #openstack-nova09:45
johnthetubaguygjayavelu: sure thing, thanks for reaching out09:45
gjayavelujohnthetubaguy: thank you!09:45
*** bklei has quit IRC09:46
*** otter76__ has joined #openstack-nova09:47
*** otter768_ has quit IRC09:48
*** tongli has quit IRC09:48
openstackgerritStephen Finucane proposed openstack/nova: releasenotes: Note on CPU thread pinning support  https://review.openstack.org/26469309:49
*** otter768_ has joined #openstack-nova09:49
*** otter76__ has quit IRC09:49
*** rk4n has joined #openstack-nova09:49
*** otter768 has quit IRC09:50
*** blaisebool has quit IRC09:50
*** otter768_ has quit IRC09:51
*** otter768 has joined #openstack-nova09:51
*** paul-carlton has joined #openstack-nova09:51
*** yamahata has quit IRC09:53
*** otter768_ has joined #openstack-nova09:53
*** Jeffrey4l has quit IRC09:53
*** otter76__ has joined #openstack-nova09:55
*** otter768 has quit IRC09:56
*** otter768 has joined #openstack-nova09:56
*** otter76__ has quit IRC09:57
*** otter768_ has quit IRC09:57
*** ndipanov has joined #openstack-nova09:57
*** haomaiwa_ has quit IRC09:58
*** gjayavelu has quit IRC09:58
*** otter768_ has joined #openstack-nova09:58
*** zhangjn has joined #openstack-nova09:59
*** otter768_ has quit IRC10:00
*** otter768_ has joined #openstack-nova10:00
*** otter768 has quit IRC10:01
*** otter768 has joined #openstack-nova10:02
*** otter768_ has quit IRC10:02
*** otter768_ has joined #openstack-nova10:04
*** otter768 has quit IRC10:04
*** otter768_ has quit IRC10:05
*** otter768 has joined #openstack-nova10:06
*** EinstCrazy has quit IRC10:07
*** haomaiwang has joined #openstack-nova10:07
*** otter768 has quit IRC10:08
*** otter768 has joined #openstack-nova10:08
*** salv-orl_ has joined #openstack-nova10:08
*** Jeffrey4l has joined #openstack-nova10:09
*** otter768_ has joined #openstack-nova10:10
*** salv-orlando has quit IRC10:11
*** otter768_ has quit IRC10:11
*** otter768_ has joined #openstack-nova10:12
*** otter768 has quit IRC10:13
*** shyama has quit IRC10:13
*** otter768 has joined #openstack-nova10:14
*** otter768_ has quit IRC10:14
*** rk4n has quit IRC10:16
*** otter768_ has joined #openstack-nova10:16
*** otter768 has quit IRC10:16
lxslijohnthetubaguy: yes thank you! I have a new one now: https://review.openstack.org/#/c/264195/10:16
johnthetubaguylxsli: awesome, thank you!10:16
*** fawadkhaliq has joined #openstack-nova10:16
johnthetubaguylxsli: did we update the wiki to point to the published docs yet?10:16
lxsliNot yet, I was waiting for the devref to get published10:17
johnthetubaguygood call :)10:17
lxsliwhich it has now been, so I'll do that10:17
johnthetubaguythanks10:17
*** otter768_ has quit IRC10:17
*** otter768 has joined #openstack-nova10:17
*** zakora has quit IRC10:18
bauzaslxsli: +2d10:18
lxslisweet, thanks!10:19
*** otter768 has quit IRC10:19
*** ankit_ag has quit IRC10:19
openstackgerritlyanchih proposed openstack/nova: Listing volumes does not provide a link to next element  https://review.openstack.org/23799910:19
*** otter768 has joined #openstack-nova10:19
*** rk4n has joined #openstack-nova10:20
*** otter768_ has joined #openstack-nova10:21
*** otter768 has quit IRC10:21
*** zakora has joined #openstack-nova10:22
*** otter768_ has quit IRC10:23
*** otter768 has joined #openstack-nova10:23
gibijohnthetubaguy: hi! you approved previously https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/soft-affinity-for-server-group, I had to rebase it due to merge conflict. Could you reapprove?10:24
*** zenoway has joined #openstack-nova10:24
lxsliOK, replaced Nova/Process and Nova/Mentoring with links to devref. Updated links to those pages in main Nova wiki page with good links.10:24
*** otter768_ has joined #openstack-nova10:25
*** otter768 has quit IRC10:26
*** otter768_ has quit IRC10:27
*** otter768 has joined #openstack-nova10:27
*** dedery has joined #openstack-nova10:28
*** otter768 has quit IRC10:28
*** otter768 has joined #openstack-nova10:29
*** otter768_ has joined #openstack-nova10:31
*** otter768 has quit IRC10:31
*** haomaiwang has quit IRC10:32
*** jaosorior has quit IRC10:34
*** jaosorior has joined #openstack-nova10:34
*** otter768_ has quit IRC10:35
*** zenoway has quit IRC10:38
*** zenoway has joined #openstack-nova10:38
swamireddyjohnthetubaguy: HI...Thanks for the quick link...Yes  I also have a similar user case as mentioend in the doc...moving the compute nodes to other rack...so I will shtidown and move other rack and restart...10:38
swamireddyjohnthetubaguy: here, Can I use the evqcute command here...before current compute node shutdown? or the nova evacute command only works after current node is power-off state only??10:39
*** apoorvad has joined #openstack-nova10:40
johnthetubaguyswamireddy: the doc should describe that, was it unclear in there?10:40
*** thorst has joined #openstack-nova10:41
johnthetubaguyswamireddy: the answer is, the host must be powered off, and never powered back on, if you evacuate the VM10:41
johnthetubaguyswamireddy: what you are likley wanting is either live-migrate or migrate10:42
johnthetubaguyswamireddy: if you VM is on local storage, evacuate looses all that data10:42
johnthetubaguyswamireddy: if your VM was on shared storage, and the previous host starts back up, you might get two VMs talking to the same disk, leading to likely data corruption10:42
*** zenoway has quit IRC10:43
*** haomaiwang has joined #openstack-nova10:45
bauzasgibi: johnthetubaguy: given that https://review.openstack.org/#/c/147516/59/nova/scheduler/weights/affinity.py reads some object value, I would have preferred to have https://review.openstack.org/#/c/148277/64 first in the series but since both of them are now approved, we're all good10:46
bauzasgibi: johnthetubaguy: because reading some field that hasn't been set raises an exception, it makes the weigher not functional until the latter patch merges, but I'm maybe nitpicking10:47
*** thorst has quit IRC10:47
*** ccarmack1 has joined #openstack-nova10:48
*** lpetrut has joined #openstack-nova10:49
*** ccarmack has quit IRC10:49
*** cdent has joined #openstack-nova10:49
*** rk4n has quit IRC10:50
*** ociuhandu has quit IRC10:51
openstackgerritIldiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments  https://review.openstack.org/19313510:53
openstackgerritIldiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments  https://review.openstack.org/19313410:53
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach  https://review.openstack.org/15303810:53
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: virt: Pass host when call attach to Cinder  https://review.openstack.org/25627310:53
johnthetubaguybauzas: I agree, but I spotted that after the others has +Ws too10:53
bauzasanyway, we're good10:54
johnthetubaguyyeah, I think so10:54
*** rk4n has joined #openstack-nova10:56
*** haomaiwang has quit IRC10:57
ildikovjohnthetubaguy: hi, I have new patches up with checking the multi attach support in the virt layer ^^10:57
ildikovjohnthetubaguy: after some checks it seems that maybe even the boot_from_volume would not be an issue either at least from detach perspective, but I'm happy to block that if we feel unsafe in general10:59
*** lyanchih has quit IRC10:59
*** chhavi has joined #openstack-nova11:00
johnthetubaguyildikov: its the problem of not protecting against folks booting two instances from the same volume that worries me.11:00
*** rk4n has quit IRC11:00
*** rk4n has joined #openstack-nova11:01
johnthetubaguyildikov: a good API should be hard to use "dangerously"11:01
ildikovjohnthetubaguy: good point, I will block it to make sure, that's prolly better than checking whether the volume is already attached or not11:01
johnthetubaguyildikov: yeah, that check will always be a little racey, I feel11:01
ildikovjohnthetubaguy: yeap, my thinking too11:02
johnthetubaguyand things outside nova might attach to it anyways11:02
ildikovthat should be in the volume info, but anyway, better to block for now11:02
ildikovI will block that and the create_snapshot11:02
gibijohnthetubaguy, bauzas: thank for the approve.11:05
*** ljxiash has quit IRC11:06
*** ljxiash has joined #openstack-nova11:06
ildikovjohnthetubaguy: one quick question11:07
*** salv-orl_ has quit IRC11:07
*** salv-orlando has joined #openstack-nova11:08
*** haomaiwang has joined #openstack-nova11:08
*** rk4n has quit IRC11:09
*** rk4n has joined #openstack-nova11:09
*** ljxiash has quit IRC11:09
ildikovjohnthetubaguy: should that be not supported if the volume can be multiattached or if it already has attachments?11:09
*** ljxiash has joined #openstack-nova11:09
*** dims has joined #openstack-nova11:10
ildikovjohnthetubaguy: in either case should the API contain the check or better to do it in a lower layer?11:10
*** coolsvap is now known as coolsvap|away11:11
*** rk4n has quit IRC11:11
johnthetubaguyildikov: I think we just check for multi-attach=true, rather than looking at the list of attachements11:11
johnthetubaguyildikov: that feels much simpler, and less confusing and racey11:11
openstackgerritJohn Garbutt proposed openstack/nova: Add argument to support-matrix sphinx extension  https://review.openstack.org/26471711:12
openstackgerritJohn Garbutt proposed openstack/nova: Refactor _load_support_matrix  https://review.openstack.org/26471811:12
openstackgerritJohn Garbutt proposed openstack/nova: Add prototype feature classification matrix  https://review.openstack.org/26471911:12
ildikovjohnthetubaguy: ok, let's do that as a starting point11:12
ildikovjohnthetubaguy: I will need to call out to Cinder though again as we don't have the volume in that call chain nowhere11:12
johnthetubaguyildikov: we can just reject that down at the compute layer for now11:13
johnthetubaguyildikov: it really does feel like we need to add mutli-attach into the BDM11:13
*** markvoelker has joined #openstack-nova11:13
ildikovjohnthetubaguy: ok, good, I don't have to touch the legacy API code this way11:13
ildikovjohnthetubaguy: there's a BP up to change it aftere volume creation already11:14
johnthetubaguyildikov: so its not ideal, but I would take a check there over no check, others may not agree with me though11:14
*** zenoway has joined #openstack-nova11:14
johnthetubaguyildikov: in cinder?11:14
johnthetubaguyildikov: I think we need to make clear how dangerous that will be11:14
ildikovyes, in Cinder, I can add you to the review11:14
ildikovwell, if we always check it with Cinder that should be fine, but if we store that in the BDM then comes the trouble for sure11:15
*** sudipto has joined #openstack-nova11:16
sudiptobauzas, good morning and happy new year! I have a strange issue that i am facing with my patchset w.r.t Jenkins, is it a good time to ask?11:16
*** markvoelker has quit IRC11:18
*** zenoway has quit IRC11:18
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [python3] byte/string conversions and enable more tests  https://review.openstack.org/26104511:18
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [python3] Port more code to python3 and enable tests  https://review.openstack.org/26208311:18
*** ociuhandu has joined #openstack-nova11:19
bauzassudipto: ty, happy new year too11:19
bauzassure, ask11:19
*** lucas-dinner is now known as lucasagomes11:20
openstackgerritVladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment  https://review.openstack.org/24901811:20
openstackgerritJohn Garbutt proposed openstack/nova: Refactor _load_support_matrix  https://review.openstack.org/26471811:20
openstackgerritJohn Garbutt proposed openstack/nova: Add prototype feature classification matrix  https://review.openstack.org/26471911:20
*** haomaiwang has quit IRC11:21
sudiptobauzas, ok this is my patchset : https://review.openstack.org/#/c/245765/ the tests fail for a strange reason here: http://logs.openstack.org/65/245765/19/check/gate-nova-python27/65319c3/ this doesn't even point to the line where i have made the change...11:22
bauzassudipto: you probably had a race, lemme check11:22
sudiptobauzas, tried this around 5-6 times, but can't seem to get around it :(11:22
sudiptobauzas, it points to the code which i have changed, but the line numbers don't match.11:22
bauzassudipto: oh that11:23
bauzassudipto: lemme verify your code11:23
sudiptobauzas, ok thanks...11:23
bauzasdon't focus on the line numbers, because the job is rebasing your change on top of master11:23
*** haomaiwang has joined #openstack-nova11:24
sudiptobauzas, ok ...11:24
bauzassudipto: FYI http://logs.openstack.org/65/245765/19/check/gate-nova-python27/65319c3/console.html#_2016-01-07_05_48_09_36011:24
sudiptobauzas, yeah - but i can't seem to understand why the module doesn't have that attribute - it does work for other functions...not sure if i am overlooking something :(11:25
bauzasyeah that's a bit weird11:27
sudipto:(11:28
bauzassudipto: I would probably rebase my change locally and run again the test11:28
*** paul-carlton2 has joined #openstack-nova11:28
bauzassudipto: to test if it's reproducible11:28
bauzassudipto: I wonder if 'utils' object is not trampled11:29
bauzasby another running test11:29
bauzassince it's a shared object11:29
bauzasI mean the module reference11:29
*** bklei has joined #openstack-nova11:30
sudiptobauzas, ok...i rebased also a couple of times...but i observe this same thing...11:34
bauzaslocally?11:34
sudiptobauzas, yeah...11:34
sudiptobauzas, still it points to a different line number...11:34
bauzaslemme look11:34
sudiptobauzas, ok...11:35
*** bklei has quit IRC11:35
*** haomaiwang has quit IRC11:36
openstackgerritjichenjc proposed openstack/nova: Use stub_out and mock to remove mox: part 1  https://review.openstack.org/26415411:40
*** ljxiash has quit IRC11:40
*** ljxiash has joined #openstack-nova11:40
openstackgerritJohn Garbutt proposed openstack/nova: Add prototype feature classification matrix  https://review.openstack.org/26471911:41
*** 16WAAH8AU has joined #openstack-nova11:41
*** eglynn has joined #openstack-nova11:41
openstackgerritjichenjc proposed openstack/nova: Use stub_out and mock to remove mox: part 2  https://review.openstack.org/26435311:42
*** thorst has joined #openstack-nova11:45
*** ljxiash has quit IRC11:45
*** BobBall_AWOL is now known as BobBall11:45
openstackgerritEoghan Glynn proposed openstack/nova: doc: minor corrections to the API version docco  https://review.openstack.org/26473211:47
*** zhangjn has quit IRC11:49
*** femnad has joined #openstack-nova11:49
*** zenoway has joined #openstack-nova11:49
*** rfolco has joined #openstack-nova11:52
*** thorst has quit IRC11:52
openstackgerritBob Ball proposed openstack/nova: XenAPI: Cope with more Cinder backends  https://review.openstack.org/24478911:53
*** 16WAAH8AU has quit IRC11:53
*** vladikr has quit IRC11:54
*** omnipresent has joined #openstack-nova11:54
*** zenoway has quit IRC11:54
*** devananda has quit IRC11:54
*** haomaiwa_ has joined #openstack-nova11:54
*** devananda has joined #openstack-nova11:55
*** ankit_ag has joined #openstack-nova11:57
*** zhangjn has joined #openstack-nova11:58
*** zhangjn has quit IRC11:58
*** zhangjn has joined #openstack-nova11:59
*** zhangjn has quit IRC11:59
*** zhangjn has joined #openstack-nova11:59
*** apoorvad has quit IRC12:00
*** zhangjn has quit IRC12:00
*** haomaiwa_ has quit IRC12:01
*** zhangjn has joined #openstack-nova12:01
*** haomaiwang has joined #openstack-nova12:06
bauzassudipto: I'm not able to reproduce the issue locally when rebasing12:09
*** rk4n has joined #openstack-nova12:09
bauzasoh sec12:09
bauzasno, I got one12:09
*** gjayavelu has joined #openstack-nova12:10
*** omnipresent has quit IRC12:11
sudiptobauzas, oh - i was not able to reproduce it either...12:12
bauzassudipto: no, I can now reproduce anytime12:13
*** haomaiwang has quit IRC12:13
*** haomaiw__ has joined #openstack-nova12:13
bauzasand by looking at the problem, it's trivial :)12:13
bauzaslemme show you12:13
sudiptobauzas, great - sure.12:14
*** markvoelker has joined #openstack-nova12:14
*** gjayavelu has quit IRC12:14
bauzassudipto: I guess you probably badly rebased your patch12:15
bauzassudipto: how do you rebase ?12:15
bauzasgit rebase -i master ?12:15
sudiptobauzas, yeah12:15
*** zhangyufei has quit IRC12:15
bauzasokay, but it means you have to update your master, right?12:15
sudiptogit pull on the master?12:15
bauzasso, could you please12:15
bauzasgit fetch master12:15
bauzasgit rebase -i origin/master12:16
sudiptook sure.12:16
markus_zPaulMurray: Regarding the midcylce in Bristol, do you know if there are ops or customers around? It would help to justify my travel approval for my employer.12:16
*** apoorvad has joined #openstack-nova12:17
bauzassudipto: FYI, here is the change you missed https://review.openstack.org/#/c/203322/12:17
*** kaisers has quit IRC12:17
johnthetubaguymarkus_z: I many ways, I am an operator, if that helps. Although thats probably cheating12:17
bauzasand I *was* an operator, HTH :p12:18
markus_zalex_xu: Hi, we talked 2-3 months ago about bug tag ownership for "api". https://wiki.openstack.org/wiki/Nova/BugTriage#Step_2:_Triage_Tagged_Bugs  Do you see yourself in that position?12:18
*** markvoelker has quit IRC12:18
markus_zjohnthetubaguy: Hm, well, could be cheating could be not :) Rackspace-admin you say?12:19
bauzassudipto: oh I saw I made a mistake when giving you the git fetch12:19
sudiptobauzas, nah - i got that :)12:19
bauzasgit fetch *origin* master12:19
sudiptobauzas, running tox... hopefully i will see the error...12:19
bauzassudipto: and ?12:19
markus_zjohnthetubaguy: Didn't we have one from cern during the liberty meetup?12:20
bauzassudipto: ok12:20
bauzassudipto: you know you can only ask for a specific set of tests, right ?12:20
sudiptobauzas, yeah12:20
bauzasok12:20
bauzasmarkus_z: we can ask belmoreira if he comes12:20
johnthetubaguymarkus_z: possibly, yes, like you say, PaulMurray will have the full list12:20
*** rk4n has quit IRC12:21
bauzasby the way, I've been asked by email to give my CC expiry date, hopefully I'm not a paranoid guy12:21
markus_zbauzas: thanks, good to have a name there12:21
markus_zjohnthetubaguy: ok, I think I wait for Paul's response12:22
*** apoorvad has quit IRC12:22
openstackgerritJohn Garbutt proposed openstack/nova: Fix sphinx warnings from signature_utils  https://review.openstack.org/26474512:22
*** zenoway has joined #openstack-nova12:23
johnthetubaguysdague: bauzas: I have a prototype for the feature classification matrix, I am curious if this is what you folks were expecting it to be: http://docs-draft.openstack.org/19/264719/3/check/gate-nova-docs/08140ba//doc/build/html/feature_classification.html#prototype-feature-support-matrix12:25
bauzasI'll certainly take a look :)12:25
*** rk4n has joined #openstack-nova12:26
*** zenoway has quit IRC12:27
PaulMurraymarkus_z, I get what you are asking, but do you actually want to speak to people or just want to say "XYZ will also attend"?12:28
markus_zPaulMurray: It's sufficient to say that they are there.12:28
*** rk4n has quit IRC12:28
*** rk4n has joined #openstack-nova12:29
*** rk4n has quit IRC12:29
PaulMurraymarkus_z, well, untill 31st Jan I'm an operator :)12:30
markus_zPaulMurray: I guess that counts :)12:30
PaulMurrayactually, most of the nova team that supports our cloud offerrings are here if that helps12:31
*** dedery has quit IRC12:31
*** dedery has joined #openstack-nova12:31
PaulMurray(I mean HPE nova team)12:31
PaulMurrayand some ops/admins12:31
*** otter768 has joined #openstack-nova12:32
markmckashyap, you tested the block migration fallback?12:32
kashyapmarkmc: Yep12:32
kashyapmarkmc: https://review.openstack.org/#/c/171098/212:32
PaulMurrayBTW johnthetubaguy, we're also hosting the swift mid cycle a few weeks later12:32
kashyapmarkmc: Please see my last comment there.12:32
PaulMurrayjohnthetubaguy, are you aware that they do swag and evening events at their mid cycles !!12:33
sudiptobauzas, thanks a lot! I wonder how i missed it but really appreciate your help!12:33
johnthetubaguyPaulMurray: thats news to me12:33
bauzasnp12:33
*** kaisers has joined #openstack-nova12:33
sudiptobauzas, this file of mine was way off.12:33
markmckashyap, any idea what the actual issue was that the reporter hit?12:34
*** raildo-afk is now known as raildo12:34
markmckashyap, in https://bugs.launchpad.net/nova/+bug/144105412:34
openstackLaunchpad bug 1441054 in OpenStack Compute (nova) "live-migration --block-migrate fails with default libvirt flags" [Medium,In progress] - Assigned to Mathieu Rohon (mathieu-rohon)12:34
markus_zPaulMurray: I guess that helps me already a bit. Can you maybe also take a look at the registration list and spot some other company names?12:34
belmoreiramarkus_z, bauzas: Unfortunately I can't be in Bristol for the nova midcycle. I will attend the ops midcycle in Manchester though.12:34
kashyapmarkmc: First, looking at my notes here (from some previous tests) - https://kashyapc.fedorapeople.org/virt/libvirt-migration-tests-with-qemu+tcp.txt12:35
kashyap(Scroll down to the bottom in case you've clicked)12:35
*** dedery has quit IRC12:35
*** haomaiw__ has quit IRC12:35
markus_zbelmoreira: Ah, that's a pity. thanks12:36
kashyapmarkmc: It's likely that the reporter misphrased it.12:36
markmcwell, he has a traceback :)12:36
openstackgerritSudipta Biswas proposed openstack/nova: libvirt: Add ppc to supported arch for NUMA  https://review.openstack.org/24576512:36
bauzasbelmoreira: roger.12:36
*** otter768 has quit IRC12:37
bauzasbelmoreira: well, next time then12:37
markmckashyap, a traceback that went away when he stopped using TUNNELLED12:37
kashyapmarkmc: That behavior _does_ manifest, but with some other combination other than default.12:37
kashyapThat's what I meant12:37
bauzasthat ops meeting in Manchester can indeed limit the number of operators that 'd like to attend both (nova + ops)12:37
markmckashyap, which combination?12:37
kashyapmarkmc: Re-testing to find that out :-)12:37
markmckashyap, ok12:38
*** doug-fish has joined #openstack-nova12:38
kashyapmarkmc: I have to step out for an errand, will add a comment there in the bug once I have empirical evidence of what flags result in that behavior.12:38
*** haomaiwa_ has joined #openstack-nova12:38
*** apoorvad has joined #openstack-nova12:39
*** baoli has joined #openstack-nova12:41
*** rk4n has joined #openstack-nova12:41
*** apoorvad has quit IRC12:43
*** rk4n has quit IRC12:44
*** deepakcs has quit IRC12:44
openstackgerritSilvan Kaiser proposed openstack/nova: Switch Nova Quobyte volume driver to rely on externally provided client mount.  https://review.openstack.org/26475212:45
openstackgerritjichenjc proposed openstack/nova: Use stub_out and mock to remove mox:part 3  https://review.openstack.org/26475312:47
*** thorst has joined #openstack-nova12:47
*** zhangjn has quit IRC12:48
*** delattec has joined #openstack-nova12:50
*** haomaiwa_ has quit IRC12:51
*** drankis has joined #openstack-nova12:52
*** cdelatte has quit IRC12:52
*** gcb has quit IRC12:52
ankit_agjohnthetubaguy: hi12:53
*** Jeffrey4l has quit IRC12:53
*** dave-mccowan has joined #openstack-nova12:56
*** houming has quit IRC12:58
*** haomaiwang has joined #openstack-nova12:58
*** ljxiash has joined #openstack-nova12:59
*** smoriya_ has quit IRC12:59
*** haomaiwang has quit IRC13:01
openstackgerritChris Dent proposed openstack/nova: Propagate qemu-img errors to compute manager  https://review.openstack.org/26434913:01
*** claudiub has joined #openstack-nova13:01
*** haomaiwa_ has joined #openstack-nova13:01
*** Jeffrey4l has joined #openstack-nova13:02
chhavihemna: Saw ur comment on https://review.openstack.org/#/c/255595/7/cinder/volume/manager.py13:02
*** salv-orlando has quit IRC13:03
*** salv-orlando has joined #openstack-nova13:03
*** ljxiash has quit IRC13:03
chhavito be clear, does it mean, you will be moving the host_attaches as an util, so it can be called via terminate connection as well13:04
*** fawadkhaliq has quit IRC13:05
*** doug-fish has quit IRC13:05
*** fawadkhaliq has joined #openstack-nova13:05
*** smatzek_ has joined #openstack-nova13:06
*** gongysh has joined #openstack-nova13:06
*** zhangjn has joined #openstack-nova13:06
*** links has quit IRC13:07
*** cfriesen__ has quit IRC13:08
*** haomaiwa_ has quit IRC13:09
*** cfriesen__ has joined #openstack-nova13:09
*** ducttape_ has joined #openstack-nova13:10
*** rajalakshmig has quit IRC13:11
johnthetubaguyankit_ag: hi, can I help?13:12
ankit_agjohnthetubaguy: Yesterday abhishekk had a discussion with you over https://review.openstack.org/#/c/16379813:13
ankit_agwhere you mentioned about exposing the DOS attack by allowing multiple force-delete requests but this issue is already handled in the current PS by restricting delete calls if task state is force_deleting13:13
*** shyama has joined #openstack-nova13:14
johnthetubaguyankit_ag: OK, so you only get one delete, then one force delete?13:14
openstackgerritDirk Mueller proposed openstack/python-novaclient: Fix W503 line break before binary operator  https://review.openstack.org/26476013:14
ankit_agjohnthetubaguy: right13:14
ankit_agso that user has one more chance to delete the instance stuck in the deleting state13:14
johnthetubaguyankit_ag: will have to re-read the patch to comment more I think13:15
*** fawadkhaliq has quit IRC13:15
ankit_agjohnthetubaguy: sure thank you13:15
*** fawadkhaliq has joined #openstack-nova13:16
*** haomaiwa_ has joined #openstack-nova13:16
*** claudiub has quit IRC13:17
*** unicell1 has quit IRC13:17
*** unicell has joined #openstack-nova13:17
*** markvoelker has joined #openstack-nova13:18
*** haomaiwa_ has quit IRC13:18
openstackgerritJohn Garbutt proposed openstack/nova: Get support matrix notes from the ini file  https://review.openstack.org/26476313:19
*** huats has quit IRC13:19
openstackgerritDirk Mueller proposed openstack/python-novaclient: Properly authenticate against Keystone V2 in a V3 env  https://review.openstack.org/26476413:19
*** bklei has joined #openstack-nova13:19
openstackgerritUkesh Kumar proposed openstack/nova: Add microversion to allow server search option metadata for non-admin  https://review.openstack.org/26476513:19
*** fawadk has joined #openstack-nova13:19
*** huats has joined #openstack-nova13:20
*** fawadkhaliq has quit IRC13:21
*** erlon has joined #openstack-nova13:22
*** edmondsw has joined #openstack-nova13:22
*** ducttape_ has quit IRC13:22
*** bklei has quit IRC13:23
*** sacharya has joined #openstack-nova13:24
*** doug-fish has joined #openstack-nova13:24
openstackgerritUkesh Kumar proposed openstack/nova: To allow server search option metadata for non-admin  https://review.openstack.org/26476513:25
*** sacharya has quit IRC13:29
*** sudipto has quit IRC13:29
*** klkumar has quit IRC13:31
*** cfriesen__ has quit IRC13:33
*** apoorvad has joined #openstack-nova13:34
*** cfriesen__ has joined #openstack-nova13:34
*** lunarlamp has joined #openstack-nova13:36
*** ndipanov has quit IRC13:36
*** zz_dimtruck is now known as dimtruck13:37
*** apoorvad has quit IRC13:38
openstackgerritChris Dent proposed openstack/nova: Versioned objects support for ResourceClass  https://review.openstack.org/26349713:40
*** Daisy has joined #openstack-nova13:40
openstackgerritDirk Mueller proposed openstack/python-novaclient: Properly authenticate against Keystone V2 in a V3 env  https://review.openstack.org/26476413:40
*** bklei has joined #openstack-nova13:43
kashyapmatrohon: Do you know the version that you found this bug with? - https://bugs.launchpad.net/nova/+bug/144105413:45
openstackLaunchpad bug 1441054 in OpenStack Compute (nova) "live-migration --block-migrate fails with default libvirt flags" [Medium,In progress] - Assigned to Mathieu Rohon (mathieu-rohon)13:45
kashyapmatrohon: Version of libvirt and QEMU13:45
*** haomaiwang has joined #openstack-nova13:46
matrohonkashyap, let me check I may still have the VM that I use for this test13:47
kashyapmatrohon: That behavior is not reproducible anymore.13:50
kashyapI'm writing a comment in the bug.13:50
kashyapPlease re-test when you get time, and write your observation in the bug.13:50
*** dimtruck is now known as zz_dimtruck13:51
matrohonkashyap, I'll do, actually I currently work on live-migration bugs in neutron13:52
*** doug-fis_ has joined #openstack-nova13:53
*** doug-fish has quit IRC13:55
kashyapmatrohon: Okay, plese see comment #5.  I tested 6 times today and checked with libvirt folks too.13:55
matrohonkashyap, it was on debian jessie and packages was probably libvirt 1.2.9-9 and qemu-kvm 1:2.1+dfsg-1114:00
matrohonI'll test with an up to date debian jessie14:00
*** ccarmack1 has quit IRC14:00
*** haomaiwang has quit IRC14:01
*** 5EXAAOC18 has joined #openstack-nova14:01
*** EinstCrazy has joined #openstack-nova14:01
*** rlrossit has joined #openstack-nova14:02
*** ankit_ag has quit IRC14:02
*** cdent has quit IRC14:02
*** xyang1 has joined #openstack-nova14:03
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/26444114:03
*** claudiub has joined #openstack-nova14:04
*** delatte has joined #openstack-nova14:05
*** ljxiash has joined #openstack-nova14:06
*** delattec has quit IRC14:08
*** EinstCrazy has quit IRC14:09
*** smatzek_ is now known as smatzek14:09
*** fawadk has quit IRC14:10
*** zenoway has joined #openstack-nova14:10
*** mkoderer_cloud has quit IRC14:11
*** yassine__ has quit IRC14:13
*** zenoway has quit IRC14:14
*** gcb_ has joined #openstack-nova14:14
*** mdrabe has joined #openstack-nova14:15
*** vladikr has joined #openstack-nova14:16
openstackgerritStephen Finucane proposed openstack/nova: config options: Make xvp proxy into vnc group  https://review.openstack.org/23539814:17
openstackgerritStephen Finucane proposed openstack/nova: config options: Centralise 'vnc' options  https://review.openstack.org/26376314:17
*** mriedem has joined #openstack-nova14:17
*** slagle has joined #openstack-nova14:19
sfinucanmikal: Got a moment?14:21
*** jichen has quit IRC14:23
bauzassfinucan: mikal is based in Australia, sincerely not sure he could answer in a timely manner :)14:23
mriedemedleafe: belmoreira: i noticed the bug for https://review.openstack.org/#/c/255405/ was mentioned in http://openstack-in-production.blogspot.com/ so i'm going to mark that fix for backport to liberty and kilo14:23
sfinucanbauzas: Ah - ZNC (or its ilk) strike again. Thanks for the heads up :)14:24
bauzasfair point14:24
*** cdent has joined #openstack-nova14:24
edleafemriedem: sure thing. Need me to do anything with that?14:24
*** zhenq has joined #openstack-nova14:24
mriedemedleafe: you could propose the backports14:24
*** mpavone has joined #openstack-nova14:24
belmoreiramriedem: great14:25
mriedemi've already marked the bug14:25
*** mpavone has quit IRC14:25
openstackgerritsahid proposed openstack/nova: libvirt: check graphic/serial consoles at pre_live_migration step  https://review.openstack.org/25470914:25
openstackgerritsahid proposed openstack/nova: libvirt: introduce block_stats to return device stats  https://review.openstack.org/25128814:25
openstackgerritsahid proposed openstack/nova: libvirt: return a root block device  https://review.openstack.org/24913914:25
openstackgerritsahid proposed openstack/nova: libvirt: introduce exceptions handler  https://review.openstack.org/25799114:25
openstackgerritsahid proposed openstack/nova: libvirt: introduce get_job_info to Guest's object  https://review.openstack.org/25074014:25
openstackgerritsahid proposed openstack/nova: libvirt: configure migrate max downtime  https://review.openstack.org/25073914:25
openstackgerritsahid proposed openstack/nova: libvirt: introduce "abort_job" to Guest's object  https://review.openstack.org/25073814:25
openstackgerritsahid proposed openstack/nova: libvirt: introduce "migrate" to Guest's object  https://review.openstack.org/25036514:25
bauzasedleafe: CC me on the backports, I'd be glad to review14:26
edleafemriedem: ok, so to make sure I'm not screwing it up: grab liberty/kilo stable, apply the fix, and then push?14:26
edleafebauzas: ack14:26
*** ndipanov has joined #openstack-nova14:26
mriedemedleafe: first i'd just try the cherry pick button i nthe gerrit ui14:26
bauzas+14:26
bauzasnothing really changed in the codebase about that - except the big Spec objectification14:27
edleafemriedem: ok, haven't played with that yet. Time to experiment!14:27
mriedemit's a 5 LOC change so i'd hope there aren'tmerge conflicts14:27
mriedembut i know you scheduler boyz like to change things14:27
bauzasmriedem: I certainly bet it's not the case14:27
*** irenab has quit IRC14:28
bauzasI mean to be clear, I expect no issue14:28
slaglemriedem: hi, could you have a look at https://review.openstack.org/#/c/264793/, it's a revert for stable/liberty14:28
mriedemslagle: gah14:29
*** ljxiash has quit IRC14:29
*** EinstCrazy has joined #openstack-nova14:30
*** ccarmack has joined #openstack-nova14:30
mriedemyeah i'll check on it14:30
mriedemthanks for finding that14:30
slaglethx14:31
*** bklei has quit IRC14:31
*** ljxiash has joined #openstack-nova14:32
edleafemriedem: easy peasy. Done.14:32
*** lucasagomes is now known as lucas-hungry14:32
*** yassine__ has joined #openstack-nova14:32
*** otter768 has joined #openstack-nova14:32
*** bklei has joined #openstack-nova14:33
*** yassine__ has quit IRC14:33
*** daemontool_ has joined #openstack-nova14:33
*** yassine__ has joined #openstack-nova14:33
*** daemontool has quit IRC14:34
*** apoorvad has joined #openstack-nova14:35
mriedemslagle: so i guess tripleo in stable/liberty isn't using upper-constraints?14:37
*** otter768 has quit IRC14:38
tobascojohnthetubaguy: you remember this one? https://bugs.launchpad.net/nova/+bug/150890714:38
openstackLaunchpad bug 1508907 in OpenStack Compute (nova) "local_gb_used wrong in compute_nodes table when using Dell Cinder backend" [Undecided,Confirmed]14:38
tobascojohnthetubaguy: i just tested this one against our deployment and it infact places the deployment into a critical state where no new instances can be spawned14:38
slaglemriedem: probably not.14:38
mriedemslagle: could you report a bug against nova with the details, i.e. the job that failed and the version of oslo.serialization used?14:38
*** zz_dimtruck is now known as dimtruck14:38
kashyapmarkmc: As to what combination caused that traceback - we'll know once the reporter tells us the version of libvirt they're using (asked them on the bug).  However, why we can't see it manifest presently is - in the meantime the bug was reported upstream libvirt must've fixed it to gracefully fallback to old implementation (libvirt devs confirm so too, on their IRC.)14:38
slaglemriedem: sure, will do14:38
mriedemslagle: i'm going to want to run this back to lifeless as a post-mortem14:38
mriedemthanks14:39
claudiubmriedem: it seems that dump_as_bytes is tagged in oslo.serialization 1.10.0. a solution would be to bump it in global-requirements, right?14:39
mriedembecause dump_as_bytes isn't in oslo.serialization 1.4.0 which is the lower bound https://github.com/openstack/requirements/blob/stable/liberty/global-requirements.txt#L9814:39
mriedembut it is in 2.1.0 as the upper bound https://github.com/openstack/requirements/blob/stable/liberty/upper-constraints.txt#L20014:39
slaglemriedem: we dont install from pip, we r using delorean which is building rpm's from the latest commits on stable/liberty14:39
mriedemclaudiub: we don't raise mins in g-r if we can help it14:39
slaglemriedem: so maybe the issue is wound up in that somehow14:39
mriedemslagle: yeah this is definitely wrong,14:39
mriedemslagle: no you're right on this one14:40
*** EinstCrazy has quit IRC14:40
mriedemwhat i'm trying to figure out is how the newly proposed backward compat process would have caught this, which i don't think it would have14:40
claudiubmriedem: ok, cool.14:40
*** apoorvad has quit IRC14:40
*** smatzek has quit IRC14:41
*** Kevin_Zheng has quit IRC14:42
mriedemedleafe: thanks, +2s all around14:45
*** zenoway has joined #openstack-nova14:45
tobascoAnybody up for a challenge? This actually breaks the implementation of counted disk usage for Nova when having Cinder volumes as root disks. https://bugs.launchpad.net/nova/+bug/150890714:46
openstackLaunchpad bug 1508907 in OpenStack Compute (nova) "local_gb_used wrong in compute_nodes table when using Dell Cinder backend" [Undecided,Confirmed]14:46
*** esracelik has quit IRC14:47
*** fawadkhaliq has joined #openstack-nova14:47
*** zenoway has quit IRC14:50
*** dslevin_ has quit IRC14:50
sdaguejohnthetubaguy et al - the optional project_ids patch I think is ready to go in as a microversion - https://review.openstack.org/#/c/233076/14:51
kashyapmatrohon: Thanks!14:52
*** liverpooler has quit IRC14:53
*** mrkz has joined #openstack-nova14:53
*** eharney has quit IRC14:54
*** ctrath has joined #openstack-nova14:56
*** ducttape_ has joined #openstack-nova14:57
*** baoli has quit IRC14:58
openstackgerritMatthew Booth proposed openstack/nova: Fix format detection in libvirt snapshot  https://review.openstack.org/26481215:00
openstackgerritMatthew Booth proposed openstack/nova: Fix format conversion in libvirt snapshot  https://review.openstack.org/26481315:00
openstackgerritMatthew Booth proposed openstack/nova: Fix backing file detection in libvirt live snapshot  https://review.openstack.org/26481415:00
*** 5EXAAOC18 has quit IRC15:01
*** baoli has joined #openstack-nova15:01
*** pradk has joined #openstack-nova15:01
*** haomaiwang has joined #openstack-nova15:01
*** zhangjn has quit IRC15:02
*** annegentle has joined #openstack-nova15:03
*** dslev has joined #openstack-nova15:03
mdboothndipanov: ^^^ Do you have time to review CVE patches?15:03
smcginnismriedem, or any others that have an understanding of nova's storage interaction: Any idea what's going on with bug 150890715:06
openstackbug 1508907 in OpenStack Compute (nova) "local_gb_used wrong in compute_nodes table when using Dell Cinder backend" [Undecided,Confirmed] https://launchpad.net/bugs/150890715:06
*** Jeffrey4l has quit IRC15:06
smcginnisLooks like local storage space is incorrectly evaluated for external storage.15:06
kashyapmdbooth: In relation to the excellent sleuthing you've done in the above bug (from your patch series), did you also see this? - https://bugs.launchpad.net/nova/+bug/141508715:07
openstackLaunchpad bug 1415087 in OpenStack Compute (nova) "[OSSA 2015-011] Format-guessing and file disclosure in image convert (CVE-2015-1850, CVE-2015-1851)" [High,Incomplete] - Assigned to Tony Breeds (o-tony)15:07
kashyapmdbooth: Maybe your patch series fixes that too.15:08
mdboothkashyap: Nope. That looks different.15:08
*** cfriesen__ has quit IRC15:09
*** Jeffrey4l has joined #openstack-nova15:09
mdboothkashyap: Although note the change in: https://review.openstack.org/#/c/264813/1/nova/virt/images.py15:09
kashyapmdbooth: It's related -- the bug I pointed involves guessing backing file format.15:10
*** mriedem is now known as mriedem_meeting15:10
*** eharney has joined #openstack-nova15:10
* kashyap clicks15:10
*** diogogmt has joined #openstack-nova15:10
mdboothkashyap: Yes, indeed.15:10
kashyapWhich is clearly not desirable.15:10
*** smatzek has joined #openstack-nova15:10
*** doug-fis_ is now known as doug-fish15:11
kashyapmdbooth: More specifically,15:11
kashyapIn this line  https://github.com/openstack/nova/blob/master/nova/virt/libvirt/utils.py#L8715:11
*** achinm has quit IRC15:11
kashyapThe base image format ought to be identified, so that we can specifiy a value for -o backing_fmt15:12
*** mragupat has joined #openstack-nova15:12
*** edtubill has joined #openstack-nova15:13
ndipanovmdbooth, ping me when jenkins comes back happy15:13
mdboothndipanov: Will do15:13
ndipanovand I'll +215:13
mdboothThanks15:14
auggysdague: omg i can't believe it's finally ready :D15:15
*** jaypipes has joined #openstack-nova15:15
*** yamahata has joined #openstack-nova15:18
*** mtanino has joined #openstack-nova15:19
*** med_` has joined #openstack-nova15:20
*** med_ has quit IRC15:20
*** Daisy_ has joined #openstack-nova15:21
*** diogogmt has quit IRC15:22
*** Daisy has quit IRC15:22
*** artom has quit IRC15:24
*** dougs1 has joined #openstack-nova15:24
*** dougs has quit IRC15:25
*** Daisy_ has quit IRC15:27
*** Daisy has joined #openstack-nova15:27
*** nelsnelson has quit IRC15:28
*** nelsnelson has joined #openstack-nova15:29
*** Daisy has quit IRC15:29
*** Daisy has joined #openstack-nova15:29
*** tonytan4ever has joined #openstack-nova15:29
*** breitz has quit IRC15:30
*** breitz has joined #openstack-nova15:30
*** annegentle has quit IRC15:30
*** annegentle has joined #openstack-nova15:31
*** zenoway has joined #openstack-nova15:33
*** lucas-hungry is now known as lucasagomes15:34
*** dims_ has joined #openstack-nova15:35
*** dims has quit IRC15:36
mriedem_meetingslagle: +W on your revert, thanks again15:36
*** zenoway has quit IRC15:38
*** slagle has quit IRC15:39
jaypipesmriedem_meeting: happy New Year, mate :)15:41
openstackgerritVladik Romanovsky proposed openstack/nova: virt: introduce libosinfo library to set hardware policy  https://review.openstack.org/14962515:41
mriedem_meetinguh oh15:41
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring network model  https://review.openstack.org/14962715:41
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: adding a class to retrieve hardware properties  https://review.openstack.org/14962615:41
mriedem_meetingjaypipes: i heard you became a monk and moved to a cave somewhere15:42
*** mriedem_meeting is now known as mriedem15:42
mriedemi guess that would technically be more of a hermit15:43
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring the disk bus  https://review.openstack.org/14962815:43
jaypipesmriedem: lol, no.. just a 2 week trip to Russia and Ukraine, then holidays15:44
*** moshele has quit IRC15:45
mriedemwelcome back15:46
mriedemfun times are to be had15:46
jaypipesmriedem: :) indeed. hopefully I can be a better contributor this quarter.15:47
mriedemthis quarter :)15:47
mriedemspoken like a true exec15:47
mriedemjust make sure you have your projections on my desk by end of day friday15:48
mriedemthanks15:48
*** _diana_1 has joined #openstack-nova15:48
bauzaswait, what? if jaypipes is a true exec, should he be wearing a suit?15:49
*** gcb_ has quit IRC15:50
auggyif he's a remote exec, then he'd probably just wear one of these - https://www.betabrand.com/mens-business-suit-onesie-hybrid.html15:50
bauzashttps://s-media-cache-ak0.pinimg.com/736x/a5/d6/af/a5d6af7969316d765ba29a6c11242a40.jpg15:50
mriedemthat is a thick looking coat15:51
*** tongli has joined #openstack-nova15:51
dansmithmriedem: it's made of sweatpants I think15:51
mriedemtoo hot15:51
*** rook has quit IRC15:51
mriedemwool sweatpants at that15:52
*** rook has joined #openstack-nova15:52
auggylooks like it's a lightweight cotton poly sweatpant fabric blend15:53
auggyand the jacket has vents15:53
openstackgerritEoghan Glynn proposed openstack/nova: doc: corrections to the server concepts docco  https://review.openstack.org/26483915:53
auggywow the things people come up with15:53
mriedemauggy: have you worked in retail before,15:53
mriedembecause you really said that too easily15:53
auggyhaha not really, i did work in an ice cream shop once15:53
bauzasthat's where I'm angry to be a French because I don't understand you, folks15:54
auggyand my first job out of college was developer support for microsoft, i had to sort of sell things15:54
mriedembauzas: "lightweight cotton poly sweatpant fabric blend" is not meant to be understood15:54
auggylike support and licenses15:54
mriedembauzas: like 'credit default swap'15:54
bauzasprobably :)15:54
jaypipesauggy: that is awesome. :)15:57
*** rlrossit has quit IRC15:57
*** apoorvad has joined #openstack-nova15:57
jaypipesmriedem: fuug, I can't believe I just wrote quarter above... guh.15:57
auggyjaypipes: at least you didn't say fiscal15:58
jaypipesauggy: tru nuf.15:58
*** sacharya has joined #openstack-nova15:58
*** rotbeard has quit IRC15:59
*** rlrossit has joined #openstack-nova15:59
*** pjm_ has joined #openstack-nova15:59
*** rlrossit_ has joined #openstack-nova15:59
pjm_Hi there15:59
pjm_I'm currently looking to develop a new driver for OpenStack15:59
pjm_but the only info I found about nova is this website: http://docs.openstack.org/developer/nova/16:00
ndipanovpjm_, what kind of driver?16:00
pjm_there is any documentation file that I could use?16:00
pjm_A driver for deploying virtual machines in a public cloud16:00
pjm_for example digital ocean16:00
*** haomaiwang has quit IRC16:01
*** haomaiwa_ has joined #openstack-nova16:01
pjm_i see that already exists an driver for the EC2 (https://github.com/openstack/ec2-driver)16:01
*** apoorvad has quit IRC16:02
mriedemfirst time i've seen ec2-driver16:02
*** Daisy has quit IRC16:02
*** sacharya has quit IRC16:02
ndipanovyeah me 216:02
*** Daisy has joined #openstack-nova16:03
*** liusheng has quit IRC16:03
ndipanovI have to say that I have my doubts about this kind of stuff16:03
*** rlrossit has quit IRC16:03
alaskihaven't we essentially said we're not interested in supporting this?16:03
mriedemprobably16:04
*** rdopiera has quit IRC16:04
dansmithalaski: correct16:04
mriedembut probably not officially in a TC meeting with a change to the governance repo16:04
mriedem:)16:04
sdagueyeh, it was an unconference pitch a couple of summits ago16:04
mriedemmaybe we need a change to the nova devref's 'scope' page16:04
mriedemhttp://docs.openstack.org/developer/nova/project_scope.html16:04
mriedemsub-section of http://docs.openstack.org/developer/nova/project_scope.html#compute-resources16:05
mriedem'thou shalt not have an in tree driver that proxies to a public cloud'16:05
ndipanovI think we'd want to be more specific than that16:06
ndipanovsince it obviously seems like a good idea to people16:06
*** Daisy has quit IRC16:07
pjm_so in Nova isn't supposed to support public cloud ?16:07
mriedemndipanov: also https://review.openstack.org/#/c/263500/16:07
*** salv-orl_ has joined #openstack-nova16:08
alaskipjm_: Nova is not an API for managing other public clouds16:08
*** zenoway has joined #openstack-nova16:09
ndipanovI am actually interested why people think this is a good idea now16:09
*** artom has joined #openstack-nova16:09
*** artom has quit IRC16:10
pjm_Hmm, I thinking that could be used because that GIT respository16:10
*** artom has joined #openstack-nova16:10
pjm_so Nova is only supposed to work directly with physical hypervisors right?16:10
*** salv-orlando has quit IRC16:11
dansmithpjm_: yeah, we've had the conversation many times about nova aggregating large sets of resources behind a single host and the conversation always seems to settle on not doing that16:12
dansmithpjm_: even the vmware driver's aggregation of multiple machines is highly problematic16:12
*** zenoway has quit IRC16:13
pjm_dansmith: thanks, sorry i misunderstood and when i saw that repository I was thinking that were acceptable16:13
pjm_its wise to make an independent component then, right?16:14
mriedempjm_: there is nothing preventing you from creating a git repo that does something like that16:14
openstackgerritsahid proposed openstack/nova: libvirt: check graphic/serial consoles at pre_live_migration step  https://review.openstack.org/25470916:14
alaskipjm_: I don't think any of us were aware of that repo16:14
*** lyanchih has joined #openstack-nova16:14
openstackgerritsahid proposed openstack/nova: libvirt: introduce block_stats to return device stats  https://review.openstack.org/25128816:14
mriedempjm_: it just means we aren't going to take it in tree16:14
openstackgerritsahid proposed openstack/nova: libvirt: return a root block device  https://review.openstack.org/24913916:14
openstackgerritsahid proposed openstack/nova: libvirt: introduce exceptions handler  https://review.openstack.org/25799116:14
openstackgerritsahid proposed openstack/nova: libvirt: introduce get_job_info to Guest's object  https://review.openstack.org/25074016:14
openstackgerritsahid proposed openstack/nova: libvirt: configure migrate max downtime  https://review.openstack.org/25073916:14
openstackgerritsahid proposed openstack/nova: libvirt: introduce "abort_job" to Guest's object  https://review.openstack.org/25073816:14
openstackgerritsahid proposed openstack/nova: libvirt: introduce "migrate" to Guest's object  https://review.openstack.org/25036516:14
dansmithpjm_: just be aware that doing it externally means you're on the hook for chasing our internal API as we do not have a stable external API for out of tree drivers16:14
*** mdrabe has quit IRC16:15
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add SerialConsoleOps class  https://review.openstack.org/18404016:17
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Nova serial console access support  https://review.openstack.org/14500416:17
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add serial console handler class  https://review.openstack.org/18403916:17
openstackgerritPetrut Lucian proposed openstack/nova: objects: Added serial port properties to ImageMetaProps  https://review.openstack.org/26103516:17
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add serial console proxy  https://review.openstack.org/18403816:17
openstackgerritJialiang proposed openstack/nova: VMware: Prefer ShutdownGuest to PowerOffVM_Task  https://review.openstack.org/26485216:19
*** diogogmt has joined #openstack-nova16:19
ndipanovmdbooth, how are the cve patches doing?16:21
sahidjaypipes: hello, any plan about osvif project?16:21
*** diogogmt_ has joined #openstack-nova16:22
*** nkrinner has quit IRC16:22
*** diogogmt has quit IRC16:24
*** diogogmt_ is now known as diogogmt16:24
*** tonytan4ever has quit IRC16:24
jaypipessahid: hi! I think we should do a meeting next week to discuss it. Everything is ready to go (to bring all the plugin repos into git.openstack.org and get test coverage up) but there's at least 6 interested parties (you, me, danpb, mleroy, smooney and moshe levi) that need to get coordinated :)16:24
mdboothndipanov: Waiting on a test to fail so I can do a recheck.16:25
mdboothndipanov: So, badly.16:25
*** kfarr has joined #openstack-nova16:25
jaypipessahid: let me send out an email with a proposed calendar event.16:26
openstackgerritAndrea Rosa proposed openstack/nova: Adding guard on None value for some helpers method  https://review.openstack.org/26485316:26
*** tonytan_brb has joined #openstack-nova16:26
sahidcool, thanks jaypipes16:26
*** pjm_ has quit IRC16:29
*** jdurgin1 has joined #openstack-nova16:30
*** lyanchih has quit IRC16:31
*** zenoway has joined #openstack-nova16:33
openstackgerritJialiang proposed openstack/nova: VMware: Prefer ShutdownGuest to PowerOffVM_Task  https://review.openstack.org/26485216:33
*** otter768 has joined #openstack-nova16:33
*** belmoreira has quit IRC16:34
*** sacharya has joined #openstack-nova16:34
*** sacharya has quit IRC16:34
*** sacharya has joined #openstack-nova16:35
openstackgerritAndrea Rosa proposed openstack/nova: Adding guard on None value for some helper methods  https://review.openstack.org/26485316:36
*** zenoway has quit IRC16:37
*** otter768 has quit IRC16:38
*** med_` is now known as med_16:39
*** med_ has quit IRC16:39
*** med_ has joined #openstack-nova16:39
*** med_ is now known as med16:40
*** med is now known as med_16:40
*** Sukhdev has joined #openstack-nova16:41
*** lyanchih has joined #openstack-nova16:44
mdboothndipanov: So I have 2 +1s from Jenkins, and a spurious -1 which is back in the gate.16:45
*** paul-carlton2 has quit IRC16:45
mdboothndipanov: I guess the 2 +1s mean that it's cherry-picking, given that they're dependent on the patch with a -1.16:45
*** mgoddard__ has joined #openstack-nova16:46
mdboothndipanov: Anyway, +1s on https://review.openstack.org/#/c/264813/1 and https://review.openstack.org/#/c/264814/116:46
*** baoli has quit IRC16:46
mdboothSpurious -1 on https://review.openstack.org/#/c/264812/116:47
*** dslev has quit IRC16:47
*** shyama has quit IRC16:48
*** mgoddard_ has quit IRC16:49
mriedemmdbooth: https://review.openstack.org/#/c/264812/ failed grenade16:51
*** mjura has quit IRC16:51
openstackgerritRaildo Mascena proposed openstack/nova: Nested Quota Driver  https://review.openstack.org/24262616:51
openstackgerritRaildo Mascena proposed openstack/nova: Nested Quota Driver: Get Hierarchy info  https://review.openstack.org/24256816:51
mriedemdid you check why before you rechecked?16:51
mdboothmriedem: Yeah. Looked like an infrastructure issue.16:51
*** unicell1 has joined #openstack-nova16:51
mdboothmriedem: Was I wrong?16:51
mriedemlooking16:51
*** mrkz has quit IRC16:52
*** zakora has quit IRC16:52
*** unicell has quit IRC16:52
mriedem10.176.160.41 | FAILED => SSH Error: ssh: connect to host 10.176.160.41 port 22: Connection timed out16:52
mriedemyeah infra issue16:52
*** mrkz has joined #openstack-nova16:53
BobBallI saw the same issue on another job - https://review.openstack.org/#/c/264812/ - hope it's just bad luck rather than a couple of cases of a big issue :)16:53
mriedemprobably just the slow nodes thing16:54
mdboothBobBall: That's the same one :)16:54
*** drankis has quit IRC16:54
BobBallOh... that's the one I just rechecked lol sorry16:54
BobBallyes, I assumed it was a different job :)16:54
BobBallAnd I _very nearly_ used "It's friday" as my excuse... ...16:55
mdboothFriday^16:55
BobBallThe first week back after Christmas is always bad!16:55
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova: Added new scheduler filter: AggregateTypeExtraSpecsAffinityFilter  https://review.openstack.org/18927916:56
openstackgerritNikola Dipanov proposed openstack/nova: pci: do not filter out any SRIOV Physical Functions  https://review.openstack.org/24901616:56
*** dane-fichter has joined #openstack-nova16:56
openstackgerritNikola Dipanov proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment  https://review.openstack.org/24901816:56
openstackgerritNikola Dipanov proposed openstack/nova: pci: adding support to specify a device_type in pci requests  https://review.openstack.org/25925416:56
openstackgerritNikola Dipanov proposed openstack/nova: objects: adding a parent_addr field to the PciDevice object  https://review.openstack.org/24901516:56
*** rlrossit_ has quit IRC16:57
*** jichen has joined #openstack-nova16:57
*** jlanoux has quit IRC16:58
*** dslevin_ has joined #openstack-nova16:59
openstackgerritNikola Dipanov proposed openstack/nova: neutron: pci_request logic considers 'direct-physical' vnic type  https://review.openstack.org/26258416:59
openstackgerritNikola Dipanov proposed openstack/nova: pci: Add utility method for getting the MAC addr  https://review.openstack.org/26258516:59
openstackgerritNikola Dipanov proposed openstack/nova: neutron: Update the port with a MAC address for PFs  https://review.openstack.org/26258616:59
openstackgerritNikola Dipanov proposed openstack/nova: Add a vnic type for PF passthrough and a new libvirt vif driver  https://review.openstack.org/26258316:59
*** dims has joined #openstack-nova16:59
*** mragupat has quit IRC17:00
*** mgoddard__ has quit IRC17:00
*** mgoddard has joined #openstack-nova17:00
*** dane-fichter has quit IRC17:00
*** mragupat has joined #openstack-nova17:00
*** rlrossit has joined #openstack-nova17:00
*** haomaiwa_ has quit IRC17:01
*** asselin has joined #openstack-nova17:01
*** 77CAAHQRX has joined #openstack-nova17:01
*** mragupat has quit IRC17:02
*** mragupat has joined #openstack-nova17:02
*** dims_ has quit IRC17:02
ndipanovdansmith, https://review.openstack.org/#/c/249015/12/nova/objects/pci_device.py17:05
*** daemontool_ has quit IRC17:06
*** burt has joined #openstack-nova17:06
*** baoli has joined #openstack-nova17:07
openstackgerritKevin L. Mitchell proposed openstack/python-novaclient: Add a mechanism to call out deprecated options  https://review.openstack.org/25759517:08
*** zenoway has joined #openstack-nova17:08
dansmithndipanov: I have it open and plan to look17:09
*** drankis has joined #openstack-nova17:09
ndipanovdansmith, also - this whole service by service business got me a little bit confused tbh17:12
ndipanovdansmith, we can't really have different generations of conductors can we/17:12
ndipanov?17:12
*** Sukhdev has quit IRC17:12
*** zenoway has quit IRC17:12
dansmithndipanov: if we do this sort of thing right we should be able to yeah, why not?17:12
*** paul-carlton2 has joined #openstack-nova17:13
ndipanovwell what if a newer compute goes to an older conductor17:13
ndipanovthat can't work17:13
*** lyanchih has quit IRC17:13
dansmithcomputes always go after conductors17:13
ndipanovok17:13
ndipanovthat makes sense17:13
dansmiththe point is to get the conductors upgraded before computes, but without taking them all down atomically17:13
*** sfinucan has quit IRC17:13
*** Sukhdev has joined #openstack-nova17:13
ndipanovyeah that makes sense17:13
*** davideagnello has joined #openstack-nova17:15
*** markus_z has quit IRC17:17
*** slagle has joined #openstack-nova17:17
doffmrlrossit: https://review.openstack.org/#/c/213041/2417:18
*** mdrabe has joined #openstack-nova17:18
*** apoorvad has joined #openstack-nova17:18
*** mragupat_ has joined #openstack-nova17:19
rlrossitdoffm: yup. exactly the same thing I was talking to dansmith about yesterday with https://review.openstack.org/#/c/263497/17:20
*** mragupat has quit IRC17:22
*** apoorvad has quit IRC17:23
*** garyk has quit IRC17:24
*** sudipto has joined #openstack-nova17:25
openstackgerritSudipta Biswas proposed openstack/nova: Remove version hardcoding for 'show' method  https://review.openstack.org/22857817:26
*** gjayavelu has joined #openstack-nova17:27
*** browne has joined #openstack-nova17:30
*** yassine__ has quit IRC17:31
*** pjm_ has joined #openstack-nova17:34
*** Sukhdev has quit IRC17:34
pjm_thanks for the previous replies17:35
johnthetubaguydansmith: mriedem: are one of you OK to do the meeting later?17:39
mriedemi can17:39
dansmithjohnthetubaguy: mriedem is17:39
johnthetubaguymriedem: thank you :)17:39
mriedemshow your appreciation by reviewing this stable/liberty backport https://review.openstack.org/#/c/258695/17:40
*** armax has joined #openstack-nova17:40
johnthetubaguysure17:40
*** dane-fichter has joined #openstack-nova17:40
mriedemfixes a gate bug17:41
johnthetubaguyah, that BDM one17:42
mriedemyeah, that's the squashed backport since the first fix on master didn't get it17:43
mriedemso it's a bit weird to look at17:43
mriedemlike rlrossit17:43
mriedemhi-o!17:44
*** zenoway has joined #openstack-nova17:44
johnthetubaguynice17:44
johnthetubaguyyeah, seems to capture those best bits of both of those changes17:45
mriedemit is both changes really17:45
mriedemjust that the 2nd change on master was essentially a redo of the first17:45
*** jistr has quit IRC17:45
*** matrohon has quit IRC17:45
*** ctrath has quit IRC17:45
johnthetubaguyyeah, only really captures the logging, but that makes sense17:46
*** gjayavelu has quit IRC17:46
rlrossitmriedem: :(17:46
* rlrossit has 2 names: nova-janitor, and now nova-punching-bag...17:47
johnthetubaguymriedem: dansmith: trying to move the testing conversation forward, got a prototype of the feature classification matrix, so we can start filling it out. Basically just adding in extra links and properties: https://review.openstack.org/#/c/264719/ and http://docs-draft.openstack.org/19/264719/3/check/gate-nova-docs/08140ba//doc/build/html/feature_classification.html#prototype-feature-support-matrix17:48
*** zenoway has quit IRC17:48
*** paul-carlton2 has quit IRC17:49
*** mrkz has quit IRC17:50
*** garyk has joined #openstack-nova17:50
dansmithjohnthetubaguy: looks pretty cool. not sure we'll be able to classify all features as mature/experimental/etc, especially as it applies to each driver17:50
dansmithbut still nice looking17:50
johnthetubaguydansmith: yeah, I tried to keep the maturity thing driver neutral, mostly for simplicity, and keep the driver thing purely passing tests17:51
*** jdurgin1 has quit IRC17:51
johnthetubaguydansmith: but you are right, thats not the full picture, but its better than nothing17:52
dansmithjohnthetubaguy: yeah17:52
dansmithjohnthetubaguy: it would *read* as the day that vmware adds live migration support that it's "complete" or whatever we had for live migration at the time for other drivers17:52
openstackgerritIldiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments  https://review.openstack.org/19313517:53
openstackgerritIldiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments  https://review.openstack.org/19313417:53
*** yarkot has joined #openstack-nova17:53
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach  https://review.openstack.org/15303817:53
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: virt: Pass host when call attach to Cinder  https://review.openstack.org/25627317:53
dansmithjohnthetubaguy: just need a note to look at the per-driver notes for that given thing I guess17:53
*** sahid has quit IRC17:54
*** tjones has joined #openstack-nova17:54
johnthetubaguyyeah, thats true17:54
mdboothThere's a swift job which has been running in the gate for 16 hours: https://review.openstack.org/#/c/259347/17:55
*** suro-patz has joined #openstack-nova17:55
*** nic has joined #openstack-nova17:55
dane-fichterguess it's not too "swift" then is it17:55
mdboothIt makes changes which reference signal handling and how the daemon runs17:55
mdboothIs anybody able to kill it with fire?17:55
dansmithmdbooth: the top thing in the gate is only 6 hours old17:56
dansmithmdbooth: that thing is running in check17:56
mdboothopenstack/swift0 min17:56
mdbooth259347,316 hr 39 min17:56
dansmithmdbooth: yeah, that's check17:56
*** can8dnSix has joined #openstack-nova17:56
dansmithmdbooth: not sure why the job timeout didn't kick it out17:57
openstackgerritIldiko Vancsa proposed openstack/nova: libvirt: Allow multiple volume attachments  https://review.openstack.org/19313517:57
openstackgerritIldiko Vancsa proposed openstack/nova: comp: Allow multiple volume attachments  https://review.openstack.org/19313417:57
openstackgerritIldiko Vancsa proposed openstack/nova: db and obj: Allow multiple volume attachments  https://review.openstack.org/19313317:57
dansmithmdbooth: I'd go ask in infra17:57
BobBallLooks like it's just running python27 as the last job which hasn't been going that long17:57
mdboothdansmith: Ah, ok. It's still stealing our precious bodily^H^H^H^H^H compute though, isn't it?17:57
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach  https://review.openstack.org/15303817:57
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: virt: Pass host when call attach to Cinder  https://review.openstack.org/25627317:57
BobBall38 minutes for the unit tests?17:57
dansmithBobBall: ah, right 39 minutes old17:57
dansmithmdbooth: indeed17:57
BobBallBut from the console it looks like someone has pressed ctrl-c ;)17:58
dansmithmdbooth: something blocked in gate would be a REAL problem17:58
dansmithBobBall: lol, nice17:58
* mdbooth is suitable chastised wrt pipeline names :)17:58
*** can8dnSix has quit IRC17:58
mdboothndipanov: w00t, 3 +1s from Jenkins :)17:59
*** cdent has quit IRC17:59
ildikovjohnthetubaguy: mriedem: I fixed the things in the multi-attach spec we talked about (virt capabilities, block snapshot create for volume and boot from volume)17:59
mdboothOr if ndipanov has gone for the day, does anybody else feel like reviewing the 3 cve patches?18:00
ildikovjohnthetubaguy: mriedem: please take a look a let me know what else to figure out with this18:00
*** chhavi has quit IRC18:00
*** 77CAAHQRX has quit IRC18:01
BobBalljohnthetubaguy: https://review.openstack.org/#/c/254022/ (the one we discussed in yesterday's XenAPI meeting) is now ready for your viewing pleasure18:01
*** haomaiwang has joined #openstack-nova18:01
johnthetubaguyBobBall: damm, I hope to get to those today, will be tomorrow now18:01
BobBallTomorrow is fine.  I have written it down.  "johnthetubaguy will review 254022 on 08/01/2016."18:02
BobBall:D18:02
*** paul-carlton2 has joined #openstack-nova18:03
*** ssurana has joined #openstack-nova18:03
*** ssurana has quit IRC18:08
*** penick has quit IRC18:08
*** ssurana has joined #openstack-nova18:09
*** eil397 has joined #openstack-nova18:09
*** thangp has joined #openstack-nova18:10
*** derekh has quit IRC18:11
*** tongli_ has joined #openstack-nova18:11
*** PaulMurray is now known as ptm_away18:12
ndipanovmdbooth, I +2ed the first two which are the most critical18:13
ndipanovI will look at the last one tomorrow18:13
ndipanov(again)18:13
*** toure has quit IRC18:14
*** tongli has quit IRC18:14
*** eharney has quit IRC18:14
*** markvoelker has quit IRC18:14
*** ajo has quit IRC18:14
*** markmc has quit IRC18:15
*** toure has joined #openstack-nova18:16
*** lpetrut has quit IRC18:17
*** READ10 has joined #openstack-nova18:20
*** markmc has joined #openstack-nova18:20
*** rook has quit IRC18:20
*** rook-desktio has joined #openstack-nova18:21
openstackgerritAndrew Laski proposed openstack/nova: Clean up filter_properties/availability_zone in boot  https://review.openstack.org/25443418:21
openstackgerritAndrew Laski proposed openstack/nova: Persist the request spec during an instance boot  https://review.openstack.org/25862818:21
*** ajo has joined #openstack-nova18:21
*** tonytan_brb has quit IRC18:21
*** ndipanov has quit IRC18:22
openstackgerritSudipta Biswas proposed openstack/nova: Remove version hardcoding for 'show' method  https://review.openstack.org/22857818:22
*** e0ne has joined #openstack-nova18:23
openstackgerritDan Smith proposed openstack/nova: Add xenapi support for XenapiLiveMigrateData objects  https://review.openstack.org/24785318:23
openstackgerritDan Smith proposed openstack/nova: Actually pass the migration data object down to the virt drivers  https://review.openstack.org/24821118:24
openstackgerritDan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire  https://review.openstack.org/25158818:24
*** eharney has joined #openstack-nova18:27
*** ctrath has joined #openstack-nova18:29
*** lihg has joined #openstack-nova18:30
*** femnad has quit IRC18:32
*** mriedem has quit IRC18:32
*** otter768 has joined #openstack-nova18:34
*** jaosorior has quit IRC18:34
*** ljxiash has quit IRC18:34
*** jaosorior has joined #openstack-nova18:35
*** haomaiwang has quit IRC18:35
*** harlowja has quit IRC18:35
*** harlowja has joined #openstack-nova18:36
openstackgerritDan Smith proposed openstack/nova: Add xenapi support for XenapiLiveMigrateData objects  https://review.openstack.org/24785318:38
openstackgerritDan Smith proposed openstack/nova: Actually pass the migration data object down to the virt drivers  https://review.openstack.org/24821118:38
openstackgerritDan Smith proposed openstack/nova: Allow sending the migrate data objects over the wire  https://review.openstack.org/25158818:38
*** mriedem has joined #openstack-nova18:38
mriedemildikov: did you actually post a change that amends the spec too?18:39
*** edmondsw has quit IRC18:39
mriedemi think we want the spec updated also for the things we have to cover with boot from volume that weren't mentioned originally18:39
*** otter768 has quit IRC18:39
ildikovmriedem: only thing left, I fixed the code first18:40
ildikovmriedem: that gives time to review it while writing up some additional text to the spec18:40
mriedemsneaky18:40
*** vilobhmm11 has joined #openstack-nova18:41
ildikovwell, I'm not as big expert especially in the Nova code I would want to, so I wanted to be sure what we already agreeded is done18:41
ildikovthe deadline is pretty close and after this amount of time and also work I really would like to see an initial version in, even if we block more things first than necessary18:42
ildikovmriedem: actually there's an open spec for the API changes, contains only the show volume, etc. things18:44
*** markstur_ has joined #openstack-nova18:44
*** edmondsw has joined #openstack-nova18:44
ildikovis it ok if I update that with what we discussed or should this be an amendment to the merged part?18:44
*** rlrossit has quit IRC18:44
mriedemcan you remind me the things we talked about blocking? boot from volume with delete_on_termination=True and multiattach=True volume?18:45
mriedemand the assisted snasphost API18:45
mriedemis what i remember18:45
*** rlrossit has joined #openstack-nova18:45
ildikovmriedem: actually the delete_on_termination should be fine18:45
mriedemi had assumed there was a single spec, not multiple18:45
*** paul-carlton2 has quit IRC18:45
ildikovthe volume delete errors are only logged in Nova everywhere18:45
mriedemyeah i saw the conversation on that a few days ago18:46
ildikovand Cinder does not remove it if there're still attachments18:46
mriedemb/c nova swallows them in _clean_volumes18:46
ildikovI blocked boot from volume in general if it's an existing multiattachable volume18:46
*** sudipto has quit IRC18:47
*** pradk has quit IRC18:47
ildikovthere was a single spec, but johnthetubaguy asked to delay the API layer changes for now as volume stuff is changing in Nova18:47
mriedemhmm, since this isn't a priority for the release is that supposed to get an exception from the spec freeze?18:47
mriedemwhich already passed18:48
ildikovI also blocked create_volume_snapshot for multattach  volumes18:48
ildikovthe API layer changes were not critical18:48
*** tonytan4ever has joined #openstack-nova18:48
ildikovin the sense of you don't need to change any API request to actually support multiattach18:49
*** rlrossit has quit IRC18:49
*** rlrossit has joined #openstack-nova18:49
jidarwhat settings are required in a RDO based install (so, haproxy, compute nodes and controllers) in order to change the default port of nova-vnc from 6080 to something else? I assume haproxy on the front end will need to listen on the address, then on the compute nodes they'll need "novncproxy_base_url" is there something else though?18:49
ildikovmriedem: so the multiattach flag is not showed on the Nova API for instance as far as I can remember18:50
mriedemjidar: you should probably ask in the rdo channel18:50
jidarmriedem: don't see how it's specific to rdo though?18:50
mriedemjidar: then see the channel topic18:51
jidarwoops!18:51
ildikovmriedem: but Cinder can show all the proper info about a volume18:51
*** lpetrut has joined #openstack-nova18:51
mriedemildikov: so cinder api will show the multiple attachments for the volume, right18:51
ildikovyes18:51
ildikovat least it should I haven't tried yet TBH18:51
*** tongli has joined #openstack-nova18:51
*** jidar has left #openstack-nova18:52
mriedemso if delete_on_termination=True isn't a problem, why are we blocking BFV with a pre-existing multiattach volume again?18:52
*** tongli has quit IRC18:53
mriedemsorry, lots of things going on this week so my memory is bad18:53
*** tongli_ has quit IRC18:53
mriedemsince that's really the same as booting an instance and then attaching that multiattach volume later18:53
ildikovjohnthetubaguy had a concern that it's not a good idea to have two instances booting from the same volume18:53
mriedemthe difference with BFV was the delete_on_termination flag18:53
mriedemhmm, i guess i don't see the difference from that with just booting those instances and attaching the same volume after they are active18:53
ildikovI would need to know how BFV works in order to answer this, sorry :(18:54
mriedemit's basically the same thing when you bring your own volume,18:54
mriedemthe difference is the delete_on_termination flag18:54
mriedemvolume attach doesn't have that18:54
mriedembut if we don't care, then i think they are the same scenario18:55
ildikovwe can discuss it in the spec amendment, it's blocked now, which is easy to remove if we're sure it's ok18:55
mriedemthe difference with BFV is nova does the orchestration of doing the attach and then waiting for the volume to be in-use18:55
*** Daisy has joined #openstack-nova18:55
*** lpetrut has quit IRC18:55
*** _diana_1 has quit IRC18:55
*** READ10 has quit IRC18:55
ildikovis there anything that can have race condition issues there for instance?18:55
mriedemmy concern was someone would BFV with delete_on_termination=True and then later attach that volume to another instance, and then delete the first and break the 2nd18:56
mriedembut cinder prevents that18:56
mriedemumm, we could maybe race to use the same device18:56
ildikovyes, it does, it would be a bug in Cinder otherwise18:56
mriedemi think the race would be basically the same as an attach request18:56
*** haomaiwang has joined #openstack-nova18:56
mriedemand the only race i see there is the race to reserve the device name18:56
*** mrkz has joined #openstack-nova18:56
ildikovyeah, I remember the test for that case18:57
*** ctrath has quit IRC18:57
ildikovyou can convince me kind of easily, it's also a small protion in the code currently, takes no time to remove18:59
*** Daisy has quit IRC18:59
ildikovwe can discuss it with John and then leave or remove the block based on the decision18:59
*** haomaiwang has quit IRC19:01
*** markvoelker has joined #openstack-nova19:03
*** moshele has joined #openstack-nova19:04
smatzekfor BFV I think the only thing you'd want to explicitly block to address johnthetubaguy 's concerns are blocking multi-attach=True for root_bdm (boot_index = 0 in BDM), and possibly check if the Cinder volume has bootable=True set on it.19:05
smatzekwhat would block two instances from having the same boot disk but allow server creations to specify attachments of mult-attached volumes.19:05
*** markvoelker_ has joined #openstack-nova19:05
*** cfriesen__ has joined #openstack-nova19:06
*** ctrath has joined #openstack-nova19:06
*** lpetrut has joined #openstack-nova19:06
*** ijw has joined #openstack-nova19:07
*** yamahata has quit IRC19:07
*** e0ne has quit IRC19:08
*** markvoelker has quit IRC19:09
*** moshele has quit IRC19:10
ildikovsmatzek: I check boot_index = 0 in BDM right now, but not Cinder bootable19:10
*** dslevin_ has quit IRC19:10
ildikovsmatzek: so I assume it might block a bit more than what's very necessary19:11
smatzekblocking multi-attach=true of volume that will be at boot_index=0 is probably good enough, and allowing all others.19:11
ildikovI check the destination_type too and only block if that's volume too19:11
ildikovyeah, this is what I'm doing at the moment19:11
ildikovor I hope the code I wrote really does the intention was this :)19:12
*** Sukhdev_ has joined #openstack-nova19:13
*** Sukhdev_ has quit IRC19:14
*** _diana_1 has joined #openstack-nova19:14
*** markvoelker has joined #openstack-nova19:14
*** pradk has joined #openstack-nova19:14
*** Sukhdev_ has joined #openstack-nova19:14
*** baoli has quit IRC19:14
ildikovsmatzek: did I understand correctly that if volume has bootable=False it should be all right?19:15
smatzekildikov: it probably would be, but it's probably an erroneous case of a mis-identified volume, or a strange case where you have a non-bootable volume first, before the volume with your master boot record.19:16
*** fawadkhaliq has quit IRC19:16
ildikovsmatzek: then it's most probably fine to not check that, I was wondering about what you wrote too19:17
*** markvoelker_ has quit IRC19:17
*** ijw has quit IRC19:17
ildikovit's a bit of a question that which error message we would like to return to the user19:17
*** renatoarmani has joined #openstack-nova19:18
*** dane-fichter has quit IRC19:19
*** penick has joined #openstack-nova19:21
*** cfriesen__ has quit IRC19:22
*** pjm_ has quit IRC19:24
*** unicell1 has quit IRC19:24
*** pjm_ has joined #openstack-nova19:25
*** cfriesen__ has joined #openstack-nova19:28
*** dslevin_ has joined #openstack-nova19:28
*** yamahata has joined #openstack-nova19:28
*** tojuvone_ has joined #openstack-nova19:28
*** Guest38743 has quit IRC19:29
*** Jeffrey4l has quit IRC19:30
*** zhenq1 has joined #openstack-nova19:30
*** pmurray has joined #openstack-nova19:30
*** nikhil_k has joined #openstack-nova19:31
*** yarkot has quit IRC19:31
*** pmurray is now known as Guest9284319:31
*** yarkot has joined #openstack-nova19:31
*** lpetrut has quit IRC19:31
*** unicell has joined #openstack-nova19:31
*** tangchen__ has joined #openstack-nova19:32
*** mriedem1 has joined #openstack-nova19:33
*** dims_ has joined #openstack-nova19:33
*** artom_ has joined #openstack-nova19:33
*** Jeffrey4l has joined #openstack-nova19:33
*** s1rp_ has joined #openstack-nova19:34
*** artom_ has quit IRC19:34
*** dims has quit IRC19:34
*** artom_ has joined #openstack-nova19:34
*** wasmum- has joined #openstack-nova19:35
*** yonglihe_ has joined #openstack-nova19:35
*** kgalanov_ has joined #openstack-nova19:35
*** ljxiash has joined #openstack-nova19:35
*** ameade_ has joined #openstack-nova19:35
*** ociuhandu has quit IRC19:36
*** powerbsd has joined #openstack-nova19:36
*** markmc` has joined #openstack-nova19:36
*** lpetrut has joined #openstack-nova19:37
*** bradjones_ has joined #openstack-nova19:37
*** bradjones_ has quit IRC19:37
*** bradjones_ has joined #openstack-nova19:37
*** stevemar_znc has joined #openstack-nova19:37
*** pradk has quit IRC19:38
*** mriedem has quit IRC19:38
*** markmc has quit IRC19:38
*** eil397 has quit IRC19:38
*** artom has quit IRC19:38
*** zhenq has quit IRC19:38
*** bradjones has quit IRC19:38
*** n0ano has quit IRC19:38
*** kbyrne has quit IRC19:38
*** ptm_away has quit IRC19:38
*** nikhil has quit IRC19:38
*** albertom has quit IRC19:38
*** ameade has quit IRC19:38
*** briancurtin has quit IRC19:38
*** kgalanov has quit IRC19:38
*** s1rp has quit IRC19:38
*** tangchen_ has quit IRC19:38
*** sileht has quit IRC19:38
*** lxsli has quit IRC19:38
*** wasmum has quit IRC19:38
*** yonglihe has quit IRC19:38
*** coreywright has quit IRC19:38
*** stevemar has quit IRC19:38
*** tojuvone has quit IRC19:38
*** bradjones_ is now known as bradjones19:38
*** powerbsd is now known as albertom19:38
*** mriedem1 is now known as mriedem19:38
*** edtubill has quit IRC19:38
*** yonglihe_ is now known as yonglihe19:39
*** kgalanov_ is now known as kgalanov19:39
*** lxsli has joined #openstack-nova19:39
*** ljxiash has quit IRC19:40
*** pradk has joined #openstack-nova19:40
*** stevemar_znc is now known as stevemar19:40
*** coreywright has joined #openstack-nova19:40
*** ijw has joined #openstack-nova19:40
*** ameade_ is now known as ameade19:40
*** lpetrut has quit IRC19:41
*** lpetrut has joined #openstack-nova19:41
*** sileht has joined #openstack-nova19:41
*** kbyrne has joined #openstack-nova19:42
* sc68cal sings the lyrics to Zappa's Token of my Extremes19:43
sc68caldarn it19:44
sc68calwrong window lol19:44
*** gjayavelu has joined #openstack-nova19:44
dansmithsc68cal: tell me more19:45
*** briancurtin has joined #openstack-nova19:45
sc68caljust having a zappa kind of day19:47
mriedemdon't eat the yellow snow19:48
*** Sukhdev_ has quit IRC19:48
*** lucasagomes is now known as lucas-dinner19:49
*** ijw has quit IRC19:53
auggyAlways wise advice19:55
*** drankis has quit IRC19:57
*** cfriesen__ has quit IRC19:59
*** dims_ has quit IRC20:00
bauzasmeeting in 1h, right?20:01
mriedemalaski: melwitt: doffm: dansmith: jaypipes: ML thread on cells v2 and soft-delete for the flavors tables http://lists.openstack.org/pipermail/openstack-dev/2016-January/083518.html20:01
mriedembauzas: yeah20:01
mriedembauzas: ^20:01
alaskimriedem: thanks20:01
mriedemnp20:01
bauzasack20:01
*** pjm_ has quit IRC20:02
mriedemsdague: the remove-mox bp is primarily in response to the py34 races right?20:02
sdaguemriedem: well, that's the final straw20:03
sdagueit was direction to stop using it a long time ago20:03
mriedemthem's are fighting words?20:03
mriedemok20:03
*** vilobhmm11 has quit IRC20:05
*** vilobhmm11 has joined #openstack-nova20:05
*** vilobhmm11 has quit IRC20:07
*** vilobhmm11 has joined #openstack-nova20:07
*** vilobhmm11 has quit IRC20:07
*** armax has quit IRC20:08
*** armax has joined #openstack-nova20:09
*** rfolco has quit IRC20:09
*** gjayavelu has quit IRC20:09
*** haomaiwang has joined #openstack-nova20:10
*** dhellmann has quit IRC20:11
*** dims has joined #openstack-nova20:12
*** cdent has joined #openstack-nova20:13
openstackgerritSean Dague proposed openstack/nova: API sample testing without project_id  https://review.openstack.org/25608920:15
openstackgerritSean Dague proposed openstack/nova: Make project_id optional in v2.1 urls  https://review.openstack.org/23307620:15
*** changbl has quit IRC20:15
*** haomaiwang has quit IRC20:17
*** ccarmack has quit IRC20:18
*** dims has quit IRC20:19
*** lpetrut has quit IRC20:21
*** dims has joined #openstack-nova20:22
*** lihg_ has joined #openstack-nova20:23
*** lpetrut has joined #openstack-nova20:23
*** openstackgerrit has quit IRC20:23
*** dims has quit IRC20:23
*** openstackgerrit has joined #openstack-nova20:25
sdaguemriedem: gate-nova-python34 is currently one of our worst gate reset offenders - http://status.openstack.org//elastic-recheck/data/uncategorized.html20:26
*** lihg has quit IRC20:26
mriedemsdague: yeah, dims had a change to skip some of those20:27
* mriedem looks for it20:27
mriedemhttps://review.openstack.org/#/c/264368/20:27
sdagueactually, it is the worst offending job20:27
*** whenry has joined #openstack-nova20:27
mriedemyeah, i know20:27
sdagueok, I think we've still had a fail after that20:27
mriedemi wonder if these are hitting ovh nodes and they are just slow enough that that tickles the mox failures20:28
mriedemit's funny how the py34 stuff hasn't been such a problem until recently20:28
sdaguethe config drive http://logs.openstack.org/26/224726/16/gate/gate-nova-python34/1dda5ee/console.html#_2016-01-07_18_17_49_11520:28
sdaguethat was rax-dfw20:28
sdaguewell, we used to run a lot less tests in this job20:28
mriedemsdague: i have a fix for that20:28
mriedemhttps://review.openstack.org/#/c/264393/20:28
*** ansiwen has quit IRC20:30
*** mdbooth has quit IRC20:30
*** migi has quit IRC20:31
*** ccarmack has joined #openstack-nova20:31
*** markmc` is now known as markmc20:32
sdaguemriedem: +220:32
sdaguewhen is the nova meeting today?20:32
mriedem28 minutes20:32
*** migi has joined #openstack-nova20:32
*** mdbooth has joined #openstack-nova20:32
*** ansiwen has joined #openstack-nova20:33
mriedemi already put py34 on the gate status section20:33
sdagueok20:33
mriedemwhich is why i was asking about the intent of the remove-mox bp20:33
mriedemsince i put that series in there20:33
jaypipesdansmith: :( since when did we agree that we would start supporting rolling upgrade of the controller services (i.e. run two different versions of the nova-conductor simultaneously)?20:33
*** lihg_ has quit IRC20:33
auggysdague: thanks for rebasing the patch, I totally forgot to do that20:34
*** jichen has quit IRC20:34
*** Marga_ has joined #openstack-nova20:35
*** Marga_ has quit IRC20:35
*** otter768 has joined #openstack-nova20:35
*** Marga_ has joined #openstack-nova20:35
rlrossitdansmith: I found my first fun-ness bug of DictCompat! http://paste.openstack.org/show/483264/20:36
rlrossityou can't set agg.availability_zone = None in kwargs or any time after20:37
rlrossitjaypipes: where was that said?20:38
*** matrohon has joined #openstack-nova20:39
*** dims has joined #openstack-nova20:39
*** otter768 has quit IRC20:40
bauzasrlrossit: IIRC az is not an object field for the Aggregate20:40
dansmithjaypipes: we talked about shooting for it during liberty, and agreed in tokyo that we'd limit any new online migrations if possible20:40
bauzasrlrossit: it's just a property20:41
dansmithjaypipes: why is that sadface? you like downtime? :)20:41
*** vilobhmm11 has joined #openstack-nova20:41
*** vilobhmm11 has quit IRC20:41
*** vilobhmm11 has joined #openstack-nova20:41
rlrossitbauzas: it's taken out of the metadata field, but the metadata field doesn't allow you to set agg.metadata['availability_zone'] = None20:41
bauzasrlrossit: yeah that https://github.com/openstack/nova/blob/master/nova/objects/aggregate.py#L150-L15120:42
*** eglynn has quit IRC20:42
bauzasrlrossit: since there is no property setter, it leaves the metadata to be set20:42
rlrossitbauzas: right, and I can't even set it to none via metadata20:42
*** cfriesen__ has joined #openstack-nova20:44
rlrossitagg.metadata['availability_zone'] = None ----> KABOOM20:45
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (functional tests)  https://review.openstack.org/26429720:45
*** lihg has joined #openstack-nova20:45
rlrossitevery time someone creates a class with NovaObjectDictCompat, a kitten dies20:46
* rlrossit looks at cdent20:46
sdagueauggy: no prob, I still don't have a full story around new samples files, as my testing does need them. But at least the testing will be in there.20:46
*** lihg_ has joined #openstack-nova20:48
*** unicell has quit IRC20:48
bauzasrlrossit: interesting20:49
bauzasrlrossit: but there is a default AZ, you know?20:49
rlrossitbauzas: test_create_with_null_availability_zone ignores that though :)20:50
*** lihg has quit IRC20:50
*** rfolco has joined #openstack-nova20:50
bauzasrlrossit: I miss the convo history20:50
rlrossitbauzas: basically, I'm trying to get rid of NovaObjectDictCompat for aggregate, and I learned the API tests were mocking a little too much out20:53
rlrossitdicts EVERYWHERE20:53
bauzasI see20:53
bauzasyeah the Aggregate object is a pretty old one AFAIR20:54
bauzasso I'm not surprised20:54
rlrossitwhy does it have to come first in the alphabet :(20:54
*** rfolco has quit IRC20:54
bauzascall it Zaggregate20:55
rlrossitso now I'm trying to convert the API tests to use the object instead of dicts, and I'm getting my new funness20:55
bauzasI see20:55
bauzasagain, I wonder something20:55
*** gjayavelu has joined #openstack-nova20:55
bauzasagg.az = foo should not update the metadata value for the az key20:55
bauzasbut rather should just set an az attr to the object20:56
bauzasbecause it's a property like I said20:56
bauzaswith no setter20:56
rlrossitbauzas: it's stored as an obj_extra_field, but the fact it's a read-only property right now you have to go through metadata20:56
rlrossitright20:56
rlrossitso we either need to make a setter or get rid of the prop altogether20:56
rlrossitis this bug-worthy?20:57
* rlrossit wonders if it's technically a bug in o.vo too20:57
bauzasI wonder the right approach20:57
*** matrohon has quit IRC20:58
rlrossitcan the almighty knowledge of dansmith come to the rescue?20:58
bauzasthat said, I feel a setter doesn't sound scary unless I'm missing some obviousness20:58
* dansmith reads20:58
mriedemnova meeting in 1 min20:59
rlrossitway to ruin the fun mriedem20:59
bauzasdansmith: tl;dr the Aggregate obj provides the az property which means that setting the az attribute directly doesn't really make sense20:59
dansmithyeah20:59
dansmithit's not a property though right?21:00
rlrossitand agg.metadata['az'] = None is also not allowed because of DictOfStrings coerce21:00
dansmithI mean,21:00
dansmithit shouldn't be a field21:00
bauzasbut since rlrossit is fool enough to change the existing dict accessors by using the dot notation21:00
rlrossitit's an obj_extra_field21:00
mriedemmikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes garyk edleafe dimsmoshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov sc68cal akuriata artom jlvillal mnestratov kashyap aloga rgeragnov bauzas xyang tpatil med_ nic scottda nagyz belliottdguitarbite sdague jroll diana_clarke _diana_ raildo jichen gjayavel21:00
jrollspam!21:00
dansmithyeah, so I would just change the tests to set it in the metadata21:00
bauzas+121:00
rlrossitdansmith: can't21:00
*** ociuhandu has joined #openstack-nova21:00
mriedemjroll: tell those people to figure out how to work a calendar21:00
rlrossitbecause of agg.metadata['az'] = None blows things up21:01
rlrossit(that was my first try)21:01
bauzaswell, that IRC ping is still useful :)21:01
jrollmriedem: blame exchange21:01
dansmithrlrossit: doesn't that mean the test is checking something that is not legit?21:01
*** changbl has joined #openstack-nova21:01
bauzasespecially when you're out of your computer but still get pings on your mobile phone when someone is bugging you :)21:01
*** bklei has quit IRC21:01
rlrossitdansmith: dunno21:01
rlrossitthere's lots of api tests for testing against null az's21:02
rlrossits/lots/like 4/21:02
*** matrohon has joined #openstack-nova21:03
*** raildo is now known as raildo-afk21:04
*** rdo has quit IRC21:04
openstackgerritIldiko Vancsa proposed openstack/nova-specs: Amend multi-attach volume spec with limitations  https://review.openstack.org/26496521:05
ildikovmriedem: amendments to the approved multi-attach spec ^^21:05
*** zenoway has joined #openstack-nova21:06
*** changbl has quit IRC21:06
*** rdo has joined #openstack-nova21:06
mriedemthanks21:06
ildikovnp, I hope I managed to capture our discussions21:07
openstackgerritIldiko Vancsa proposed openstack/nova-specs: Amend multi-attach volume spec with limitations  https://review.openstack.org/26496521:09
*** zenoway has quit IRC21:10
*** lbeliveau has quit IRC21:12
openstackgerritMark McLoughlin proposed openstack/nova: libvirt: tweak to migration flags mangling reno  https://review.openstack.org/26496921:12
*** lbeliveau has joined #openstack-nova21:12
openstackgerritIldiko Vancsa proposed openstack/nova-specs: Amend multi-attach volume spec with limitations  https://review.openstack.org/26496521:13
*** lpetrut has quit IRC21:13
*** haomaiwang has joined #openstack-nova21:14
openstackgerritSean Dague proposed openstack/nova: API sample testing without project_id  https://review.openstack.org/25608921:15
openstackgerritDiana Clarke proposed openstack/nova: Replace stubs.Set with stub_out (functional tests)  https://review.openstack.org/26429721:16
*** gcb has joined #openstack-nova21:16
*** unicell has joined #openstack-nova21:17
*** ccarmack has left #openstack-nova21:18
*** haomaiwang has quit IRC21:19
*** penick has quit IRC21:20
*** unicell has quit IRC21:21
*** unicell has joined #openstack-nova21:21
*** vladikr has quit IRC21:24
*** baoli has joined #openstack-nova21:25
*** baoli_ has joined #openstack-nova21:26
*** ccarmack1 has joined #openstack-nova21:26
*** smatzek has quit IRC21:28
*** nikhil_k is now known as nikhil21:29
*** baoli has quit IRC21:30
*** penick has joined #openstack-nova21:34
*** ijw has joined #openstack-nova21:34
*** ccarmack has joined #openstack-nova21:35
*** bklei has joined #openstack-nova21:35
*** baoli_ has quit IRC21:36
*** ljxiash has joined #openstack-nova21:36
*** _diana_1 has quit IRC21:37
*** baoli has joined #openstack-nova21:37
*** matrohon has quit IRC21:37
*** Sukhdev has joined #openstack-nova21:37
*** ccarmack1 has quit IRC21:38
*** matrohon has joined #openstack-nova21:38
*** baoli has quit IRC21:40
*** baoli has joined #openstack-nova21:40
*** ljxiash has quit IRC21:41
*** zenoway has joined #openstack-nova21:41
*** baoli has quit IRC21:45
*** ccarmack has left #openstack-nova21:45
*** zenoway has quit IRC21:45
*** bklei has quit IRC21:45
*** baoli has joined #openstack-nova21:46
rlrossitdansmith: bauzas: don't know if this ever came to a resolution, but how do we want to handle the null az's? remove the tests or fix the object?21:47
dansmithI don't know what the right answer is21:47
dansmithI suspect bauzas does though21:47
bauzasrlrossit: tbh, it's pretty late for me discussing that21:47
dansmithif null az is wrong, then fix the tests, otherwise fix the object to allow null21:47
bauzasrlrossit: but I need to check if we allow the az to be nullable by the API21:48
dansmithrlrossit: jaypipes might be a good one to ask21:48
bauzasrlrossit: because IIRC, az is just a key21:48
bauzasfor the aggregate21:48
bauzasrlrossit: which means that you can provide an agg without providing a metadata az21:48
sdagueI thought if you didn't set az it came back as 'nova'21:49
*** lihg_ has quit IRC21:49
bauzasand since it's a API call, it would mean that setting the az metadata would mean that it would be an empty string, not a None21:49
bauzassdague: 'nova' is the default AZ value for services21:49
melwitt there's a default az that applies if one isn't set on the aggregate, right?21:49
*** markmc has quit IRC21:50
*** kgalanov has quit IRC21:50
*** wasmum- has quit IRC21:50
*** renatoarmani has quit IRC21:50
*** jaosorior has quit IRC21:50
*** slagle has quit IRC21:50
*** jaypipes has quit IRC21:50
*** gongysh has quit IRC21:50
*** DinaBelova has quit IRC21:50
*** boris-42 has quit IRC21:50
*** ircuser-1 has quit IRC21:50
*** gsilvis has quit IRC21:50
bauzashere, the problem is purely about the aggregate resource AFAIU21:50
bauzasmeh, netsplit21:50
bauzasI mean21:51
rlrossitbauzas: sdague: the main test in question is https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/compute/test_aggregates.py#L220 which is stubbing out what the ag object returns21:51
bauzasthe default AZ value is for hosts21:51
rlrossitcurrently it returns a null az if one isn't specified21:51
*** markmc has joined #openstack-nova21:51
*** kgalanov has joined #openstack-nova21:51
*** wasmum- has joined #openstack-nova21:51
*** renatoarmani has joined #openstack-nova21:51
*** jaosorior has joined #openstack-nova21:51
*** slagle has joined #openstack-nova21:51
*** jaypipes has joined #openstack-nova21:51
*** gongysh has joined #openstack-nova21:51
*** DinaBelova has joined #openstack-nova21:51
*** boris-42 has joined #openstack-nova21:51
*** ircuser-1 has joined #openstack-nova21:51
*** gsilvis has joined #openstack-nova21:51
*** ircuser-1 has quit IRC21:51
bauzassec, checking the API21:51
rlrossitsdague: if it always returns at least *something* I would say these tests are invalid because the AggregateAPI will always *return* something21:52
* rlrossit *likes* *asterisks*21:52
*** Kennan has quit IRC21:52
sdaguethe asserts in a closure used as a stub is making me a little nauseous21:53
*** Kennan has joined #openstack-nova21:53
rlrossitsdague: agreed21:53
*** ircuser-1 has joined #openstack-nova21:53
rlrossitwhoever's doing self.stubs.Set is going to love this file21:54
*** cdent has quit IRC21:54
sdaguerlrossit: honestly, I just figure out what a better test is and write that, and delete this test21:55
bauzasokay, I verified21:55
*** thangp has quit IRC21:55
dansmithsdague: we don't know if we should allow the thing it's doing or not though21:56
bauzasand there is no way to set a nullable AZ21:56
bauzaslemme show you, folks21:56
dansmiththe other way to look at this is,21:56
bauzashttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L3496-L349721:56
dansmiththe unit tests can be wrong, but if we messed this up in icehouse (when this object was added) and haven't heard about it yet ...21:56
dansmiththat'd be fairly amazing21:56
dimssdague : the -2 on ec2 patch ...21:56
bauzasrlrossit: sdague: ^21:56
dimssdague : looks like i missed the meeting an hour ago (https://review.openstack.org/#/c/263368/)21:56
bauzasso if the az argument is none, we avoid it to be set21:57
sdaguedims: damn, we never talked about that one. johnthetubaguy was going to add the agenda21:57
*** lihg has joined #openstack-nova21:57
bauzasrlrossit: so, kill the testes IMHO21:57
bauzastests21:57
sdaguebauzas: yeh, that's what I vaguely remembered21:57
openstackgerritEd Leafe proposed openstack/nova: Move config options from nova/api directory  https://review.openstack.org/25818021:57
openstackgerritEd Leafe proposed openstack/nova: Move nova/opts.py to nova/conf/  https://review.openstack.org/25543621:57
openstackgerritEd Leafe proposed openstack/nova: single point of entry for sample config generation  https://review.openstack.org/26001521:57
openstackgerritEd Leafe proposed openstack/nova: Improve the help text for the API options (2)  https://review.openstack.org/26058721:57
openstackgerritEd Leafe proposed openstack/nova: Improve the help text for the API options (1)  https://review.openstack.org/26058621:57
openstackgerritEd Leafe proposed openstack/nova: Improve the help text for the API options (3)  https://review.openstack.org/26058821:57
rlrossitbauzas: heh, if they wouldn't have stubbed out so much they would've found that those tests are wrong by blowing up on what I'm seeing now21:57
rlrossitI will give them the axe then21:58
bauzasI checked and we allow to pass a None value from the wsgi resource to the compute API, but that above conditional avoids it to be persisted21:58
rlrossitor do we want me to remove them in a separate prequel patch?21:58
dansmithbauzas: please don't kill the testes21:58
dansmithrlrossit: prequel21:58
rlrossitrlrossit_commits++21:59
*** lihg has quit IRC21:59
bauzasyay21:59
bauzasI meant, it should be done in a sep patch21:59
*** suro-patz has quit IRC21:59
bauzasand probably we should have an unittest verifying that if we pass a nullable az, it's not part of the metadata21:59
dimssdague : want me to throw it to openstack-dev?21:59
bauzasunless it's already written somewhere :)22:00
dansmithbauzas: well, the object will reject that for us22:00
*** lihg has joined #openstack-nova22:00
dansmithbauzas: unless you mean confirm that it won't stringify the None, but I don't think it will22:00
bauzasdansmith: I'm thinking that at the API level22:00
*** cfriesen__ has quit IRC22:00
bauzasdansmith: to prevent some behavioural change that could occur if someone would drop that lofic22:00
bauzaslogic22:00
dansmithmeh, maybe, we rely on known-working validation in a lot of places but..22:01
bauzassurely, it would be raised by the object, but we could have some unpredictable side-effects22:01
*** thorst has quit IRC22:02
mriedemauggy: thanks for jumping on the bug triage week thing22:02
auggynp, i am still pretty new but maybe know just enough to maybe be useful22:02
*** xyang1 has quit IRC22:02
bauzasoh sec22:02
bauzasman22:02
bauzashttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L3519-L3520 is awesome22:02
tjonesim looking for a description on what i need to do to run python 34 tests - when i run tox -epy34 i get a bunch of errors about dependencys and unable to find python.h.  so i clearly have missed someting in my setup.  any pointers?22:02
bauzasbecause we can update the metadata by passing a null az22:03
_diana_22:03
bauzasI guess it's a bug22:03
rlrossitbauzas: which the object will reject22:03
bauzasfor good reasons22:03
*** vilobhmm11 has quit IRC22:03
edleafe_diana_: I agree!22:03
bauzasso, drop drop drop22:03
*** vilobhmm11 has joined #openstack-nova22:03
rlrossitshould I open a bug to link this to, or does this count as "clean up"?22:04
rlrossitbauzas: dansmith ^22:04
dansmithrlrossit: mriedem will say yes22:04
dansmithI would say no, given a healthy commit message22:04
mriedemso the test is expecting someting that is not real22:05
*** baoli has quit IRC22:05
mriedemor stubbing out not real things that would happen22:05
rlrossitthat one ^22:06
mriedemi don't care, if you're going to clean it up real quick22:06
*** baoli has joined #openstack-nova22:06
mriedemif i were passing through and wasn't going to work on it right now i'd report a bug and move on22:06
*** gjayavelu has quit IRC22:06
*** _diana_ has quit IRC22:07
*** markmc has quit IRC22:08
*** salv-orlando has joined #openstack-nova22:09
jaypipesdansmith: the sadface was for the ugliness this adds to the codebase..22:09
*** markmc has joined #openstack-nova22:09
*** gsilvis has quit IRC22:09
*** gsilvis has joined #openstack-nova22:10
dansmithjaypipes: that we need to handle both cases? or the specific way it does it?22:10
*** Marga_ has quit IRC22:10
jaypipesdansmith: both.22:10
dansmithjaypipes: so you'd rather not handle both cases and always require anything that talks to the database (api,conductor,scheduler) to have to be fully atomic any time we do an upgrade?22:11
jaypipesdansmith: I've never seen any scenario in the wild where a conductor node was not upgraded at the same time as api and other conductors... we're talking about milliseconds of downtime here...22:11
*** salv-orl_ has quit IRC22:11
dansmithjaypipes: we've never been able to tolerate it before... how is this different from what we do to support older computes talking to newer conductors?22:12
dansmithwe could go back to telling everyone they have to upgrade all the computes alongside the control services...22:12
*** matrohon has quit IRC22:12
*** bklei has joined #openstack-nova22:12
*** bklei has quit IRC22:13
dansmithit's also not milliseconds at all if we have to kill a conductor or api and expect it to finish what it was doing before it exits22:13
*** bklei has joined #openstack-nova22:13
jaypipesdansmith: at least with that we have an obj_make_compatible() interface that is standardized and not just random "here's a method that goes and grabs version information from the database via the objects.Service.get_xxx() call and determines every time we call save() whether or not to backport the object to some version.22:13
dansmithwhich is another goal we've had for a while.. conductors can be doing longer-running things like waiting for a scheduler reply22:14
openstackgerritChuck Carmack proposed openstack/nova: config options: centralize 'spice' options  https://review.openstack.org/26427122:14
dansmithjaypipes: save() is that standard interface, IMHO, and just like obj_make_compatible(), it's _highly_ dependent on each case22:14
*** penick has quit IRC22:15
openstackgerritRyan Rossiter proposed openstack/nova: Remove null AZ tests from API tests  https://review.openstack.org/26498722:15
dansmithjaypipes: if you've got ideas for how to make this more standard and repeatable, then let's do it22:15
rlrossitdansmith: let me know if the commit message of ^ is healthy enough22:15
*** gcb has quit IRC22:16
jaypipesdansmith: I would be much happier about this change in direction if I had yet been able to witness us being able to get rid of *any* old cruft from the object classes. as it stands now, it just seems like we are doomed to live in a cluttered house of code with no end in sight to backporting and backversioning schemas.22:16
*** suro-patz has joined #openstack-nova22:16
dansmithrlrossit: looks good to me22:16
*** claudiub has quit IRC22:16
*** haomaiwa_ has joined #openstack-nova22:16
anteayamriedem: carl_baldwin pointed out line 53 on the etherpad to me: https://etherpad.openstack.org/p/mitaka-nova-midcycle22:17
*** zenoway has joined #openstack-nova22:17
dansmithjaypipes: really? we dropped Instance v1 this cycle, and also did the full flavor metadata migration in an online migration just like this, and dropped *all* that code as soon as we could22:17
anteayamriedem: he also shared this link http://lists.openstack.org/pipermail/openstack-dev/2016-January/083342.html22:17
jaypipesdansmith: oh?22:17
dansmithjaypipes: yeah let me get you some links22:17
*** baoli has quit IRC22:17
anteayamriedem: he said he has been talking to John and the folks working on the scheduler22:17
anteayamriedem: I'm not sure, perhaps that information hasn't been conveyed to you yet22:18
dansmithjaypipes: https://github.com/openstack/nova/commit/1337890ace918fa2555046c01c8624be014ce2d822:18
*** baoli has joined #openstack-nova22:18
dansmithjaypipes: https://github.com/openstack/nova/commit/6f79646f71b66399599a4f56bcdc16471a20e00522:18
dansmithjaypipes: https://github.com/openstack/nova/commit/5e9df4baf97fd1f495b81d3b8b612e24f344b32522:19
dansmiththe last one was a removal from objects and code, then a drop from sqla models, then finally a collapse of the column from the actual schema22:19
dansmithfull lifecycke22:19
dansmither, lifecycle22:19
*** zenoway has quit IRC22:21
mriedemanteaya: doesn't need to be, i just haven't read through the midcycle etherpad yet22:21
mriedemamong many other things22:22
dansmithjaypipes: that -005 one is dropping the full online migration of flavors from being key=value pairs in system metadata to json blobs in instance_extra, with extra_specs merged in from the proper flavor22:22
anteayamriedem: ah okay, sorry my mistake22:22
*** haomaiwa_ has quit IRC22:22
anteayamriedem: well so it appears that neutron has at least one thing they would like to discuss at the mid-cycle22:22
dansmithjaypipes: a much larger and more complicated dance than ndipanov is doing by moving one thing out of a metadata blob into a proper field, and we held that compatibility code for one cycle22:22
anteayamriedem: I've told armax if he has more items to add them22:22
jaypipesdansmith: well, I stand corrected. apologies.22:23
*** renatoarmani has quit IRC22:23
dansmithjaypipes: I'd like to be dropping more cruft from more objects, but I'm pretty much the only one that works on those, and things have been a little busy this cycle, as I'm sure you can appreciate :)22:24
*** pradk has quit IRC22:24
jaypipesdansmith: I do appreciate that. do we have a list of them that can be deprecated or is it up to individuals to grep-fu it?22:24
dansmithjaypipes: rlrossit is dropping dict cruft from objects as we speak22:24
dansmithjaypipes: I could make up a list based on the length of obj_make_compatible() in each object22:25
rlrossitdansmith: hey! I work on them too!22:25
dansmithjaypipes: some have a bunch of changes, but they aren't changes that cost us anything (i.e. new methods)22:25
jaypipesk22:25
dansmithrlrossit: hence me showering you with glory just a few lines above! :)22:25
jaypipesdansmith: would be great to get an etherpad to track what needs to be cleaned up and divvy up the work.22:25
dansmithsure22:26
*** thorst_ has joined #openstack-nova22:27
*** suro-patz has quit IRC22:27
*** baoli has quit IRC22:29
*** baoli has joined #openstack-nova22:30
dimsdansmith : fyi, Add a grenade job for testing oslo.versionedobjects  https://review.openstack.org/26499222:30
*** vishwanathj has quit IRC22:30
dimsdansmith : mriedem : based on our conversations yesterday22:30
dansmithdims: okay, we really need that to be the grenade multinode job22:31
dimsdansmith : ack. let me switch22:31
dansmithdims: I mean, probably worth having this too I guess, but  the multinode one is how we'll detect if we broke something on the wire22:31
dansmithdims: thanks for doing that22:31
mriedemdims: you want it to be non-voting?22:31
*** thorst_ has quit IRC22:32
dansmithI thought this was going to be "check experimental" type stuff22:32
dimsdansmith : voting false22:32
*** penick has joined #openstack-nova22:32
dimsdansmith : runs only for o.vo project, not nova, we don't have that much churn22:33
mriedemand it's only check queue22:33
dansmithokay cool22:33
dimsmriedem : y22:33
mriedemso check non-voting22:33
mriedemwhy not gating and voting?22:33
dansmithnon-voting is better, I just thought maybe not because it's super expensive22:33
dansmith(better than experimental)22:33
dimsdansmith : y, if it was for nova22:34
mriedemi guess i don't understand22:34
mriedemif you're going to run it anyway, and it's working,22:34
mriedemit might as well be voting22:34
mriedemthrowing it in the experimental queue makes it less expensive since it's on-demand, but then people forget to run it22:35
dimsmriedem : we are using it for oslo.messaging as non-voting for a while22:35
mriedemso this just copies that22:35
dimsy22:35
mriedemis it passing in oslo.messaging?22:35
dansmithit can be voting in check if we want, I think that's probably fine22:35
dimssome hiccups off and on. don't want it voting22:36
dansmithI mean it can be voting in gate too, I just didn't think it was  .. yeah ^22:36
mriedemwhateves22:36
mriedemignore the peanut gallery22:36
*** otter768 has joined #openstack-nova22:36
dansmithI'm also terrified of the love pentagon of testing that we're about to unleash,22:36
*** gyee has joined #openstack-nova22:36
dansmithwhich means more things gated leads to more squashed hail-mary patch fixes22:36
dansmithespecially on something like o.vo I think22:37
dimsdansmith : like sdague said, too much then we can't land anything22:37
dansmithright22:37
mriedemtoo much running or too much voting?22:37
dansmithvoting22:37
mriedemb/c this makes it still running22:37
mriedemok22:37
mriedemmaking a job non-voting is easy if it's busted/blocked22:37
dansmithcome on I get nothing for "love pentagon" ?22:37
mriedemi smiled22:38
dansmithhah22:38
mriedemmakes me think of something something dick cheney22:38
mriedemhalliburton + dick cheney + NSA + pentagon + multinode job22:38
mriedem= kaboom22:38
dansmithhmm.22:39
dimslol22:39
*** suro-patz has joined #openstack-nova22:39
dansmithnot the pentagonal relationship I was alluding to22:39
mriedemsatan?!22:39
*** suro-patz has quit IRC22:39
dansmithalthough similarly incestuous22:39
*** eharney has quit IRC22:39
*** otter768 has quit IRC22:40
*** baoli has quit IRC22:41
*** baoli has joined #openstack-nova22:41
*** suro-patz has joined #openstack-nova22:42
*** tonytan4ever has quit IRC22:42
*** suro-patz has quit IRC22:42
*** lihg has quit IRC22:43
dimsdansmith : do you want to figure out which pieces of code from o.vo master needs to be in stable/liberty branch?22:44
dansmithdims: well, I tested against nova's stable unit tests for 0.9 and 0.10 and both are fine, now that we backported the removal of that test22:44
dimsdansmith : we could cut a release and get it ready for use22:44
dansmithdims: so I _think_ that we're probably okay to roll back, but I wanted to see those test runs to be sure22:44
dimsdansmith : ah ok22:45
*** _diana_ has joined #openstack-nova22:45
*** _diana_ has quit IRC22:45
openstackgerritRyan Rossiter proposed openstack/nova: Remove NovaObjectDictCompat from Aggregate  https://review.openstack.org/26439722:45
rlrossitcome on tempest, show me some love22:45
mriedemrlrossit: question in https://review.openstack.org/#/c/264987/22:47
rlrossitthere goes mriedem ruining the party again22:47
rlrossitdansmith: mriedem raises a good point in his question22:49
dansmithrlrossit: not possible22:49
rlrossitso instead of setting it to None, I could just not set it, and the property will return it as none22:49
*** ijw has quit IRC22:50
mriedemi have to head out anyway, i might just be missing something obvious22:51
dansmithI just commented22:52
dansmithit's really just a holdover from the dict days where we had a .get() and checked for none to do the default22:52
dansmithit was for checking the "not specified" case, not the "is actually set to None" case22:52
*** zenoway has joined #openstack-nova22:52
rlrossitdansmith: but does leaving the none test in affect how https://review.openstack.org/#/c/264397/2/nova/api/openstack/compute/aggregates.py will work?22:53
*** baoli has quit IRC22:53
*** mriedem has quit IRC22:54
rlrossit(not affect how it works, but at least how it's tested)22:54
*** baoli has joined #openstack-nova22:54
dansmithrlrossit: you could fix the tests to use an object with no az in the metadata22:55
rlrossitI'm going to -W my removal change for now, and change over the tests to not have metadata in my other change22:56
*** zenoway has quit IRC22:57
dansmithI dunno, I'm conflicted22:57
dansmithactually, let me comment on there again22:58
dansmithrlrossit: was bauzas saying that doing this create and specifying None was actually allowed?22:58
dansmithnm, foudn it22:59
rlrossitpassing None to the API is allowed, but I think it doesn't set metadata then?22:59
dansmithright23:00
dansmithwhich this doesn't test anyway23:00
*** dims_ has joined #openstack-nova23:00
*** dims has quit IRC23:01
rlrossitdansmith: the one thing I did find though is if you don't set metadata, and then try to access the az prop, it will blow up because metadata isn't loaded23:01
dansmithrlrossit: where is the None actually ignored? I thought it was in compute_api, which is mocked out here?23:01
*** burt has quit IRC23:02
dansmithyeah, it is23:02
dansmithwhich means this test, even if fixed, is not testing anything23:02
*** sacharya has quit IRC23:02
rlrossitdansmith: yeah, the None is ignored in the compute_api23:02
*** sacharya has joined #openstack-nova23:03
rlrossitbut None being ignored by the API, followed by an AZ access results in None23:03
dansmiththe compute_api just passes what it got (if anything) for az to compute_api, which does the actual test23:04
dansmiththis is mocking the ignoring behavior out23:04
dansmithwhich means it's not testing anything real, IMHO23:04
rlrossitdansmith: to be fair, most of these tests are mocking, like, all functionality out23:05
dansmithrlrossit: yeah, I know.. tons of useless in there23:05
*** baoli has quit IRC23:05
rlrossitdansmith: I'm just going to leave this until tomorrow... it's time to go home23:05
*** baoli has joined #openstack-nova23:06
dansmithas you wish :)23:06
*** erlon has quit IRC23:06
openstackgerritEoghan Glynn proposed openstack/nova: doc: corrections to the server concepts docco  https://review.openstack.org/26483923:06
*** rlrossit has quit IRC23:07
*** sacharya has quit IRC23:07
*** ctrath has quit IRC23:13
*** tonytan4ever has joined #openstack-nova23:14
*** ijw has joined #openstack-nova23:15
*** baoli has quit IRC23:17
*** tonytan4ever has quit IRC23:18
*** baoli has joined #openstack-nova23:18
*** mdrabe has quit IRC23:18
*** mragupat_ has quit IRC23:20
*** bklei has quit IRC23:23
*** zenoway has joined #openstack-nova23:28
*** baoli has quit IRC23:29
*** baoli has joined #openstack-nova23:30
*** zenoway has quit IRC23:32
*** kfarr has quit IRC23:33
*** ljxiash has joined #openstack-nova23:38
*** dims_ has quit IRC23:39
*** doug-fish has quit IRC23:41
*** ljxiash has quit IRC23:43
*** baoli has quit IRC23:44
*** mriedem has joined #openstack-nova23:44
*** dims_ has joined #openstack-nova23:45
*** lihg has joined #openstack-nova23:46
*** lihg_ has joined #openstack-nova23:46
*** sacharya has joined #openstack-nova23:47
*** dims_ has quit IRC23:50
*** ducttape_ has quit IRC23:51
*** sacharya has quit IRC23:51
*** ducttape_ has joined #openstack-nova23:51
*** ducttape_ has quit IRC23:55

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