Thursday, 2015-06-25

*** claudiub has quit IRC00:05
*** salv-orlando has joined #openstack-nova00:05
*** david-ly_ has joined #openstack-nova00:06
*** jwcroppe has quit IRC00:07
*** ijw_ has joined #openstack-nova00:07
*** jwcroppe has joined #openstack-nova00:07
*** david-lyle has quit IRC00:08
*** jwcroppe has quit IRC00:10
*** ijw__ has quit IRC00:10
*** salv-orlando has quit IRC00:12
*** Longgeek has joined #openstack-nova00:12
*** jwcroppe has joined #openstack-nova00:13
*** dims_ has joined #openstack-nova00:14
*** edmondsw has joined #openstack-nova00:16
*** dims has quit IRC00:16
*** edmondsw_ has joined #openstack-nova00:16
*** edmondsw_ has quit IRC00:17
*** salv-orlando has joined #openstack-nova00:21
*** Longgeek has quit IRC00:23
*** salv-orlando has quit IRC00:27
*** marun has quit IRC00:27
*** ssurana has joined #openstack-nova00:28
*** ijw_ has quit IRC00:29
*** rushiagr_away is now known as rushiagr00:32
*** dboik has quit IRC00:32
*** park_heijlong has joined #openstack-nova00:33
*** electrocucaracha has quit IRC00:34
*** otter768 has quit IRC00:35
*** dprince has joined #openstack-nova00:37
*** jmccrory has quit IRC00:38
*** rwsu has quit IRC00:38
*** thorst has joined #openstack-nova00:39
*** jmccrory has joined #openstack-nova00:40
*** boris-42 has quit IRC00:42
*** husanu has quit IRC00:43
*** mriedem has quit IRC00:45
*** husanu has joined #openstack-nova00:45
*** jwcroppe_ has joined #openstack-nova00:47
*** edmondsw has quit IRC00:47
*** diogogmt has quit IRC00:48
*** r-daneel has quit IRC00:48
*** rushiagr is now known as rushiagr_away00:49
*** jwcroppe has quit IRC00:49
*** zhenguo has joined #openstack-nova00:52
*** tobe has joined #openstack-nova00:56
*** thorst has quit IRC01:00
*** amotoki has joined #openstack-nova01:01
*** Jeffrey4l has joined #openstack-nova01:02
PietThe UX team is still looking for folks that are willing to participant in a one hour interview.  You should be currently using Nova-Network and chat about why/why not you're moving to Neutron01:03
PietFeel free to reach-out to me at pieter.c.kruithof-jr <@> hp.com01:04
*** busterswt has joined #openstack-nova01:04
*** tjones1 has joined #openstack-nova01:05
*** amotoki has quit IRC01:05
*** park_heijlong has quit IRC01:10
*** ssurana has quit IRC01:10
*** baoli has quit IRC01:13
*** dprince has quit IRC01:22
*** READ10 has joined #openstack-nova01:26
*** sdake has joined #openstack-nova01:28
*** sdake_ has joined #openstack-nova01:29
*** eliqiao1 has joined #openstack-nova01:32
*** eliqiao1 has left #openstack-nova01:33
*** sdake has quit IRC01:33
*** david-ly_ is now known as david-lyle01:36
*** yingjun has joined #openstack-nova01:39
*** rfolco has quit IRC01:45
openstackgerritZhenyu Zheng proposed openstack/nova: Add instance action events for live migration  https://review.openstack.org/19537201:45
*** dims_ has quit IRC01:45
*** mdrabe has joined #openstack-nova01:50
*** dims has joined #openstack-nova01:50
*** tjones1 has quit IRC01:52
openstackgerritDoug Royal proposed openstack/nova: fix typos in docs  https://review.openstack.org/19537401:52
*** dontalton has quit IRC01:56
*** mdrabe has quit IRC01:58
*** tjones1 has joined #openstack-nova01:58
*** _diana_ has joined #openstack-nova02:01
*** yingjun has quit IRC02:02
*** asettle is now known as asettle-afk02:02
*** dims has quit IRC02:04
*** yingjun has joined #openstack-nova02:08
*** dims has joined #openstack-nova02:10
*** park_heijlong has joined #openstack-nova02:12
*** sdake has joined #openstack-nova02:13
*** dramakri has quit IRC02:15
*** mdrabe has joined #openstack-nova02:15
*** sdake_ has quit IRC02:17
*** sharkJing has joined #openstack-nova02:17
*** unicell has quit IRC02:17
*** READ10 has quit IRC02:20
*** patrickeast has quit IRC02:22
*** yamahata has quit IRC02:28
*** tjones1 has quit IRC02:28
*** salv-orl_ has joined #openstack-nova02:29
*** armax has joined #openstack-nova02:32
*** bkopilov has quit IRC02:33
*** salv-orl_ has quit IRC02:34
*** otter768 has joined #openstack-nova02:36
*** diogogmt has joined #openstack-nova02:40
*** otter768 has quit IRC02:41
*** Jeffrey4l has quit IRC02:41
*** Piet has quit IRC02:42
*** Piet has joined #openstack-nova02:43
*** mestery has joined #openstack-nova02:44
openstackgerritmelanie witt proposed openstack/nova-specs: Database connection switching for cells  https://review.openstack.org/19493502:45
*** IanGovett has quit IRC02:49
*** pixelb has quit IRC02:52
*** Jeffrey4l has joined #openstack-nova02:53
*** mestery has quit IRC02:57
*** mestery has joined #openstack-nova02:57
*** dims has quit IRC02:58
*** dave-mccowan has quit IRC02:59
*** Jeffrey4l has quit IRC03:00
*** Sukhdev has joined #openstack-nova03:01
*** setuid has quit IRC03:02
*** amotoki has joined #openstack-nova03:02
*** amotoki has quit IRC03:06
*** _diana_ has quit IRC03:06
*** setuid has joined #openstack-nova03:11
*** zul has quit IRC03:13
*** Jeffrey4l has joined #openstack-nova03:13
*** asettle-afk is now known as asettle03:15
*** zul has joined #openstack-nova03:16
*** Longgeek has joined #openstack-nova03:20
*** Longgeek has quit IRC03:21
*** Longgeek has joined #openstack-nova03:21
*** mdrabe has quit IRC03:23
openstackgerritZhenyu Zheng proposed openstack/nova: Add instance action events for live migration  https://review.openstack.org/19537203:24
*** tobe has quit IRC03:26
*** yingjun has quit IRC03:27
*** mtanino has quit IRC03:31
*** dave-mccowan has joined #openstack-nova03:37
*** diogogmt has quit IRC03:38
*** tojuvone has joined #openstack-nova03:38
*** tobe has joined #openstack-nova03:39
*** Jeffrey4l has quit IRC03:45
*** xyhuang has joined #openstack-nova03:46
*** bkopilov has joined #openstack-nova03:54
*** deepthi has joined #openstack-nova03:55
*** Jeffrey4l has joined #openstack-nova03:57
*** arnaud____ has joined #openstack-nova03:57
*** coolsvap|away is now known as coolsvap04:04
*** dramakri has joined #openstack-nova04:04
*** rajesht has joined #openstack-nova04:08
*** busterswt has quit IRC04:09
*** vilobhmm has joined #openstack-nova04:14
*** hemna has joined #openstack-nova04:19
*** mestery has quit IRC04:19
*** mestery_ has joined #openstack-nova04:19
*** achanda has joined #openstack-nova04:23
*** tojuvone has quit IRC04:30
*** garyk has quit IRC04:32
*** annashen has joined #openstack-nova04:36
*** otter768 has joined #openstack-nova04:37
*** Longgeek has quit IRC04:37
*** Longgeek has joined #openstack-nova04:37
*** hemna has quit IRC04:40
*** unicell has joined #openstack-nova04:40
*** salv-orlando has joined #openstack-nova04:41
*** hemna has joined #openstack-nova04:42
*** otter768 has quit IRC04:42
*** isd has joined #openstack-nova04:42
*** Longgeek has quit IRC04:44
*** sorrison_laptop has quit IRC04:45
*** salv-orlando has quit IRC04:47
*** tjones1 has joined #openstack-nova04:53
*** cfriesen_ has quit IRC04:54
*** _gryf|afk is now known as _gryf04:55
*** Sukhdev has quit IRC05:01
*** Marga_ has joined #openstack-nova05:02
*** arnaud____ has quit IRC05:03
*** hemna has quit IRC05:05
*** dramakri has left #openstack-nova05:05
*** dramakri has quit IRC05:05
*** annashen has quit IRC05:05
*** achanda has quit IRC05:06
*** mestery_ has quit IRC05:09
*** tjones1 has quit IRC05:12
*** tojuvone has joined #openstack-nova05:14
*** irenab has joined #openstack-nova05:14
*** Longgeek has joined #openstack-nova05:17
*** sorrison_laptop has joined #openstack-nova05:19
*** unicell1 has joined #openstack-nova05:24
openstackgerritGhanshyam Mann proposed openstack/nova: Server list filter with tenant_id implies all_tenants  https://review.openstack.org/19541205:24
openstackgerritNikhil Komawar proposed openstack/nova-specs: Add support for Glance v2 API  https://review.openstack.org/19494505:25
*** unicell has quit IRC05:25
*** ildikov has quit IRC05:26
*** vilobhmm has quit IRC05:26
openstackgerritNikhil Komawar proposed openstack/nova-specs: Add support for Glance v2 API  https://review.openstack.org/19494505:27
park__heijlongdansmith: are you around?05:27
*** garyk has joined #openstack-nova05:33
openstackgerritGhanshyam Mann proposed openstack/nova: Merge availability_zone tests between v2 and v2.1  https://review.openstack.org/17101705:35
openstackgerritMadhuri Kumari proposed openstack/nova: Merge SchedulerHints functional tests of v2 and v2.1  https://review.openstack.org/16790305:35
*** arnaud____ has joined #openstack-nova05:36
*** nkrinner has joined #openstack-nova05:39
*** markvoelker has quit IRC05:43
*** armax has quit IRC05:49
*** asettle has quit IRC05:51
openstackgerritjunxu proposed openstack/nova-specs: Allow force stop server in nova rest api  https://review.openstack.org/19541805:51
jheskethjohnthetubaguy: ping05:54
*** moshele has joined #openstack-nova05:58
openstackgerritjunxu proposed openstack/nova-specs: Allow force stop server in nova rest api  https://review.openstack.org/19541806:00
*** browne has quit IRC06:01
*** radek__ has joined #openstack-nova06:01
*** raildo has quit IRC06:04
*** samueldmq has quit IRC06:04
*** tellesnobrega has quit IRC06:05
*** ericksonsantos has quit IRC06:05
*** destiny has joined #openstack-nova06:07
*** destiny has left #openstack-nova06:08
*** Mike_D_laptop has quit IRC06:08
*** Murali has joined #openstack-nova06:08
*** ildikov has joined #openstack-nova06:13
*** tobe has quit IRC06:14
*** jmccrory has quit IRC06:15
*** changbl_ has quit IRC06:15
*** jmccrory has joined #openstack-nova06:20
*** mirrorbox has joined #openstack-nova06:24
*** sudipto has joined #openstack-nova06:31
*** claudiub has joined #openstack-nova06:33
*** tobe has joined #openstack-nova06:33
*** jmccrory has quit IRC06:33
moshelejohnthetubaguy: hi06:36
*** otter768 has joined #openstack-nova06:38
*** scheuran has joined #openstack-nova06:39
*** mkoderer_cloud has joined #openstack-nova06:39
*** ian_brown has quit IRC06:40
*** jmccrory has joined #openstack-nova06:42
*** otter768 has quit IRC06:43
*** ssurana has joined #openstack-nova06:43
*** markvoelker has joined #openstack-nova06:43
*** achanda has joined #openstack-nova06:44
*** nihilifer has joined #openstack-nova06:45
*** markvoelker has quit IRC06:49
*** sahid has joined #openstack-nova06:52
*** belmoreira has joined #openstack-nova06:53
*** exploreshaifali has joined #openstack-nova06:53
*** salv-orlando has joined #openstack-nova06:54
*** jmccrory has quit IRC06:55
*** salv-orlando has quit IRC06:56
*** rotbeard has joined #openstack-nova06:56
*** salv-orlando has joined #openstack-nova06:56
*** aix has joined #openstack-nova06:56
*** markus_z has joined #openstack-nova06:57
*** oro has joined #openstack-nova06:57
*** arnaud____ has quit IRC06:57
*** achanda has quit IRC06:58
*** bradjones has quit IRC07:02
*** lennyb has quit IRC07:03
*** bradjones has joined #openstack-nova07:04
*** bradjones has quit IRC07:04
*** bradjones has joined #openstack-nova07:04
*** rotbeard has quit IRC07:06
*** root_ has joined #openstack-nova07:07
*** achanda has joined #openstack-nova07:08
*** root_ has quit IRC07:09
rgerganovalex_xu_: you around?07:09
*** lennyb has joined #openstack-nova07:12
*** jmccrory has joined #openstack-nova07:18
*** bfic has joined #openstack-nova07:19
*** lan has quit IRC07:20
rgerganovgilliard: morning, you around?07:20
*** lan has joined #openstack-nova07:21
*** romainh has joined #openstack-nova07:21
*** welldannit has joined #openstack-nova07:22
*** unicell has joined #openstack-nova07:22
*** welldannit has quit IRC07:22
*** unicell1 has quit IRC07:23
*** lsmola has joined #openstack-nova07:27
openstackgerritAbhijeet Malawade proposed openstack/nova: Test patch libvirt race condition (do not merge)  https://review.openstack.org/19173207:28
*** unicell has quit IRC07:29
*** jmccrory has quit IRC07:29
*** isd has quit IRC07:32
*** jmccrory has joined #openstack-nova07:32
*** doude has joined #openstack-nova07:33
*** hshiina has joined #openstack-nova07:38
*** mpaolino has joined #openstack-nova07:38
*** mpaolino has quit IRC07:39
*** ssurana has quit IRC07:40
*** lpetrut has joined #openstack-nova07:41
*** sorrison_laptop has quit IRC07:41
*** oro has quit IRC07:44
*** dsanders has quit IRC07:45
*** tobe has quit IRC07:48
*** mpavone has joined #openstack-nova07:53
*** sorrison_laptop has joined #openstack-nova07:54
*** rajesht has quit IRC07:57
*** atuvenie has joined #openstack-nova08:02
*** lucasagomes has joined #openstack-nova08:03
*** alex_klimov has joined #openstack-nova08:03
*** husanu has quit IRC08:03
*** tobe has joined #openstack-nova08:04
*** dsanders has joined #openstack-nova08:04
*** mpavone has quit IRC08:05
*** deepthi has quit IRC08:05
*** husanu has joined #openstack-nova08:05
*** ndipanov has joined #openstack-nova08:06
*** Nikolay_St has joined #openstack-nova08:13
*** oro has joined #openstack-nova08:15
*** exploreshaifali has quit IRC08:16
*** coolsvap is now known as coolsvap|away08:16
*** mpavone has joined #openstack-nova08:17
moshelejohnthetubaguy: hi08:18
*** coolsvap|away is now known as coolsvap08:19
*** karimb has joined #openstack-nova08:25
*** dsanders has quit IRC08:28
*** kiwik-chenrui has quit IRC08:28
*** RuiChen has joined #openstack-nova08:29
openstackgerritgaryk proposed openstack/nova: Cells: add instance cell registration utility to nova-manage  https://review.openstack.org/19388908:30
*** sorrison_laptop has quit IRC08:30
*** markvoelker has joined #openstack-nova08:32
*** Jeffrey4l has quit IRC08:33
*** gszasz has joined #openstack-nova08:35
*** yassine_ has joined #openstack-nova08:36
*** markvoelker has quit IRC08:37
*** haypo has joined #openstack-nova08:38
*** otter768 has joined #openstack-nova08:39
*** danpb has joined #openstack-nova08:42
*** otter768 has quit IRC08:44
openstackgerritgaryk proposed openstack/nova: VMware: add in vnic resource limitations  https://review.openstack.org/18020208:45
*** Jeffrey4l has joined #openstack-nova08:45
*** jcoufal has joined #openstack-nova08:46
garykdanpb:  can you please look at https://review.openstack.org/192675 it has one +2 and it will help unblock the BP. sorry to keep on bugging about this08:46
*** BobBall_AWOL has quit IRC08:47
openstackgerritgaryk proposed openstack/nova: VMware: support limits via image meta data  https://review.openstack.org/18094208:47
*** doude has quit IRC08:49
*** doude has joined #openstack-nova08:49
openstackgerritgaryk proposed openstack/nova: VMware: Resource limits for memory  https://review.openstack.org/17906008:49
*** derekh has joined #openstack-nova08:51
*** deepthi has joined #openstack-nova08:52
*** mpavone_ has joined #openstack-nova08:53
*** salv-orl_ has joined #openstack-nova08:53
*** mpavone has quit IRC08:54
*** BobBall has joined #openstack-nova08:54
openstackgerritgaryk proposed openstack/nova: VMware: add in vnic resource limitations  https://review.openstack.org/18020208:56
openstackgerritgaryk proposed openstack/nova: VMware: support limits via image meta data  https://review.openstack.org/18094208:56
openstackgerritgaryk proposed openstack/nova: VMware: store extra_specs object  https://review.openstack.org/17906108:56
openstackgerritgaryk proposed openstack/nova: VMware: add resource limits for disk  https://review.openstack.org/17906208:56
*** salv-orlando has quit IRC08:57
*** salv-orl_ has quit IRC08:58
*** huats has joined #openstack-nova09:00
*** masayukig_ has quit IRC09:00
*** oro has quit IRC09:01
*** oro has joined #openstack-nova09:02
*** deepthi has quit IRC09:02
*** masayukig_ has joined #openstack-nova09:04
openstackgerritRui Chen proposed openstack/nova-specs: Split network plane for live migration  https://review.openstack.org/19499009:06
kashyapIs it recommended to cleanup the existing .tox directory from the Nova git repo before running unit tests?09:08
johnthetubaguyjhesketh: I am about now, but I guess you may have gone?09:09
jheskethjohnthetubaguy: nope09:09
jheskethjohnthetubaguy: pm okay?09:09
johnthetubaguyjhesketh: ah, good good, how can I help?09:09
*** masayukig_ has quit IRC09:09
* kashyap does it anyhow, for good measure09:10
johnthetubaguykashyap: should be needed09:10
johnthetubaguyjhesketh: sure09:10
kashyapjohnthetubaguy: "should be needed" meaning cleaning up .tox before running unit tests is recommended?09:11
haypokashyap: FYI since pip 7, pip creates wheel packages on demand, so recreating a tox test env is much faster!09:11
*** tojuvone has quit IRC09:12
haypokashyap: sometimes i use "rm -rf .tox" when i modify code in .tox/py{27,34}/lib/python{2.7,3.4}/site-packages/ for debug purpose. or when dependencies change09:12
kashyaphaypo: I see.09:12
haypokashyap: in the common case, there is no good reason to create tox09:12
haypokashyap: (oh, and there is also "tox -r" option, maybe better than rm -rf .tox. last week i typed "rm -rf .git" by mistake, i lost one day of work...)09:13
*** oro_ has joined #openstack-nova09:13
kashyaphaypo: Thank you. I also came across this one - https://www.berrange.com/posts/2014/11/14/faster-rebuilds-for-python-virtualenv-trees/09:13
*** masayukig_ has joined #openstack-nova09:14
openstackgerritAbhijeet Malawade proposed openstack/nova: Test patch libvirt race condition (do not merge)  https://review.openstack.org/19173209:14
kashyaphaypo: Ouch. "rm -rf .git", I thought `rm -rf` doesn't is not allowed any more by coreutils :-)09:15
haypokashyap: yeah, before i also had a setup using devpi-server. but it required to compile explicitly wheel packages. pip 7 is better because it does that on demand, and it doesn't need a local server09:15
*** deepthi has joined #openstack-nova09:18
kashyaphaypo: I have this python-pip-1.5.6-3.fc21.noarch - not sure if it's "pip 7" or not. But, don't bother, I'll look up.09:19
openstackgerritgaryk proposed openstack/nova: VMware: add in vnic resource limitations  https://review.openstack.org/18020209:19
openstackgerritgaryk proposed openstack/nova: VMware: add support for cores per socket  https://review.openstack.org/18794209:19
openstackgerritgaryk proposed openstack/nova: VMware: support limits via image meta data  https://review.openstack.org/18094209:19
openstackgerritgaryk proposed openstack/nova: VMware: create common object for limits, reservations and shares  https://review.openstack.org/17905909:19
openstackgerritgaryk proposed openstack/nova: VMware: Resource limits for memory  https://review.openstack.org/17906009:19
openstackgerritgaryk proposed openstack/nova: VMware: store extra_specs object  https://review.openstack.org/17906109:19
openstackgerritgaryk proposed openstack/nova: VMware: add resource limits for disk  https://review.openstack.org/17906209:19
johnthetubaguykashyap: I missed a crucial bit09:20
johnthetubaguykashyap: shouldn't be needed, but it can help get you unstuck if things go wrong09:20
haypokashyap: pip 1.5 is pip 1.5, but not pip 7 :)09:20
kashyapjohnthetubaguy: Yep, got it. Thanks. Was testing a upstream Kilo backport and trying to resolve some conflicts.09:21
kashyaphaypo: Felt F21 shouldn't be _that_ far off and thought it's some kind of Fedora's own versioning mechansim for this.09:23
haypokashyap: i'm using fedora 22 and i upgraded pip manually (pip install -U pip)09:24
haypo(which introduces new funny issues)09:24
*** rajesht has joined #openstack-nova09:25
moshelesahid: hi09:28
*** lpetrut has quit IRC09:28
tdurakovdanpb: hi Dan, i left question in comment for your patch: https://review.openstack.org/#/c/183331/7 Could you answer it?09:29
*** masayukig_ has quit IRC09:33
*** oro has quit IRC09:33
*** oro_ has quit IRC09:34
openstackgerritMoshe Levi proposed openstack/nova: libvirt: mlnx_direct vif type removal  https://review.openstack.org/18167109:35
*** masayukig_ has joined #openstack-nova09:36
sahidmoshele: hello09:36
shihanzhangwho can help review this patch #https://review.openstack.org/#/c/175742/09:38
moshelesahid: what tests related   ibvirt/test_vif.py did I missed in https://review.openstack.org/181671 ?09:38
moshelesahid: I removed the constant09:39
danpbtdurakov: i'm not the dan you are looking for09:39
tdurakovdanpb, oh, sorry... my fault)09:40
sahidmoshele: yep seems good09:40
moshelesahid: thanks09:40
danpbgaryk: i added comments to it yesterday which you didn't respond to yet09:40
tdurakovdansmith, hi Dan, i left question in comment for your patch: https://review.openstack.org/#/c/183331/7 Could you answer it?09:41
johnthetubaguykashyap: ah, so switching to stable branches, I often switch out that directory, not sure if thats the best approach09:41
garykdanpb: i did not see. i will check now.09:41
danpbi didn't -1 it because it wasn't really a critisim - i just wanted some clarification on interpretation09:42
moshelejohnthetubaguy: can this https://review.openstack.org/#/c/179577/ be approved?09:42
*** masayukig_ has quit IRC09:43
bauzasjohnthetubaguy: hey, wondering if https://review.openstack.org/#/c/191914/ is not opening a can of worms for people wanting to get rid of the scheduler...09:43
kashyapjohnthetubaguy: I just clunkily do a checkout foo/bar branch.09:43
bauzasjohnthetubaguy: I honestly think we could have alternative drivers (that's fair to consider it) but I dislike about any big-bang09:43
garykdanpb: the limits are per disk/vif. i answered on gerrit09:44
danpbok, cool09:44
danpbthough unfortunately it means i'll also for a naming change, because those semantics do match libvirt in some cases09:45
*** alexpilotti has joined #openstack-nova09:45
*** masayukig_ has joined #openstack-nova09:45
danpbs/also/ask/09:45
*** sudipto has quit IRC09:45
garykok, which ones? i have not found any that are a 1:1 match09:46
danpbi believe disk_io_limit matches   disk_total_iops_sec09:46
danpbthey both specify a limit on total io operations per sec IIUC09:46
danpbi have to  double check the libvirt VIF  bandwidth semantics - it is possible one of the libvirt settings matches  vif_limit too - just not quite sure yet09:47
garykyes, the disk io limit is one and the same.09:47
garykit would add a few extra ifs to the code to parse that and not be able to use a generic parse for the names, but that is ok09:48
danpbbefore you change the spec, just give me a chance to double check the vif stuff too09:48
garykok09:48
*** oro_ has joined #openstack-nova09:49
garyki think that maybe it would be easier for an admin to maybe have the same contructs for the vmware limits for disks, cpu, memory and vifs. that could make their life easier and less error prone09:49
garykhaving it as the same for all except the disk io limits could be confusing09:49
garykcan you please consider that09:50
garyki understand that we would like generic and shared constructs but maybe here we should try and be consistent09:50
*** salv-orlando has joined #openstack-nova09:51
*** oro has joined #openstack-nova09:52
*** park_heijlong has quit IRC09:53
hayporunning nova tests (tox -e py27) fails in the subunit-trace command with an AttributeError in find_test_run_time_diff(): http://paste.openstack.org/show/320860/ -- does anyone know this bug?09:53
bauzashaypo: looking09:53
garykdanpb: i am also thinking about and admin creating a heat template where they could define the flavors. it would require some tweaking on their behalf for one item that does not fir the model09:53
bauzashaypo: ah, did you upgraded your venv ?09:53
bauzashaypo: nvm, I can see it created09:54
haypobauzas: it's a fresh test env, see the second line: "py27 create: /home/haypo/prog/openstack/nova/.tox/py27"09:54
bauzashaypo: yup, see my last comment (nvm)09:55
*** salv-orl_ has joined #openstack-nova09:56
*** lpetrut has joined #openstack-nova09:57
bauzashaypo: pip freeze | grep os_testr ?09:57
bauzashaypo: see https://github.com/openstack/os-testr/commit/de604832c3ece2aaeb56cc7a427dcb1dd833c0e1 which is not released yet09:58
*** salv-orlando has quit IRC09:58
haypobauzas: it's os-testr 0.1.0 (you can also see in the paste ;-)). there is no more recent version09:58
haypobauzas: ah!09:59
*** salv-orl_ has quit IRC09:59
*** salv-orlando has joined #openstack-nova09:59
bauzashaypo: yeah I was lazy about the version checking :p09:59
haypobauzas: np10:00
haypobauzas: i will install the development version10:00
bauzas haypo: or give a try about that https://github.com/openstack/os-testr/blob/master/os_testr/subunit_trace.py#L34-L3610:01
*** salv-orl_ has joined #openstack-nova10:01
*** salv-orlando has quit IRC10:03
*** edmondsw has joined #openstack-nova10:06
*** edmondsw has quit IRC10:06
*** edmondsw has joined #openstack-nova10:07
openstackgerritgaryk proposed openstack/nova: VMware: add in vnic resource limitations  https://review.openstack.org/18020210:07
openstackgerritgaryk proposed openstack/nova: VMware: add support for cores per socket  https://review.openstack.org/18794210:07
openstackgerritgaryk proposed openstack/nova: VMware: support limits via image meta data  https://review.openstack.org/18094210:07
openstackgerritgaryk proposed openstack/nova: VMware: create common object for limits, reservations and shares  https://review.openstack.org/17905910:07
openstackgerritgaryk proposed openstack/nova: VMware: Resource limits for memory  https://review.openstack.org/17906010:07
openstackgerritgaryk proposed openstack/nova: VMware: store extra_specs object  https://review.openstack.org/17906110:07
openstackgerritgaryk proposed openstack/nova: VMware: add resource limits for disk  https://review.openstack.org/17906210:07
openstackgerritVictor Stinner proposed openstack/nova: Fix Python 3 issues in nova.db.sqlalchemy  https://review.openstack.org/19519110:09
*** karimb has quit IRC10:13
*** karimb has joined #openstack-nova10:13
*** Kennan2 has joined #openstack-nova10:14
*** dheeraj-gupta-4 has joined #openstack-nova10:14
*** Kennan has quit IRC10:14
*** dims has joined #openstack-nova10:14
hayposdague: FYI https://review.openstack.org/195191 is now ready to be reviewed10:17
*** Kennan2 is now known as Kennan10:17
*** husanu3 has joined #openstack-nova10:18
openstackgerritAbhijeet Malawade proposed openstack/nova: Test patch libvirt race condition (do not merge)  https://review.openstack.org/19173210:19
*** exploreshaifali has joined #openstack-nova10:20
*** markvoelker has joined #openstack-nova10:21
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py  https://review.openstack.org/18778510:24
*** husanu3 has quit IRC10:25
*** markvoelker has quit IRC10:25
*** husanu3 has joined #openstack-nova10:26
*** sudipto has joined #openstack-nova10:27
*** rajesht_ has joined #openstack-nova10:28
*** rajesht has quit IRC10:28
*** husanu3 has quit IRC10:28
*** husanu1 has joined #openstack-nova10:29
*** husanu1 has quit IRC10:31
*** jogo has quit IRC10:31
*** jasondot_ has joined #openstack-nova10:32
*** jogo has joined #openstack-nova10:35
*** karimb has quit IRC10:36
*** jasondot_ is now known as jasondotstar10:37
*** Nikolay_St has quit IRC10:38
*** karimb has joined #openstack-nova10:39
*** otter768 has joined #openstack-nova10:40
openstackgerritPetrut Lucian proposed openstack/nova: Fixes delayed instance lifecycle events issue  https://review.openstack.org/19518010:40
openstackgerritAnkit Agrawal proposed openstack/nova: Test patch libvirt race condition (do not merge)  https://review.openstack.org/19173210:41
*** deepthi has quit IRC10:42
dimshaypo: replied in the review. no worries.10:42
*** otter768 has quit IRC10:44
BobBalljohnthetubaguy / others: What's the expected behaviour of consoles with shut off servers?  There is a tempest test that expects to be able to get the console, however XenAPI's implementation was written to explicitely remove consoles for VMs that were no longer running and so there is a race condition there...10:47
johnthetubaguyBobBall: I don't know10:48
johnthetubaguyBobBall: never really dug deep in console land10:48
johnthetubaguyBobBall: whats the race you are seeing?10:48
BobBallI thought you wrote the script that deletes the logs? :P10:48
BobBallJust that if the cronjob happens after the server has been shutoff but before the tempest test reads it then there is no log for it to read and it fails :)10:49
johnthetubaguyBobBall: oh you mean the console log? I thought you meant the VNC console. Thats never been tested or deployed, AFAIK10:49
BobBallIt's tested by tempest10:50
johnthetubaguyBobBall: yeah, that sounds broken10:50
johnthetubaguyBobBall: it should be when the server is deleted that its removed10:50
*** Marga_ has quit IRC10:50
*** sdake has quit IRC10:50
BobBallSo the tempest test is bad - and probably libvirt driver either deletes after a race or incorrectly leaves them hanging about?10:50
johnthetubaguyBobBall: but the problem is around the dom ids and reboots and information leakage10:51
johnthetubaguyBobBall: tempest test sounds good, honestly10:51
*** pixelb has joined #openstack-nova10:51
BobBallThe tempest test checks if we can read the logs after the server has disappeared10:51
*** abhishekk has joined #openstack-nova10:52
johnthetubaguyBobBall: yep, seems like a good thing to check10:52
haypodims: my https://review.openstack.org/195191 change works with the current oslo.db version10:52
johnthetubaguyBobBall: well, is it just stopped, or deleted?10:53
BobBallSHUTOFF10:53
dimshaypo: nice10:53
BobBallXenAPI removes all logs for guests that are not running10:53
johnthetubaguyBobBall: right, you should be able to read it when the VM is shutoff, that seems reasonable10:53
haypodims: i will write a new patch when a new version of oslo.db will be release, to replace remaining obj.iteritems()10:53
dimshaypo: ++10:53
johnthetubaguyBobBall: right, because of the issue I mentioned above, its a bug10:53
BobBallRight - so your view is XenAPI bug.10:53
johnthetubaguyBobBall: yes10:54
BobBallgood10:54
dimshaypo: should be early next week, though g-r update will take time10:54
haypodims: i prefer to replace map() with list-comprehension because it will be less motivated to rework the code later if the code is already compatible with python 310:54
haypodims: it's harder to argue for refactoring patches if they don't fix a test10:54
johnthetubaguyBobBall: I think you are the first person to try and test that code, FWIW10:54
dimshaypo: sure, if yours lands first, i'll rebase10:54
johnthetubaguyBobBall: other than me just trying out the API with the CLI10:54
dimshaypo: in the end, we need both patches as they target different tests10:55
BobBall\o/ (I did already post a replacement script BTW - https://review.openstack.org/#/c/193647/ - to fix some races)10:55
johnthetubaguyBobBall: cool10:55
openstackgerritgaryk proposed openstack/nova: libvirt: add in missing doc string for hypervisor_version  https://review.openstack.org/19551810:56
johnthetubaguyBobBall: is that just me, or does that not work on older version of XenServer because it requires xl?10:56
BobBallxl has always been there I think10:56
johnthetubaguyBobBall: I thought that was the new xen toolstack to replace xe10:56
johnthetubaguy(xen light?)10:57
BobBallto replace xm.  xe is XAPI's CLI10:57
johnthetubaguyBobBall: ah, yeah, thats right10:57
johnthetubaguyBobBall: added in Xen 4.1 so should be Ok for most things: http://wiki.xen.org/wiki/XL10:58
garykdanpb: not sure if you saw the comments that i wrote above about the names being consistent.10:58
BobBallWe've been 4.1+ since XS 6.010:58
*** IanGovett has joined #openstack-nova10:59
johnthetubaguyBobBall: right, but we should add that check in the code somewhere, to exclude 5.610:59
BobBallWas list_domains even in 5.6?10:59
johnthetubaguyBobBall: no idea10:59
ildikovdanpb: johnthetubaguy: could you please take a look on this patch: https://review.openstack.org/#/c/192755/ ?11:00
ildikovdanpb: johnthetubaguy: I'm not sure whether the Liberty-1 deadline is applicable for this patch too, I would like to ask for some help to clarify11:02
*** dims has quit IRC11:02
johnthetubaguyildikov: the current deadline applies, its not a priority item, what the current deadline means is open to debate.11:03
ildikovjohnthetubaguy: this patch is an extension to an already merged blueprint to cover the API impacts which was unfortunately left out from the original one11:04
ildikovjohnthetubaguy: the corresponding code is already up for review and there is a -1 on it that highlighted this issue with the original spec11:05
*** dims has joined #openstack-nova11:05
johnthetubaguyildikov: ah, in which case, its probably not covered by the freeze, or at least its a great exception candidate11:05
*** moshele has quit IRC11:05
ildikovjohnthetubaguy: if the deadline has to be applied to this extension too, will there be a freeze exception process?11:06
johnthetubaguyildikov: the exception process is documented on a wiki page I send to the ML a few times, here is the link: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule11:06
ildikovjohnthetubaguy: so I just pinged you to avoid the issue that we cannot merge the last patch for the feature because of this extension patch is not merged11:07
*** LukeHinds has joined #openstack-nova11:07
johnthetubaguyildikov: agreed, right now I have lots of specs that need urgent attention, so its just not something I can jump on right now11:07
ildikovjohnthetubaguy: sorry I'm a bit behind with mails :(11:08
johnthetubaguyildikov: me too11:08
* johnthetubaguy runs away for lunch11:08
*** moshele has joined #openstack-nova11:09
ildikovjohnthetubaguy: if the freeze exception is the best way to follow with this, then I will do that or if we can agree on the patch has to be merged before the corresponding code but not until today, that's fine with me too11:09
*** moshele has quit IRC11:10
*** dims_ has joined #openstack-nova11:10
ildikovjohnthetubaguy: enjoy :)11:11
*** dims has quit IRC11:11
*** tobe has quit IRC11:11
*** moshele has joined #openstack-nova11:11
*** abhishekk has quit IRC11:13
*** ZZelle-away has quit IRC11:14
*** salv-orl_ has quit IRC11:14
*** ZZelle-away has joined #openstack-nova11:15
*** kexiaodong1 has joined #openstack-nova11:15
*** moshele has quit IRC11:15
*** dims_ has quit IRC11:16
*** moshele has joined #openstack-nova11:16
*** david-lyle has quit IRC11:17
*** dheeraj-gupta-4 has quit IRC11:20
*** dims has joined #openstack-nova11:21
*** david-lyle has joined #openstack-nova11:21
*** lucasagomes has quit IRC11:22
*** achanda has quit IRC11:25
*** oro_ has quit IRC11:28
*** oro has quit IRC11:28
*** Marga_ has joined #openstack-nova11:32
*** ociuhandu has joined #openstack-nova11:33
*** ericksonsantos has joined #openstack-nova11:33
*** lucasagomes has joined #openstack-nova11:34
*** tellesnobrega has joined #openstack-nova11:36
*** markvoelker has joined #openstack-nova11:37
*** samueldmq has joined #openstack-nova11:37
*** bradjones has quit IRC11:38
*** EmilienM|off is now known as EmilienM11:38
*** ociuhandu_ has joined #openstack-nova11:39
*** ociuhandu has quit IRC11:40
*** ociuhandu_ is now known as ociuhandu11:40
sdaguedanpb: is there an overall sketch (bp, spec, ml post, readme) on the big picture of what's going on with - https://review.openstack.org/#/c/192627 and friends? Just so I can have a bit more context on where this is going to feel comfortable reviewing them.11:40
*** Nikolay_St has joined #openstack-nova11:40
*** bradjones has joined #openstack-nova11:41
*** bradjones has quit IRC11:41
*** bradjones has joined #openstack-nova11:41
*** oro_ has joined #openstack-nova11:41
*** oro has joined #openstack-nova11:41
*** markvoelker has quit IRC11:41
*** lucasagomes is now known as lucas-hungry11:43
*** mwagner_afk has quit IRC11:46
*** Marga_ has quit IRC11:48
*** salv-orlando has joined #openstack-nova11:49
*** VW_ has joined #openstack-nova11:50
*** jasondotstar has quit IRC11:50
*** _diana_ has joined #openstack-nova11:53
openstackgerritKarim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0  https://review.openstack.org/19026711:55
*** apuimedo has joined #openstack-nova11:55
*** raildo has joined #openstack-nova11:56
*** markvoelker has joined #openstack-nova11:58
*** zhenguo has quit IRC12:04
*** sudipto has quit IRC12:06
openstackgerritClaudiu Belu proposed openstack/nova-specs: Adds Hyper-V vTPM devices spec  https://review.openstack.org/19506812:07
*** VW_ has quit IRC12:08
dimsjohnthetubaguy: please unblock -2 on https://review.openstack.org/#/c/180695/ as the spec (https://review.openstack.org/#/c/105404/) has merged12:10
johnthetubaguydims: spooky, I am just updating the blueprint now so I can remove that12:10
dimshaha :)12:10
dimsthanks12:10
johnthetubaguydims: FWIW, I am not sure that needed a spec, but it has one now, so thats cool12:11
*** htruta has joined #openstack-nova12:12
*** rfolco has joined #openstack-nova12:17
dimsjohnthetubaguy: y, spec was already files, so i just adopted it12:22
dimss/files/filed/12:22
johnthetubaguydims: cool12:22
*** aysyanne has joined #openstack-nova12:22
danpbsdague: there's probably not any single doc describing it i'm afraid but i can explain quickly12:23
danpbsdague: the high level goal was to objectify our use of image metadata properties12:24
danpbsdague: so for that we've merged this new ImageMeta & ImageMetaProps object https://github.com/openstack/nova/blob/master/nova/objects/image_meta.py12:24
danpbsdague: the patches that i have up in that series you point to https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:virtimageprops-17,n,z12:24
danpbsdague: are now doing the work to convert the codebase over to use the object instead of dict12:25
danpbsdague: i've first converted the nova/virt/hardware.py shared code, then i convert each virt driver and finally convert the compute manager12:25
danpbsdague: the first patch in the series is dealing with a bug i uncovered whereby we're storing truncated data in some fields of system_metadata12:26
danpbsdague: because it overflows the 255 byte limit  in the column12:26
*** coolsvap is now known as coolsvap|away12:26
danpbmost of the patches are a fairly straightforward & tedious conversion from dict access to object access12:26
sdagueyep, the patches all looked straight forward, I was just trying to get the big picture, because it was kind of looking through a soda straw12:27
danpbthe first patch probably deserves close attention from several nova-cores, since there might be aspects that I'm not aware of https://review.openstack.org/#/c/192627/12:27
sdagueyeh12:28
*** busterswt has joined #openstack-nova12:29
*** dprince has joined #openstack-nova12:29
*** mwagner_afk has joined #openstack-nova12:30
sdagueso... I left the comment in the libosinfo review (as a zero score), but it still seems weird to me to have python gobject -> c lib to get access to a bunch of xml data, which is what we want. I get that in C accessing XML is hard. But in python, it's built in to stdlib. Is there no way to just access the XML directly ourselves?12:30
*** dims has quit IRC12:31
*** dims has joined #openstack-nova12:31
*** bkopilov has quit IRC12:32
*** exploreshaifali has quit IRC12:32
*** VW_ has joined #openstack-nova12:33
johnthetubaguysdague: danpb: I guess I see the data here: https://git.fedorahosted.org/cgit/libosinfo.git/tree/data but maybe its only released as a C lib, is that the issue here?12:34
sdaguejohnthetubaguy: yeh, and distros (at least debian) don't consider this like tzdata, which is something they have to keep reving12:35
*** coolsvap|away is now known as coolsvap12:35
sdaguehttps://packages.debian.org/search?keywords=libosinfo&searchon=names&suite=all&section=all12:35
sdaguebecause, effectively, this is kind of like tzdata, something that you always want the latest version of12:35
*** david-ly_ has joined #openstack-nova12:35
*** _diana_ has quit IRC12:35
*** coolsvap is now known as coolsvap|away12:36
danpbsdague: the libosinfo library is not simply about providing a way to read the XML files12:37
danpbsdague: it is providing an higher level API to make logic decisions based on metadata in the XML12:38
danpbthe fact that it uses XML underneath is really considered a private impl detail that apps should not care / know about12:38
danpbso from that POV, it would not be appropriate for Nova to just read the XML directly12:38
*** david-lyle has quit IRC12:39
*** Redcavalier has joined #openstack-nova12:39
openstackgerritPuneet proposed openstack/nova-specs: Implement Block Volume Quota per Instance  https://review.openstack.org/19555512:40
johnthetubaguydanpb: ah, so we do get some business logic that we use too, I overlooked that12:40
*** dane-fichter has joined #openstack-nova12:40
*** otter768 has joined #openstack-nova12:41
*** ivasev has joined #openstack-nova12:41
danpbin the future the data may also come from sources which are not local XML files too12:41
danpbso we really don't want apps to be touching the XML directly at all12:42
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Kick tires for oslo.reports  https://review.openstack.org/19555812:43
sdaguedanpb: the business logic that is being consumed seems pretty minimal. I do get that this is the way libosinfo has chosen to structure itself, it's just pretty unfortunate, because the patch level of your host OS will now dictate which guest OSes you can support.12:46
*** otter768 has quit IRC12:46
*** jasondotstar has joined #openstack-nova12:46
openstackgerritSylvain Bauza proposed openstack/nova-specs: Remove priority for Servicegroup API control plane spec  https://review.openstack.org/19555912:46
bauzasjohnthetubaguy: ^ I uploaded a spec change for removing the priority12:46
bauzasjohnthetubaguy: since you were +2 on the spec, I preferred to provide a dependent change12:47
*** lucas-hungry is now known as lucasagomes12:47
*** bradjones has quit IRC12:47
johnthetubaguybauzas: yeah, I liked that being part of the priority list of things, as its an important refactor that impacts scheduler related things12:47
johnthetubaguybauzas: maybe I am missing something big here?12:48
bauzasjohnthetubaguy: well, I wanted to make sure that scheduler priority is about modifying the scheduler12:48
*** hshiina has quit IRC12:48
bauzasjohnthetubaguy: I know this change is really important hence my +112:48
*** lpetrut1 has joined #openstack-nova12:49
danpbsdague: two things - as an admin you can provide new data files to augment those provided as standard12:49
bauzasjohnthetubaguy: but I also think that leaving some specs saying "eh, sched prio" would just be a placeholder for having one more milestone time12:49
danpbsdague: and there is an intention to extend the library so it can fetch refreshed data from upstream and cache it locally12:49
bauzasjohnthetubaguy: so given that, I'd prefer to see the discussion about that coming in the subteam meeting12:49
bauzasjohnthetubaguy: so the subteam could accept it as a prio or not12:49
danpbthe ability to augment the database also allows organizations to provide information about any custom in-house operating systems they deal with12:50
*** bradjones has joined #openstack-nova12:50
*** bradjones has quit IRC12:50
*** bradjones has joined #openstack-nova12:50
*** ajayaa has joined #openstack-nova12:50
sdaguedanpb: library dynamically refreshing from upstream is pretty much a non starter, most of these environments are going to have firewalls to prevent that kind of thing12:50
*** lpetrut has quit IRC12:50
bauzasjohnthetubaguy: tbc, I don't want to have the scheduler priority being a proxy for getting more time for implementing12:50
*** husanu4 has joined #openstack-nova12:50
danpbsdague: it would not be something that was unconditionally enabled by default12:51
sdagueand it could be addressed if the libosinfo folks published their database as a release artifact with an standardized format, like tzdata.12:51
johnthetubaguybauzas: if the subteam isn't interested I am cool with that12:51
openstackgerritTimofey Durakov proposed openstack/nova: Split instance resize operation into 2 phases  https://review.openstack.org/19508812:51
openstackgerritTimofey Durakov proposed openstack/nova: Create class hierarchy for tasks in conductor  https://review.openstack.org/18449512:51
johnthetubaguybauzas: its certainly boarderline if it is accepted12:51
bauzasjohnthetubaguy: what I can be saying is that let's merge the main spec and discuss with the subteam if they +1 on my own change12:51
*** _diana_ has joined #openstack-nova12:52
*** husanu4 has quit IRC12:52
johnthetubaguybauzas: sounds good12:52
bauzasjohnthetubaguy: so it would be helping the proposer to have its spec accepted, and it would help the subteam reviewing that as a prio12:52
johnthetubaguybauzas: agreed12:52
bauzasjohnthetubaguy: if the subteam thinks it's a good prio, then I'll abandon my change12:52
danpbsdague: in the future we might split off the database and distribute it independently, but it still wouldn't be likely to be something we'd expect apps to directly parse & consume12:52
sdagueit just seems like fundamentally the wrong release model for an open data set to be hidden in an execution environment, and be told that's the only way you can access it.12:52
*** claudiub|2 has joined #openstack-nova12:53
johnthetubaguybauzas: it does affect the host.is_up call, well, sort of anyway12:53
bauzasjohnthetubaguy: I just -W'd my change, so it means we will discuss on that during next sched meeting12:54
johnthetubaguybauzas: but I am out of touch with where we are right now on all that, so I totally want to defer that to your subteam12:54
*** husanu1 has joined #openstack-nova12:54
bauzasjohnthetubaguy: what I want to make sure is that the subteam knows about the specs we're engaged to review and we're accepting12:54
*** tellesnobrega_ has joined #openstack-nova12:55
*** claudiub has quit IRC12:55
bauzasjohnthetubaguy: I want all of us to be committed to the success to the specs we agree as a prio12:55
*** vladikr has joined #openstack-nova12:55
johnthetubaguybauzas: agreed, I really want everyone reviewing specs, just like everyone is reviewing code, but that just aint happening right now, although you scheduler folks are actually doing a great job there12:55
johnthetubaguybauzas: we are violently agreeing with each other here12:56
bauzasjohnthetubaguy: hence why we need to not overcommit ourselves with reviewing lots of changes if we can't - or subteams would then be not that good12:56
*** husanu1 has quit IRC12:56
bauzasjohnthetubaguy: I want to have the subteam offloading the reviewing time by making you cores sure that we at least gave a vote12:56
*** sdake has joined #openstack-nova12:56
*** husanu3 has joined #openstack-nova12:56
johnthetubaguybauzas: you and me both12:57
*** mdrabe has joined #openstack-nova12:57
bauzascoolness, I'll speak about that next Tues then12:57
garykis the nova meeting now?12:57
sdaguegaryk: an hour12:57
garyksdague: ok, thanks12:57
johnthetubaguygaryk: there is a link to do the translation for you on the agenda: https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting12:58
johnthetubaguyat least its meant to!12:58
ndipanovI think it's in 1h12:59
*** coolsvap|away is now known as coolsvap13:00
*** atuvenie_ has joined #openstack-nova13:00
*** husanu3 has quit IRC13:00
claudiub|2danpb: Hi. If you have a bit of time, can you take a look at the hyper-v uefi secure boot spec? Your comments have been addressed: https://review.openstack.org/#/c/190997/13:01
*** beagles is now known as beagles_brb13:01
bauzaseh eh, 1 hour left for pinging people about specs \o/13:02
*** husanux0 has joined #openstack-nova13:03
* danpb goes to lunch for the next hour ;-)13:03
*** atuvenie has quit IRC13:04
*** sudipto has joined #openstack-nova13:04
*** rlrossit has joined #openstack-nova13:05
*** belmoreira has quit IRC13:05
garykjohnthetubaguy: i just commented on the limits spec - what you mention is neutron related and this is epcfically for nova - it also works with nova network.13:07
garykthe implementation is done by the VC and not neutron13:08
garykneutron has that stuff in discussion and design13:08
*** afazekas has joined #openstack-nova13:08
johnthetubaguygaryk: what happens if the port is from neutron, but you have that in your settings?13:08
bauzasjaypipes: do you know the difference between a student and me ?13:08
garykthe ESX implements this13:08
*** rfolco has quit IRC13:08
garykso it will always be driven by Nova13:08
bauzasjaypipes: the student doesn't ask to be noted13:08
*** husanux0 has quit IRC13:08
bauzas:)13:09
garykjohnthetubaguy: the is done by the ESX on the VNOC level and not the neutron port.13:09
johnthetubaguygaryk: but if you create the port in neutron, what happens to the nova setting?13:09
jaypipesbauzas: :)13:09
garykjohnthetubaguy: nothing - it just gets the VC DVS port group13:09
johnthetubaguygaryk: similar with cinder volumes really, I guess13:09
jaypipesbauzas: what did I do now?13:09
bauzasjaypipes: orly ? my bad, you know, French trains always have delays13:10
garykjohnthetubaguy: the neutron stuff is something othognal and not related at all to what this feature is trying to do.13:10
johnthetubaguygaryk: OK, so if you use neutron, rather than nova-network, those settings just don't mean anything I guess?13:10
*** jecarey has joined #openstack-nova13:10
garykjohnthetubaguy: at the moment they will be implemented by the VC.13:10
garykthe neutron stuff is in discussion and the API may land in L. then each vendor may see how they implement this.13:11
garykToday the DVS has network limitations which we may decide to leverage later on, but not now13:11
*** neiljerram has joined #openstack-nova13:11
garykjohnthetubaguy: today in most cases the rxtx factor is treated by nova and not neutron13:12
*** husanux0 has joined #openstack-nova13:13
garykneutron does not control anythin on the ESX - it just gets a virtual network13:13
*** husanux0 has quit IRC13:13
lpetrut1johnthetubaguy: Hi13:13
*** irenab has quit IRC13:14
*** cfriesen has joined #openstack-nova13:14
johnthetubaguygaryk: so the qos is applied by the neutron-agent I guess, in theory (or the vif driver), hmm13:14
*** kylek3h has joined #openstack-nova13:14
markus_zkashyap: A comeback to the problem you faces yesterday wrt https://review.openstack.org/#/c/188058/13:14
lpetrut1johnthetubaguy: Could you please take a look over this patch whenever you have time? https://review.openstack.org/#/c/195180/ It's the one adding an extra check in the manager event handling method13:14
markus_zkashyap: Is this still present?13:14
markus_z*faced13:14
garykjohnthetubaguy: there is no neutron agent with the VMware driver13:14
*** _diana_ has quit IRC13:14
*** husanux4 has joined #openstack-nova13:14
kashyapmarkus_z: 1 moment, in the middle of shitstorm trying to resolve `tox` deps in my local Nova environment :-(13:15
garykjohnthetubaguy: the qos is done by the VC interafces and enforced by the ESX13:15
*** ZZelle-away is now known as ZZelle13:15
markus_zkashyap: OK, good luck13:15
lxslikashyap: I find activating the venv and running this a couple of times can help:13:16
kashyapmarkus_z: I'll try and comment on the review. I need to step out for some late lunch too.13:16
garykjohnthetubaguy: it is kind of like asking why do we let libvirt implement all kinds of networking features if it can be done via neutron. answer is that libvirt controls those specific bits and the same is in this case13:16
lxslikashyap: for X in `cat requirements.txt test-requirements.txt | sed 's/#.*//;/^$/d'`; do echo $X; pip install $X; done13:16
markus_zkashyap: sure, thanks13:17
kashyaplxsli: That's it I guess. I need to get used to `pip` and ignore system deps13:17
lxslikashyap: yes I find it's much easier to use venvs for almost everything13:18
kashyaplxsli: Thank you, I was resisting so far to go the pip route, now that you mention it again, I'll just do it, and get some lunch13:18
*** husanux4 has quit IRC13:18
kashyaplxsli: Yeah, I began to use Doug Hellman's virtualenvwrapper lately13:18
*** lyarwood has quit IRC13:19
lxsliI should check that out13:19
*** lyarwood has joined #openstack-nova13:19
johnthetubaguygaryk: I wish we could decide what were doing there long term, it would make life easier. Frankly I think my brain is totally burnt out from all the spec reviews right now, I am certainly emotionally totally burnt out from them all.13:19
mdboothlxsli: Surely you can just activate the venv, then do pip install -r requirements.txt -r test-requirements.txt ?13:19
*** husanux8 has joined #openstack-nova13:20
lxslimdbooth: I was a bit frustrated when I wrote it, installing each thing individually means failures are handled separately13:20
mdboothAh, ok13:20
mdboothI wonder if pip can list requirements13:21
mdboothThen you could toposort it and just run it once13:21
*** husanux8 has quit IRC13:21
kashyaplxsli: Do you run that `pip` with sudo? Please tell me no.13:21
lxslikashyap: absolutely not13:21
*** exploreshaifali has joined #openstack-nova13:21
kashyapYou said, virtualenv, so I'll take it as no.13:21
kashyapYep13:21
*** coolsvap is now known as coolsvap|away13:21
garykjohnthetubaguy: i understand and respect the fact that you have invested a lot of time and effort here. the fact that i feel that everything we do in the driver is blocked - for whatever reasons is problematic. this is very frustrating as i see parallel drivers get their similar bits though. so i really feel crap about it and it is not encouraging.13:21
mdboothkashyap: Do you use devpi-server and a local wheel cache, btw?13:22
*** Marga_ has joined #openstack-nova13:22
garykjohnthetubaguy: in addition to that it is totally contained within the driver.13:22
kashyapmdbooth: No, I need to. I've heard the trick, just have to do it. Will do it post lunch13:22
kashyapmdbooth: Something like this? - https://www.berrange.com/posts/2014/11/14/faster-rebuilds-for-python-virtualenv-trees/13:22
garyki give up.13:22
mdboothkashyap: It's a job of minutes.13:22
mdboothkashyap: I do it every time I setup a new dev env.13:23
lxslimdbooth: Pip 7 automatically caches wheels right?13:23
*** aysyanne has quit IRC13:23
mdboothlxsli: Might do. I don't follow it closely, tbh.13:23
kashyapmdbooth: I see, thanks for reminder, I'll get my act together and prepare proper environments.13:24
*** lyarwood has quit IRC13:24
*** lyarwood has joined #openstack-nova13:24
*** ferest has joined #openstack-nova13:24
*** jdurgin has joined #openstack-nova13:24
*** _gryf is now known as _gryf|afk13:25
*** husanux2 has joined #openstack-nova13:25
neiljerramjohnthetubaguy: Sorry to hear that you feel burnt out by the specs situation.13:25
johnthetubaguygaryk: understood, and that sucks. I have attempted to get folks to do more reviews on specs, but its just not happened, and has left me totally burnt out right now, which certainly wasn't my intention, in fact I tried very hard to not let that happen, but thats just where I am right now13:25
johnthetubaguygaryk: sorry for that, just being honest, as thats my default mode of operation!13:25
mdboothkashyap: Yup, that's it, btw13:26
garykjohnthetubaguy: i appreciate the honesty and efforts. its just that things are screwed up and they will not change overnight13:26
ndipanovjohnthetubaguy, fwiw - I don't think it's fair to frame the problem as just people aren't reviewing enough13:26
ndipanovit's not like that13:26
jaypipesneiljerram: I think a lot of us are at this point :/13:26
johnthetubaguygaryk: agreed13:27
garykchange only comes if people want change. and it seems like nova does not want change so we are all screwed13:27
*** salv-orl_ has joined #openstack-nova13:27
ndipanovyou can't invent work and then be pissed when people don't happily show up to do it... but it's been discussed on the thread so no need to polute the channel now13:27
johnthetubaguyndipanov: so agreed there is way way more to it than that, it just feels like that from where I am sat, which is a dumb place13:27
*** apuimedo has quit IRC13:28
johnthetubaguyndipanov: trust me, I want less dumb work for everyone, and we certainly have less of it than previous releases, but yes, the ML if full of that debate13:28
jaypipeskashyap: sudo pip install sanity.13:28
*** ferest has quit IRC13:29
*** husanux2 has quit IRC13:29
* kashyap out for a walk, too much screen staring13:29
garykjohnthetubaguy: ndipanov: at the moment it is all about scatology13:29
ndipanovjohnthetubaguy, sure - if I didn't think we could get better I wouldn't have brought it up13:29
neiljerramjaypipes, johnthetubaguy I'm afraid I don't have enough experience to offer any solution, so can only express sympathy.13:30
garykpeople here want to work and put their efforts into working on things. the problem is that the 'coomunity' will not let us work13:30
jaypipesgaryk: come on, I don't think that's the case, Gary...13:30
garykso instead of delaing with development, but fixing, traiaging and working together. we are all nitpiking about crap that is strbgling the projects13:30
garykjaypipes: yes, it really is. seriously13:31
jaypipesgaryk: it may seem like that, but I assure you that we are slammed with reviews and are doing as many as humanly possible.13:31
*** Marga_ has quit IRC13:31
*** salv-orlando has quit IRC13:31
garykjaypipes: that i understand. the community is growing, the core group is shrining. the numbers speak for themselves.13:31
*** toure_ has joined #openstack-nova13:31
*** toure_ has quit IRC13:31
*** mriedem has joined #openstack-nova13:31
*** Marga_ has joined #openstack-nova13:31
garykis our qulity better - no.13:31
garykis our product more stable - no13:31
garykare our contributors happy - no13:32
jaypipesgaryk: I disagree with the first two.13:32
*** nkrinner has quit IRC13:32
ndipanovyeah I do too - a ton of good work has happened13:32
* mriedem lurks13:32
garykno one is saying that the work done is not good13:33
garykwhat i am saying is that we can be doing it more efficiently - but we are not13:33
johnthetubaguyndipanov: I am not against raising the issue, btw, we sure need some healthy debate and solutions13:33
dimssudipto: is this enough for your scenario? https://review.openstack.org/19557813:34
bauzascan we just take time to see how some new things are done by Liberty before saying contributors are unhappy ?13:34
*** liusheng has quit IRC13:35
*** eharney has quit IRC13:35
bauzasbecause we now have a Mentoring process, we have a backlog specs process, we have some subteams, etc.13:35
johnthetubaguygaryk: I am not disagreeing with that statement either, and we are actively trying out ideas as we speak right, although they are not obviously making massive changes right now13:35
*** ijuwang has joined #openstack-nova13:35
bauzaswhich means we by far have more time for helping people13:35
garykjohnthetubaguy: i am open to change and am happy that attempts are being made. that is progress.13:36
bauzasso, people, please give time for seeing if all of these efforts are good or not13:36
johnthetubaguybauzas: many contributors are unhappy, I am trying to engage with describing why we do what we do so they can help change things for the better, and thats all good13:36
bauzasjohnthetubaguy: we can't just make them happy by using a magic wand13:36
garykbauzas: what do you mean? cause my patches are not being reviewd i have more time to review? i do that all the time ....13:36
openstackgerritAnkit Agrawal proposed openstack/nova: Test patch libvirt race condition (do not merge)  https://review.openstack.org/19173213:36
johnthetubaguybauzas: right agreed with you there too, we don't know if the changes we have made will work out in the end, as they are not getting to the payoff part of whats been tried here13:37
bauzasgaryk: I mean that we know that we have some things to change, but some new programs kicked-off, so please leave some time before giving a status IMHO13:37
*** elmiko has joined #openstack-nova13:37
*** mmedvede has joined #openstack-nova13:37
garykbauzas: what is frustrating is the following:13:38
*** zzzeek has joined #openstack-nova13:39
bauzasgaryk: (waiting your comments)13:39
garykis that instead of having a feeling of being productive whatever one does feelis like they are stonewalled13:39
bauzasgaryk: okay, I think it's all about our mindsets13:39
garyklook at the scheduler for an example - it has been ove r 2years that we are talking about splitting this out13:40
bauzasgaryk: I know I have some features to push13:40
garykare we closer no. we are locking this more and more into nova13:40
bauzasgaryk: but for me, it's by far more important to help with Nova than pushing my own features even if I do want to13:40
sudiptodims, so whatever is obtained currently as a diff - the implementing subclasses could choose to filter it further - based on the black/whitelist?13:40
*** changbl_ has joined #openstack-nova13:40
garykare we able to develop things in drivers that are not libvirt - yeah, 1 out of 15 specs, if you are lucky13:40
bauzasgaryk: do you honestly know why we take 2 years for improving the scheduler ? because we sucked in the past13:41
garykbauzas: no, i disagree. we are the same group of people13:41
garykbauzas: cause now we are getting stuck in process13:41
*** Nikolay_St has quit IRC13:41
johnthetubaguyso the way I see it, we are moving forward and thats great13:42
bauzasgaryk: talking about the sched split again, I'm now not really wanting to having it split13:42
garykthe specs in some cases provide some brilliant clarify, but in others they are pointlesss, time consuming and a huge overhead13:42
bauzasgaryk: you know why ? because by the previous years, I understood all the problems we have and what we need to have13:42
bauzasgaryk: we need to have features, not technology13:42
claudiub|2I dunno, maybe the driver decomposition is needed? A lot of the driver specs impacts only X driver and can be implemented with just code in the XDriver. A good part of them is for parity as well. And what's worse, there are proposals for integrating new drivers into nova, like docker and Hyper, or others I'm not really aware of, which will only increase the congesstion even further. :)13:43
*** _diana_ has joined #openstack-nova13:43
bauzasgaryk: meaning that it's more important to serve usecases and find how to do that rather than just splitting for the god's sake13:43
johnthetubaguyso we need to start writing down *why* we have decided the current path13:43
garykclaudiub|2: yes, maybe the time has come to do a driver split. in neutron it worked fine and developers there are actually a lot happier13:43
johnthetubaguyI am trying to get to that point, but we are not there13:43
*** kudryashova has joined #openstack-nova13:44
johnthetubaguyonce we have that, the debate will be much healthier13:44
garykbauzas: what do you mean technology?13:44
johnthetubaguyhelping doing that would be very welcome13:44
bauzasgaryk: "I want the scheduler to be spit" is not an usecase IMHO13:44
bauzassplit eh13:44
* bauzas is like this kind of kids who spit13:44
garykbauzas: so how would you provide a feature like network proximity or volume proximity? without having a global picture of all resources13:44
bauzasgaryk: we need to have a sane discussion on how to provide those resources13:45
*** kexiaodong2 has joined #openstack-nova13:45
claudiub|2garyk: as the one that handled the Hyper-V neutron agent decomposition, I can say that it felt liberating, no longer having to wait days / weeks for reviews. :)13:45
garykbauzas: how can you if you only have a one side view of the compute resources and not anything else.13:45
bauzasgaryk: but do you want to know how many people came to our x-session meetings for discussing about cross-project resources ? crickets.13:45
garykbauzas: cause we chased them away.13:46
garykseriously13:46
*** dboik has joined #openstack-nova13:46
*** kexiaodong1 has quit IRC13:46
garyki do not want to mention names, but there were some serious scheuler players involved and we stonewalled them13:46
bauzasgaryk: so, yeah, I'm gonna help the usecase by refining the APIs, like we agreed 2 cycles before, and then we'll kick-off a discussion in Tokyo for seeing how we can get other projects' resources hopefully13:46
johnthetubaguyclaudiub|2: like I was saying there are a bunch of reasons why we are not doing the split out, and I think its something worth debating, but I really want to frame that conversation, and help doing that is super super welcome13:47
garykwe did not encourage and welcome them to the comminuty13:47
*** ctrath has joined #openstack-nova13:47
garykanyway, i am sorry for the crap. i am off to rebase some things before the meeting13:47
bauzasgaryk: sorry ? say it again ? we didn't welcomed people contributing to the scheduler ? shall I mention lxsli, edleafe or myself ?13:47
*** VW_ has quit IRC13:47
openstackgerritAndreas Scheuring proposed openstack/nova-specs: Spec to Add 'macvtap' as vif type to novas libvirt driver.  https://review.openstack.org/18228013:48
bauzasdidn't alex_xu_ being welcomed for his help with the API policy stuff ?13:48
garykbauzas: this was before you guys started to work on it13:48
*** thangp has joined #openstack-nova13:48
bauzasgaryk: I certainly understand we sucked in the past13:48
*** jcoufal has quit IRC13:49
sahidjohnthetubaguy: perhaps we can think about to have team lead for each drivers and somes lieutenants13:49
* sudipto working off dims changes right away13:49
sahidon libvirt we are happy to have severals core to reviews the code and make it merged13:49
*** VW_ has joined #openstack-nova13:49
sahidin xen it is the same i think13:49
johnthetubaguysahid: we proposed the subteam system at the summit, which is very similar to that, so they can ad hoc form and lead that area, and start to gain independence13:49
bauzashonestly, I'm going back to code as well, this whole convo makes me uncomfortable13:49
*** kexiaodong2 has quit IRC13:50
sahidbut for vmware and hyperv :/13:50
sahidjohnthetubaguy: yep13:50
johnthetubaguysahid: is it working as I would like yet, no, not really, I still think the idea is worth trying properly13:50
*** samueldmq has quit IRC13:51
*** tellesnobrega has quit IRC13:51
*** tellesnobrega_ has quit IRC13:51
mriedemgaryk: a few things to cleanup here https://review.openstack.org/#/c/165060/13:51
*** ericksonsantos has quit IRC13:51
*** raildo has quit IRC13:51
johnthetubaguysahid: for reference, you become a subteam on this etherpad: https://etherpad.openstack.org/p/liberty-nova-priorities-tracking13:51
*** htruta has quit IRC13:51
garykdanpb: i am not sure that ui understand your comment on the limits spec. Do this mean we need to abandon?13:52
garykit is really a parity feature13:52
*** baoli has joined #openstack-nova13:52
*** beagles_brb is now known as beagles13:53
garykdanpb: how does it differ from https://github.com/openstack/nova-specs/commit/3b32baa07a50536b31362e683d82c66bbe8aca88?13:53
sahidjohnthetubaguy: actually i set my own name on this, i wanted to refer people they use to contribute on libvirt13:53
garykdoes a tenant need to know that?13:53
garykmriedem: thanks13:54
sahidi'm not sure to know whether that helped to make things moving forward13:54
claudiub|2sahid: yeah, there is the concept of subteams / teamleads, as johnthetubaguy mentioned and also linked the etherpad, but we still have commits in that priority list that is weeks old.13:55
sahidi just say that we need to make code merged - split drivers or have "real" team will help13:56
*** baoli has quit IRC13:56
johnthetubaguyclaudiub|2: thats the bit were I say its not working yet, but people have not really all got used to the concept yet, and I suspect we need to refine that quite a bit13:56
garyksahid: that was done in neutron and the model works very well.13:56
*** baoli has joined #openstack-nova13:57
claudiub|2johnthetubaguy: IMO, there should be at least 1 core to each subteam, that reviews those commits. Otherwise, they won't get any attention13:57
*** earlephilhower has quit IRC13:57
openstackgerritSylvain Bauza proposed openstack/nova: Add RequestSpec methods for primitiving into dicts  https://review.openstack.org/18719713:58
openstackgerritSylvain Bauza proposed openstack/nova: Use RequestSpec object in HostManager  https://review.openstack.org/19125113:58
openstackgerritSylvain Bauza proposed openstack/nova: Use the same pci_requests field for all filters and HostManager  https://review.openstack.org/19125013:58
openstackgerritSylvain Bauza proposed openstack/nova: Remove unnecessary method in FilterScheduler  https://review.openstack.org/18848513:58
openstackgerritSylvain Bauza proposed openstack/nova: Consider that all scheduler calls are IO Ops  https://review.openstack.org/18848613:58
openstackgerritSylvain Bauza proposed openstack/nova: Create RequestSpec object  https://review.openstack.org/14552813:58
openstackgerritSylvain Bauza proposed openstack/nova: Use RequestSpec object in the FilterScheduler  https://review.openstack.org/18848713:58
*** xyang1 has joined #openstack-nova13:58
johnthetubaguyclaudiub|2: ideally two cores in each subteam, but yes, it would work better that way, but I don't want to restrict the subteams in a way where they must have a core, I would like cores to join subteams where possible, but thats not happening yet either13:58
*** exploreshaifali has quit IRC13:58
*** iamjarvo has joined #openstack-nova13:59
mriedemmeeting in 1 minute13:59
sahidjohnthetubaguy: right two core is needed ..ideally 3 !13:59
bauzasmriedem: hell, so refreshing ping13:59
mriedemoprah: you get a core! and you get a core! everyone gets a core!13:59
claudiub|2mriedem: lol13:59
*** artom has joined #openstack-nova13:59
claudiub|2sahid: there is also the concept that the "all subteam's +1 = +2"14:00
claudiub|2that will have to be implemented somehow.14:00
sahidclaudiub|2: is that make thing merged, so yes :)14:00
bauzasclaudiub|2: there is no discrete number of members within the subteam, it's open14:00
sahidbut if it is to have 8+1 waiting for a core to find the time to review it14:01
sahidi mean a last core14:01
bauzas(gosh, I promised myself to not be in the discussion)14:01
bauzasmeeting !14:01
johnthetubaguyclaudiub|2: we said at the summit, the idea is that the subteam recommendation needs to gain the trust of the core members who will treat it as a +2, although the plan was to discuss if we start doing that at the midcycle14:01
*** tellesnobrega has joined #openstack-nova14:01
*** eharney has joined #openstack-nova14:01
*** bkopilov has joined #openstack-nova14:02
*** marun has joined #openstack-nova14:06
*** mnestratov|2 is now known as mnestratov14:07
garykalaski: https://review.openstack.org/195518 - pagination and all14:07
*** bfic has quit IRC14:07
*** r-daneel has joined #openstack-nova14:09
*** neelashah has joined #openstack-nova14:10
*** READ10 has joined #openstack-nova14:10
*** ajayaa has quit IRC14:10
*** tellesnobrega has quit IRC14:11
*** tellesnobrega has joined #openstack-nova14:14
*** romainh has quit IRC14:15
*** romainh has joined #openstack-nova14:15
elmikohi all, i have a question about Eric Brown's comment on https://review.openstack.org/#/c/194290/14:16
elmikothis change was cherry-picked, should i have done something different with the patch to indicate this?14:16
elmikooh wait, just saw the new comment from Dave Walker....14:17
* elmiko feels sheepish14:17
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 1  https://review.openstack.org/15256914:18
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 2  https://review.openstack.org/16740814:18
*** Piet has quit IRC14:19
*** tellesnobrega_ has joined #openstack-nova14:21
mriedemelmiko: typically you should cherry pick to stable using 'git cherry-pick -x <git commit hash of source commit>'14:21
mriedemelmiko: so i agree with eriwb14:21
mriedem*ericwb14:22
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Change default compute api version  https://review.openstack.org/18465814:23
*** tellesnobrega has quit IRC14:24
*** ferest has joined #openstack-nova14:25
*** sudipto has quit IRC14:26
*** raildo has joined #openstack-nova14:28
*** yamahata has joined #openstack-nova14:30
*** ferest has quit IRC14:30
nagyzdo I see correctly that the ImagePropertiesFilter cannot filter on disk_format?14:32
bauzasalaski: uh, you probably missed the beginning of the nova meeting with such big email :p14:32
*** htruta has joined #openstack-nova14:33
alaskibauzas: heh, I was just finishing it up then14:33
*** eharney has quit IRC14:33
elmikomriedem: ahh, that was my flub, i missed the -x option. thanks for the info =)14:33
*** xyhuang has quit IRC14:33
mriedemelmiko: np14:33
*** eglynn has joined #openstack-nova14:33
*** vilobhmm has joined #openstack-nova14:33
*** samueldmq has joined #openstack-nova14:33
bauzaselmiko: https://wiki.openstack.org/wiki/StableBranch#Proposing_Fixes14:33
*** READ10 has quit IRC14:34
elmikobauzas: yea, i followed that. just not closely enough ;)14:34
*** xyhuang has joined #openstack-nova14:34
mriedemctrath: rlrossit: we have a lot of nova bugs marked as high severity, might be good experience to comb through some of those, since some have reviews - could check out the reviews to see if they are sane, and some might not be high severity after all14:35
mriedemhttps://bugs.launchpad.net/nova/+bugs?orderby=-importance&start=014:35
*** READ10 has joined #openstack-nova14:36
bauzasnagyz: no, you're rightr14:39
*** electrocucaracha has joined #openstack-nova14:39
*** electrocucaracha has quit IRC14:40
nagyzbauzas, I have some nodes where the ephemeral is on ceph so I must use raw images while on some other nodes it's on local SSD...14:40
*** electrocucaracha has joined #openstack-nova14:40
*** eharney has joined #openstack-nova14:40
nagyzso there I want qcow214:40
ctrathmriedem: Ok, I'll take a look at some today14:40
bauzasnagyz: you can easily create your own out-of-tree filter which would look at that property14:40
bauzasnagyz: since we pass the whole image dictionary14:41
nagyzright. I'll probably do that.. but I figured the upstream filter should be able to do this as well14:41
*** otter768 has joined #openstack-nova14:41
mdboothdansmith: I was just writing a -1 review for that patch :)14:42
mdboothMeh, I've started so I'll finish14:42
openstackgerritMatthew Gilliard proposed openstack/nova-specs: Query and cancel ongoing live migrations  https://review.openstack.org/17914914:42
dansmithmdbooth: which, mriedem's ?14:42
mdboothYeah14:42
mdboothNothing earth shattering14:43
*** marun has quit IRC14:43
dansmithmdbooth: well, hurry up and I'll remove my -W if need be14:43
dansmithbefore it merges I mean14:43
mdboothNah, I was just pointing out that it's still racy14:43
mdboothJust less so14:43
mdboothSo merge away14:43
mriedemmdbooth: there is another thing that can be done to completely lock it down14:43
mriedemwhich i already know about14:43
dansmithmdbooth: this is the "fix the thing I know is completely wrong but refuse to check" level of fix14:44
mdboothmriedem: I was going to suggest passing the mac to disassociate(), and doing a compare and swap in the db14:44
*** xyhuang has quit IRC14:44
dansmithmdbooth: like, we didn't even check to see if we *think* the IP is properly owned by the de-lease-er14:44
mdboothdansmith: Indeed.14:44
mdboothmriedem: That way, disassociate() would be atomic wrt to mac, no race at all.14:45
mriedemmdbooth: mine was basically the opposite, pass the vif to fixedip.associated and set the allocated flag at the same time that the instance is associated with the fixedip14:45
mriedemwhich requires a db api and rpc version change in the fixed ip object14:45
dansmithmdbooth: right, we know that needs to happen, but without the mac in that function, we can't make that call14:45
mdboothYeah, I saw that14:45
mriedemthe allocated=True flag just means that fixedip.virtual_interface_id is set14:45
mdboothAnyway, as I say the patch as is is an obvious improvement14:45
mriedemwhich, btw, couldn't we do that with some sqlalchemy magic?14:45
mriedemso we don't actually need the allocated column?14:46
mdboothmriedem: It's not that magic :)14:46
mriedemcouldn't it just be a property on the model?14:46
mriedem@property, like name in instances14:46
*** otter768 has quit IRC14:46
mdboothI'm still in the middle of it, but assuming it's not a direct property you can still do it with an exists clause14:46
bauzasalaski: ooooooh, just seeing https://review.openstack.org/#/c/192875/3/nova/cells/filters/different_cell.py,cm14:46
mdboothi.e. update foo set bar=blah where exists(select 1 from baz where grok=iverunout)14:47
bauzasalaski: 'routing_path' is a filt_props key already provided ?14:47
mriedemalaski: melwitt: bauzas: not sure we're going to get to voting on the cells job in the meeting today :(14:47
bauzasmriedem: yep, no real time for the discussion14:47
alaskibauzas: yes, the cells scheduler adds it.  well, I should double check that the upstream one does14:47
alaskimriedem: I'm getting that feeling14:48
bauzasalaski: please do, I should add that to the object then :(14:48
mriedemi'll horn in when there is 7.5 minutes left14:48
*** busterswt has quit IRC14:48
bauzasalaski: that's just like gold-mining to create that object14:48
bauzasalaski: digging and digging again for finding all the keys provided in filt_props14:48
melwittmriedem, alaski: yeah, also getting the feeling14:49
melwittspoke too soon :P14:49
alaskiheh14:51
melwittI'll add a Depends-On to the voting patch14:52
alaskibauzas: http://git.openstack.org/cgit/openstack/nova/tree/nova/cells/scheduler.py#n21314:52
mriedemmelwitt: you could just rebase on top of the regex patc hright?14:52
mriedemhttps://review.openstack.org/#/c/194410/14:52
* bauzas whispers14:52
melwittmriedem: oh, right. I'm in project-config, not nova.14:52
openstackgerritMichal Dulko proposed openstack/nova: Allow Cinder to raise when attaching a volume  https://review.openstack.org/19561714:53
alaskibauzas: it would probably be better to update those method calls than to modify filter_properties, if that can be done sanely14:53
bauzasalaski: you mean getting the routing_path only when needed in the filter ?14:54
*** mestery has joined #openstack-nova14:56
*** Jeffrey4l has quit IRC14:57
*** VW_ has quit IRC14:57
*** VW_ has joined #openstack-nova14:58
*** browne has joined #openstack-nova14:59
*** fifieldt has joined #openstack-nova15:00
*** david-ly_ is now known as david-lyle15:01
*** fifieldt has quit IRC15:01
*** beagles is now known as beagles_brb15:01
*** thedodd has joined #openstack-nova15:01
mdboothmriedem: commented15:01
artomSo, I didn't want to bring it up during the meeting because it's not stuck per se, but if API folks could go over my all tenants floating IPs spec one last time: https://review.openstack.org/#/c/171389/15:01
*** fifieldt has joined #openstack-nova15:01
melwittjohnthetubaguy, mriedem: do you think I should write a ML post about the quotas? I'm concerned about nested quotas being blocked (patches are already up), if quota simplification has to happen first, is there a way we can make progress on that in L considering we already approved the nested quotas for L?15:02
*** tdurakov has quit IRC15:02
*** ajayaa has joined #openstack-nova15:02
johnthetubaguymelwitt: I think we have to not block nested, if we didn't get the simplification spec up and approved15:02
*** armax has joined #openstack-nova15:03
johnthetubaguymelwitt: now I wish it was the other way around, but they have been waiting ages at this point, and we already told them yes twice, even if the second time was slightly accidental15:03
melwittjohnthetubaguy: okay, that's my thought as well15:03
mdboothIncidentally, I'd love to land this fix for a really old race condition affecting all api operations on instances: https://review.openstack.org/#/c/141115/15:04
*** mpaolino has joined #openstack-nova15:04
*** mpaolino has quit IRC15:04
*** agireud has joined #openstack-nova15:04
danpbjohnthetubaguy: i'm wondering what you think we should do with the VIF type nova/neutron integration/plugins spec(s)15:04
*** mtanino has joined #openstack-nova15:05
*** ferest has joined #openstack-nova15:05
danpbjohnthetubaguy: previously we debated ability to have neutron specify scripts to invoke by nova, but  then i switched to suggest a more formal object model & plugin system https://review.openstack.org/#/c/193668/15:05
johnthetubaguydanpb: we just discussed that in the meeting it seems like the consensus was to not block new drivers, while the new lib happens in paralell15:05
danpbgiven the spec deadline though and lack of interest from most of nova-core, its feeling  like its unlikely to make it15:05
*** tdurakov has joined #openstack-nova15:06
johnthetubaguydanpb: that all sounds important enough for an exception to me15:06
danpboh that's a shame i couldn't make the meeting15:06
johnthetubaguydanpb: if we agree that direction, we can sort out the paperwork to make that possible15:06
danpbok, so you think there's still a realistic chance we could do it for liberty if we can agree on exception15:07
*** Nikolay_St has joined #openstack-nova15:07
johnthetubaguydanpb: so the review timeline is getting crazy short at this point, but we should still try agree the direction at least, even if we can't quite make it, we should agree that direction right?15:07
*** unicell has joined #openstack-nova15:07
danpbyeah, it'd be nice to do that15:07
johnthetubaguydanpb: thats my gut feeling here, anyways15:07
edleafebauzas: answered you on the spec: https://review.openstack.org/#/c/18773915:08
johnthetubaguydanpb: cool, lets try for that15:08
*** moshele has quit IRC15:08
danpbas we could at least start some of the work to move things forward15:08
johnthetubaguydanpb: +115:08
*** hshiina has joined #openstack-nova15:08
edleafebauzas: if that's still confusing, let's discuss here15:08
danpbeven if we don't end up merging it15:08
johnthetubaguydanpb: so if we need a feature branch for this, we could do that, but not sure if that really helps15:08
bauzasedleafe: oh I see your confusion15:09
*** med_` is now known as med_15:09
bauzasedleafe: if you see my series, you'll find the latest one as being the HostManager change15:09
*** med_ is now known as med___15:09
bauzasedleafe: that one is having a Partially-Implements tag, which means it's not the last from the series15:09
*** hshiina has left #openstack-nova15:09
johnthetubaguymelwitt: let me know if there is something I can do to say we are not blocking that quota stuff, not had chance to review that yet15:10
edleafebauzas: ok, understood15:10
bauzasedleafe: that's because I'm iteratively modifying portions of code to verify that Nova is happy15:10
bauzasedleafe: and it took me a certain amount of time to fix the HostManager use15:10
*** iamjarvo has quit IRC15:10
edleafebauzas: but how does that affect filter.py15:10
*** jasondotstar has quit IRC15:10
scheurandanpb: johnthetubaguy: along to the decission to not block new vif_types, could you have another look at the macvtap-vif spec if there are some show stoppers to get it in in the old way?: https://review.openstack.org/#/c/182280/15:11
bauzasedleafe: that's the next patch I'm going to deliver, because I'll set a new method for filters.py passing a ReqSpec object15:11
johnthetubaguyscheuran: not today, I am afraid, but in general I hope to15:11
edleafebauzas: I think you're missing the point15:11
bauzasedleafe: probably, I don't see the relationship with your change15:11
scheuranjohnthetubaguy: no problem, thx15:12
openstackgerritMaxim Nestratov proposed openstack/nova: libvirt: resize ploop disks with prl_disk_tool  https://review.openstack.org/18239015:12
edleafebauzas: I'm proposing recording the results of each filter. How does ReqSpec have anything to do with that?15:12
*** ferest has quit IRC15:12
edleafebauzas: it's just a better way of getting filter info to the filters, but it won't change their result15:12
bauzasedleafe: you know the funny part of it ? I have no idea why ReqSpec is mentioned in your spec, since I didn't told about it - at all15:13
melwittjohnthetubaguy: okay. I *think* everything is fine, that is, I think no one thinks it's blocked, but I will check on the patches to make sure they're still moving15:13
bauzasedleafe: see all my comments, you won't see me saying I'd like to see the ReqSpec object implemented before your spec15:13
edleafebauzas: because the first idea was to attach the filter results to ReqSpec, where they would be persisted15:14
edleafebauzas: that was changed in favor of simply logging the failures15:14
edleafebauzas: see the earlier revisions15:14
johnthetubaguymelwitt: cool15:15
bauzasedleafe: I see it now...15:15
*** mtanino_ has joined #openstack-nova15:15
bauzasedleafe: so, what's your problem since I was not aware of that ?15:15
*** iamjarvo has joined #openstack-nova15:15
bauzasedleafe: I mean, I was just replying to your point which was saying "I don't think filters will get updated by having a Spec object15:16
*** mtanino has quit IRC15:16
edleafebauzas: it was jaypipes' comment about "how you plan to attach this information to the RequestSpec object...15:16
mnestratovrepeating here what I hadn't chance to discuss on nova-meeting jus a moment ago about parallels/virtuozzo driver feature parity changes15:16
mnestratovwe have a set of small fixes and they don't have enough attention. is is possible to look at them somehow?15:16
mnestratovhttps://review.openstack.org/#/c/182257/15:16
mnestratovhttps://review.openstack.org/#/c/184153/15:16
mnestratovhttps://review.openstack.org/#/c/186858/15:16
mnestratovhttps://review.openstack.org/#/c/182390/15:16
mnestratovhttps://review.openstack.org/#/c/184311/15:16
*** oro_ has quit IRC15:16
*** oro has quit IRC15:16
bauzasedleafe: okay, so, please consider that you'll get a host_passes(RequestSpec) method for each filter15:16
*** edmondsw has quit IRC15:16
mriedemmnestratov: you just did15:17
edleafebauzas: I was wrong to guess that ReqSpec wasn't going to be finished by L, but that's not really relevant to the current proposal15:17
bauzasedleafe: yup I don't think it's related15:17
mriedemmnestratov: make sure your ci is reporting on those, i don't see it on all of them15:17
edleafebauzas: there is no interaction at all with either filter_properties or ReqSpec involved15:17
bauzasedleafe: I mean, I began to review your spec when you dropped the idea of using the ReqSpec object15:17
mnestratovmriedem: ok sure15:18
edleafebauzas: yeah, the idea of attaching to ReqSpec was from some discussions at the Summit15:18
bauzasedleafe: since I missed that point, I need to think about that and see if the original proposition was better - like jaypipes said15:18
bauzasedleafe: gotcha15:18
bauzasedleafe: ok, now I understand better how the ReqSpec object popped up in the discussion15:19
edleafebauzas: there was a lot of pushbackk about persisting all results, whether in ReqSpec or logs15:19
edleafebauzas: if we're only going to record the failures, logs seemed like a better choice15:19
bauzasedleafe: so, given that, probably a discussion with jaypipes is worth it, no ?15:19
edleafebauzas: sure15:19
mnestratovmriedem: last changes on these were made before CI started to report on regular basis15:20
bauzasedleafe: as I said, I need to think more about the opportunity of amending the Spec object for that puropo15:20
edleafebauzas: I pinged him yesterday, but I was in and out a lot, and never followed up on IRC15:20
mriedemmnestratov: can't you recheck the ci on them manually?15:20
bauzaspurpose even15:20
*** alex_klimov has quit IRC15:20
mnestratovmriedem: sure15:20
*** alex_klimov has joined #openstack-nova15:20
bauzasedleafe: okay, I'll review your spec carefully then, and look at the very first PS15:20
edleafebauzas: the pain point is operators who can't tell why no hosts were found15:20
*** ijw_ has joined #openstack-nova15:21
edleafebauzas: I don't think we really need to muck up the ReqSpec for that15:21
bauzasedleafe: I appreciate the usecase15:21
edleafebauzas: logs would work better, IMO15:21
*** xyhuang has joined #openstack-nova15:21
bauzasedleafe: not 100% sure about that still, mriedem made a good point about instance_faults and instance_actions15:22
bauzasedleafe: I was originally thinking about that in Kilo15:22
bauzasedleafe: and my take was that probably select_destinations() wasn't rich enough when there was a failure15:22
edleafebauzas: this spec only addresses when no hosts pass the filters15:23
edleafebauzas: not when hosts pass but the instance fails to spawn15:23
bauzasedleafe: instead of throwing an exception, we should rather give the list of filters and hosts and leave the conductor probably do the update15:23
edleafebauzas: that's a whole 'nother issue15:23
bauzasedleafe: you misunderstood me15:23
*** ajayaa has quit IRC15:23
edleafebauzas: it wouldn't be the first time :)15:23
bauzasedleafe: when a filter is saying 0 hosts, then the scheduler throws an exception15:23
bauzasedleafe: instead of throwing an exception, we could return a rich set of information back to the conductor so it could use it15:24
kfox1111johnthetubaguy: any movement on the instance user spec?15:24
edleafebauzas: use it how?15:25
bauzasedleafe: logging an instance_fault by example15:25
edleafebauzas: no hosts matched what was requested.15:25
bauzasoops, s/logging/updating15:25
*** _diana_ has quit IRC15:25
*** edmondsw has joined #openstack-nova15:25
bauzasthat way, an operator could use the REST API for giving the faults related to the instance, and he would know which filter is wrong15:26
bauzasedleafe: without needing to read thru the logs15:26
bauzasfor getting, not giving...15:26
mriedemmnestratov: this is not something you can leak into the compute api https://review.openstack.org/#/c/186858/15:27
edleafebauzas: what about mriedem's point about Horizon leaking that information?15:27
bauzasedleafe: tbh, the more I think about that, the more I care about providing a way to use the REST API for this15:27
mriedemmnestratov: the compute api shouldn't be checking some specific config from libvirt15:27
bauzasedleafe: eh, isn't that an RBAC policy ?15:28
*** mtanino has joined #openstack-nova15:28
edleafebauzas: "I know we have instance_actions and instance_faults which are shown in Horizon and we don't want to leak details to non-admin users"15:28
bauzasedleafe: that doesn't necessarly mean we can't add a specific method for getting the scheduler faults which would be admin-only15:28
*** _diana_ has joined #openstack-nova15:29
johnthetubaguykfox1111: same message as before, except the freeze is in place now I am afraid15:29
bauzasedleafe: or we could just rip the scheduler details on the API if the admin policy isn't enforced15:29
*** angdraug has joined #openstack-nova15:30
mnestratovmriedem: I see, but what would you recomend if we don't support 'virtio' bus which reflects vda devices15:30
jaypipesbauzas, edleafe: would you catch me up to this conversation please?15:30
kfox1111damn.15:30
edleafebauzas: Relying on policy for security doesn't seems wise15:30
mriedemmnestratov: i'm saying, you can't have this in the nova.compute.api: if CONF.libvirt.virt_type == 'parallels':15:30
mriedemmnestratov: my suggestions are inline15:30
*** Murali has quit IRC15:30
edleafejaypipes: it's logging vs. returning filter failures to conductor15:30
*** neiljerr` has joined #openstack-nova15:30
kfox1111very unfortunate for all the other projecdts that will now be working around the lack of the feature. :/15:30
*** _diana_ has quit IRC15:31
edleafejaypipes: and then having conductor add those to instance_faults15:31
bauzasjaypipes: so, given the problem of providing a good interface for the operator to get information about a scheduler failure, I think we should give back the information to the conductor (instead of raising a NoValidHost exception) so that we could use that and persist that in like instance_faults15:31
kfox1111johnthetubaguy: what's the spec freeze exception process like? I want to formally request one.15:31
edleafejaypipes: this would require adding an admin-only API call so that operators could query the filter results15:31
*** mtanino_ has quit IRC15:31
bauzasjaypipes: and do the necessary policy control stuff for making sure it's admin only15:31
edleafejaypipes: instead of having to go through logs15:31
jaypipesbauzas: the problem with that is that instance_faults would leak implementation details of the underlying resource providers.15:31
mriedemmnestratov: unless there is something i'm not finding with image/volume metadata that would have the virt type info in it, i would add a get_default_root_device() method to the virt driver api which returns 'vda' in the ComputeDriver base class, and is overridden in the libvirt driver to return 'sda' if you're using parallels15:31
edleafejaypipes: this is only for the case where all hosts are filtered out15:31
*** Piet has joined #openstack-nova15:32
edleafejaypipes: not for build failures15:32
bauzasjaypipes: that's why I say it has to be policy driven15:32
kfox1111I believe the really contentious stuff is already out of the way, and the rest can be dealt with in code review.15:32
jaypipesedleafe: right, but we don't want, for example, instance_faults to contain something like "first filter got X hosts, second winnowed to Y hosts, etc"15:32
mnestratovmriedem: got you, sounds reasonable15:32
edleafebauzas: right now ops has no view. This would give them the view into that info15:32
bauzasjaypipes: ie. if only if it's an admin, we give that info15:32
edleafejaypipes: but that's pretty much what bauzas is thinking (if I'm understanding him correctly)15:33
edleafejaypipes: and it's not even that15:33
jaypipesbauzas, edleafe: how about just having that level of detail in the logging records, and giving a generic "Failed filter X" message in NoValidHost?15:33
kfox1111we're seeing cases like Octavia which are coming up with their own solutions just to work aroudn the lack of instance users. and several other projects are starting to go down that path too. :/15:33
edleafejaypipes: it would be "Filter Foo returned HostA, HostC, HostQ. Filter Bar returned HostC. Filter Baz returned nothing."15:33
*** neiljerram has quit IRC15:34
jaypipesedleafe: no, I don't want to have that in the fault message.15:34
kfox1111its fracturing openstack further. :/15:34
jaypipesedleafe: that is leaking way too much internal info.15:34
edleafejaypipes: that's what my spec calls for: logging the filtering results when no hosts pass filtering15:34
edleafejaypipes: in logs??15:34
edleafejaypipes: who has access to logs except people who already know the internal info?15:35
openstackgerritDan Smith proposed openstack/nova: Make evacuate update the migration object along the way  https://review.openstack.org/19437315:35
openstackgerritDan Smith proposed openstack/nova: API support for migration_type  https://review.openstack.org/18320015:35
openstackgerritDan Smith proposed openstack/nova: Add some notifications to the evacuate path  https://review.openstack.org/19437215:35
openstackgerritDan Smith proposed openstack/nova: Make live migration create a migration object record  https://review.openstack.org/18333115:35
openstackgerritDan Smith proposed openstack/nova: Make evacuate leave a record for the source compute host to process  https://review.openstack.org/18335415:35
jaypipesedleafe: yes, I support the logging in your spec. What I do not support is putting that information as-is in the fault returned to the user (i.e. NoValidHost)15:35
bauzasjaypipes: don't get me wrong, I'm not saying it would be leaked to the user15:35
edleafejaypipes: of course not15:35
edleafejaypipes: that's not what I proposed at all15:35
jaypipesedleafe: I do not want any changes to the REST API in this spec.15:36
bauzasjaypipes: I'm explicitely saying since 10 mins that it has to be RBAC driven by an API endpoint15:36
jaypipesbauzas: no.15:36
edleafejaypipes: it was what bauzas thought would be helpful in the instance_faults table15:36
jaypipesbauzas: that's what I'm saying is not a good idea.15:36
edleafejaypipes: +1 to no API change15:36
*** _diana_ has joined #openstack-nova15:36
*** med___ is now known as med_15:36
*** med_ has quit IRC15:36
*** med_ has joined #openstack-nova15:36
jaypipesbauzas: no changes to the REST API. just use the existing NoValidHost, and have a better fault message.15:36
bauzasjaypipes: so you prefer having operators using Kibana and other awks to find the relevant entries in the rotated logs to match with a request-id they would get using /os-instance-actions ?15:36
*** jdurgin has quit IRC15:37
jaypipesbauzas: yes. this isn't an os-instance-action. It's a log-level record of the details of filter decisions.15:37
edleafejaypipes: can you re-review https://review.openstack.org/#/c/187739 and see if you agree? If so, removing the -1 would be a big help15:37
bauzasjaypipes: okay, how the operator is getting the relevant scheduler log entry corresponding to the faulty instance, since the scheduler doesn't know the instance uuid ?15:37
jaypipesedleafe: OK, so the thing I wanted in that review was an indication of how you are returning the detailed filter result information (via the request_spec object) to the conductor, which then would build the informative fault message from that info for the NoValidHost return.15:38
*** hemnafk is now known as hemna15:38
danpbdoes nova have any persistent record in the database about VIFs attached to a guest ?15:38
jaypipesbauzas: the operator knows the request_id. she can look up the relevant log messages via that id, filtering by "nova.scheduler.*"15:38
edleafejaypipes: what more information would you want other than "no hosts met the filter criteria"?15:39
danpbsomething equivalent to the block device mapping but for nics ?15:39
jaypipesdanpb: meh, kinda...15:39
bauzasjaypipes: how does he know the request id ?15:39
openstackgerritKarim Boumedhel proposed openstack/nova: report an error for hw:numa_nodes=0  https://review.openstack.org/19026715:39
jaypipesdanpb: the virtual_interfaces table has *some* of the info.15:39
bauzasjaypipes: because it's not given by a nova show15:39
jaypipesbauzas: it's in the HTTP response.15:39
bauzasjaypipes: I see, other way, he greps the nova-api log to get the req-id from the nova boot, and then look at n-sch to get the log15:40
bauzasjaypipes: I just think having a REST API doesn't sound that mad you're thinking about15:40
edleafejaypipes: I also have POC code for the spec here: https://review.openstack.org/#/c/192184/15:40
*** mtanino has quit IRC15:40
bauzasjaypipes: in particular when logs are rotated once per day and gzipped15:40
*** annegentle has joined #openstack-nova15:40
*** dsanders has joined #openstack-nova15:41
bauzasjaypipes: that just makes me think that Rackspace for example would prefer something better than just digging in the logs15:41
danpbjaypipes: ok thanks for the pointer15:41
bauzasjaypipes: because you would imagine 10 nova-api processes, 1 week retention and one nova-sch per cell...15:42
bauzasjaypipes: so, are you -2 on providing an API because it requires an API change or because you care about any possible leakage ?15:43
mriedemmnestratov: sorry, but the virt driver idea won't work in the nova.compute.api since there isn't a virt driver reference in there15:43
mnestratovmriedem: I saw your comment15:43
mriedemmnestratov: so you'll have to move that to the compute manager somehow, or fix it in the virt driver, or use some image/volume metadata key if available15:43
mriedemmnestratov: unless ndipanov has other ideas15:43
bauzasjaypipes: (I also wish good pleasure to operators once we fix the n-sch scalability issue and then we can scale out using multiple processes)15:44
jaypipesbauzas: don't assume a logrotate'd deployment..15:44
*** mkoderer_cloud has quit IRC15:44
ndipanovmriedem, link pls15:44
mriedemndipanov: https://review.openstack.org/#/c/186858/15:44
bauzasjaypipes: don't assume a 1x n-api and 1x n-sch :)15:44
jaypipesbauzas: I am -2 on using an existing os-instance-actions API that wasn't designed or fits this situation.15:44
jwcroppe_bauzas: FWIW, I think anything we can do to get even some primitive strides forward (read: simple log improvements) to better understand why a VM couldn't schedule would be tremendously useful for operators15:44
bauzasjaypipes: -2 on os-instance-actions, that I understand because I was unclear15:45
ndipanovmriedem, ah15:45
ndipanovsuper sad15:45
*** jasondotstar has joined #openstack-nova15:45
mnestratovmriedem: moving this code from api would mean that we should move assigning vda from here too. Is this what you assumed?15:46
ndipanovmnestratov, not so fast15:46
ndipanovyou have to assume vda there15:46
bauzasjwcroppe_: fair point, I'm just thinking of the usability of that15:46
*** ajayaa has joined #openstack-nova15:46
ndipanovthere is a reason for it - it has to do with block device mapping format15:46
ndipanovbut not too important for this patch15:46
ndipanovjust assume that it has to be there15:47
bauzasjwcroppe_: since the scheduler doesn't know the instance UUIDs, you will only get failure information regarding a request-id15:47
*** dboik has quit IRC15:47
mnestratovndipanov: then we should rewrite root_device in case of parallels virt_type ignoring what was specified15:48
ndipanovmnestratov, however compute manager will call default_root_device_name15:48
bauzasjwcroppe_: that's the main thing I think it's just not that simple as a grep "instance_uuid" /var/log/nova/nova-scheduler.lgo15:48
*** dboik has joined #openstack-nova15:48
jwcroppe_bauzas: we've had some proto code that creates some textual reports when schedules fail and I can tell you it saves me a *boat load* of time troubleshooting which of 25 filters failed :)15:48
ndipanovand there it will delegate to the virt driver15:48
ndipanovwhich will know that it's running parallels and can override it15:48
ndipanovso your fix should be there15:48
bauzasjwcroppe_: how do you match the relationship between a schedule request and an instance boot ?15:49
ndipanovunless libvirt already checks that15:49
ndipanovand then the fix is to make it always override what is supplied15:49
ndipanovwhich I've already fixed in all other places15:49
*** alex_klimov has quit IRC15:50
ndipanovthough mriedem won't approve my patches for what I can only attribute to personal grudge15:50
jwcroppe_bauzas: I'd have to check, but I thought the inst uuid is somewhere in the request spec once it gets to select_destinations15:50
*** sdake_ has joined #openstack-nova15:50
bauzasjwcroppe_: tss tss tss, it isn't15:50
mnestratovndipanov: ok, thanks. I just thought, when I did my change, that I should take into account cases that you had in your mind when you added this default vda assigning15:50
*** beagles_brb is now known as beagles15:50
jwcroppe_looking at the code now...sec15:50
bauzasjwcroppe_: it has been removed since Juno or Kilo, I don't remember15:50
ndipanovmnestratov, see https://review.openstack.org/#/c/189632/15:50
*** vilobhmm has quit IRC15:51
ndipanovjk mriedem btw :)15:51
openstackgerritTimofey Durakov proposed openstack/nova: Split instance resize operation into 2 phases  https://review.openstack.org/19508815:51
openstackgerritTimofey Durakov proposed openstack/nova: Create class hierarchy for tasks in conductor  https://review.openstack.org/18449515:51
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: add feature "serial console"  https://review.openstack.org/18091215:52
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: fix "evacuate" for s390 and hyper-v  https://review.openstack.org/19473015:52
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: add CLI commands to features  https://review.openstack.org/19564715:52
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: add feature "evacuate"  https://review.openstack.org/19564815:52
*** annegentle has quit IRC15:52
*** annegentle has joined #openstack-nova15:52
*** sdake has quit IRC15:54
mriedemndipanov: just added myself to that this morning, you bastard :)15:54
*** unicell has quit IRC15:54
jwcroppe_bauzas: request_spec['instance_properties']['uuid']15:54
mriedemer, to the base change15:54
*** oro has joined #openstack-nova15:54
ndipanovmriedem, in all seriousness - it might fix a bunch of problems we're having with _get_disk_info15:55
mriedemndipanov: ok, it's in a tab15:55
mriedemi plan on just doing reviews today15:55
ndipanovnice15:55
*** dsanders has quit IRC15:57
*** arnaud____ has joined #openstack-nova15:57
*** _diana_ has quit IRC15:57
*** elmiko has left #openstack-nova15:59
*** armax_ has joined #openstack-nova15:59
*** Longgeek has quit IRC15:59
*** atuvenie_ has quit IRC16:00
*** Longgeek has joined #openstack-nova16:00
kashyapmdbooth: Thank you for reminding the devpi+wheel. I did fix my `pip` pain.16:00
*** armax has quit IRC16:01
*** armax_ is now known as armax16:01
kashyapmdbooth: But, I should shamed publicly because, I (incorrectly) _thought_ I had some deps and didn't double-check them. So, I deserved the pain too.16:01
jwcroppe_bauzas: scheduler utils's build_request_spec still puts the 'instance_properties' in there with the instance data - https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L7316:02
*** Sukhdev has joined #openstack-nova16:03
*** tojuvone has joined #openstack-nova16:04
*** scheuran has quit IRC16:05
artommelwitt, alex_xu_, johnthetubaguy, mriedem could the all-tenants floating IPs spec get a final look over please? https://review.openstack.org/#/c/171389/16:06
*** VW_ has quit IRC16:07
*** Nikolay_St has quit IRC16:08
*** samueldmq has quit IRC16:12
*** greghaynes has joined #openstack-nova16:12
*** htruta has quit IRC16:12
*** tellesnobrega_ has quit IRC16:12
*** raildo has quit IRC16:12
*** VW_ has joined #openstack-nova16:13
*** Nikolay_St has joined #openstack-nova16:13
*** VW_ has quit IRC16:14
*** VW_ has joined #openstack-nova16:15
openstackgerritDaniel Berrange proposed openstack/nova-specs: Virtual guest device role tagging  https://review.openstack.org/19566216:15
*** sahid has quit IRC16:15
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: add feature "serial console"  https://review.openstack.org/18091216:16
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: add feature "evacuate"  https://review.openstack.org/19564816:16
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: add CLI commands to features  https://review.openstack.org/19564716:16
markus_zsorry for the spam, I rebased too interactive...16:17
openstackgerritVladik Romanovsky proposed openstack/nova: Adding user_id handling to keypair index, show and create api calls  https://review.openstack.org/7048516:17
openstackgerritVladik Romanovsky proposed openstack/nova: tests: functional tests for keypairs create in API microversion 2.5  https://review.openstack.org/18364516:17
*** armax has quit IRC16:17
*** annegentle has quit IRC16:17
*** mpavone_ has quit IRC16:18
*** Nikolay_St has quit IRC16:20
*** subscope has quit IRC16:20
*** annashen has joined #openstack-nova16:20
*** Nikolay_St has joined #openstack-nova16:21
*** armax has joined #openstack-nova16:21
*** VW_ has quit IRC16:22
markus_zmriedem: maybe you can have a look (stable/kilo patch): https://review.openstack.org/#/c/186335/16:22
openstackgerritEugeniya Kudryashova proposed openstack/python-novaclient: Raise exception when API does't support microversions  https://review.openstack.org/18881616:22
*** tellesnobrega has joined #openstack-nova16:22
*** afazekas has quit IRC16:23
openstackgerritEugeniya Kudryashova proposed openstack/python-novaclient: Raise exception when API does't support microversions  https://review.openstack.org/18881616:23
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 1  https://review.openstack.org/15256916:25
*** sdake_ is now known as sdake16:25
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 2  https://review.openstack.org/16740816:25
*** raildo has joined #openstack-nova16:27
*** Nikolay_St has quit IRC16:29
*** erlon has joined #openstack-nova16:29
*** markus_z has quit IRC16:29
*** atuvenie has joined #openstack-nova16:30
*** Nikolay_St has joined #openstack-nova16:30
*** achanda has joined #openstack-nova16:31
*** romainh has left #openstack-nova16:34
*** Nikolay_St has quit IRC16:34
*** neiljerr` has quit IRC16:34
openstackgerritIvan Mironov proposed openstack/nova: libvirt: Minimal qemu version for discard support depends on image backend.  https://review.openstack.org/19567216:36
*** romainh has joined #openstack-nova16:36
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Implements 'microversions' api type - Part 1  https://review.openstack.org/15256916:36
*** nic has joined #openstack-nova16:36
*** agireud_ has joined #openstack-nova16:36
*** Nikolay_St has joined #openstack-nova16:37
baolisdague: ping, RE: https://review.openstack.org/#/c/179390/16:38
*** romainh has quit IRC16:38
*** agireud has quit IRC16:39
*** agireud_ is now known as agireud16:39
baolisdague: please see my comments, and I'd appreciate your feedback on how to move it forward. thanks16:39
*** derekh has quit IRC16:40
*** romainh has joined #openstack-nova16:40
*** yamahata has quit IRC16:40
*** VW_ has joined #openstack-nova16:41
*** ndipanov has quit IRC16:42
*** kiranr has joined #openstack-nova16:42
*** otter768 has joined #openstack-nova16:42
*** htruta has joined #openstack-nova16:43
*** kiranr has quit IRC16:43
*** unicell has joined #openstack-nova16:43
*** arnaud____ has quit IRC16:44
bauzasjwcroppe_: sorry had to step out, but instance_props['uuid'] only relates the first instance16:45
bauzasjwcroppe_: which means that if you have 2 instances queried, then you would have 2 logs having the same uuid for 2 different instances, which would be not good16:45
*** eglynn has quit IRC16:45
*** otter768 has quit IRC16:47
*** artom has quit IRC16:49
*** ijw_ has quit IRC16:49
jwcroppe_bauzas: good point - perhaps we could consider putting something in the request spec with all uuids... need to think16:50
jwcroppe_bauzas: stepping away for a few16:51
*** jwcroppe_ has quit IRC16:51
*** jwcroppe has joined #openstack-nova16:51
*** rwsu has joined #openstack-nova16:52
*** busterswt has joined #openstack-nova16:52
*** Nikolay_St has quit IRC16:53
openstackgerritIvan Mironov proposed openstack/nova: libvirt: Minimal qemu version for discard support depends on image backend.  https://review.openstack.org/19567216:54
*** artom has joined #openstack-nova16:55
openstackgerritsulochan-acharya proposed openstack/nova: xapi: ensure pv driver info is present prior to live-migration  https://review.openstack.org/18973116:55
*** jasondotstar has quit IRC16:55
*** romainh has left #openstack-nova16:56
*** jwcroppe has quit IRC16:56
*** electrocucaracha has left #openstack-nova16:58
*** Nikolay_St has joined #openstack-nova16:59
*** rajesht_ has quit IRC16:59
*** igordcard_ has joined #openstack-nova17:01
*** dboik has quit IRC17:03
*** htruta has quit IRC17:04
*** Nikolay_St has quit IRC17:04
*** mwagner_afk has quit IRC17:04
*** raildo has quit IRC17:05
*** hemna_ has joined #openstack-nova17:06
*** tjones1 has joined #openstack-nova17:06
*** danpb has quit IRC17:06
*** isd has joined #openstack-nova17:07
*** yamahata has joined #openstack-nova17:07
*** jasondotstar has joined #openstack-nova17:08
*** achanda has quit IRC17:08
*** patrickeast has joined #openstack-nova17:08
mdboothIs anybody using Sem-Ver pseudoheaders?17:09
*** hemna_ has quit IRC17:09
*** Marga_ has quit IRC17:09
*** tellesnobrega has quit IRC17:10
mdboothAlso, has anybody ever patched pbr out of Nova's setup.py?17:10
*** Marga_ has joined #openstack-nova17:10
mdboothWondering how much effort it was17:10
mdboothi.e. how much do you lose by trying to rid yourself of pbr's versioning madness?17:10
*** lucasagomes is now known as lucas-dinner17:12
mriedemmdbooth: i doubt you gain much since it's in all projects now17:12
*** tjones2 has joined #openstack-nova17:12
mriedemmdbooth: so you'd have to do it universally17:12
mdboothmriedem: This would be downstream-only17:12
mriedemmdbooth: i know17:12
*** tjones2 has quit IRC17:13
*** tjones2 has joined #openstack-nova17:14
*** tjones1 has quit IRC17:15
*** ijw_ has joined #openstack-nova17:15
*** dsanders has joined #openstack-nova17:15
*** karimb has quit IRC17:15
*** angdraug has quit IRC17:17
mdboothI wonder... you might write a build tool which takes output from pbr and writes a static config which you can then edit for unreasonableness17:18
*** tonytan4ever has joined #openstack-nova17:18
* mdbooth will look under the covers17:19
*** tjones2 has quit IRC17:20
*** sdake_ has joined #openstack-nova17:20
*** yamahata has quit IRC17:20
*** yamahata has joined #openstack-nova17:20
mdboothLooks like it's already there: util.cfg_to_args()17:21
*** tjones1 has joined #openstack-nova17:22
*** isd has quit IRC17:23
*** sdake has quit IRC17:24
*** achanda has joined #openstack-nova17:25
*** bradjones has quit IRC17:26
*** _diana_ has joined #openstack-nova17:27
mriedemmdbooth: my impression is that things are pretty tightly coupled with pbr being baked in so undoing might work for pbr 1.2 but could splode in 1.7 or 2.0, et17:27
mriedem*etc17:27
mriedemif there were bigger issues, like pluggability/overridabilty, effort might be better spent in getting that into pbr itself to enable whatever you need to do17:28
*** bradjones has joined #openstack-nova17:28
*** bradjones has quit IRC17:28
*** bradjones has joined #openstack-nova17:28
*** sdake_ is now known as sdake17:29
mdboothmriedem: I'm thinking it might be worth frigging an environment to make pbr.utils.cfg_to_args() not explode, and writing that as static config17:29
mdboothAnd replacing setup.py with the static version17:29
mdboothThat way you could take the pain of pbr periodically, only when you make a setup change17:30
mdboothTomorrow...17:32
*** marun has joined #openstack-nova17:32
*** jasondotstar has quit IRC17:34
*** garyk has quit IRC17:38
artommelwitt, alex_xu_, johnthetubaguy, mriedem sorry for the spam, but I don't want the reason the spec didn't merge in liberty to be "the submitter didn't harass the cores enough." Could the all-tenants floating IPs spec get a final look over please? https://review.openstack.org/#/c/171389/17:38
*** jasondotstar has joined #openstack-nova17:39
*** mwagner_afk has joined #openstack-nova17:40
*** dboik has joined #openstack-nova17:40
*** samueldmq has joined #openstack-nova17:41
*** jwcroppe has joined #openstack-nova17:41
mriedemartom: noted17:42
*** Marga_ has quit IRC17:42
mriedemartom: i'm reviewing other things today17:42
mriedems/today/atm/17:42
mriedemif this were a deli, i'd have a ticket thingy17:42
mriedemand i'd be eating a lot of salted meats, but that's beside the point17:43
*** VW_ has quit IRC17:44
*** yassine_ has quit IRC17:45
*** afazekas has joined #openstack-nova17:49
mriedemsdague: https://review.openstack.org/#/c/195606/ is failing jenkins mega hard17:52
artommriedem, awesome, thanks :D17:52
artomI'm trying to riff on the deli smoked meat thing (since we're famous for it over here) but I'm coming up empty.17:53
*** boris-42 has joined #openstack-nova17:54
*** SourabhP has joined #openstack-nova17:54
*** afazekas has quit IRC17:55
*** Marga_ has joined #openstack-nova17:55
*** irenab has joined #openstack-nova17:55
*** tellesnobrega has joined #openstack-nova17:56
beaglesartom: the bagels are pretty decent too17:57
artombeagles, food in general is pretty awesome :)17:59
artomYou've been?17:59
beaglesartom: long long time ago17:59
edleafeartom: where is there?18:00
*** husanu0 has joined #openstack-nova18:02
*** dontalton has joined #openstack-nova18:02
artomedleafe, Montreal.18:02
*** ctrath1 has joined #openstack-nova18:03
*** Sukhdev has quit IRC18:03
edleafeartom: ah, yes. I was just there for PyCon.18:03
*** VW_ has joined #openstack-nova18:03
*** ctrath has left #openstack-nova18:04
openstackgerritMatt Riedemann proposed openstack/nova: hypervisor support matrix: fix "evacuate" for s390 and hyper-v  https://review.openstack.org/19473018:04
*** VW_ has quit IRC18:05
mriedemrlrossit: you add that hacking checking thing yet?18:05
*** VW_ has joined #openstack-nova18:05
*** jasondotstar has quit IRC18:05
*** isd has joined #openstack-nova18:07
*** Sukhdev has joined #openstack-nova18:07
*** ildikov has quit IRC18:07
*** jwcroppe_ has joined #openstack-nova18:08
mriedembauzas: you can probably +1 this now given the dependency https://review.openstack.org/#/c/190894/18:08
bauzasmriedem: sure thing18:08
bauzasmriedem: done18:08
*** jwcroppe_ has quit IRC18:09
*** jwcroppe_ has joined #openstack-nova18:09
*** jwcroppe has quit IRC18:11
*** husanu0 has quit IRC18:14
*** lsmola has quit IRC18:19
openstackgerritXinyuan Huang proposed openstack/nova-specs: Proposes passing network requests to scheduler  https://review.openstack.org/19569618:19
*** jasondotstar has joined #openstack-nova18:19
xyhuangbauzas: We had some discussions about whether the above really needs a spec, I still doubt about this but I just wrote one anyway to explain and clarify in more details… for your reference. Please could you take a look if you have time. Thanks!18:19
mriedemjogo: isn't http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/unit/virt/test_virt_drivers.py#n214 meant to test all interface methods on the ComputeDriver class?18:21
*** Nikolay_St has joined #openstack-nova18:22
*** ctrath1 has quit IRC18:25
*** annashen has quit IRC18:28
*** annegentle has joined #openstack-nova18:28
*** krtaylor has quit IRC18:31
*** tojuvone has quit IRC18:32
*** arnaud____ has joined #openstack-nova18:33
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py  https://review.openstack.org/18778518:33
bauzasxyhuang: sure thing, adding it to my review queue18:34
xyhuangbauzas: Thanks a lot!18:34
*** lpetrut1 has quit IRC18:36
*** ericksonsantos has joined #openstack-nova18:37
edleafejaypipes: hate to bug you, but after our discussion, can you remove your -1 from https://review.openstack.org/187739 ?18:37
*** ericksonsantos has quit IRC18:37
*** arnaud____ has quit IRC18:37
edleafejaypipes: I'm sure that's discouraging anyone else from reviewing it.18:37
*** VW_ has quit IRC18:37
*** ericksonsantos has joined #openstack-nova18:38
mriedemso i'm going to ask a dumb bdm question, but can someone explain the significance of boot_index -1?18:39
*** faraujo has joined #openstack-nova18:39
dansmithnot bootable?18:39
mriedemidk18:39
dansmithI remember going round about that when I reviewed that code18:39
mriedemi kind of wish there was a nice comment in the code somewhere, probably bdm.boot_index field, about what the values are and their meaning18:40
mriedemi feel like a jackass asking otherwise18:40
mriedemlike, is 0 or 1 root device?18:40
*** xyhuang has left #openstack-nova18:40
*** samueldmq has quit IRC18:43
*** VW_ has joined #openstack-nova18:43
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Kicking the oslo.service tires  https://review.openstack.org/19290018:43
*** ericksonsantos has quit IRC18:43
*** otter768 has joined #openstack-nova18:43
rlrossitmriedem: I've been working on the hacking check and I think I'm close18:43
rlrossitmriedem: will we want to swap over spawn_n() too I assume?18:44
openstackgerritDan Smith proposed openstack/nova: Make evacuate update the migration object along the way  https://review.openstack.org/19437318:44
*** dontalton has quit IRC18:44
mriedemrlrossit: for eventlet? not sure18:45
*** annegentle has quit IRC18:45
rlrossitI saw it in nova.utils also so I figured I would roll that in with it18:45
*** ZZelle_ has joined #openstack-nova18:46
*** jasondotstar has quit IRC18:47
*** ociuhandu has quit IRC18:48
*** otter768 has quit IRC18:48
*** gszasz has quit IRC18:53
openstackgerritgaryk proposed openstack/nova: VMware: add in folder support on VC  https://review.openstack.org/16506018:54
*** r-daneel has quit IRC18:54
*** r-daneel has joined #openstack-nova18:55
*** jwcroppe_ has quit IRC18:55
*** jwcroppe has joined #openstack-nova18:56
*** ericksonsantos has joined #openstack-nova18:56
*** rajesht has joined #openstack-nova18:56
*** jwcroppe has quit IRC18:57
*** jwcroppe has joined #openstack-nova18:57
*** agireud has quit IRC18:58
*** rajesht has quit IRC19:00
*** VW_ has quit IRC19:01
*** _diana_ has quit IRC19:02
*** VW_ has joined #openstack-nova19:02
*** VW_ has quit IRC19:03
*** ericksonsantos has quit IRC19:03
*** VW_ has joined #openstack-nova19:03
*** Raildo has joined #openstack-nova19:04
*** jasondotstar has joined #openstack-nova19:07
*** agireud has joined #openstack-nova19:08
*** tellesnobrega has quit IRC19:10
*** achanda has quit IRC19:11
*** dprince has quit IRC19:11
*** rotbeard has joined #openstack-nova19:12
*** Piet has quit IRC19:13
*** romainh has joined #openstack-nova19:18
openstackgerritMaxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo  https://review.openstack.org/18431119:21
*** ildikov has joined #openstack-nova19:22
*** Raildo is now known as raildo19:23
*** jdurgin has joined #openstack-nova19:24
*** dane-fichter has quit IRC19:26
openstackgerritRyan Rossiter proposed openstack/nova: Add hacking check for greenthread.spawn()  https://review.openstack.org/19571319:28
*** annashen has joined #openstack-nova19:29
*** tellesnobrega has joined #openstack-nova19:29
*** rushiagr_away is now known as rushiagr19:32
*** isd has quit IRC19:32
*** arnaud____ has joined #openstack-nova19:33
*** annashen has quit IRC19:34
*** tellesnobrega has quit IRC19:35
*** annashen has joined #openstack-nova19:35
*** ctrath1 has joined #openstack-nova19:35
*** jdurgin has quit IRC19:37
*** Nikolay_St has quit IRC19:38
*** ajayaa has quit IRC19:38
*** arnaud____ has quit IRC19:38
*** Nikolay_St has joined #openstack-nova19:39
*** ijw_ has quit IRC19:45
*** lyarwood has quit IRC19:45
*** lyarwood has joined #openstack-nova19:45
*** _diana_ has joined #openstack-nova19:46
*** Piet has joined #openstack-nova19:47
*** lyarwood has quit IRC19:51
*** lyarwood has joined #openstack-nova19:52
openstackgerritPeter Savage proposed openstack/python-novaclient: Added marker functionality to flavours and images  https://review.openstack.org/19321219:52
*** lpetrut has joined #openstack-nova19:52
*** _diana_ has quit IRC19:55
*** VW_ has quit IRC19:59
*** pixelb has quit IRC20:01
*** mtanino has joined #openstack-nova20:01
*** VW_ has joined #openstack-nova20:03
*** atuvenie has quit IRC20:05
*** Sukhdev has quit IRC20:07
*** jecarey has quit IRC20:09
*** krtaylor has joined #openstack-nova20:09
*** ctrath1 has quit IRC20:09
*** isd has joined #openstack-nova20:11
*** tonytan4ever has quit IRC20:11
*** Longgeek has quit IRC20:13
*** tonytan4ever has joined #openstack-nova20:16
*** alex_klimov has joined #openstack-nova20:17
*** gabriel-bezerra has quit IRC20:17
*** VW_ has quit IRC20:18
*** gabriel-bezerra has joined #openstack-nova20:20
openstackgerritPeter Savage proposed openstack/python-novaclient: Added marker functionality to flavours and images  https://review.openstack.org/19321220:22
*** sdake_ has joined #openstack-nova20:24
*** thangp has quit IRC20:24
*** sdake has quit IRC20:28
*** sdake_ is now known as sdake20:29
*** Redcavalier has quit IRC20:29
*** ivasev has quit IRC20:30
*** BobBall has quit IRC20:32
*** BobBall has joined #openstack-nova20:34
*** arnaud____ has joined #openstack-nova20:34
*** BobBall is now known as Guest5495820:34
*** ijw_ has joined #openstack-nova20:36
openstackgerritMiguel Lavalle proposed openstack/nova-specs: Integrate DNS resolution with Neutron using Nova hostname  https://review.openstack.org/9015020:37
*** _diana_ has joined #openstack-nova20:37
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Specify current directory during processutils.execute using new cwd param  https://review.openstack.org/19160420:39
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Specify current directory during processutils.execute using new cwd param  https://review.openstack.org/19160420:39
*** jasondotstar has quit IRC20:39
*** vladikr has quit IRC20:39
*** jaypipes has quit IRC20:39
*** arnaud____ has quit IRC20:39
*** obondarev has quit IRC20:41
*** ijw_ has quit IRC20:41
*** obondarev has joined #openstack-nova20:42
*** otter768 has joined #openstack-nova20:44
*** otter768 has quit IRC20:49
*** dboik has quit IRC20:52
jogomriedem: want to review my  pictures https://review.openstack.org/#/c/194840/20:53
*** oro has quit IRC20:55
dansmithmriedem: careful, lots of malware starts off with the same sort of phishing lure20:55
jogodansmith: shhh don't give it away20:59
dansmithmriedem: jogo is a twenty-something dude in SFO. If he tells you anything different, don't send any money20:59
jogolol21:00
dansmithjogo: we have to be careful.. mriedem runs windows, so clicking on links is really dangerous :)21:00
jogohaha  I wonder if SVG graphics can be used to attack21:00
dansmithwindows has had jpeg exploits before :)21:00
jogoSVG files contain a small JavaScript entry, which attackers exploited in the campaign observed by AppRiver to redirect victims to a webpage set up to push a piece of malware.21:01
jogotitle: Cybercriminals Use SVG Files to Distribute Ransomware21:01
dansmithoof21:01
*** ijw_ has joined #openstack-nova21:02
*** patrickeast_ has joined #openstack-nova21:03
*** annegentle has joined #openstack-nova21:03
*** patrickeast has quit IRC21:05
*** patrickeast_ is now known as patrickeast21:05
kashyapdansmith: When you have a moment, would you mind taking a look at this unit test failure, and tell me what glaring mistake am I making there -- http://paste.openstack.org/show/321179/21:08
kashyapThe modified code is below the test run.21:09
dansmithjust a sec21:10
kashyapSure, no rush. It's an upstream backport of a fix by Matt (Riedemann). I bothered him on #openstack-stable, seems like he's afk. So, spamming you here. :-)21:11
*** isd has quit IRC21:11
dansmithkashyap:  I think you want to mock get_domain_info and not get_domain21:12
dansmithbut it looks like it's already mocked to me21:12
dansmithpresumably by fakelibvirt21:12
dansmithMagicMock name='get_domain().info().__getitem__()21:13
dansmiththis tells you what it did21:13
kashyapI see,21:13
dansmithit called get_domain(), and then .info() on the return value from that, and then is doing [foo] on the result of that21:13
*** KurtMartin has joined #openstack-nova21:13
dansmithand since you didn't set [foo] on the end thing, it's throwing a keyerror21:13
kashyapYeah, realized the reason of KeyError.21:14
*** IanGovett has quit IRC21:14
* kashyap is super illiterate on Mocks and MagicMocks, etc (was reading a bit about them earlier).21:14
kashyapdansmith: Thanks, I'll test a bit more and get to this tomorrow.21:15
dansmithit's usually just something like you're missing a step in a chain of mocks,21:15
dansmithor doing it on the wrong mock21:15
kashyapYeah, the original test is here - https://review.openstack.org/#/c/181781/3/nova/tests/unit/virt/libvirt/test_driver.py,cm21:15
*** burt has quit IRC21:16
kashyapThe libvirt_guest is not present in stable/kilo21:16
dansmithoh, and I'm looking at the libvirt driver in master,21:16
dansmithso if it's a backport, knowing what changed is probably the key there21:16
kashyapSorry for not providing the full context.21:16
dansmithping mriedem tomorrow and he can probably point to it quicker, or I can help again tomorrow on a fresh brain21:17
kashyapdansmith: No worries. I just want to educate myself on these, hence the attempt. I know mriedem would do it in his sleep :-)21:17
dansmithsure, make him tell you, not fix it :)21:17
kashyap:-) /me heads off for the night.21:18
*** VW_ has joined #openstack-nova21:20
*** IanGovett has joined #openstack-nova21:20
*** Longgeek has joined #openstack-nova21:21
*** faraujo has quit IRC21:22
*** LukeHinds has quit IRC21:22
*** dboik has joined #openstack-nova21:22
*** dboik_ has joined #openstack-nova21:24
*** iamjarvo has quit IRC21:25
*** dboik has quit IRC21:27
*** eharney has quit IRC21:28
*** marun has quit IRC21:33
*** ijw_ has quit IRC21:38
*** mtanino has quit IRC21:38
*** ijw_ has joined #openstack-nova21:38
*** hogepodge has quit IRC21:38
mriedemwhat now?21:39
*** isd has joined #openstack-nova21:39
* mriedem has been away for 2 hours21:39
mriedemand is leaving now21:39
*** _diana_ has quit IRC21:39
*** SourabhP has quit IRC21:39
mriedemi'll just fix it :)21:40
*** SourabhP has joined #openstack-nova21:40
*** Longgeek has quit IRC21:40
*** _diana_ has joined #openstack-nova21:41
*** ian_brown has joined #openstack-nova21:41
*** mriedem has quit IRC21:44
*** tonytan4ever has quit IRC21:45
*** krtaylor has quit IRC21:47
*** romainh has quit IRC21:47
*** ian_brown has quit IRC21:48
openstackgerritJesse J. Cook proposed openstack/nova: Cells: Add cells filter RamByInstanceTypeFilter  https://review.openstack.org/19575521:50
openstackgerritJesse J. Cook proposed openstack/nova: Cells: limit slots by io_ops  https://review.openstack.org/19575621:50
openstackgerritJesse J. Cook proposed openstack/nova: WIP: cells: weighted cell list randomization  https://review.openstack.org/19575721:50
*** lpetrut has quit IRC21:50
*** rlrossit has quit IRC21:51
*** Piet has quit IRC21:52
*** faraujo has joined #openstack-nova21:54
*** Marga_ has quit IRC21:55
*** agireud has quit IRC21:58
*** annegentle has quit IRC21:58
*** nic has quit IRC22:00
*** nic has joined #openstack-nova22:00
*** apuimedo has joined #openstack-nova22:02
*** rushiagr is now known as rushiagr_away22:03
*** SourabhP has quit IRC22:04
*** jwcroppe_ has joined #openstack-nova22:05
*** mdrabe has quit IRC22:07
*** jwcroppe has quit IRC22:07
*** zzzeek has quit IRC22:07
*** mdrabe has joined #openstack-nova22:07
*** SourabhP has joined #openstack-nova22:07
*** pixelb has joined #openstack-nova22:14
*** KurtMartin has quit IRC22:14
*** KurtMartin has joined #openstack-nova22:14
*** cfriesen has quit IRC22:14
*** _diana_ has quit IRC22:16
*** faraujo has quit IRC22:17
*** ctrath has joined #openstack-nova22:18
*** SourabhP has quit IRC22:21
*** ctrath has quit IRC22:21
*** iamjarvo has joined #openstack-nova22:22
*** arnaud____ has joined #openstack-nova22:23
*** ctrath has joined #openstack-nova22:26
*** annashen has quit IRC22:27
*** edmondsw has quit IRC22:27
*** ctrath has quit IRC22:28
*** arnaud____ has quit IRC22:28
*** KurtMartin has quit IRC22:30
*** krtaylor has joined #openstack-nova22:30
*** dims has quit IRC22:30
*** ctrath has joined #openstack-nova22:31
*** eglynn has joined #openstack-nova22:31
*** dontalton has joined #openstack-nova22:32
*** ctrath has quit IRC22:32
*** SourabhP has joined #openstack-nova22:34
*** _diana_ has joined #openstack-nova22:34
*** alex_klimov has quit IRC22:35
*** Mike_D_laptop has joined #openstack-nova22:36
*** Longgeek has joined #openstack-nova22:37
*** dims_ has joined #openstack-nova22:38
*** nic has quit IRC22:42
*** Longgeek has quit IRC22:42
*** armax has quit IRC22:45
*** Administrator__ has joined #openstack-nova22:45
*** ijuwang has quit IRC22:45
*** otter768 has joined #openstack-nova22:45
*** dontalton has quit IRC22:45
*** dhellmann has quit IRC22:46
*** dhellmann has joined #openstack-nova22:46
*** _diana_ has quit IRC22:47
*** Kevin_Zheng has quit IRC22:47
*** baoli has quit IRC22:47
*** baoli has joined #openstack-nova22:48
*** Mike_D_laptop has quit IRC22:48
*** lucas-dinner has quit IRC22:48
*** ijw__ has joined #openstack-nova22:49
*** _diana_ has joined #openstack-nova22:50
*** otter768 has quit IRC22:50
*** armax has joined #openstack-nova22:51
*** ijw_ has quit IRC22:52
*** VW_ has quit IRC22:53
*** iamjarvo has quit IRC22:55
*** hogepodge has joined #openstack-nova22:58
*** annegentle has joined #openstack-nova22:59
*** sdake_ has joined #openstack-nova23:00
*** busterswt has quit IRC23:01
*** mriedem has joined #openstack-nova23:02
openstackgerritMichal Rostecki proposed openstack/nova: [WIP] Dedicate aggregates for specific tenants  https://review.openstack.org/19578323:03
*** sdake has quit IRC23:04
*** annegentle has quit IRC23:05
*** mriedem has quit IRC23:06
*** mriedem has joined #openstack-nova23:06
*** mestery has quit IRC23:07
*** sdake has joined #openstack-nova23:07
tonybHas anyeon seen devstack (stable/juno) stalling trying to create images?23:09
mikalHuh, one hour until spec freeze according to my reading of the emails23:10
*** ijw_ has joined #openstack-nova23:10
mikalGuess I better do a last couple of reviews23:10
*** sdake_ has quit IRC23:11
tonybmikal: go nuts.23:11
*** isd has quit IRC23:13
*** ijw__ has quit IRC23:13
*** ZZelle_ has quit IRC23:13
*** _diana_ has quit IRC23:13
*** thedodd has quit IRC23:16
*** zhenguo has joined #openstack-nova23:18
*** raildo has quit IRC23:20
*** Sukhdev has joined #openstack-nova23:21
*** _diana_ has joined #openstack-nova23:22
*** armax has quit IRC23:24
*** markvoelker has quit IRC23:24
*** otter768 has joined #openstack-nova23:26
openstackgerritIvan Mironov proposed openstack/nova: libvirt: Minimal qemu version for discard support depends on image backend.  https://review.openstack.org/19567223:29
*** ijw__ has joined #openstack-nova23:30
*** kylek3h has quit IRC23:32
*** ijw_ has quit IRC23:33
openstackgerritClaudiu Belu proposed openstack/nova-specs: Adds Hyper-V vTPM devices spec  https://review.openstack.org/19506823:37
*** apuimedo has quit IRC23:41
*** SourabhP has quit IRC23:45
*** otter768 has quit IRC23:49
*** ijw_ has joined #openstack-nova23:51
*** erlon has quit IRC23:51
*** faraujo has joined #openstack-nova23:51
*** leseb has quit IRC23:52
*** ijw__ has quit IRC23:53
*** sdake has quit IRC23:54
*** leseb has joined #openstack-nova23:55
*** sdake has joined #openstack-nova23:55
*** Sukhdev has quit IRC23:56
*** mdrabe has quit IRC23:57

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