*** mikeit_ has quit IRC | 00:00 | |
*** Alexandra_ has joined #openstack-nova | 00:01 | |
*** mikeit_ has joined #openstack-nova | 00:02 | |
*** oro has quit IRC | 00:06 | |
*** sdake_ has joined #openstack-nova | 00:07 | |
*** sdake has quit IRC | 00:11 | |
*** mikeit_ has quit IRC | 00:18 | |
*** markvoelker has joined #openstack-nova | 00:21 | |
*** mikeitt has joined #openstack-nova | 00:21 | |
*** armax has joined #openstack-nova | 00:22 | |
*** markvoelker has quit IRC | 00:25 | |
*** eliqiao1 has left #openstack-nova | 00:29 | |
*** eliqiao1 has joined #openstack-nova | 00:29 | |
*** mikeitt has quit IRC | 00:31 | |
*** mikeit has joined #openstack-nova | 00:32 | |
*** mpaolino has joined #openstack-nova | 00:34 | |
*** mpaolino has quit IRC | 00:36 | |
*** promulo has joined #openstack-nova | 00:36 | |
*** haomaiwa_ has quit IRC | 00:39 | |
*** promulo__ has quit IRC | 00:40 | |
*** liusheng has joined #openstack-nova | 00:41 | |
*** Kevin_Zheng has joined #openstack-nova | 00:41 | |
*** yingjun has joined #openstack-nova | 00:45 | |
*** otter768 has joined #openstack-nova | 00:46 | |
*** salv-orlando has joined #openstack-nova | 00:47 | |
*** salv-orlando has quit IRC | 00:50 | |
*** FL1SK has quit IRC | 00:52 | |
*** bjornar has quit IRC | 01:04 | |
*** salv-orlando has joined #openstack-nova | 01:05 | |
*** hanzhang has joined #openstack-nova | 01:06 | |
*** bjornar has joined #openstack-nova | 01:06 | |
*** hanzhang has quit IRC | 01:07 | |
*** armax has quit IRC | 01:08 | |
*** vilobhmm1 has quit IRC | 01:12 | |
*** salv-orlando has quit IRC | 01:12 | |
*** markvoelker has joined #openstack-nova | 01:21 | |
openstackgerrit | Kylin CG proposed openstack/python-novaclient: Make _discover_extensions public https://review.openstack.org/171067 | 01:23 |
---|---|---|
*** sdake has joined #openstack-nova | 01:24 | |
*** sdake_ has quit IRC | 01:26 | |
*** markvoelker has quit IRC | 01:26 | |
*** sdake_ has joined #openstack-nova | 01:27 | |
*** gtt116_ has quit IRC | 01:28 | |
*** sdake has quit IRC | 01:30 | |
*** popw has joined #openstack-nova | 01:35 | |
openstackgerrit | Eli Qiao proposed openstack/nova: API: remove admin require from compute_node_get_all_by_* from db layer https://review.openstack.org/175334 | 01:38 |
openstackgerrit | Eli Qiao proposed openstack/nova: API: Add policy enforcement test cases for pci API https://review.openstack.org/175745 | 01:38 |
openstackgerrit | Eli Qiao proposed openstack/nova: API: remove instance_get_all_by_host hard-code admin check from db https://review.openstack.org/177673 | 01:38 |
openstackgerrit | Eli Qiao proposed openstack/nova: API: remove admin require for compute_node(get_all/search_by_hyperviso) from db https://review.openstack.org/175744 | 01:38 |
openstackgerrit | Eli Qiao proposed openstack/nova: API: remove admin require for compute_node_create/update/delete from db layer https://review.openstack.org/175693 | 01:38 |
*** Alexandra_ has quit IRC | 01:41 | |
openstackgerrit | zhangjl proposed openstack/nova: Check if destination_type in block_device_mapping_v2 https://review.openstack.org/173357 | 01:45 |
*** coreywright has quit IRC | 01:49 | |
*** patrickeast has joined #openstack-nova | 01:49 | |
*** renatoarmani has joined #openstack-nova | 01:52 | |
*** renatoarmani has quit IRC | 01:53 | |
*** coreywright has joined #openstack-nova | 01:56 | |
*** gokrokve has joined #openstack-nova | 01:57 | |
*** patrickeast has quit IRC | 02:00 | |
*** dimsum__ has quit IRC | 02:04 | |
*** liusheng has quit IRC | 02:04 | |
openstackgerrit | zhangjl proposed openstack/nova: Check if destination_type in block_device_mapping_v2 https://review.openstack.org/173357 | 02:06 |
*** kaufer has joined #openstack-nova | 02:06 | |
*** emagana has joined #openstack-nova | 02:11 | |
*** nickrobison has joined #openstack-nova | 02:11 | |
*** nickrobison has quit IRC | 02:16 | |
*** markvoelker has joined #openstack-nova | 02:22 | |
*** dsanders has quit IRC | 02:23 | |
*** achanda has quit IRC | 02:25 | |
*** yingjun has quit IRC | 02:26 | |
*** markvoelker has quit IRC | 02:27 | |
*** yingjun has joined #openstack-nova | 02:30 | |
*** dsanders has joined #openstack-nova | 02:32 | |
*** mikeit has quit IRC | 02:38 | |
*** yingjun has quit IRC | 02:39 | |
*** yingjun has joined #openstack-nova | 02:40 | |
*** patrickeast has joined #openstack-nova | 02:41 | |
*** patrickeast has quit IRC | 02:42 | |
*** sdake_ is now known as sdake | 02:44 | |
openstackgerrit | shihanzhang proposed openstack/nova: Optimize validate_networks https://review.openstack.org/175742 | 02:50 |
openstackgerrit | Eli Qiao proposed openstack/nova-specs: Correct lock_server api return code https://review.openstack.org/163275 | 02:50 |
openstackgerrit | shihanzhang proposed openstack/nova: Optimize validate_networks https://review.openstack.org/175742 | 02:52 |
*** FL1SK has joined #openstack-nova | 02:53 | |
*** salv-orlando has joined #openstack-nova | 02:56 | |
*** Jeffrey4l has joined #openstack-nova | 02:56 | |
*** salv-orlando has quit IRC | 03:01 | |
*** dimsum__ has joined #openstack-nova | 03:05 | |
*** dimsum__ has quit IRC | 03:10 | |
*** dsanders has quit IRC | 03:10 | |
*** dsanders has joined #openstack-nova | 03:10 | |
*** yingjun has quit IRC | 03:12 | |
*** yingjun has joined #openstack-nova | 03:14 | |
*** salv-orlando has joined #openstack-nova | 03:14 | |
*** salv-orlando has quit IRC | 03:19 | |
*** coolsvapl is now known as coolsvap | 03:22 | |
*** markvoelker has joined #openstack-nova | 03:23 | |
*** achanda has joined #openstack-nova | 03:26 | |
*** liusheng has joined #openstack-nova | 03:27 | |
*** markvoelker has quit IRC | 03:27 | |
*** dsanders has quit IRC | 03:28 | |
*** unicell has joined #openstack-nova | 03:30 | |
*** achanda has quit IRC | 03:31 | |
*** unicell1 has quit IRC | 03:32 | |
*** pixelbeat has quit IRC | 03:32 | |
*** pixelbeat has joined #openstack-nova | 03:33 | |
*** tojuvone_ has joined #openstack-nova | 03:35 | |
*** haomaiwang has joined #openstack-nova | 03:38 | |
*** nickrobison has joined #openstack-nova | 03:43 | |
*** deepthi has joined #openstack-nova | 03:51 | |
*** neelashah has joined #openstack-nova | 03:52 | |
*** yingjun has quit IRC | 03:58 | |
*** neelashah has quit IRC | 03:59 | |
*** josecastroleon has joined #openstack-nova | 04:03 | |
*** otter768 has quit IRC | 04:04 | |
*** josecastroleon has quit IRC | 04:05 | |
*** dsanders has joined #openstack-nova | 04:18 | |
*** arnaud____ has joined #openstack-nova | 04:19 | |
*** markvoelker has joined #openstack-nova | 04:24 | |
*** markvoelker has quit IRC | 04:28 | |
*** heyongli has quit IRC | 04:30 | |
*** heyongli has joined #openstack-nova | 04:31 | |
*** tjones1 has left #openstack-nova | 04:31 | |
*** unicell has quit IRC | 04:41 | |
*** unicell has joined #openstack-nova | 04:43 | |
*** irenab has quit IRC | 04:43 | |
*** pixelbeat has quit IRC | 04:45 | |
*** bkopilov has joined #openstack-nova | 04:46 | |
*** tjones1 has joined #openstack-nova | 04:51 | |
openstackgerrit | Lan Qi Song proposed openstack/nova: Remove useless volume when boot from volume failed https://review.openstack.org/177084 | 04:56 |
*** achanda has joined #openstack-nova | 05:02 | |
*** vilobhmm1 has joined #openstack-nova | 05:09 | |
*** moshele has joined #openstack-nova | 05:13 | |
openstackgerrit | Chhavi 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/179697 | 05:14 |
*** vilobhmm1 has quit IRC | 05:17 | |
*** _gryf|afk is now known as _gryf | 05:20 | |
*** markvoelker has joined #openstack-nova | 05:24 | |
*** markvoelker has quit IRC | 05:29 | |
*** irenab has joined #openstack-nova | 05:31 | |
*** jyuso has joined #openstack-nova | 05:35 | |
*** gokrokve has quit IRC | 05:37 | |
*** vilobhmm1 has joined #openstack-nova | 05:37 | |
openstackgerrit | Deepak C Shetty proposed openstack/nova: libvirt: Use 'relative' flag for online snapshot's commit/rebase operations https://review.openstack.org/168805 | 05:37 |
*** chhavi has joined #openstack-nova | 05:37 | |
*** gokrokve has joined #openstack-nova | 05:37 | |
*** vilobhmm1 has quit IRC | 05:38 | |
*** gokrokve has quit IRC | 05:42 | |
*** vilobhmm1 has joined #openstack-nova | 05:47 | |
*** vilobhmm1 has quit IRC | 05:48 | |
*** kedkul has joined #openstack-nova | 05:49 | |
kedkul | Hi, 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 IRC | 05:51 | |
*** salv-orlando has joined #openstack-nova | 05:53 | |
*** chhavi has quit IRC | 05:54 | |
*** cfriesen_ has quit IRC | 05:56 | |
*** salv-orlando has quit IRC | 05:56 | |
*** vilobhmm1 has joined #openstack-nova | 05:58 | |
*** salv-orlando has joined #openstack-nova | 05:59 | |
*** salv-orlando has quit IRC | 05:59 | |
*** zhangtralon has joined #openstack-nova | 05:59 | |
*** vilobhmm1 has quit IRC | 06:00 | |
*** josecastroleon has joined #openstack-nova | 06:02 | |
*** kaufer has quit IRC | 06:03 | |
*** otter768 has joined #openstack-nova | 06:05 | |
*** Murali has joined #openstack-nova | 06:05 | |
*** zhangtralon has quit IRC | 06:05 | |
*** gokrokve has joined #openstack-nova | 06:08 | |
*** gokrokve has quit IRC | 06:09 | |
*** gokrokve has joined #openstack-nova | 06:09 | |
*** otter768 has quit IRC | 06:10 | |
*** salv-orlando has joined #openstack-nova | 06:11 | |
*** doron has joined #openstack-nova | 06:15 | |
*** doron is now known as doron_afk | 06:15 | |
*** pkoniszewski has joined #openstack-nova | 06:15 | |
*** salv-orlando has quit IRC | 06:15 | |
*** pkoniszewski has quit IRC | 06:16 | |
*** pkoniszewski has joined #openstack-nova | 06:17 | |
*** ajayaa has joined #openstack-nova | 06:21 | |
*** sdake_ has joined #openstack-nova | 06:22 | |
kedkul | Hi, 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 IRC | 06:25 | |
*** vilobhmm1 has joined #openstack-nova | 06:28 | |
*** vilobhmm1 has quit IRC | 06:30 | |
*** sahid has joined #openstack-nova | 06:30 | |
*** sdake_ is now known as sdake | 06:35 | |
openstackgerrit | Alex Xu proposed openstack/nova-specs: Add separated policy rule for each v2.1 api https://review.openstack.org/127863 | 06:40 |
*** sudipto has joined #openstack-nova | 06:42 | |
*** dimsum__ has joined #openstack-nova | 06:42 | |
*** xylan_kong has left #openstack-nova | 06:44 | |
*** salv-orl_ has joined #openstack-nova | 06:45 | |
*** dimsum__ has quit IRC | 06:47 | |
*** Anticimex has quit IRC | 06:48 | |
*** dsanders has quit IRC | 06:52 | |
*** jcoufal has joined #openstack-nova | 06:55 | |
ttx | sdague, gilllliard: it's actually tonyb keeping the wiki and ics in sync those days. (giving credit where due) | 06:58 |
*** garyk has joined #openstack-nova | 07:02 | |
tonyb | sdague, gilllliard: what'd I miss? | 07:03 |
*** ssurana has joined #openstack-nova | 07:03 | |
*** doron_afk is now known as doron | 07:07 | |
*** alanf-mc has joined #openstack-nova | 07:07 | |
ttx | tonyb: I don't think you missed anything, I was just unjustly credited for something you do | 07:08 |
*** alanf-mc_ has joined #openstack-nova | 07:08 | |
*** pask81 has joined #openstack-nova | 07:09 | |
*** jichen has joined #openstack-nova | 07:10 | |
*** alanf-mc has quit IRC | 07:12 | |
*** yamahata has quit IRC | 07:14 | |
*** romainh has joined #openstack-nova | 07:18 | |
*** moshele has quit IRC | 07:20 | |
*** zhangtralon has joined #openstack-nova | 07:20 | |
*** sdake has quit IRC | 07:21 | |
*** doude has joined #openstack-nova | 07:21 | |
openstackgerrit | Eli Qiao proposed openstack/nova-specs: Add separated policy rule for each v2.1 api https://review.openstack.org/127863 | 07:23 |
*** achanda has quit IRC | 07:23 | |
*** dsanders has joined #openstack-nova | 07:24 | |
*** markvoelker has joined #openstack-nova | 07:26 | |
*** achanda has joined #openstack-nova | 07:26 | |
*** ildikov has joined #openstack-nova | 07:29 | |
*** markvoelker has quit IRC | 07:30 | |
*** nlahouti has quit IRC | 07:31 | |
*** doron is now known as doron_afk | 07:31 | |
*** moshele has joined #openstack-nova | 07:34 | |
*** markus_z has joined #openstack-nova | 07:36 | |
*** alanf-mc_ has quit IRC | 07:37 | |
*** lpetrut has joined #openstack-nova | 07:37 | |
*** achanda has quit IRC | 07:37 | |
*** alanf-mc has joined #openstack-nova | 07:37 | |
*** mpavone has joined #openstack-nova | 07:40 | |
*** dsanders has quit IRC | 07:42 | |
*** ssurana has quit IRC | 07:45 | |
*** aswadr has joined #openstack-nova | 07:50 | |
*** yingjun has joined #openstack-nova | 07:51 | |
*** alanf-mc has quit IRC | 07:51 | |
*** romainh has quit IRC | 07:52 | |
*** chhavi has joined #openstack-nova | 07:56 | |
*** jistr has joined #openstack-nova | 07:59 | |
*** yingjun has quit IRC | 07:59 | |
*** yingjun has joined #openstack-nova | 07:59 | |
*** yingjun_ has joined #openstack-nova | 08:04 | |
*** yingjun has quit IRC | 08:04 | |
*** oro has joined #openstack-nova | 08:04 | |
*** otter768 has joined #openstack-nova | 08:05 | |
*** romainh has joined #openstack-nova | 08:08 | |
*** ildikov has quit IRC | 08:10 | |
*** otter768 has quit IRC | 08:10 | |
*** yingjun_ has quit IRC | 08:17 | |
openstackgerrit | Sailaja proposed openstack/nova: Project quota update should not be less than user quota. https://review.openstack.org/179726 | 08:17 |
openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver https://review.openstack.org/140045 | 08:23 |
openstackgerrit | Adelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location https://review.openstack.org/179727 | 08:23 |
openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: verify vCenter server certificate https://review.openstack.org/179728 | 08:24 |
*** yuntong1 has quit IRC | 08:25 | |
*** ildikov has joined #openstack-nova | 08:26 | |
openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation https://review.openstack.org/138255 | 08:26 |
*** nickrobison has quit IRC | 08:27 | |
*** markvoelker has joined #openstack-nova | 08:27 | |
*** Hosam has joined #openstack-nova | 08:30 | |
*** markvoelker has quit IRC | 08:31 | |
*** Kevin_Zheng has quit IRC | 08:40 | |
*** Kevin_Zheng has joined #openstack-nova | 08:41 | |
*** pczesno has quit IRC | 08:44 | |
*** alexpilotti has quit IRC | 08:45 | |
*** ociuhandu has joined #openstack-nova | 08:48 | |
*** kedkul has quit IRC | 08:48 | |
*** deepakcs has joined #openstack-nova | 08:48 | |
deepakcs | Hi all, anybody seeing issues with gate-nova-python27 ? | 08:48 |
deepakcs | I 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 |
deepakcs | More details are here @ http://logs.openstack.org/05/168805/8/check/gate-nova-python27/584359c/console.html#_2015-05-04_07_23_32_995 | 08:49 |
*** haypo has joined #openstack-nova | 08:50 | |
*** arnaud____ has joined #openstack-nova | 08:52 | |
*** alexpilotti has joined #openstack-nova | 08:54 | |
openstackgerrit | sahid proposed openstack/nova-specs: console: convert consoles code to use objects framework https://review.openstack.org/165838 | 08:54 |
*** arnaud____ has quit IRC | 08:56 | |
openstackgerrit | sahid proposed openstack/nova-specs: console: convert consoles code to use objects framework https://review.openstack.org/165838 | 08:56 |
openstackgerrit | zhangtralon proposed openstack/nova: resize-revert can break anti-affinity policy https://review.openstack.org/160110 | 09:02 |
*** Hosam has quit IRC | 09:02 | |
*** Hosam has joined #openstack-nova | 09:03 | |
*** Hosam has quit IRC | 09:07 | |
tonyb | ttx: Ahh okay thanks. | 09:08 |
deepakcs | garyk, ^^ | 09:08 |
*** asrangne has joined #openstack-nova | 09:11 | |
*** salv-orlando has joined #openstack-nova | 09:13 | |
*** salv-orl_ has quit IRC | 09:13 | |
*** aswadr has quit IRC | 09:13 | |
*** markvoelker has joined #openstack-nova | 09:27 | |
*** jyuso has quit IRC | 09:27 | |
*** mpaolino has joined #openstack-nova | 09:29 | |
*** markvoelker has quit IRC | 09:32 | |
*** mpaolino has quit IRC | 09:35 | |
*** salv-orlando has quit IRC | 09:43 | |
*** ociuhandu has quit IRC | 09:44 | |
*** salv-orl_ has joined #openstack-nova | 09:46 | |
garyk | deepakcs: let me take alook | 09:47 |
*** salv-orlando has joined #openstack-nova | 09:50 | |
openstackgerrit | Chhavi 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/179697 | 09:52 |
deepakcs | garyk, thanks | 09:53 |
openstackgerrit | garyk proposed openstack/nova: VMware: fix unit tests to unblock gate https://review.openstack.org/179746 | 09:53 |
*** salv-orl_ has quit IRC | 09:53 | |
*** dimsum__ has joined #openstack-nova | 09:54 | |
deepakcs | garyk, i +1'ed it, just curious, was params=None added recently ? | 09:54 |
*** pixelbeat has joined #openstack-nova | 09:55 | |
*** tojuvone_ has quit IRC | 09:55 | |
garyk | deepakcs: i think that it is related to the commit https://github.com/openstack/nova/commit/bf8c75b19291b7ed9a4822a9b2d2a523c269afd0 | 09:56 |
garyk | still investigating | 09:56 |
openstackgerrit | Chhavi 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/179697 | 09:56 |
deepakcs | garyk, 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's | 09:57 |
garyk | deepakcs: i hope that it addresses the issue. lets wait for the gate to give an indication | 09:58 |
garyk | hopefully in about 10 minutes we will know | 09:58 |
deepakcs | garyk, agree, but then the 'gloabl requirement' patch should have indicated the -1, how did that pass ? | 09:58 |
*** tojuvone_ has joined #openstack-nova | 09:58 | |
openstackgerrit | Chhavi 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/179697 | 09:59 |
garyk | deepakcs: i am not sure that it is something related to the requirement. it looks like the requests library. | 10:01 |
*** lpetrut has quit IRC | 10:02 | |
*** doron has joined #openstack-nova | 10:04 | |
*** doron_afk has quit IRC | 10:04 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Helper scripts for running under Apache2 https://review.openstack.org/179672 | 10:04 |
*** dguitarbite has quit IRC | 10:05 | |
*** otter768 has joined #openstack-nova | 10:06 | |
garyk | deepakcs: the reason is an update in the requests library - this was updated yesterday (https://pypi.python.org/pypi/requests) | 10:07 |
deepakcs | garyk, good to know :) | 10:09 |
garyk | deepakcs: once we have a +1 i'll update the commit message | 10:09 |
deepakcs | garyk, sure | 10:09 |
deepakcs | garyk, pls do add Reported-By: if u wish :) | 10:09 |
garyk | deepakcs: sure. i'll open a bug | 10:11 |
*** otter768 has quit IRC | 10:11 | |
*** leo_ has joined #openstack-nova | 10:11 | |
deepakcs | garyk, cool | 10:11 |
leo_ | can i install/run kvm on an OpenStack VM? (running a VM on VM) | 10:12 |
*** moshele has quit IRC | 10:14 | |
*** lpetrut has joined #openstack-nova | 10:15 | |
deepakcs | leo_, you can provided you enable nested virtualization support in the host (google for it) | 10:15 |
openstackgerrit | garyk proposed openstack/nova: VMware: fix unit tests to unblock gate https://review.openstack.org/179746 | 10:15 |
garyk | Can nova cores please look at the above 0 this unblocks the gate | 10:15 |
*** armax has joined #openstack-nova | 10:15 | |
garyk | johnthetubaguy: ^^ | 10:16 |
garyk | sdague: ^^ | 10:16 |
leo_ | thanks deepakcs | 10:16 |
*** dguitarbite has joined #openstack-nova | 10:17 | |
*** zhangtralon has quit IRC | 10:18 | |
*** IanGovett has joined #openstack-nova | 10:21 | |
*** moshele has joined #openstack-nova | 10:23 | |
*** moshele has quit IRC | 10:24 | |
sdague | garyk: so ... https://review.openstack.org/#/c/179746/2/nova/tests/unit/virt/vmwareapi/test_read_write_util.py,cm confuses me | 10:28 |
*** kiran-r has joined #openstack-nova | 10:28 | |
sdague | because that means that if you have a lower version of requests, it'll now fail? | 10:28 |
garyk | sdague: no, it should not fail. the parameter has a defualt value. the problem is mock | 10:29 |
garyk | that is it checks all of the default params (as far as I understand) | 10:29 |
garyk | the api should be backwards compatible. if not then we need to cap the requests library | 10:29 |
garyk | does that make sense? | 10:30 |
*** salv-orl_ has joined #openstack-nova | 10:31 | |
*** vladikr has quit IRC | 10:31 | |
*** salv-orlando has quit IRC | 10:34 | |
*** sahid has quit IRC | 10:35 | |
*** kiran-r is now known as _kiran_ | 10:35 | |
*** leo_ has quit IRC | 10:36 | |
sdague | I 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-nova | 10:40 | |
*** moshele has joined #openstack-nova | 10:41 | |
garyk | sdague: do you mean skip the rest? | 10:41 |
sdague | yeh, there's not anything particularlly relevant about the requests call right? | 10:42 |
sdague | it was just being used to ensure the url was created effective | 10:42 |
sdague | that seems to be the point of the test | 10:43 |
garyk | yeah. | 10:43 |
garyk | i'll update | 10:43 |
*** asrangne has quit IRC | 10:43 | |
*** atuvenie has joined #openstack-nova | 10:43 | |
openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds RemoteFX support to the Hyper-V driver https://review.openstack.org/42529 | 10:44 |
*** armax has quit IRC | 10:44 | |
*** salv-orl_ has quit IRC | 10:46 | |
*** salv-orlando has joined #openstack-nova | 10:47 | |
*** lpetrut has left #openstack-nova | 10:48 | |
*** dikonoor has joined #openstack-nova | 10:48 | |
*** ociuhandu has joined #openstack-nova | 10:49 | |
*** bfic has joined #openstack-nova | 10:51 | |
*** dulek has joined #openstack-nova | 10:52 | |
garyk | sdague: 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 |
sdague | cool, thanks | 10:53 |
*** tjones2 has joined #openstack-nova | 10:54 | |
*** doron is now known as doron_afk | 10:55 | |
*** tjones1 has quit IRC | 10:57 | |
*** exploreshaifali has quit IRC | 10:59 | |
*** claudiub has joined #openstack-nova | 11:03 | |
*** baoli has joined #openstack-nova | 11:04 | |
*** baoli has quit IRC | 11:05 | |
openstackgerrit | garyk proposed openstack/nova: VMware: fix unit tests to unblock gate https://review.openstack.org/179746 | 11:09 |
garyk | sdague: ^ | 11:09 |
sdague | garyk: hmmm... ok, so that wasn't really what I was thinking here | 11:10 |
garyk | sdague: then i misunderstood - you wanted that we check the base url | 11:12 |
garyk | so the addition of new params here does not break anything and it is like the header defined | 11:12 |
garyk | sdague: http://docs.python-requests.org/en/latest/api/ | 11:13 |
*** Longgeek has joined #openstack-nova | 11:15 | |
*** bfic has quit IRC | 11:16 | |
*** pixelbeat has quit IRC | 11:16 | |
*** redbeard has joined #openstack-nova | 11:17 | |
openstackgerrit | Sean Dague proposed openstack/nova: VMWare: Isolate unit tests from requests https://review.openstack.org/179757 | 11:18 |
sdague | ok, that's what I was thinking | 11:18 |
sdague | garyk: see if - https://review.openstack.org/179757 is reasonable to you | 11:20 |
*** bfic has joined #openstack-nova | 11:21 | |
*** bfic has quit IRC | 11:22 | |
*** salv-orlando has quit IRC | 11:23 | |
sudipto | sdague, 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 |
garyk | sdague: it is 6 of one and half a dozen of the other. maybe consider mocking _create_read_connection and not the requests. | 11:23 |
sdague | I guess, it's the request signature check that's the problem though | 11:24 |
sdague | that's *not* our api | 11:24 |
sdague | so it shouldn't be mocked with a signature check | 11:24 |
garyk | sdague: correct, but what is importnat is what we pass to the requests. | 11:24 |
*** bfic has joined #openstack-nova | 11:25 | |
garyk | so that is what my patch is validating. | 11:25 |
sdague | garyk: but it's mocking the requests signature | 11:25 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Helper scripts for running under Apache2 https://review.openstack.org/179672 | 11:26 |
sdague | what is the test supposed to check? it seems like the url building is all that it actually is supposed to check | 11:26 |
openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation https://review.openstack.org/138255 | 11:26 |
garyk | sdague: 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 IRC | 11:27 | |
*** zhangtralon has joined #openstack-nova | 11:27 | |
sdague | the place that should be validated, it at all, is in the oslo.vmware library right? | 11:27 |
garyk | my understanding of the test is that is should validate the headers, the url and the flags passed to the requests | 11:27 |
openstackgerrit | zhangtralon proposed openstack/nova: resize-revert can break anti-affinity policy https://review.openstack.org/160110 | 11:27 |
sdague | why does it need to validate the headers | 11:28 |
sdague | why are those material to this call? | 11:28 |
*** eharney has quit IRC | 11:28 | |
sdague | they are set by oslo_vmware.rw_handles.FileHandle implicitly right? no nova code sets them | 11:29 |
garyk | yeah, i guess so. | 11:29 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Helper scripts for running under Apache2 https://review.openstack.org/179672 | 11:31 |
*** bfic has quit IRC | 11:31 | |
garyk | sdague: added some comments to the patch - you should mock out _create_read_connection instead of requests. | 11:33 |
*** sahid has joined #openstack-nova | 11:33 | |
garyk | that is the code that calls the requests | 11:34 |
sdague | ok, sure | 11:34 |
sdague | it was quick, and I haven't finished my first cup of coffee yet | 11:34 |
garyk | :) | 11:34 |
chhavi | There is Jenkin failure for gate-nova-python27 FAILURE in 17m 00s | 11:35 |
*** deepthi has quit IRC | 11:35 | |
*** deepakcs has quit IRC | 11:36 | |
sdague | chhavi: yes, we know | 11:36 |
chhavi | my change-id has not modified this test, so is this issue been handled by some other change-id | 11:36 |
*** bfic has joined #openstack-nova | 11:36 | |
garyk | chhavi: the problem is one of the python libraries was updated and the test in question breaks due to a signature mismatch | 11:37 |
chhavi | ok sure | 11:37 |
*** Anticimex has joined #openstack-nova | 11:38 | |
*** pask81 has quit IRC | 11:40 | |
openstackgerrit | zhangtralon proposed openstack/nova: Support force_hosts in the ChanceScheduler https://review.openstack.org/171951 | 11: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 dims | 11:41 | |
sdague | dims: 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 |
dims | sdague: ack thanks! no worries | 11:42 |
*** ildikov has joined #openstack-nova | 11:43 | |
openstackgerrit | Sean Dague proposed openstack/nova: VMWare: Isolate unit tests from requests https://review.openstack.org/179757 | 11:43 |
dims | johnthetubaguy: 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 |
sdague | garyk: ok - is https://review.openstack.org/179757 to your liking? | 11:44 |
*** claudiub_ has joined #openstack-nova | 11:44 | |
*** claudiub has quit IRC | 11:45 | |
garyk | sdague: works for me. looks like there may be a pep8 issue with line length. | 11:46 |
garyk | line 44 (not 100% sure) | 11:46 |
garyk | code looks good | 11:47 |
*** sudipto has quit IRC | 11:50 | |
*** markvoelker has joined #openstack-nova | 11:50 | |
*** alexpilotti has quit IRC | 11:51 | |
*** tojuvone_ has quit IRC | 11:54 | |
*** exploreshaifali has joined #openstack-nova | 11:56 | |
*** dikonoor has quit IRC | 11:57 | |
*** raildo has joined #openstack-nova | 11:58 | |
*** rfolco has joined #openstack-nova | 11:58 | |
*** lpetrut has joined #openstack-nova | 11:59 | |
*** lpetrut has left #openstack-nova | 11:59 | |
*** exploreshaifali has quit IRC | 12:00 | |
*** exploreshaifali has joined #openstack-nova | 12:00 | |
sdague | flake8 didn't trigger for me locally, so I don't think so | 12:01 |
*** exploreshaifali has quit IRC | 12:03 | |
*** exploreshaifali has joined #openstack-nova | 12:04 | |
*** aix has joined #openstack-nova | 12:05 | |
*** exploreshaifali has quit IRC | 12:07 | |
*** popw has quit IRC | 12:09 | |
garyk | sdague: the grumpy old man has given his +1. can you get other cores to look at this to unblock the gate | 12:10 |
garyk | johnthetubaguy: ^ (https://review.openstack.org/#/c/179757/) | 12:10 |
*** dave-mccowan has joined #openstack-nova | 12:10 | |
*** shakamunyi has quit IRC | 12:18 | |
*** aysyd has joined #openstack-nova | 12:19 | |
*** ferest has joined #openstack-nova | 12:20 | |
*** VW_ has joined #openstack-nova | 12:21 | |
*** htruta has joined #openstack-nova | 12:21 | |
*** ferest_ has joined #openstack-nova | 12:23 | |
*** salv-orlando has joined #openstack-nova | 12:24 | |
*** ivasev has joined #openstack-nova | 12:24 | |
sdague | johnthetubaguy / mikal / superdan - would be good to get another nova-core to look at https://review.openstack.org/#/c/179757/ and fix unit tests | 12:25 |
*** ferest_ has quit IRC | 12:27 | |
*** ferest has quit IRC | 12:27 | |
*** ivasev has quit IRC | 12:28 | |
*** pixelbeat has joined #openstack-nova | 12:29 | |
*** subscope has joined #openstack-nova | 12:35 | |
*** alexpilotti has joined #openstack-nova | 12:35 | |
*** salv-orlando has quit IRC | 12:37 | |
*** ivasev has joined #openstack-nova | 12:38 | |
*** dprince has joined #openstack-nova | 12:39 | |
*** shyama has joined #openstack-nova | 12:43 | |
bauzas | dims: today is UK holiday | 12:45 |
bauzas | dims: so you won't get johnthetubaguy unless he wants to work :) | 12:46 |
dims | bauzas: ack thanks. just lining it up for scrollback :) | 12:46 |
bauzas | we're entering May, aka. France vacation month :-) | 12:47 |
bauzas | like 40% of our holidays are in the same month... | 12:47 |
bauzas | and I guess some other EU countries are quite similar | 12:47 |
*** tojuvone_ has joined #openstack-nova | 12:48 | |
*** dprince has quit IRC | 12:48 | |
*** exploreshaifali has joined #openstack-nova | 12:48 | |
*** Jeffrey4l has quit IRC | 12:49 | |
*** cbader has joined #openstack-nova | 12:49 | |
dims | bauzas: i envy you all :) | 12:51 |
bauzas | dims: there is a Mirantis Grenoble office y'know :p | 12:51 |
*** exploreshaifali has quit IRC | 12:51 | |
alex_xu | johnthetubaguy: 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 |
dims | bauzas: ++ :) | 12:52 |
bauzas | dims: I've been told that someone from MV, CA just made the move to Grenoble :) | 12:52 |
*** mikeit has joined #openstack-nova | 12:53 | |
*** exploreshaifali has joined #openstack-nova | 12:53 | |
dims | bauzas: ha! | 12:54 |
*** links has joined #openstack-nova | 12:56 | |
*** exploreshaifali has quit IRC | 12:57 | |
*** sdake has joined #openstack-nova | 12:57 | |
*** jaypipes has joined #openstack-nova | 12:58 | |
*** exploreshaifali has joined #openstack-nova | 12:58 | |
*** exploreshaifali has quit IRC | 12:59 | |
openstackgerrit | Alex Xu proposed openstack/nova: API: remove instance_get_all_by_host hard-code admin check from db https://review.openstack.org/177673 | 12:59 |
*** exploreshaifali has joined #openstack-nova | 13:00 | |
*** exploreshaifali has quit IRC | 13:00 | |
*** Jeffrey4l has joined #openstack-nova | 13:01 | |
*** dikonoor has joined #openstack-nova | 13:03 | |
*** moshele has quit IRC | 13:04 | |
*** moshele has joined #openstack-nova | 13:04 | |
*** kaufer has joined #openstack-nova | 13:08 | |
*** signed8bit has joined #openstack-nova | 13:09 | |
*** signed8bit has quit IRC | 13:09 | |
*** ajayaa has quit IRC | 13:10 | |
*** VW_ has quit IRC | 13:12 | |
*** VW_ has joined #openstack-nova | 13:12 | |
*** dims has quit IRC | 13:14 | |
*** dims has joined #openstack-nova | 13:15 | |
*** erlon_awaY has joined #openstack-nova | 13:15 | |
*** alexpilotti has quit IRC | 13:16 | |
*** _kiran_ is now known as kiran-r | 13:18 | |
*** neelashah has joined #openstack-nova | 13:20 | |
*** baoli has joined #openstack-nova | 13:21 | |
*** jcoufal_ has joined #openstack-nova | 13:24 | |
*** mestery_ is now known as mestery | 13:25 | |
*** jcoufal has quit IRC | 13:27 | |
*** erlon_awaY is now known as erlon | 13:27 | |
openstackgerrit | Adelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation https://review.openstack.org/138255 | 13:29 |
*** sdake_ has joined #openstack-nova | 13:31 | |
*** dprince has joined #openstack-nova | 13:32 | |
*** bfic has quit IRC | 13:32 | |
*** sdake has quit IRC | 13:34 | |
*** kiran-r has quit IRC | 13:37 | |
*** coolsvap is now known as coolsvap|afk | 13:37 | |
*** apuimedo has joined #openstack-nova | 13:37 | |
dikonoor | sdague : could you take a look at https://review.openstack.org/#/c/178059/ ? | 13:40 |
dikonoor | sdague:This needs another +2 | 13:40 |
*** thangp has joined #openstack-nova | 13:42 | |
sdague | superdan: you want to fast approve https://review.openstack.org/#/c/179757/ ? 0 nova jobs will pass until it's in | 13:43 |
sdague | and apparently we are for lack of nova cores this morning | 13:43 |
superdan | I'm easy to trick before I've had coffee | 13:43 |
* superdan gets coffee | 13:44 | |
*** jeblair_ is now known as jeblair | 13:44 | |
*** superdan is now known as dansmith | 13:44 | |
*** kodoku has joined #openstack-nova | 13:45 | |
*** ajayaa has joined #openstack-nova | 13:46 | |
*** mriedem has joined #openstack-nova | 13:46 | |
*** dboik has joined #openstack-nova | 13:46 | |
kodoku | Hi, 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 |
kodoku | Maybe I need to execute command for change this delay ? | 13:47 |
*** lascii is now known as alaski | 13:47 | |
dansmith | kodoku: please see topic | 13:48 |
kodoku | dansmith ok sorry | 13:49 |
moshele | dansmith: hi | 13:53 |
*** emagana has quit IRC | 13:53 | |
dansmith | moshele: hi | 13:53 |
chhavi | dansmith: hi | 13:53 |
chhavi | I have submitted one review | 13:53 |
chhavi | https://review.openstack.org/#/c/179697/ | 13:53 |
sdague | dansmith: thanks | 13:54 |
chhavi | this is with respect to the changes which you have done earlier to include _ComputeV4Proxy | 13:54 |
moshele | dansmith: Mellanox CI can we start it on nova as none voting ? | 13:54 |
*** kodoku has left #openstack-nova | 13:54 | |
chhavi | there is a small change needed for detach_volume definition. please check | 13:54 |
mriedem | chhavi: do you have a stack trace you can put in the bug report? | 13:55 |
mriedem | chhavi: your commit message could use some work too | 13:55 |
dansmith | moshele: 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 |
dikonoor | mriedem :https://review.openstack.org/#/c/178059/ - could you please take a look and approve :) | 13:56 |
dansmith | mriedem: you okay with the mellanox CI bits? and if not, can you reply on their thread? | 13:56 |
chhavi | ok | 13:56 |
mriedem | just got online | 13:56 |
mriedem | did someone chum the waters here? | 13:56 |
*** xuhanp_ has joined #openstack-nova | 13:57 | |
*** xuhanp has joined #openstack-nova | 13:57 | |
*** atuvenie_ has joined #openstack-nova | 13:57 | |
*** atuvenie has quit IRC | 14:00 | |
mriedem | dansmith: i'll look at the ML thread | 14:00 |
mriedem | garyk: 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_907 | 14:01 |
mriedem | that's hitting multiple changes | 14:01 |
sdague | mriedem: if you'd been logged in, you would see the scrollback :) | 14:01 |
openstackgerrit | Victor Stinner proposed openstack/nova: Rename iteritems() methods to items() on Python 3 https://review.openstack.org/179783 | 14:01 |
openstackgerrit | Victor Stinner proposed openstack/nova: Replace dict.iteritems() with six.iteritems(dict) https://review.openstack.org/179784 | 14:01 |
sdague | mriedem: https://review.openstack.org/#/c/179757/ | 14:01 |
mriedem | sorry, slept in | 14:02 |
*** atuvenie_ has quit IRC | 14:02 | |
mriedem | kid puked 3 times throughout the night | 14:02 |
sdague | oof, sorry | 14:02 |
*** atuvenie_ has joined #openstack-nova | 14:02 | |
sdague | anyway, dansmith fast approved it | 14:02 |
mriedem | i'm not the one shampooing the carpets today | 14:02 |
dansmith | my kid puked zero times throughout the night | 14:02 |
dansmith | just sayin' | 14:02 |
sdague | mriedem: if you want to build an ER signature for it, that would be cool | 14:02 |
sdague | but it should be merged shortly | 14:03 |
mriedem | yeah can do | 14:03 |
*** tojuvone_ has quit IRC | 14:03 | |
*** dave-mccowan has quit IRC | 14:03 | |
*** sahid has quit IRC | 14:04 | |
*** zz_naotok has quit IRC | 14:04 | |
*** sahid has joined #openstack-nova | 14:05 | |
*** zz_naotok has joined #openstack-nova | 14: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-nova | 14:08 | |
*** otter768 has joined #openstack-nova | 14:08 | |
sdague | haypo: 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 IRC | 14:09 | |
garyk | mriedem: no, that was just due to a update in the requests library | 14:09 |
sdague | and not care about which python version, you'll just implement the interface for both | 14:09 |
garyk | mriedem: please see https://review.openstack.org/179757 | 14:10 |
haypo | sdague, dict has no iteritems method on python 3, i prefer to mimick the exact dict API | 14:10 |
*** cfriesen_ has joined #openstack-nova | 14:11 | |
haypo | sdague, this change is related to its following change to use six.iteritems() everywhere | 14:11 |
sdague | yeh, seems silly. Though this incompat issue with python3 also seems silly. | 14:12 |
openstackgerrit | Jens Rosenboom proposed openstack/nova: Allow server search option ip6 for non-admin https://review.openstack.org/179569 | 14:12 |
*** otter768 has quit IRC | 14:13 | |
haypo | sdague, 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 refactoring | 14:15 |
haypo | sdague, IMO it's safer to use six.iteritems() everywhere | 14:15 |
*** pkoniszewski has quit IRC | 14:15 | |
*** dave-mccowan has joined #openstack-nova | 14:15 | |
*** salv-orlando has joined #openstack-nova | 14:16 | |
*** garyk has quit IRC | 14:17 | |
sdague | I 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 |
haypo | sdague, 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 layers | 14:19 |
*** exploreshaifali has joined #openstack-nova | 14:19 | |
*** signed8bit has joined #openstack-nova | 14:19 | |
haypo | sdague, 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 |
sdague | but, 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 3 | 14:20 |
sdague | haypo: right, so come back in 10 years :) | 14:20 |
*** vladikr has joined #openstack-nova | 14:20 | |
sdague | anyway, just frustrated that python 3 didn't think to provide some of this compatibility itself | 14:21 |
haypo | sdague, six is a nice way to write code compatible with python 2 and python 3 with little efforts | 14:23 |
haypo | sdague, 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 IRC | 14:23 | |
dansmith | it's really not | 14:28 |
dansmith | it's terrible | 14:28 |
dansmith | it's good that there's a way to do it, but it's far from "nice" IMHO :) | 14:29 |
sdague | dansmith: ++ | 14:30 |
*** dboik_ has joined #openstack-nova | 14:30 | |
edleafe | dansmith: sdague: haypo: OTOH, the idea was not to make it easy to stay with 2.x | 14:31 |
sdague | especially 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 smoothly | 14:31 |
dansmith | edleafe: that's very clear, but not really a factor in my evaluation of niceness :) | 14:31 |
sdague | edleafe: right, that's clear. That worked out really well in getting people to move :) | 14:32 |
edleafe | Somehow seeing 'dansmith' and 'niceness' in the same sentence feels odd | 14:32 |
sdague | heh | 14:32 |
sdague | edleafe: ++ | 14:32 |
*** salv-orlando has quit IRC | 14:32 | |
*** armax has joined #openstack-nova | 14:33 | |
edleafe | sdague: yeah, the problem was that 3.x didn't offer anything especially wonderful to justify the pain of moving | 14:33 |
bauzas | I have to agree with sdague, having .items() returing different results if py3 instead of py2 seems just errorprone | 14:33 |
bauzas | but unfortunately, life is bad - we have to use six for that | 14:33 |
edleafe | bauzas: but for people working solely in 2.x, having one .items() act differently than every other .items() is worse | 14:33 |
*** gokrokve_ has joined #openstack-nova | 14:34 | |
*** dboik has quit IRC | 14:34 | |
bauzas | edleafe: 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 IMHO | 14:34 |
sdague | edleafe: that's fine, but it still doesn't excuse the lack of an alias for viewitems | 14:34 |
edleafe | sdague: true | 14:35 |
sdague | that's just people not thinking through actual code base usage when they did the backport | 14:35 |
bauzas | but as we can't just use the delorean to ask for that, it needs us to use six | 14:35 |
*** zz_jgrimm is now known as jgrimm | 14:35 | |
edleafe | sdague: it's surprising that they even added that back to 2.7, as they are pretty dead set against enhancing anything in 2 anymore | 14:36 |
*** shyama has quit IRC | 14:36 | |
sdague | edleafe: it was 5 years ago or something | 14:37 |
*** gokrokve has quit IRC | 14:37 | |
*** jgrimm is now known as zz_jgrimm | 14:38 | |
sdague | anyway, beer rant | 14:38 |
edleafe | sdague: exactly | 14:39 |
alex_xu | johnthetubaguy: 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 |
edleafe | sdague: I've had way too many of these discussions at the bars during PyCons | 14:39 |
sdague | edleafe: sadly no on listened? :) | 14:39 |
sdague | http://blog.brentlaabs.com/2015/05/three-tales-of-second-system-syndrome.html - was an interesting read from this morning | 14:39 |
bauzas | alex_xu: as said previously, it's holiday for UK peers | 14:40 |
edleafe | sdague: I probably wasn't at my most convincing | 14:40 |
alex_xu | bauzas: ah, thanks | 14:40 |
*** mtanino has joined #openstack-nova | 14:40 | |
alex_xu | bauzas: just know John at UK also | 14:40 |
*** unicell1 has joined #openstack-nova | 14:40 | |
*** packet has joined #openstack-nova | 14:41 | |
*** unicell has quit IRC | 14:42 | |
*** emagana has joined #openstack-nova | 14:42 | |
openstackgerrit | jichenjc proposed openstack/nova-specs: New API for get service info from id https://review.openstack.org/172412 | 14:43 |
*** claudiub_ has quit IRC | 14:44 | |
*** xuhanp has quit IRC | 14:49 | |
*** xuhanp_ has quit IRC | 14:49 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Eventlet supports python logging directly https://review.openstack.org/179801 | 14:49 |
*** tonytan4ever has joined #openstack-nova | 14:52 | |
haypo | dansmith, 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 py3 | 14:52 |
openstackgerrit | Victor Stinner proposed openstack/nova: Rename iteritems() methods to items() on Python 3 https://review.openstack.org/179783 | 14:53 |
openstackgerrit | Victor Stinner proposed openstack/nova: Replace dict.iteritems() with six.iteritems(dict) https://review.openstack.org/179784 | 14:53 |
*** pankaj2934 has joined #openstack-nova | 14:53 | |
dansmith | pick a different word then :) | 14:53 |
dansmith | "bare effing minimum" might be appropriate :) | 14:53 |
haypo | dansmith, six is convinient | 14:53 |
*** alexpilotti has joined #openstack-nova | 14:53 | |
*** Murali has quit IRC | 14:54 | |
*** atuvenie has joined #openstack-nova | 14:54 | |
dansmith | a 7/11 is convenient, six is a necessary evil | 14:55 |
*** gokrokve_ has quit IRC | 14:55 | |
*** gokrokve has joined #openstack-nova | 14:55 | |
*** links has quit IRC | 14:56 | |
*** garyk has joined #openstack-nova | 14:57 | |
*** gokrokve has quit IRC | 14:57 | |
*** atuvenie_ has quit IRC | 14:57 | |
*** gokrokve has joined #openstack-nova | 14:58 | |
*** sdake has joined #openstack-nova | 15:01 | |
*** ijw has joined #openstack-nova | 15:02 | |
*** sdake__ has joined #openstack-nova | 15:03 | |
*** sdake_ has quit IRC | 15:03 | |
*** ajayaa has quit IRC | 15:06 | |
*** sdake has quit IRC | 15:07 | |
*** ociuhandu_ has joined #openstack-nova | 15:07 | |
mriedem | moshele: for the mellanox ci, | 15:10 |
mriedem | there are tests that live outside of tempest right? | 15:10 |
moshele | mremdem: no we are using the tempest scenarios with port direct | 15:11 |
*** zzzeek has joined #openstack-nova | 15:11 | |
mriedem | moshele: 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.gz | 15:12 |
mriedem | the rest are api tests | 15:12 |
*** aix has quit IRC | 15:13 | |
openstackgerrit | jichenjc proposed openstack/nova-specs: New API for get service info from id https://review.openstack.org/172412 | 15:13 |
moshele | mriedem: yes this is the important one | 15:13 |
*** iamjarvo has joined #openstack-nova | 15:13 | |
moshele | mriedem: I with a customer I will ping you later | 15:13 |
*** cfriesen_ is now known as cfriesen | 15:14 | |
*** claudiub has joined #openstack-nova | 15:14 | |
*** aix has joined #openstack-nova | 15:15 | |
cfriesen | any 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 IRC | 15:18 | |
pankaj2934 | is there a way to set a proper hostname to a instance and haev a A record too in infoblox for the instacne | 15:19 |
dansmith | pankaj2934: please see topic | 15:20 |
mriedem | dansmith: moshele: +1 from me on mellanox ci non-voting | 15:20 |
dansmith | mriedem: what about that scenario test? | 15:20 |
dansmith | anteaya: what do we need to do to grant them comment perms? | 15:20 |
mriedem | dansmith: i assumed this was like the intel pci ci where the tests are not in tempest but some other private repo | 15:21 |
dansmith | mriedem: okay, yeah, I don't think so | 15:21 |
anteaya | dansmith: not a thing, other than tell them they may | 15:21 |
dansmith | anteaya: ah, okay, well, we have two cores +1, and nobody else seems to care, so I guess we're good | 15:22 |
anteaya | so that if I ever come and ask does foo ci have permission to comment on nova patches you can say "yes" | 15:22 |
anteaya | dansmith: thank you | 15:22 |
anteaya | that works for me | 15:22 |
dansmith | moshele: I think you're good to go | 15:22 |
*** sudipto has joined #openstack-nova | 15:22 | |
moshele | dansmith: thanks | 15:22 |
*** dsanders has joined #openstack-nova | 15:23 | |
*** penick has joined #openstack-nova | 15:26 | |
*** pankaj2934 has left #openstack-nova | 15:28 | |
*** apuimedo has quit IRC | 15:29 | |
mriedem | jaypipes: johnthetubaguy: i'm looking at this https://review.openstack.org/#/c/107185/ | 15:30 |
mriedem | that 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 IRC | 15:31 | |
mriedem | am i missing something? | 15:31 |
mriedem | i don't see a spec out here either http://specs.openstack.org/openstack/nova-specs/specs/kilo/approved/ | 15:31 |
mriedem | we have a customer asking for this but i don't know what kind of half baked state it's in | 15:31 |
mriedem | alexpilotti: claudiub: ^ | 15:31 |
*** mtanino has quit IRC | 15:33 | |
dansmith | alaski: on this: https://review.openstack.org/#/c/177470/6/nova/compute/manager.py,cm | 15:34 |
dansmith | alaski: we know that when the race happens, we get the save() as part of the cleanup and reschedule | 15:34 |
dansmith | alaski: 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-nova | 15:35 | |
dansmith | alaski: 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 wonder | 15:35 |
dansmith | JoshNang: jroll ^ | 15:35 |
alaski | I dunno, I'll have to check | 15:35 |
*** yamahata has joined #openstack-nova | 15:35 | |
sahid | so we don't have meeting today | 15:36 |
sahid | hm forget about me :) | 15:37 |
*** vilobhmm1 has quit IRC | 15:37 | |
JoshNang | dansmith: i'll check. i assume so because it was working with and without lots of reschedules. | 15:38 |
dansmith | right | 15:38 |
dansmith | I just want to make sure we're not opening a new race where it might *not* get saved | 15:38 |
alaski | the last thing we do before a reschedule is instance.save() | 15:39 |
*** mfisch has quit IRC | 15:40 | |
alaski | dansmith: ^, 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 anyways | 15:40 |
*** Longgeek_ has joined #openstack-nova | 15:40 | |
*** ildikov has quit IRC | 15:41 | |
openstackgerrit | Jens Rosenboom proposed openstack/nova: Allow server search option ip6 for non-admin https://review.openstack.org/179569 | 15:41 |
*** mfisch has joined #openstack-nova | 15:41 | |
*** salv-orlando has joined #openstack-nova | 15:41 | |
alaski | dansmith: but there is an instance.save after a successful return from the virt driver | 15:41 |
*** dsanders has quit IRC | 15:41 | |
*** mfisch is now known as Guest65197 | 15:41 | |
*** Longgeek has quit IRC | 15:42 | |
dansmith | alaski: okay cool | 15:42 |
dansmith | just wanted to make sure we weren't focusing on the 10% and losing sight of the 90% :) | 15:42 |
*** penick has quit IRC | 15:42 | |
alaski | dansmith: yeah, definitely | 15:43 |
frickler | regarding 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 list | 15:43 |
openstack | Launchpad 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 IRC | 15:43 | |
alaski | JoshNang: dansmith we should probably comment those saves, # This needs to be here because blah | 15:43 |
dansmith | alaski: yeah, so we don't remove them later for some reason | 15:44 |
JoshNang | alaski: ++. i can add more comments later today | 15:45 |
*** ijw has quit IRC | 15:46 | |
*** mwagner_lap has quit IRC | 15:47 | |
alaski | dansmith: also, 4 minutes, not bad :) | 15:49 |
openstackgerrit | Victor Stinner proposed openstack/nova: Replace iter.next() with next(iter) https://review.openstack.org/179821 | 15:49 |
dansmith | alaski: heh | 15:49 |
*** htruta has quit IRC | 15:50 | |
*** htruta has joined #openstack-nova | 15:53 | |
*** mtanino has joined #openstack-nova | 15:54 | |
openstackgerrit | Jens Rosenboom proposed openstack/nova: Allow server search option ip6 for non-admin https://review.openstack.org/179569 | 15:54 |
*** ijw has joined #openstack-nova | 15:54 | |
*** pixelbeat has quit IRC | 15:54 | |
*** armax has quit IRC | 15:56 | |
*** dguitarbite has joined #openstack-nova | 15:59 | |
*** gokrokve_ has joined #openstack-nova | 16:01 | |
*** gokrokve_ has quit IRC | 16:01 | |
*** Guest65197 has quit IRC | 16:02 | |
*** emagana has quit IRC | 16:03 | |
*** artom has joined #openstack-nova | 16:03 | |
*** gokrokve has quit IRC | 16:04 | |
*** emagana has joined #openstack-nova | 16:04 | |
*** atuvenie has quit IRC | 16:04 | |
*** dulek has quit IRC | 16:05 | |
openstackgerrit | Victor Stinner proposed openstack/nova: Drop L from literal integer numbers for Python 3 https://review.openstack.org/179828 | 16:05 |
*** mikeit has quit IRC | 16:06 | |
*** unicell1 has quit IRC | 16:06 | |
*** gokrokve has joined #openstack-nova | 16:06 | |
*** gokrokve has quit IRC | 16:06 | |
*** shyama has joined #openstack-nova | 16:08 | |
*** angdraug has joined #openstack-nova | 16:08 | |
*** mpavone has quit IRC | 16:08 | |
*** yamahata has quit IRC | 16:09 | |
*** otter768 has joined #openstack-nova | 16:09 | |
openstackgerrit | jichenjc proposed openstack/nova-specs: flavor access create should check public/private https://review.openstack.org/172388 | 16:09 |
*** ijw has quit IRC | 16:11 | |
*** ijw has joined #openstack-nova | 16:11 | |
*** markus_z has quit IRC | 16:11 | |
*** salv-orlando has quit IRC | 16:12 | |
*** Nic has joined #openstack-nova | 16:12 | |
openstackgerrit | Dan Smith proposed openstack/nova: Move our ObjectListBase to subclass from the Oslo one https://review.openstack.org/173995 | 16:14 |
openstackgerrit | Dan Smith proposed openstack/nova: Start the conversion to oslo.versionedobjects https://review.openstack.org/173479 | 16:14 |
dansmith | these have been sitting for a while and could use some eyes ^ | 16:14 |
*** otter768 has quit IRC | 16:14 | |
dansmith | pretty easy stuff, just some beginnings of cutting over to o.vo | 16:14 |
jaypipes | mriedem: hmm, I' | 16:16 |
*** ijw has quit IRC | 16:16 | |
jaypipes | m really not sure about that one. I remember reviewing the code, but never lookied at a spec. | 16:16 |
*** romainh has left #openstack-nova | 16:18 | |
*** zz_jgrimm is now known as jgrimm | 16:18 | |
mriedem | jaypipes: i posted to the ML where alexpilotti responded | 16:19 |
*** sahid has quit IRC | 16:19 | |
*** jichen has quit IRC | 16:20 | |
openstackgerrit | Dan Smith proposed openstack/nova: Move our ObjectListBase to subclass from the Oslo one https://review.openstack.org/173995 | 16:24 |
openstackgerrit | Dan Smith proposed openstack/nova: Start the conversion to oslo.versionedobjects https://review.openstack.org/173479 | 16:24 |
dansmith | any 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 |
dansmith | clarkb: sdague ^ | 16:24 |
*** tjones1 has joined #openstack-nova | 16:24 | |
clarkb | dansmith: beause it has that explanatory string explaining what the failure means | 16:25 |
clarkb | dansmith: many people were confused (and still are) when that job fails because they don't know about our reqs enforcement | 16:25 |
dansmith | clarkb: hmm, okay, well, it means that it's less visible when it fails if you're looking at the report | 16:25 |
dansmith | clarkb: 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 IRC | 16:26 | |
dansmith | most people click on failed jobs to go look at the console output to start I think | 16:26 |
clarkb | we could | 16:27 |
clarkb | its probably worth trying | 16:27 |
dansmith | or I could stop complaining :) | 16:27 |
dansmith | we made a big stink about proper external CI formatting, yet we break it ourselves | 16:27 |
dansmith | and when I say "we" I mean "sdague" in the first case :) | 16:27 |
clarkb | though 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 thing | 16:27 |
dansmith | those people should be tarred and feathered | 16:28 |
cfriesen | dansmith: I'll open them, but I have to say that I'm guilty of not creating submitting new bug signatures for elastic recheck | 16:30 |
*** ajayaa has joined #openstack-nova | 16:30 | |
dansmith | cfriesen: that's different than just rechecking your patch a few times before bothering to see why they're failing | 16:30 |
dansmith | hell hath many rings, as you know :) | 16:31 |
cfriesen | :) | 16:31 |
*** ijw has joined #openstack-nova | 16:33 | |
*** yamahata has joined #openstack-nova | 16:33 | |
*** mfisch has joined #openstack-nova | 16:34 | |
*** mfisch is now known as Guest19502 | 16:34 | |
kashyap | Maybe the no. of rechecks (after the first one) should be rate-limited if there's no details of investigation :-P | 16:36 |
*** moshele has quit IRC | 16:36 | |
*** unicell has joined #openstack-nova | 16:37 | |
sdague | kashyap: you'd need a lot of infrastructure to support that | 16:38 |
*** baoli has quit IRC | 16:38 | |
kashyap | sdague: 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 rechecks | 16:38 |
sdague | infra doesn't have pain with it | 16:39 |
dansmith | and that's not the problem with it, it's that people grind reviews until they land by chance | 16:39 |
sdague | all the developers do if all the nodes get taken up with un passable jobs | 16:39 |
dansmith | instead of finding that three runs of their stuff failed twice for real reasons | 16:39 |
sdague | yep | 16:39 |
kashyap | I thought it's kind of a DOS-attach where all the "unlimited" CI infra machines will be used up. | 16:40 |
dansmith | there's that too | 16:40 |
dansmith | but that's not the worst result of the behavior, IMHO | 16:40 |
*** flashgordon is now known as jogo | 16:40 | |
*** emagana has quit IRC | 16:41 | |
sdague | yeh, 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 IRC | 16:41 | |
dansmith | maybe we should require a corp-like recertification quiz each year to have recheck rights :D | 16:41 |
dansmith | "Alice submitted a patch and jenkins gave it a -1. What should Alice do?" | 16:42 |
dansmith | 1. Recheck and go get ice cream while it runs | 16:42 |
dansmith | 2. Say "eff it, I'm important" and recheck until it goes in | 16:42 |
dansmith | 3. Call Bob and ask for it to be pushed in because it's obviously correct | 16:42 |
dansmith | 4. Read the job report | 16:42 |
*** emagana has joined #openstack-nova | 16:42 | |
*** neelashah has quit IRC | 16:43 | |
*** redbeard has quit IRC | 16:43 | |
mtreinish | dansmith: I would probably fail, because between ice cream and reading the job results ice cream wins | 16:44 |
dansmith | mtreinish: see, that's a trap to catch the weak ones | 16:45 |
sdague | heh | 16:46 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Eventlet supports python logging directly https://review.openstack.org/179801 | 16:48 |
*** gokrokve has joined #openstack-nova | 16:50 | |
*** Nic has quit IRC | 16:51 | |
*** Nic has joined #openstack-nova | 16:52 | |
sdague | dansmith / mriedem - https://review.openstack.org/#/c/176806/ - the nova backport of the oslo sync so that services can shut down | 16:53 |
sdague | mriedem: I think you have +2 now, right? | 16:53 |
kashyap | dansmith: 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 |
mriedem | yes now i do | 16:54 |
sdague | now you both +2ed it :) | 16:54 |
sdague | one of you want to +A? | 16:54 |
*** rdo has joined #openstack-nova | 16:54 | |
*** doron_afk has quit IRC | 16:56 | |
*** sdake__ is now known as sdake | 16:58 | |
edleafe | dansmith: why 'ovoo' instead of 'ovo'? | 16:58 |
edleafe | dansmith: for oslo.versionedobjects | 16:58 |
dansmith | edleafe: where? | 16:59 |
edleafe | https://review.openstack.org/#/c/173479 | 16:59 |
edleafe | https://review.openstack.org/#/c/173479/5/nova/objects/base.py,cm L28 | 16:59 |
*** alanf-mc has joined #openstack-nova | 17:00 | |
dansmith | edleafe: in lots of nova code, we import base as obj_base | 17:00 |
dansmith | edleafe: so I was probably adding "ob" that after ovo in that sort of idea | 17:00 |
dansmith | doesn't matter of course | 17:00 |
*** dsanders has joined #openstack-nova | 17:01 | |
*** whenry has joined #openstack-nova | 17:01 | |
*** neelashah has joined #openstack-nova | 17:02 | |
edleafe | dansmith: yeah, nbd - just curious as I read the code | 17:02 |
*** baoli has joined #openstack-nova | 17:03 | |
dims | edleafe: its like evoo it's good for you! | 17:04 |
*** dboik_ has quit IRC | 17:04 | |
*** chhavi has quit IRC | 17:04 | |
*** dboik has joined #openstack-nova | 17:05 | |
*** baoli has quit IRC | 17:06 | |
*** zul has quit IRC | 17:06 | |
*** baoli has joined #openstack-nova | 17:06 | |
*** zul has joined #openstack-nova | 17:07 | |
*** Longgeek_ has quit IRC | 17:10 | |
openstackgerrit | garyk proposed openstack/nova: VMware: add support for NFS 4.1 https://review.openstack.org/179842 | 17:10 |
*** achanda has joined #openstack-nova | 17:11 | |
*** dikonoor has quit IRC | 17:12 | |
*** vilobhmm1 has joined #openstack-nova | 17:12 | |
*** ajayaa has quit IRC | 17:13 | |
mriedem | sdague: 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 IRC | 17:15 | |
*** Guest19502 has quit IRC | 17:15 | |
*** Guest19502 has joined #openstack-nova | 17:15 | |
*** patrickeast has joined #openstack-nova | 17:18 | |
mriedem | especially if using neutron since you should assume neutron is running with network namespaces | 17:19 |
dims | haypo: you have local changes to run py34 with tox? | 17:20 |
*** Sukhdev has joined #openstack-nova | 17:22 | |
*** armax has joined #openstack-nova | 17:23 | |
*** ssurana has joined #openstack-nova | 17:24 | |
* bauzas just standing up after 4 hours reworking on Summit session slides | 17:24 | |
*** ociuhandu has quit IRC | 17:24 | |
*** Guest19502 is now known as mfisch | 17:26 | |
haypo | dims, to run tox -e py34, i commented MySQL-python, oslo.vmware and suds in test-requirements.txt | 17:26 |
cfriesen | just 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.html | 17:26 |
*** gokrokve has quit IRC | 17:27 | |
haypo | dims, but with the current master branch, testr is simply unable to import python modules, and so fail badly | 17:27 |
mriedem | cfriesen: http://logs.openstack.org/80/170780/11/gate/gate-tempest-dsvm-postgres-full/b2e3fd4/console.html#_2015-05-04_17_05_04_323 | 17:27 |
mriedem | check the n-cpu logs for errors | 17:27 |
mriedem | cfriesen: 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_948 | 17:28 |
*** angdraug has quit IRC | 17:28 | |
haypo | dims, you need at least https://review.openstack.org/#/c/179828/ to fix syntax errors | 17:28 |
mriedem | cfriesen: so you have "libvirtError: internal error: received hangup / error event on socket" | 17:28 |
mriedem | so plug that into logstash.openstack.org | 17:28 |
mriedem | cfriesen: logstash shows it's not just your change http://goo.gl/PIVpb6 | 17:29 |
*** dboik_ has joined #openstack-nova | 17:29 | |
mriedem | i don't see any nova bugs specifically with "libvirtError: internal error: received hangup / error event on socket" in them | 17:30 |
*** salv-orlando has joined #openstack-nova | 17:30 | |
cfriesen | mriedem: thanks for the lesson | 17:30 |
*** armax has quit IRC | 17:31 | |
mriedem | cfriesen: well, https://docs.google.com/presentation/d/1xUxNeQrA-1nK1T3VD1JGsWNBD8In7qI3SXXwQYgOr-Y/edit?usp=sharing | 17:31 |
haypo | dims, 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 |
mriedem | i might use this actually :) | 17:31 |
*** ociuhandu has joined #openstack-nova | 17:31 | |
mriedem | mtreinish: ^ | 17:31 |
*** IanGovett has quit IRC | 17:31 | |
*** IanGovett has joined #openstack-nova | 17:31 | |
cfriesen | mriedem: I found the right person then. :) | 17:32 |
*** dguitarbite has quit IRC | 17:32 | |
openstackgerrit | Victor Stinner proposed openstack/nova: Replace unicode with six.text_type https://review.openstack.org/179850 | 17:33 |
*** dboik has quit IRC | 17:34 | |
*** harlowja has joined #openstack-nova | 17:34 | |
mriedem | cfriesen: https://bugs.launchpad.net/nova/+bug/1451506 | 17:35 |
openstack | Launchpad 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 |
dansmith | alaski: it's run_instance() we get to delete now, right? | 17:36 |
*** baoli has quit IRC | 17:36 | |
dansmith | alaski: it wasn't marked for deprecation so I just want to be sure | 17:36 |
alaski | dansmith: yep. hmm, it should have been | 17:36 |
dansmith | t'wasn't | 17:36 |
dansmith | although it's not in rpcapi | 17:36 |
dansmith | alaski: will be looking for you to review the drop patch for cruft and issues :) | 17:37 |
*** baoli has joined #openstack-nova | 17:37 | |
*** thomasem has quit IRC | 17:37 | |
alaski | dansmith: I will review with a vengeance | 17:37 |
cfriesen | mriedem: thanks. given that it's not blocking everything, do I just do a recheck on my patch? | 17:37 |
alaski | dansmith: https://github.com/openstack/nova/tree/master/nova/compute#L2532 | 17:38 |
mriedem | cfriesen: yeah, there is actually another test that failed in there, a timeout waiting for a state change | 17:38 |
mriedem | i didn't dig into that one | 17:38 |
*** bkopilov has quit IRC | 17:38 | |
dansmith | alaski: hmm, I hope the versions are off on that | 17:38 |
*** baoli has quit IRC | 17:38 | |
*** salv-orlando has quit IRC | 17:39 | |
*** baoli has joined #openstack-nova | 17:39 | |
alaski | dansmith: I think I wrote that when we reliably incremented each release | 17:39 |
dansmith | alaski: 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.x | 17:39 |
dansmith | hmm, okay | 17:39 |
dansmith | anyway | 17:39 |
alaski | so I really should have written Kilo/L or something | 17:40 |
*** gokrokve has joined #openstack-nova | 17:41 | |
dansmith | well, writing a version number is more helpful | 17:41 |
dansmith | because we have things in the code like: | 17:41 |
dansmith | # NOTE: remove in grizzly | 17:41 |
alaski | heh | 17:41 |
*** russellb has quit IRC | 17:42 | |
*** alanf-mc has quit IRC | 17:42 | |
mriedem | remove in manwich | 17:42 |
*** jogo has quit IRC | 17:42 | |
mriedem | we might not call it manwich though | 17:42 |
dansmith | manwich would be awesome, | 17:43 |
alaski | +1 | 17:43 |
dansmith | but I bet people outside the US wouldn't think much of it | 17:43 |
mriedem | it would be a cheap mess | 17:43 |
*** alanf-mc has joined #openstack-nova | 17:43 | |
dansmith | and it's probably a gender sin | 17:43 |
mriedem | ladywich? | 17:43 |
dansmith | so it should be personwich, which isn't good for lany other releases | 17:43 |
dansmith | and kinda loses the snazz | 17:43 |
mtreinish | mriedem: what am I supposed to be looking at? | 17:43 |
*** jogo has joined #openstack-nova | 17:43 | |
*** russellb has joined #openstack-nova | 17:43 | |
mriedem | mtreinish: heh - i found an example for the e-r workflow, | 17:44 |
mriedem | from 'what failed' to e-r query | 17:44 |
mtreinish | mriedem: ah cool | 17:44 |
*** gokrokve has quit IRC | 17:45 | |
*** gokrokve has joined #openstack-nova | 17:45 | |
*** mwagner_lap has joined #openstack-nova | 17:45 | |
*** haypo has quit IRC | 17:48 | |
*** apuimedo has joined #openstack-nova | 17:48 | |
sdague | mriedem: we should actually deprecate file injection entirely | 17:50 |
sdague | we actually don't use it in most of the upstream testing any more | 17:50 |
sdague | but the whole mechanism should be deprecated and removed | 17:50 |
dansmith | and then spit upon with disdain | 17:51 |
*** shyama has quit IRC | 17:52 | |
mriedem | anything stopping us from doing that? | 17:52 |
*** ZZelle has quit IRC | 17:53 | |
*** ZZelle has joined #openstack-nova | 17:53 | |
alaski | some providers still rely on it :| | 17:55 |
alaski | though with a deprecation cycle, I would still like to see it removed | 17:55 |
mriedem | yeah so assuming deprecation cycle, | 17:55 |
mriedem | should 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 |
alaski | seems reasonable | 17:56 |
mriedem | not it? | 17:56 |
*** aix has quit IRC | 17:58 | |
*** harlowja has quit IRC | 17:59 | |
*** harlowja_ has joined #openstack-nova | 17:59 | |
*** tonytan4ever has quit IRC | 17:59 | |
alaski | I'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 terrible | 17:59 |
*** thomasem has joined #openstack-nova | 17:59 | |
mriedem | that's why i'm not volunteering to write that up | 17:59 |
mriedem | i don't know enough about the details | 17:59 |
dansmith | philosophically terrible isn't enough? | 17:59 |
dansmith | IMHO, | 17:59 |
alaski | dansmith: for me yes, operators probably not | 18:00 |
dansmith | things like this are part of the inward-facing "API" to to speak, | 18:00 |
dansmith | which is the environment of the guest | 18:00 |
dansmith | it's a HUGE part of actual portability between clouds | 18:00 |
dansmith | which is one reason I want a minimum level of functionality for each driver | 18:00 |
*** armax has joined #openstack-nova | 18:00 | |
*** kaufer1 has joined #openstack-nova | 18:00 | |
dansmith | however, 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 it | 18:01 |
jogo | johnthetubaguy: https://review.openstack.org/#/c/179553/ FYI | 18:01 |
*** emagana has quit IRC | 18:02 | |
alaski | dansmith: thanks, that helps | 18:02 |
*** kaufer has quit IRC | 18:02 | |
mriedem | when 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 IRC | 18:03 | |
*** exploreshaifali has joined #openstack-nova | 18:03 | |
*** exploreshaifali_ has joined #openstack-nova | 18:03 | |
*** exploreshaifali has quit IRC | 18:04 | |
*** exploreshaifali_ has quit IRC | 18:04 | |
*** exploreshaifali has joined #openstack-nova | 18:04 | |
dansmith | mriedem: correct | 18:05 |
*** EmilienM is now known as EmilienM|mtg | 18:05 | |
*** sandywalsh_ has joined #openstack-nova | 18:05 | |
*** sandywalsh has quit IRC | 18:06 | |
*** promulo has quit IRC | 18:06 | |
*** jcoufal has joined #openstack-nova | 18:08 | |
*** promulo has joined #openstack-nova | 18:09 | |
*** vilobhmm1 has quit IRC | 18:09 | |
*** packet has quit IRC | 18:09 | |
*** Sukhdev has quit IRC | 18:09 | |
*** vilobhmm1 has joined #openstack-nova | 18:09 | |
*** otter768 has joined #openstack-nova | 18:10 | |
*** emagana has joined #openstack-nova | 18:11 | |
*** Marga_ has joined #openstack-nova | 18:12 | |
*** exploreshaifali has quit IRC | 18:13 | |
*** otter768 has quit IRC | 18:15 | |
sdague | dansmith: so, you sound like you have the strongest opinions, I nominate you to handle this deprecation :) | 18:20 |
*** pixelbeat has joined #openstack-nova | 18:20 | |
mriedem | 2nd | 18:20 |
dansmith | no | 18:21 |
sdague | dansmith / alaski either of you want to fix stable/kilo unit tests with a magic +A - https://review.openstack.org/#/c/179805/2 | 18:21 |
dansmith | thanks though :) | 18:21 |
openstackgerrit | Joe Gordon proposed openstack/nova: Mark ironic credential config as secret https://review.openstack.org/179857 | 18:21 |
sdague | dansmith: hey, I did ec2, that was much harder to navigate | 18:21 |
*** EmilienM|mtg is now known as EmilienM | 18:21 | |
mtreinish | sdague: +A | 18:22 |
sdague | if it hadn't been my original patch I'd have fast approved it, but I felt weird doing so, so thanks mtreinish | 18:23 |
alaski | sdague: I don't have +2 on stable | 18:25 |
sdague | alaski: oh, right | 18:25 |
alaski | but +2 from the sidelines | 18:25 |
*** jichen has joined #openstack-nova | 18:27 | |
*** Sukhdev has joined #openstack-nova | 18:29 | |
dansmith | alaski: guess what sucks... | 18:30 |
*** sdake_ has joined #openstack-nova | 18:30 | |
alaski | dansmith: I can't even narrow it down for a guess | 18:31 |
dansmith | well, | 18:31 |
dansmith | let me do some more digging first | 18:31 |
*** dboik_ has quit IRC | 18:31 | |
dansmith | yeah | 18:33 |
*** tonytan4ever has joined #openstack-nova | 18:33 | |
dansmith | alaski: we have a bunch of tests for things that use run_instance() to poke a condition | 18:33 |
dansmith | which puts me at pretty much FML status given where I am in this rpc cleanup | 18:33 |
*** sdake has quit IRC | 18:34 | |
*** sdake_ is now known as sdake | 18:34 | |
alaski | dansmith: and these conditions still need poking after run_instance is removed? | 18:35 |
dansmith | alaski: yeah, they're testing things like instance lock, file injection, ect | 18:35 |
dansmith | er, etc | 18:35 |
alaski | dansmith: damn | 18:35 |
dansmith | I guess I should refactor these in a patch before the compute rpc cleanup | 18:36 |
dansmith | because it's already large | 18:36 |
dansmith | (although oh-so-sweet) | 18:36 |
alaski | I 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 forward | 18:37 |
dansmith | no, | 18:37 |
*** boris-42 has joined #openstack-nova | 18:37 | |
dansmith | I'm not asking you to, I'm just bitching an moaning | 18:37 |
dansmith | act like you feel sorry for me and that's all I need :P | 18:37 |
alaski | cool, well complain on. I know it's bad, that's why I replaced it :) so I feel your pain | 18:38 |
dansmith | I just hate finding debt under rocks :) | 18:38 |
dansmith | so far, the cleanup is: 3 files changed, 103 insertions(+), 1465 deletions(-) | 18:39 |
alaski | that's excellent | 18:39 |
alaski | deleted code is the best code | 18:39 |
*** igordcard has joined #openstack-nova | 18:40 | |
dansmith | generally yeah | 18:40 |
*** lpetrut has joined #openstack-nova | 18:41 | |
*** gokrokve has quit IRC | 18:44 | |
dansmith | sdague: don't we have a spawn fixture that prevents actually spawning and just runs immediately? | 18:44 |
openstackgerrit | Michael Bayer proposed openstack/nova: Add an index to virtual_interfaces.uuid https://review.openstack.org/154243 | 18:45 |
openstackgerrit | Michael Bayer proposed openstack/nova: Add an index to virtual_interfaces.uuid https://review.openstack.org/154243 | 18:47 |
*** gokrokve has joined #openstack-nova | 18:48 | |
*** subscope_ has joined #openstack-nova | 18:49 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/179870 | 18:51 |
*** IanGovett1 has joined #openstack-nova | 18:52 | |
*** ildikov has joined #openstack-nova | 18:52 | |
*** ijw has quit IRC | 18:52 | |
*** oro has joined #openstack-nova | 18:52 | |
sdague | dansmith: I'm not sure | 18:53 |
*** IanGovett has quit IRC | 18:54 | |
lifeless | dansmith: theres a fakePopen fixture in fixtures | 18:55 |
*** ijw has joined #openstack-nova | 18:56 | |
dansmith | lifeless: not that spwan | 18:57 |
dansmith | sdague: could have sworn, but I don't see it.. so, level 3 rabbit hole :/ | 18:57 |
lifeless | dansmith: oh, doh :) | 18:57 |
*** VW_ has quit IRC | 19:02 | |
*** kaufer has joined #openstack-nova | 19:02 | |
*** VW_ has joined #openstack-nova | 19:02 | |
*** kaufer1 has quit IRC | 19:02 | |
*** Marga_ has quit IRC | 19:04 | |
*** dboik has joined #openstack-nova | 19:09 | |
*** sdake_ has joined #openstack-nova | 19:09 | |
*** VW_ has quit IRC | 19:09 | |
*** sandywalsh_ has quit IRC | 19:09 | |
*** VW_ has joined #openstack-nova | 19:10 | |
*** VW_ has quit IRC | 19:10 | |
*** neelashah has quit IRC | 19:10 | |
*** VW_ has joined #openstack-nova | 19:10 | |
*** mjturek1 has quit IRC | 19:11 | |
*** pixelbeat has quit IRC | 19:11 | |
*** VW_ has quit IRC | 19:11 | |
*** VW_ has joined #openstack-nova | 19:12 | |
*** artom has quit IRC | 19:13 | |
*** sdake has quit IRC | 19:13 | |
*** dprince has quit IRC | 19:13 | |
*** sandywalsh has joined #openstack-nova | 19:15 | |
*** artom has joined #openstack-nova | 19:16 | |
cfriesen | what's the URL to the overview page for all the various openstack CI jobs? my google-fu is failing. | 19:17 |
*** artom has quit IRC | 19:17 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Try running a single test module with python34 https://review.openstack.org/179893 | 19:17 |
*** artom has joined #openstack-nova | 19:17 | |
dansmith | cfriesen: status.openstack.org | 19:18 |
dansmith | and /zuul probably is what you want | 19:18 |
*** mjturek1 has joined #openstack-nova | 19:18 | |
cfriesen | dansmith: thanks, that's exactly what I was looking for. Had seen it previously and then couldn't find it again. | 19:18 |
dansmith | alaski: https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute.py#L1418 | 19:19 |
dansmith | alaski: that seems to be looking for specific behavior related to run_instance | 19:19 |
dansmith | alaski: we definitely don't honor that anymore because we spawn() the work right away | 19:19 |
dansmith | alaski: think there is value in trying to convert that somehow, or just drop it? | 19:19 |
*** ildikov has quit IRC | 19:19 | |
*** mjturek1 has quit IRC | 19:19 | |
*** mjturek1 has joined #openstack-nova | 19:20 | |
*** emagana has quit IRC | 19:21 | |
*** ZZelle_ has joined #openstack-nova | 19:22 | |
dansmith | alaski: also the _ram_fail() one a couple down from it | 19:22 |
*** salv-orlando has joined #openstack-nova | 19:22 | |
*** claudiub has quit IRC | 19:26 | |
*** alexpilotti has quit IRC | 19:29 | |
alaski | dansmith: yeah, build_and_run instance won't raise the exception. | 19:30 |
alaski | but _do_build_and_run_instance should, so you could try using that | 19:30 |
dansmith | alaski: well, I'm mocking spawn_n() to just run the function and I'm not getting the same behavior | 19:30 |
*** sudipto has quit IRC | 19:31 | |
dansmith | I seem to be for some of the ComputeResourcesUnavailable tests, but not all | 19:31 |
dansmith | I guess I'll finish them all and then see which ones don't pass and then try to decide why and what to do | 19:32 |
*** ildikov has joined #openstack-nova | 19:32 | |
*** thomasem has quit IRC | 19:32 | |
*** salv-orlando has quit IRC | 19:33 | |
alaski | dansmith: the only value I see in that test is ensuring that a resource failure hits the wrappers which log/notify/whatever | 19:34 |
alaski | not that it's raised to the caller | 19:34 |
*** mjturek2 has joined #openstack-nova | 19:34 | |
dansmith | okay | 19:35 |
dansmith | wasn't run_instance a cast before anyway? | 19:35 |
alaski | for ram_fail the important thing is that limits are properly passed to rt.instance_claim | 19:35 |
alaski | dansmith: yep | 19:35 |
dansmith | okay, that gives me some leeway | 19:36 |
alaski | these tests seem very roudabout in how they're checking their conditions | 19:36 |
*** mjturek1 has quit IRC | 19:36 | |
dansmith | yeah | 19:36 |
*** jgrimm is now known as zz_jgrimm | 19:37 | |
*** mjturek2 has quit IRC | 19:38 | |
*** mjturek1 has joined #openstack-nova | 19:42 | |
*** thangp has quit IRC | 19:44 | |
*** igordcard has quit IRC | 19:45 | |
*** emagana has joined #openstack-nova | 19:48 | |
*** unicell has quit IRC | 19:49 | |
*** emagana_ has joined #openstack-nova | 19:51 | |
*** dsanders has quit IRC | 19:52 | |
*** emagana has quit IRC | 19:52 | |
*** dsanders has joined #openstack-nova | 19:52 | |
openstackgerrit | Vilobh Meshram proposed openstack/nova-specs: Replace service groups with the tooz groups https://review.openstack.org/138607 | 19:54 |
*** thomasem has joined #openstack-nova | 19:56 | |
*** dboik has quit IRC | 19:59 | |
*** mriedem has quit IRC | 19:59 | |
*** dboik has joined #openstack-nova | 19:59 | |
*** jcoufal has quit IRC | 20:00 | |
*** dboik_ has joined #openstack-nova | 20:02 | |
*** claudiub has joined #openstack-nova | 20:03 | |
*** jdurgin has quit IRC | 20:04 | |
*** dboik has quit IRC | 20:05 | |
*** josecastroleon has quit IRC | 20:07 | |
bauzas | alaski: 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 IMHO | 20:08 |
*** subscope_ has quit IRC | 20:08 | |
bauzas | alaski: while I definitely prefer the latter, the former just fixes a small bug so I'm looking at another guy opinion :) | 20:08 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/179870 | 20:09 |
*** igordcard has joined #openstack-nova | 20:09 | |
bauzas | alaski: since force_hosts just bypasses the whole filters, it doesn't need to be in the sched | 20:09 |
*** josecastroleon has joined #openstack-nova | 20:09 | |
*** doude has quit IRC | 20:10 | |
alaski | bauzas: I'm not against that patch landing. though I'll admit to some surprise that anyone wants to run the chance scheduler | 20:10 |
*** doude has joined #openstack-nova | 20:10 | |
bauzas | alaski: agreed | 20:10 |
bauzas | alaski: but since it's in-tree, we have to accept that :) | 20:11 |
*** otter768 has joined #openstack-nova | 20:11 | |
alaski | well it's simple for testing, which is really what I thought it's purpose was | 20:13 |
*** Anticime1 has joined #openstack-nova | 20:14 | |
*** Anticimex has quit IRC | 20:15 | |
johnthetubaguy | jogo: thanks for that patch, got some dev ref stuff cooking too, but UK holiday today getting in my way | 20:15 |
bauzas | oh johnthetubaguy working on holiday ? :) | 20:15 |
*** otter768 has quit IRC | 20:16 | |
jogo | johnthetubaguy: yeah I saw, I am trying to remove dead content and not add new right now | 20:18 |
jogo | and work around your patches too | 20:18 |
bauzas | alaski: 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 |
bauzas | alaski: that means that it checks your hosts and only return which ones are good | 20:19 |
bauzas | alaski: so I won't eventually open a new BP for that | 20:19 |
*** jdurgin has joined #openstack-nova | 20:22 | |
openstackgerrit | John Garbutt proposed openstack/nova: devref: add information to clarify nova scope https://review.openstack.org/178623 | 20:22 |
cfriesen | dansmith: I know how much you like evacuate...https://review.openstack.org/#/c/169827/ | 20:22 |
*** matrohon has joined #openstack-nova | 20:23 | |
dansmith | cfriesen: busy with other stuff today, so doubt I'll get to it | 20:23 |
*** d34dh0r53 has quit IRC | 20:23 | |
*** xyhuang has joined #openstack-nova | 20:24 | |
cfriesen | dansmith: fair enough. | 20:24 |
*** toan has quit IRC | 20:24 | |
*** d34dh0r53 has joined #openstack-nova | 20:24 | |
*** toan has joined #openstack-nova | 20:25 | |
*** htruta has quit IRC | 20:26 | |
*** ociuhandu has quit IRC | 20:29 | |
*** Sukhdev has quit IRC | 20:29 | |
*** exploreshaifali has joined #openstack-nova | 20:29 | |
*** boris-42 has quit IRC | 20:30 | |
*** ramishra has quit IRC | 20:30 | |
*** kun_huang has quit IRC | 20:30 | |
*** cyeoh has quit IRC | 20:31 | |
*** apmelton has quit IRC | 20:31 | |
*** toan has quit IRC | 20:31 | |
*** dboik_ has quit IRC | 20:31 | |
*** Nakato has quit IRC | 20:32 | |
*** lpetrut has quit IRC | 20:32 | |
*** toan has joined #openstack-nova | 20:33 | |
*** Nakato has joined #openstack-nova | 20:33 | |
*** ivasev has quit IRC | 20:33 | |
*** apmelton has joined #openstack-nova | 20:34 | |
*** unicell has joined #openstack-nova | 20:34 | |
*** cyeoh has joined #openstack-nova | 20:34 | |
*** d34dh0r53 has quit IRC | 20:34 | |
*** dboik has joined #openstack-nova | 20:34 | |
*** ssurana1 has joined #openstack-nova | 20:34 | |
*** kun_huang has joined #openstack-nova | 20:34 | |
*** htruta has joined #openstack-nova | 20:34 | |
*** d34dh0r53 has joined #openstack-nova | 20:35 | |
*** Marga_ has joined #openstack-nova | 20:35 | |
*** ramishra has joined #openstack-nova | 20:35 | |
*** ssurana has quit IRC | 20:35 | |
*** boris-42 has joined #openstack-nova | 20:36 | |
*** jdurgin has quit IRC | 20:37 | |
*** xgsa has joined #openstack-nova | 20:39 | |
*** vladikr has quit IRC | 20:39 | |
*** fawadkhaliq has joined #openstack-nova | 20:40 | |
*** zz_jgrimm has quit IRC | 20:43 | |
*** maurosr has quit IRC | 20:44 | |
*** apuimedo has quit IRC | 20:45 | |
*** sdake has joined #openstack-nova | 20:45 | |
*** maurosr has joined #openstack-nova | 20:46 | |
*** zz_jgrimm has joined #openstack-nova | 20:46 | |
jogo | cfriesen: any chance you have a bug fix for making shelve unshelve work? | 20:48 |
*** sdake_ has quit IRC | 20:49 | |
*** exploreshaifali has quit IRC | 20:51 | |
*** sdake_ has joined #openstack-nova | 20:52 | |
*** browne has joined #openstack-nova | 20:53 | |
*** jdurgin has joined #openstack-nova | 20:53 | |
*** fawadkhaliq has quit IRC | 20:55 | |
*** salv-orlando has joined #openstack-nova | 20:55 | |
*** sdake has quit IRC | 20:56 | |
*** matrohon has quit IRC | 20:56 | |
cfriesen | jogo: don't think so. haven't tried it recently | 20:59 |
cfriesen | jogo: what's wrong with it? | 21:00 |
*** Sukhdev has joined #openstack-nova | 21:00 | |
jogo | cfriesen: it doesn't work well with nova-network | 21:01 |
jogo | cfriesen: are multinode testing http://logs.openstack.org/80/170780/11/check/check-tempest-dsvm-multinode-full/31d7552/console.html | 21:01 |
*** busterswt has joined #openstack-nova | 21:03 | |
*** tonytan4ever has quit IRC | 21:05 | |
*** raildo has quit IRC | 21:05 | |
*** artom has quit IRC | 21:05 | |
*** artom has joined #openstack-nova | 21:05 | |
cfriesen | jogo: 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 |
jogo | cfriesen: it may move the instance to a different machine | 21:06 |
cfriesen | jogo: ah, okay. does that testcase use floatingIP? | 21:07 |
*** pixelbeat has joined #openstack-nova | 21:07 | |
*** xgsa has quit IRC | 21:07 | |
*** artom has quit IRC | 21:07 | |
*** artom has joined #openstack-nova | 21:08 | |
*** thomasem has quit IRC | 21:08 | |
cfriesen | jogo: 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 path | 21:09 |
jogo | cfriesen: AFAIK yes | 21:10 |
jogo | cfriesen: so the test works sometimes but not always | 21:10 |
cfriesen | jogo: do the failures correlate with changing hosts? | 21:10 |
*** jichen has quit IRC | 21:10 | |
jogo | cfriesen: not 100% | 21:11 |
cfriesen | jogo: annoying. sorry, I've got nothing else. :) | 21:15 |
jogo | cfriesen: also unshelve does call self.network_api.setup_instance_network_on_host | 21:16 |
jogo | cfriesen: thanks anyway | 21:16 |
*** haypo has joined #openstack-nova | 21:17 | |
*** mmedvede has quit IRC | 21:18 | |
*** eharney has quit IRC | 21:22 | |
openstackgerrit | Victor Stinner proposed openstack/nova: Fix raise syntax for Python 3 https://review.openstack.org/179946 | 21:28 |
openstackgerrit | Artom Lifshitz proposed openstack/nova: Make get_pagination_params respect osapi_max_limit https://review.openstack.org/179947 | 21:29 |
openstackgerrit | Joe Gordon proposed openstack/nova: Mark ironic credential config as secret https://review.openstack.org/179857 | 21:32 |
*** Jeffrey4l has quit IRC | 21:35 | |
*** mwagner_lap has quit IRC | 21:35 | |
*** igordcard has quit IRC | 21:36 | |
openstackgerrit | Dan Smith proposed openstack/nova: Add SpawnFixture https://review.openstack.org/179949 | 21:38 |
openstackgerrit | Dan Smith proposed openstack/nova: WIP: Refactor tests that use compute's deprecated run_instance() method https://review.openstack.org/179950 | 21:38 |
openstackgerrit | Dan Smith proposed openstack/nova: Drop compute RPC 3.x support https://review.openstack.org/179951 | 21:38 |
dansmith | alaski: that middle patch isn't quite there yet.. there seem to be some actual serious differences between the two, around things like notifications especially | 21:38 |
dansmith | alaski: I'm hoping that maybe tomorrow you could take a look at some of those failures and see what you thihnk | 21:38 |
dansmith | I've had it with that crap and have to go do something else before I go crazy | 21:39 |
alaski | dansmith: can do | 21:41 |
dansmith | thanks | 21:42 |
* dansmith & | 21:42 | |
openstackgerrit | Victor Stinner proposed openstack/nova: Replace dict.itervalues() with six.itervalues(dict) https://review.openstack.org/179952 | 21:45 |
*** VW_ has quit IRC | 21:46 | |
*** aix has joined #openstack-nova | 21:46 | |
*** Jeffrey4l has joined #openstack-nova | 21:47 | |
jogo | alaski: ping shelve/unshelve | 21:48 |
*** artom has quit IRC | 21:49 | |
jogo | alaski: 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 on | 21:49 |
openstack | Launchpad bug 1445569 in OpenStack Compute (nova) "No dhcp lease after shelve unshelve" [High,Confirmed] | 21:49 |
jogo | alaski: and being you wrote the shelve code originally ... I908474915c5325e120a64af9b9af13fcbce1f01c | 21:50 |
alaski | jogo: I'm not really familiar with the n-net networking stuff, I would have to spend some time there to be helpful | 21:51 |
alaski | jogo: when writing shelve I used a lot of what resize did | 21:51 |
*** rfolco has quit IRC | 21:51 | |
*** dave-mccowan has quit IRC | 21:51 | |
alaski | all of the manipulation to move networking to a new host should copy what's used for resize | 21:52 |
jogo | alaski: hmm that is a good point | 21:52 |
*** ekhugen has quit IRC | 21:52 | |
jogo | I wonder if we have a resize test | 21:52 |
alaski | it's possible that I missed something, or they've drifted apart. or that resize would have the same issue | 21:53 |
*** zz_jgrimm has quit IRC | 21:54 | |
*** timello has quit IRC | 21:54 | |
jogo | alaski: ahh we have allow_migrate and allow resize to same host = True | 21:55 |
jogo | so we may be seeing those issues | 21:55 |
*** kaufer has quit IRC | 21:55 | |
*** ekhugen has joined #openstack-nova | 21:55 | |
*** apuimedo has joined #openstack-nova | 21:56 | |
*** zz_jgrimm has joined #openstack-nova | 21:56 | |
*** timello has joined #openstack-nova | 21:57 | |
jogo | maybe not, looking at the wrong log | 21:57 |
alaski | wife putting dinner out, gotta run | 21:59 |
jogo | alaski: thanks o/ | 21:59 |
*** dboik has quit IRC | 22:02 | |
*** thomasem has joined #openstack-nova | 22:02 | |
*** harlowja_ has quit IRC | 22:06 | |
*** harlowja has joined #openstack-nova | 22:07 | |
*** ekhugen has quit IRC | 22:08 | |
*** zz_jgrimm has quit IRC | 22:09 | |
*** mmedvede has joined #openstack-nova | 22:09 | |
*** maurosr has quit IRC | 22:11 | |
*** ekhugen has joined #openstack-nova | 22:11 | |
*** pixelbeat has quit IRC | 22:11 | |
*** otter768 has joined #openstack-nova | 22:12 | |
*** zz_jgrimm has joined #openstack-nova | 22:12 | |
*** busterswt has quit IRC | 22:12 | |
*** oro has quit IRC | 22:13 | |
*** maurosr has joined #openstack-nova | 22:14 | |
*** otter768 has quit IRC | 22:16 | |
*** zz_jgrimm has quit IRC | 22:16 | |
openstackgerrit | Mitsuhiro Tanino proposed openstack/nova: Prevent unintentional detaching volume from instance https://review.openstack.org/166695 | 22:16 |
*** maurosr has quit IRC | 22:19 | |
*** ekhugen has quit IRC | 22:19 | |
*** zz_jgrimm has joined #openstack-nova | 22:19 | |
*** ildikov has quit IRC | 22:20 | |
*** maurosr has joined #openstack-nova | 22:20 | |
*** igordcard has joined #openstack-nova | 22:21 | |
*** ekhugen has joined #openstack-nova | 22:21 | |
*** igordcard has joined #openstack-nova | 22:21 | |
*** mwagner_lap has joined #openstack-nova | 22:22 | |
*** igordcard has quit IRC | 22:31 | |
openstackgerrit | Eric Brown proposed openstack/nova: Deprecate nova ironic driver's admin_auth_token https://review.openstack.org/179960 | 22:32 |
*** mmedvede has quit IRC | 22:32 | |
*** aysyd has quit IRC | 22:33 | |
*** ildikov has joined #openstack-nova | 22:33 | |
*** oro has joined #openstack-nova | 22:35 | |
*** igordcard has joined #openstack-nova | 22:37 | |
*** baoli has quit IRC | 22:37 | |
*** VW_ has joined #openstack-nova | 22:42 | |
*** VW_ has quit IRC | 22:42 | |
*** VW_ has joined #openstack-nova | 22:42 | |
*** VW_ has quit IRC | 22:43 | |
*** IanGovett1 has quit IRC | 22:47 | |
*** VW_ has joined #openstack-nova | 22:49 | |
*** ZZelle_ has quit IRC | 22:55 | |
*** armax has quit IRC | 22:57 | |
*** thomasem has quit IRC | 22:57 | |
*** Sukhdev has quit IRC | 23:00 | |
*** xyhuang has quit IRC | 23:01 | |
*** oro has quit IRC | 23:01 | |
*** thomasem has joined #openstack-nova | 23:01 | |
*** zz_jgrimm has quit IRC | 23:03 | |
*** sdake has joined #openstack-nova | 23:04 | |
*** thomasem has quit IRC | 23:05 | |
*** zz_jgrimm has joined #openstack-nova | 23:06 | |
*** vilobhmm1 has quit IRC | 23:08 | |
*** vilobhmm1 has joined #openstack-nova | 23:08 | |
*** sdake_ has quit IRC | 23:08 | |
*** Marga_ has quit IRC | 23:09 | |
*** Marga_ has joined #openstack-nova | 23:12 | |
*** thomasem has joined #openstack-nova | 23:15 | |
*** Marga_ has quit IRC | 23:15 | |
*** thomasem has quit IRC | 23:16 | |
*** thomasem has joined #openstack-nova | 23:17 | |
*** thomasem has quit IRC | 23:18 | |
*** VW_ has quit IRC | 23:20 | |
*** zz_jgrimm has quit IRC | 23:27 | |
*** maurosr has quit IRC | 23:28 | |
*** maurosr has joined #openstack-nova | 23:29 | |
*** zz_jgrimm has joined #openstack-nova | 23:29 | |
*** IanGovett has joined #openstack-nova | 23:35 | |
*** hemna is now known as hemnafk | 23:37 | |
*** claudiub has quit IRC | 23:37 | |
*** Marga_ has joined #openstack-nova | 23:38 | |
*** gokrokve has quit IRC | 23:39 | |
*** vilobhmm1 has quit IRC | 23:41 | |
*** dims_ has joined #openstack-nova | 23:42 | |
*** dims has quit IRC | 23:44 | |
*** flwang1 has joined #openstack-nova | 23:45 | |
flwang1 | garyk: could you pls help review https://review.openstack.org/168759 ? | 23:45 |
flwang1 | adam_g: ^ thanks | 23:46 |
flwang1 | it's a cherrypick to juno | 23:46 |
*** signed8bit is now known as signed8bit_ZZZzz | 23:50 | |
*** thomasem has joined #openstack-nova | 23:52 | |
*** Raildo has joined #openstack-nova | 23:54 | |
*** dave-mccowan has joined #openstack-nova | 23:56 | |
*** IanGovett has quit IRC | 23:56 | |
*** BadCub is now known as BadCub_Hiding | 23:58 | |
*** sdake_ has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!