Thursday, 2016-03-10

*** openstack has joined #openstack-nova00:10
*** ducttape_ has quit IRC00:12
*** davideag_ has quit IRC00:14
*** davideagnello has joined #openstack-nova00:14
*** shaohe_feng has quit IRC00:16
*** shaohe_feng has joined #openstack-nova00:16
*** openstackgerrit has quit IRC00:17
*** dimtruck is now known as zz_dimtruck00:17
*** openstackgerrit has joined #openstack-nova00:18
*** dave-mccowan has joined #openstack-nova00:20
*** vilobhmm11 has joined #openstack-nova00:20
*** haukebruno has joined #openstack-nova00:22
*** shaohe_feng has quit IRC00:26
*** irenab has quit IRC00:26
*** shaohe_feng has joined #openstack-nova00:26
*** irenab has joined #openstack-nova00:29
*** yamahata has joined #openstack-nova00:29
*** dims has quit IRC00:35
*** shaohe_feng has quit IRC00:36
*** rk4n has quit IRC00:37
*** shaohe_feng has joined #openstack-nova00:38
*** BobBall_ has joined #openstack-nova00:43
*** BobBall_ is now known as Guest310200:44
*** vilobhmm11 has quit IRC00:44
*** isq_ has joined #openstack-nova00:46
*** shaohe_feng has quit IRC00:46
*** shaohe_feng has joined #openstack-nova00:46
*** aysyd has quit IRC00:47
*** cfriesen__ has joined #openstack-nova00:47
*** Nakato_ has joined #openstack-nova00:48
*** clduser_ has joined #openstack-nova00:48
*** allen_gao has quit IRC00:49
*** efried has quit IRC00:49
*** cfriesen_ has quit IRC00:49
*** mkoderer__ has quit IRC00:49
*** harlowja_at_home has quit IRC00:49
*** zzzeek has quit IRC00:49
*** gmann has quit IRC00:49
*** clduser has quit IRC00:49
*** edleafe has quit IRC00:49
*** BobBall has quit IRC00:49
*** jbernard_ has quit IRC00:49
*** ccesario has quit IRC00:49
*** isq has quit IRC00:49
*** jdillaman has quit IRC00:49
*** Nakato has quit IRC00:49
*** gmann has joined #openstack-nova00:49
*** ccesario has joined #openstack-nova00:49
*** harlowja_at_home has joined #openstack-nova00:50
*** allen_gao has joined #openstack-nova00:50
*** mkoderer___ has joined #openstack-nova00:50
*** jbernard_ has joined #openstack-nova00:50
*** zzzeek has joined #openstack-nova00:50
*** efried has joined #openstack-nova00:51
*** edleafe has joined #openstack-nova00:52
*** jdillaman has joined #openstack-nova00:52
*** ssurana has quit IRC00:55
*** shaohe_feng has quit IRC00:57
*** jschlueter has joined #openstack-nova00:57
*** shaohe_feng has joined #openstack-nova00:57
*** Jeffrey4l has joined #openstack-nova00:59
*** haomaiwang has quit IRC01:01
*** haomaiwang has joined #openstack-nova01:01
*** markus_z has quit IRC01:04
*** Jeffrey4l has quit IRC01:04
*** shaohe_feng has quit IRC01:07
*** shaohe_feng has joined #openstack-nova01:08
*** markusz has joined #openstack-nova01:08
*** ducttape_ has joined #openstack-nova01:08
*** angdraug has quit IRC01:09
*** Daisy has joined #openstack-nova01:11
*** ducttape_ has quit IRC01:14
*** wasmum has quit IRC01:15
*** ducttape_ has joined #openstack-nova01:15
*** yonglihe has joined #openstack-nova01:15
*** thorst_ has quit IRC01:15
*** Daisy has quit IRC01:16
*** shaohe_feng has quit IRC01:17
*** thorst_ has joined #openstack-nova01:17
*** shaohe_feng has joined #openstack-nova01:17
openstackgerritGiridhar Jayavelu proposed openstack/nova: tests: fix false positive in test_set_host_enabled*  https://review.openstack.org/29091901:18
*** Sukhdev has joined #openstack-nova01:20
*** wasmum has joined #openstack-nova01:20
*** RuiChen has quit IRC01:20
*** RuiChen has joined #openstack-nova01:21
*** Sukhdev has quit IRC01:22
*** RuiChen has quit IRC01:23
*** RuiChen has joined #openstack-nova01:24
*** thorst_ has quit IRC01:24
*** RuiChen has quit IRC01:24
*** Sukhdev has joined #openstack-nova01:24
*** mriedem has joined #openstack-nova01:25
*** RuiChen has joined #openstack-nova01:25
*** RuiChen has quit IRC01:25
*** RuiChen has joined #openstack-nova01:26
*** RuiChen has quit IRC01:27
*** allen_gao has quit IRC01:27
*** shaohe_feng has quit IRC01:27
*** Sukhdev has quit IRC01:27
*** RuiChen has joined #openstack-nova01:28
*** Sukhdev has joined #openstack-nova01:28
*** aorourke is now known as aorourke-afk01:28
*** RuiChen has quit IRC01:28
*** RuiChen has joined #openstack-nova01:29
*** shaohe_feng has joined #openstack-nova01:30
*** dims has joined #openstack-nova01:31
mriedemi wonder if we can enable live snapshots in the ceph job, or if the issues with libvirt 1.2.2 would also happen in the ceph job01:32
*** RuiChen has quit IRC01:32
*** RuiChen has joined #openstack-nova01:33
*** shaohe_feng has quit IRC01:38
*** rk4n has joined #openstack-nova01:38
*** allen_gao has joined #openstack-nova01:38
*** shaohe_feng has joined #openstack-nova01:39
*** Sukhdev has quit IRC01:39
*** Sukhdev has joined #openstack-nova01:39
*** zenoway has joined #openstack-nova01:39
*** Sukhdev has quit IRC01:41
*** Sukhdev has joined #openstack-nova01:42
*** mtanino has quit IRC01:43
*** zenoway has quit IRC01:43
*** edmondsw has quit IRC01:44
*** rk4n has quit IRC01:45
*** ducttape_ has quit IRC01:45
*** armax has quit IRC01:46
*** Sukhdev has quit IRC01:46
*** Sukhdev_ has joined #openstack-nova01:46
auggyi'm trying to start up my devstack and i keep getting this error from the nova api startup "ImportError: cannot import name secretutils"01:47
auggyi tried doing a pip upgrade on oslo.utils but it says it's up to date?? yargh i'm so confused01:47
*** shaohe_feng has quit IRC01:48
*** shaohe_feng has joined #openstack-nova01:49
*** Sukhdev has joined #openstack-nova01:49
dimsauggy : i typically go to my /opt/stack/nova and run "pip install -U -r requirements.txt -r test-requirements.txt" to update to latest of everything01:49
*** Sukhdev_ has quit IRC01:49
auggydims: oh right, thanks, i usually do that too ;D01:49
* auggy is on pain meds for a back issue so a little spacy01:49
*** fawadk has quit IRC01:49
auggyi'll see if that gets me the right thing01:49
*** Sukhdev has quit IRC01:51
*** Sukhdev has joined #openstack-nova01:52
*** mingyu has joined #openstack-nova01:52
*** thorst_ has joined #openstack-nova01:52
auggydims: thanks that totally works01:53
*** Sukhdev has quit IRC01:53
* auggy slaps forehead01:53
dimsauggy : been there done that :)01:53
*** Sukhdev has joined #openstack-nova01:54
auggy:)01:54
*** gongysh has joined #openstack-nova01:55
*** Sukhdev has quit IRC01:56
*** Sukhdev has joined #openstack-nova01:56
*** Jeffrey4l has joined #openstack-nova01:57
*** Sukhdev has quit IRC01:58
*** shaohe_feng has quit IRC01:58
*** haomaiwang has quit IRC01:58
*** Sukhdev has joined #openstack-nova01:59
*** shaohe_feng has joined #openstack-nova01:59
*** Sukhdev has quit IRC02:01
*** Sukhdev has joined #openstack-nova02:01
openstackgerritBin Zhou proposed openstack/nova: upgrade connection_info when Ceph mon IP changed  https://review.openstack.org/28981302:04
*** fawadkhaliq has joined #openstack-nova02:05
openstackgerritBin Zhou proposed openstack/nova: upgrade connection_info when Ceph mon IP changed  https://review.openstack.org/28981302:05
*** diana_clarke2 has quit IRC02:06
*** shaohe_feng has quit IRC02:08
*** shaohe_feng has joined #openstack-nova02:09
dansmithmriedem: do you want to go ahead and land those ceph fixes on stable or what?02:16
dansmithmriedem: ah, nevermind02:17
*** hoangcx has joined #openstack-nova02:18
*** thorst_ has quit IRC02:18
*** shaohe_feng has quit IRC02:19
*** thorst_ has joined #openstack-nova02:19
*** shaohe_feng has joined #openstack-nova02:20
mriedemdansmith: wayyyy ahead of you02:20
mriedemdansmith: also, you might get the itch to -2 this https://review.openstack.org/#/c/289813/02:20
dansmitho.my02:21
*** Sukhdev_ has joined #openstack-nova02:24
*** Sukhdev has quit IRC02:24
*** baoli has joined #openstack-nova02:27
*** thorst_ has quit IRC02:28
*** shaohe_feng has quit IRC02:29
*** shaohe_feng has joined #openstack-nova02:30
*** Sukhdev_ has quit IRC02:30
*** Sukhdev has joined #openstack-nova02:31
*** Sukhdev has quit IRC02:35
*** Sukhdev has joined #openstack-nova02:35
*** zenoway has joined #openstack-nova02:38
*** ociuhandu has quit IRC02:38
*** Sukhdev has quit IRC02:38
*** ociuhandu has joined #openstack-nova02:39
*** Sukhdev has joined #openstack-nova02:39
*** shaohe_feng has quit IRC02:39
*** baoli has quit IRC02:40
*** shaohe_feng has joined #openstack-nova02:40
*** eharney has quit IRC02:40
*** Jeffrey4l has quit IRC02:41
*** tbachman has quit IRC02:41
*** zenoway has quit IRC02:42
*** Sukhdev has quit IRC02:42
*** Sukhdev has joined #openstack-nova02:42
*** Jeffrey4l has joined #openstack-nova02:44
*** Sukhdev has quit IRC02:45
*** Sukhdev has joined #openstack-nova02:46
*** houming has joined #openstack-nova02:47
*** efried has quit IRC02:47
openstackgerritMatt Riedemann proposed openstack/nova: Set personality/injected_files to empty list if not specified  https://review.openstack.org/24866202:48
*** Sukhdev has quit IRC02:48
*** Sukhdev has joined #openstack-nova02:49
*** shaohe_feng has quit IRC02:49
*** shaohe_feng has joined #openstack-nova02:50
*** Sukhdev has quit IRC02:52
*** Sukhdev has joined #openstack-nova02:52
*** fawadkhaliq has quit IRC02:53
*** Marga_ has quit IRC02:54
*** Sukhdev has quit IRC02:55
*** Sukhdev_ has joined #openstack-nova02:55
mriedemtonyb: normally i wouldn't do https://review.openstack.org/#/c/289428/ but it's needed to get the tempest change to land https://review.openstack.org/#/c/288966/02:55
*** Marga_ has joined #openstack-nova02:56
*** Sukhdev_ has quit IRC02:58
*** Sukhdev has joined #openstack-nova02:59
*** shaohe_feng has quit IRC03:00
*** dansmith has quit IRC03:01
*** shaohe_feng has joined #openstack-nova03:01
*** Marga_ has quit IRC03:01
*** dansmith has joined #openstack-nova03:01
*** Sukhdev has quit IRC03:01
*** RuiChen has quit IRC03:02
*** Sukhdev has joined #openstack-nova03:02
*** RuiChen has joined #openstack-nova03:03
*** Sukhdev has quit IRC03:05
*** Sukhdev_ has joined #openstack-nova03:05
mriedemholy crap maybe we should fix this in newton https://blueprints.launchpad.net/nova/+spec/validate-project-with-keystone03:05
mriedemcheck the ever expanding list of bugs for that03:06
mriedemauggy: maybe something you want to tackle in newton? ^03:06
mriedemthere was POC code at one point, would just have to be revived and polished03:06
auggymriedem: sure i'll take a look03:07
auggywow lots of bugs against that one03:08
*** Sukhdev_ has quit IRC03:08
*** Sukhdev has joined #openstack-nova03:08
* auggy has been wrasslin with dev environments today03:08
mriedembasically,03:08
mriedemour quota-update API allows you to pass garbage for a tenant03:09
mriedemif the quotas don't exist for the tenant, it creates new defaults for it03:09
mriedemeven if the tenant doesn't actually exist in keystone03:09
auggyoh lovely03:09
mriedemso it's just a change to add a validation of the tenant in keystone03:09
mriedembut it was awhile ago when i dug into that one, and there might be more to it03:09
*** shaohe_feng has quit IRC03:10
auggyoh there always is03:10
* auggy has come to expect it ;D03:10
*** thorst_ has joined #openstack-nova03:10
auggyshould i try to contact the spec owner or just take over the spec?03:10
*** ducttape_ has joined #openstack-nova03:11
*** shaohe_feng has joined #openstack-nova03:11
*** ducttape_ has quit IRC03:11
*** Sukhdev_ has joined #openstack-nova03:11
*** Sukhdev has quit IRC03:11
openstackgerritJiajun Liu proposed openstack/nova: Replace deprecated LOG.warn with LOG.warning  https://review.openstack.org/29095803:12
*** tbachman has joined #openstack-nova03:12
*** zenoway has joined #openstack-nova03:13
*** Sukhdev_ has quit IRC03:14
*** thorst_ has quit IRC03:14
*** dims has quit IRC03:14
*** Sukhdev has joined #openstack-nova03:14
mriedemauggy: you could try contacting them first03:14
*** baoli has joined #openstack-nova03:14
mriedembut if no dice, just take it over03:14
*** thorst_ has joined #openstack-nova03:15
auggymriedem: sure thing03:15
*** Sukhdev has quit IRC03:15
* auggy adds it to her todo list03:15
auggymriedem: thanks for thinking of me!03:15
mriedemnp03:16
mriedemmaybe you can justify working on it with sdague because it remotely has something to do with project IDs03:16
mriedemand he hates all of that stuff03:17
*** zz_dimtruck is now known as dimtruck03:17
* mriedem sneaks off for the night, ttyl03:17
*** mriedem has quit IRC03:17
*** zenoway has quit IRC03:17
*** apoorvad has quit IRC03:18
*** shaohe_feng has quit IRC03:20
*** shaohe_feng has joined #openstack-nova03:21
tonybmriedem: That's basically what I thought.  I'm just a little worried about setting a "we'll do this for tempest" precedent03:21
*** thorst_ has quit IRC03:23
*** bapalm has quit IRC03:24
*** baoli has quit IRC03:24
*** fawadkhaliq has joined #openstack-nova03:24
*** RuiChen has quit IRC03:24
*** RuiChen has joined #openstack-nova03:26
*** markvoelker has joined #openstack-nova03:29
*** shaohe_feng has quit IRC03:30
*** shaohe_feng has joined #openstack-nova03:31
*** fawadkhaliq has quit IRC03:34
*** houming has quit IRC03:39
*** shaohe_feng has quit IRC03:41
*** shaohe_feng has joined #openstack-nova03:41
*** bapalm has joined #openstack-nova03:43
*** rk4n has joined #openstack-nova03:43
*** markvoelker has quit IRC03:43
*** RuiChen has quit IRC03:45
*** houming has joined #openstack-nova03:45
*** rk4n has quit IRC03:47
*** rk4n has joined #openstack-nova03:47
*** yamahata has quit IRC03:49
*** zenoway has joined #openstack-nova03:49
*** Marga_ has joined #openstack-nova03:50
*** links has joined #openstack-nova03:50
*** shaohe_feng has quit IRC03:51
*** Marga_ has quit IRC03:51
*** shaohe_feng has joined #openstack-nova03:51
*** Marga_ has joined #openstack-nova03:51
*** zenoway has quit IRC03:53
*** diana_clarke1 has joined #openstack-nova03:54
*** vilobhmm11 has joined #openstack-nova03:58
*** shaohe_feng has quit IRC04:01
*** dims has joined #openstack-nova04:01
*** shaohe_feng has joined #openstack-nova04:02
*** jwcroppe has joined #openstack-nova04:02
*** jaosorior has quit IRC04:02
*** jaosorior has joined #openstack-nova04:03
*** mingyu has quit IRC04:03
*** gjayavel_ has joined #openstack-nova04:05
*** vilobhmm11 has quit IRC04:10
*** gongysh has quit IRC04:11
*** shaohe_feng has quit IRC04:11
*** shaohe_feng has joined #openstack-nova04:12
*** dims has quit IRC04:13
*** nikhil has joined #openstack-nova04:15
*** vilobhmm11 has joined #openstack-nova04:16
*** thorst_ has joined #openstack-nova04:21
*** shaohe_feng has quit IRC04:22
*** shaohe_feng has joined #openstack-nova04:23
*** zenoway has joined #openstack-nova04:25
*** mingyu has joined #openstack-nova04:26
*** shaohe_feng has quit IRC04:28
*** thorst_ has quit IRC04:28
*** zenoway has quit IRC04:29
*** shaohe_feng_ has joined #openstack-nova04:30
*** mingyu has quit IRC04:30
*** shaohe_feng has joined #openstack-nova04:33
*** shaohe_feng_ has quit IRC04:36
*** amotoki has joined #openstack-nova04:38
*** harlowja_at_home has quit IRC04:39
*** harlowja_at_home has joined #openstack-nova04:39
*** sudipto has joined #openstack-nova04:41
*** jamielennox is now known as jamielennox|away04:41
*** shaohe_feng has quit IRC04:42
*** shaohe_feng has joined #openstack-nova04:43
yonglihemriedem, ping04:47
*** DinaBelova has quit IRC04:51
*** kragniz has quit IRC04:52
*** shaohe_feng has quit IRC04:52
*** shaohe_feng has joined #openstack-nova04:53
*** DinaBelova has joined #openstack-nova04:53
yonglihemriedem: it's seems late for you,  will try to find you next day.   i leave the summary message here:  about SRIOV testing for Nova, for our CI, SRIOV now just post on Neutron, and is different configurations/setup/machines.   To get it work for nova, it's like a new CI internally perspective.   let's discuss this when you back.04:53
*** kragniz has joined #openstack-nova04:54
yonglihemriedem: in case you did not catch me,  send me email also work.04:54
*** heyongli has joined #openstack-nova04:57
*** heyongli has quit IRC05:00
*** heyongli has joined #openstack-nova05:00
*** zenoway has joined #openstack-nova05:00
*** shaohe_feng has quit IRC05:03
*** shaohe_feng has joined #openstack-nova05:03
*** zenoway has quit IRC05:04
*** kragniz has quit IRC05:05
*** kragniz has joined #openstack-nova05:05
*** heyongli has quit IRC05:10
*** heyongli has joined #openstack-nova05:10
*** houming has quit IRC05:11
*** shaohe_feng has quit IRC05:13
*** shaohe_feng has joined #openstack-nova05:13
*** Jeffrey4l has quit IRC05:15
*** heyongli has quit IRC05:20
*** heyongli has joined #openstack-nova05:20
*** EinstCrazy has joined #openstack-nova05:21
*** shaohe_feng has quit IRC05:23
*** shaohe_feng has joined #openstack-nova05:24
*** dimtruck is now known as zz_dimtruck05:25
*** mdrabe has quit IRC05:25
*** jwcroppe has quit IRC05:25
*** zz_dimtruck is now known as dimtruck05:25
*** jwcroppe has joined #openstack-nova05:25
*** thorst_ has joined #openstack-nova05:26
*** jwcroppe has quit IRC05:30
*** heyongli has quit IRC05:30
*** hieulq has joined #openstack-nova05:31
*** hieulq has quit IRC05:31
*** FL1SK has quit IRC05:32
*** thorst_ has quit IRC05:33
*** sridharg has joined #openstack-nova05:33
*** shaohe_feng has quit IRC05:33
*** shaohe_feng has joined #openstack-nova05:34
*** baoli has joined #openstack-nova05:36
*** zenoway has joined #openstack-nova05:36
*** y_sawai has joined #openstack-nova05:36
*** harlowja_at_home has quit IRC05:36
*** jaosorior has quit IRC05:40
*** dansmith has quit IRC05:40
*** yonglihe has quit IRC05:40
*** cfriesen__ has quit IRC05:40
*** whenry has quit IRC05:40
*** cfarquhar has quit IRC05:40
*** baoli has quit IRC05:40
*** zenoway has quit IRC05:40
openstackgerritAbhishek Talwar proposed openstack/python-novaclient: flavor-id should be only integer or UUID  https://review.openstack.org/28770205:40
*** shaohe_feng has quit IRC05:44
*** shaohe_feng has joined #openstack-nova05:45
*** dansmith has joined #openstack-nova05:46
*** yonglihe has joined #openstack-nova05:46
*** cfriesen__ has joined #openstack-nova05:46
*** whenry has joined #openstack-nova05:46
*** cfarquhar has joined #openstack-nova05:46
*** jaosorior has joined #openstack-nova05:46
*** jaosorior has quit IRC05:46
*** 16WAADQPV has joined #openstack-nova05:49
*** armax has joined #openstack-nova05:50
*** irenab has quit IRC05:51
*** shaohe_feng has quit IRC05:54
*** melwitt has quit IRC05:54
*** shaohe_feng has joined #openstack-nova05:55
*** brad[] has quit IRC05:56
*** med_ has quit IRC05:56
*** mfisch has quit IRC05:56
*** med_ has joined #openstack-nova05:56
*** brad[] has joined #openstack-nova05:56
*** med_ is now known as Guest9366405:56
*** mingyu has joined #openstack-nova05:57
*** melwitt has joined #openstack-nova05:58
*** mingyu has quit IRC05:59
*** 16WAADQPV has quit IRC06:01
*** haomaiwang has joined #openstack-nova06:01
*** rk4n has quit IRC06:02
*** shaohe_feng has quit IRC06:02
*** dansmith has quit IRC06:02
*** yonglihe has quit IRC06:02
*** cfriesen__ has quit IRC06:02
*** whenry has quit IRC06:02
*** cfarquhar has quit IRC06:02
*** dimtruck is now known as zz_dimtruck06:03
*** EinstCrazy has quit IRC06:05
*** shaohe_feng has joined #openstack-nova06:05
*** fawadkhaliq has joined #openstack-nova06:07
*** dansmith has joined #openstack-nova06:08
*** yonglihe has joined #openstack-nova06:08
*** cfriesen__ has joined #openstack-nova06:08
*** whenry has joined #openstack-nova06:08
*** cfarquhar has joined #openstack-nova06:08
openstackgerritZhiQiang Fan proposed openstack/nova: Use oslo_config new type PortOpt for port options  https://review.openstack.org/23834706:09
*** zenoway has joined #openstack-nova06:11
*** mingyu has joined #openstack-nova06:13
*** gjayave__ has joined #openstack-nova06:14
*** shaohe_feng has quit IRC06:14
*** zenoway has quit IRC06:17
*** gjayavel_ has quit IRC06:17
*** shaohe_feng has joined #openstack-nova06:17
*** ivase_ has joined #openstack-nova06:17
*** rk4n has joined #openstack-nova06:19
*** ivase has quit IRC06:20
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata  https://review.openstack.org/28884506:21
*** bapalm has quit IRC06:23
*** shaohe_feng has quit IRC06:25
*** shaohe_feng has joined #openstack-nova06:26
*** mfisch has joined #openstack-nova06:27
*** mfisch is now known as Guest3848406:27
*** fawadkhaliq has quit IRC06:30
*** diana_clarke1 has quit IRC06:30
*** thorst_ has joined #openstack-nova06:31
*** shaohe_feng has quit IRC06:35
*** shaohe_feng has joined #openstack-nova06:36
*** thorst_ has quit IRC06:38
*** sridharg has quit IRC06:40
*** bapalm has joined #openstack-nova06:42
*** melwitt has quit IRC06:45
*** shaohe_feng has quit IRC06:45
*** shaohe_feng has joined #openstack-nova06:46
*** zenoway has joined #openstack-nova06:47
*** irenab has joined #openstack-nova06:47
*** melwitt has joined #openstack-nova06:50
*** esracelik has joined #openstack-nova06:50
*** melwitt is now known as Guest9853406:50
*** mjura has joined #openstack-nova06:51
*** gongysh has joined #openstack-nova06:51
*** nkrinner has joined #openstack-nova06:53
*** dave-mccowan has quit IRC06:55
*** shaohe_feng has quit IRC06:55
*** Guest98534 is now known as melwitt06:55
*** shaohe_feng has joined #openstack-nova06:56
*** melwitt has quit IRC06:58
*** mingyu has quit IRC06:59
*** melwitt has joined #openstack-nova06:59
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-nova07:01
*** melwitt has quit IRC07:03
*** haomaiwang has quit IRC07:04
*** melwitt has joined #openstack-nova07:04
*** haomaiwang has joined #openstack-nova07:04
*** melwitt is now known as Guest3702007:05
*** obondarev has quit IRC07:05
*** shaohe_feng has quit IRC07:06
*** shaohe_feng has joined #openstack-nova07:07
*** obondarev has joined #openstack-nova07:07
*** yamahata has joined #openstack-nova07:07
*** shaohe_feng has quit IRC07:09
*** paul-carlton2 has joined #openstack-nova07:10
*** mingyu has joined #openstack-nova07:12
*** moshele has joined #openstack-nova07:13
openstackgerritAbhishek Talwar proposed openstack/python-novaclient: flavor-id should be only integer or UUID  https://review.openstack.org/28770207:14
*** vishwanathj is now known as vishwanathj_zzz07:15
*** Kennan has quit IRC07:18
*** sc68cal has quit IRC07:22
*** sridharg has joined #openstack-nova07:22
*** ildikov has quit IRC07:22
*** Kennan has joined #openstack-nova07:23
*** hoangcx has quit IRC07:23
*** liverpooler has quit IRC07:23
*** lpetrut has joined #openstack-nova07:25
*** Jeffrey4l has joined #openstack-nova07:26
*** andreykurilin__ has joined #openstack-nova07:29
*** melwitt_ has joined #openstack-nova07:30
*** melwitt_ has quit IRC07:32
*** hoangcx has joined #openstack-nova07:33
*** scheuran has joined #openstack-nova07:33
*** rcernin has joined #openstack-nova07:33
*** ildikov has joined #openstack-nova07:34
*** thorst_ has joined #openstack-nova07:36
*** browne has quit IRC07:37
*** melwitt_ has joined #openstack-nova07:39
*** melwitt_ is now known as melwitt07:40
*** melwitt has quit IRC07:40
*** belmoreira has joined #openstack-nova07:41
*** melwitt has joined #openstack-nova07:41
*** rk4n has quit IRC07:41
*** pece has joined #openstack-nova07:42
*** pbrooko has joined #openstack-nova07:42
*** thorst_ has quit IRC07:43
*** nikhil has quit IRC07:45
*** nikhil has joined #openstack-nova07:47
*** rdopiera has joined #openstack-nova07:48
*** baoli has joined #openstack-nova07:48
*** palexster has joined #openstack-nova07:49
*** mpavone has joined #openstack-nova07:52
*** baoli has quit IRC07:53
*** rk4n has joined #openstack-nova07:53
*** e0ne has joined #openstack-nova07:53
*** allen_gao has quit IRC07:54
*** jlanoux has joined #openstack-nova07:55
*** atuvenie has joined #openstack-nova07:57
openstackgerritJiajun Liu proposed openstack/nova: Replace deprecated LOG.warn with LOG.warning  https://review.openstack.org/29095807:58
*** erno has joined #openstack-nova07:59
*** FL1SK has joined #openstack-nova08:00
openstackgerritMoshe Levi proposed openstack/nova-specs: Add spec to enhance PCI passthrough whitelist to support regex  https://review.openstack.org/29104608:00
*** mpavone has quit IRC08:00
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-nova08:01
*** mpavone has joined #openstack-nova08:01
*** tbachman has quit IRC08:03
*** allen_gao has joined #openstack-nova08:05
*** tbachman has joined #openstack-nova08:07
*** ihrachys has joined #openstack-nova08:07
*** ihrachys has quit IRC08:08
*** gjayave__ has quit IRC08:10
*** rk4n has quit IRC08:11
*** daemontool_ has joined #openstack-nova08:11
*** hoangcx has quit IRC08:12
*** ihrachys has joined #openstack-nova08:13
*** paul-carlton2 has quit IRC08:14
*** daemontool has quit IRC08:14
*** armax has quit IRC08:17
*** hoangcx has joined #openstack-nova08:17
*** dedery has joined #openstack-nova08:20
*** e0ne has quit IRC08:21
*** e0ne has joined #openstack-nova08:25
*** matrohon has joined #openstack-nova08:25
*** jichen has joined #openstack-nova08:26
*** lpetrut has quit IRC08:27
*** gcb has joined #openstack-nova08:28
*** pbrooko has quit IRC08:30
*** rk4n has joined #openstack-nova08:31
*** rk4n has quit IRC08:32
*** rk4n has joined #openstack-nova08:32
*** dguitarbite has quit IRC08:32
*** paul-carlton2 has joined #openstack-nova08:32
*** pbrooko has joined #openstack-nova08:32
*** davideagnello has quit IRC08:32
*** atuvenie has quit IRC08:33
*** y_sawai has quit IRC08:33
*** pcaruana has joined #openstack-nova08:34
*** liverpooler has joined #openstack-nova08:34
*** vishwanathj_zzz has quit IRC08:34
*** vishwanathj_zzz has joined #openstack-nova08:35
*** davideagnello has joined #openstack-nova08:35
*** liverpooler has quit IRC08:35
*** permalac has joined #openstack-nova08:35
*** permalac has quit IRC08:35
*** liverpooler has joined #openstack-nova08:35
*** andreykurilin__ has quit IRC08:36
*** betherly has quit IRC08:36
*** angular_mike___ has quit IRC08:36
*** zhiyan has quit IRC08:36
*** zhenguo_ has quit IRC08:36
*** fmccrthy has quit IRC08:36
*** kgalanov has quit IRC08:36
*** zzzeek has quit IRC08:36
*** andrewbogott has quit IRC08:36
*** ameade has quit IRC08:36
*** scottda has quit IRC08:36
*** junjie has quit IRC08:36
*** zzzeek has joined #openstack-nova08:38
*** zhiyan has joined #openstack-nova08:39
*** scottda has joined #openstack-nova08:39
*** ameade has joined #openstack-nova08:39
*** betherly has joined #openstack-nova08:40
*** permalac has joined #openstack-nova08:41
*** permalac has quit IRC08:41
*** thorst_ has joined #openstack-nova08:41
openstackgerritPaul Carlton proposed openstack/nova: Add include_disabled parameter to service_get_all_by_binary  https://review.openstack.org/28837308:41
*** kgalanov has joined #openstack-nova08:42
*** fmccrthy has joined #openstack-nova08:42
*** e0ne has quit IRC08:42
*** permalac has joined #openstack-nova08:43
permalachello guys.08:43
bauzasgood morning Nova08:43
*** josecastroleon has quit IRC08:44
*** zhenguo_ has joined #openstack-nova08:44
permalacI have an issue with qemu-virsh and I can not migrate machines from one node to another. I looking at shared storage to do this for the future situations.  Can I use swift as /var/lib/instances mountpoint ? Or how Should I do it?08:44
*** andrewbogott has joined #openstack-nova08:44
*** andreykurilin__ has joined #openstack-nova08:44
*** josecastroleon has joined #openstack-nova08:45
*** gszasz has joined #openstack-nova08:46
*** lpetrut has joined #openstack-nova08:46
paul-carlton2bauzas, good morning, see my update of the release note on https://review.openstack.org/#/c/288373, hope this meets with everyones approval08:46
*** thorst_ has quit IRC08:48
*** gjayavel_ has joined #openstack-nova08:48
*** wuhao has quit IRC08:49
*** wuhao has joined #openstack-nova08:49
*** zhangjn has quit IRC08:49
openstackgerritPaul Carlton proposed openstack/nova: Add include_disabled parameter to service_get_all_by_binary  https://review.openstack.org/28837308:49
*** mingyu has quit IRC08:50
*** junjie has joined #openstack-nova08:50
*** angular_mike___ has joined #openstack-nova08:50
*** mingyu has joined #openstack-nova08:50
*** Jeffrey4l has quit IRC08:50
*** pbrooko has quit IRC08:51
*** Jeffrey4l has joined #openstack-nova08:51
bauzaspaul-carlton2: sure, will do08:51
*** jaosorior has joined #openstack-nova08:52
*** zhangjn has joined #openstack-nova08:55
*** salv-orlando has joined #openstack-nova08:55
*** pbrooko has joined #openstack-nova08:56
*** paul-carlton2 has quit IRC08:57
*** ishant has joined #openstack-nova08:58
*** permalac_ has joined #openstack-nova09:00
*** permalac_ has quit IRC09:00
*** haomaiwang has quit IRC09:01
*** permalac has quit IRC09:01
*** haomaiwa_ has joined #openstack-nova09:01
*** ndipanov has joined #openstack-nova09:03
*** lpetrut has quit IRC09:03
*** zz_dimtruck is now known as dimtruck09:03
*** Jeffrey4l has quit IRC09:05
*** mgoddard has joined #openstack-nova09:06
*** pbrooko has quit IRC09:09
openstackgerritPavel Kholkin proposed openstack/nova: removed db_exc.DBDuplicateEntry in bw_usage_update  https://review.openstack.org/25080709:09
*** jaosorior has quit IRC09:09
*** jaosorior has joined #openstack-nova09:09
*** jistr has joined #openstack-nova09:09
*** mvk has joined #openstack-nova09:10
*** gcb has quit IRC09:11
*** dimtruck is now known as zz_dimtruck09:12
*** lucas-dinner is now known as lucasagomes09:14
*** amotoki has quit IRC09:15
*** danpb has joined #openstack-nova09:16
*** salv-orlando has quit IRC09:20
*** pbrooko has joined #openstack-nova09:21
*** paul-carlton2 has joined #openstack-nova09:23
*** gcb_ has joined #openstack-nova09:24
*** pbrooko has quit IRC09:25
*** efoley has joined #openstack-nova09:26
openstackgerritPavel Kholkin proposed openstack/nova: fixed log warning in sqlalchemy/api.py  https://review.openstack.org/24690109:29
*** openstackgerrit has quit IRC09:30
*** openstackgerrit_ has joined #openstack-nova09:30
*** openstackgerrit_ is now known as openstackgerrit09:31
*** openstackgerrit has quit IRC09:31
*** openstackgerrit_ has joined #openstack-nova09:31
*** openstackgerrit_ is now known as openstackgerrit09:32
*** openstackgerrit has quit IRC09:32
*** openstackgerrit_ has joined #openstack-nova09:33
*** openstackgerrit_ is now known as openstackgerrit09:33
*** openstackgerrit has quit IRC09:33
*** e0ne has joined #openstack-nova09:33
*** openstackgerrit_ has joined #openstack-nova09:34
*** yassine__ has joined #openstack-nova09:34
*** sfinucan has joined #openstack-nova09:34
openstackgerrit_Sergey Nikitin proposed openstack/nova: Added db API layer to add instance tag-list filtering support  https://review.openstack.org/24880509:34
openstackgerrit_Sergey Nikitin proposed openstack/nova: Added server tags controller  https://review.openstack.org/12894009:34
openstackgerrit_Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/26893209:34
*** openstackgerrit_ is now known as openstackgerrit09:34
openstackgerritSergey Nikitin proposed openstack/nova: Moved tags filtering tests to TestInstanceTagsFiltering test case  https://review.openstack.org/25851509:39
*** gjayavel_ has quit IRC09:40
*** zenoway has quit IRC09:40
*** zenoway has joined #openstack-nova09:40
*** ekuris_ has joined #openstack-nova09:41
*** dguitarbite has joined #openstack-nova09:44
*** dguitarbite has quit IRC09:44
*** pkarikh has left #openstack-nova09:44
*** zenoway has quit IRC09:45
*** roeyc has joined #openstack-nova09:45
*** daemontool__ has joined #openstack-nova09:45
*** thorst_ has joined #openstack-nova09:45
*** daemontool_ has quit IRC09:49
*** amotoki has joined #openstack-nova09:50
*** acabot has joined #openstack-nova09:51
*** PaulMurray has joined #openstack-nova09:51
*** thorst_ has quit IRC09:53
PaulMurraygood morning09:53
*** rk4n has quit IRC09:53
* johnthetubaguy waves good morning09:53
gmanngood morning09:55
gmannPaulMurray: johnthetubaguy need your feedback on https://review.openstack.org/29019609:55
ndipanovPaulMurray, hello09:55
gmannadded patch as discussed in API meeting- for GET volume attachment APIs09:55
ndipanovPaulMurray, if you get a chance go back to the live mig claims patch - I've updated it\09:56
ndipanovjohnthetubaguy, would be interested in your thoughts on this https://review.openstack.org/#/c/290705/09:56
PaulMurrayndipanov, yep, had a brief look yesterday but doing it this morning09:56
ndipanovin light of RC1 even if you don't think it's too risky09:57
PaulMurraygmann, I'll look at that to09:57
gmannPaulMurray: Thanks09:57
johnthetubaguyndipanov: it has a +2, I should hit that stuff on my morning rounds09:57
*** mingyu has quit IRC09:57
johnthetubaguyndipanov: that device reservation, I think that stopped being useful when we stopped trying to honor the device people specify, I guess?09:59
johnthetubaguyanyways, will try get back to that in a bit09:59
*** markusz is now known as markus_z10:00
*** haomaiwa_ has quit IRC10:01
ndipanovjohnthetubaguy, weeeel10:01
*** haomaiwang has joined #openstack-nova10:01
ndipanovthis thing just moves stuff around a bit10:01
openstackgerritStephen Finucane proposed openstack/nova: libvirt: Don't misuse InstanceNotFound exception  https://review.openstack.org/29070610:01
ndipanovI would not remove it for RC 110:01
ndipanov(the reservation I mean)10:01
ndipanovbut do a volume reservation before it would prevent a class of bugs we keep hitting for years10:02
ndipanovand that's what the patch does10:02
*** efoley_ has joined #openstack-nova10:02
*** Marga_ has quit IRC10:03
*** rk4n has joined #openstack-nova10:03
*** efoley has quit IRC10:04
johnthetubaguyndipanov: OK, like I say, will try read through that more later10:04
ndipanovjohnthetubaguy, awesome - I also left comments on the bug to clarify10:04
*** Guest3102 is now known as BobBall10:04
johnthetubaguycools10:04
*** liverpooler has quit IRC10:05
*** sayalilunkad_lap has joined #openstack-nova10:06
*** atuvenie has joined #openstack-nova10:07
openstackgerritAbhishek Talwar proposed openstack/python-novaclient: flavor-id should be only integer or UUID  https://review.openstack.org/28770210:08
*** eglynn has joined #openstack-nova10:09
*** zenoway has joined #openstack-nova10:09
*** liverpooler has joined #openstack-nova10:10
*** derekh has joined #openstack-nova10:13
*** vilobhmm11 has quit IRC10:13
*** paul-carlton2 has quit IRC10:14
*** liverpooler has quit IRC10:17
*** paul-carlton2 has joined #openstack-nova10:17
*** atuvenie has quit IRC10:19
openstackgerritCao ShuFeng proposed openstack/nova: Update the doc of notification  https://review.openstack.org/29108410:20
*** lpetrut has joined #openstack-nova10:28
*** liverpooler has joined #openstack-nova10:29
*** hoangcx has quit IRC10:32
*** paul-carlton2 has quit IRC10:33
johnthetubaguyandreykurilin__ did you see any python-novaclient issues pop up that mean we should wait on creating our stable branch for mitaka? (thats a general question, too!)10:37
*** josecastroleon has quit IRC10:38
openstackgerritGábor Antal proposed openstack/nova: Extract duplicated code from compute manager  https://review.openstack.org/28132610:38
*** josecastroleon has joined #openstack-nova10:39
*** atuvenie has joined #openstack-nova10:41
andreykurilin__johnthetubaguy: hi! I know only one issue, but it is not critical(maybe low priority) and we have it for a long time:) I suppose it is not blocking and relates not only to novaclient, but to api-side too. So we are ready to create new stable branch10:41
johnthetubaguycool, thats what I am seeing from where I am sat, so thats good to know, thanks andreykurilin__10:41
*** mvk has quit IRC10:42
andreykurilin__johnthetubaguy: btw, do you have a bit of time to talk about one issue? or lets talk a bit later(after release)?10:43
johnthetubaguyandreykurilin__: depends if its quick ;) whats up?10:44
*** rk4n has quit IRC10:44
*** dims has joined #openstack-nova10:47
*** Marga_ has joined #openstack-nova10:47
andreykurilin__johnthetubaguy: it is about `nova quota-show` command. Nova stores only "no-default" quotas in database. quota-show command accepts tenant-id to show quotas for. In this case nova does not validate tenant and just do the query. The results are empty and Nova assumes that there are no-default quotas and returns dict with default quotas.10:47
andreykurilin__johnthetubaguy: users try to put tenant_name instead of tenant_id to `quota show` and nova displays wrong quotas10:50
johnthetubaguyandreykurilin__: oh, thats an old tricky issue. We don't validate the tenant when we set quotas either, because its really an arbitrary external id to Nova. The problem is having a new plugin point of "authenticators" to validate tenant ids. There might be some old history on this conversation somewhere10:50
*** thorst_ has joined #openstack-nova10:50
andreykurilin__johnthetubaguy: got it, i'll llok at mailing list history10:51
*** bapalm has quit IRC10:51
*** dims has quit IRC10:52
*** yassine__ has quit IRC10:54
andreykurilin__johnthetubaguy: I had a talk with keystone folks and they said that there is no way to validate tenant via keystoneauth(it used on both novaclient and api sides) :(10:54
*** cdent has joined #openstack-nova10:55
*** dims has joined #openstack-nova10:56
*** omnipresent has joined #openstack-nova10:56
*** ociuhandu has quit IRC10:56
*** salv-orlando has joined #openstack-nova10:57
*** zenoway has quit IRC10:57
*** thorst_ has quit IRC10:58
johnthetubaguyandreykurilin__: oh, thats a good point, we could look at finding a way to validate that inside python-novaclient10:58
johnthetubaguyandreykurilin__: its a shame its not possible, but thats an angle I don't remember us fully thinking through yet10:59
*** sdague has joined #openstack-nova11:00
*** haomaiwang has quit IRC11:01
*** haomaiwang has joined #openstack-nova11:01
*** roeyc has quit IRC11:03
*** atuvenie has quit IRC11:06
*** cristicalin has joined #openstack-nova11:06
*** bapalm has joined #openstack-nova11:08
*** gongysh has quit IRC11:09
*** josecastroleon has quit IRC11:09
*** efoley_ has quit IRC11:09
*** mvk has joined #openstack-nova11:10
*** josecastroleon has joined #openstack-nova11:10
*** efoley_ has joined #openstack-nova11:11
*** rk4n has joined #openstack-nova11:12
*** dguitarbite has joined #openstack-nova11:14
*** rk4n has quit IRC11:16
*** dims has quit IRC11:19
openstackgerritAlexis Lee proposed openstack/nova: Wait for device to be mapped  https://review.openstack.org/21561311:19
openstackgerritAlexis Lee proposed openstack/nova: Wait for device to be mapped  https://review.openstack.org/21561311:21
*** dims has joined #openstack-nova11:21
*** bapalm has quit IRC11:24
*** ishant has quit IRC11:25
*** kmartin has quit IRC11:29
*** salv-orlando has quit IRC11:30
*** cristicalin has quit IRC11:35
*** rk4n has joined #openstack-nova11:39
*** josecastroleon has quit IRC11:40
*** EinstCrazy has joined #openstack-nova11:40
*** josecastroleon has joined #openstack-nova11:41
*** smatzek has joined #openstack-nova11:42
*** ivase_ has quit IRC11:42
*** bapalm has joined #openstack-nova11:44
*** sayalilunkad_lap has quit IRC11:45
*** roeyc has joined #openstack-nova11:46
*** atuvenie has joined #openstack-nova11:48
*** tbachman_ has joined #openstack-nova11:50
*** rfolco has joined #openstack-nova11:50
*** rk4n has quit IRC11:50
*** gjayavel_ has joined #openstack-nova11:52
*** rk4n has joined #openstack-nova11:52
*** tbachman has quit IRC11:53
*** tbachman_ is now known as tbachman11:53
johnthetubaguylyarwood: any chance you could review this bug for me, it feels related: https://review.openstack.org/#/c/288334/411:54
paul-carlton1johnthetubaguy, two questions on https://review.openstack.org/#/c/288373/11:55
*** thorst_ has joined #openstack-nova11:56
*** dims has quit IRC11:56
*** gjayavel_ has quit IRC11:56
paul-carlton1firstly HyperV CI has failed do I need to recheck that or is that irrelevant now you've +1 workflow'ed it?11:56
*** sfinucan has quit IRC11:56
*** efoley__ has joined #openstack-nova11:57
paul-carlton1Secondly, just to clarify you are thinking that we should change CellsStateManager usage of objects.ServiceList.get_by_binary to set the include_disabled parameter to True?11:58
*** moshele has quit IRC11:58
*** aysyd has joined #openstack-nova11:59
cdentp!spy11:59
paul-carlton1I think I've answer the first question, since it is now in gate in zuul HyperV CI check is irrelevant12:00
*** efoley_ has quit IRC12:00
openstackgerritsahid proposed openstack/nova: network: make nova to handle port_security_enabled=False  https://review.openstack.org/28409512:00
*** haomaiwang has quit IRC12:01
*** haomaiwang has joined #openstack-nova12:01
*** rk4n has quit IRC12:01
*** jaosorior has quit IRC12:02
*** thorst_ has quit IRC12:03
*** jaosorior has joined #openstack-nova12:03
johnthetubaguypaul-carlton1: yeah, only jenkins will block it in the gate, didn't seem specific to hyper-v, so I ignored that12:03
*** rk4n has joined #openstack-nova12:04
paul-carlton1johnthetubaguy, discussed with PaulMurray and now understand the Cells change is to optimize12:04
*** dims has joined #openstack-nova12:04
johnthetubaguypaul-carlton1: hmm, I suppose it is an optimisation there, slightly12:05
*** vladikr has joined #openstack-nova12:05
*** erlon has joined #openstack-nova12:05
*** rk4n has quit IRC12:05
johnthetubaguypaul-carlton1: although if we are keeping that in cells, maybe best to explicitly pass False, and drop the extra if, and your comment about removing that arg in the next version12:05
paul-carlton1very slightly, select will not filter on disabled but the CellsStateManager strips out disabled hosts after doing select anyway12:06
*** rk4n has joined #openstack-nova12:07
sdaguehmmm... I guess we have no unit tests for this - https://github.com/openstack/nova/blob/3d7e403cc7a5d9ebcd9a011d6c2055bfbf56cb05/nova/compute/resource_tracker.py#L97712:08
sdaguebecause that's not going to do what anyone thinks12:08
paul-carlton1don't think we can drop the ...12:08
paul-carlton1if not include_disabled:12:08
paul-carlton1        query = query.filter_by(disabled=False)12:08
paul-carlton1until v2.0 of object, per dansmith wanting the existing object behaviour to remain unchanged12:09
*** josecastroleon has quit IRC12:11
*** rk4n has quit IRC12:12
*** josecastroleon has joined #openstack-nova12:13
bauzasjohnthetubaguy: correct me if I'm wrong, but configdrives are r/o, right?12:16
*** moshele has joined #openstack-nova12:18
openstackgerritThomas Bachman proposed openstack/nova: VMware: Use Port Group and Key in binding details  https://review.openstack.org/28807612:18
*** eglynn has quit IRC12:20
sdaguebauzas: yes12:21
openstackgerritSean Dague proposed openstack/nova: Make tuple actually a tuple  https://review.openstack.org/29113812:21
bauzassdague: okay, thanks12:22
sdaguebauzas / ndipanov - https://review.openstack.org/291138 saw that when I was looking at an unrelated review. There are definitely no tests in that area, but I don't know the resource tracker enough to know what those should be.12:23
bauzassdague: so, we had a legacy test_resource_tracker12:24
bauzassdague: which was doing kind of DB tests12:24
sdagueoh, the initial review even has comments about that, it works by accident12:24
bauzassdague: and each time we were modifying some RT stuff, there were like 20 impacted tests litterally needing to change a mocked signature12:24
bauzassdague: so jaypipes worked on moving those tests to another testcase in test_tracker12:25
bauzaswell, another testcases even12:25
openstackgerritjaveme proposed openstack/nova: Don't-merge: test for can_send_version  https://review.openstack.org/29114112:25
sdaguebauzas: yeh, no, this is one of those things where python bits are weird12:25
bauzas(with an 's')12:25
*** eglynn has joined #openstack-nova12:25
sdague'resize' in 'resize' == True12:25
bauzassdague: so, if you want to add a test for RT, then test_tracker is the right thing12:25
sdague'resize' in ('resize',) == True12:25
sdague'resiz' in 'resize' == True12:26
openstackgerritsean mooney proposed openstack/nova: stop setting mtu when pluging vhostuser port  https://review.openstack.org/27144412:26
sdague'resiz' in ('resize',) == False12:26
bauzasyeah :)12:26
bauzasbecause strings are containers :)12:26
sdagueso it's accidentally working, for the wrong reasons12:26
bauzasI'm all good with your change, just thinking of whether we need a test12:27
*** tbachman_ has joined #openstack-nova12:27
bauzasmigration.migration_type is an object field AFAIR12:27
bauzaswhich is Enum12:27
bauzassdague: so there would litterally be no way to provide like a substring12:28
*** Daisy has joined #openstack-nova12:28
bauzas(and the field isn't nullable)12:28
*** tbachman has quit IRC12:29
*** tbachman_ is now known as tbachman12:29
*** rodrigods has quit IRC12:33
*** rodrigods has joined #openstack-nova12:33
sdaguebauzas: sure, it does make things confusing though. Because it's not the intent.12:33
*** Daisy has quit IRC12:33
bauzassdague: I agree, so I +2d the patch but downgrade the bug to Low12:34
bauzasdowngraded12:34
*** jwcroppe has joined #openstack-nova12:34
bauzasthat's not impactful12:34
*** Daisy has joined #openstack-nova12:34
*** lucasagomes is now known as lucas-hungry12:34
*** hamzy has quit IRC12:35
*** thorst_ has joined #openstack-nova12:35
bauzassdague: non-related, http://lists.openstack.org/pipermail/openstack-dev/2016-March/088931.html12:35
bauzassdague: I don't know where we could discuss that, in -dev chan ?12:36
*** rtheis has joined #openstack-nova12:37
openstackgerritBin Zhou proposed openstack/nova: upgrade connection_info when Ceph mon IP changed  https://review.openstack.org/28981312:38
openstackgerritBin Zhou proposed openstack/nova: upgrade connection_info when Ceph mon IP changed  https://review.openstack.org/28981312:38
*** Daisy has quit IRC12:38
*** doug-fish has quit IRC12:38
jaypipesOMG I want to slap the face of these PCI unit tests. how many levels of hell must I go through in order to unwind these :(12:39
johnthetubaguybauzas: they should be, although I think the ext ones are not quite read only12:39
* johnthetubaguy gets some lunch12:39
sdaguebauzas: that we should keep on the mailing list probably12:39
*** doug-fish has joined #openstack-nova12:39
bauzasjohnthetubaguy: ack12:40
*** rk4n has joined #openstack-nova12:40
bauzassdague: np, just wanted to get your opinion on that12:40
*** rk4n has quit IRC12:40
* bauzas goes eating as well12:41
*** fesp has joined #openstack-nova12:43
sdaguebauzas: sure12:43
*** doug-fish has quit IRC12:44
*** rk4n has joined #openstack-nova12:44
*** moshele has quit IRC12:45
*** pcaruana has quit IRC12:45
*** gongysh has joined #openstack-nova12:46
*** ildikov has quit IRC12:48
*** purplerbot has quit IRC12:49
*** purplerbot has joined #openstack-nova12:49
*** tdurakov_ has joined #openstack-nova12:49
*** tbachman_ has joined #openstack-nova12:51
sdaguecdent: you have a patch on top of -  https://review.openstack.org/#/c/192878/ which is merge conflicted12:51
*** fesp has quit IRC12:51
*** sayalilunkad has quit IRC12:51
*** sayalilunkad has joined #openstack-nova12:52
* cdent looks12:52
*** efoley__ has quit IRC12:53
cdentthanks sdague, I meant to make that independent, will fix12:53
*** tbachman has quit IRC12:53
*** tbachman_ is now known as tbachman12:53
*** e0ne_ has joined #openstack-nova12:53
*** e0ne has quit IRC12:54
cdentsdague: oh, I did12:54
cdentmy change should be independent, it's the same topic though12:54
*** baoli has joined #openstack-nova12:55
tdurakov_pkoniszewski, johnthetubaguy, hey folks,do you have time to chat about this bug: https://bugs.launchpad.net/nova/+bug/155230312:56
openstackLaunchpad bug 1552303 in OpenStack Compute (nova) "Block live migrations are broken when nova calculates live migration type by itself" [Critical,In progress] - Assigned to Pawel Koniszewski (pawel-koniszewski)12:56
pkoniszewskitdurakov: hi, sure12:57
*** cdelatte has quit IRC12:57
sdaguecdent: ah, my bad12:57
cdentsdague: I rewrote it indepdendently because the other version is doing too much to be just a bug fix. so I borrowed the tests and did my own implementation without the refactor12:58
sdaguegotcha, cool12:58
tdurakov_pkoniszewski, will your patch be merged prior to release?12:59
pkoniszewskilive migrations in 2.25 API are broken, so yes, i think it will be12:59
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Comment non-obvious security implications of migrate code  https://review.openstack.org/29115512:59
*** yamahata has quit IRC13:00
pkoniszewskiit affects both, live migrations and block migrations13:00
*** pcaruana has joined #openstack-nova13:00
pkoniszewskionly force block live migrations still work13:00
pkoniszewskiforced *13:01
*** haomaiwang has quit IRC13:01
mdbooth^^^ The above patch simply adds a comment, btw. It describes why we shouldn't touch 2 lines of code which look wrong, because doing that would introduce (another) severe security bug in the libvirt driver.13:01
*** haomaiwang has joined #openstack-nova13:01
tdurakov_pkoniszewski, ok, while i like your approach to get rid of block_migration param, we could go with alternative and much easier solution by setting block_migration in conductor, smth like: http://xsnippet.org/361517/ but i'd rather go with existing approach if we have enough time13:03
*** efried has joined #openstack-nova13:04
pkoniszewskitdurakov: this was my first proposition, but it is ugly and bad13:04
tdurakov_pkoniszewski, agree13:04
*** moshele has joined #openstack-nova13:04
openstackgerritMaciej Szankin proposed openstack/nova: Live migration failure in API leaves VM in MIGRATING state  https://review.openstack.org/16891613:04
openstackgerritPawel Koniszewski proposed openstack/nova: Remove block_migration from virt drivers  https://review.openstack.org/29051913:05
openstackgerritPawel Koniszewski proposed openstack/nova: Use migrate_data.block_migration instead of block_migration  https://review.openstack.org/28736313:05
tdurakov_pkoniszewski, imo all live migration methods should operate with instance and migrate_data params only13:05
pkoniszewskicorrect13:05
*** thumpba has joined #openstack-nova13:06
mdboothtdurakov_: Much the same as your cleanup patch, block migration should be firmly in the domain of the driver, imho13:07
openstackgerritTimofey Durakov proposed openstack/nova: Cleanup for post/rollback of live-migrations  https://review.openstack.org/28065313:08
openstackgerritTimofey Durakov proposed openstack/nova: New field for LibvirtLiveMigrateData  https://review.openstack.org/28674513:08
openstackgerritTimofey Durakov proposed openstack/nova: move disk_info calc to check methods  https://review.openstack.org/29116013:08
openstackgerritTimofey Durakov proposed openstack/nova: implement async rpc for live-migration  https://review.openstack.org/29116113:08
mdboothExposing it in the api is obviously really bad, but conductor isn't great either13:08
tdurakov_mdbooth, pkoniszewski check this^13:08
tdurakov_it's part of PoC will submit last one in an hour13:09
*** Daisy has joined #openstack-nova13:09
openstackgerritMaciej Szankin proposed openstack/nova: Live migration failure in API leaves VM in MIGRATING state  https://review.openstack.org/16891613:10
*** atuvenie_ has joined #openstack-nova13:10
*** Daisy has quit IRC13:11
*** haomaiwang has quit IRC13:11
pkoniszewskitdurakov: i like one that moves cleanup flags to drivers13:11
*** erno has quit IRC13:11
*** Daisy has joined #openstack-nova13:11
tdurakov_pkoniszewski, which one?13:11
pkoniszewskitdurakov: this one https://review.openstack.org/28065313:12
mdboothpkoniszewski: Me too, btw.13:12
*** ducttape_ has joined #openstack-nova13:12
mdboothHowever, I was -1 the details, and I still thought the patch could be broken up a bit.13:12
*** moshele has quit IRC13:12
tdurakov_yes, got feedback, need to split more:-p13:12
pkoniszewskitdurakov: i wonder why didn't u continue it here https://review.openstack.org/#/c/216329/13:12
mdbooth:)13:12
pkoniszewskitdurakov: it's simple enough and just moves this one method13:13
*** atuvenie__ has joined #openstack-nova13:13
*** pradk has joined #openstack-nova13:13
*** atuvenie has quit IRC13:13
*** dave-mccowan has joined #openstack-nova13:13
tdurakov_pkoniszewski, we discussed it with johnthetubaguy, so this 2 are kind of alternatives, and we should pick one13:13
*** erno has joined #openstack-nova13:14
*** roeyc has quit IRC13:14
tdurakov_pkoniszewski, do you think it's worth to implement both?13:14
*** moshele has joined #openstack-nova13:14
pkoniszewskitdurakov: let me compare both, i will let you know what i think13:15
*** ekuris_ has quit IRC13:15
*** Daisy has quit IRC13:15
tdurakov_i mean, it seems like first one(that moves method to driver side) looks bit useless as we aiming to get rid of it at all13:15
*** ekuris_ has joined #openstack-nova13:16
tdurakov_pkoniszewski, yes, please, left comment than13:16
*** atuvenie_ has quit IRC13:16
pkoniszewskitdurakov: well, this can be true13:16
*** atuvenie has joined #openstack-nova13:16
*** lucas-hungry is now known as lucasagomes13:17
*** jmhale has joined #openstack-nova13:17
*** atuvenie__ has quit IRC13:18
pkoniszewskitdurakov: you are right, moving this method to driver side is a bit useless, drivers already have all the data so improving cleanup logic sounds better13:20
*** cdelatte has joined #openstack-nova13:20
*** edmondsw has joined #openstack-nova13:21
tdurakov_pkoniszewski, mdbooth, could you folks review this patch also: https://review.openstack.org/#/c/291161/13:21
*** hamzy has joined #openstack-nova13:21
tdurakov_interested in feedback, as now I'm describing this in spec13:21
tdurakov_mostly about new methods and rpc api13:21
tdurakov_p.s. last one adds state to compute to add safety to this13:22
mdboothtdurakov_: Yup, will do.13:24
tdurakov_mdbooth, thanks in advance13:24
pkoniszewskitdurakov: cool, will do too13:26
*** doug-fish has joined #openstack-nova13:26
tdurakov_pkoniszewski, thx:)13:26
johnthetubaguytdurakov_: by add state, do you mean put more data in the migrate_data blob that gets passed around?13:26
openstackgerritGleb Stepanov proposed openstack/nova: Redundant parentheses removed  https://review.openstack.org/29041013:26
pkoniszewskitdurakov: i think i should reabse https://review.openstack.org/#/c/290519 somewhere in the middle of your patch series, so don't conflict much13:26
*** mtanino has joined #openstack-nova13:27
*** doug-fish has quit IRC13:27
tdurakov_johnthetubaguy, no, as we discussed it during mid-cycle, storing state of migration so l-m will survive after crash and continue async process13:27
tdurakov_on compute side i mean13:27
*** doug-fish has joined #openstack-nova13:27
johnthetubaguytdurakov_: oh, that bit, yeah, gotcha13:28
*** ivase has joined #openstack-nova13:28
*** lixiaoy1 has joined #openstack-nova13:28
*** lefaiseur has joined #openstack-nova13:28
*** ducttape_ has quit IRC13:29
johnthetubaguytdurakov_: just error out and clean up would be good enough, but it might be just as easy to pick up where the sync monitoring process left off, which is cool13:29
tdurakov_pkoniszewski, in a middle I think will be good, as last 2 are mostly about bp implementation13:29
*** prateek has joined #openstack-nova13:29
*** prateek is now known as Guest2246413:29
*** Guest22464 is now known as parora13:29
tdurakov_johnthetubaguy, let me upload the last part, and then we could pick best solution13:30
*** roeyc has joined #openstack-nova13:30
parorado we have any locking mechanism for instances in api.py file in nova ?13:30
johnthetubaguyparora: depends what you want to achieve, we do a best effort thing to stop conflicting API operations.13:31
*** ivase has quit IRC13:32
parorajohnthetubaguy, i am trying to make sure that while i am taking snapshot, the instances are not deleted13:32
johnthetubaguyparora: that breaks the API though, its intentional that we allow that13:32
*** sayalilunkad has quit IRC13:32
*** tdurakov_ has quit IRC13:33
johnthetubaguyparora: basically delete is the one operation that is allowed at any time13:33
mdboothtdurakov_: Re the cleanup patch, did you see my comment about removing the last use of cleanup?13:33
*** tdurakov_ has joined #openstack-nova13:33
parorajohnthetubaguy, intentional, why ?13:33
*** ivase has joined #openstack-nova13:33
mdboothtdurakov_: Re the cleanup patch, did you see my comment about removing the last use of cleanup?13:33
johnthetubaguyparora: it comes from thinking about when users get billed for instances, you want to always allow delete, if thats what they want to do.13:33
mdboothtdurakov_: It's weird, because you're making a bunch of changes in there, but I don't think anything's calling it any more13:33
sdagueparora: it's the universal cancel operation. It would be like blocking Ctrl-C to not allow it.13:34
johnthetubaguyparora: now it does have a habit of triggering obscure bugs in the system, but they should be fixed to allow delete to happen at any time.13:34
johnthetubaguysdague: I like that comparison, +113:34
parorasdague, well said13:34
parorajohnthetubaguy, sdague i agree but that sends the image into saving state for an eternity13:35
tdurakov_mdbooth, 5 mins13:35
johnthetubaguyparora: so thats totally a bug that needs fixing, it should gracefully fail the snapshot, and clean up the half created image, like we do for other snapshot failure modes13:35
*** sayalilunkad has joined #openstack-nova13:35
parorajohnthetubaguy, sounds good13:36
mdboothjohnthetubaguy: Remind me, do we have Tasks (with a T, not a t) yet?13:36
johnthetubaguymdbooth: neither really, but we do have elements of what you might need, like instance actions, instance faults, task state detection to spot partial operations, etc13:37
mdboothjohnthetubaguy: Thanks.13:38
*** acabot has quit IRC13:38
*** dims has quit IRC13:38
johnthetubaguymdbooth: totally want to see more structure there, and I know alaski is super interested in all that (expect for that cells v2 thing he has right now)13:39
*** dims has joined #openstack-nova13:39
mdboothjohnthetubaguy: Was just thinking in the context of live migration13:39
* mdbooth is reading tdurakov_ 's async live migration api13:40
johnthetubaguythe tricker bit now, is not breaking the several years of work on upgrades we have in place13:40
*** porunov has joined #openstack-nova13:40
mdboothYup13:40
johnthetubaguyyeah, totally, the stuff we have for instance create and instance snapshot helps establishes a lot of the patterns there two13:41
johnthetubaguyinstance delete as well, I guess13:41
*** tbachman_ has joined #openstack-nova13:42
*** thumpba has quit IRC13:42
*** tbachman has quit IRC13:43
*** tbachman_ is now known as tbachman13:43
*** gongysh has quit IRC13:43
*** burgerk has joined #openstack-nova13:43
johnthetubaguypkoniszewski: bauzas: how are we feeling about this fix now? I am just taking a peak again myself: https://review.openstack.org/#/c/287363/13:44
*** tdurakov_ has quit IRC13:44
*** rmart04 has joined #openstack-nova13:44
*** Daisy has joined #openstack-nova13:45
alaskimdbooth: as johnthetubaguy there are no Tasks yet, but tdurakov and I have discussed this in relation to his work and see what he's doing as a precursor to getting there13:47
pkoniszewskijohnthetubaguy: my feeling is that this is unwanted beginning of a live migration code cleanup13:48
mdboothalaski: Cool. Seemed like an obvious fit from what little I know of the details of Task.13:48
*** tdurakov_ has joined #openstack-nova13:48
*** links has quit IRC13:48
johnthetubaguypkoniszewski: are we not totally broken in the new API if we don't merge something like this one though: https://review.openstack.org/#/c/28736313:48
*** delattec has joined #openstack-nova13:49
pkoniszewskijohnthetubaguy: yeah, without this only forced block migrations will work, if we don't want to use block_migration anymore this is a good (and bad) start13:49
*** acabot has joined #openstack-nova13:50
*** markvoelker has joined #openstack-nova13:50
pkoniszewskiand we should focus on cleaning up all this stuff in Newton I think13:50
pkoniszewskii mean, cleanup that tdurakov started in his series13:50
*** tdurakov has quit IRC13:51
*** tdurakov_ is now known as tdurakov13:51
* alex_xu back from bug smash13:51
*** Guest32605 has joined #openstack-nova13:52
*** rfolco_ has joined #openstack-nova13:52
johnthetubaguypkoniszewski: so the API added block_migration=auto, for that to work, we need your change right?13:52
johnthetubaguypkoniszewski: agreed there is a wider cleanup needed13:52
pkoniszewskijohnthetubaguy: exactly13:52
johnthetubaguybut the wider clean up is totally for newton13:52
pkoniszewskiagree13:52
*** cdelatte has quit IRC13:52
johnthetubaguyright, so we still need your patch in mitaka, I think13:53
tdurakovjohnthetubaguy, we totally need pkoniszewski fix before mitaka release13:53
*** rfolco has quit IRC13:53
johnthetubaguycool, just making sure we are all on the same page here13:53
johnthetubaguyits looking good now, for me, just grepping through things to double check a few things13:54
*** liusheng has joined #openstack-nova13:54
*** rlrossit has joined #openstack-nova13:55
*** rk4n has quit IRC13:55
pkoniszewskijohnthetubaguy: yes, we need it to fix new API13:56
*** thumpba has joined #openstack-nova13:56
johnthetubaguypkoniszewski: so, I think we have an issue in your patch, that fix you did for jaypipes I think is actually a bad fix, once I dug deeper, I think we should instead add a FIXME saying to remove that variable at a later date :( Now its possible it doesn't actually create a bug, but I think we should keep it all consistent.13:57
johnthetubaguypkoniszewski: its these bits: https://review.openstack.org/#/c/287363/7/nova/virt/libvirt/driver.py@6407 sorry, I didn't see those before13:58
pkoniszewskijohnthetubaguy: i have a meeting in few minutes, i will take a look on it right after13:59
*** mriedem has joined #openstack-nova13:59
johnthetubaguyso I have to go run the nova meeting now myself13:59
*** salv-orlando has joined #openstack-nova13:59
*** jaosorior has quit IRC13:59
johnthetubaguyadrian_otto akuriata alevine alexpilotti aloga andreykurilin anteaya artom auggy13:59
johnthetubaguybauzas belliott belmoreira bobball cburgess claudiub danpb dguitarbite _diana_13:59
johnthetubaguydiana_clarke dims duncant edleafe efried flip214 funzo garyk gcb gjayavelu13:59
johnthetubaguyirina_pov jaypipes jcookekhugen jgrimm jichen jlvillal jroll kashyap klindgren13:59
johnthetubaguykrtaylor lbeliveau lxsli macsz markus_z mdorman med_ mikal mjturek mnestratov13:59
johnthetubaguymoshele mrda nagyz ndipanov neiljerram nic Nisha PaulMurray raildo rgeragnov13:59
johnthetubaguysc68cal scottda sdague sileht sorrison swamireddy thomasem thorst tjones tonyb13:59
johnthetubaguytpatil tpatzig xyang rdopiera13:59
* johnthetubaguy make ping noise14:00
kashyapjohnthetubaguy: Which IRC channel?14:00
johnthetubaguyits time for the Nova meeting in #openstack-meeting14:00
kashyapThx14:00
*** tdurakov has quit IRC14:00
*** eliqiao1 has joined #openstack-nova14:01
* danpb wonders why we need shared meeting channels at all - each meeting could have its own meeting channel #openstack-meeting-$meeting-name avoiding the mess of finding free slots14:01
*** tbachman has quit IRC14:02
*** rk4n has joined #openstack-nova14:03
*** burgerk has quit IRC14:03
*** tdurakov_ has joined #openstack-nova14:03
*** openstackgerrit has quit IRC14:03
*** openstackgerrit_ has joined #openstack-nova14:04
*** akshai has joined #openstack-nova14:04
*** jwcroppe has quit IRC14:04
*** haomaiwa_ has joined #openstack-nova14:04
*** openstackgerrit_ is now known as openstackgerrit14:04
*** openstackgerrit has quit IRC14:04
*** jwcroppe has joined #openstack-nova14:04
*** openstackgerrit_ has joined #openstack-nova14:05
*** openstackgerrit_ is now known as openstackgerrit14:05
*** openstackgerrit has quit IRC14:05
*** openstackgerrit_ has joined #openstack-nova14:06
*** openstackgerrit_ is now known as openstackgerrit14:06
*** openstackgerrit has quit IRC14:07
*** pwnall has joined #openstack-nova14:07
*** openstackgerrit_ has joined #openstack-nova14:07
*** gongysh has joined #openstack-nova14:07
thomasemhello14:07
*** openstackgerrit_ is now known as openstackgerrit14:08
*** openstackgerrit has quit IRC14:08
*** mdrabe has joined #openstack-nova14:08
*** openstackgerrit_ has joined #openstack-nova14:08
thomasemdanpb: +114:09
*** openstackgerrit_ is now known as openstackgerrit14:09
*** gjayavel_ has joined #openstack-nova14:09
*** jwcroppe has quit IRC14:09
*** openstackgerrit has quit IRC14:09
*** palexster has quit IRC14:09
*** openstackgerrit_ has joined #openstack-nova14:09
*** amotoki has quit IRC14:10
*** openstackgerrit_ is now known as openstackgerrit14:10
*** Guest41345 has joined #openstack-nova14:10
*** palexster has joined #openstack-nova14:11
openstackgerritSean Dague proposed openstack/nova: API: Raise up HTTPNotFound when no availabe while get_console_output  https://review.openstack.org/27069514:11
mriedemjichen: alex_xu: can we mark https://bugs.launchpad.net/nova/+bug/1552888 as fixed since https://review.openstack.org/#/c/288125/ is merged?14:11
openstackLaunchpad bug 1552888 in OpenStack Compute (nova) "ERROR (BadRequest): expected string or buffer" [High,In progress] - Assigned to Alex Xu (xuhj)14:11
sdaguemriedem: yeh, I think so14:12
*** burgerk has joined #openstack-nova14:12
alex_xumriedem: yea14:12
sdaguethere is a regression test up as well which can get approved14:12
alex_xuI should create another bug for https://review.openstack.org/288268, and that looks like not very critical14:12
mriedemdone, thanks14:12
mriedemsdague: i +Wed the regression test14:12
sdaguemriedem: ok, cool14:13
sdaguedanpb: the conflict is intentional so there isn't every team meeting at 15:00 UTC and you can't participate in more than one14:13
openstackgerritAndrew Laski proposed openstack/nova: Change SpawnIsSynchronous fixture return  https://review.openstack.org/29078014:14
alaskimriedem: fixed up ^. you'll need to rebase14:14
mriedemalaski: ok14:15
*** cleong has joined #openstack-nova14:16
*** claudiub has joined #openstack-nova14:16
mriedembauzas: we just had that cve fix go through yesterday so i guess we need to doc that14:18
bauzasmriedem: I agree14:18
mriedemi can do one for that14:18
bauzasmriedem: we also had 2 security fixes that I can't find as reno notes14:18
*** salv-orlando has quit IRC14:19
mriedemi'm pretty sure we did one for https://bugs.launchpad.net/nova/+bug/151676514:19
openstackLaunchpad bug 1516765 in OpenStack Compute (nova) liberty "[OSSA 2016-002] xenapi: volume_utils._parse_volume_info can leak connection password via StorageError (CVE-2015-8749)" [High,Fix committed] - Assigned to Matt Riedemann (mriedem)14:19
bauzasmriedem: that'd be appreciated, I'm not that horizontally scalable :)14:19
bauzasmriedem: yeah, that's ony of my duties this weeky14:19
bauzasweek14:19
mriedembauzas: hmm, we have a reno in liberty for CVE-2015-8749 but not mitaka, i'll forward port that14:20
openstackgerritMaciej Szankin proposed openstack/nova: Live migration failure in API leaves VM in MIGRATING state  https://review.openstack.org/16891614:20
bauzasmriedem: ack14:20
johnthetubaguymriedem: alex_xu: awesome14:21
*** jichen has quit IRC14:21
*** mtanino has quit IRC14:22
markus_zbauzas: What did you mean with the "automatic dash" in the meeting?14:22
bauzasmarkus_z: https://github.com/sbauza/gerrit-dashboard-nfv14:23
openstackgerritDongcan Ye proposed openstack/nova: VMware: Always update image size for sparse image  https://review.openstack.org/29120314:23
*** garyk2 has joined #openstack-nova14:23
johnthetubaguybauzas: I can look into some of those notes, if we have a list of the ones we need (which I guess is in the etherpad), goes to look...14:23
bauzasmarkus_z: that's basically looking at launchpad whiteboard for getting the list and providing the dash14:23
garyk2johnthetubaguy: is there anychance that someone can please look at https://review.openstack.org/28412114:23
bauzasmarkus_z: but what we could be doing is calling an bitly API for modifying the long URL14:24
bauzasjohnthetubaguy: for the moment, nothing really catched by me14:24
bauzasexcept the security bugs that I need to take care14:25
markus_zbauzas: nice, let me take a look at it14:25
bauzasjohnthetubaguy: the features and upgrades should be quite good now14:25
*** irenab has quit IRC14:25
*** tbachman has joined #openstack-nova14:25
*** Daisy has quit IRC14:25
*** ildikov has joined #openstack-nova14:25
*** Daisy has joined #openstack-nova14:26
*** salv-orlando has joined #openstack-nova14:26
johnthetubaguygaryk2: I can try to hit that one, it would be good to clean up the subteam list on the etherpad, seems to be stale14:27
*** parora has quit IRC14:28
mriedemgaryk2: can you update the commit message to explain what the issue is and how this fixes it?14:28
johnthetubaguybauzas: OK, thats cool14:28
*** artom has joined #openstack-nova14:28
*** dims has quit IRC14:29
*** salv-orlando has quit IRC14:29
*** artom has quit IRC14:29
*** artom has joined #openstack-nova14:29
openstackgerritMaciej Szankin proposed openstack/nova: Live migration failure in API leaves VM in MIGRATING state  https://review.openstack.org/16891614:30
*** Daisy has quit IRC14:30
garyk2johnthetubaguy: mriedem: sure, will update and get the greater team to look at it.14:30
*** ociuhandu has joined #openstack-nova14:30
mriedemthanks14:30
mriedemgaryk2: and i'm assuming vmware nsx ci is not hitting that code path?14:31
*** bharath has joined #openstack-nova14:31
mriedemi wasn't sure if it was using opaque networks / nsxv314:31
garyk2mriedem: at the moment the NSX CI is just running the neutron API tests. We hit this issue when running the tempest tests with Nova.14:31
garyk2opaque networks == nsx_v314:31
mriedemorly14:31
garyk2the CI running for the VCdriver is with the nova network14:32
*** gongysh has quit IRC14:32
mriedemoh...14:32
mriedemany plans to change that to neutron at some point?14:32
garyk2we are working on getting it to post for neutron too.14:32
*** xyang1 has joined #openstack-nova14:34
*** tdurakov_ has quit IRC14:35
*** edtubill has joined #openstack-nova14:35
*** dims has joined #openstack-nova14:35
*** eliqiao1 has quit IRC14:36
*** zz_dimtruck is now known as dimtruck14:36
markus_zbauzas: not sure if the dashboard is the right thing. How do I update it when the data in launchpad changes?14:36
bauzasmarkus_z: using cron14:36
bauzasmarkus_z: but I provided you this for seeing how to use launchpadlib14:37
openstackgerritMatt Riedemann proposed openstack/nova: Add release notes for security fixes in 13.0.0 mitaka GA  https://review.openstack.org/29120814:37
mriedembauzas: ^14:37
sdaguealex_xu: +2 on https://review.openstack.org/#/c/288268/14:37
markus_zbauzas: ah, thanks. I already know how to work with launchpadlib: https://github.com/markuszoeller/openstack/tree/master/scripts/launchpad14:38
alex_xusdague: thanks!14:38
markus_zbauzas: ^ that's how I build the stuff for the etherpad14:38
bauzasmriedem: just a note for the prelude section14:38
*** erno has quit IRC14:38
mriedemwill fix14:39
bauzasmarkus_z: so probably using cron for calling launchpadlib and then modifying a short URL ?14:39
markus_zbauzas: getting the shorturl is done in a few seconds14:40
johnthetubaguymriedem: FYI, is looking like the OSIC folks will put some real effort into that feature classification stuff, so we can ramp up the testing ASAP14:40
bauzasmarkus_z: sure, but that's an human thing14:40
sdaguebauzas: honestly, I feel like it's going to be a losing battle14:40
*** edtubill has quit IRC14:40
sdaguebecause short urls use redirects, so the thing in your webbrowser can't be bookmarked14:40
bauzassdague: tbh, I'm using the new Gerrit UI for my own dashes14:41
bauzassdague: I mean, the ones I use with your own project14:41
*** pcaruana has quit IRC14:41
sdaguebauzas: ok, I just make them bookmarks14:41
bauzassdague: but since the URL is dynamic14:41
bauzassdague: it's difficult14:41
sdaguehonestly, the thing I'd do, is make it a tool that you run, and it opens your browser14:41
bauzassdague: why not14:42
sdaguewebbrowser.open(url)14:42
sdagueit's baked into python14:42
bauzassdague: I was thinking of a shortening service because everyone can use that without the tool14:42
*** haukebruno has quit IRC14:42
sdagueright, but you get the persistent link issues14:42
*** haukebruno has joined #openstack-nova14:42
bauzassdague: hence calling the shortening service API to modify the long URL14:43
*** mrkz has joined #openstack-nova14:43
sdaguesure14:43
bauzasunless I misunderstood your point14:43
markus_zThe tool I have prints the long url to the gerrit query. The short url in the etherpad was just for convenience.14:43
*** edtubill has joined #openstack-nova14:43
*** ducttape_ has joined #openstack-nova14:43
sdaguebut if you actually follow that link, it will resolve in your browser14:43
bauzaswhat I'd like to avoid is waiting markus_z or hassling it to update the query :p14:43
openstackgerritMatt Riedemann proposed openstack/nova: Add release notes for security fixes in 13.0.0 mitaka GA  https://review.openstack.org/29120814:43
bauzashassling him*14:44
markus_zbauzas: You can clone my repo and execute it as you like. :)14:44
*** mgoddard_ has joined #openstack-nova14:44
*** cdent has quit IRC14:44
* markus_z doesn't want do be hasselhoffed14:44
bauzasmarkus_z: heh, okay14:44
bauzasmarkus_z: sure, I can play scripts locally :p14:44
*** esracelik has quit IRC14:44
alex_xupkoniszewski: hope you take a look at this https://review.openstack.org/29028914:44
bauzasI was just thinking of people wanting to contribute reviewing those and *not* running that script, but anyway :)14:45
*** masayukig has quit IRC14:45
markus_zbauzas: that's too easy :)  I have to go early today, maybe in the evening I can put it on a server of mine.14:46
* tbachman wonders what it means to be hasselhoffed14:46
tbachmanNight Driver?14:46
tbachmanBaywatch?14:46
bauzasmarkus_z: heh14:46
tbachmanNight Rider?14:46
* tbachman has a bad memory14:46
markus_ztbachman: vodka soaked burgers will be thrown at you ;)14:46
tbachmanmarkus_z: if you can eat fast enough, is that a bad thing?14:46
tbachman;)14:47
*** masayukig has joined #openstack-nova14:47
tbachmanentertaining, to be sure14:47
markus_ztbachman: I'm not as young anymore as I used to be :)14:47
tbachmanlol14:47
*** mgoddard has quit IRC14:47
tbachmanI must have missed the Baywatch episode with the vodka soaked burgers14:47
tbachmanactually, I missed every episode14:47
*** roeyc has quit IRC14:48
markus_zIt's a classic, every episode is a classic, you should binge-watch it this evening.14:48
tbachmanlol14:48
*** roeyc has joined #openstack-nova14:48
* markus_z leaves for today, see you14:48
*** haukebruno has quit IRC14:48
tbachmaninternet: tool for…. progress?14:48
tbachmanmarkus_z: l8r14:48
*** salv-orlando has joined #openstack-nova14:50
*** edtubill has quit IRC14:50
*** eharney has joined #openstack-nova14:52
cfriesen__is there a way to make "rebuild" overwrite the rootfs in the boot-from-volume case?14:53
*** shangxdy has joined #openstack-nova14:53
*** pcaruana has joined #openstack-nova14:53
*** lefaiseur has quit IRC14:54
ansiwengate-nova-pep8 fails here, although locally `tox -e pep8` is successful. it complains "F821 undefined name 'utils'", although there is a `from nova import utils` in the beginning: https://review.openstack.org/#/c/28556214:54
ansiwenany idea?14:55
*** ekuris_ has quit IRC14:55
*** lefaiseur has joined #openstack-nova14:55
*** tdurakov_ has joined #openstack-nova14:55
*** jaosorior has joined #openstack-nova14:56
mriedemansiwen: huh, looks correct to me14:57
pkoniszewskijohnthetubaguy: just a question regarding your comment here https://review.openstack.org/#/c/287363/7/nova/virt/libvirt/driver.py@640714:57
johnthetubaguycfriesen__: no, but I know our product managers are requesting that feature actually, or something kinda like that, also relates to PaulMurray's work on detaching the root volume of a boot from volume instance14:57
pkoniszewskijohnthetubaguy: we don't allow new->old live migrations, i mean, mitaka->liberty14:57
pkoniszewskido we?14:57
johnthetubaguypkoniszewski: in general, I thought we did allow that, but we actually stopped the new API working until the upgrade is completed14:58
*** edtubill has joined #openstack-nova14:58
*** bharath has quit IRC14:58
*** belmoreira has quit IRC14:59
johnthetubaguypkoniszewski: this logic isn't quite what if first looks like: https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L15214:59
ansiwenmriedem: my code looks correct, you mean?14:59
mriedemansiwen: yeah14:59
ansiwenmriedem: yes, that's why I'm wondering15:00
openstackgerritGleb Stepanov proposed openstack/nova: Redundant argument removed  https://review.openstack.org/29121815:00
ansiwenit's weird15:00
*** haomaiwa_ has quit IRC15:01
*** haomaiwang has joined #openstack-nova15:01
*** mtanino has joined #openstack-nova15:01
pkoniszewskijohnthetubaguy: so, i think we won't break things there, because block_migration is not used anywhere during post operations in both, liberty and mitaka15:02
*** edtubill has quit IRC15:02
openstackgerritAlex Xu proposed openstack/nova: Report instance-actions for live migration force complete API  https://review.openstack.org/29028915:02
*** mgoddard_ has quit IRC15:03
*** mgoddard has joined #openstack-nova15:03
*** dimtruck is now known as zz_dimtruck15:04
*** Jeffrey4l has joined #openstack-nova15:04
johnthetubaguypkoniszewski: so I thought that was probably true, but it feels risky to change that at this point, I am thinking of going back to a minimal fix, then revisiting with the bigger refactor. what do you think?15:04
alex_xuoomichi: hope you can take look at this one when you have time https://review.openstack.org/288268, thanks!15:04
pkoniszewskijohnthetubaguy: this sounds good, let me change it15:05
tdurakov_pkoniszewski, you could pick code from snippet for that15:05
johnthetubaguypkoniszewski: well, the compute manager changes seem worthwhile, I am just thinking about keeping the driver changes minimal, maybe add some FIXMEs for later, so we don't forget when we revisit this15:05
*** cdent has joined #openstack-nova15:05
pkoniszewskijohnthetubaguy: yes, thats what im thinking about too15:05
johnthetubaguypkoniszewski: cool15:06
*** burgerk has quit IRC15:06
*** rdopiera has quit IRC15:06
*** moshele has quit IRC15:07
openstackgerritJianghua Wang proposed openstack/nova: XenAPI: add unit test for plugin bandwidth  https://review.openstack.org/28644515:08
*** cdent has quit IRC15:09
*** cdent has joined #openstack-nova15:09
*** tdurakov_ has quit IRC15:11
lennybmriedem: Hi, you were looking for me, how can I help?15:12
mriedemlennyb: i was looking to run the mellanox ci on a patch, tried to figure out how to recheck for that, it wasn't on the wiki and the contact email on the wiki is blocked15:12
mriedemhere https://wiki.openstack.org/wiki/ThirdPartySystems/Mellanox_CI15:13
bauzasansiwen: could you please make sure that your change is locally  rebased on a master up-to-date ?15:13
openstackgerritYingxin Cheng proposed openstack/nova: split host_state.least_disk_mb out of free_disk_mb  https://review.openstack.org/29030215:13
mriedemlennyb: i see you've updated the contact info, to be lennyb@mellanox.com15:13
mriedemthanks15:13
mriedemlennyb: is there a way to recheck the mellanox ci on a patch?15:13
bauzasansiwen: chances are that you're getting a pep8 conflict because your change is rebased on a top master15:14
lennybmriedem: it's 'recheck nova-mlnx' as written in #link https://review.openstack.org/#/c/242573/ ( look for Mellanox CI comments)15:14
ansiwenbauzas: ok, I will try15:14
bauzasansiwen: I can test that too, lemme verify15:14
mriedemlennyb: ok, i looked at an older post from the ci and the recheck info wasn't in there15:14
mriedemlennyb: can you also get that into the wiki?15:15
lennybmriedem: sure.15:15
mriedemlennyb: and finally :)15:15
mriedemlennyb: would it be possible to run a compute api resize test on that patch alone?15:15
*** efried has quit IRC15:15
mriedemsince i'm told resize with sriov is busted and that patch fixes it,15:15
mriedembut i'd like at least one of the sriov 3rd party CIs to verify that15:15
mriedem*resize/migrate15:15
ansiwenbauzas: right, that was the problem15:16
*** salv-orlando has quit IRC15:16
lennybmriedem, I will check it and update you soon.15:16
mriedemlennyb: thanks15:16
ansiwenbauzas: so, gate is rebasing the change even for the first verify?15:16
bauzasansiwen: glad to hear15:16
bauzasansiwen: what you call 'gate' is the check pipeline and yes, indeed15:16
ansiwenbauzas: well, I call it gate, because the job is called "gate-nova-pep8" ;-)15:17
dansmithbecause it's the same job15:17
*** gjayavel_ has quit IRC15:18
bauzasansiwen: http://logs.openstack.org/62/285562/5/check/gate-nova-pep8/17be69b/console.html#_2016-03-09_17_16_37_75115:18
*** knikolla has joined #openstack-nova15:19
bauzasansiwen: it's now hidden behind the zuul cloner script, but that's basically a rebase everytime15:19
bauzasansiwen: if you want to know more about zuul and infra, I can point you some summit session videos15:19
lennybmriedem: regarding posting how to recheck in wiki, I am not sure that this is a accepted format. None of the third parties CIs has it. I can verify this with anteaya, but as I've mentioned it is written in CI comment of the patch set15:19
*** lefaiseur has quit IRC15:19
*** zz_dimtruck is now known as dimtruck15:19
mriedemlennyb: i've noticed that, but i'm not sure why,15:20
mriedembecause a lot of the time, 3rd party ci hasn't shown up on a change, and it's not marked as down in the wiki,15:20
mriedemso i want to manually kick it15:20
bauzasansiwen: http://docs.openstack.org/infra/system-config/zuul.html#overview15:20
mriedemso i can't use the recheck from the comment b/c hte comment hasn't come yet15:20
anteayalennyb: you can add anything you want to your wikipage15:20
lennybanteaya: thanks15:20
anteayalennyb: the template is just asking for minimal things, you can add any additional items to your page15:21
anteayaeither in the table or below it15:21
anteayaand responding to dev's requests is always a great idea15:21
anteayaso thank you15:21
*** jaosorior has quit IRC15:23
BobBallalaski: Not sure if you're receiving comments on https://bugs.launchpad.net/nova/+bug/1512955 but the XenAPI fix is https://review.openstack.org/#/c/241127/ - not sure why LaunchPad/Gerrit didn't add the link when it was first proposed (but clearly it didn't help ease confusion when a similar Xen fix was posted...)15:25
openstackLaunchpad bug 1512955 in OpenStack Compute (nova) "Race condition in nova/neutron when booting instance with XenAPI driver" [Medium,In progress] - Assigned to huan (huan-xie)15:25
mriedemmikal: dansmith: sdague: johnthetubaguy: tonyb: garyk2: i'll probably request a liberty 12.0.3 release next week for that cve fix, we should get the release note in for that https://review.openstack.org/#/c/291221/15:26
dansmithcool15:27
sdaguemriedem: all the backport patches landed?15:27
sdaguesounds like it should trigger on kilo as well, no?15:27
mriedemsdague: yeah15:28
sdague+A on the reno15:28
mriedemkilo is still a mass push at once15:28
mriedemso i probably need to talk to Daviey about that15:28
pkoniszewskione question - can nova-api write to DB by itself, or does it write everything through conductor?15:28
alaskiBobBall: I am not receiving comments, so thanks15:28
mriedempkoniszewski: you can enable local conductor,15:28
mriedembut it's deprecated15:28
dansmithpkoniszewski: api can always write to the db, yes15:29
johnthetubaguypkoniszewski: but it makes most DB calls directly15:29
johnthetubaguywell, all DB calls, it does also use the conductor, but thats the task API15:29
*** Daisy has joined #openstack-nova15:29
*** efried has joined #openstack-nova15:29
*** aysyd has quit IRC15:30
pkoniszewskithanks! :)15:30
*** aysyd has joined #openstack-nova15:30
bauzaspkoniszewski: all our services are using the object facade, but only the computes are having an indirection set15:31
bauzaspkoniszewski: https://github.com/openstack/nova/blob/master/nova/cmd/compute.py#L64-L6715:31
dansmithbauzas: and network15:31
mriedembauzas: and nova-net15:31
dansmithbauzas: and metadata :)15:31
bauzasoh right, dammit15:31
bauzasthose15:31
pkoniszewskiwell, i think i need to dig through this, thanks bauzas15:31
mriedemand cert!15:31
bauzasokay okay15:32
*** liusheng has quit IRC15:32
mriedempkoniszewski: you can see which services by looking at their commands under nova/cmd/15:32
*** READ10 has joined #openstack-nova15:32
bauzass/only computes/only services running on compute/15:32
mriedemhttp://codesearch.openstack.org/?q=block_db_access&i=nope&files=&repos=15:32
openstackgerritPawel Koniszewski proposed openstack/nova: Use migrate_data.block_migration instead of block_migration  https://review.openstack.org/28736315:32
bauzasmriedem: I jinxed you, see above15:32
*** eharney has quit IRC15:32
bauzasnah/15:32
mriedemoh not cert i guess15:32
bauzasanyway, /me runs15:33
*** garyk2 has quit IRC15:33
*** sridharg has quit IRC15:34
*** aorourke-afk is now known as aorourke15:34
pkoniszewskimriedem: thanks, that's the easy way :)15:34
*** burgerk has joined #openstack-nova15:35
*** eharney has joined #openstack-nova15:35
mdboothMy change, which simply adds a comment to the libvirt driver, got a -1 from xenserver ci :)15:36
BobBallwe don't like comments mdbooth15:36
mriedemdansmith: do we care about making a Certificate object?15:36
mriedemBobBall: the xenlight failure is pretty bad in the xenproject ci15:36
mdboothBobBall: Neither does the libvirt driver, so you're in good company :)15:36
BobBallThe CI thinks that code should be self-commenting ;)15:37
dansmithmriedem: I dunno, do we?15:38
lennybmriedem: which exact tempest tests are you interested in #link  http://paste.openstack.org/show/490019/ ?15:38
BobBallmriedem: I believe there's a fix pending for that (it's around networking) - https://review.openstack.org/#/c/199093/ (although clearly Anthony has work to do to get it working)15:38
sdaguedo we remember where the glance bug is for this - https://bugs.launchpad.net/nova/+bug/1555638  ?15:38
openstackLaunchpad bug 1555638 in OpenStack Compute (nova) "nova image-list return HTTP500" [Undecided,New]15:38
mriedemBobBall: hasn't been updated in 3 months15:39
BobBallmriedem: I know.  I will happily send him an email.  Do you have an example failure to hand I can use to kick Anthony into reworking his patch?15:39
mriedemBobBall: honestly i kind of feel like we should silence the xen project ci until that's fixed15:39
mriedemBobBall: no, but i've rechecked several this week15:39
mriedemBobBall: and it's gotten to the point that i hardly pay attention to that ci anymore15:39
*** jwcroppe has joined #openstack-nova15:39
danpbjaypipes:can you take another look at  https://review.openstack.org/#/c/282436/15:39
mriedemdansmith: sure. in newton.15:40
mriedemdansmith: i'm sure rlrossit would be happy to do that.15:40
rlrossitdo what now?15:40
mriedemlennyb: i suppose at least tempest.api.compute.servers.test_server_actions.ServerActionsTestJSON.test_resize_server_confirm15:41
BobBallmriedem: Indeed.  I will make a note of it to Anthony and the rest of the Xen team.  I hope they will fix it soon.15:41
mriedemlennyb: and tempest.api.compute.servers.test_server_actions.ServerActionsTestJSON.test_resize_server_revert15:41
mriedemlennyb: those are the 2 normal/happy path tests for resize15:41
mriedemBobBall: thanks15:41
mriedemrlrossit: create an object for the Certificate db model15:42
sdaguehttps://bugs.launchpad.net/nova/+bug/1527925 looks like it is it15:42
openstackLaunchpad bug 1527925 in Glance "glanceclient.exc.HTTPInternalServerError when running nova image-list" [High,Confirmed]15:42
rlrossitah I remember talking to doffm about it15:42
mriedemsdague: there was a new bug for the same thing15:42
*** lefaiseur has joined #openstack-nova15:42
mriedemsomeone must have duped it15:42
*** Daisy has quit IRC15:42
*** Daisy has joined #openstack-nova15:42
sdaguemriedem: right, it gets reported a lot15:43
sdagueit's the issue of glanceclient exploding in v1 vs v2 attributes15:43
sdagueiirc15:43
jaypipesdanpb: trying my best. meeting hell today unfortunately. I asked dansmith to take a looksie as well from an o.vo perspecitve.15:44
danpbjaypipes: i changed it to use ovo since your comment15:45
*** pece has quit IRC15:45
*** gjayavel_ has joined #openstack-nova15:45
jaypipesdanpb: cool, thank you. I will try my best to get to that later today. apologies for delay.15:45
dansmithdanpb: anything that flows from neutron to nova is in response to nova having done something and requested neutron to do a thing, right?15:46
danpbdansmith: yes15:46
dansmithdanpb: so the way we handle this now from compute->conductor, is whenever we make a call, we pass a manifest of our object versions, which is small, to conductor and it then knows what needs to happen for us15:47
dansmithnot based on any package version, or parent object version with assumptions, etc15:47
dansmithso can we pass that (or a similar) manifest to neutron on the initial call so that it already knows what we want back? that would avoid hard-coding, hard ordering requirements,15:48
dansmithand let one neutron service multiple novas without having to clamp to the minimum version15:48
danpbdansmith: that problems already dealt with15:48
mriedemjohnthetubaguy: there is no version bump on https://review.openstack.org/#/c/288927/ because a version bump means we need a compat handler, and there is no compat handling for this change15:48
mriedemjohnthetubaguy: it's not like adding a new attribute15:49
lennybmriedem: I've added those tests to Mellanox CI and asked for recheck. I will try to check the result later15:49
dansmithdanpb: it sounded like from your last comment (before your last rev) that it was a thing needing solving.. so what's the plan now?15:49
danpbwhen nova tells neutron what VIFs are supported for a particular plugin,the plugin says what versions it is willing to accept15:49
mriedemlennyb: awesome, thanks15:49
*** browne has joined #openstack-nova15:49
*** dimtruck is now known as zz_dimtruck15:49
dansmithdanpb: okay good15:49
*** EinstCrazy has quit IRC15:49
danpbthe problem i'm having is actually about encoding this info that nova sends to neutron15:49
danpbie when we tell neutron what plugins we support, we have send this info to neutron15:49
danpbpreviously i suggested just sending a json doc with it, but jaypipes thought it should also be a versioned object15:50
*** baoli has quit IRC15:50
dansmithdanpb: okay, my reading of this back then was that you were concerned about the other way around15:50
danpbso https://review.openstack.org/#/c/282436/2/os_vif/objects/compute_info.py  introduces a ComputeInfo object that contains info about the plugins15:50
*** ildikov has quit IRC15:50
dansmithdanpb: so you're concerned about us sending a newer version than it can handle?15:50
*** baoli has joined #openstack-nova15:51
danpbwith the idea that we'll call obj_to_primitive on it, and sent the result to neutron15:51
danpbyeah, this obviously means that neutron's version of os-vif, has to be at least as new as nova's15:51
danpbi was concerned about that initially, but IIUC for live upgrades we already mandated that neutron is updated before nova15:51
danpbso i'm now thinking this requirement is no big deal15:51
dansmithdanpb: so, we could ask the neutron api to deserialize the object syncrhonously, and if it fails, return a 400 with "max-version is $foo" and then we can try again (and cache that version)15:51
*** angdraug has joined #openstack-nova15:52
*** yamahata has joined #openstack-nova15:52
danpboh that's an interesting idea15:52
dansmithshould be an easy thing to do15:52
*** zz_dimtruck is now known as dimtruck15:53
*** palexster has quit IRC15:53
*** ssurana has joined #openstack-nova15:53
danpbok, so using ovo is fine then15:54
*** palexster has joined #openstack-nova15:54
jaypipesthat would pretty much match what we do now with the API and conductor services, right dansmith?15:54
dansmithjaypipes: yar15:54
dansmithjaypipes: and be more "just works" user experience which I think is good15:55
jaypipes++15:55
*** rk4n has quit IRC15:57
*** kfarr has joined #openstack-nova15:59
*** jistr has quit IRC15:59
*** vishwanathj_zzz is now known as vishwanathj16:00
*** Jeffrey4l has quit IRC16:00
*** armax has joined #openstack-nova16:00
*** rk4n has joined #openstack-nova16:00
*** haomaiwang has quit IRC16:01
*** pwnall has quit IRC16:01
*** haomaiwang has joined #openstack-nova16:01
openstackgerritSven Anderson proposed openstack/nova: Avoid parallel DB queries of identical metadata  https://review.openstack.org/28556216:01
*** READ10 is now known as READ10|away16:02
ansiwenbauzas: thanks for the hin16:04
ansiwenhint16:04
*** pwnall has joined #openstack-nova16:05
johnthetubaguymriedem: yeah, agreed its not needed16:05
*** pradk has quit IRC16:06
*** oomichi_ has joined #openstack-nova16:06
*** haomaiwang has quit IRC16:07
*** nkrinner has quit IRC16:08
*** gjayavelu has quit IRC16:08
*** mjura has quit IRC16:09
*** pushkaru has joined #openstack-nova16:09
*** kmartin has joined #openstack-nova16:16
*** tbachman has quit IRC16:16
*** sayalilunkad has quit IRC16:16
*** oomichi_ has quit IRC16:18
*** sayalilunkad has joined #openstack-nova16:19
*** oomichi_ has joined #openstack-nova16:19
*** edtubill has joined #openstack-nova16:20
mriedemsdague: i think i'm going to remove these redundant not implemented checks in https://review.openstack.org/#/c/290134/6/nova/tests/unit/virt/vmwareapi/test_driver_api.py16:22
mriedemthose are tested in test_virt_drivers16:22
*** baoli has quit IRC16:23
*** tbachman has joined #openstack-nova16:26
*** ZZelle has quit IRC16:27
*** bharath has joined #openstack-nova16:28
*** ildikov has joined #openstack-nova16:29
*** baoli has joined #openstack-nova16:29
*** sc68cal has joined #openstack-nova16:30
*** jdurgin1 has joined #openstack-nova16:31
*** sc68cal_ has joined #openstack-nova16:31
*** bharathk has joined #openstack-nova16:32
*** bharath has quit IRC16:32
*** kfarr has quit IRC16:33
*** harlowja_at_home has joined #openstack-nova16:33
*** shangxdy has quit IRC16:35
*** jlanoux has quit IRC16:36
*** irenab has joined #openstack-nova16:37
*** ctrath has joined #openstack-nova16:38
*** Daisy has quit IRC16:39
*** Daisy has joined #openstack-nova16:40
*** rfolco has joined #openstack-nova16:40
*** ducttape_ has quit IRC16:42
*** rfolco_ has quit IRC16:43
*** Daisy has quit IRC16:44
*** efried has quit IRC16:45
*** e0ne_ has quit IRC16:46
*** roeyc has quit IRC16:47
*** pcaruana has quit IRC16:47
*** tbachman has quit IRC16:48
*** ihrachys has quit IRC16:48
*** scheuran has quit IRC16:49
*** burt has joined #openstack-nova16:49
*** READ10|away is now known as READ1016:53
rlrossitmriedem: for your comment in https://review.openstack.org/#/c/290550/2/nova/volume/cinder.py I think the reason they have translate_volume_exception is so they can catch NotFound on the volume_id16:54
*** tbachman has joined #openstack-nova16:54
*** READ10 has quit IRC16:54
rlrossitwow this feels like it will be terrible to try to do the snapshot stuff16:54
*** ducttape_ has joined #openstack-nova16:55
*** jwcroppe has quit IRC16:58
*** jwcroppe has joined #openstack-nova16:59
*** efried has joined #openstack-nova16:59
*** ducttape_ has quit IRC16:59
mriedemrlrossit: damn, you're right https://github.com/openstack/cinder/blob/master/cinder/api/v2/snapshots.py#L16817:00
*** ihrachys has joined #openstack-nova17:00
mriedemrlrossit: idk, maybe snapshot_create needs the snapshot decorator and has to handle NotFound explicitly17:01
rlrossitmriedem: yeah I left a comment on that with an overview of what we need to do in that func17:01
mriedembut i've already closed the tab!17:01
rlrossitctrl+shift+t17:02
rlrossituse it17:02
*** EinstCrazy has joined #openstack-nova17:03
*** jwcroppe has quit IRC17:03
*** lucasagomes is now known as lucas-afk17:04
mriedemrlrossit: if you could use your volume exception decorator also a context manager...17:04
mriedemyou could use that in the snapshot_create method17:04
mriedemand decorate the method with the snapshot decorator17:04
rlrossityeesh17:04
*** dims has quit IRC17:04
*** ndipanov has quit IRC17:04
mriedemdon't yeesh me17:04
*** mpavone has quit IRC17:05
*** derekh has quit IRC17:05
*** rk4n has quit IRC17:05
rlrossitmriedem: can't use the volume decorator as a context manager though17:05
rlrossitit'll steal the OverLimit from the snapshot decorator them17:05
rlrossit*then17:05
mriedemoh, right17:05
*** rmart04 has quit IRC17:05
mriedemwell,17:06
mriedemf it17:06
*** dims has joined #openstack-nova17:06
rlrossitha17:06
mriedemjust try/except like a mofo17:06
mriedemthat's how the neutronv2 wrapper code works17:06
mriedemand if there is one thing i know, we want more of nova to look like nova.network.neutronv2.api17:06
rlrossitI will make it as ugly as possible, just for you17:06
*** Bjoern_ has joined #openstack-nova17:07
rlrossitIs oomichi around to answer that question we had about the API yesterday?17:07
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: check for optional LibvirtLiveMigrateData attrs before loading  https://review.openstack.org/28682817:07
openstackgerritMatt Riedemann proposed openstack/nova: Raise VirtNotImplemented for ComputeDriver methods  https://review.openstack.org/29013417:07
openstackgerritMatt Riedemann proposed openstack/nova: Change SpawnIsSynchronous fixture return  https://review.openstack.org/29078017:07
mriedemrlrossit: i think you have to say his name 2 more times17:07
rlrossitoomichi:17:07
rlrossitbeetlejuice17:07
rlrossitah crap17:07
oomichi_rlrossit: ?17:07
rlrossitmriedem and I had a question yesterday about being able to toss 403 back for pretty much everything on the os-volumes17:08
* rlrossit goes back in the logs to dig out the question17:08
*** EinstCrazy has quit IRC17:08
mriedembasically is there a simple way to handle 403 generically in the REST API code17:09
*** rk4n has joined #openstack-nova17:09
mriedemi.e. if nova.exception.Forbidden comes back17:09
* oomichi_ checking the code...17:09
mriedemcan something translate that to a webob exception and raise it up17:09
mriedemrather than having to add that try/except in every possible volume api method that could hit it17:09
rlrossitand also having to put 403 in the possible returns on all of the functions too17:10
rlrossitwhich, arguably, isn't all that terrible17:10
mriedemrlrossit: well, i think jsonvalidator will require that anyway17:10
rlrossitI have no idea what that means, but I'll nod in agreement17:10
mriedemthe schema validation stuff in the v2.1 api17:11
mriedemwhere the rest api methods have those expected exception decorators17:11
rlrossitwait aren't those 2 different things?17:11
*** paul-carlton2 has joined #openstack-nova17:12
*** thumpba has quit IRC17:12
*** atuvenie has quit IRC17:12
*** thumpba has joined #openstack-nova17:13
*** sc68cal_ has quit IRC17:14
*** tbachman has quit IRC17:15
*** rk4n has quit IRC17:15
oomichi_mriedem: rlrossit: I cannot get the point yet, that means 403 is good exception for all os-volumes cases?17:16
mriedemoomichi: well, it's possible for several of them,17:16
mriedemand we're trying to avoid returning a 500 because we didn't handle the 403 in the RES TAPI17:17
mriedem*REST API17:17
oomichi_mriedem: yeah, 500 is wrong. but the status code should depends on case by case17:17
mriedemalaski: https://review.openstack.org/#/c/286828/ is rebased now,17:17
mriedemalaski: i also replied to your comment in ps3, i didn't get quite get the question17:18
rlrossitoomichi_:  Isn't exceptions.Forbidden always going to be a 403?17:20
*** Sukhdev has joined #openstack-nova17:21
mriedemfor example, https://github.com/openstack/nova/blob/master/nova/compute/api.py#L105617:22
mriedemi could be booting from volume with a volume snapshot17:22
mriedemoh nvm, that would be a 404 i'm thinking of17:22
oomichi_rlrossit: yeah, I think so17:22
oomichi_rlrossit: it is nice to transpate Forbidden to 40317:22
rlrossitI mean I guess we could make a new decorator to stack on top of all of the other ones on the api functions that transforms exceptions.Forbidden into exc.HTTPForbidden...17:23
mriedemhere is the example i was looking for17:23
mriedemhttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L235517:23
*** haomaiwang has joined #openstack-nova17:23
oomichi_mriedem: rlrossit: sorry, I need to go out now for renting a house. I will see it 4 hours later17:24
mriedemif we're doing a snapshot of a volume-backed instance, we could get a 403 from cinder for creating the snapshot (or an overlimit 409)17:24
rlrossitso that's something outside of the volumes API that could still 500 because of a forbidden from cinder?17:24
mriedemwell, it's not handled here https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L110317:25
*** ducttape_ has joined #openstack-nova17:25
rlrossitholy decorators batman17:25
*** READ10 has joined #openstack-nova17:25
*** sudipto has quit IRC17:25
mriedemso if we are trying to snapshot a volume-backed instance, and get a 403 from cinder, we don't handle that in the REST API and return a 50017:25
rlrossitthat one's expecting 403 though17:25
mriedemi need to dig into what that decorator is checking17:26
*** matrohon has quit IRC17:26
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/extensions.py#L46717:26
mriedemi think that's a 50017:27
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/extensions.py#L50417:27
rlrossitI'm guessing wsgi.action('createImage') can raise a 403?17:27
mriedemsure17:27
mriedembut for https://github.com/openstack/nova/blob/master/nova/api/openstack/extensions.py#L48717:27
mriedemi'd think17:27
mriedemnova policy might say you can't17:27
mriedembut if nova policy says you can, and cinder policy says you can't,17:27
rlrossitoh I get it17:27
mriedemthen kablammo17:27
rlrossityup17:27
mriedemso,17:28
mriedemwhat you could *maybe* do in https://github.com/openstack/nova/blob/master/nova/api/openstack/extensions.py#L46717:28
mriedemis add a check for NovaException because NovaException is supposed to have a 'code' kwarg17:28
*** haomaiwang has quit IRC17:28
mriedemand you could translate the generic ones to webob exceptions17:28
rlrossitwe want to generalize it more outside of forbidden?17:28
*** oomichi_ has quit IRC17:29
mriedem403 would be a POC17:29
rlrossitok17:29
mriedemif isinstance(NovaException) and ex.code in errors,17:29
rlrossitbut would I check for types, or code?17:29
mriedemi think code17:29
mriedemnot sure17:29
rlrossitisn't that what L481-486 is already doing?17:30
mriedemi don't know that it would matter since the nova exceptions are supposed to follow a hierarchy17:30
mriedemnot for nova exceptions17:30
mriedemif isinstance(exc, webob.exc.WSGIHTTPException):17:30
rlrossitoh nvm17:30
mriedembut yeah, i think you'd follow that pattern17:30
rlrossitwe aren't in that category17:30
alaskimriedem: I guess I'm not clear on how target_connect_addr would not be available. it seems that migration relies on there being a destination address in there17:30
mriedemalaski: well, there is a dest parameter passed in17:30
mriedemthis overwrites it17:30
mriedemand i don't know why17:30
mriedemalaski: b/c that seems sneaky to me17:31
mriedemalaski: the more i looked at that, the more i thought, if what's passed in is not None and is overwritten, we should probably log something there17:31
alaskiyeah. that alleviates my concern about not having dest, but does seem like something that should be called out17:32
mriedemalaski: i can put up a patch to warn on that17:32
mriedemif it's missing or different17:32
alaskicool17:32
mriedemalaski: but the field is nullable https://github.com/openstack/nova/blob/master/nova/objects/migrate_data.py#L12617:33
mriedemso that's why i wasn't trying to deal with it like it wasn't17:33
alaskimakes sense17:34
mriedemrlrossit: i'm thinking you should write a test patch that raises nova.exception.Forbidden in a rest api that expects 403 but doesn't get a webob exception, like create_image for volume-backed snapshot17:34
*** apoorvad has joined #openstack-nova17:34
mriedemrlrossit: and then you could test out converting that nova exception to a webob exception in the decorator17:34
mriedemrlrossit: and then have ken'ichi take a look17:34
*** tbachman has joined #openstack-nova17:35
*** vilobhmm11 has joined #openstack-nova17:37
*** liverpooler has quit IRC17:37
*** n0ano has quit IRC17:38
*** vilobhmm111 has joined #openstack-nova17:39
*** mgarza has joined #openstack-nova17:40
alaskiI'm not really up to speed on what y'all are discussing so feel free to ignore me, but exception.Forbidden is converted deep down in the wsgi layer with http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/wsgi.py#n42117:40
*** vilobhmm11 has quit IRC17:41
*** browne has quit IRC17:41
mriedemalaski: ooo17:42
mriedemthis is why i said we should just write a test17:42
mriedembecause i know there are layers to this17:42
*** gjayavel_ has quit IRC17:42
*** ivase_ has joined #openstack-nova17:44
*** nic has joined #openstack-nova17:46
*** ivase has quit IRC17:46
*** fawadkhaliq has joined #openstack-nova17:47
*** edtubill has quit IRC17:47
*** nikhil has quit IRC17:51
*** jdurgin1 has quit IRC17:51
*** Sukhdev has quit IRC17:52
*** e0ne has joined #openstack-nova17:53
*** dims has quit IRC17:54
mriedemalaski: unrelated, but since this is in the compute manager network alloc/reschedule flow, you might want to take a look https://review.openstack.org/#/c/283198/17:56
alaskilooking17:58
*** baoli has quit IRC18:00
*** baoli has joined #openstack-nova18:01
* mriedem reboots18:01
*** mriedem has quit IRC18:01
openstackgerritDoug Hellmann proposed openstack/python-novaclient: Update reno for stable/mitaka  https://review.openstack.org/29134118:02
johnthetubaguydansmith: mriedem: does that wait for neutron events stuff still help at all if you don't pause the VM first?18:03
*** dims has joined #openstack-nova18:03
*** vladikr has quit IRC18:05
dansmithjohnthetubaguy: no18:06
dansmithjohnthetubaguy: but the libvirt driver starts paused18:06
johnthetubaguydansmith: I was just looking at this: https://review.openstack.org/#/c/85398/218:06
dansmithjohnthetubaguy: if other hypervisors could do the long neutron bit first and then start the guest, that would help them I guess18:06
johnthetubaguyoops18:06
*** efoley__ has joined #openstack-nova18:06
johnthetubaguyhttps://review.openstack.org/#/c/268258/318:06
*** Swami has joined #openstack-nova18:07
dansmithjohnthetubaguy: I'll comment18:07
johnthetubaguydansmith: thanks18:07
openstackgerritAugustina Ragwitz proposed openstack/nova: Double check aggregate metadata keys due to case insensitivity  https://review.openstack.org/28386818:08
*** moshele has joined #openstack-nova18:09
*** Marga_ has quit IRC18:11
*** Marga_ has joined #openstack-nova18:12
openstackgerritLudovic Beliveau proposed openstack/nova: Update binding:profile for SR-IOV ports  https://review.openstack.org/24257318:12
*** mriedem has joined #openstack-nova18:13
*** nic has quit IRC18:13
*** nic has joined #openstack-nova18:14
*** diana_clarke has quit IRC18:15
*** lpetrut has quit IRC18:16
auggysdague: so i may have actually fixed this, but the fix may need to go in higher up in the stack, i'm not sure https://review.openstack.org/28386818:16
auggysdague: basically the novaclient gets the success status and then assumes that whatever the user submitted is what was actually written to the db18:17
auggysdague: but i'm lowercasing the aggregate metadata key before they submit it, so that's not actually true18:17
auggysdague: so anyways, i'm still poking around at this to see where the novaclient is getting that from and to make sure it's also not the case in the api18:18
openstackgerritChris Dent proposed openstack/nova-specs: Declarative API Tests With Gabbi  https://review.openstack.org/29135218:18
*** moshele has quit IRC18:19
sdagueok, cool do we still have the db case on this? I feel like it's worth probably hitting this with some known data and make sure it acts the right way18:19
cdentsdague ^^ I've come to kill your api samples tests18:19
cdentbecause I'm all out of bubblegum18:20
auggysdague: yeah the functional test is still on it and it should run sometime in the next hour i hope, i am suspecting it will pass18:20
alaskijohnthetubaguy: dansmith thanks for pointing out https://review.openstack.org/#/c/268258, the context manager did not work as I expected18:21
*** dims has quit IRC18:21
dansmithalaski: np, it's not very obvious18:21
*** dims has joined #openstack-nova18:22
auggycdent: those api samples tests are kerazy, hopefully the work you're doing will improve things :)18:22
dansmithalaski: also, I pretty much cherish every rare instance where I know something you don't18:22
*** ihrachys has quit IRC18:23
sdaguecdent: ok, though I'm not really sure that we can evaluate this just at a spec level. The API samples tests are basically a huge part of our API testing. So we've got to at least see what gabbi looks like for a couple of the more complicated instances there.18:23
cdentsdague: yeah, there's going to poc concept code coming along side this18:23
cdents/to/to be/18:23
cdentalready in the works18:24
*** vladikr has joined #openstack-nova18:24
*** ctrath has quit IRC18:24
alaskidansmith: hah, it's not rare at all. I just keep quiet in that case, which explains why I'm always quiet18:24
dansmithheh18:25
*** neophy has joined #openstack-nova18:25
*** omnipresent has quit IRC18:25
*** pece has joined #openstack-nova18:25
*** zenoway has joined #openstack-nova18:26
*** bharath has joined #openstack-nova18:26
*** ducttape_ has quit IRC18:26
*** diana_clarke has joined #openstack-nova18:26
*** ducttape_ has joined #openstack-nova18:26
*** bharathk has quit IRC18:26
*** ctrath has joined #openstack-nova18:27
*** jaosorior has joined #openstack-nova18:28
*** browne has joined #openstack-nova18:29
*** openstackgerrit_ has joined #openstack-nova18:30
*** zenoway has quit IRC18:30
*** kfarr has joined #openstack-nova18:34
*** efoley__ has quit IRC18:36
*** PaulMurray has quit IRC18:38
*** angdraug has quit IRC18:43
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: check for optional LibvirtLiveMigrateData attrs before loading  https://review.openstack.org/28682818:45
openstackgerritMatt Riedemann proposed openstack/nova: Raise VirtNotImplemented for ComputeDriver methods  https://review.openstack.org/29013418:45
openstackgerritMatt Riedemann proposed openstack/nova: Change SpawnIsSynchronous fixture return  https://review.openstack.org/29078018:45
dansmithsdague: so I took a chance on this: https://review.openstack.org/#/c/290077/18:45
dansmithbut it doesn't work18:45
dansmithis there some within-master thing I can use or something?18:45
*** jichen has joined #openstack-nova18:46
alaskimriedem: you undid my updates with that rebase18:46
mriedemwah?!18:47
dansmithsdague: I know I'm kinda cheating, but I can't run those migration commands until after db schema has run, and from-liberty is too early18:47
mriedemgdi18:47
dansmithman, haven't had a good "gdi" in here in months18:47
openstackgerritMatt Riedemann proposed openstack/nova: Change SpawnIsSynchronous fixture return  https://review.openstack.org/29078018:47
*** eharney has quit IRC18:48
openstackgerritAndrew Laski proposed openstack/nova: Change SpawnIsSynchronous fixture return  https://review.openstack.org/29078018:49
alaskimriedem: rebased, but push over that if you need to18:49
mriedemoy18:50
mriedemhere goes18:52
mriedemof course i can't push it up18:52
cburgessgdi?18:52
mriedemno18:52
mriedemsigh18:53
cburgessNo I mean what does that even mean?18:53
mriedemgod damn it18:53
cburgessI feel stupid for not realizing that.18:53
cburgessI'm going to crawl back under my rock now.18:53
*** paul-carlton2 has quit IRC18:53
mriedemadorable18:53
*** READ10 has quit IRC18:54
edleafelol18:55
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: check for optional LibvirtLiveMigrateData attrs before loading  https://review.openstack.org/28682818:55
openstackgerritMatt Riedemann proposed openstack/nova: Raise VirtNotImplemented for ComputeDriver methods  https://review.openstack.org/29013418:55
*** ducttape_ has quit IRC18:56
mriedemHUZZAH!18:56
*** aginwala has joined #openstack-nova18:56
* tbachman wonders if that’s the request to be hasselhoffed18:59
*** vilobhmm111 has quit IRC19:00
*** eharney has joined #openstack-nova19:03
*** danpb has quit IRC19:04
mriedemalaski: so i guess that target_connect_addr would be set in pre live migration here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L670819:04
mriedembut, it defaults to None19:04
mriedemand if not specified, defaults to the dest compute hostname19:04
mriedemit actually seems like we shouldn't even have the target_connect_addr in the object19:05
mriedemit's a config option, and it's only used to override the dest compute hostname19:05
mriedemall of these https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L6703-L670919:06
mriedemdansmith: ^ does that seem weird to you?19:07
alaskiit must be there in case you have a seperate network interface that should be used19:07
mriedemwe're passing config options over the wire for live migration19:07
mriedemalaski: sure, the config option itself is fine,19:07
mriedemit just seems odd that we're storing this as migration data and passing it around19:07
dansmithmriedem: you need to pass the addr the host is configured to use to the other side right?19:08
mriedemyeah i suppose19:08
openstackgerritEd Leafe proposed openstack/python-novaclient: Fixes TypeError on Client instantiation  https://review.openstack.org/24027719:08
openstackgerritEd Leafe proposed openstack/python-novaclient: Change all test URLs to use example.com  https://review.openstack.org/29137019:08
alaskimriedem: it would be better to not overwrite dest in that other method later, and just set migrate_data.target_connect_addr properly once and always use it19:10
*** paul_ has joined #openstack-nova19:11
mriedemalaski: we dont have dest in pre_live_migration where it's set though19:11
*** lpetrut has joined #openstack-nova19:13
mriedemhmm, when it's called we actually do have dest19:13
alaskiI thought it would just be the hostname, but I should just shut my mouth about the virt driver. Nothing is ever as it seems there19:13
*** dimtruck is now known as zz_dimtruck19:14
*** andrewbogott has quit IRC19:14
*** andrewbogott has joined #openstack-nova19:14
mriedemalaski: as in CONF.host?19:14
mriedemi think that's right19:14
mriedempre_live_migration in the compute rpcapi just uses dest to prepare the call19:15
mriedemit doesn't pass it over, because it'll be on that host,19:15
mriedemso CONF.host19:15
mriedemif that host has target_connect_addr set, it uses it, else it's None (and dest will be used during live migration later)19:15
*** jistr has joined #openstack-nova19:15
*** zenoway has joined #openstack-nova19:16
mriedemanyway, i'm not sure i want to get within 10 miles of touching any of this code...19:16
*** ducttape_ has joined #openstack-nova19:17
dansmithmriedem: I can recommend not getting near it :)19:17
*** edtubill has joined #openstack-nova19:18
alaskiafter tracing this I have no clue. dest originates in the API from user input so it may or may not match CONF.host.19:19
*** kfarr has quit IRC19:20
dansmithdest has to be a hypervisor hostname I think19:20
dansmithso we know which one we're talking to19:20
dansmithi.e. the same as the topic19:20
*** moshele has joined #openstack-nova19:20
dansmithwhich may not be (at all) how the two communicate19:20
mriedemi thought dest came from the live_migrate task in conductor19:24
mriedemhttps://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L16419:24
*** vilobhmm11 has joined #openstack-nova19:25
mriedemyou can request a dest, which is then validated here https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L10219:25
*** vilobhmm11 has quit IRC19:25
*** zul has quit IRC19:25
*** vilobhmm11 has joined #openstack-nova19:25
*** zul has joined #openstack-nova19:26
alaskidestination is passed in at https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L30719:26
alaskipulled from a scheduler hint set in compute/api19:26
*** dims_ has joined #openstack-nova19:28
*** moshele has quit IRC19:28
*** mvk has quit IRC19:28
mriedembut couldn't that be None?19:28
alaskimaybe it is possible to just say live-migrate to wherever, though that seems a bit crazy19:28
mriedemwhy is that crazier than cold migrate to wherever?19:29
*** dims has quit IRC19:29
*** Sukhdev has joined #openstack-nova19:29
alaskilive migrate is usally done by admins trying to clear hosts for upgrade, in which case they want to carefully control where things go. I guess in the 'host is about to die' case an unspecified live-migrate could be useful19:30
alaskithe resource cost of it just seems like something that should be controlled19:31
*** bharath has quit IRC19:31
*** gszasz has quit IRC19:31
*** jaosorior has quit IRC19:31
alaskieither way, it seems the api allows destination to be None19:32
openstackgerritMark Doffman proposed openstack/nova-specs: Move networking tables to the API db.  https://review.openstack.org/29138219:32
mriedemalaski: yeah, this is why i was asking the other day if it was possible to migrate from a compute service that is disabled19:32
mriedemi.e. i want to upgrade compute A, so disable the service for compute A so nothing schedules to it19:32
*** andreykurilin__ has quit IRC19:33
mriedemthe docs don't really talk about doing anything like that for maintenance19:33
alaskiI believe that works19:34
*** rlrossit has quit IRC19:34
*** rlrossi__ has joined #openstack-nova19:37
*** rlrossi__ has quit IRC19:37
*** rlrossi__ has joined #openstack-nova19:38
*** rlrossi__ has quit IRC19:38
*** rcernin has quit IRC19:40
*** cdent has quit IRC19:42
*** e0ne has quit IRC19:43
*** Sukhdev has quit IRC19:43
*** armax has quit IRC19:46
*** nelsnelson has quit IRC19:47
*** baoli has quit IRC19:50
doffmalaski: Have you thought much about the quota system for cells? AFAICT they quota tables will have to go in the API db to work, but that will lead to a hammering of the api db. (Touched for any request across the deployment that affects quota)19:50
doffmMy current idea is to put quota tables in API db so that it 'works' (but might not scale very well)19:50
doffmThen worry about alternative quota drivers later.19:51
alaskidoffm: just a little. I agree it needs to be in the api db19:51
alaskithe big question is whether there is anything that can happen to affect quotas which doesn't pass through the API19:51
alaskibecause if not the cells never need to touch the api db for that19:52
doffmI'm not sure about that. But right now quota is updated from the compute nodes right?19:52
doffmWe would have to change the way quota is recorded.19:52
alaskiquotas are a mess so I won't claim any real knowledge, but my understanding is that it's all/mostly handled in the api via reservations19:53
alaskithe computes just confirm/deny the reservation19:53
doffmalaski: Ok. You're right its not easy to comprehend whats going on with quotas.19:53
alaskiin cellsv1 quotas are handled only in the top level cell, equivalent to api for this discussion, and take an optimistic view of just commiting things before sending to the cells19:54
openstackgerritSylvain Bauza proposed openstack/python-novaclient: Update reno for stable/mitaka  https://review.openstack.org/29134119:55
bauzasdhellmann: FYI, just pushed an edit on https://review.openstack.org/#/c/291341/19:55
doffmalaski: Well the fact that they were doing that in cellsv1 and didn't see any issues with it is encouraging.19:55
alaskidoffm: well, no more issues than anyone else I think :)19:55
dhellmannbauzas : thanks!19:56
doffmalaski: Heh, yeah. I'm not talking about all the *other* quota issue.19:56
bauzasdhellmann: hopefully it should fix the job19:56
dhellmannbauzas : +1 LGTM19:56
doffmalaski: Thanks, I guess i'll propose moving all the quota tables to API. I'll mention possible scaling issues, but not worry about it for now.19:57
*** ildikov has quit IRC19:58
bauzasdhellmann: https://github.com/openstack/releases/blob/master/deliverables/mitaka/python-novaclient.yaml is not mentioning a stable branch, have you already cut the branch ?19:58
bauzasnvm https://github.com/openstack/python-novaclient/tree/stable/mitaka20:00
*** ildikov has joined #openstack-nova20:00
*** aginwala has quit IRC20:03
*** lefaiseur has quit IRC20:04
*** rlrossit has joined #openstack-nova20:07
*** paul_ has quit IRC20:07
*** aginwala has joined #openstack-nova20:08
*** eglynn has quit IRC20:08
*** irenab_ has joined #openstack-nova20:08
*** irenab has quit IRC20:10
*** irenab_ is now known as irenab20:10
*** e0ne has joined #openstack-nova20:12
*** diogogmt has joined #openstack-nova20:13
*** Swami has quit IRC20:15
*** ducttape_ has quit IRC20:18
*** matrohon has joined #openstack-nova20:19
*** armax has joined #openstack-nova20:20
*** ducttape_ has joined #openstack-nova20:20
*** irenab_ has joined #openstack-nova20:21
*** vilobhmm11 has quit IRC20:22
*** irenab has quit IRC20:23
*** vilobhmm11 has joined #openstack-nova20:23
*** zz_dimtruck is now known as dimtruck20:23
*** e0ne has quit IRC20:23
*** ociuhandu has quit IRC20:24
*** irenab_ has quit IRC20:26
*** cirros has joined #openstack-nova20:27
*** pushkaru has quit IRC20:27
*** irenab has joined #openstack-nova20:27
*** cirros has quit IRC20:28
*** vilobhmm11 has quit IRC20:29
*** vilobhmm11 has joined #openstack-nova20:29
*** Sukhdev has joined #openstack-nova20:30
*** vilobhmm11 has quit IRC20:33
*** erlon has quit IRC20:36
*** pushkaru has joined #openstack-nova20:37
*** sheel has quit IRC20:37
*** angdraug has joined #openstack-nova20:38
*** thumpba has quit IRC20:50
*** thumpba has joined #openstack-nova20:51
*** yamahata has quit IRC20:52
*** thumpba has quit IRC20:55
*** matrohon has quit IRC20:55
*** smatzek has quit IRC20:56
*** haomaiwang has joined #openstack-nova20:56
*** rfolco has quit IRC21:00
*** haomaiwang has quit IRC21:01
mriedemdhellmann: question in https://review.openstack.org/#/c/291341/21:02
tonybmriedem: re 12.0.3, sounds good.21:05
*** toanster has quit IRC21:10
dhellmannbauzas : yes, the branches aren't listed in the deliverables files21:11
*** toan has joined #openstack-nova21:11
dhellmannmriedem: answered21:12
*** krtaylor has quit IRC21:13
openstackgerritMatt Riedemann proposed openstack/python-novaclient: Update reno for stable/mitaka  https://review.openstack.org/29134121:13
mriedemdhellmann: done, thanks21:13
mriedemdhellmann: also, is it safe to merge that now or do we need to wait for stable/mitaka to be created?21:14
dhellmannmriedem : when I looked at the update, I noticed that it was in the toctree but didn't pay attention to ordering21:14
dhellmannmriedem : the stable branch for the client should already exist, that's where the patch came from (all one script)21:14
mriedemoh it's the client21:14
mriedemdidn't notice21:14
dhellmannmriedem : ETOOMANYPATCHES21:15
*** ctrath has quit IRC21:16
openstackgerritGage Hugo proposed openstack/nova: Added extra_resources field to ComputeNode  https://review.openstack.org/29143621:18
*** aginwala has quit IRC21:18
*** ctrath has joined #openstack-nova21:18
*** vilobhmm11 has joined #openstack-nova21:19
*** vilobhmm11 has quit IRC21:19
*** vilobhmm11 has joined #openstack-nova21:19
*** Marga_ has quit IRC21:20
*** penick has joined #openstack-nova21:21
*** Sukhdev has quit IRC21:25
*** burt has quit IRC21:27
*** cristicalin has joined #openstack-nova21:28
*** cristicalin has quit IRC21:28
*** jichen has quit IRC21:29
*** boris-42 has joined #openstack-nova21:30
*** liverpooler has joined #openstack-nova21:32
*** aginwala has joined #openstack-nova21:34
*** haplo37 has joined #openstack-nova21:34
*** aginwala has quit IRC21:34
*** gjayavelu has joined #openstack-nova21:34
*** Sukhdev has joined #openstack-nova21:40
*** Sukhdev has quit IRC21:40
*** jistr has quit IRC21:40
*** hamzy has quit IRC21:40
*** Sukhdev has joined #openstack-nova21:41
*** openstackstatus has quit IRC21:42
*** openstackstatus has joined #openstack-nova21:44
*** ChanServ sets mode: +v openstackstatus21:44
*** rk4n has joined #openstack-nova21:48
*** ihrachys has joined #openstack-nova21:49
*** edtubill has quit IRC21:49
*** sdague has quit IRC21:52
*** dims has joined #openstack-nova21:52
*** jamielennox|away is now known as jamielennox21:52
*** dims_ has quit IRC21:52
*** pece has quit IRC21:53
*** Marga_ has joined #openstack-nova21:54
*** neophy has quit IRC21:54
*** ociuhandu has joined #openstack-nova21:55
*** mdrabe has quit IRC21:56
*** thorst_ has quit IRC21:56
*** ociuhandu has quit IRC21:56
*** ociuhandu has joined #openstack-nova21:57
rlrossitmriedem: https://bugs.launchpad.net/nova/+bug/1555826 ur fave21:58
openstackLaunchpad bug 1555826 in OpenStack Compute (nova) "Functional API client has incorrect URL for volumes" [Undecided,New]21:58
*** doug-fish has quit IRC21:59
*** melwitt has quit IRC22:00
*** delattec has quit IRC22:02
*** thorst_ has joined #openstack-nova22:02
*** thorst__ has joined #openstack-nova22:04
*** cleong has quit IRC22:06
*** thorst_ has quit IRC22:06
*** thorst__ has quit IRC22:08
*** mrkz has quit IRC22:10
*** mrkz has joined #openstack-nova22:10
mriedemdansmith: we should get a FutureWarning for this right? https://review.openstack.org/#/c/259813/7/nova/tests/unit/network/security_group/test_neutron_driver.py22:10
mriedemb/c i don't see one http://logs.openstack.org/13/259813/7/check/gate-nova-python27/57b09b3/console.html#_2016-03-06_15_27_26_09522:11
*** rcernin has joined #openstack-nova22:11
dansmithyeah, if we're using the new enough o.vo22:11
dansmithI have no idea if we are, but regardless, you're right on the comment of course22:12
*** pushkaru has quit IRC22:12
mriedemwe must be http://logs.openstack.org/13/259813/7/check/gate-nova-python27/57b09b3/console.html#_2016-03-06_15_27_26_17922:12
mriedemit warned on another one22:12
rlrossitdansmith: mriedem: doesn't matter if we are, we will eventually22:12
dansmithyeah22:12
rlrossitdrop the hammer22:12
mriedemi dropped the hammer,22:13
mriedembut it didn't warn, so i'm confused22:13
*** knikolla has quit IRC22:13
*** aysyd has quit IRC22:13
*** porunov has quit IRC22:14
*** manjeets has joined #openstack-nova22:14
manjeetstoday i clonned nova repo and built a one node devstack22:15
manjeetsbut when i spin instance scheduler keep on trying22:16
*** melwitt has joined #openstack-nova22:16
*** melwitt has quit IRC22:16
*** pushkaru has joined #openstack-nova22:17
openstackgerritRyan Rossiter proposed openstack/nova: Add regression test for Cinder 403 forwarding  https://review.openstack.org/29146422:17
rlrossitmriedem: functional will blow up on that ^22:17
rlrossitmriedem: I realize I can do this too... http://paste.openstack.org/show/490075/ is what it's barfing out22:22
*** thorst_ has joined #openstack-nova22:22
*** rcernin has quit IRC22:22
*** rk4n has quit IRC22:24
*** pushkaru has quit IRC22:24
*** pushkaru has joined #openstack-nova22:24
mriedemrlrossit: alaski: aha22:25
mriedemhttp://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/extensions.py#n50122:25
mriedemthat's in the decorator in the v2.1 api22:25
mriedemwhich translates to a 50022:25
*** thorst__ has joined #openstack-nova22:26
mriedemso the thing wayyyyy down in the wsgi layer that alaski pointed out is busted22:26
mriedemb/c it's not getting a NovaException, it's getting a webob 50022:26
mriedemwell, it's not busted, but it's being short-circuited22:26
mriedemhttp://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/wsgi.py#n42122:28
mriedemso you're hitting this http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/wsgi.py#n43822:28
alaskiPolicyNotAuthorized is a subclass of Forbidden, so isinstance() isn't the right check there22:29
*** thorst_ has quit IRC22:29
mriedemb/c expected_errors barfed on nova.exception.Forbidden22:29
*** pushkaru has quit IRC22:29
*** woodster_ has joined #openstack-nova22:29
mriedemalaski: yeah good point22:29
alaskiissubclass would be better there22:30
mriedemchange http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/extensions.py#n487 to Forbidden and the functional test should work22:30
alaskiI think you still want to allow PolicyNotAuthorized there22:30
mriedemisn't PolicyNotAuthorized an instance of Forbidden?22:31
mriedemlike objects.Instance is an instance of nova.objects.base.NovaObject22:31
mriedemif it's not, that's terrible22:31
mriedemand we have to rewrite nova in java immediately22:31
alaskino you're right22:31
rlrossitmriedem: it is22:31
mriedemwhew22:31
alaskiI had the hierarchy backwards in my head22:32
*** diogogmt has quit IRC22:32
* mriedem leaves22:34
*** mriedem has quit IRC22:34
*** ctrath has quit IRC22:35
openstackgerritGiridhar Jayavelu proposed openstack/nova: VMware: use datacenter path to fetch image  https://review.openstack.org/28961322:37
manjeetstoday when i stacked on devstack on able to schedule instances22:38
manjeetshttp://paste.openstack.org/show/490076/ I am getting in n-cond logs22:39
*** melwitt has joined #openstack-nova22:39
manjeetsbut n-api and n-crt, n-cpu, n-sch i don't see any trace22:39
*** pushkaru has joined #openstack-nova22:39
*** gyee has joined #openstack-nova22:40
*** melwitt has quit IRC22:40
*** zenoway has quit IRC22:40
*** melwitt has joined #openstack-nova22:41
*** melwitt has quit IRC22:41
openstackgerritSujitha proposed openstack/nova: Added new exception to handle CinderClientException  https://review.openstack.org/29015622:42
*** melwitt has joined #openstack-nova22:42
*** cdelatte has joined #openstack-nova22:43
*** pushkaru has quit IRC22:43
*** pushkaru has joined #openstack-nova22:43
*** rtheis has quit IRC22:44
*** Bjoern_ has quit IRC22:44
*** burgerk has quit IRC22:44
*** vilobhmm11 has quit IRC22:45
*** vilobhmm11 has joined #openstack-nova22:46
*** melwitt has quit IRC22:46
*** melwitt has joined #openstack-nova22:47
*** aginwala has joined #openstack-nova22:47
*** mgarza has quit IRC22:54
*** zenoway has joined #openstack-nova22:54
*** melwitt has quit IRC22:55
*** melwitt has joined #openstack-nova22:55
*** melwitt has quit IRC22:56
*** melwitt has joined #openstack-nova22:56
*** rlrossit has quit IRC22:58
*** doug-fish has joined #openstack-nova22:59
*** zenoway has quit IRC22:59
*** melwitt has quit IRC23:00
*** melwitt has joined #openstack-nova23:00
*** doug-fish has quit IRC23:03
*** doug-fish has joined #openstack-nova23:03
*** melwitt has quit IRC23:04
*** melwitt has joined #openstack-nova23:04
*** diogogmt has joined #openstack-nova23:04
*** krtaylor has joined #openstack-nova23:05
*** doug-fish has quit IRC23:08
*** derekh has joined #openstack-nova23:09
*** haplo37 has quit IRC23:11
*** krtaylor has quit IRC23:11
*** doug-fish has joined #openstack-nova23:14
*** ivase_ has quit IRC23:15
*** ducttape_ has quit IRC23:15
*** ihrachys has quit IRC23:16
*** ivase has joined #openstack-nova23:18
*** doug-fish has quit IRC23:19
*** ivase has quit IRC23:23
*** renatoarmani has joined #openstack-nova23:23
*** jwcroppe has joined #openstack-nova23:24
*** krtaylor has joined #openstack-nova23:25
*** xyang1 has quit IRC23:26
*** mrkz has quit IRC23:26
*** mrkz has joined #openstack-nova23:27
*** haomaiwang has joined #openstack-nova23:27
*** zenoway has joined #openstack-nova23:30
*** vladikr has quit IRC23:31
*** haomaiwang has quit IRC23:32
*** renatoarmani has quit IRC23:34
*** zenoway has quit IRC23:35
*** rk4n has joined #openstack-nova23:36
openstackgerritAllen Gao proposed openstack/python-novaclient: replace six.iteritems() with dict.items()  https://review.openstack.org/29119423:37
*** derekh has quit IRC23:39
*** Sukhdev has quit IRC23:44
*** Sukhdev has joined #openstack-nova23:45
*** yamahata has joined #openstack-nova23:46
*** gongysh has joined #openstack-nova23:47
*** penick has quit IRC23:51
*** penick has joined #openstack-nova23:54
*** mrkz has quit IRC23:55
*** Swami has joined #openstack-nova23:56
*** penick has quit IRC23:57
*** jwcroppe has quit IRC23:57
*** Sukhdev_ has joined #openstack-nova23:57
*** Sukhdev has quit IRC23:57
*** sheel has joined #openstack-nova23:58

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