Monday, 2015-05-04

*** mikeit_ has quit IRC00:00
*** Alexandra_ has joined #openstack-nova00:01
*** mikeit_ has joined #openstack-nova00:02
*** oro has quit IRC00:06
*** sdake_ has joined #openstack-nova00:07
*** sdake has quit IRC00:11
*** mikeit_ has quit IRC00:18
*** markvoelker has joined #openstack-nova00:21
*** mikeitt has joined #openstack-nova00:21
*** armax has joined #openstack-nova00:22
*** markvoelker has quit IRC00:25
*** eliqiao1 has left #openstack-nova00:29
*** eliqiao1 has joined #openstack-nova00:29
*** mikeitt has quit IRC00:31
*** mikeit has joined #openstack-nova00:32
*** mpaolino has joined #openstack-nova00:34
*** mpaolino has quit IRC00:36
*** promulo has joined #openstack-nova00:36
*** haomaiwa_ has quit IRC00:39
*** promulo__ has quit IRC00:40
*** liusheng has joined #openstack-nova00:41
*** Kevin_Zheng has joined #openstack-nova00:41
*** yingjun has joined #openstack-nova00:45
*** otter768 has joined #openstack-nova00:46
*** salv-orlando has joined #openstack-nova00:47
*** salv-orlando has quit IRC00:50
*** FL1SK has quit IRC00:52
*** bjornar has quit IRC01:04
*** salv-orlando has joined #openstack-nova01:05
*** hanzhang has joined #openstack-nova01:06
*** bjornar has joined #openstack-nova01:06
*** hanzhang has quit IRC01:07
*** armax has quit IRC01:08
*** vilobhmm1 has quit IRC01:12
*** salv-orlando has quit IRC01:12
*** markvoelker has joined #openstack-nova01:21
openstackgerritKylin CG proposed openstack/python-novaclient: Make _discover_extensions public  https://review.openstack.org/17106701:23
*** sdake has joined #openstack-nova01:24
*** sdake_ has quit IRC01:26
*** markvoelker has quit IRC01:26
*** sdake_ has joined #openstack-nova01:27
*** gtt116_ has quit IRC01:28
*** sdake has quit IRC01:30
*** popw has joined #openstack-nova01:35
openstackgerritEli Qiao proposed openstack/nova: API: remove admin require from compute_node_get_all_by_* from db layer  https://review.openstack.org/17533401:38
openstackgerritEli Qiao proposed openstack/nova: API: Add policy enforcement test cases for pci API  https://review.openstack.org/17574501:38
openstackgerritEli Qiao proposed openstack/nova: API: remove instance_get_all_by_host hard-code admin check from db  https://review.openstack.org/17767301:38
openstackgerritEli Qiao proposed openstack/nova: API: remove admin require for compute_node(get_all/search_by_hyperviso) from db  https://review.openstack.org/17574401:38
openstackgerritEli Qiao proposed openstack/nova: API: remove admin require for compute_node_create/update/delete from db layer  https://review.openstack.org/17569301:38
*** Alexandra_ has quit IRC01:41
openstackgerritzhangjl proposed openstack/nova: Check if destination_type in block_device_mapping_v2  https://review.openstack.org/17335701:45
*** coreywright has quit IRC01:49
*** patrickeast has joined #openstack-nova01:49
*** renatoarmani has joined #openstack-nova01:52
*** renatoarmani has quit IRC01:53
*** coreywright has joined #openstack-nova01:56
*** gokrokve has joined #openstack-nova01:57
*** patrickeast has quit IRC02:00
*** dimsum__ has quit IRC02:04
*** liusheng has quit IRC02:04
openstackgerritzhangjl proposed openstack/nova: Check if destination_type in block_device_mapping_v2  https://review.openstack.org/17335702:06
*** kaufer has joined #openstack-nova02:06
*** emagana has joined #openstack-nova02:11
*** nickrobison has joined #openstack-nova02:11
*** nickrobison has quit IRC02:16
*** markvoelker has joined #openstack-nova02:22
*** dsanders has quit IRC02:23
*** achanda has quit IRC02:25
*** yingjun has quit IRC02:26
*** markvoelker has quit IRC02:27
*** yingjun has joined #openstack-nova02:30
*** dsanders has joined #openstack-nova02:32
*** mikeit has quit IRC02:38
*** yingjun has quit IRC02:39
*** yingjun has joined #openstack-nova02:40
*** patrickeast has joined #openstack-nova02:41
*** patrickeast has quit IRC02:42
*** sdake_ is now known as sdake02:44
openstackgerritshihanzhang proposed openstack/nova: Optimize validate_networks  https://review.openstack.org/17574202:50
openstackgerritEli Qiao proposed openstack/nova-specs: Correct lock_server api return code  https://review.openstack.org/16327502:50
openstackgerritshihanzhang proposed openstack/nova: Optimize validate_networks  https://review.openstack.org/17574202:52
*** FL1SK has joined #openstack-nova02:53
*** salv-orlando has joined #openstack-nova02:56
*** Jeffrey4l has joined #openstack-nova02:56
*** salv-orlando has quit IRC03:01
*** dimsum__ has joined #openstack-nova03:05
*** dimsum__ has quit IRC03:10
*** dsanders has quit IRC03:10
*** dsanders has joined #openstack-nova03:10
*** yingjun has quit IRC03:12
*** yingjun has joined #openstack-nova03:14
*** salv-orlando has joined #openstack-nova03:14
*** salv-orlando has quit IRC03:19
*** coolsvapl is now known as coolsvap03:22
*** markvoelker has joined #openstack-nova03:23
*** achanda has joined #openstack-nova03:26
*** liusheng has joined #openstack-nova03:27
*** markvoelker has quit IRC03:27
*** dsanders has quit IRC03:28
*** unicell has joined #openstack-nova03:30
*** achanda has quit IRC03:31
*** unicell1 has quit IRC03:32
*** pixelbeat has quit IRC03:32
*** pixelbeat has joined #openstack-nova03:33
*** tojuvone_ has joined #openstack-nova03:35
*** haomaiwang has joined #openstack-nova03:38
*** nickrobison has joined #openstack-nova03:43
*** deepthi has joined #openstack-nova03:51
*** neelashah has joined #openstack-nova03:52
*** yingjun has quit IRC03:58
*** neelashah has quit IRC03:59
*** josecastroleon has joined #openstack-nova04:03
*** otter768 has quit IRC04:04
*** josecastroleon has quit IRC04:05
*** dsanders has joined #openstack-nova04:18
*** arnaud____ has joined #openstack-nova04:19
*** markvoelker has joined #openstack-nova04:24
*** markvoelker has quit IRC04:28
*** heyongli has quit IRC04:30
*** heyongli has joined #openstack-nova04:31
*** tjones1 has left #openstack-nova04:31
*** unicell has quit IRC04:41
*** unicell has joined #openstack-nova04:43
*** irenab has quit IRC04:43
*** pixelbeat has quit IRC04:45
*** bkopilov has joined #openstack-nova04:46
*** tjones1 has joined #openstack-nova04:51
openstackgerritLan Qi Song proposed openstack/nova: Remove useless volume when boot from volume failed  https://review.openstack.org/17708404:56
*** achanda has joined #openstack-nova05:02
*** vilobhmm1 has joined #openstack-nova05:09
*** moshele has joined #openstack-nova05:13
openstackgerritChhavi Agarwal proposed openstack/nova: This patch provides changes the detach_volume to pass volume_id as kwarg to allow object_compat decorator.  https://review.openstack.org/17969705:14
*** vilobhmm1 has quit IRC05:17
*** _gryf|afk is now known as _gryf05:20
*** markvoelker has joined #openstack-nova05:24
*** markvoelker has quit IRC05:29
*** irenab has joined #openstack-nova05:31
*** jyuso has joined #openstack-nova05:35
*** gokrokve has quit IRC05:37
*** vilobhmm1 has joined #openstack-nova05:37
openstackgerritDeepak C Shetty proposed openstack/nova: libvirt: Use 'relative' flag for online snapshot's commit/rebase operations  https://review.openstack.org/16880505:37
*** chhavi has joined #openstack-nova05:37
*** gokrokve has joined #openstack-nova05:37
*** vilobhmm1 has quit IRC05:38
*** gokrokve has quit IRC05:42
*** vilobhmm1 has joined #openstack-nova05:47
*** vilobhmm1 has quit IRC05:48
*** kedkul has joined #openstack-nova05:49
kedkulHi, nova interface-attach command adds vnic inside the vm but it is in down state, we have to manually make it up. Is there any way to avoid that, and vnic gets ip address?05:50
*** arnaud____ has quit IRC05:51
*** salv-orlando has joined #openstack-nova05:53
*** chhavi has quit IRC05:54
*** cfriesen_ has quit IRC05:56
*** salv-orlando has quit IRC05:56
*** vilobhmm1 has joined #openstack-nova05:58
*** salv-orlando has joined #openstack-nova05:59
*** salv-orlando has quit IRC05:59
*** zhangtralon has joined #openstack-nova05:59
*** vilobhmm1 has quit IRC06:00
*** josecastroleon has joined #openstack-nova06:02
*** kaufer has quit IRC06:03
*** otter768 has joined #openstack-nova06:05
*** Murali has joined #openstack-nova06:05
*** zhangtralon has quit IRC06:05
*** gokrokve has joined #openstack-nova06:08
*** gokrokve has quit IRC06:09
*** gokrokve has joined #openstack-nova06:09
*** otter768 has quit IRC06:10
*** salv-orlando has joined #openstack-nova06:11
*** doron has joined #openstack-nova06:15
*** doron is now known as doron_afk06:15
*** pkoniszewski has joined #openstack-nova06:15
*** salv-orlando has quit IRC06:15
*** pkoniszewski has quit IRC06:16
*** pkoniszewski has joined #openstack-nova06:17
*** ajayaa has joined #openstack-nova06:21
*** sdake_ has joined #openstack-nova06:22
kedkulHi, nova interface-attach command adds vnic inside the vm but it is in down state, we have to manually make it up. Is there any way to avoid that, and vnic gets ip address?06:23
*** sdake has quit IRC06:25
*** vilobhmm1 has joined #openstack-nova06:28
*** vilobhmm1 has quit IRC06:30
*** sahid has joined #openstack-nova06:30
*** sdake_ is now known as sdake06:35
openstackgerritAlex Xu proposed openstack/nova-specs: Add separated policy rule for each v2.1 api  https://review.openstack.org/12786306:40
*** sudipto has joined #openstack-nova06:42
*** dimsum__ has joined #openstack-nova06:42
*** xylan_kong has left #openstack-nova06:44
*** salv-orl_ has joined #openstack-nova06:45
*** dimsum__ has quit IRC06:47
*** Anticimex has quit IRC06:48
*** dsanders has quit IRC06:52
*** jcoufal has joined #openstack-nova06:55
ttxsdague, gilllliard: it's actually tonyb keeping the wiki and ics in sync those days. (giving credit where due)06:58
*** garyk has joined #openstack-nova07:02
tonybsdague, gilllliard: what'd I miss?07:03
*** ssurana has joined #openstack-nova07:03
*** doron_afk is now known as doron07:07
*** alanf-mc has joined #openstack-nova07:07
ttxtonyb: I don't think you missed anything, I was just unjustly credited for something you do07:08
*** alanf-mc_ has joined #openstack-nova07:08
*** pask81 has joined #openstack-nova07:09
*** jichen has joined #openstack-nova07:10
*** alanf-mc has quit IRC07:12
*** yamahata has quit IRC07:14
*** romainh has joined #openstack-nova07:18
*** moshele has quit IRC07:20
*** zhangtralon has joined #openstack-nova07:20
*** sdake has quit IRC07:21
*** doude has joined #openstack-nova07:21
openstackgerritEli Qiao proposed openstack/nova-specs: Add separated policy rule for each v2.1 api  https://review.openstack.org/12786307:23
*** achanda has quit IRC07:23
*** dsanders has joined #openstack-nova07:24
*** markvoelker has joined #openstack-nova07:26
*** achanda has joined #openstack-nova07:26
*** ildikov has joined #openstack-nova07:29
*** markvoelker has quit IRC07:30
*** nlahouti has quit IRC07:31
*** doron is now known as doron_afk07:31
*** moshele has joined #openstack-nova07:34
*** markus_z has joined #openstack-nova07:36
*** alanf-mc_ has quit IRC07:37
*** lpetrut has joined #openstack-nova07:37
*** achanda has quit IRC07:37
*** alanf-mc has joined #openstack-nova07:37
*** mpavone has joined #openstack-nova07:40
*** dsanders has quit IRC07:42
*** ssurana has quit IRC07:45
*** aswadr has joined #openstack-nova07:50
*** yingjun has joined #openstack-nova07:51
*** alanf-mc has quit IRC07:51
*** romainh has quit IRC07:52
*** chhavi has joined #openstack-nova07:56
*** jistr has joined #openstack-nova07:59
*** yingjun has quit IRC07:59
*** yingjun has joined #openstack-nova07:59
*** yingjun_ has joined #openstack-nova08:04
*** yingjun has quit IRC08:04
*** oro has joined #openstack-nova08:04
*** otter768 has joined #openstack-nova08:05
*** romainh has joined #openstack-nova08:08
*** ildikov has quit IRC08:10
*** otter768 has quit IRC08:10
*** yingjun_ has quit IRC08:17
openstackgerritSailaja proposed openstack/nova: Project quota update should not be less than user quota.  https://review.openstack.org/17972608:17
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004508:23
openstackgerritAdelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location  https://review.openstack.org/17972708:23
openstackgerritRadoslav Gerganov proposed openstack/nova: VMware: verify vCenter server certificate  https://review.openstack.org/17972808:24
*** yuntong1 has quit IRC08:25
*** ildikov has joined #openstack-nova08:26
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation  https://review.openstack.org/13825508:26
*** nickrobison has quit IRC08:27
*** markvoelker has joined #openstack-nova08:27
*** Hosam has joined #openstack-nova08:30
*** markvoelker has quit IRC08:31
*** Kevin_Zheng has quit IRC08:40
*** Kevin_Zheng has joined #openstack-nova08:41
*** pczesno has quit IRC08:44
*** alexpilotti has quit IRC08:45
*** ociuhandu has joined #openstack-nova08:48
*** kedkul has quit IRC08:48
*** deepakcs has joined #openstack-nova08:48
deepakcsHi all, anybody seeing issues with gate-nova-python27 ?08:48
deepakcsI see a vmwareapi error failing the jenkins and giving -1 (which AFAICS isn't related to the change i did in the patch)08:49
deepakcsMore details are here @ http://logs.openstack.org/05/168805/8/check/gate-nova-python27/584359c/console.html#_2015-05-04_07_23_32_99508:49
*** haypo has joined #openstack-nova08:50
*** arnaud____ has joined #openstack-nova08:52
*** alexpilotti has joined #openstack-nova08:54
openstackgerritsahid proposed openstack/nova-specs: console: convert consoles code to use objects framework  https://review.openstack.org/16583808:54
*** arnaud____ has quit IRC08:56
openstackgerritsahid proposed openstack/nova-specs: console: convert consoles code to use objects framework  https://review.openstack.org/16583808:56
openstackgerritzhangtralon proposed openstack/nova: resize-revert can break anti-affinity policy  https://review.openstack.org/16011009:02
*** Hosam has quit IRC09:02
*** Hosam has joined #openstack-nova09:03
*** Hosam has quit IRC09:07
tonybttx: Ahh okay thanks.09:08
deepakcsgaryk, ^^09:08
*** asrangne has joined #openstack-nova09:11
*** salv-orlando has joined #openstack-nova09:13
*** salv-orl_ has quit IRC09:13
*** aswadr has quit IRC09:13
*** markvoelker has joined #openstack-nova09:27
*** jyuso has quit IRC09:27
*** mpaolino has joined #openstack-nova09:29
*** markvoelker has quit IRC09:32
*** mpaolino has quit IRC09:35
*** salv-orlando has quit IRC09:43
*** ociuhandu has quit IRC09:44
*** salv-orl_ has joined #openstack-nova09:46
garykdeepakcs: let me take alook09:47
*** salv-orlando has joined #openstack-nova09:50
openstackgerritChhavi Agarwal proposed openstack/nova: This patch provides changes the detach_volume to pass volume_id as kwarg to allow object_compat decorator.  https://review.openstack.org/17969709:52
deepakcsgaryk, thanks09:53
openstackgerritgaryk proposed openstack/nova: VMware: fix unit tests to unblock gate  https://review.openstack.org/17974609:53
*** salv-orl_ has quit IRC09:53
*** dimsum__ has joined #openstack-nova09:54
deepakcsgaryk, i +1'ed it, just curious, was params=None added recently ?09:54
*** pixelbeat has joined #openstack-nova09:55
*** tojuvone_ has quit IRC09:55
garykdeepakcs: i think that it is related to the commit https://github.com/openstack/nova/commit/bf8c75b19291b7ed9a4822a9b2d2a523c269afd009:56
garykstill investigating09:56
openstackgerritChhavi Agarwal proposed openstack/nova: This patch provides changes the detach_volume  to pass volume_id as kwarg to allow object_compat decorator.  https://review.openstack.org/17969709:56
deepakcsgaryk, ok, i couldn't backtrack it when i tried :) anyways, can u pls have it +2'ed quick so that many patches can avoid the false -1's09:57
garykdeepakcs: i hope that it addresses the issue. lets wait for the gate to give an indication09:58
garykhopefully in about 10 minutes we will know09:58
deepakcsgaryk, agree, but then the 'gloabl requirement' patch should have indicated the -1, how did that pass ?09:58
*** tojuvone_ has joined #openstack-nova09:58
openstackgerritChhavi Agarwal proposed openstack/nova: This patch provides changes the detach_volume  to pass volume_id as kwarg to allow object_compat decorator.  https://review.openstack.org/17969709:59
garykdeepakcs: i am not sure that it is something related to the requirement. it looks like the requests library.10:01
*** lpetrut has quit IRC10:02
*** doron has joined #openstack-nova10:04
*** doron_afk has quit IRC10:04
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Helper scripts for running under Apache2  https://review.openstack.org/17967210:04
*** dguitarbite has quit IRC10:05
*** otter768 has joined #openstack-nova10:06
garykdeepakcs: the reason is an update in the requests library - this was updated yesterday (https://pypi.python.org/pypi/requests)10:07
deepakcsgaryk, good to know :)10:09
garykdeepakcs: once we have a +1 i'll update the commit message10:09
deepakcsgaryk, sure10:09
deepakcsgaryk, pls do add Reported-By: if u wish :)10:09
garykdeepakcs: sure. i'll open a bug10:11
*** otter768 has quit IRC10:11
*** leo_ has joined #openstack-nova10:11
deepakcsgaryk, cool10:11
leo_can i install/run kvm on an OpenStack VM? (running a VM on VM)10:12
*** moshele has quit IRC10:14
*** lpetrut has joined #openstack-nova10:15
deepakcsleo_, you can provided you enable nested virtualization support in the host (google for it)10:15
openstackgerritgaryk proposed openstack/nova: VMware: fix unit tests to unblock gate  https://review.openstack.org/17974610:15
garykCan nova cores please look at the above 0 this unblocks the gate10:15
*** armax has joined #openstack-nova10:15
garykjohnthetubaguy: ^^10:16
garyksdague: ^^10:16
leo_thanks deepakcs10:16
*** dguitarbite has joined #openstack-nova10:17
*** zhangtralon has quit IRC10:18
*** IanGovett has joined #openstack-nova10:21
*** moshele has joined #openstack-nova10:23
*** moshele has quit IRC10:24
sdaguegaryk: so ... https://review.openstack.org/#/c/179746/2/nova/tests/unit/virt/vmwareapi/test_read_write_util.py,cm confuses me10:28
*** kiran-r has joined #openstack-nova10:28
sdaguebecause that means that if you have a lower version of requests, it'll now fail?10:28
garyksdague: no, it should not fail. the parameter has a defualt value. the problem is mock10:29
garykthat is it checks all of the default params (as far as I understand)10:29
garykthe api should be backwards compatible. if not then we need to cap the requests library10:29
garykdoes that make sense?10:30
*** salv-orl_ has joined #openstack-nova10:31
*** vladikr has quit IRC10:31
*** salv-orlando has quit IRC10:34
*** sahid has quit IRC10:35
*** kiran-r is now known as _kiran_10:35
*** leo_ has quit IRC10:36
sdagueI supposed, mocking the requests signature is probably going to bite us in the future though. Instead of doing that, it seems like you could just test for _base_url being correct right?10:38
*** exploreshaifali has joined #openstack-nova10:40
*** moshele has joined #openstack-nova10:41
garyksdague: do you mean skip the rest?10:41
sdagueyeh, there's not anything particularlly relevant about the requests call right?10:42
sdagueit was just being used to ensure the url was created effective10:42
sdaguethat seems to be the point of the test10:43
garykyeah.10:43
garyki'll update10:43
*** asrangne has quit IRC10:43
*** atuvenie has joined #openstack-nova10:43
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds RemoteFX support to the Hyper-V driver  https://review.openstack.org/4252910:44
*** armax has quit IRC10:44
*** salv-orl_ has quit IRC10:46
*** salv-orlando has joined #openstack-nova10:47
*** lpetrut has left #openstack-nova10:48
*** dikonoor has joined #openstack-nova10:48
*** ociuhandu has joined #openstack-nova10:49
*** bfic has joined #openstack-nova10:51
*** dulek has joined #openstack-nova10:52
garyksdague: ok, i am done, just waiting for the tests to pass locally (i blew away the .tox dir so it will take a few minutes)10:53
sdaguecool, thanks10:53
*** tjones2 has joined #openstack-nova10:54
*** doron is now known as doron_afk10:55
*** tjones1 has quit IRC10:57
*** exploreshaifali has quit IRC10:59
*** claudiub has joined #openstack-nova11:03
*** baoli has joined #openstack-nova11:04
*** baoli has quit IRC11:05
openstackgerritgaryk proposed openstack/nova: VMware: fix unit tests to unblock gate  https://review.openstack.org/17974611:09
garyksdague: ^11:09
sdaguegaryk: hmmm... ok, so that wasn't really what I was thinking here11:10
garyksdague: then i misunderstood - you wanted that we check the base url11:12
garykso the addition of new params here does not break anything and it is like the header defined11:12
garyksdague: http://docs.python-requests.org/en/latest/api/11:13
*** Longgeek has joined #openstack-nova11:15
*** bfic has quit IRC11:16
*** pixelbeat has quit IRC11:16
*** redbeard has joined #openstack-nova11:17
openstackgerritSean Dague proposed openstack/nova: VMWare: Isolate unit tests from requests  https://review.openstack.org/17975711:18
sdagueok, that's what I was thinking11:18
sdaguegaryk: see if - https://review.openstack.org/179757 is reasonable to you11:20
*** bfic has joined #openstack-nova11:21
*** bfic has quit IRC11:22
*** salv-orlando has quit IRC11:23
sudiptosdague, johnthetubaguy currently can nova have specific resource monitors for different platforms? Since the Resource monitors seems to be a nova.conf option and can be added optionally...11:23
garyksdague: it is 6 of one and half a dozen of the other. maybe consider mocking _create_read_connection and not the requests.11:23
sdagueI guess, it's the request signature check that's the problem though11:24
sdaguethat's *not* our api11:24
sdagueso it shouldn't be mocked with a signature check11:24
garyksdague: correct, but what is importnat is what we pass to the requests.11:24
*** bfic has joined #openstack-nova11:25
garykso that is what my patch is validating.11:25
sdaguegaryk: but it's mocking the requests signature11:25
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Helper scripts for running under Apache2  https://review.openstack.org/17967211:26
sdaguewhat is the test supposed to check? it seems like the url building is all that it actually is supposed to check11:26
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation  https://review.openstack.org/13825511:26
garyksdague: that is the code that is invoked at the end of the day. and my understaning is that is what needs to be validated.11:27
*** ildikov has quit IRC11:27
*** zhangtralon has joined #openstack-nova11:27
sdaguethe place that should be validated, it at all, is in the oslo.vmware library right?11:27
garykmy understanding of the test is that is should validate the headers, the url and the flags passed to the requests11:27
openstackgerritzhangtralon proposed openstack/nova: resize-revert can break anti-affinity policy  https://review.openstack.org/16011011:27
sdaguewhy does it need to validate the headers11:28
sdaguewhy are those material to this call?11:28
*** eharney has quit IRC11:28
sdaguethey are set by oslo_vmware.rw_handles.FileHandle implicitly right? no nova code sets them11:29
garykyeah, i guess so.11:29
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Helper scripts for running under Apache2  https://review.openstack.org/17967211:31
*** bfic has quit IRC11:31
garyksdague: added some comments to the patch - you should mock out _create_read_connection instead of requests.11:33
*** sahid has joined #openstack-nova11:33
garykthat is the code that calls the requests11:34
sdagueok, sure11:34
sdagueit was quick, and I haven't finished my first cup of coffee yet11:34
garyk:)11:34
chhaviThere is Jenkin failure for gate-nova-python27 FAILURE in 17m 00s11:35
*** deepthi has quit IRC11:35
*** deepakcs has quit IRC11:36
sdaguechhavi: yes, we know11:36
chhavimy change-id has not modified this test, so is this issue been handled by some other change-id11:36
*** bfic has joined #openstack-nova11:36
garykchhavi: the problem is one of the python libraries was updated and the test in question breaks due to a signature mismatch11:37
chhaviok sure11:37
*** Anticimex has joined #openstack-nova11:38
*** pask81 has quit IRC11:40
openstackgerritzhangtralon proposed openstack/nova: Support force_hosts in the ChanceScheduler  https://review.openstack.org/17195111:40
dimsum__sdague: i have a pair of reviews in (nova and devstack) to run nova-api and nova-ec2-api under Apache. i was wondering if there should be a new tempest job to exercise it or if we can enable it in an existing job. thoughts?11:41
*** dimsum__ is now known as dims11:41
sdaguedims: so, I'll be honestly, until I get my summit content in the bag, thinking through problem spaces like that is something I'm going to have to punt on :)11:42
dimssdague: ack thanks! no worries11:42
*** ildikov has joined #openstack-nova11:43
openstackgerritSean Dague proposed openstack/nova: VMWare: Isolate unit tests from requests  https://review.openstack.org/17975711:43
dimsjohnthetubaguy: is a blueprint enough? https://blueprints.launchpad.net/nova/+spec/run-nova-services-under-apache2 - i just need a couple of scripts in nova tree (https://review.openstack.org/#/c/179672/)11:43
sdaguegaryk: ok - is https://review.openstack.org/179757 to your liking?11:44
*** claudiub_ has joined #openstack-nova11:44
*** claudiub has quit IRC11:45
garyksdague: works for me. looks like there may be a pep8 issue with line length.11:46
garykline 44 (not 100% sure)11:46
garykcode looks good11:47
*** sudipto has quit IRC11:50
*** markvoelker has joined #openstack-nova11:50
*** alexpilotti has quit IRC11:51
*** tojuvone_ has quit IRC11:54
*** exploreshaifali has joined #openstack-nova11:56
*** dikonoor has quit IRC11:57
*** raildo has joined #openstack-nova11:58
*** rfolco has joined #openstack-nova11:58
*** lpetrut has joined #openstack-nova11:59
*** lpetrut has left #openstack-nova11:59
*** exploreshaifali has quit IRC12:00
*** exploreshaifali has joined #openstack-nova12:00
sdagueflake8 didn't trigger for me locally, so I don't think so12:01
*** exploreshaifali has quit IRC12:03
*** exploreshaifali has joined #openstack-nova12:04
*** aix has joined #openstack-nova12:05
*** exploreshaifali has quit IRC12:07
*** popw has quit IRC12:09
garyksdague: the grumpy old man has given his +1. can you get other cores to look at this to unblock the gate12:10
garykjohnthetubaguy: ^ (https://review.openstack.org/#/c/179757/)12:10
*** dave-mccowan has joined #openstack-nova12:10
*** shakamunyi has quit IRC12:18
*** aysyd has joined #openstack-nova12:19
*** ferest has joined #openstack-nova12:20
*** VW_ has joined #openstack-nova12:21
*** htruta has joined #openstack-nova12:21
*** ferest_ has joined #openstack-nova12:23
*** salv-orlando has joined #openstack-nova12:24
*** ivasev has joined #openstack-nova12:24
sdaguejohnthetubaguy / mikal / superdan - would be good to get another nova-core to look at https://review.openstack.org/#/c/179757/ and fix unit tests12:25
*** ferest_ has quit IRC12:27
*** ferest has quit IRC12:27
*** ivasev has quit IRC12:28
*** pixelbeat has joined #openstack-nova12:29
*** subscope has joined #openstack-nova12:35
*** alexpilotti has joined #openstack-nova12:35
*** salv-orlando has quit IRC12:37
*** ivasev has joined #openstack-nova12:38
*** dprince has joined #openstack-nova12:39
*** shyama has joined #openstack-nova12:43
bauzasdims: today is UK holiday12:45
bauzasdims: so you won't get johnthetubaguy unless he wants to work :)12:46
dimsbauzas: ack thanks. just lining it up for scrollback :)12:46
bauzaswe're entering May, aka. France vacation month :-)12:47
bauzaslike 40% of our holidays are in the same month...12:47
bauzasand I guess some other EU countries are quite similar12:47
*** tojuvone_ has joined #openstack-nova12:48
*** dprince has quit IRC12:48
*** exploreshaifali has joined #openstack-nova12:48
*** Jeffrey4l has quit IRC12:49
*** cbader has joined #openstack-nova12:49
dimsbauzas: i envy you all :)12:51
bauzasdims: there is a Mirantis Grenoble office y'know :p12:51
*** exploreshaifali has quit IRC12:51
alex_xujohnthetubaguy: I worked out https://review.openstack.org/127863 today, and the related oslo.policy spec worked out also https://review.openstack.org/179685 , let me know your opinion when you have time, thanks in advance!12:52
dimsbauzas: ++ :)12:52
bauzasdims: I've been told that someone from MV, CA just made the move to Grenoble :)12:52
*** mikeit has joined #openstack-nova12:53
*** exploreshaifali has joined #openstack-nova12:53
dimsbauzas: ha!12:54
*** links has joined #openstack-nova12:56
*** exploreshaifali has quit IRC12:57
*** sdake has joined #openstack-nova12:57
*** jaypipes has joined #openstack-nova12:58
*** exploreshaifali has joined #openstack-nova12:58
*** exploreshaifali has quit IRC12:59
openstackgerritAlex Xu proposed openstack/nova: API: remove instance_get_all_by_host hard-code admin check from db  https://review.openstack.org/17767312:59
*** exploreshaifali has joined #openstack-nova13:00
*** exploreshaifali has quit IRC13:00
*** Jeffrey4l has joined #openstack-nova13:01
*** dikonoor has joined #openstack-nova13:03
*** moshele has quit IRC13:04
*** moshele has joined #openstack-nova13:04
*** kaufer has joined #openstack-nova13:08
*** signed8bit has joined #openstack-nova13:09
*** signed8bit has quit IRC13:09
*** ajayaa has quit IRC13:10
*** VW_ has quit IRC13:12
*** VW_ has joined #openstack-nova13:12
*** dims has quit IRC13:14
*** dims has joined #openstack-nova13:15
*** erlon_awaY has joined #openstack-nova13:15
*** alexpilotti has quit IRC13:16
*** _kiran_ is now known as kiran-r13:18
*** neelashah has joined #openstack-nova13:20
*** baoli has joined #openstack-nova13:21
*** jcoufal_ has joined #openstack-nova13:24
*** mestery_ is now known as mestery13:25
*** jcoufal has quit IRC13:27
*** erlon_awaY is now known as erlon13:27
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation  https://review.openstack.org/13825513:29
*** sdake_ has joined #openstack-nova13:31
*** dprince has joined #openstack-nova13:32
*** bfic has quit IRC13:32
*** sdake has quit IRC13:34
*** kiran-r has quit IRC13:37
*** coolsvap is now known as coolsvap|afk13:37
*** apuimedo has joined #openstack-nova13:37
dikonoorsdague : could you take a look at https://review.openstack.org/#/c/178059/ ?13:40
dikonoorsdague:This needs another +213:40
*** thangp has joined #openstack-nova13:42
sdaguesuperdan: you want to fast approve https://review.openstack.org/#/c/179757/ ? 0 nova jobs will pass until it's in13:43
sdagueand apparently we are for lack of nova cores this morning13:43
superdanI'm easy to trick before I've had coffee13:43
* superdan gets coffee13:44
*** jeblair_ is now known as jeblair13:44
*** superdan is now known as dansmith13:44
*** kodoku has joined #openstack-nova13:45
*** ajayaa has joined #openstack-nova13:46
*** mriedem has joined #openstack-nova13:46
*** dboik has joined #openstack-nova13:46
kodokuHi, I try to change 'block_device_allocate_retries' in "/nova/compute/manager.py" because when I launch instance with empty volume, cinder make a long time for create volume (35 min ==> Vmware backend). I edit default with 3600 but instance is in error state after 5 min.13:47
kodokuMaybe I need to execute command for change this delay ?13:47
*** lascii is now known as alaski13:47
dansmithkodoku: please see topic13:48
kodokudansmith ok sorry13:49
mosheledansmith: hi13:53
*** emagana has quit IRC13:53
dansmithmoshele: hi13:53
chhavidansmith: hi13:53
chhaviI have submitted one review13:53
chhavihttps://review.openstack.org/#/c/179697/13:53
sdaguedansmith: thanks13:54
chhavithis is with respect to the changes which you have done earlier to include  _ComputeV4Proxy13:54
mosheledansmith: Mellanox CI can we start it on nova as none voting ?13:54
*** kodoku has left #openstack-nova13:54
chhavithere is a small change needed for detach_volume definition. please check13:54
mriedemchhavi: do you have a stack trace you can put in the bug report?13:55
mriedemchhavi: your commit message could use some work too13:55
dansmithmoshele: it's okay with me, as I said in my email.. I wish some other people would reply there.. I think you need two of us, right?13:55
dikonoormriedem :https://review.openstack.org/#/c/178059/ - could you please take a look and approve :)13:56
dansmithmriedem: you okay with the mellanox CI bits? and if not, can you reply on their thread?13:56
chhaviok13:56
mriedemjust got online13:56
mriedemdid someone chum the waters here?13:56
*** xuhanp_ has joined #openstack-nova13:57
*** xuhanp has joined #openstack-nova13:57
*** atuvenie_ has joined #openstack-nova13:57
*** atuvenie has quit IRC14:00
mriedemdansmith: i'll look at the ML thread14:00
mriedemgaryk: looks like there is a race in a vmware unit test http://logs.openstack.org/97/179697/4/check/gate-nova-python27/6b80015/console.html#_2015-05-04_10_24_10_90714:01
mriedemthat's hitting multiple changes14:01
sdaguemriedem: if you'd been logged in, you would see the scrollback :)14:01
openstackgerritVictor Stinner proposed openstack/nova: Rename iteritems() methods to items() on Python 3  https://review.openstack.org/17978314:01
openstackgerritVictor Stinner proposed openstack/nova: Replace dict.iteritems() with six.iteritems(dict)  https://review.openstack.org/17978414:01
sdaguemriedem: https://review.openstack.org/#/c/179757/14:01
mriedemsorry, slept in14:02
*** atuvenie_ has quit IRC14:02
mriedemkid puked 3 times throughout the night14:02
sdagueoof, sorry14:02
*** atuvenie_ has joined #openstack-nova14:02
sdagueanyway, dansmith fast approved it14:02
mriedemi'm not the one shampooing the carpets today14:02
dansmithmy kid puked zero times throughout the night14:02
dansmithjust sayin'14:02
sdaguemriedem: if you want to build an ER signature for it, that would be cool14:02
sdaguebut it should be merged shortly14:03
mriedemyeah can do14:03
*** tojuvone_ has quit IRC14:03
*** dave-mccowan has quit IRC14:03
*** sahid has quit IRC14:04
*** zz_naotok has quit IRC14:04
*** sahid has joined #openstack-nova14:05
*** zz_naotok has joined #openstack-nova14:06
haypo^^ i just pushed my 2 first patches for Python 3 ;-) let's have fun with Python 3: the second patch changes 111 files :-) hopefully, i wrote a tool to generate it ;)14:07
*** eharney has joined #openstack-nova14:08
*** otter768 has joined #openstack-nova14:08
sdaguehaypo: for https://review.openstack.org/#/c/179783/1/nova/cells/utils.py,cm why not just alias items to iteritems ?14:09
*** dguitarbite has quit IRC14:09
garykmriedem: no, that was just due to a update in the requests library14:09
sdagueand not care about which python version, you'll just implement the interface for both14:09
garykmriedem: please see https://review.openstack.org/17975714:10
hayposdague, dict has no iteritems method on python 3, i prefer to mimick the exact dict API14:10
*** cfriesen_ has joined #openstack-nova14:11
hayposdague, this change is related to its following change to use six.iteritems() everywhere14:11
sdagueyeh, seems silly. Though this incompat issue with python3 also seems silly.14:12
openstackgerritJens Rosenboom proposed openstack/nova: Allow server search option ip6 for non-admin  https://review.openstack.org/17956914:12
*** otter768 has quit IRC14:13
hayposdague, it would be strange that only NovaObjectDictCompat has an iteritems() method, whereas all other dictionaries don't have this method, it may introduce bugs during refactoring14:15
hayposdague, IMO it's safer to use six.iteritems() everywhere14:15
*** pkoniszewski has quit IRC14:15
*** dave-mccowan has joined #openstack-nova14:15
*** salv-orlando has joined #openstack-nova14:16
*** garyk has quit IRC14:17
sdagueI guess, the blanket use of six just gets annoying. I don't suppose they built a compat layer for viewitems in python 3 ?14:17
hayposdague, the whole purpose of python 3 was to design a new and better language, sometimes seen as simpler and easier to learn, so it doesn't contain compat layers14:19
*** exploreshaifali has joined #openstack-nova14:19
*** signed8bit has joined #openstack-nova14:19
hayposdague, the pain point occurs when you try to write code compatible with python 2 and python 3. if you write python 3 only code, it should look better ;)14:20
sdaguebut, when that function was brought back to python 2.7 under a different name, it would have been really easy to also put those new names in python 314:20
sdaguehaypo: right, so come back in 10 years :)14:20
*** vladikr has joined #openstack-nova14:20
sdagueanyway, just frustrated that python 3 didn't think to provide some of this compatibility itself14:21
hayposdague, six is a nice way to write code compatible with python 2 and python 3 with little efforts14:23
hayposdague, it's not part of the python stdlib, but it would not be possible since python 2.6 don't accept new features (nor python 2.7)14:23
*** irenab has quit IRC14:23
dansmithit's really not14:28
dansmithit's terrible14:28
dansmithit's good that there's a way to do it, but it's far from "nice" IMHO :)14:29
sdaguedansmith: ++14:30
*** dboik_ has joined #openstack-nova14:30
edleafedansmith: sdague: haypo: OTOH, the idea was not to make it easy to stay with 2.x14:31
sdagueespecially in this case where using python 2.7 viewitems would be behaviorly identical to python 3 items, it was backported. you'd think that they'd have aliased that function so you could move forward smoothly14:31
dansmithedleafe: that's very clear, but not really a factor in my evaluation of niceness :)14:31
sdagueedleafe: right, that's clear. That worked out really well in getting people to move :)14:32
edleafeSomehow seeing 'dansmith' and 'niceness' in the same sentence feels odd14:32
sdagueheh14:32
sdagueedleafe: ++14:32
*** salv-orlando has quit IRC14:32
*** armax has joined #openstack-nova14:33
edleafesdague: yeah, the problem was that 3.x didn't offer anything especially wonderful to justify the pain of moving14:33
bauzasI have to agree with sdague, having .items() returing different results if py3 instead of py2 seems just errorprone14:33
bauzasbut unfortunately, life is bad - we have to use six for that14:33
edleafebauzas: but for people working solely in 2.x, having one .items() act differently than every other .items() is worse14:33
*** gokrokve_ has joined #openstack-nova14:34
*** dboik has quit IRC14:34
bauzasedleafe: my point is that iterators are done differently in Py3, but I think it would have been better to have some different methods than the ones already in use IMHO14:34
sdagueedleafe: that's fine, but it still doesn't excuse the lack of an alias for viewitems14:34
edleafesdague: true14:35
sdaguethat's just people not thinking through actual code base usage when they did the backport14:35
bauzasbut as we can't just use the delorean to ask for that, it needs us to use six14:35
*** zz_jgrimm is now known as jgrimm14:35
edleafesdague: it's surprising that they even added that back to 2.7, as they are pretty dead set against enhancing anything in 2 anymore14:36
*** shyama has quit IRC14:36
sdagueedleafe: it was 5 years ago or something14:37
*** gokrokve has quit IRC14:37
*** jgrimm is now known as zz_jgrimm14:38
sdagueanyway, beer rant14:38
edleafesdague: exactly14:39
alex_xujohnthetubaguy: do we really want to deprecate os-getSerialConsole os-getVNCConsole and so on since microversion 2.4? The spec said https://review.openstack.org/#/c/163790 deprecate all of those API in 'proposed change' section, but not mentioned at 'REST API Impact' section.14:39
edleafesdague: I've had way too many of these discussions at the bars during PyCons14:39
sdagueedleafe: sadly no on listened? :)14:39
sdaguehttp://blog.brentlaabs.com/2015/05/three-tales-of-second-system-syndrome.html - was an interesting read from this morning14:39
bauzasalex_xu: as said previously, it's holiday for UK peers14:40
edleafesdague: I probably wasn't at my most convincing14:40
alex_xubauzas: ah, thanks14:40
*** mtanino has joined #openstack-nova14:40
alex_xubauzas: just know John at UK also14:40
*** unicell1 has joined #openstack-nova14:40
*** packet has joined #openstack-nova14:41
*** unicell has quit IRC14:42
*** emagana has joined #openstack-nova14:42
openstackgerritjichenjc proposed openstack/nova-specs: New API for get service info from id  https://review.openstack.org/17241214:43
*** claudiub_ has quit IRC14:44
*** xuhanp has quit IRC14:49
*** xuhanp_ has quit IRC14:49
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Eventlet supports python logging directly  https://review.openstack.org/17980114:49
*** tonytan4ever has joined #openstack-nova14:52
haypodansmith, by "nice", i mean that you don't have to duplicate your whole application to have a py2 version and a py3 version, only a few lines need to be modified to handle subtle differences between py2 and py314:52
openstackgerritVictor Stinner proposed openstack/nova: Rename iteritems() methods to items() on Python 3  https://review.openstack.org/17978314:53
openstackgerritVictor Stinner proposed openstack/nova: Replace dict.iteritems() with six.iteritems(dict)  https://review.openstack.org/17978414:53
*** pankaj2934 has joined #openstack-nova14:53
dansmithpick a different word then :)14:53
dansmith"bare effing minimum" might be appropriate :)14:53
haypodansmith, six is convinient14:53
*** alexpilotti has joined #openstack-nova14:53
*** Murali has quit IRC14:54
*** atuvenie has joined #openstack-nova14:54
dansmitha 7/11 is convenient, six is a necessary evil14:55
*** gokrokve_ has quit IRC14:55
*** gokrokve has joined #openstack-nova14:55
*** links has quit IRC14:56
*** garyk has joined #openstack-nova14:57
*** gokrokve has quit IRC14:57
*** atuvenie_ has quit IRC14:57
*** gokrokve has joined #openstack-nova14:58
*** sdake has joined #openstack-nova15:01
*** ijw has joined #openstack-nova15:02
*** sdake__ has joined #openstack-nova15:03
*** sdake_ has quit IRC15:03
*** ajayaa has quit IRC15:06
*** sdake has quit IRC15:07
*** ociuhandu_ has joined #openstack-nova15:07
mriedemmoshele: for the mellanox ci,15:10
mriedemthere are tests that live outside of tempest right?15:10
moshelemremdem: no we are using the tempest scenarios with port direct15:11
*** zzzeek has joined #openstack-nova15:11
mriedemmoshele: there is one scenario test here http://144.76.193.39/ci-artifacts/Check-MLNX-Nova-ML2-Sriov-driver_PORT_20150503_1854/testr_results.html.gz15:12
mriedemthe rest are api tests15:12
*** aix has quit IRC15:13
openstackgerritjichenjc proposed openstack/nova-specs: New API for get service info from id  https://review.openstack.org/17241215:13
moshelemriedem: yes this is the important  one15:13
*** iamjarvo has joined #openstack-nova15:13
moshelemriedem: I with a customer I will ping you later15:13
*** cfriesen_ is now known as cfriesen15:14
*** claudiub has joined #openstack-nova15:14
*** aix has joined #openstack-nova15:15
cfriesenany chance of getting someone to look at https://review.openstack.org/#/c/170780/ (which mriedem has already +2'd) and https://review.openstack.org/#/c/175542/   (which has a couple of +1s).  Both are simple and uncontroversial.15:15
*** jcoufal_ has quit IRC15:18
pankaj2934 is there a way to set a proper hostname to a instance and haev a A record too in infoblox for the instacne15:19
dansmithpankaj2934: please see topic15:20
mriedemdansmith: moshele: +1 from me on mellanox ci non-voting15:20
dansmithmriedem: what about that scenario test?15:20
dansmithanteaya: what do we need to do to grant them comment perms?15:20
mriedemdansmith: i assumed this was like the intel pci ci where the tests are not in tempest but some other private repo15:21
dansmithmriedem: okay, yeah, I don't think so15:21
anteayadansmith: not a thing, other than tell them they may15:21
dansmithanteaya: ah, okay, well, we have two cores +1, and nobody else seems to care, so I guess we're good15:22
anteayaso that if I ever come and ask does foo ci have permission to comment on nova patches you can say "yes"15:22
anteayadansmith: thank you15:22
anteayathat works for me15:22
dansmithmoshele: I think you're good to go15:22
*** sudipto has joined #openstack-nova15:22
mosheledansmith: thanks15:22
*** dsanders has joined #openstack-nova15:23
*** penick has joined #openstack-nova15:26
*** pankaj2934 has left #openstack-nova15:28
*** apuimedo has quit IRC15:29
mriedemjaypipes: johnthetubaguy: i'm looking at this https://review.openstack.org/#/c/107185/15:30
mriedemthat merged in kilo, but the spec was never approved or re-proposed for kilo from what i can see https://review.openstack.org/#/c/103945/15:31
*** ociuhandu_ has quit IRC15:31
mriedemam i missing something?15:31
mriedemi don't see a spec out here either http://specs.openstack.org/openstack/nova-specs/specs/kilo/approved/15:31
mriedemwe have a customer asking for this but i don't know what kind of half baked state it's in15:31
mriedemalexpilotti: claudiub: ^15:31
*** mtanino has quit IRC15:33
dansmithalaski: on this: https://review.openstack.org/#/c/177470/6/nova/compute/manager.py,cm15:34
dansmithalaski: we know that when the race happens, we get the save() as part of the cleanup and reschedule15:34
dansmithalaski: are we sure that we also get it in the normal success case and the non-race reschedule case?15:35
*** vilobhmm1 has joined #openstack-nova15:35
dansmithalaski: I would assume we have to else it wouldn't have passed your testing, but looking at the updated unit tests just now made me wonder15:35
dansmithJoshNang: jroll ^15:35
alaskiI dunno, I'll have to check15:35
*** yamahata has joined #openstack-nova15:35
sahidso we don't have meeting today15:36
sahidhm forget about me :)15:37
*** vilobhmm1 has quit IRC15:37
JoshNangdansmith: i'll check. i assume so because it was working with and without lots of reschedules.15:38
dansmithright15:38
dansmithI just want to make sure we're not opening a new race where it  might *not* get saved15:38
alaskithe last thing we do before a reschedule is instance.save()15:39
*** mfisch has quit IRC15:40
alaskidansmith: ^, plus a normal success case doesn't really matter as we'll never use that value again.  Though it would be good to ensure it's consistent anyways15:40
*** Longgeek_ has joined #openstack-nova15:40
*** ildikov has quit IRC15:41
openstackgerritJens Rosenboom proposed openstack/nova: Allow server search option ip6 for non-admin  https://review.openstack.org/17956915:41
*** mfisch has joined #openstack-nova15:41
*** salv-orlando has joined #openstack-nova15:41
alaskidansmith: but there is an instance.save after a successful return from the virt driver15:41
*** dsanders has quit IRC15:41
*** mfisch is now known as Guest6519715:41
*** Longgeek has quit IRC15:42
dansmithalaski: okay cool15:42
dansmithjust wanted to make sure we weren't focusing on the 10% and losing sight of the 90% :)15:42
*** penick has quit IRC15:42
alaskidansmith: yeah, definitely15:43
fricklerregarding https://launchpad.net/bugs/1450859, can someone explain to me why the API should simple ignore invalid options? the issue would have been much more obvious if there had been some error returned instead of simply returning the complete list15:43
openstackLaunchpad bug 1450859 in OpenStack Compute (nova) "list servers always returns all with 'ip6' filter" [Undecided,In progress] - Assigned to Dr. Jens Rosenboom (j-rosenboom-j)15:43
*** oro has quit IRC15:43
alaskiJoshNang: dansmith we should probably comment those saves, # This needs to be here because blah15:43
dansmithalaski: yeah, so we don't remove them later for some reason15:44
JoshNangalaski: ++. i can add more comments later today15:45
*** ijw has quit IRC15:46
*** mwagner_lap has quit IRC15:47
alaskidansmith: also, 4 minutes, not bad :)15:49
openstackgerritVictor Stinner proposed openstack/nova: Replace iter.next() with next(iter)  https://review.openstack.org/17982115:49
dansmithalaski: heh15:49
*** htruta has quit IRC15:50
*** htruta has joined #openstack-nova15:53
*** mtanino has joined #openstack-nova15:54
openstackgerritJens Rosenboom proposed openstack/nova: Allow server search option ip6 for non-admin  https://review.openstack.org/17956915:54
*** ijw has joined #openstack-nova15:54
*** pixelbeat has quit IRC15:54
*** armax has quit IRC15:56
*** dguitarbite has joined #openstack-nova15:59
*** gokrokve_ has joined #openstack-nova16:01
*** gokrokve_ has quit IRC16:01
*** Guest65197 has quit IRC16:02
*** emagana has quit IRC16:03
*** artom has joined #openstack-nova16:03
*** gokrokve has quit IRC16:04
*** emagana has joined #openstack-nova16:04
*** atuvenie has quit IRC16:04
*** dulek has quit IRC16:05
openstackgerritVictor Stinner proposed openstack/nova: Drop L from literal integer numbers for Python 3  https://review.openstack.org/17982816:05
*** mikeit has quit IRC16:06
*** unicell1 has quit IRC16:06
*** gokrokve has joined #openstack-nova16:06
*** gokrokve has quit IRC16:06
*** shyama has joined #openstack-nova16:08
*** angdraug has joined #openstack-nova16:08
*** mpavone has quit IRC16:08
*** yamahata has quit IRC16:09
*** otter768 has joined #openstack-nova16:09
openstackgerritjichenjc proposed openstack/nova-specs: flavor access create should check public/private  https://review.openstack.org/17238816:09
*** ijw has quit IRC16:11
*** ijw has joined #openstack-nova16:11
*** markus_z has quit IRC16:11
*** salv-orlando has quit IRC16:12
*** Nic has joined #openstack-nova16:12
openstackgerritDan Smith proposed openstack/nova: Move our ObjectListBase to subclass from the Oslo one  https://review.openstack.org/17399516:14
openstackgerritDan Smith proposed openstack/nova: Start the conversion to oslo.versionedobjects  https://review.openstack.org/17347916:14
dansmiththese have been sitting for a while and could use some eyes ^16:14
*** otter768 has quit IRC16:14
dansmithpretty easy stuff, just some beginnings of cutting over to o.vo16:14
jaypipesmriedem: hmm, I'16:16
*** ijw has quit IRC16:16
jaypipesm really not sure about that one. I remember reviewing the code, but never lookied at a spec.16:16
*** romainh has left #openstack-nova16:18
*** zz_jgrimm is now known as jgrimm16:18
mriedemjaypipes: i posted to the ML where alexpilotti responded16:19
*** sahid has quit IRC16:19
*** jichen has quit IRC16:20
openstackgerritDan Smith proposed openstack/nova: Move our ObjectListBase to subclass from the Oslo one  https://review.openstack.org/17399516:24
openstackgerritDan Smith proposed openstack/nova: Start the conversion to oslo.versionedobjects  https://review.openstack.org/17347916:24
dansmithany idea why the requirements check in jenkins doesn't report in the normal form such that it doesn't show up in the rollup ?16:24
dansmithclarkb: sdague ^16:24
*** tjones1 has joined #openstack-nova16:24
clarkbdansmith: beause it has that explanatory string explaining what the failure means16:25
clarkbdansmith: many people were confused (and still are) when that job fails because they don't know about our reqs enforcement16:25
dansmithclarkb: hmm, okay, well, it means that it's less visible when it fails if you're looking at the report16:25
dansmithclarkb: couldn't we have it just fail like normal and emit a blob of text in the job output explaining?16:26
*** iamjarvo has quit IRC16:26
dansmithmost people click on failed jobs to go look at the console output to start I think16:26
clarkbwe could16:27
clarkbits probably worth trying16:27
dansmithor I could stop complaining :)16:27
dansmithwe made a big stink about proper external CI formatting, yet we break it ourselves16:27
dansmithand when I say "we" I mean "sdague" in the first case :)16:27
clarkbthough I am not sure how many people open failed jobs before `recheck`, but current system has problems too so its not like we are replacing a perfect thing16:27
dansmiththose people should be tarred and feathered16:28
cfriesendansmith: I'll open them, but I have to say that I'm guilty of not creating submitting new bug signatures for elastic recheck16:30
*** ajayaa has joined #openstack-nova16:30
dansmithcfriesen: that's different than just rechecking your patch a few times before bothering to see why they're failing16:30
dansmithhell hath many rings, as you know :)16:31
cfriesen:)16:31
*** ijw has joined #openstack-nova16:33
*** yamahata has joined #openstack-nova16:33
*** mfisch has joined #openstack-nova16:34
*** mfisch is now known as Guest1950216:34
kashyapMaybe the no. of rechecks (after the first one) should be rate-limited if there's no details of investigation :-P16:36
*** moshele has quit IRC16:36
*** unicell has joined #openstack-nova16:37
sdaguekashyap: you'd need a lot of infrastructure to support that16:38
*** baoli has quit IRC16:38
kashyapsdague: Yeah. What made me say that comment was I was wondering how much pain does infra goes through when there's too many no. of rechecks16:38
sdagueinfra doesn't have pain with it16:39
dansmithand that's not the problem with it, it's that people grind reviews until they land by chance16:39
sdagueall the developers do if all the nodes get taken up with un passable jobs16:39
dansmithinstead of finding that three runs of their stuff failed twice for real reasons16:39
sdagueyep16:39
kashyapI thought it's kind of a DOS-attach where all the "unlimited" CI infra machines will be used up.16:40
dansmiththere's that too16:40
dansmithbut that's not the worst result of the behavior, IMHO16:40
*** flashgordon is now known as jogo16:40
*** emagana has quit IRC16:41
sdagueyeh, we get bugs landed where people recheck grind for their patches to land when they get lucky and don't hit the race twice in a row.16:41
*** rdo has quit IRC16:41
dansmithmaybe we should require a corp-like recertification quiz each year to have recheck rights :D16:41
dansmith"Alice submitted a patch and jenkins gave it a -1. What should Alice do?"16:42
dansmith1. Recheck and go get ice cream while it runs16:42
dansmith2. Say "eff it, I'm important" and recheck until it goes in16:42
dansmith3. Call Bob and ask for it to be pushed in because it's obviously correct16:42
dansmith4. Read the job report16:42
*** emagana has joined #openstack-nova16:42
*** neelashah has quit IRC16:43
*** redbeard has quit IRC16:43
mtreinishdansmith: I would probably fail, because between ice cream and reading the job results ice cream wins16:44
dansmithmtreinish: see, that's a trap to catch the weak ones16:45
sdagueheh16:46
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Eventlet supports python logging directly  https://review.openstack.org/17980116:48
*** gokrokve has joined #openstack-nova16:50
*** Nic has quit IRC16:51
*** Nic has joined #openstack-nova16:52
sdaguedansmith / mriedem - https://review.openstack.org/#/c/176806/ - the nova backport of the oslo sync so that services can shut down16:53
sdaguemriedem: I think you have +2 now, right?16:53
kashyapdansmith: I can't recall the xkcd (or somewhere similar) where it says you can convince any technical person to pay attention if you involve the "Alice and Bob" characters :-)16:54
mriedemyes now i do16:54
sdaguenow you both +2ed it :)16:54
sdagueone of you want to +A?16:54
*** rdo has joined #openstack-nova16:54
*** doron_afk has quit IRC16:56
*** sdake__ is now known as sdake16:58
edleafedansmith: why 'ovoo' instead of 'ovo'?16:58
edleafedansmith: for oslo.versionedobjects16:58
dansmithedleafe: where?16:59
edleafehttps://review.openstack.org/#/c/17347916:59
edleafehttps://review.openstack.org/#/c/173479/5/nova/objects/base.py,cm L2816:59
*** alanf-mc has joined #openstack-nova17:00
dansmithedleafe: in lots of nova code, we import base as obj_base17:00
dansmithedleafe: so I was probably adding "ob" that after ovo in that sort of idea17:00
dansmithdoesn't matter of course17:00
*** dsanders has joined #openstack-nova17:01
*** whenry has joined #openstack-nova17:01
*** neelashah has joined #openstack-nova17:02
edleafedansmith: yeah, nbd - just curious as I read the code17:02
*** baoli has joined #openstack-nova17:03
dimsedleafe: its like evoo it's good for you!17:04
*** dboik_ has quit IRC17:04
*** chhavi has quit IRC17:04
*** dboik has joined #openstack-nova17:05
*** baoli has quit IRC17:06
*** zul has quit IRC17:06
*** baoli has joined #openstack-nova17:06
*** zul has joined #openstack-nova17:07
*** Longgeek_ has quit IRC17:10
openstackgerritgaryk proposed openstack/nova: VMware: add support for NFS 4.1  https://review.openstack.org/17984217:10
*** achanda has joined #openstack-nova17:11
*** dikonoor has quit IRC17:12
*** vilobhmm1 has joined #openstack-nova17:12
*** ajayaa has quit IRC17:13
mriedemsdague: if nbd is so problematic, should we put a warning in the nbd code in nova that says use libguestfs instead?17:14
*** jistr has quit IRC17:15
*** Guest19502 has quit IRC17:15
*** Guest19502 has joined #openstack-nova17:15
*** patrickeast has joined #openstack-nova17:18
mriedemespecially if using neutron since you should assume neutron is running with network namespaces17:19
dimshaypo: you have local changes to run py34 with tox?17:20
*** Sukhdev has joined #openstack-nova17:22
*** armax has joined #openstack-nova17:23
*** ssurana has joined #openstack-nova17:24
* bauzas just standing up after 4 hours reworking on Summit session slides17:24
*** ociuhandu has quit IRC17:24
*** Guest19502 is now known as mfisch17:26
haypodims, to run tox -e py34, i commented MySQL-python, oslo.vmware and suds in test-requirements.txt17:26
cfriesenjust had gate-tempest-dsvm-postgres-full fail for a test of mine...failure seems unrelated to the code change.  This a known issue?    http://logs.openstack.org/80/170780/11/gate/gate-tempest-dsvm-postgres-full/b2e3fd4/console.html17:26
*** gokrokve has quit IRC17:27
haypodims, but with the current master branch, testr is simply unable to import python modules, and so fail badly17:27
mriedemcfriesen: http://logs.openstack.org/80/170780/11/gate/gate-tempest-dsvm-postgres-full/b2e3fd4/console.html#_2015-05-04_17_05_04_32317:27
mriedemcheck the n-cpu logs for errors17:27
mriedemcfriesen: http://logs.openstack.org/80/170780/11/gate/gate-tempest-dsvm-postgres-full/b2e3fd4/logs/screen-n-cpu.txt.gz?level=TRACE#_2015-05-04_16_49_01_94817:28
*** angdraug has quit IRC17:28
haypodims, you need at least https://review.openstack.org/#/c/179828/ to fix syntax errors17:28
mriedemcfriesen: so you have "libvirtError: internal error: received hangup / error event on socket"17:28
mriedemso plug that into logstash.openstack.org17:28
mriedemcfriesen: logstash shows it's not just your change http://goo.gl/PIVpb617:29
*** dboik_ has joined #openstack-nova17:29
mriedemi don't see any nova bugs specifically with "libvirtError: internal error: received hangup / error event on socket" in them17:30
*** salv-orlando has joined #openstack-nova17:30
cfriesenmriedem: thanks for the lesson17:30
*** armax has quit IRC17:31
mriedemcfriesen: well, https://docs.google.com/presentation/d/1xUxNeQrA-1nK1T3VD1JGsWNBD8In7qI3SXXwQYgOr-Y/edit?usp=sharing17:31
haypodims, i'm working on the first part, the most boring patches (iteritems(), next(), unicode(), replace 123L with 123), with these patches, we will be able to work on more interesting patches like fixing bytes/unicode issues (and being able to run tox)17:31
mriedemi might use this actually :)17:31
*** ociuhandu has joined #openstack-nova17:31
mriedemmtreinish: ^17:31
*** IanGovett has quit IRC17:31
*** IanGovett has joined #openstack-nova17:31
cfriesenmriedem: I found the right person then. :)17:32
*** dguitarbite has quit IRC17:32
openstackgerritVictor Stinner proposed openstack/nova: Replace unicode with six.text_type  https://review.openstack.org/17985017:33
*** dboik has quit IRC17:34
*** harlowja has joined #openstack-nova17:34
mriedemcfriesen: https://bugs.launchpad.net/nova/+bug/145150617:35
openstackLaunchpad bug 1451506 in OpenStack Compute (nova) "spawn failed with "libvirtError: internal error: received hangup / error event on socket" in the gate" [Undecided,New]17:35
dansmithalaski: it's run_instance() we get to delete now, right?17:36
*** baoli has quit IRC17:36
dansmithalaski: it wasn't marked for deprecation so I just want to be sure17:36
alaskidansmith: yep.  hmm, it should have been17:36
dansmitht'wasn't17:36
dansmithalthough it's not in rpcapi17:36
dansmithalaski: will be looking for you to review the drop patch for cruft and issues :)17:37
*** baoli has joined #openstack-nova17:37
*** thomasem has quit IRC17:37
alaskidansmith: I will review with a vengeance17:37
cfriesenmriedem: thanks. given that it's not blocking everything, do I just do a recheck on my patch?17:37
alaskidansmith: https://github.com/openstack/nova/tree/master/nova/compute#L253217:38
mriedemcfriesen: yeah, there is actually another test that failed in there, a timeout waiting for a state change17:38
mriedemi didn't dig into that one17:38
*** bkopilov has quit IRC17:38
dansmithalaski: hmm, I hope the versions are off on that17:38
*** baoli has quit IRC17:38
*** salv-orlando has quit IRC17:39
*** baoli has joined #openstack-nova17:39
alaskidansmith: I think I wrote that when we reliably incremented each release17:39
dansmithalaski: yeah, we removed use of it in 3.x so when we drop 3.x compat, we can drop the function from manager (i.e. 4.x not 5.x17:39
dansmithhmm, okay17:39
dansmithanyway17:39
alaskiso I really should have written Kilo/L or something17:40
*** gokrokve has joined #openstack-nova17:41
dansmithwell, writing a version number is more helpful17:41
dansmithbecause we have things in the code like:17:41
dansmith# NOTE: remove in grizzly17:41
alaskiheh17:41
*** russellb has quit IRC17:42
*** alanf-mc has quit IRC17:42
mriedemremove in manwich17:42
*** jogo has quit IRC17:42
mriedemwe might not call it manwich though17:42
dansmithmanwich would be awesome,17:43
alaski+117:43
dansmithbut I bet people outside the US wouldn't think much of it17:43
mriedemit would be a cheap mess17:43
*** alanf-mc has joined #openstack-nova17:43
dansmithand it's probably a gender sin17:43
mriedemladywich?17:43
dansmithso it should be personwich, which isn't good for lany other releases17:43
dansmithand kinda loses the snazz17:43
mtreinishmriedem: what am I supposed to be looking at?17:43
*** jogo has joined #openstack-nova17:43
*** russellb has joined #openstack-nova17:43
mriedemmtreinish: heh - i found an example for the e-r workflow,17:44
mriedemfrom 'what failed' to e-r query17:44
mtreinishmriedem: ah cool17:44
*** gokrokve has quit IRC17:45
*** gokrokve has joined #openstack-nova17:45
*** mwagner_lap has joined #openstack-nova17:45
*** haypo has quit IRC17:48
*** apuimedo has joined #openstack-nova17:48
sdaguemriedem: we should actually deprecate file injection entirely17:50
sdaguewe actually don't use it in most of the upstream testing any more17:50
sdaguebut the whole mechanism should be deprecated and removed17:50
dansmithand then spit upon with disdain17:51
*** shyama has quit IRC17:52
mriedemanything stopping us from doing that?17:52
*** ZZelle has quit IRC17:53
*** ZZelle has joined #openstack-nova17:53
alaskisome providers still rely on it :|17:55
alaskithough with a deprecation cycle, I would still like to see it removed17:55
mriedemyeah so assuming deprecation cycle,17:55
mriedemshould we just put something in the operators mailing list seeing who screams if we say we're deprecating in L and removing in manwich?17:56
alaskiseems reasonable17:56
mriedemnot it?17:56
*** aix has quit IRC17:58
*** harlowja has quit IRC17:59
*** harlowja_ has joined #openstack-nova17:59
*** tonytan4ever has quit IRC17:59
alaskiI'm on both sides of the issue so I'm not sure if I'm the best person, but I could float the idea.  I'd want to understand the actual issues more besides that it's philosophically terrible17:59
*** thomasem has joined #openstack-nova17:59
mriedemthat's why i'm not volunteering to write that up17:59
mriedemi don't know enough about the details17:59
dansmithphilosophically terrible isn't enough?17:59
dansmithIMHO,17:59
alaskidansmith: for me yes, operators probably not18:00
dansmiththings like this are part of the inward-facing "API" to to speak,18:00
dansmithwhich is the environment of the guest18:00
dansmithit's a HUGE part of actual portability between clouds18:00
dansmithwhich is one reason I want a minimum level of functionality for each driver18:00
*** armax has joined #openstack-nova18:00
*** kaufer1 has joined #openstack-nova18:00
dansmithhowever, file injection fits into this I think, where it's a terrible thing to do to the guest and, like our external api, we can help drive consistency, and consistency in the right direction, by removing it18:01
jogojohnthetubaguy: https://review.openstack.org/#/c/179553/ FYI18:01
*** emagana has quit IRC18:02
alaskidansmith: thanks, that helps18:02
*** kaufer has quit IRC18:02
mriedemwhen my manager munges my nick in irc and i don't get notified, it means i never saw the issue right?18:03
*** exploreshaifali has quit IRC18:03
*** exploreshaifali has joined #openstack-nova18:03
*** exploreshaifali_ has joined #openstack-nova18:03
*** exploreshaifali has quit IRC18:04
*** exploreshaifali_ has quit IRC18:04
*** exploreshaifali has joined #openstack-nova18:04
dansmithmriedem: correct18:05
*** EmilienM is now known as EmilienM|mtg18:05
*** sandywalsh_ has joined #openstack-nova18:05
*** sandywalsh has quit IRC18:06
*** promulo has quit IRC18:06
*** jcoufal has joined #openstack-nova18:08
*** promulo has joined #openstack-nova18:09
*** vilobhmm1 has quit IRC18:09
*** packet has quit IRC18:09
*** Sukhdev has quit IRC18:09
*** vilobhmm1 has joined #openstack-nova18:09
*** otter768 has joined #openstack-nova18:10
*** emagana has joined #openstack-nova18:11
*** Marga_ has joined #openstack-nova18:12
*** exploreshaifali has quit IRC18:13
*** otter768 has quit IRC18:15
sdaguedansmith: so, you sound like you have the strongest opinions, I nominate you to handle this deprecation :)18:20
*** pixelbeat has joined #openstack-nova18:20
mriedem2nd18:20
dansmithno18:21
sdaguedansmith / alaski either of you want to fix stable/kilo unit tests with a magic +A - https://review.openstack.org/#/c/179805/218:21
dansmiththanks though :)18:21
openstackgerritJoe Gordon proposed openstack/nova: Mark ironic credential config as secret  https://review.openstack.org/17985718:21
sdaguedansmith: hey, I did ec2, that was much harder to navigate18:21
*** EmilienM|mtg is now known as EmilienM18:21
mtreinishsdague: +A18:22
sdagueif it hadn't been my original patch I'd have fast approved it, but I felt weird doing so, so thanks mtreinish18:23
alaskisdague: I don't have +2 on stable18:25
sdaguealaski: oh, right18:25
alaskibut +2 from the sidelines18:25
*** jichen has joined #openstack-nova18:27
*** Sukhdev has joined #openstack-nova18:29
dansmithalaski: guess what sucks...18:30
*** sdake_ has joined #openstack-nova18:30
alaskidansmith: I can't even narrow it down for a guess18:31
dansmithwell,18:31
dansmithlet me do some more digging first18:31
*** dboik_ has quit IRC18:31
dansmithyeah18:33
*** tonytan4ever has joined #openstack-nova18:33
dansmithalaski: we have a bunch of tests for things that use run_instance() to poke a condition18:33
dansmithwhich puts me at pretty much FML status given where I am in this rpc cleanup18:33
*** sdake has quit IRC18:34
*** sdake_ is now known as sdake18:34
alaskidansmith: and these conditions still need poking after run_instance is removed?18:35
dansmithalaski: yeah, they're testing things like instance lock, file injection, ect18:35
dansmither, etc18:35
alaskidansmith: damn18:35
dansmithI guess I should refactor these in a patch before the compute rpc cleanup18:36
dansmithbecause it's already large18:36
dansmith(although oh-so-sweet)18:36
alaskiI don't think I'll have time to help today, but I could take a crack at porting some tests if that would help this move forward18:37
dansmithno,18:37
*** boris-42 has joined #openstack-nova18:37
dansmithI'm not asking you to, I'm just bitching an moaning18:37
dansmithact like you feel sorry for me and that's all I need :P18:37
alaskicool, well complain on.  I know it's bad, that's why I replaced it :)  so I feel your pain18:38
dansmithI just hate finding debt under rocks :)18:38
dansmithso far, the cleanup is:  3 files changed, 103 insertions(+), 1465 deletions(-)18:39
alaskithat's excellent18:39
alaskideleted code is the best code18:39
*** igordcard has joined #openstack-nova18:40
dansmithgenerally yeah18:40
*** lpetrut has joined #openstack-nova18:41
*** gokrokve has quit IRC18:44
dansmithsdague: don't we have a spawn fixture that prevents actually spawning and just runs immediately?18:44
openstackgerritMichael Bayer proposed openstack/nova: Add an index to virtual_interfaces.uuid  https://review.openstack.org/15424318:45
openstackgerritMichael Bayer proposed openstack/nova: Add an index to virtual_interfaces.uuid  https://review.openstack.org/15424318:47
*** gokrokve has joined #openstack-nova18:48
*** subscope_ has joined #openstack-nova18:49
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements  https://review.openstack.org/17987018:51
*** IanGovett1 has joined #openstack-nova18:52
*** ildikov has joined #openstack-nova18:52
*** ijw has quit IRC18:52
*** oro has joined #openstack-nova18:52
sdaguedansmith: I'm not sure18:53
*** IanGovett has quit IRC18:54
lifelessdansmith: theres a fakePopen fixture in fixtures18:55
*** ijw has joined #openstack-nova18:56
dansmithlifeless: not that spwan18:57
dansmithsdague: could have sworn, but I don't see it.. so, level 3 rabbit hole :/18:57
lifelessdansmith: oh, doh :)18:57
*** VW_ has quit IRC19:02
*** kaufer has joined #openstack-nova19:02
*** VW_ has joined #openstack-nova19:02
*** kaufer1 has quit IRC19:02
*** Marga_ has quit IRC19:04
*** dboik has joined #openstack-nova19:09
*** sdake_ has joined #openstack-nova19:09
*** VW_ has quit IRC19:09
*** sandywalsh_ has quit IRC19:09
*** VW_ has joined #openstack-nova19:10
*** VW_ has quit IRC19:10
*** neelashah has quit IRC19:10
*** VW_ has joined #openstack-nova19:10
*** mjturek1 has quit IRC19:11
*** pixelbeat has quit IRC19:11
*** VW_ has quit IRC19:11
*** VW_ has joined #openstack-nova19:12
*** artom has quit IRC19:13
*** sdake has quit IRC19:13
*** dprince has quit IRC19:13
*** sandywalsh has joined #openstack-nova19:15
*** artom has joined #openstack-nova19:16
cfriesenwhat's the URL to the overview page for all the various openstack CI jobs?  my google-fu is failing.19:17
*** artom has quit IRC19:17
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Try running a single test module with python34  https://review.openstack.org/17989319:17
*** artom has joined #openstack-nova19:17
dansmithcfriesen: status.openstack.org19:18
dansmithand /zuul probably is what you want19:18
*** mjturek1 has joined #openstack-nova19:18
cfriesendansmith: thanks, that's exactly what I was looking for.  Had seen it previously and then couldn't find it again.19:18
dansmithalaski: https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute.py#L141819:19
dansmithalaski: that seems to be looking for specific behavior related to run_instance19:19
dansmithalaski: we definitely don't honor that anymore because we spawn() the work right away19:19
dansmithalaski: think there is value in trying to convert that somehow, or just drop it?19:19
*** ildikov has quit IRC19:19
*** mjturek1 has quit IRC19:19
*** mjturek1 has joined #openstack-nova19:20
*** emagana has quit IRC19:21
*** ZZelle_ has joined #openstack-nova19:22
dansmithalaski: also the _ram_fail() one a couple down from it19:22
*** salv-orlando has joined #openstack-nova19:22
*** claudiub has quit IRC19:26
*** alexpilotti has quit IRC19:29
alaskidansmith: yeah, build_and_run instance won't raise the exception.19:30
alaskibut _do_build_and_run_instance should, so you could try using that19:30
dansmithalaski: well, I'm mocking spawn_n() to just run the function and I'm not getting the same behavior19:30
*** sudipto has quit IRC19:31
dansmithI seem to be for some of the ComputeResourcesUnavailable tests, but not all19:31
dansmithI guess I'll finish them all and then see which ones don't pass and then try to decide why and what to do19:32
*** ildikov has joined #openstack-nova19:32
*** thomasem has quit IRC19:32
*** salv-orlando has quit IRC19:33
alaskidansmith: the only value I see in that test is ensuring that a resource failure hits the wrappers which log/notify/whatever19:34
alaskinot that it's raised to the caller19:34
*** mjturek2 has joined #openstack-nova19:34
dansmithokay19:35
dansmithwasn't run_instance a cast before anyway?19:35
alaskifor ram_fail the important thing is that limits are properly passed to rt.instance_claim19:35
alaskidansmith: yep19:35
dansmithokay, that gives me some leeway19:36
alaskithese tests seem very roudabout in how they're checking their conditions19:36
*** mjturek1 has quit IRC19:36
dansmithyeah19:36
*** jgrimm is now known as zz_jgrimm19:37
*** mjturek2 has quit IRC19:38
*** mjturek1 has joined #openstack-nova19:42
*** thangp has quit IRC19:44
*** igordcard has quit IRC19:45
*** emagana has joined #openstack-nova19:48
*** unicell has quit IRC19:49
*** emagana_ has joined #openstack-nova19:51
*** dsanders has quit IRC19:52
*** emagana has quit IRC19:52
*** dsanders has joined #openstack-nova19:52
openstackgerritVilobh Meshram proposed openstack/nova-specs: Replace service groups with the tooz groups  https://review.openstack.org/13860719:54
*** thomasem has joined #openstack-nova19:56
*** dboik has quit IRC19:59
*** mriedem has quit IRC19:59
*** dboik has joined #openstack-nova19:59
*** jcoufal has quit IRC20:00
*** dboik_ has joined #openstack-nova20:02
*** claudiub has joined #openstack-nova20:03
*** jdurgin has quit IRC20:04
*** dboik has quit IRC20:05
*** josecastroleon has quit IRC20:07
bauzasalaski: around ? just wonder if we should leave https://review.openstack.org/#/c/171951/1 to be landed but also see if we should remove force_hosts from the scheduler IMHO20:08
*** subscope_ has quit IRC20:08
bauzasalaski: while I definitely prefer the latter, the former just fixes a small bug so I'm looking at another guy opinion :)20:08
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements  https://review.openstack.org/17987020:09
*** igordcard has joined #openstack-nova20:09
bauzasalaski: since force_hosts just bypasses the whole filters, it doesn't need to be in the sched20:09
*** josecastroleon has joined #openstack-nova20:09
*** doude has quit IRC20:10
alaskibauzas: I'm not against that patch landing.  though I'll admit to some surprise that anyone wants to run the chance scheduler20:10
*** doude has joined #openstack-nova20:10
bauzasalaski: agreed20:10
bauzasalaski: but since it's in-tree, we have to accept that :)20:11
*** otter768 has joined #openstack-nova20:11
alaskiwell it's simple for testing, which is really what I thought it's purpose was20:13
*** Anticime1 has joined #openstack-nova20:14
*** Anticimex has quit IRC20:15
johnthetubaguyjogo: thanks for that patch, got some dev ref stuff cooking too, but UK holiday today getting in my way20:15
bauzasoh johnthetubaguy working on holiday ? :)20:15
*** otter768 has quit IRC20:16
jogojohnthetubaguy: yeah I saw, I am trying to remove dead content and not add new right now20:18
jogoand work around your patches too20:18
bauzasalaski: mmm, by actually looking further, sounds like we need to support force_hosts in the scheduler since it's actually checking that the hosts are up (calling the SG API)20:18
bauzasalaski: that means that it checks your hosts and only return which ones are good20:19
bauzasalaski: so I won't eventually open a new BP for that20:19
*** jdurgin has joined #openstack-nova20:22
openstackgerritJohn Garbutt proposed openstack/nova: devref: add information to clarify nova scope  https://review.openstack.org/17862320:22
cfriesendansmith: I know how much you like evacuate...https://review.openstack.org/#/c/169827/20:22
*** matrohon has joined #openstack-nova20:23
dansmithcfriesen: busy with other stuff today, so doubt I'll get to it20:23
*** d34dh0r53 has quit IRC20:23
*** xyhuang has joined #openstack-nova20:24
cfriesendansmith: fair enough.20:24
*** toan has quit IRC20:24
*** d34dh0r53 has joined #openstack-nova20:24
*** toan has joined #openstack-nova20:25
*** htruta has quit IRC20:26
*** ociuhandu has quit IRC20:29
*** Sukhdev has quit IRC20:29
*** exploreshaifali has joined #openstack-nova20:29
*** boris-42 has quit IRC20:30
*** ramishra has quit IRC20:30
*** kun_huang has quit IRC20:30
*** cyeoh has quit IRC20:31
*** apmelton has quit IRC20:31
*** toan has quit IRC20:31
*** dboik_ has quit IRC20:31
*** Nakato has quit IRC20:32
*** lpetrut has quit IRC20:32
*** toan has joined #openstack-nova20:33
*** Nakato has joined #openstack-nova20:33
*** ivasev has quit IRC20:33
*** apmelton has joined #openstack-nova20:34
*** unicell has joined #openstack-nova20:34
*** cyeoh has joined #openstack-nova20:34
*** d34dh0r53 has quit IRC20:34
*** dboik has joined #openstack-nova20:34
*** ssurana1 has joined #openstack-nova20:34
*** kun_huang has joined #openstack-nova20:34
*** htruta has joined #openstack-nova20:34
*** d34dh0r53 has joined #openstack-nova20:35
*** Marga_ has joined #openstack-nova20:35
*** ramishra has joined #openstack-nova20:35
*** ssurana has quit IRC20:35
*** boris-42 has joined #openstack-nova20:36
*** jdurgin has quit IRC20:37
*** xgsa has joined #openstack-nova20:39
*** vladikr has quit IRC20:39
*** fawadkhaliq has joined #openstack-nova20:40
*** zz_jgrimm has quit IRC20:43
*** maurosr has quit IRC20:44
*** apuimedo has quit IRC20:45
*** sdake has joined #openstack-nova20:45
*** maurosr has joined #openstack-nova20:46
*** zz_jgrimm has joined #openstack-nova20:46
jogocfriesen: any chance you have a bug fix for making shelve unshelve work?20:48
*** sdake_ has quit IRC20:49
*** exploreshaifali has quit IRC20:51
*** sdake_ has joined #openstack-nova20:52
*** browne has joined #openstack-nova20:53
*** jdurgin has joined #openstack-nova20:53
*** fawadkhaliq has quit IRC20:55
*** salv-orlando has joined #openstack-nova20:55
*** sdake has quit IRC20:56
*** matrohon has quit IRC20:56
cfriesenjogo: don't think so.  haven't tried it recently20:59
cfriesenjogo: what's wrong with it?21:00
*** Sukhdev has joined #openstack-nova21:00
jogocfriesen: it doesn't work well with nova-network21:01
jogocfriesen: are multinode testing http://logs.openstack.org/80/170780/11/check/check-tempest-dsvm-multinode-full/31d7552/console.html21:01
*** busterswt has joined #openstack-nova21:03
*** tonytan4ever has quit IRC21:05
*** raildo has quit IRC21:05
*** artom has quit IRC21:05
*** artom has joined #openstack-nova21:05
cfriesenjogo: so you can't connect to it after unshelving?  I haven't really used shelve all that much...how does it differ from "nova stop" in practice?21:05
jogocfriesen: it may move the instance to a different machine21:06
cfriesenjogo: ah, okay.  does that testcase use floatingIP?21:07
*** pixelbeat has joined #openstack-nova21:07
*** xgsa has quit IRC21:07
*** artom has quit IRC21:07
*** artom has joined #openstack-nova21:08
*** thomasem has quit IRC21:08
cfriesenjogo: looks like it might...if so it might be worth checking whether it calls self.network_api.setup_instance_network_on_host() in the unshelve path21:09
jogocfriesen: AFAIK yes21:10
jogocfriesen: so the test works sometimes but not always21:10
cfriesenjogo: do the failures correlate with changing hosts?21:10
*** jichen has quit IRC21:10
jogocfriesen: not 100%21:11
cfriesenjogo: annoying.  sorry, I've got nothing else. :)21:15
jogocfriesen: also unshelve does call self.network_api.setup_instance_network_on_host21:16
jogocfriesen: thanks anyway21:16
*** haypo has joined #openstack-nova21:17
*** mmedvede has quit IRC21:18
*** eharney has quit IRC21:22
openstackgerritVictor Stinner proposed openstack/nova: Fix raise syntax for Python 3  https://review.openstack.org/17994621:28
openstackgerritArtom Lifshitz proposed openstack/nova: Make get_pagination_params respect osapi_max_limit  https://review.openstack.org/17994721:29
openstackgerritJoe Gordon proposed openstack/nova: Mark ironic credential config as secret  https://review.openstack.org/17985721:32
*** Jeffrey4l has quit IRC21:35
*** mwagner_lap has quit IRC21:35
*** igordcard has quit IRC21:36
openstackgerritDan Smith proposed openstack/nova: Add SpawnFixture  https://review.openstack.org/17994921:38
openstackgerritDan Smith proposed openstack/nova: WIP: Refactor tests that use compute's deprecated run_instance() method  https://review.openstack.org/17995021:38
openstackgerritDan Smith proposed openstack/nova: Drop compute RPC 3.x support  https://review.openstack.org/17995121:38
dansmithalaski: that middle patch isn't quite there yet.. there seem to be some actual serious differences between the two, around things like notifications especially21:38
dansmithalaski: I'm hoping that maybe tomorrow you could take a look at some of those failures and see what you thihnk21:38
dansmithI've had it with that crap and have to go do something else before I go crazy21:39
alaskidansmith: can do21:41
dansmiththanks21:42
* dansmith &21:42
openstackgerritVictor Stinner proposed openstack/nova: Replace dict.itervalues() with six.itervalues(dict)  https://review.openstack.org/17995221:45
*** VW_ has quit IRC21:46
*** aix has joined #openstack-nova21:46
*** Jeffrey4l has joined #openstack-nova21:47
jogoalaski: ping shelve/unshelve21:48
*** artom has quit IRC21:49
jogoalaski: shelve/unshelve breaking due to bug https://bugs.launchpad.net/nova/+bug/1445569 is the last big issue blocking the mulitnode job from being stable enough to gate on21:49
openstackLaunchpad bug 1445569 in OpenStack Compute (nova) "No dhcp lease after shelve unshelve" [High,Confirmed]21:49
jogoalaski: and being you wrote the shelve code originally ... I908474915c5325e120a64af9b9af13fcbce1f01c21:50
alaskijogo: I'm not really familiar with the n-net networking stuff, I would have to spend some time there to be helpful21:51
alaskijogo: when writing shelve I used a lot of what resize did21:51
*** rfolco has quit IRC21:51
*** dave-mccowan has quit IRC21:51
alaskiall of the manipulation to move networking to a new host should copy what's used for resize21:52
jogoalaski:  hmm that is a good point21:52
*** ekhugen has quit IRC21:52
jogoI wonder if we have a resize test21:52
alaskiit's possible that I missed something, or they've drifted apart.  or that resize would have the same issue21:53
*** zz_jgrimm has quit IRC21:54
*** timello has quit IRC21:54
jogoalaski: ahh we have allow_migrate and allow resize to same host = True21:55
jogoso we may be seeing those issues21:55
*** kaufer has quit IRC21:55
*** ekhugen has joined #openstack-nova21:55
*** apuimedo has joined #openstack-nova21:56
*** zz_jgrimm has joined #openstack-nova21:56
*** timello has joined #openstack-nova21:57
jogomaybe not, looking at the wrong log21:57
alaskiwife putting dinner out, gotta run21:59
jogoalaski: thanks o/21:59
*** dboik has quit IRC22:02
*** thomasem has joined #openstack-nova22:02
*** harlowja_ has quit IRC22:06
*** harlowja has joined #openstack-nova22:07
*** ekhugen has quit IRC22:08
*** zz_jgrimm has quit IRC22:09
*** mmedvede has joined #openstack-nova22:09
*** maurosr has quit IRC22:11
*** ekhugen has joined #openstack-nova22:11
*** pixelbeat has quit IRC22:11
*** otter768 has joined #openstack-nova22:12
*** zz_jgrimm has joined #openstack-nova22:12
*** busterswt has quit IRC22:12
*** oro has quit IRC22:13
*** maurosr has joined #openstack-nova22:14
*** otter768 has quit IRC22:16
*** zz_jgrimm has quit IRC22:16
openstackgerritMitsuhiro Tanino proposed openstack/nova: Prevent unintentional detaching volume from instance  https://review.openstack.org/16669522:16
*** maurosr has quit IRC22:19
*** ekhugen has quit IRC22:19
*** zz_jgrimm has joined #openstack-nova22:19
*** ildikov has quit IRC22:20
*** maurosr has joined #openstack-nova22:20
*** igordcard has joined #openstack-nova22:21
*** ekhugen has joined #openstack-nova22:21
*** igordcard has joined #openstack-nova22:21
*** mwagner_lap has joined #openstack-nova22:22
*** igordcard has quit IRC22:31
openstackgerritEric Brown proposed openstack/nova: Deprecate nova ironic driver's admin_auth_token  https://review.openstack.org/17996022:32
*** mmedvede has quit IRC22:32
*** aysyd has quit IRC22:33
*** ildikov has joined #openstack-nova22:33
*** oro has joined #openstack-nova22:35
*** igordcard has joined #openstack-nova22:37
*** baoli has quit IRC22:37
*** VW_ has joined #openstack-nova22:42
*** VW_ has quit IRC22:42
*** VW_ has joined #openstack-nova22:42
*** VW_ has quit IRC22:43
*** IanGovett1 has quit IRC22:47
*** VW_ has joined #openstack-nova22:49
*** ZZelle_ has quit IRC22:55
*** armax has quit IRC22:57
*** thomasem has quit IRC22:57
*** Sukhdev has quit IRC23:00
*** xyhuang has quit IRC23:01
*** oro has quit IRC23:01
*** thomasem has joined #openstack-nova23:01
*** zz_jgrimm has quit IRC23:03
*** sdake has joined #openstack-nova23:04
*** thomasem has quit IRC23:05
*** zz_jgrimm has joined #openstack-nova23:06
*** vilobhmm1 has quit IRC23:08
*** vilobhmm1 has joined #openstack-nova23:08
*** sdake_ has quit IRC23:08
*** Marga_ has quit IRC23:09
*** Marga_ has joined #openstack-nova23:12
*** thomasem has joined #openstack-nova23:15
*** Marga_ has quit IRC23:15
*** thomasem has quit IRC23:16
*** thomasem has joined #openstack-nova23:17
*** thomasem has quit IRC23:18
*** VW_ has quit IRC23:20
*** zz_jgrimm has quit IRC23:27
*** maurosr has quit IRC23:28
*** maurosr has joined #openstack-nova23:29
*** zz_jgrimm has joined #openstack-nova23:29
*** IanGovett has joined #openstack-nova23:35
*** hemna is now known as hemnafk23:37
*** claudiub has quit IRC23:37
*** Marga_ has joined #openstack-nova23:38
*** gokrokve has quit IRC23:39
*** vilobhmm1 has quit IRC23:41
*** dims_ has joined #openstack-nova23:42
*** dims has quit IRC23:44
*** flwang1 has joined #openstack-nova23:45
flwang1garyk: could you pls help review https://review.openstack.org/168759 ?23:45
flwang1adam_g: ^ thanks23:46
flwang1it's a cherrypick to juno23:46
*** signed8bit is now known as signed8bit_ZZZzz23:50
*** thomasem has joined #openstack-nova23:52
*** Raildo has joined #openstack-nova23:54
*** dave-mccowan has joined #openstack-nova23:56
*** IanGovett has quit IRC23:56
*** BadCub is now known as BadCub_Hiding23:58
*** sdake_ has joined #openstack-nova23:59

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