Monday, 2016-12-19

*** thorst has quit IRC00:06
openstackgerritTakashi NATSUME proposed openstack/nova-specs: List/show all server migration types  https://review.openstack.org/40723700:08
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Abort Cold Migration  https://review.openstack.org/33473200:08
*** ducttape_ has joined #openstack-nova00:08
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Fix formatting errors and warnings  https://review.openstack.org/40736400:08
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec  https://review.openstack.org/39719300:08
openstackgerritTakashi NATSUME proposed openstack/nova: Remove mox in test_block_device_mapping_v1.py  https://review.openstack.org/30874600:08
openstackgerritTakashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases  https://review.openstack.org/39543600:09
*** baoli has quit IRC00:09
*** dimtruck is now known as zz_dimtruck00:15
*** fragatina has joined #openstack-nova00:40
*** yingjun has joined #openstack-nova00:41
*** Hazelesque has quit IRC00:42
*** yingjun has quit IRC00:46
*** Hazelesque has joined #openstack-nova00:51
*** liusheng has quit IRC00:51
*** ducttape_ has quit IRC00:52
*** roeyc has quit IRC00:58
*** hoangcx has joined #openstack-nova00:59
*** phuongnh has joined #openstack-nova01:00
*** baoli has joined #openstack-nova01:00
*** ducnc has joined #openstack-nova01:03
*** thorst has joined #openstack-nova01:03
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/41214301:03
*** ducttape_ has joined #openstack-nova01:04
mriedemandreykurilin: i don't have a strong opinion01:04
mriedemprobably not a bad idea01:04
mriedembtw i still haven't gotten the trove tempest tests to run on this change https://review.openstack.org/#/c/412113/01:05
mriedemi'm not sure what i'm missing there, i'll probably ask amrith to look tomorrow01:05
*** baoli has quit IRC01:05
*** baoli has joined #openstack-nova01:07
*** thorst has quit IRC01:11
andreykurilinmriedem: I think most of openstack projects, at least "integrated", uses manually created keystone session and transmit it to novaclient. Otherwise they should be broken, since keystone switched to V3 by default(I did not see a lot of complains about this switch)01:12
andreykurilinSo my patches for most cases are just cleanup(removing HTTPClient implementation)01:13
*** ducttape_ has quit IRC01:15
*** liusheng has joined #openstack-nova01:15
*** zhangyujun has joined #openstack-nova01:16
oomichialex_xu: I put comments on https://review.openstack.org/#/c/38900301:17
oomichialex_xu: I feel we are avoiding outputting logs related to BadRequest thing to avoid much logs on the server,01:18
andreykurilinhm... It looks like I was wrong https://github.com/openstack/trove/blob/48dcbb6dcd1b120e668d8330104a6c63a3c01a96/trove/common/remote.py#L10101:19
andreykurilinmriedem: omg. Trove s*cks. They do not read any out warnings... https://github.com/openstack/trove/blob/48dcbb6dcd1b120e668d8330104a6c63a3c01a96/tools/trove-pylint.config#L36301:22
*** zhangjl has joined #openstack-nova01:22
andreykurilin*our01:22
*** kevinz has joined #openstack-nova01:22
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: replace 'osapi_max_limit' with 'max_limit'  https://review.openstack.org/41224501:24
*** ducttape_ has joined #openstack-nova01:29
*** yingjun has joined #openstack-nova01:37
*** yuntongjin has joined #openstack-nova01:40
*** yufei has joined #openstack-nova01:41
*** tuanluong has joined #openstack-nova01:46
*** trananhkma has joined #openstack-nova01:46
*** yuntongjin has quit IRC01:49
*** ducttape_ has quit IRC01:50
*** elynn has joined #openstack-nova01:54
*** thorst has joined #openstack-nova01:57
*** elynn_ has joined #openstack-nova01:58
*** thorst has quit IRC01:58
*** elynn has quit IRC01:59
*** zhs_ has joined #openstack-nova02:01
*** gongysh has joined #openstack-nova02:07
*** zz_dimtruck is now known as dimtruck02:08
*** yuntongjin has joined #openstack-nova02:11
*** zhangyujun has quit IRC02:11
*** ducttape_ has joined #openstack-nova02:12
*** fragatina has quit IRC02:17
*** dimtruck is now known as zz_dimtruck02:18
*** asettle has joined #openstack-nova02:22
*** asettle has quit IRC02:26
*** zhangyujun has joined #openstack-nova02:27
*** yingjun has quit IRC02:28
*** catintheroof has quit IRC02:28
openstackgerritint32bit proposed openstack/nova: Fix wrong disk size for sparse flat files  https://review.openstack.org/39687302:29
*** catintheroof has joined #openstack-nova02:30
*** catintheroof has quit IRC02:34
*** ducttape_ has quit IRC02:37
*** ducttape_ has joined #openstack-nova02:37
openstackgerritint32bit proposed openstack/nova: Fix error if free_disk_gb is None in CellStateManager  https://review.openstack.org/40950102:37
*** yingjun has joined #openstack-nova02:42
openstackgerritCao Xuan Hoang proposed openstack/nova: Removes unnecessary utf-8 encoding  https://review.openstack.org/41005402:50
*** ducttape_ has quit IRC02:52
*** tbachman has joined #openstack-nova02:54
*** baoli has quit IRC02:55
*** yingjun has quit IRC02:56
*** gouthamr has joined #openstack-nova02:56
*** baoli has joined #openstack-nova02:57
*** FL1SK has quit IRC03:00
*** FL1SK has joined #openstack-nova03:01
*** thorst has joined #openstack-nova03:03
openstackgerritRocDove proposed openstack/python-novaclient: modified:   novaclient/v2/servers.py  https://review.openstack.org/41225803:04
*** zz_dimtruck has quit IRC03:05
*** Jeffrey4l has quit IRC03:05
*** zz_dimtruck has joined #openstack-nova03:05
*** Jeffrey4l has joined #openstack-nova03:05
*** zz_dimtruck is now known as dimtruck03:06
*** ediardo has quit IRC03:09
*** yingjun has joined #openstack-nova03:09
*** Nakato has quit IRC03:10
*** Nakato has joined #openstack-nova03:10
*** gouthamr has quit IRC03:11
*** ediardo has joined #openstack-nova03:11
*** noama has quit IRC03:12
*** thorst has quit IRC03:12
*** noama has joined #openstack-nova03:12
*** kuzko has quit IRC03:14
*** irenab_ has joined #openstack-nova03:15
*** ducttape_ has joined #openstack-nova03:15
*** kuzko has joined #openstack-nova03:15
*** irenab has quit IRC03:17
*** irenab_ is now known as irenab03:17
*** frickler_ has joined #openstack-nova03:20
*** frickler has quit IRC03:21
*** elynn_ has quit IRC03:24
*** elynn_ has joined #openstack-nova03:25
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port libvirt.test_vif to Python 3  https://review.openstack.org/40341903:25
*** Prophidys has quit IRC03:25
*** Prophidys has joined #openstack-nova03:26
*** ducttape_ has quit IRC03:26
*** mwhahaha has quit IRC03:26
*** mwhahaha has joined #openstack-nova03:29
*** baoli has quit IRC03:30
*** baoli has joined #openstack-nova03:30
*** baoli has quit IRC03:30
alex_xuoomichi: nice idea!03:31
*** elynn_ has left #openstack-nova03:31
*** yingjun has quit IRC03:31
alex_xuoomichi: good point. so that is why there is option to turn that log off03:36
alex_xuoomichi: the previous 'nice idea' point to the improvement of log message03:37
openstackgerritZhenyu Zheng proposed openstack/nova: Check destination_type when booting with bdm provided  https://review.openstack.org/40237203:40
*** nicolasbock has joined #openstack-nova03:53
*** psachin has joined #openstack-nova03:53
*** sudipto has joined #openstack-nova03:56
*** thorst has joined #openstack-nova04:09
*** marst has quit IRC04:11
*** thorst has quit IRC04:18
*** dimtruck is now known as zz_dimtruck04:18
openstackgerritint32bit proposed openstack/nova: Add more actions to instance action logs  https://review.openstack.org/39651004:20
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port xenapi test_vm_utils to Python 3  https://review.openstack.org/40182704:22
*** yuntongjin has quit IRC04:26
*** ducttape_ has joined #openstack-nova04:26
*** ducttape_ has quit IRC04:31
*** baoli has joined #openstack-nova04:32
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/3)  https://review.openstack.org/35710504:32
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/3)  https://review.openstack.org/40895504:33
*** nicolasbock has quit IRC04:43
*** zz_dimtruck is now known as dimtruck04:46
*** siva_krish has joined #openstack-nova04:48
*** prateek has joined #openstack-nova04:49
*** Jeffrey4l has quit IRC04:53
*** baoli has quit IRC04:53
*** pbandark has joined #openstack-nova04:56
oomichialex_xu: I feel the option should be disabled at the default at least04:59
*** ducttape_ has joined #openstack-nova05:00
*** lpetrut has joined #openstack-nova05:04
openstackgerritint32bit proposed openstack/nova: Add soft delete wrinkle to api-ref  https://review.openstack.org/41144005:11
openstackgerritint32bit proposed openstack/python-novaclient: Add CLI to show instance usage audit logs  https://review.openstack.org/40941305:12
openstackgerritint32bit proposed openstack/nova: Choose VMware datastore by the least provisioned space  https://review.openstack.org/40263205:12
*** sreeram_v has joined #openstack-nova05:14
*** thorst has joined #openstack-nova05:15
*** unicell1 has joined #openstack-nova05:16
*** unicell has quit IRC05:16
*** preethipy has joined #openstack-nova05:16
*** preethi has joined #openstack-nova05:18
*** sudipto has quit IRC05:18
*** ratailor has joined #openstack-nova05:20
*** preethipy has quit IRC05:21
*** thorst has quit IRC05:22
*** bks has joined #openstack-nova05:24
*** hfu has joined #openstack-nova05:24
*** lpetrut has quit IRC05:26
*** diga has joined #openstack-nova05:31
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(3/3)  https://review.openstack.org/40896405:36
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Microversion 2.41 - Enable cold migration woth target host  https://review.openstack.org/40670705:37
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004205:37
*** ducttape_ has quit IRC05:39
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004205:40
*** sudipto has joined #openstack-nova05:47
*** mdnadeem has joined #openstack-nova05:47
*** yuntongjin has joined #openstack-nova05:47
*** gcb has joined #openstack-nova05:49
*** tojuvone has joined #openstack-nova05:53
*** amotoki has joined #openstack-nova05:55
*** amotoki has quit IRC05:58
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(3/3)  https://review.openstack.org/40896406:03
*** bkopilov has joined #openstack-nova06:04
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004206:04
openstackgerritKen'ichi Ohmichi proposed openstack/nova: Remove unused validation code from block_device  https://review.openstack.org/41000606:06
*** jaosorior has joined #openstack-nova06:06
*** ayogi has joined #openstack-nova06:07
*** sridharg has joined #openstack-nova06:10
alex_xuoomichi: or we should let people notice the change first?06:13
oomichialex_xu: humm, I cannot catch your meaning.. Is that related to reno?06:13
alex_xuoomichi: the option for the log06:14
alex_xuoomichi: if we enable it by default, then people will be noticed there is a change06:14
alex_xuoomichi: if they don't like it, they can turn it off.06:14
alex_xuoomichi: if the option is disable default. I feel no-one will notice there is new option06:15
*** yingjun has joined #openstack-nova06:15
oomichialex_xu: humm, it is better to avoid much logs which related to BadRequest on REST API layer basically06:15
alex_xuoomichi: btw, sounds like you back to Japan, due to you didn't sleep now06:15
oomichialex_xu: yeah, gmann is back here :)06:16
alex_xuoomichi: yea, agree with that, but thinking of the purpose of this option, it want to notice people there is a change06:16
*** yufei has quit IRC06:17
alex_xuoomichi: gmann back to Japan? I thought he was in Japan06:17
*** bkopilov has quit IRC06:17
oomichialex_xu: hehe, after they know, they just start blaming us like "Why we should see much logs related to that?"06:17
oomichialex_xu: yeah, gmann is also in Japan now06:18
alex_xuoomichi: then they notice that change, that is right our purpose06:18
*** amotoki has joined #openstack-nova06:18
oomichialex_xu: I don't want to see blaming :-(06:18
alex_xuoomichi: if we make it disable by default, I didn't see any reason why we want that option06:19
*** thorst has joined #openstack-nova06:19
*** yufei has joined #openstack-nova06:21
*** asettle has joined #openstack-nova06:23
alex_xuoomichi: also replied this patch https://review.openstack.org/#/c/398755/706:24
*** asettle has quit IRC06:27
*** thorst has quit IRC06:28
*** enginy has joined #openstack-nova06:32
openstackgerritZhenyu Zheng proposed openstack/nova: Fix misleading port delete description  https://review.openstack.org/41058206:32
*** yingjun has quit IRC06:33
*** enginy has quit IRC06:34
*** enginy has joined #openstack-nova06:35
*** enginy has quit IRC06:36
*** mnestratov has joined #openstack-nova06:36
*** qwertyco has joined #openstack-nova06:36
*** dimtruck is now known as zz_dimtruck06:37
*** zerda2 has joined #openstack-nova06:42
oomichijohnthetubaguy: do you want to make the option enabled on the default with https://review.openstack.org/#/c/389003/15/nova/conf/api.py ?06:44
*** moshele has joined #openstack-nova06:48
openstackgerritAlex Xu proposed openstack/nova: Use JSON-Schema to validate query parameters for keypairs API  https://review.openstack.org/39875506:48
openstackgerritAlex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters  https://review.openstack.org/38900306:48
*** lpetrut has joined #openstack-nova06:49
*** ducttape_ has joined #openstack-nova06:57
*** nherciu_ has joined #openstack-nova06:58
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Add some missing modules in API reference  https://review.openstack.org/41231506:59
*** oanson has joined #openstack-nova06:59
openstackgerritZhenyu Zheng proposed openstack/nova: Check destination_type when booting with bdm provided  https://review.openstack.org/40237207:00
*** ducttape_ has quit IRC07:02
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Microversion 2.41 - Enable cold migration woth target host  https://review.openstack.org/40670707:03
*** tbachman has quit IRC07:03
*** mjura has joined #openstack-nova07:03
*** tesseract has joined #openstack-nova07:04
*** tesseract is now known as Guest3325407:05
*** mjura has quit IRC07:09
*** mjura has joined #openstack-nova07:10
*** amotoki has quit IRC07:10
*** Oku_OS-away is now known as Oku_OS07:11
*** yingjun has joined #openstack-nova07:15
*** fragatina has joined #openstack-nova07:18
*** baoli has joined #openstack-nova07:21
*** sahid has joined #openstack-nova07:23
*** thorst has joined #openstack-nova07:25
*** baoli has quit IRC07:26
*** amotoki has joined #openstack-nova07:32
*** thorst has quit IRC07:33
*** pcaruana has joined #openstack-nova07:33
*** avolkov has joined #openstack-nova07:38
*** takashin_ has joined #openstack-nova07:40
*** andreas_s has joined #openstack-nova07:41
*** takashin has quit IRC07:42
openstackgerritSergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API.  https://review.openstack.org/35554007:44
openstackgerritzhangyanxian proposed openstack/nova-specs: Fix typo in same-instances-scheduling.rst  https://review.openstack.org/41233807:50
*** lpetrut has quit IRC07:50
openstackgerritzhangyanxian proposed openstack/nova-specs: Fix typo in same-instances-scheduling.rst  https://review.openstack.org/41233807:51
lennybwznoinsk, pls review https://review.openstack.org/#/c/407958/07:53
*** jaosorior has quit IRC07:58
*** claudiub has joined #openstack-nova07:59
*** tuanluong has quit IRC08:00
tdurakovsfinucan: hi, are you around?08:02
*** takashin_ has left #openstack-nova08:03
*** ihrachys has joined #openstack-nova08:04
*** amotoki has quit IRC08:06
*** gongysh has quit IRC08:10
Kevin_Zhengping oomichi08:16
oomichihello08:16
*** uu has joined #openstack-nova08:16
oomichiKevin_Zheng: hi08:17
Kevin_Zhengoomichi: Hi, I did a test after you left your comment08:17
oomichiKevin_Zheng: which one?08:17
oomichis/one/patch/ ?08:18
*** yingjun has quit IRC08:18
Kevin_Zhengthe destination_type one08:19
Kevin_ZhengDEBUG (session:342) REQ: curl -g -i -X POST http://10.3.150.17:8774/v2.1/os-volumes_boot -H "Accept: application/json" -H "User-Agent: python-novaclient" -H "OpenStack-API-Version: compute 2.37" -H "X-OpenStack-Nova-API-Version: 2.37" -H "X-Auth-Token: {SHA1}8cbcc4004bcc8e2acccfaaa4c0c8015657901267" -H "Content-Type: application/json" -d '{"server": {"name":08:19
Kevin_Zheng"test3", "imageRef": "2ff3a1d3-ed70-4c3f-94ac-941461153bc0", "block_device_mapping_v2": [{"source_type": "image", "delete_on_termination": true, "boot_index": 0, "uuid": "2ff3a1d3-ed70-4c3f-94ac-941461153bc0", "destination_type": "local"}, {"source_type": "volume", "uuid": "99d92140-3d0c-4ea5-a49c-f94c38c607f0", "destination_type": "volume1"}], "flavorRef":08:19
Kevin_Zheng"2", "max_count": 1, "min_count": 1, "networks": [{"uuid": "474cde7c-d959-46c1-a62e-4810058ba3a8"}]}}'08:19
Kevin_Zhengah maybe I can ping you in personal08:19
*** baoli has joined #openstack-nova08:22
*** jaosorior has joined #openstack-nova08:24
*** qwertyco has quit IRC08:24
*** amotoki has joined #openstack-nova08:25
*** avolkov has quit IRC08:26
*** ducttape_ has joined #openstack-nova08:28
*** thorst has joined #openstack-nova08:29
*** ducttape_ has quit IRC08:33
*** qwertyco has joined #openstack-nova08:36
*** yuntongjin has quit IRC08:38
*** thorst has quit IRC08:38
*** masber has joined #openstack-nova08:39
*** amotoki has quit IRC08:40
*** avolkov has joined #openstack-nova08:44
openstackgerritSergey Nikitin proposed openstack/nova-specs: Added new fields in Diagnostics info  https://review.openstack.org/41234908:44
*** baoli has quit IRC08:45
*** jaosorior has quit IRC08:45
*** jaosorior has joined #openstack-nova08:46
*** noama has quit IRC08:48
*** pbandark has quit IRC08:49
*** pbandark has joined #openstack-nova08:52
openstackgerritZhenyu Zheng proposed openstack/nova: Check destination_type when booting with bdm provided  https://review.openstack.org/40237208:53
*** rdo has quit IRC08:54
sfinucantdurakov: I am now08:55
tdurakovsfinucan: left a comment on https://review.openstack.org/#/c/410542/ already08:56
*** zzzeek has quit IRC09:00
*** tuanluong has joined #openstack-nova09:01
*** zzzeek has joined #openstack-nova09:01
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/41214309:04
*** yamahata has quit IRC09:04
*** hrw has joined #openstack-nova09:05
hrwhi09:05
openstackgerritStephen Finucane proposed openstack/nova: rbd: Remove unnecessary 'encode' calls  https://review.openstack.org/41235609:06
*** qwertyco has quit IRC09:06
*** portdirect has joined #openstack-nova09:10
*** roeyc has joined #openstack-nova09:11
openstackgerritStephen Finucane proposed openstack/nova: Always set MTU when plugging devices  https://review.openstack.org/40878109:11
*** tuanluong_ has joined #openstack-nova09:11
*** sahid has quit IRC09:13
*** sreeram_v has quit IRC09:15
*** sreeram_v has joined #openstack-nova09:16
*** sreeram_v has quit IRC09:20
openstackgerritStephen Finucane proposed openstack/nova: relnotes: Fix config options deprecation notes  https://review.openstack.org/37540009:23
*** yassine has joined #openstack-nova09:23
*** asettle has joined #openstack-nova09:23
*** yassine is now known as Guest2420509:23
*** hrw has left #openstack-nova09:24
openstackgerritFrancesco Santoro proposed openstack/nova: bump os-vif min version  https://review.openstack.org/41199109:25
openstackgerritFrancesco Santoro proposed openstack/nova: os-vif-util: set vif_name for vhostuser ovs os-vif port  https://review.openstack.org/41073709:25
openstackgerritFrancesco Santoro proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser  https://review.openstack.org/38506109:25
*** lucas-afk is now known as lucasagomes09:26
*** yingjun has joined #openstack-nova09:26
*** asettle has quit IRC09:28
*** hieulq has joined #openstack-nova09:29
*** gongysh has joined #openstack-nova09:31
*** asettle has joined #openstack-nova09:32
*** thorst has joined #openstack-nova09:35
openstackgerritAlex Xu proposed openstack/nova: Use JSON-Schema to validate query parameters for keypairs API  https://review.openstack.org/39875509:37
openstackgerritAlex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters  https://review.openstack.org/38900309:37
*** rdo has joined #openstack-nova09:38
alex_xusnikitin: just in case I didn't clear in the review. I mean we should update api reference document http://developer.openstack.org/api-ref/compute/, the source is under https://github.com/openstack/nova/tree/master/api-ref/source. It isn't spec09:40
*** thorst has quit IRC09:42
*** frickler_ is now known as frickler09:44
*** yingjun has quit IRC09:45
snikitinalex_xu, Oh! now I understand. thank you I totally forgot about docs09:51
alex_xusnikitin: np :)09:52
*** bkopilov has joined #openstack-nova09:52
*** zenoway has joined #openstack-nova09:53
*** Dinesh_Bhor has joined #openstack-nova09:54
*** kashyap has joined #openstack-nova09:54
*** amotoki has joined #openstack-nova09:56
*** kevinz has quit IRC09:57
*** ducttape_ has joined #openstack-nova09:58
johnthetubaguyoomichi: the worry was when admins get users complaining that their query has stop returning the results they are used to, there is at least a log you can look at to find out what happened09:59
*** tyrola has quit IRC10:00
*** tyrola has joined #openstack-nova10:01
*** pksingh has joined #openstack-nova10:01
johnthetubaguyoomichi: I am not sure if we need a configuration option to turn that off myself, we do add too many config options10:02
*** lpetrut has joined #openstack-nova10:03
*** ducttape_ has quit IRC10:03
*** amotoki has quit IRC10:06
*** gongysh has quit IRC10:07
*** bkopilov_ has joined #openstack-nova10:08
*** amotoki has joined #openstack-nova10:09
*** hfu has quit IRC10:15
*** tuanluong has quit IRC10:16
openstackgerritchenyingnan proposed openstack/nova: Port nova.cloudpipe.pipelib and nova.tests.unit.virt.xenapi.test_agent to Python 3  https://review.openstack.org/41167610:17
*** sahid has joined #openstack-nova10:18
*** trananhkma has quit IRC10:20
openstackgerritStephen Finucane proposed openstack/nova: conf: Remove 'virt' file  https://review.openstack.org/39256610:23
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate two more nova-net options  https://review.openstack.org/41239010:23
*** amotoki has quit IRC10:23
*** Jeffrey4l has joined #openstack-nova10:24
*** greeny___ has joined #openstack-nova10:29
*** phuongnh has quit IRC10:38
*** hoangcx has quit IRC10:38
*** amotoki has joined #openstack-nova10:38
*** thorst has joined #openstack-nova10:40
*** derekh has joined #openstack-nova10:43
*** bks has quit IRC10:43
*** amotoki has quit IRC10:45
*** thorst has quit IRC10:48
*** greeny___ has quit IRC10:53
*** tuanluong_ has quit IRC10:55
*** ducttape_ has joined #openstack-nova11:03
*** amoralej is now known as amoralej|brb11:05
*** asettle has quit IRC11:05
*** tbachman has joined #openstack-nova11:06
*** sreeram_v has joined #openstack-nova11:06
*** asettle has joined #openstack-nova11:09
*** ociuhandu has joined #openstack-nova11:10
*** sudipto has quit IRC11:10
*** sreeram_v has quit IRC11:11
*** cdent has joined #openstack-nova11:11
*** ihrachys has quit IRC11:12
*** asettle has quit IRC11:17
*** yufei has quit IRC11:17
*** ducttape_ has quit IRC11:21
*** rfolco has joined #openstack-nova11:25
*** dave-mccowan has joined #openstack-nova11:26
*** pksingh has quit IRC11:28
*** esmiurium has joined #openstack-nova11:30
*** mnasiadka has joined #openstack-nova11:33
*** ducttape_ has joined #openstack-nova11:33
*** sdague has joined #openstack-nova11:33
*** Dave has quit IRC11:33
*** portdirect is now known as portdirect_away11:34
*** pksingh has joined #openstack-nova11:36
*** pksingh has quit IRC11:37
*** nicolasbock has joined #openstack-nova11:38
openstackgerritSergey Nikitin proposed openstack/nova: Docs: Added 'tags' to server show, update and detail responses  https://review.openstack.org/41241111:41
*** smatzek has joined #openstack-nova11:43
*** ihrachys has joined #openstack-nova11:43
*** baoli has joined #openstack-nova11:44
*** tbachman has quit IRC11:44
*** Dave has joined #openstack-nova11:45
*** dave-mcc_ has joined #openstack-nova11:45
*** thorst has joined #openstack-nova11:45
*** tbachman has joined #openstack-nova11:45
*** tbachman has quit IRC11:45
*** baoli has quit IRC11:48
*** dave-mccowan has quit IRC11:48
*** ducttape_ has quit IRC11:49
*** ratailor has quit IRC11:49
*** Guest24205 has quit IRC11:52
*** thorst has quit IRC11:52
*** lucasagomes is now known as lucas-hungry11:55
*** diga has quit IRC11:55
*** ociuhandu has quit IRC11:57
openstackgerritStephen Finucane proposed openstack/nova: rbd: Remove unnecessary 'encode' calls  https://review.openstack.org/41235611:58
*** hfu has joined #openstack-nova11:59
sfinucangcb: I think I have all of the goal-python35 changes reviewed now. Just three that need another look at12:00
*** gszasz has joined #openstack-nova12:07
*** kevinz has joined #openstack-nova12:09
*** Jeffrey4l has quit IRC12:10
gcbsfinucan, thanks.  I still have at least patches to submit12:10
*** zhangjl has left #openstack-nova12:11
gcbsfinucan,  will address your comments later12:11
*** thorst has joined #openstack-nova12:15
*** amoralej|brb is now known as amoralej12:15
*** catintheroof has joined #openstack-nova12:18
*** ducnc has quit IRC12:18
*** dave-mcc_ has quit IRC12:18
*** ducnc has joined #openstack-nova12:18
openstackgerritVasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt  https://review.openstack.org/36441312:21
*** prateek has quit IRC12:24
*** rtheis has joined #openstack-nova12:25
*** yingjun has joined #openstack-nova12:27
*** portdirect_away is now known as portdirect12:29
*** hfu has quit IRC12:31
*** hfu has joined #openstack-nova12:33
*** vladikr has joined #openstack-nova12:37
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: [WIP] Add version field to LibvirtLiveMigrateData  https://review.openstack.org/40460712:38
*** hfu has quit IRC12:40
*** bkopilov has quit IRC12:42
*** bkopilov_ has quit IRC12:42
*** portdirect is now known as portdirect_away12:43
*** Jeffrey4l has joined #openstack-nova12:43
*** baoli has joined #openstack-nova12:44
*** flaper87 has joined #openstack-nova12:45
*** siva_krish has quit IRC12:45
wznoinsklennyb, reviewed12:47
*** vladikr has quit IRC12:47
lennybwznoinsk, 10x12:47
*** ducttape_ has joined #openstack-nova12:49
*** baoli has quit IRC12:49
*** preethi has quit IRC12:53
*** ducttape_ has quit IRC12:54
*** siva_krish has joined #openstack-nova12:56
*** ociuhandu has joined #openstack-nova12:58
*** gibi has quit IRC12:59
*** moshele has quit IRC13:01
*** lucas-hungry is now known as lucasagomes13:01
*** moshele has joined #openstack-nova13:01
*** tbachman has joined #openstack-nova13:03
*** yassine has joined #openstack-nova13:05
sahiddansmith: can you have a look at this serie https://review.openstack.org/#/c/408240/ - so i can rebase the work for emulator threads on top of it13:05
*** yassine is now known as Guest9516713:06
*** tbachman has quit IRC13:08
*** hfu has joined #openstack-nova13:08
*** portdirect_away is now known as portdirect13:09
*** dbutts29 has joined #openstack-nova13:11
*** lamt has quit IRC13:12
*** gcb has quit IRC13:14
*** rfolco has quit IRC13:16
*** rfolco has joined #openstack-nova13:17
*** tbachman has joined #openstack-nova13:21
*** asettle has joined #openstack-nova13:22
*** rfolco has quit IRC13:23
*** jwcroppe_ has quit IRC13:26
*** gouthamr has joined #openstack-nova13:26
*** jaypipes has joined #openstack-nova13:27
*** pradk has joined #openstack-nova13:31
*** prateek has joined #openstack-nova13:31
*** weshay_lata is now known as weshay13:31
sdaguesahid: I think dansmith is out for the rest of the year13:32
*** rfolco has joined #openstack-nova13:33
*** rfolco has quit IRC13:33
*** rfolco has joined #openstack-nova13:33
*** gcb has joined #openstack-nova13:36
*** noama has joined #openstack-nova13:37
*** portdirect has quit IRC13:38
*** jheroux has joined #openstack-nova13:39
*** noama has left #openstack-nova13:39
alex_xujohnthetubaguy: sdague, oomichi , hi, looks like we have different opinion about the config for invalid query message log https://review.openstack.org/#/c/389003/17, it will be great to have quick discussion if you are online13:39
sdaguealex_xu: sure13:41
johnthetubaguyalex_xu: I am around13:41
alex_xuas i know oomichi back to Japan, probably it is late for him13:42
sdagueok, so the idea of logging the invalid content was a last minute add that I had13:42
sdagueit feels like *if* we do that, we should make it easy to turn off, because it could be really noisey13:42
sdaguejohnthetubaguy: I think you were objecting to the conf value, but that's why I had suggested it13:42
alex_xuI +1 for noisey13:42
sdagueI also agree with johnthetubaguy that WARN is way too high a value13:43
sdagueI guess I also wonder if there is enough information in the log message to make it worth while at all, because the idea was that the operators that care would be able to tell their users their app was wrong13:43
sdaguebut that's going to require having a pretty detailed request log, including client IP / user-agent logged with this13:44
sdagueto the point that maybe we take this logging out of this patch all together, just keep with the old validation, and think about communicating things are wrong in January?13:45
johnthetubaguythats true, I had assumed everyone had that info for every request-id already13:45
johnthetubaguysdague: thats quite a good idea, we could punt on that for now13:45
*** baoli has joined #openstack-nova13:45
johnthetubaguylets not block the good stuff on getting the logging right13:45
*** erhudy has joined #openstack-nova13:45
alex_xucool :)13:46
alex_xusdague: do you mean remove the log and config for now?13:46
alex_xuI think so, I didn't read wrong13:46
sdaguealex_xu: yeh13:46
alex_xuoh yeah \o/13:46
sdaguealex_xu: make it a dedicated patch after the existing ones13:46
*** clenimar has joined #openstack-nova13:46
alex_xugot it13:46
sdaguewe can get the rest of this in, and talk about that piece later13:46
alex_xuok13:47
alex_xujohnthetubaguy: sdague thanks13:47
sdaguealex_xu: thanks for driving this13:47
alex_xusdague: np13:47
*** mdnadeem has quit IRC13:49
openstackgerritPawel Koniszewski proposed openstack/nova: WIP: testing LM grenade job  https://review.openstack.org/41166213:49
*** pbandark has quit IRC13:50
sdaguecdent: when you have a chance, if you want to see that I got the placement API status checking right - https://review.openstack.org/#/c/411883/4/nova/cmd/status.py13:50
*** baoli has quit IRC13:50
*** ducttape_ has joined #openstack-nova13:50
cdentsdague: certainly, will look at that asap13:51
openstackgerritPawel Koniszewski proposed openstack/nova: WIP: testing LM grenade job  https://review.openstack.org/41166213:51
*** jwcroppe has joined #openstack-nova13:51
*** diga has joined #openstack-nova13:55
*** ducttape_ has quit IRC13:55
jaypipesedleafe-: we doing scheduler meeting today? can't remember if we cancelled it for the year...13:58
cdentjaypipes, edleafe- I remember "if we feel like it and people are around"13:59
jaypipesmriedem: thx for fixing up that one docstring comment on my patch over the weekend.14:03
*** hfu has quit IRC14:06
*** sudipto has joined #openstack-nova14:08
*** ayogi has quit IRC14:09
*** amoralej is now known as amoralej|lunch14:10
sahidsdague: hum that is unfortunate, he asked me to make some clean before moving forward with my spec emulator threads https://review.openstack.org/#/c/400874/5/nova/objects/instance_numa_topology.py14:10
*** xyang1 has joined #openstack-nova14:11
*** lamt has joined #openstack-nova14:12
*** yogesh_ has joined #openstack-nova14:12
*** yogesh_ has quit IRC14:12
*** cleong has joined #openstack-nova14:14
*** bkopilov has joined #openstack-nova14:15
*** cdent has quit IRC14:16
*** bkopilov_ has joined #openstack-nova14:16
*** preethipy has joined #openstack-nova14:16
*** moshele has quit IRC14:17
sfinucansahid: I'll take a gawk now, but I'm not dansmith :)14:18
*** baoli has joined #openstack-nova14:19
*** sdague_ has joined #openstack-nova14:21
*** sdague has quit IRC14:24
*** pradk has quit IRC14:25
*** baoli has quit IRC14:25
*** sdague_ is now known as sdague14:27
*** preethipy has quit IRC14:27
*** boggle has joined #openstack-nova14:28
bauzassdague: hola, just a silly question14:28
bauzassdague: why are you adding https://review.openstack.org/#/c/409871/2/features.yaml@243 ?14:28
*** preethipy has joined #openstack-nova14:28
openstackgerritAlex Xu proposed openstack/nova: Use JSON-Schema to validate query parameters for keypairs API  https://review.openstack.org/39875514:29
openstackgerritAlex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters  https://review.openstack.org/38900314:29
openstackgerritAlex Xu proposed openstack/nova: Refactor the code to add generic schema validation helper  https://review.openstack.org/41247314:29
sdaguebauzas: how do you tell a subnode that it should be configured with placement credentials?14:29
bauzassdague: I can see it's related to a "subnode" but I need to understand a bit more of the d-g logic14:29
sdaguesee - https://review.openstack.org/#/c/411510/14:29
*** baoli has joined #openstack-nova14:29
bauzassdague: ah, thanks, that was the puzzle piece I was missing14:30
sdagueit was in the Depends-On :)14:31
bauzasI missed that14:31
*** ducttape_ has joined #openstack-nova14:32
*** psachin has quit IRC14:37
*** esberglu has joined #openstack-nova14:38
*** mnasiadka has quit IRC14:38
*** smatzek has quit IRC14:39
mriedemandreykurilin: so does trove break with the latest novaclient code?14:41
openstackgerritPawel Koniszewski proposed openstack/nova: DO NOT MERGE THE STEGOSAURUS  ___________________________________ < I'M DEBUGGING THE LM+GRENADE GATE >  ----------------------------------- \                             .       .  \                           / `.   .' "   \                  .---.  <      https://review.openstack.org/41166214:42
*** ducttape_ has quit IRC14:42
*** jamesdenton has joined #openstack-nova14:42
andreykurilinmriedem: hi! based on the code which I saw there, trove doesn't work with https://github.com/openstack/python-novaclient/releases/tag/3.0.0 which was cut during mitaka cycle(it is even not the latest version of mitaka)14:43
*** yingjun has quit IRC14:44
mriedemandreykurilin: well, they might just not use the v1_1 module and don't actually get the warning from pylint anymore14:44
*** liverpooler has joined #openstack-nova14:44
mriedemif they are using upper-constraints in their ci jobs they are testing against novaclient 6.0.014:44
andreykurilinmriedem: I think it is our fault, since we did not raise novaclient version for a long time https://github.com/openstack/requirements/blob/master/global-requirements.txt#L22514:44
mriedemandreykurilin: that's not what we test against thogh14:45
mriedem*though14:45
andreykurilinmriedem: they use unexist module - https://github.com/openstack/trove/blob/48dcbb6dcd1b120e668d8330104a6c63a3c01a96/trove/common/single_tenant_remote.py#L2214:45
mriedemhmm, they are testing against novaclient 6.0.0 though http://logs.openstack.org/99/412299/1/check/gate-tempest-dsvm-trove-ubuntu-xenial/c75e3cc/logs/pip2-freeze.txt.gz14:46
mriedemso i don't see how any of that would be passing14:46
*** prateek has quit IRC14:46
andreykurilinmriedem: well, they have several modules which initialize novaclient. one of them is fully broken.  another one looks ok to me - https://github.com/openstack/trove/blob/48dcbb6dcd1b120e668d8330104a6c63a3c01a96/trove/common/remote.py#L25 . It should not be broken(initialization looks good to me)14:47
mriedemandreykurilin: i have a feeling that code isn't used http://git.openstack.org/cgit/openstack/trove/tree/trove/common/cfg.py#n32514:48
mriedemyeah that's the one that's used in the config14:48
mriedemi agree that https://github.com/openstack/trove/blob/48dcbb6dcd1b120e668d8330104a6c63a3c01a96/trove/common/single_tenant_remote.py#L22 is broken and it's not tested14:48
mriedemit's probably faked out14:48
andreykurilinmriedem: and they do not transmit session to novaclient in remote.py, so my patches should help trove to support keystone v3:)14:48
*** burt has joined #openstack-nova14:49
mriedemi'll report a bug on the other busted code14:49
*** liverpooler has quit IRC14:49
*** boggle has quit IRC14:49
andreykurilinmriedem: hm... https://github.com/openstack/trove/blob/48dcbb6dcd1b120e668d8330104a6c63a3c01a96/trove/common/remote.py#L104 this line is wrong and doesn't work any more. it looks like it is originaly was a bad behaviour and it is better to fix it at trove side14:50
*** kevinz has quit IRC14:50
*** liverpooler has joined #openstack-nova14:51
*** shaner_ has quit IRC14:51
*** prateek has joined #openstack-nova14:52
*** shaner has joined #openstack-nova14:52
mriedemnice14:52
mriedemandreykurilin: anyway https://bugs.launchpad.net/trove/+bug/165115414:52
openstackLaunchpad bug 1651154 in OpenStack DBaaS (Trove) "ImportError for novaclient v1_1 in trove/common/single_tenant_remote.py" [Undecided,New]14:52
andreykurilinmriedem: despite the fact that it is too late, I think we should update min version of novaclient for mitaka, newton and for master too14:52
*** eharney has joined #openstack-nova14:53
mriedemwe aren't going to bump the minimum required versions of novaclient for mitaka or newton at this point14:53
mriedembumping minimum required versions of dependencies in stable is a big no-no14:53
andreykurilinok14:55
*** cfriesen has joined #openstack-nova14:55
pkoniszewskimriedem: hi, i did some more debugging of the LM+grenade gate, currently i'm stuck on this https://review.openstack.org/#/c/393441/ grenade change, I wonder whether this could break RPC somehow14:55
mriedemsdague: on https://review.openstack.org/#/c/411883/ i was wondering what your thoughts were on testing the placement API parts of that? mock out the calls or use an actual API fixture like we have elsewhere in functional tests?14:55
andreykurilinsdague: but at least for master, I hope we will raise min version :)14:55
*** cdent has joined #openstack-nova14:55
*** tbachman_ has joined #openstack-nova14:55
*** jamielennox is now known as jamielennox|away14:56
*** tbachman has quit IRC14:56
*** tbachman_ is now known as tbachman14:56
mriedemandreykurilin: but to what min version?14:56
*** amoralej|lunch is now known as amoralej14:56
pkoniszewskimriedem: I can see that live migration pre checks work, but the message to start pre_live_migration is lost14:56
mriedemusually you don't raise the minimum until something actually requires a higher minimum for like a feature or a fix14:56
mriedemcdent: thanks for the review on https://review.openstack.org/#/c/411883/ - you don't have to +1 though, it's a wip :)14:57
sdaguemriedem: honestly, I would do it with mocks. I can write that test class if you like14:57
*** mdnadeem has joined #openstack-nova14:57
sdaguemostly I wrote the code by walking through devstack setup one step at a time14:57
sdagueand seeing the failures14:58
mriedemsdague: yeah i pulled it down and played with some of that too14:58
mriedemstarting w/o placement, and then with placement but no auth config14:58
andreykurilinmriedem: ideally, to uncoming release:) Imo, all releases between min and max should be compatible. Now we have several big releases which are not compatible at all. It is impossible to implement several use cases by one way which will work with all these versions14:58
cdentmriedem: I like reviewing wips: sooner sanity gets checkec the sooner the insane can stop.14:58
sdaguemriedem: so, I'll to the test file you have here - https://review.openstack.org/#/c/411525/6 and explain how to reconstruct that in the "real" world14:58
mriedemsdague: doesn't matter to me which of us writes the test class, i was going to work on the api/cell db query comparisons today14:58
*** sudipto has quit IRC14:59
mriedemsdague: sure, throw it in it's own test class14:59
andreykurilinmriedem: In future, I think min version should be equal to version that supports all features from specific openstack release.14:59
mriedemso when we drop these it's a clean drop14:59
*** sudipto has joined #openstack-nova14:59
openstackgerritMaciej Szankin proposed openstack/nova: conf: Remove 'virt' file  https://review.openstack.org/39256614:59
sdaguemriedem: will do14:59
*** sudipto has quit IRC14:59
cdentmriedem: and besides I gotta keep up my "+1s later minused by a core" stats14:59
*** sudipto has joined #openstack-nova14:59
*** sudipto_ has joined #openstack-nova15:00
openstackgerritMaciej Szankin proposed openstack/nova: conf: Deprecate two more nova-net options  https://review.openstack.org/41239015:00
*** sudipto_ has quit IRC15:00
*** sudipto has quit IRC15:00
*** sudipto has joined #openstack-nova15:00
*** eharney has quit IRC15:00
*** sudipto has quit IRC15:00
*** sudipto has joined #openstack-nova15:01
*** sudipto_ has joined #openstack-nova15:01
openstackgerritint32bit proposed openstack/nova: Add unit test for extract_snapshot with compression enabled  https://review.openstack.org/41248515:02
*** smatzek has joined #openstack-nova15:02
mriedempkholkin: you were asking for things to work on, this hasn't started but was re-approved from newton https://blueprints.launchpad.net/nova/+spec/validate-project-with-keystone15:02
mriedempkholkin: or this https://blueprints.launchpad.net/nova/+spec/virt-device-tagged-attach-detach - might want to ask artom if he's going to start that?15:03
mriedempkholkin: or this https://blueprints.launchpad.net/nova/+spec/vendordata-reboot-ocata15:03
mriedempkholkin: order of importance in my mind on those would be vendordata, keystone project validation, then attach/detach tagged devices15:04
*** pradk has joined #openstack-nova15:04
mriedemsdague: ooo you didn't even question my enum34 usage :)15:05
*** jamesdenton has quit IRC15:07
pkholkinmriedem: oh thanks Matt! I will look these specs15:07
pkholkinmaybe will take smth for me )15:08
*** zz_dimtruck is now known as dimtruck15:10
*** eharney has joined #openstack-nova15:14
openstackgerritint32bit proposed openstack/nova: Add unit test for extract_snapshot with compression enabled  https://review.openstack.org/41248515:14
*** liverpooler has quit IRC15:15
*** liverpooler has joined #openstack-nova15:15
sdaguemriedem: meh, seems like the cool kids all want to do it15:16
*** ducttape_ has joined #openstack-nova15:17
*** tbachman has quit IRC15:17
*** sshen has quit IRC15:19
mriedemsdague: i do prefer proper enums over hacky int constants :)15:19
*** preethipy has quit IRC15:19
mriedemthat's why i went with it15:19
sdagueI'm most amused that apparently we have python 2.6 markers for it15:19
mriedemeven old school c guys could appreciate that15:20
mriedemyeah i saw that15:20
mriedemwould have removed but you know, reqs15:20
sdagueok, I'm all through with my mandatory training in the cyber, which I hear is going to be very big15:20
sdagueso I'll do those unit tests15:21
bauzasoh that's fun, someone pointed out internally how the Hilton is far cheaper than the Sheraton for the Atlanta PTG15:25
bauzaswe're talking of $120 vs. $18515:26
*** dirk has left #openstack-nova15:27
mriedemsdague: mandatory training in the cyber?15:29
*** mlavalle has joined #openstack-nova15:29
mriedemoh15:29
mriedemsecurity15:29
mriedemthe fate of our company rests on your shoulders sdague15:29
mriedemdon't click those links15:29
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate two more nova-net options  https://review.openstack.org/41239015:32
andreykurilinmriedem: https://review.openstack.org/412497 should fix trove to work with old and new novaclient releases15:33
*** marst has joined #openstack-nova15:34
mriedemlooking for another core to go through the first 3 changes in this series https://review.openstack.org/#/c/398470/ i'm +2 on the bottom 315:37
mriedemwould be good to move those along as it's a large refactor so prone to rebase issues15:38
mriedemandreykurilin: heh, have you ever noticed that this is a method and not a class? https://github.com/openstack/python-novaclient/blob/master/novaclient/client.py#L27515:40
mriedemit's a factory so i guess that makes sense, just the Client bit has always thrown me15:41
*** weshay is now known as weshay_afk15:41
*** aarefiev is now known as aarefiev_afk15:41
openstackgerritSarafraj Singh proposed openstack/nova: Objectify requests_and_created_ports  https://review.openstack.org/40539815:41
andreykurilinmriedm: sure, I know that. My  first work in OpenStack was about unification all clients, so I spent a lot of time discovering them :) Btw, several clients named such method just like "get_client", but I prefer "Client" for entrypoint15:42
*** int32bit has joined #openstack-nova15:44
*** baoli has quit IRC15:44
*** prateek has quit IRC15:44
*** baoli has joined #openstack-nova15:45
*** weshay_afk is now known as weshay15:45
*** mtanino has joined #openstack-nova15:50
*** lbeliveau has quit IRC15:50
*** lbeliveau has joined #openstack-nova15:51
*** Sukhdev has joined #openstack-nova15:51
dansmithsahid: I am not here15:52
jrollwhat a bum15:52
dansmithsahid: I also meant that cleanup patch could come after your real set.. bonus points for rebasing on the cleanup, but not critical15:52
andreykurilinmriedem: http://docs.openstack.org/developer/python-novaclient/ref/v2/index.html omg...we still have v1.1 reference in docs...15:53
*** Sukhdev has quit IRC15:53
*** yamahata has joined #openstack-nova15:53
mriedemandreykurilin: heh yeah that's an easy cleanup15:54
mriedemit's in doc/source/conf.py i think15:54
*** yamahata has quit IRC15:56
bauzassdague: do you feel we would still need to update devstack for running the placement service by default ?15:56
bauzassdague: or is the d-g change only needed ?15:56
bauzasI tend to think about the former15:57
*** yamahata has joined #openstack-nova15:57
sahiddansmith: ah cool happy to hear that so at least i can continue to expect to get reviews on the features then i will rebase the work done to clean the numa objects15:57
sdaguebauzas: we do need to *also* update devstack15:57
sdaguebut the d-g change should go first15:58
openstackgerritSean Dague proposed openstack/nova: nova-status: add basic placement status checking  https://review.openstack.org/41188315:58
sdaguemriedem: ^^^15:58
sdagueI think that's about right15:58
mriedemsdague: cool15:59
*** dimtruck is now known as zz_dimtruck15:59
mriedembauzas: there is a default ENABLED_SERVICES in stackrc i think which could be updated independenty of sean's changes15:59
*** mvk has quit IRC16:00
bauzasmriedem: sdague: okay, I'll write that change then, I just wanted to doublecheck16:00
bauzasmriedem: yeah, I was thinking about that one16:00
*** _ducttape_ has joined #openstack-nova16:00
sdagueok, d-g change is approved now16:02
* mriedem wishes gerrit had an 'expand all common lines' button16:02
*** Guest33254 has quit IRC16:02
*** roeyc has quit IRC16:03
*** ratailor has joined #openstack-nova16:03
mriedemooo there is one16:03
*** Sukhdev_ has joined #openstack-nova16:03
*** ducttape_ has quit IRC16:04
*** sdague has quit IRC16:06
*** diga has quit IRC16:08
*** keekz has quit IRC16:09
*** schegi has quit IRC16:10
*** schegi has joined #openstack-nova16:10
*** hongbin has joined #openstack-nova16:10
*** zz_dimtruck is now known as dimtruck16:10
*** mjura has quit IRC16:11
*** keekz has joined #openstack-nova16:11
*** pcaruana has quit IRC16:15
*** sreeram_v has joined #openstack-nova16:17
*** sridharg has quit IRC16:20
mriedemjaypipes: how does RT.disabled ever return True?16:21
mriedemgiven in update_available_resource we either have a compute node or we create one16:21
*** sreeram_v has quit IRC16:22
*** zenoway has quit IRC16:24
*** browne has joined #openstack-nova16:24
*** cdent has quit IRC16:26
jaypipesmriedem: it returns True if things like instance_claim() are called before update_available_resource()./16:26
*** tjones has joined #openstack-nova16:26
mriedemthat seems...wrong16:27
mriedemlike, see https://review.openstack.org/#/c/398473/15/nova/compute/resource_tracker.py@12116:27
mriedem"# compute_driver doesn't support resource tracking"16:27
*** _ducttape_ has quit IRC16:27
*** ducttape_ has joined #openstack-nova16:28
jaypipesmriedem: the disabled() thing really hasn't meant "doesn't support resource tracking" ever :) It was a leftover remnant of when we conflated the Service object (and servicegroup API) with the ComputeNode object.16:29
jaypipesmriedem: at this point, it's more of a vestigial tail, but what I'm changing in this patch doesn't change the existing behaviour. That code comment is just wrong...16:29
mriedemyeah i figured it meant, is the service record related to this compute node enabled or disabled16:29
mriedemand shouldn't have claims made against it16:29
jaypipesit used to be, yes.16:29
jaypipesit used to mean that, yes'16:30
mriedemok...making notes inline16:31
mriedemif we actually have a case where instance_claim can be called before update_available_resource, i'd think we should call update_available_resource from instance_claim rather than do a noop claim on the host16:32
mriedembut i thought update_available_resource was called when the compute service started up16:32
mriedemyeah pre_start_hook in compute manager16:32
jaypipesmriedem: it is, yes.16:32
mriedemso this disabled thing just all seems like malarky16:33
mriedemand we'll just torch that at some point?16:33
*** moshele has joined #openstack-nova16:33
jaypipesmriedem: yes sir.16:34
*** smatzek has quit IRC16:34
mriedemok i just wanted to be able to say malarkey16:34
*** weshay is now known as weshay_bbiab16:34
jaypipesmriedem: the check for disablement really should be in the nova.compute.manager.Manager class.16:34
jaypipesmriedem: which is the thing that == the Service object == the servicegroup API.16:35
mriedemand then we wouldn't erroneously recreate deleted compute nodes after you disabled the service via the REST API16:35
mriedemand end up with ComputeNodeNotFound errors later16:35
mriedemwhen listing services16:35
*** Oku_OS is now known as Oku_OS-away16:35
jaypipesmriedem: right.16:36
mriedemew, test_compute mucking with RT internals16:37
mriedemthat's fun16:37
*** smatzek has joined #openstack-nova16:38
jaypipesmriedem: yuuuup. :(16:39
*** Sukhdev_ has quit IRC16:39
*** dbutts29 has quit IRC16:40
andreafmriedem, sdague: it looks like https://bugs.launchpad.net/tempest/+bug/1646779 and https://bugs.launchpad.net/nova/+bug/1643911 may be different manifestations of the same issue - they both happen around the same tests in Tempest, and they both relate to a libvirt crash, even though it manifests in  three different way16:42
openstackLaunchpad bug 1646779 in OpenStack Compute (nova) "libvirt killed by kernel on general protection or stack segment traps" [Undecided,Incomplete]16:42
openstackLaunchpad bug 1643911 in OpenStack Compute (nova) "libvirt randomly crashes on xenial nodes with "*** Error in `/usr/sbin/libvirtd': malloc(): memory corruption:"" [Medium,Confirmed]16:42
andreafmemory corruption, general protection or stack segment traps16:43
andreafmriedem, sdague: I tried to reproduce the issue on my VM but unsuccessfully so far16:44
andreafwould it be ok to merge the two issues? I have an e-r query merged now for one the two, I could extend the query16:44
mriedemjaypipes: 2 tiny things in https://review.openstack.org/#/c/398473/ - do you want me to just address them quick so you don't get lost in the comments?16:44
lbeliveaujaypipes: hi, would you have time to look at my rt bug ?  https://review.openstack.org/#/c/405575/16:45
andreafactually there is a e-r query for both issues already16:46
*** jaosorior has quit IRC16:48
jaypipesmriedem: nope, I'm actually addressiong the WIP patch at the HEAD of that series, so I'll do that in a rebase to master shortly.16:48
mriedemok16:48
jaypipeslbeliveau: yup, soon as I finish up what I'm on right this second.16:48
lbeliveaujaypipes: no rush :)16:48
*** jaosorior has joined #openstack-nova16:48
jaypipesmriedem: I'm adding the "auto-create these Ironic node resource classes in the placement API" stuff.16:49
jaypipesmriedem: as requested by yourself and jroll in the ML thread on the topic.16:50
*** Guest95167 has quit IRC16:50
*** yassine has joined #openstack-nova16:50
*** tbachman has joined #openstack-nova16:51
mriedemok, i'm in nova/cinder meeting fun times for the next hour, then will come back16:51
*** yassine is now known as Guest4586716:51
openstackgerritAkshil Verma proposed openstack/python-novaclient: Fixed the __ne__ implementation in base.Resource  https://review.openstack.org/40954916:51
*** cdent has joined #openstack-nova16:56
cdentjaypipes: I'm not sure what the right thing is on that latest not latest question I raised. Mentioned it because I simply dunno.16:57
jaypipescdent: I'd actually prefer to leave it at latest... it's for non-scheduler-report-client clients that I think is important to specify.16:58
cdentthat's cool with me16:59
*** ratailor has quit IRC17:01
*** nic has joined #openstack-nova17:05
*** _ducttape_ has joined #openstack-nova17:08
*** Guest45867 has quit IRC17:08
bauzasmriedem: jaypipes: FYI, https://review.openstack.org/41253717:09
bauzassean disappeared but I'd love getting his feedback on that devstack patch from a noobie17:09
* bauzas bails out but is back in the evening17:10
jaypipesbauzas: asked a ? on the reivew...17:11
*** ducttape_ has quit IRC17:11
bauzasjaypipes: replied17:12
mriedembauzas: let's add some context to the commit message for 51a225c5d7f7d8655be62cab0e4d677b6839f35a and the d-g change17:13
bauzasthat's for multinode devstacks17:13
bauzasmriedem: sure, np17:13
bauzasmriedem: on it17:13
*** cleong has quit IRC17:15
*** cleong has joined #openstack-nova17:15
*** siva_krish has quit IRC17:16
*** moshele has quit IRC17:17
*** Zer0Byte__ has joined #openstack-nova17:17
snetijohnthetubaguy, Can you please check this when you have time https://review.openstack.org/#/c/392305/. rebased this patch you previously +2d.17:17
jaypipesbauzas: done. :)17:23
*** corey_ has joined #openstack-nova17:23
*** cleong has quit IRC17:23
*** corey_ is now known as Guest162217:24
*** sudipto has quit IRC17:25
*** sudipto_ has quit IRC17:25
*** sudipto_ has joined #openstack-nova17:25
*** sudipto has joined #openstack-nova17:25
robcresswello/ Hey all, anyone particularly familiar with the api_versions.py in the pythonclient? Attempting to use the discover_version function with the client that Horizon has, but it alwars gets a 'versions' KeyError. I'm not entirely sure why.17:25
robcresswellalways*17:26
mriedemrobcresswell: andreykurilin is your guy17:26
robcresswellmriedem: Thanks, any particular tz?17:27
andreykurilinrobcresswell:  hi! can you share any trace?17:27
mriedemrobcresswell: andrey is on at all TZs17:28
robcresswellandreykurilin: http://paste.openstack.org/show/592814/17:28
andreykurilinmriedem: heh17:28
robcresswellThe construction of the client on Horizons side is https://github.com/openstack/horizon/blob/master/openstack_dashboard/api/nova.py#L46417:29
andreykurilinmriedem: https://github.com/openstack/horizon/blob/master/openstack_dashboard/api/nova.py#L474 they use the same way of initialization novaclient17:30
andreykurilinbut it is unrelated17:30
andreykurilinrobcresswell: does cli `nova version-list` works?17:31
mriedemtranscend was the word i was thinking of17:31
robcresswellandreykurilin: It does17:32
*** dtp has joined #openstack-nova17:34
andreykurilinrobcresswell: sorry, I need to switch to some urgent task. will return in 20 minutes17:35
robcresswellandreykurilin: Sure, no rush17:35
robcresswellI appreciate the help!17:35
*** mdnadeem has quit IRC17:37
*** andreas_s has quit IRC17:39
*** _ducttape_ has quit IRC17:40
*** ducttape_ has joined #openstack-nova17:41
*** siva_krish has joined #openstack-nova17:42
*** nicolasbock has quit IRC17:43
*** derekh has quit IRC17:44
openstackgerritint32bit proposed openstack/nova: Page the db query when filter instances with IP  https://review.openstack.org/40517117:45
*** moshele has joined #openstack-nova17:45
*** raunak has joined #openstack-nova17:46
*** nicolasbock has joined #openstack-nova17:48
*** lpetrut has quit IRC17:48
*** sdague has joined #openstack-nova17:48
*** ociuhandu has quit IRC17:49
*** lucasagomes is now known as lucas-afk17:50
openstackgerritJay Pipes proposed openstack/nova: rt: pass the nodename to public methods  https://review.openstack.org/39847017:50
openstackgerritJay Pipes proposed openstack/nova: rt: remove fluff from test_resource_tracker  https://review.openstack.org/39847117:50
openstackgerritJay Pipes proposed openstack/nova: placement: RT now adds proper Ironic inventory  https://review.openstack.org/40447217:50
openstackgerritJay Pipes proposed openstack/nova: rt: explicitly pass compute node to _update()  https://review.openstack.org/39847217:50
openstackgerritJay Pipes proposed openstack/nova: rt: use a single ResourceTracker object instance  https://review.openstack.org/39847317:50
*** sahid has quit IRC17:50
*** weshay_bbiab is now known as weshay17:51
*** sudipto_ has quit IRC17:51
*** sudipto has quit IRC17:51
*** Swami has joined #openstack-nova17:52
*** liverpooler has quit IRC17:53
*** liverpooler has joined #openstack-nova17:55
openstackgerritint32bit proposed openstack/nova: Page the db query when filter instances with IP  https://review.openstack.org/40517117:55
*** slaweq_ has quit IRC17:56
*** slaweq_ has joined #openstack-nova17:58
*** unicell1 has quit IRC18:00
*** gszasz has quit IRC18:00
*** yamahata has quit IRC18:01
*** int32bit has quit IRC18:01
*** Jeffrey4l has quit IRC18:07
*** slaweq_ has quit IRC18:08
openstackgerritAkshil Verma proposed openstack/python-novaclient: Fixed the __ne__ implementation in base.Resource  https://review.openstack.org/40954918:09
andreykurilin robcresswell: do you have local env with horizon and novaclient?18:18
*** sreeram_v has joined #openstack-nova18:18
robcresswellandreykurilin: Yep18:18
andreykurilinrobcresswell: nice.18:18
robcresswellIs that causing the issue?18:19
openstackgerritSean Dague proposed openstack/nova: nova-status: add basic placement status checking  https://review.openstack.org/41188318:19
sdaguemriedem: ok, I think I fixed all your concerns18:19
andreykurilinrobcresswell: no)18:20
*** ociuhandu has joined #openstack-nova18:20
andreykurilinrobcresswell: which novaclient's version do you use?18:20
robcresswellandreykurilin: 6.0.018:21
andreykurilinrobcresswell: Can you modify horizon code to transmit `auth_token=token_id, bypass_url=nova_url` to https://github.com/openstack/horizon/blob/master/openstack_dashboard/api/nova.py#L466 and remove L474 and L475? and recheck everything(nothing should change, but at least old features should continue to work, i.e listing servers)?18:21
*** sreeram_v has quit IRC18:22
andreykurilinrobcresswell: next step will be updating novaclient to master. it can help(after changes posted above)18:22
*** siva_krishnan_ has joined #openstack-nova18:23
robcresswellandreykurilin: Should L469 still be token_id? Or is that to be removed oto?18:24
robcresswelltoo*18:24
*** armax has joined #openstack-nova18:25
andreykurilinrobcresswell: let it be token_id for now.18:28
andreykurilinactually, novaclient expects password there, but keystone allows to transmit token there18:28
robcresswellthat didnt seem to change anything, same issues18:29
*** unicell has joined #openstack-nova18:34
openstackgerritDan Peschman proposed openstack/nova: Option to always create network metadata information as static  https://review.openstack.org/33629318:34
openstackgerritSean Dague proposed openstack/nova: nova-status: add basic placement status checking  https://review.openstack.org/41188318:35
*** matrohon has joined #openstack-nova18:36
*** ihrachys has quit IRC18:37
andreykurilinrobcresswell: can you update novaclient to master?18:40
*** yamahata has joined #openstack-nova18:40
*** lpetrut has joined #openstack-nova18:40
*** moshele has quit IRC18:41
*** nic has quit IRC18:42
*** nic has joined #openstack-nova18:42
robcresswellandreykurilin: ooo, new errors!18:43
robcresswellI call that progress18:43
robcresswellgimme a minute to figure out whats changed18:43
andreykurilinrobcresswell: waiting for paste:)18:43
openstackgerritMaciej Szankin proposed openstack/nova: conf: refactor conf_fixture.py  https://review.openstack.org/41258118:50
robcresswellandreykurilin: Its working now18:52
andreykurilincool18:53
robcresswellandreykurilin: So, is this a novaclient bug? Or a Horizon implementation bug?18:53
robcresswell(or both)18:54
andreykurilinmriedem: usage of SessionClient instead of HTTPClient helped to discover microversions in horizon ^ We need to recheck everything and cut a release:)18:54
andreykurilinrobcresswell: I think both18:54
andreykurilinrobcresswell: recently, we merged a big change to novaclient which should fix a lot of issues. it is unreleased yet.18:55
openstackgerritMaciej Szankin proposed openstack/nova: conf: added notifications group  https://review.openstack.org/39214518:55
andreykurilinrobcresswell: from horizon side, it sets properties of inner novaclient's object. it is wrong behaviour. Please, propose a patch to horizon repo with changes you made for initialization of novaclient18:56
openstackgerritMaciej Szankin proposed openstack/nova: conf: remove deprecated ironic options  https://review.openstack.org/40559718:56
robcresswellandreykurilin: Any rough ETA on that release?18:56
robcresswellandreykurilin: Sure, I can just push that as part of the microversion patch18:57
openstackgerritMaciej Szankin proposed openstack/nova: Transform instance.reboot notifications  https://review.openstack.org/38295918:57
andreykurilinrobcresswell: we need to check that our changes doesn't break anything:) also, there are small nits that should be fixed before release. I can't give you any ETA, mriedem may help you.18:58
andreykurilinFrom my side, I'll propose all required patches this week18:58
mriedemcan someone summarize? i'm debugging a customer issue atm so i haven't been following along.18:58
*** baoli has quit IRC18:59
*** baoli has joined #openstack-nova19:00
robcresswellmriedem: I'm not sure to be honest. In short, novaclient 6.0.0 failed consistently on api_versions.discover_version(). Updating it to master caused it to work as expected.19:00
robcresswellI don't know any further details on the nova side.19:00
openstackgerritMaciej Szankin proposed openstack/nova: conf: move few console opts to xenserver group  https://review.openstack.org/38964719:03
andreykurilinmriedem, sum: horizon doesn't create keystone session manually, they transmit all arguments to novaclient. HTTPClient impl s*cks and for some reason doesn't work. SessionClient fixes all issues(after switching to use master branch) and horizon can use Nova's microversions now! :)19:04
openstackgerritDiana Clarke proposed openstack/python-novaclient: Microversion 2.40 - Simple tenant usage pagination  https://review.openstack.org/40782519:04
openstackgerritMaciej Szankin proposed openstack/nova: conf: move few console opts to xenserver group  https://review.openstack.org/38964719:04
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: WIP: restrict some properties  https://review.openstack.org/41258819:05
*** browne has quit IRC19:05
*** stvnoyes has quit IRC19:06
robcresswellandreykurilin: So, we should be using SessionClient(...) instead of Client(...)?19:07
andreykurilinrobcresswell: no no no19:07
andreykurilinit is our inner stuff19:07
andreykurilineverything is done for you in master/branch of novaclient:)19:07
*** kylek3h is now known as kylek3h_19:08
robcresswellandreykurilin: Right... just a little frustrating to not be able to implement microversions even by Ocata :/19:13
*** fragatina has quit IRC19:13
*** david-lyle_ has joined #openstack-nova19:13
*** openstackstatus has quit IRC19:13
*** jamespag` has joined #openstack-nova19:13
*** jamespag` has quit IRC19:13
*** jamespag` has joined #openstack-nova19:13
*** david-lyle has quit IRC19:13
*** jamespage has quit IRC19:13
*** openstack has joined #openstack-nova19:15
andreykurilin@robcresswell: actually, there is a way to fix everything on horizon side without waiting our release, but it require more lines of code, so it is better to wait a bit.19:19
robcresswellandreykurilin: Could you explain? I really dont think we have time to wait for a release without an ETA.19:19
*** matrohon has quit IRC19:20
*** armax has quit IRC19:21
mriedemrobcresswell: i'm hoping to have a novaclient release this week,19:21
mriedembut trying to test it out against trove to make sure we don't break someone19:21
*** eharney has quit IRC19:22
*** kfarr has joined #openstack-nova19:24
mriedemcburgess: nic: so before you guys had fast snapshots with rbd, how were you dealing with large disk snapshots not failing? i.e. the token expiring between the time that we're getting the disk snapshot and then uploading that to glance? just long token timeouts?19:24
mriedemit seems really odd that nova creates a new session and new token with requests to other services like ironic, neutron and cinder but for glance we just pass the token through, which can timeout during a long running operation like snapshot19:26
*** tbachman_ has joined #openstack-nova19:26
*** tbachman has quit IRC19:26
*** tbachman_ is now known as tbachman19:26
robcresswellandreykurilin: Would you mind sharing how to fix this on the Horizon side? Cinder is broken for the exact same reason.19:27
*** armax has joined #openstack-nova19:31
*** baoli has quit IRC19:32
*** baoli has joined #openstack-nova19:32
*** ducttape_ has quit IRC19:37
*** eharney has joined #openstack-nova19:38
openstackgerritChris Dent proposed openstack/nova: [placement] placement_dev info for testing and gabbi  https://review.openstack.org/41260019:48
*** jaosorior has quit IRC19:52
*** roeyc has joined #openstack-nova19:55
*** rcernin has joined #openstack-nova19:56
*** lpetrut has quit IRC20:00
*** ducttape_ has joined #openstack-nova20:05
*** rfolco has quit IRC20:11
*** amoralej is now known as amoralej|off20:17
*** sreeram_v has joined #openstack-nova20:18
*** sreeram_v has quit IRC20:23
alaskimriedem: long timeouts is the only solution I've seen for that issue so far. It's been a long standing problem that trusts were supposed to help with, and more recently I think there have been other proposed solutions. But nothing solid ever materialized that I saw.20:24
mriedemalaski: yeah https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/use-service-tokens.html is the future20:25
mriedem"User start snapshot operation in Nova. User token expires during this operation. Nova call Glance to update final bits and that request is failed by Keystone due to expired user token."20:25
mriedem^ is exactly what i'm hitting with an 80GB disk :)20:25
alaskiyeah, it was happening fairly regularly in the large deployment I was involved in20:28
*** roeyc has quit IRC20:29
alaskilooks like OSIC is on that spec. Hopefully they can push that forward20:29
*** jaypipes has quit IRC20:30
*** Sukhdev has quit IRC20:33
*** stvnoyes has joined #openstack-nova20:35
*** cdent has quit IRC20:36
*** slaweq_ has joined #openstack-nova20:38
*** annegentle has joined #openstack-nova20:41
*** weshay is now known as weshay_relocatin20:42
sdaguemriedem: it does the same thing for cinder IIRC20:48
sdaguethe neutron code is misleading20:48
*** baoli has quit IRC20:48
mriedemsdague: we have ksa stuff for cinder20:48
sdaguesome times we work on behalf of the user with a user scoped token20:48
sdagueand sometimes we just admin it20:49
sdaguemriedem: hmmm... ok, I guess the cinder bits are a little different than I last remember them20:50
sdaguemaybe when we purged all the proxies we got rid of some of that other stuff20:51
mriedemspeaking of purging things, we still need to purge the glance v1 code in nova20:56
*** siva_krishnan_ has quit IRC21:00
*** siva_krishnan_ has joined #openstack-nova21:01
*** tlian has joined #openstack-nova21:01
*** jamielennox|away is now known as jamielennox21:04
*** baoli has joined #openstack-nova21:04
mriedemalright, crisis postponed, i'll work on adding the RP counting to the placement nova-status patch now21:09
*** GMAzrael_ has joined #openstack-nova21:12
*** rtheis has quit IRC21:13
GMAzrael_Oh Wise developers. How would I increaase the idle timeout for websocketify?21:14
*** gouthamr has quit IRC21:17
*** sreeram_v has joined #openstack-nova21:17
stevemarmriedem: hmm, jamielennox has been working on that stuff21:18
*** Guest1622 has quit IRC21:18
stevemarmriedem: actually, all the keystone and keystonemiddleware parts have landed21:18
stevemarmriedem: http://specs.openstack.org/openstack/keystone-specs/specs/keystone/ocata/allow-expired.html21:18
mriedemstevemar: yeah21:19
mriedemhence the nova spec to use them21:19
stevemarmriedem: yessss, just making sure that you knew it revolved around validating expired tokens21:20
stevemarand using a service token isn't enough21:21
*** sreeram_v has quit IRC21:22
mriedemalright21:24
*** smatzek has quit IRC21:29
*** kfarr has quit IRC21:29
*** _ducttape_ has joined #openstack-nova21:29
*** ducttape_ has quit IRC21:33
*** _ducttape_ has quit IRC21:34
openstackgerritMatt Riedemann proposed openstack/nova: rt: pass the nodename to public methods  https://review.openstack.org/39847021:34
openstackgerritMatt Riedemann proposed openstack/nova: rt: remove fluff from test_resource_tracker  https://review.openstack.org/39847121:34
openstackgerritMatt Riedemann proposed openstack/nova: placement: RT now adds proper Ironic inventory  https://review.openstack.org/40447221:34
openstackgerritMatt Riedemann proposed openstack/nova: rt: explicitly pass compute node to _update()  https://review.openstack.org/39847221:34
openstackgerritMatt Riedemann proposed openstack/nova: rt: use a single ResourceTracker object instance  https://review.openstack.org/39847321:34
*** ssurana has joined #openstack-nova21:36
openstackgerritAnusha Unnam proposed openstack/nova: Do not merge: Add post hook script to run tempest with old n-api  https://review.openstack.org/40139521:39
*** eharney has quit IRC21:40
*** liverpooler has quit IRC21:41
jamielennoxstevemar, mriedem: the extension of that spec however would be that we should change the policies for service->service operations so that a user/service token is sufficient rather than need the nova user to have admin priviledges21:46
*** thorst has quit IRC21:46
*** thorst has joined #openstack-nova21:46
*** fragatina has joined #openstack-nova21:48
mordredsdague: (or anyone else) ... http://developer.openstack.org/api-ref/compute/?expanded=show-usage-statistics-for-tenant-detail#usage-reports-os-simple-tenant-usage shows v2.40 in the URL - my understanding of microversions was that it was all header-based21:49
mordredis that doc correct?21:49
mordred(I also dont see a version in the URL in python-novaclient21:50
mriedemmordred: the doc is busted21:51
mriedemmordred: report a bug?21:51
mriedemhttps://github.com/openstack/nova/blob/master/api-ref/source/os-simple-tenant-usage.inc#L7821:52
mriedemthat shouldn't have v2.40 hard-coded into it21:52
mordredI'll make a bug/patch21:52
mriedemthere might be v2.40 request/response wrinkles for the limit/marker, but those go into the parameters.yaml21:52
mriedemfyi it comes from this recently merged patch https://github.com/openstack/nova/commit/83404013cb53aef16b97b5616c0627c50af76ac821:52
mriedemdiana_clarke: ^21:53
*** browne has joined #openstack-nova21:53
mriedemi bet it was just a mistake when making a similar change to the ../../doc/api_samples/os-simple-tenant-usage/v2.40/simple-tenant-usage-get-detail.json example21:53
mordredyah - that's what it seems like21:53
diana_clarkemriedem: looking21:53
diana_clarkemriedem: I'll toss a patch up that removes the min version21:54
openstackgerritMonty Taylor proposed openstack/nova: Remove v2.40 from URL string in usage API docs  https://review.openstack.org/41262421:54
mordreddiana_clarke: ^^21:54
*** fragatina has quit IRC21:54
*** thorst has quit IRC21:55
*** fragatina has joined #openstack-nova21:55
mriedem+121:56
mriedemoops +221:56
mriedemthe response examples in there now are for 2.40 and confuse someone that assumes they are for v2.121:56
diana_clarkemriedem: I originally left the old examples, but was told to switch them to the new examples.21:57
mriedemok, might be good to put a (v2.40) in the example title or something21:58
mriedemalso, it doesn't look like we have a description of tenant_usage_links in the response descriptions21:58
mordredwhy do I get the feeling I'm about to have a BUNCH of fun learning what you fine people have been doing with microversions all this time21:58
*** fragatina has quit IRC21:58
diana_clarkemriedem: ah, good call. I can do those minor fixes over the holidays.21:59
*** fragatina has joined #openstack-nova21:59
mriedemmordred: as long as you're (1) not using nova-network and (2) are using v2.1 you should be fine21:59
mriedemfrom a client pov21:59
mordredmriedem: well.... that's just the thing22:00
mordredI get the fun of using all of those things22:00
mriedemmordred: then you should be happy about microversions22:00
mordredI probably will be22:01
mordredI've just been ignoring them so far because I wasn't doing direct REST calls for anything22:01
mordredyay new learning22:01
mriedemdiana_clarke: my xmas gift to you https://bugs.launchpad.net/nova/+bug/165125422:02
openstackLaunchpad bug 1651254 in OpenStack Compute (nova) ""tenant_usage_links" isn't described in the v2.40 os-simple-tenant-usage API reference" [Low,Confirmed]22:02
*** tbachman has quit IRC22:03
*** tbachman has joined #openstack-nova22:03
mriedemmordred: btw, thanks for bringing up the api-ref issue - things are much better than before but there are still plenty of loose ends and warts22:04
*** smatzek has joined #openstack-nova22:05
mordredmriedem: yah - the docs are actually REALLY good and usable now - to the point where when there is a weirdness it makes sense to ask about it22:05
*** fragatina has quit IRC22:06
kukaczhi, I got a couple of instances which can't be deleted using nova delete commands, neither reset-state helps22:07
kukaczthose instances are deleted from hypervisor however22:08
kukaczis there a recommended process how to complete the deletion?22:09
*** Sukhdev has joined #openstack-nova22:10
*** dave-mccowan has joined #openstack-nova22:11
kukaczI could find several blog posts about how to soft-delete them by sql calls on database22:12
kukaczbut there's also a nova_nuke.py https://gist.github.com/DavidWittman/6831237 which does that via python nova.db.sqlalchemy  calls22:13
kukaczI wonder if there's something more or less recommended by nova developers?22:14
*** annegentle has quit IRC22:17
openstackgerritDiana Clarke proposed openstack/nova: Missing usage next links in api-ref  https://review.openstack.org/41263222:19
diana_clarkemriedem: ^ that's just following the established pattern which is pretty vague. Anyhoo, I also owe you and alex_xu a paragraph about 2.40. That, I'll do later.22:20
diana_clarkefrom mexico!!!22:20
diana_clarkemriedem: Oh, and before I forget. Here's the other follow-up patch I promised you last week. https://review.openstack.org/#/c/411441/22:24
mriedemthanks22:24
diana_clarkemriedem: It documents & tests all the endpoints that support offset & limit. (I originally only fixed the one I needed fixed downstream)22:24
mriedemkukacz: forceDelete? http://developer.openstack.org/api-ref/compute/?expanded=delete-server-detail,force-delete-server-forcedelete-action-detail#force-delete-server-forcedelete-action22:26
mriedemdiana_clarke: where in mexico?22:27
mriedemwe'll be in cabo in late january22:27
mriedembtw, enjoy the 6 hour flight with a baby22:27
mriedem:)22:27
openstackgerritSarafraj Singh proposed openstack/nova: WIP: Add service_token for nova-glance interaction  https://review.openstack.org/41263422:28
*** baoli has quit IRC22:29
kukaczmriedem: I've tried that earlier from nova CLI - did not help. the error I'm seeing logged at instance is 504 Gateway error22:30
diana_clarkemriedem: My uncle just built a place near/in Punta de Mita. I'm pretty excited.22:33
diana_clarkemriedem: Pro tip: if you don't pay for seat selection, they randomly place your family and strangers can look after your kids.22:33
diana_clarkewhy would I pay to look after my kids?!22:34
*** smatzek has quit IRC22:37
*** jheroux has quit IRC22:39
kukaczmriedem: it's Kilo release, instances booted from volumes, Ceph backend in Cinder. In most cases instance deletion deletion works smoothly for us22:40
kukaczthese instances are hanging there in error state for some time already. all seem to have passed through an unsuccesfull deletion attempt22:41
openstackgerritDiana Clarke proposed openstack/nova: Missing usage next links in api-ref  https://review.openstack.org/41263222:42
mriedemsdague: just starting the rp / compute node comparisons, we might want to actually make that it's own check22:44
mriedembecause it's not trivial22:44
mriedemlike, cells v2 check, placement API check, resource providers check22:44
mriedemtesting it is going to get a bit funky22:45
mriedemespecially with stubbing out compute_nodes in multiple cell databases and relying on querying the api db22:45
mriedemoh my22:45
*** slaweq_ has quit IRC22:46
*** esberglu has quit IRC22:47
*** esberglu has joined #openstack-nova22:48
*** mvk has joined #openstack-nova22:51
*** esberglu has quit IRC22:52
*** thorst has joined #openstack-nova22:52
*** xyang1 has quit IRC22:53
*** Swami has quit IRC22:56
*** baoli has joined #openstack-nova22:56
*** eharney has joined #openstack-nova22:58
*** thorst has quit IRC23:00
*** baoli has quit IRC23:01
*** esberglu has joined #openstack-nova23:01
*** pradk has quit IRC23:01
*** annegentle has joined #openstack-nova23:04
*** esberglu has quit IRC23:05
*** annegentle has quit IRC23:12
*** mlavalle has quit IRC23:19
*** nherciu_ has quit IRC23:21
*** ducttape_ has joined #openstack-nova23:31
*** lamt has quit IRC23:32
*** Sukhdev has quit IRC23:34
*** ducttape_ has quit IRC23:34
*** ducttape_ has joined #openstack-nova23:35
*** tlian has quit IRC23:35
*** ducttape_ has quit IRC23:39
openstackgerritSujitha proposed openstack/nova: Adding tags to InstancePayload  https://review.openstack.org/40722823:42
*** sdague has quit IRC23:44
*** rfolco has joined #openstack-nova23:50
*** rfolco has quit IRC23:50
*** dave-mccowan has quit IRC23:50
*** nicolasbock has quit IRC23:54
openstackgerritSujitha proposed openstack/nova: Adding tags to InstancePayload  https://review.openstack.org/40722823:54
*** ducttape_ has joined #openstack-nova23:55
*** thorst has joined #openstack-nova23:57
*** gouthamr has joined #openstack-nova23:58
*** jwcroppe has quit IRC23:58
*** jwcroppe has joined #openstack-nova23:59
*** tlian has joined #openstack-nova23:59
*** Sukhdev has joined #openstack-nova23:59
*** ducttape_ has quit IRC23:59

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