Friday, 2016-12-23

*** ijw has quit IRC00:01
*** catintheroof has joined #openstack-nova00:05
*** ducttape_ has quit IRC00:09
*** rcernin has quit IRC00:16
*** raunak has quit IRC00:19
*** ducttape_ has joined #openstack-nova00:20
*** xinliang has joined #openstack-nova00:22
*** ijw has joined #openstack-nova00:22
*** ijw has quit IRC00:27
*** Zer0Byte__ has quit IRC00:28
*** yingjun has joined #openstack-nova00:30
mriedemjaypipes: weird, the generation id isn't changing00:32
mriedemhttp://logs.openstack.org/30/414230/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/7fd2379/logs/screen-n-cpu.txt.gz#_2016-12-22_17_21_16_09700:32
mriedemthat's with my debug patch00:32
*** ijw has joined #openstack-nova00:32
mriedemwell, i guess the gen id wouldn't change if we just never have a successful update00:32
mriedemand yeah just everything is reporting as 000:32
mriedemram/cpu/disk00:32
mriedemvsaienk0: ^00:34
mriedemthe ironic driver isn't reporting any resources it looks like00:35
*** ijw has quit IRC00:37
*** fragatina has joined #openstack-nova00:40
*** fragatina has quit IRC00:41
*** fragatina has joined #openstack-nova00:41
*** ducttape_ has quit IRC00:48
*** kylek3h__ has joined #openstack-nova00:53
*** ducttape_ has joined #openstack-nova00:55
*** kylek3h_ has quit IRC00:56
*** roeyc has joined #openstack-nova01:02
*** lbeliveau_ has joined #openstack-nova01:03
*** yamahata has quit IRC01:06
*** yamahata has joined #openstack-nova01:06
alex_xumriedem: just replied in spec https://review.openstack.org/#/c/393205/19/specs/ocata/approved/add-whitelist-for-server-list-filter-sort-parameters.rst01:11
alex_xumriedem: we will make the filter name 'hypervisor_hostname' rather than node in the future. and with new microversion i think01:12
*** ssurana has joined #openstack-nova01:12
*** ssurana has quit IRC01:12
*** ducttape_ has quit IRC01:12
alex_xuotherwise we expose some new filters without any signal, that break the case when people access different cloud some clouds have new filters and some clouds haven't01:13
*** ijw has joined #openstack-nova01:14
*** yamahata has quit IRC01:17
*** ijw has quit IRC01:19
openstackgerritAlex Xu proposed openstack/nova: Use JSON-Schema to validate query parameters for keypairs API  https://review.openstack.org/39875501:20
openstackgerrithuangtianhua proposed openstack/python-novaclient: Make _console() public  https://review.openstack.org/41436401:22
openstackgerritAlex Xu proposed openstack/nova: Use JSON-Schema to validate query parameters for keypairs API  https://review.openstack.org/39875501:25
openstackgerritAlex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters  https://review.openstack.org/38900301:25
alex_xuoomichi: mriedem Kevin_Zheng ^ the patch revert to the right version01:26
Kevin_Zhengalex_xu: Thanks, sorry for the accident :(01:27
alex_xuKevin_Zheng: no worries :)01:27
* alex_xu will back to online after an hour01:27
*** hfu has joined #openstack-nova01:28
*** zhangjl has joined #openstack-nova01:28
*** zsli has joined #openstack-nova01:30
openstackgerritZhenyu Zheng proposed openstack/nova: Update hostname of server when updating its name  https://review.openstack.org/40458301:33
*** ijw has joined #openstack-nova01:35
*** ijw has quit IRC01:40
*** catintheroof has quit IRC01:40
*** fragatina has quit IRC01:42
*** unicell has quit IRC01:43
mriedemalex_xu: ah ok, i guess i hadn't thought about people actually trying to filter on node today01:45
*** unicell has joined #openstack-nova01:46
*** unicell has quit IRC01:50
*** yufei has joined #openstack-nova01:54
*** zz_dimtruck is now known as dimtruck02:05
*** ducttape_ has joined #openstack-nova02:13
*** roeyc has quit IRC02:14
*** roeyc has joined #openstack-nova02:17
*** ducttape_ has quit IRC02:18
*** yufei has left #openstack-nova02:23
*** kylek3h__ has quit IRC02:30
*** Jinxing has joined #openstack-nova02:35
mriedemalex_xu: Kevin_Zheng: another question i had on that spec was if we can filter/sort on the instances.name field, which isn't actually a column in the instances table, it's a generated property on the instances model object02:36
mriedemoh i see https://github.com/openstack/nova/blob/master/nova/compute/api.py#L233802:37
alex_xumriedem: 'name' is only for filters. 'display_name' is only for sorts02:38
alex_xumriedem: also found that 'instance_type_id' doesn't works, but the reason isn't that it is integer. The vcpus is integer in the db model, but it works.02:39
mriedemalex_xu: how about L111 and L143 in https://review.openstack.org/#/c/393205/19/specs/ocata/approved/add-whitelist-for-server-list-filter-sort-parameters.rst ?02:40
mriedemi'm wondering about the parts where it says we'll error on joined tables but ignore other parameters, i'm not sure how we tell the difference without jsonschema?02:41
*** ijw has joined #openstack-nova02:41
alex_xumriedem: we haven't put the joined-tables filters/sorts in the schema. But in a constant variable, then we return 400 for those filters/sorts in python02:44
alex_xu/haven't/have to/02:44
mriedemyou mean don't put them in the schema? if they aren't in the schema, then wouldn't they get a 400 from the jsonschema validation? or are you planning on using additionalProperties: True (i think you were, which was in the spec)02:45
*** ijw has quit IRC02:46
mriedemi might be confusing this with something else though02:46
alex_xumriedem: I mean put them in the schema. But raise 400 in the python code02:46
mriedemwhy would we put them in the schema though?02:46
mriedemif they aren't in the schema and additionalProperties=False, then jsonschema will handle the 400 for us02:46
alex_xumriedem: but we need to set additionalProperties=True for now02:47
alex_xuwe don't want to return 400 for all the invalid parameters, that may break user, due to we removed few filters in this propose02:47
mriedemok, so if additionalProperties=True, and we're going to blacklist those joined table columns in python, then i don't see why we'd put them in the schema, that seems redundant02:48
openstackgerritZhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed  https://review.openstack.org/40356802:48
mriedemmaybe this is worth just hashing out in the actual code review :)02:48
mriedemhow about the policy default change question i have on L142?02:49
alex_xumriedem: if addtionalProperties=True, then https://review.openstack.org/#/c/389003/21/nova/api/validation/__init__.py@89 will strip all the invalid parameters out. Then it won't trigger any 40002:50
alex_xumriedem: yea, we are going to change the default rule to '@'02:50
alex_xuand should have a note about the old rule is going to deprecated, maybe remove in the future02:51
mriedemwon't that make all_tenants available to everyone in a fresh install?02:51
mriedemor,02:52
alex_xumriedem: yes, but the new rule 'os_compute_api:servers:all_tenants_visible' will take care about that02:52
mriedemi guess the idea is that on a fresh install, get_all_tenants is @ for everyone, but not used? and then the new all_tenants_visible is admin-only?02:52
alex_xuyea02:52
mriedemok, i guess i wouldn't even change the default value for get_all_tenants02:53
mriedemif we just are going to replace it and not use it, then i don't see the point in changing the default value02:53
alex_xuthat as the config option, we can't remove the old one directly, we should deprecate it first then remove it?02:53
mriedemwe don't need to just remove it, but we don't need to change it's default value to @ either02:54
alex_xuif we replace it, i guess if the user custom the rule of 'all_tenants', then it can't startup the service after upgrade02:54
mriedemwell i mean replace it's usage in the code02:55
mriedemi.e. after this change will we even use the get_all_tenants option in code?02:55
mriedems/option/policy rule/02:55
alex_xuyou mean keep it, but we didn't use the rule 'all_tenants' in the code anymore?02:56
*** ijw has joined #openstack-nova02:56
mriedemthat's what i'm trying to figure out02:57
mriedemin other words, when we release ocata,02:58
mriedemwill any of the code read the policy rule for 'os_compute_api:servers:detail:get_all_tenants'?02:58
mriedembecause if not we could just drop it, or at least not change it's default value02:58
alex_xuin the propose, yes. if the user change get_all_tenants back to admin-only, it still return 403 for user02:59
alex_xufor non-admin user02:59
mriedemso we're going to check that rule and os_compute_api:servers:all_tenants_visible at the same time?03:00
alex_xuyea03:00
alex_xuand due to the default rule is '@', so 'get_all_tenants' actually just passed for any case03:01
*** ijw has quit IRC03:01
mriedemin a fresh install03:01
alex_xualso for the user didn't change the default rule before03:02
mriedemfor anyone that has get_all_tenants as rule:admin_api from newton it will still be a 40303:02
mriedemAND it would have to be explicitly in their policy.json03:02
alex_xuwe already put the default value into the python code03:02
mriedemand not just the policy in code03:02
mriedemok, i think i follow now03:03
alex_xuyea, if they didn't clearup their policy.json, still be a 40303:03
mriedemif you still want it to be admin_api and fail with a 403 for non-admins, put that in your policy.json as an override03:03
alex_xumriedem: yea03:03
mriedemotherwise we'll change the default in code and it'll be admin-only but not be a 403 for non-admins03:03
mriedemsomehow, with code magic03:03
mriedemdo we have code today that doesn't make a policy rule violation not fail with Forbidden?03:04
alex_xuI guess no, Kevin_Zheng work on that part, but I guess he didn't get chance to update the code yet. Due to we just propose this new rule last week03:05
openstackgerritZhenyu Zheng proposed openstack/nova: Update hostname of server when updating its name  https://review.openstack.org/40458303:05
Kevin_Zhengyes, not started the policy part yet03:05
*** baoli has joined #openstack-nova03:07
*** yamahata has joined #openstack-nova03:08
mriedemok,03:08
mriedemif i were an operator setting this up it'd probably be confusing03:08
mriedemif the rule is admin-only but non-admins can make the request but not fail03:09
*** raunak has joined #openstack-nova03:09
*** wlhc has joined #openstack-nova03:11
alex_xumriedem: there is discussion about hierarchy tenants, that is the reason sdague and johnthetubaguy thinks the all_tenants should be allow for everyone, and be controlled by role in the future03:11
mriedemok i think your logic on the response to L74 is also probably part of it03:13
mriedemthe fatal=False part of the policy check03:13
mriedemfail the policy, but don't raise an error, just return False03:13
mriedemso we don't elevate the context03:13
alex_xuyea03:14
alex_xumriedem: and you can catch few discussion at http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2016-12-15.log.html#t2016-12-15T13:27:2503:14
*** mdnadeem has joined #openstack-nova03:14
*** ducttape_ has joined #openstack-nova03:14
mriedemalex_xu: i left some more notes/scenarios from our irc discussion in the deployer impact section, i think it would be good to explain or give examples for the different scenarios for the policy changes for a fresh install with both admin and non-admin users, and then an upgrade from newton where get_all_tenants is overridden in policy.json03:14
mriedemi think i'm clear on everything now03:14
alex_xumriedem: thanks, i will update :)03:15
mriedemwe just need some clarifications in the spec i think so that when we write release notes on this it's clear to operators that are upgrading03:15
alex_xugot it03:15
mriedemand with that, i'm done for the day - thanks for being patient on this one and explaining everything.03:15
Kevin_Zhengmriedem: Merry Christmas :)03:16
alex_xumriedem: np, have a good night03:16
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/41429903:16
alex_xuyea, Merry Christmas :)03:16
*** baoli has quit IRC03:17
*** roeyc has quit IRC03:18
*** roeyc has joined #openstack-nova03:18
*** ducttape_ has quit IRC03:19
*** baoli has joined #openstack-nova03:21
openstackgerritint32bit proposed openstack/nova: Add more actions to instance action logs  https://review.openstack.org/39651003:23
*** ijw has joined #openstack-nova03:31
*** fragatina has joined #openstack-nova03:33
*** mdnadeem has quit IRC03:35
*** ijw has quit IRC03:36
*** fragatin_ has joined #openstack-nova03:41
*** browne has quit IRC03:42
*** fragatina has quit IRC03:43
*** fragatin_ has quit IRC03:43
*** fragatina has joined #openstack-nova03:44
*** yuntongjin has joined #openstack-nova03:47
*** raunak has quit IRC03:48
*** baoli has quit IRC03:50
*** raunak has joined #openstack-nova03:56
*** raunak has quit IRC03:56
*** lbeliveau_ has quit IRC04:00
*** zsli has quit IRC04:05
*** diga has joined #openstack-nova04:06
*** ijw has joined #openstack-nova04:07
*** zsli has joined #openstack-nova04:08
*** psachin has joined #openstack-nova04:12
*** ratailor has joined #openstack-nova04:13
*** ducttape_ has joined #openstack-nova04:15
*** zsli_ has joined #openstack-nova04:17
*** yingjun has quit IRC04:17
*** zsli has quit IRC04:19
*** ducttape_ has quit IRC04:20
*** gouthamr has quit IRC04:20
*** mdnadeem has joined #openstack-nova04:21
*** tjones has joined #openstack-nova04:26
*** yamahata has quit IRC04:28
*** yamahata has joined #openstack-nova04:29
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port api.openstack.compute.test_disk_config to Python 3  https://review.openstack.org/40353604:29
*** yuntongjin has quit IRC04:35
*** yingjun has joined #openstack-nova04:37
*** ishant|away has quit IRC04:37
*** zsli_ has quit IRC04:43
openstackgerritZhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed  https://review.openstack.org/40356804:43
*** nicolasbock has quit IRC04:48
*** raunak has joined #openstack-nova04:52
*** roeyc has quit IRC04:53
*** weshay_afk is now known as weshay_pto04:57
*** cheran has quit IRC04:57
*** jamespag` has quit IRC05:00
*** jamespage has joined #openstack-nova05:01
*** kevinz has joined #openstack-nova05:02
*** raunak has quit IRC05:06
*** pbandark has joined #openstack-nova05:08
*** artom has joined #openstack-nova05:09
*** ducttape_ has joined #openstack-nova05:16
*** ducttape_ has quit IRC05:20
*** ayogi has joined #openstack-nova05:21
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enble compute.test_compute_mgr.ComputeManagerUnitTestCase on Python 3  https://review.openstack.org/41054205:21
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enble network.test_neutronv2.TestNeutronv2 on Python 3  https://review.openstack.org/41441805:21
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enable virt.test_virt_drivers.AbstractDriverTestCase on Python 3  https://review.openstack.org/41441905:21
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enable virt.vmwareapi test cases on Python  https://review.openstack.org/41442005:21
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enable more virt.xenapi tests on Python 3  https://review.openstack.org/41442105:22
*** unicell has joined #openstack-nova05:22
*** zsli has joined #openstack-nova05:31
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port security group related tests to Python 3  https://review.openstack.org/40124105:33
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port compute.test_user_data.ServersControllerCreateTest to Python 3  https://review.openstack.org/40643505:33
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port compute unit tests to Python 3  https://review.openstack.org/40201005:33
*** prateek has joined #openstack-nova05:34
*** ijw has quit IRC05:53
*** yuntongjin has joined #openstack-nova05:53
*** ijw has joined #openstack-nova05:54
*** stvnoyes1 has quit IRC05:55
*** stvnoyes has joined #openstack-nova05:55
*** ijw has quit IRC05:55
*** ijw has joined #openstack-nova05:56
openstackgerritint32bit proposed openstack/nova: Add more actions to instance action logs  https://review.openstack.org/39651005:58
*** ijw_ has joined #openstack-nova06:00
*** adisky_ has joined #openstack-nova06:00
*** ijw has quit IRC06:02
*** ijw_ has quit IRC06:04
*** ijw has joined #openstack-nova06:06
*** Sukhdev has joined #openstack-nova06:07
openstackgerritAlex Xu proposed openstack/nova-specs: Add whitelist for filter and sort query parameters for server list API  https://review.openstack.org/39320506:08
alex_xuKevin_Zheng: mriedem ^ updated06:09
Kevin_ZhengI saw :)06:13
*** raunak has joined #openstack-nova06:14
*** ducttape_ has joined #openstack-nova06:17
*** hfu has quit IRC06:17
*** ducttape_ has quit IRC06:21
*** ijw_ has joined #openstack-nova06:24
*** ijw_ has quit IRC06:25
*** ijw_ has joined #openstack-nova06:26
*** ijw has quit IRC06:28
*** ijw_ has quit IRC06:31
*** salv-orlando has joined #openstack-nova06:44
openstackgerritZhenyu Zheng proposed openstack/nova: Trival fix typo  https://review.openstack.org/41444006:46
*** sridharg has joined #openstack-nova06:46
*** diga has quit IRC06:49
*** ecelik has joined #openstack-nova06:50
*** ecelik has left #openstack-nova06:50
*** dimtruck is now known as zz_dimtruck06:55
*** avolkov has quit IRC07:03
*** mnasiadka has joined #openstack-nova07:10
*** yamahata has quit IRC07:14
*** yuntongjin has quit IRC07:17
*** raunak has quit IRC07:18
*** tesseract has joined #openstack-nova07:22
*** tesseract is now known as Guest3157607:22
*** ducnc has quit IRC07:25
openstackgerritTao Li proposed openstack/nova: Fix the display of updated_at time when using memcache driver.  https://review.openstack.org/41446307:34
*** salv-orlando has quit IRC07:40
*** zhangjl1 has joined #openstack-nova07:40
*** zhangjl has quit IRC07:42
*** yuntongjin has joined #openstack-nova07:42
*** mjura has joined #openstack-nova07:44
openstackgerritTao Li proposed openstack/nova: Fix the display of updated_at time when using memcache driver.  https://review.openstack.org/41446307:50
*** juszczus has quit IRC07:52
*** pcaruana has quit IRC07:54
*** pcaruana has joined #openstack-nova07:55
*** avolkov has joined #openstack-nova07:57
*** diga has joined #openstack-nova08:10
*** gszasz has joined #openstack-nova08:11
*** armax has quit IRC08:12
*** ijw has joined #openstack-nova08:12
*** slaweq has joined #openstack-nova08:12
*** ijw has quit IRC08:17
*** sahid has joined #openstack-nova08:18
*** amoralej|off is now known as amoralej08:20
*** yingjun has quit IRC08:24
*** flaper87 has quit IRC08:24
*** ijw has joined #openstack-nova08:25
*** yingjun has joined #openstack-nova08:25
*** flaper87 has joined #openstack-nova08:26
*** zenoway has joined #openstack-nova08:32
*** ihrachys has joined #openstack-nova08:39
*** ralonsoh has joined #openstack-nova08:44
*** ijw_ has joined #openstack-nova08:53
*** ijw_ has quit IRC08:54
*** ijw has quit IRC08:54
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:02
*** yuntongjin has quit IRC09:03
vsaienk0vsaienk009:04
*** ducnc has joined #openstack-nova09:09
*** ihrachys has quit IRC09:10
*** derekh has joined #openstack-nova09:10
*** wlhc has quit IRC09:14
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Fix urllib.urlencode issue in functional tests on Python 3  https://review.openstack.org/41055409:14
*** yingjun_1 has joined #openstack-nova09:17
*** yingjun has quit IRC09:18
*** amotoki has joined #openstack-nova09:18
gcbsfinucan,  can you approve https://review.openstack.org/#/c/403423/ again09:20
sfinucangcb: Sure09:21
gcbsfinucan,  thanks09:21
gcbsfinucan,  I split commit https://review.openstack.org/#/c/410542/  into small ones as you and mriedem suggested09:29
sfinucangcb: Great. I would guess the plan is to merge them slowly over a few weeks, so I can look at the first for now09:29
*** slaweq has quit IRC09:30
gcbsfinucan,  yeah that's also what I expect, have enough time between merging them :-)09:31
*** zsli has quit IRC09:31
*** lucas-afk is now known as lucasagomes09:34
openstackgerritAndrea Visnyei proposed openstack/nova: Parameter verification of os-security-groups  https://review.openstack.org/38359809:39
*** salv-orlando has joined #openstack-nova09:41
*** vtech has joined #openstack-nova09:42
*** amotoki has quit IRC09:42
*** yingjun_1 has quit IRC09:44
*** salv-orlando has quit IRC09:46
*** Sukhdev has quit IRC09:48
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Fix "Message object do not support addition".  https://review.openstack.org/41452910:00
*** salv-orlando has joined #openstack-nova10:05
openstackgerritint32bit proposed openstack/nova: Add more actions to instance action logs  https://review.openstack.org/39651010:16
*** slaweq has joined #openstack-nova10:20
*** kevinz has quit IRC10:24
*** slaweq has quit IRC10:28
*** Jinxing has quit IRC10:30
*** zhangjl1 has left #openstack-nova10:30
*** vtech has quit IRC10:34
*** rcernin has joined #openstack-nova10:36
*** mvk has joined #openstack-nova10:39
*** rcernin has quit IRC10:45
*** derekh has quit IRC10:57
*** rmart04 has joined #openstack-nova10:57
*** ducnc has quit IRC11:01
*** derekh has joined #openstack-nova11:05
*** rnoriega has quit IRC11:07
*** rnoriega has joined #openstack-nova11:09
*** armax has joined #openstack-nova11:13
*** diga has quit IRC11:17
*** rfolco has joined #openstack-nova11:23
*** smatzek has joined #openstack-nova11:24
*** sahid has quit IRC11:26
*** sahid has joined #openstack-nova11:33
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port security group related tests to Python 3  https://review.openstack.org/40124111:35
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port compute.test_user_data.ServersControllerCreateTest to Python 3  https://review.openstack.org/40643511:35
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port compute unit tests to Python 3  https://review.openstack.org/40201011:35
*** nicolasbock has joined #openstack-nova11:42
*** dhellmann has quit IRC11:42
*** dhellmann has joined #openstack-nova11:42
*** gabor_antal has quit IRC11:48
*** ducttape_ has joined #openstack-nova11:48
*** ijw has joined #openstack-nova11:55
*** ratailor has quit IRC11:56
*** cdent has joined #openstack-nova11:58
*** jamesdenton has quit IRC11:58
*** jamesdenton has joined #openstack-nova11:58
*** derekh has quit IRC12:00
*** ijw has quit IRC12:00
*** ducttape_ has quit IRC12:09
*** roeyc has joined #openstack-nova12:15
*** roeyc has quit IRC12:19
openstackgerritNeil Jerram proposed openstack/nova: libvirt: avoid generating script with empty path  https://review.openstack.org/41193612:20
cdentmriedem: I don't reckon there's an easy fix to https://review.openstack.org/#/c/411327/12:21
cdent(I wrote a lengthy but mostly useless comment)12:22
*** slaweq has joined #openstack-nova12:24
openstackgerritPawel Koniszewski proposed openstack/nova: Add new configuration option live_migration_scheme  https://review.openstack.org/41081712:25
*** prateek has quit IRC12:30
*** mdnadeem has quit IRC12:32
*** rtheis has joined #openstack-nova12:35
*** sridharg has quit IRC12:41
*** salv-orlando has quit IRC12:43
gcbsfinucan,  another commit needs re-approve : https://review.openstack.org/#/c/410554  thanks12:46
*** slaweq has quit IRC12:51
*** roeyc has joined #openstack-nova12:52
*** lucasagomes is now known as lucas-hungry12:53
*** zhs_ has quit IRC12:55
*** salv-orlando has joined #openstack-nova12:55
*** irenab__ has left #openstack-nova13:00
*** prateek has joined #openstack-nova13:04
*** ayogi has quit IRC13:04
*** ducttape_ has joined #openstack-nova13:10
*** gcb has quit IRC13:11
*** pradk has quit IRC13:11
*** ducttape_ has quit IRC13:15
*** sandanar has joined #openstack-nova13:16
*** sandanar_ has joined #openstack-nova13:16
*** ociuhandu has quit IRC13:18
*** jheroux has joined #openstack-nova13:19
*** cdent has quit IRC13:19
*** nklenke has quit IRC13:19
*** gouthamr has joined #openstack-nova13:36
*** amoralej is now known as amoralej|lunch13:40
*** pradk has joined #openstack-nova13:44
*** sandanar has quit IRC13:46
*** mdnadeem has joined #openstack-nova13:46
*** pradk has quit IRC13:46
*** lucas-hungry is now known as lucasagomes13:48
*** narayrak has joined #openstack-nova13:49
*** ducttape_ has joined #openstack-nova13:53
*** ociuhandu has joined #openstack-nova13:53
*** nherciu has joined #openstack-nova14:00
*** cdent has joined #openstack-nova14:06
*** amoralej|lunch is now known as amoralej14:07
*** amotoki has joined #openstack-nova14:09
*** baoli has joined #openstack-nova14:15
*** baoli has quit IRC14:17
*** baoli has joined #openstack-nova14:17
*** baoli has quit IRC14:17
*** roeyc has quit IRC14:18
*** baoli has joined #openstack-nova14:18
*** roeyc has joined #openstack-nova14:18
*** gszasz has quit IRC14:22
*** jamesdenton has quit IRC14:24
*** amotoki has quit IRC14:27
*** salv-orl_ has joined #openstack-nova14:31
*** salv-orlando has quit IRC14:34
openstackgerritVladyslav Drok proposed openstack/nova: Do not merge  https://review.openstack.org/41460314:35
*** armax has quit IRC14:42
*** rtheis has quit IRC14:47
*** mdnadeem has quit IRC14:48
*** jamesdenton has joined #openstack-nova14:48
*** ducttape_ has quit IRC14:54
*** jamesdenton has quit IRC14:55
*** jamesdenton has joined #openstack-nova14:55
*** xyang1 has joined #openstack-nova14:55
*** hemanth_ has joined #openstack-nova14:57
*** hemanth_ has left #openstack-nova14:57
*** narayrak has quit IRC15:00
*** jaypipes is now known as leakypipes15:11
leakypipesvdrok: mornin. I see you are investigating the ironic virt driver resources 0 issue..15:12
*** salv-orl_ has quit IRC15:12
vdrokleakypipes: kind of :) both me and vsaienk015:13
vdrokgood morning!15:13
leakypipesvdrok: I think that code in the above patch you commented out is indeed the culprit.15:13
*** zz_dimtruck is now known as dimtruck15:14
*** siva_krish has joined #openstack-nova15:15
vdrokleakypipes: I'm wondering, both None and 0 are not an option now for total resources present?15:16
vsaienk0leakpipes: I've found that on failed builds n-cpu _do_build_and_run_instance stuck on acquiring lock(http://paste.openstack.org/show/593274/), since it is acquired by nova.compute.resource_tracker._update_available_resource so it looks like lock starvation15:16
leakypipesvdrok: correct. there's no such thing as an inventory of 0 :)15:16
*** sandanar_ has quit IRC15:16
leakypipesvdrok: just like there is no such thing as cold. only lack of heat.15:17
vdrokheh, gotcha15:17
leakypipesvsaienk0: I don't think this is due to a locking issue. All of the resource tracker's calls (and thus the calls to the scheduler report client) are protected by a semaphore on the compute node.15:17
vsaienk0leakypipes: I'm not sure why nova.compute.resource_tracker._update_available_resource (Lock "compute_resources") acquire lock too frequently http://logs.openstack.org/46/327046/37/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial/ee529db/logs/screen-n-cpu.txt.gz15:18
*** psachin has quit IRC15:18
leakypipesvdrok: we definitely need to add code into the report client that bails when there's no inventory to record.15:18
vsaienk0leakypipes:  I've tried to bump instance association timeout in Ironic https://review.openstack.org/#/c/413499/ and the reproduce rate seems to be lowered15:19
leakypipesvsaienk0: that lock is there to prevent multiple writers to the same compute node at the same time. I think the instance association timeout is a separate concern.15:20
*** zenoway has quit IRC15:21
vdrokleakypipes: Im not sure about all the interactions there, would it be enough to just check the values and skip the update for those that are 0? or skip everything if any key is 0?15:21
*** mlavalle has joined #openstack-nova15:22
leakypipesvdrok: yeah, I'm going to push a patch to the scheduler report client that does that. gimme about a half hour.15:22
vdrokleakypipes: sure, thanks! :)15:22
leakypipesvdrok: also see jvillal's comment on the bug report pointing to http://logs.openstack.org/26/414426/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial-nv/af7b923/logs/screen-n-cpu.txt.gz?level=WARNING#_2016-12-23_06_21_16_40115:22
*** jamesdenton has quit IRC15:24
*** hongbin has joined #openstack-nova15:24
*** roeyc has quit IRC15:26
*** salv-orlando has joined #openstack-nova15:27
*** siva_krish has quit IRC15:27
*** dimtruck is now known as zz_dimtruck15:27
EmilienMcdent, bauzas: hey! I changed the placement api to use https + 443 port + vhost /placement and it works fine. Do you mind looking at logs to make sure we're good?15:28
EmilienMthe vhost: http://logs.openstack.org/01/406301/6/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/e96e118/logs/apache_config/10-placement_wsgi.conf.txt.gz15:29
EmilienMlogs: http://logs.openstack.org/01/406301/6/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/e96e118/logs/nova/nova-placement-api.txt.gz15:29
cdentEmilienM: looking15:29
*** prateek has quit IRC15:31
*** ijw has joined #openstack-nova15:31
*** salv-orlando has quit IRC15:31
cdentEmilienM: looks right. The errors that are happening in the log are expected until some other changes merge15:33
EmilienMcdent: yeah, no worries for that.15:34
EmilienMcdent: cool so I can remove the 8778 stuff15:34
EmilienMthanks!15:34
*** ijw has quit IRC15:35
*** jamesdenton has joined #openstack-nova15:36
*** zz_dimtruck is now known as dimtruck15:38
*** xinliang has quit IRC15:41
*** xinliang has joined #openstack-nova15:41
*** mnasiadka has quit IRC15:42
*** rmart04 has quit IRC15:44
vdrokleakypipes: mhm, so we can't even get to the point when we do _init_compute_node, until we get to the node that will report something?15:46
leakypipesvdrok: sorry, I'm not following you...15:46
*** ducttape_ has joined #openstack-nova15:49
vdrokleakypipes: nevermind, I'm just not sure that that excpetion by jlvillal is related to this problem? as at the point when self.compute_node.uuid is done, we have not called _init_compute_node in the resource tracker?15:52
*** yamahata has joined #openstack-nova15:52
*** Guest31576 has quit IRC15:52
*** raunak has joined #openstack-nova15:54
leakypipesvdrok: right, understood. I'm not sure *why* that is occurring, though... :) trying to figure that out locally. The pre_start_hook of the nova-compute manager object runs RT.update_available_resource() which should create all ComputeNode objects properly...15:55
*** ducttape_ has quit IRC16:02
*** mjura has quit IRC16:04
*** ducttape_ has joined #openstack-nova16:05
vdrokleakypipes: I might be just missing some knowledge in nova :) but from what I see, RT.update_available_resource does RT._verify_resources before RT._update_available_resource (inside of which RT_init_compute_node), so in RT._verify_resources self.compute_node is still None, seems alright to me16:06
*** pbandark has quit IRC16:06
vdrokand looks like it's almost the same procedure in pre_start_hook, or in a periodic task16:07
*** armax has joined #openstack-nova16:07
leakypipesvdrok: doh, yeah, you're totally correct...16:08
leakypipesvdrok: verify_resources() is called before _update_available_resource() :(16:08
leakypipesnot sure what I was thinking.... in any case, I'm going to rework this and push shortly.16:08
vdrokyah, so just moving that log after _update_available_resource would be all good :)16:08
vdrokthanks!16:08
*** dtp has joined #openstack-nova16:12
*** ssurana has joined #openstack-nova16:17
jlvillalleakypipes, So do you think you know reason for failure? Or just fix for better logging?16:18
leakypipesjlvillal: yeah, I know the reason for the failure.16:18
jlvillalWoo hoo! :)16:18
jlvillalleakypipes, Thanks!16:18
openstackgerritJay Pipes proposed openstack/nova: placement: correct improper test case inheritance  https://review.openstack.org/40636316:26
*** salv-orlando has joined #openstack-nova16:27
*** ducttape_ has quit IRC16:29
*** jamesdenton has quit IRC16:29
openstackgerritJay Pipes proposed openstack/nova: placement: Do not save 0-valued inventory  https://review.openstack.org/41421416:31
*** salv-orlando has quit IRC16:32
*** ociuhandu has quit IRC16:33
*** sahid has quit IRC16:36
*** yamahata has quit IRC16:41
*** raunak has quit IRC16:43
*** ralonsoh has quit IRC16:45
*** unicell has quit IRC16:46
*** raunak has joined #openstack-nova16:47
artomDid http://developer.openstack.org/api-ref/compute/?expanded=create-interface-detail,list-port-interfaces-detail,list-virtual-interfaces-detail#list-port-interfaces get deprecated entirely? I thought not...16:47
*** ssurana has quit IRC16:49
*** tjones1 has joined #openstack-nova16:51
*** tjones has quit IRC16:52
*** dimtruck is now known as zz_dimtruck16:53
*** tjones1 has quit IRC16:53
artomOr do I need to enable something in the config to allow interface attachments? PTO has made me rusty...16:55
*** liverpooler has joined #openstack-nova16:56
*** krypto has joined #openstack-nova17:03
*** zz_dimtruck is now known as dimtruck17:04
kryptohello all i have seen this issue before in Juno but didnot care much. : Compute node is powered off but in nova list vm state is still up,hypervisor is marked down but still vm on that node is showing as up.Any idea about this behaviour17:05
*** vtech has joined #openstack-nova17:09
*** tjones has joined #openstack-nova17:09
*** ociuhandu has joined #openstack-nova17:12
*** raunak has quit IRC17:13
*** cdent has quit IRC17:13
*** nic has joined #openstack-nova17:14
*** smatzek has quit IRC17:14
*** pcaruana has quit IRC17:14
*** smatzek has joined #openstack-nova17:15
*** raunak has joined #openstack-nova17:17
*** ijw has joined #openstack-nova17:19
*** Zer0Byte__ has joined #openstack-nova17:20
*** lucasagomes is now known as lucas-pto17:22
openstackgerritArtom Lifshitz proposed openstack/nova: Tagged attach compute manager and net API support  https://review.openstack.org/39194117:24
openstackgerritArtom Lifshitz proposed openstack/nova: Tagged attach API support  https://review.openstack.org/37602117:24
*** ijw has quit IRC17:24
*** slaweq has joined #openstack-nova17:24
jlvillalleakypipes, pep8 doesn't like you :(17:25
jlvillalleakypipes, http://logs.openstack.org/14/414214/2/check/gate-nova-pep8-ubuntu-xenial/2c95daa/17:26
*** ducttape_ has joined #openstack-nova17:30
openstackgerritJohn L. Villalovos proposed openstack/nova: placement: Do not save 0-valued inventory  https://review.openstack.org/41421417:30
jlvillalleakypipes, ^^^ I added a blank line to fix the pep8 error17:31
*** amoralej is now known as amoralej|pto17:31
*** ducttape_ has quit IRC17:35
*** ijw has joined #openstack-nova17:38
*** ssurana has joined #openstack-nova17:39
*** ftersin has quit IRC17:40
*** liverpooler has quit IRC17:43
*** slaweq has quit IRC17:45
openstackgerritint32bit proposed openstack/python-novaclient: Add batch add/remove server tags support  https://review.openstack.org/41463517:47
leakypipesjlvillal: ty sir! :)17:50
* leakypipes was taking the girls to the groomers..17:50
*** ijw has quit IRC17:51
*** ijw has joined #openstack-nova17:52
jlvillalleakypipes, Glad to help :)17:52
openstackgerritint32bit proposed openstack/python-novaclient: Add batch add/remove server tags support  https://review.openstack.org/41463517:52
*** ijw_ has joined #openstack-nova17:53
*** ijw has quit IRC17:56
*** gouthamr has quit IRC18:18
*** yamahata has joined #openstack-nova18:20
*** vtech has quit IRC18:24
openstackgerritJay Pipes proposed openstack/nova: placement: create aggregate map in report client  https://review.openstack.org/40730918:25
*** Zer0Byte__ has quit IRC18:27
*** Zer0Byte__ has joined #openstack-nova18:28
*** raunak has quit IRC18:28
*** esberglu has quit IRC18:38
*** esberglu has joined #openstack-nova18:39
*** gouthamr has joined #openstack-nova18:42
*** esberglu has quit IRC18:43
*** mdnadeem has joined #openstack-nova18:46
*** adreznec has quit IRC18:51
*** ducttape_ has joined #openstack-nova18:53
*** mdnadeem has quit IRC18:55
*** adreznec has joined #openstack-nova18:57
jlvillalleakypipes, FYI: One job failed for: https://review.openstack.org/#/c/414214/19:06
*** gouthamr has quit IRC19:06
*** siva_krish has joined #openstack-nova19:07
leakypipesjlvillal: yeah, unrelated... timeout error. will recheck.19:09
*** rfolco has quit IRC19:09
jlvillalleakypipes, Thanks19:09
*** portdirect_ has joined #openstack-nova19:09
*** portdirect_ has quit IRC19:13
lbeliveauleakypipes: do you know why https://review.openstack.org/#/c/405575/ is failing ?  Looks like there are failing db tests.19:15
leakypipeslbeliveau: yes. just fixing that up locally.19:16
leakypipeslbeliveau: it's the result of patches that merged yesterday.19:16
leakypipeslbeliveau: that changed the RT's public call interfaces.19:16
lbeliveauleakypipes: cool, thanks :)19:17
*** portdirect_ has joined #openstack-nova19:19
*** dtp has quit IRC19:23
*** ducttape_ has quit IRC19:26
*** gouthamr has joined #openstack-nova19:26
*** MVenesio has joined #openstack-nova19:26
*** unicell1 has joined #openstack-nova19:34
*** krypto has quit IRC19:46
*** salv-orlando has joined #openstack-nova19:47
*** dimtruck is now known as zz_dimtruck19:52
*** adisky_ has quit IRC19:59
*** ociuhandu has quit IRC20:03
*** ssurana has quit IRC20:09
*** nicolasbock has quit IRC20:10
*** ssurana has joined #openstack-nova20:10
jlvillalleakypipes, FYI: I think it is going to fail the ceph job again. I'm watching the telnet log and see tempest failures20:12
jlvillalleakypipes, From the previous run I see: http://logs.openstack.org/14/414214/3/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/70b537a/logs/screen-n-cpu.txt.gz?level=WARNING20:12
jlvillalleakypipes, Any chance related to this patch?20:13
*** MVenesio has quit IRC20:18
*** rcernin has joined #openstack-nova20:25
*** ducttape_ has joined #openstack-nova20:26
*** ijw has joined #openstack-nova20:26
*** ijw_ has quit IRC20:27
*** smatzek has quit IRC20:30
openstackgerritJohn L. Villalovos proposed openstack/nova: WIP/DNM: Testing  https://review.openstack.org/41464620:30
*** salv-orl_ has joined #openstack-nova20:31
*** ducttape_ has quit IRC20:31
*** rcernin has quit IRC20:34
*** salv-orlando has quit IRC20:34
*** ociuhandu has joined #openstack-nova20:43
*** rnoriega has quit IRC20:44
*** vtech has joined #openstack-nova20:45
*** rnoriega has joined #openstack-nova20:50
*** ducttape_ has joined #openstack-nova20:51
*** slaweq has joined #openstack-nova20:52
*** smatzek has joined #openstack-nova20:55
*** slaweq has quit IRC20:56
*** nherciu has quit IRC20:56
*** slaweq has joined #openstack-nova20:57
*** baoli has quit IRC20:58
*** vtech has quit IRC21:21
*** zz_dimtruck is now known as dimtruck21:21
*** jheroux has quit IRC21:24
*** smatzek has quit IRC21:28
*** xyang1 has quit IRC21:28
*** Jeffrey4l has quit IRC21:35
*** lpetrut has joined #openstack-nova21:36
*** dimtruck is now known as zz_dimtruck21:40
*** ijw_ has joined #openstack-nova21:41
*** ijw has quit IRC21:44
*** zz_dimtruck is now known as dimtruck21:47
*** Jeffrey4l has joined #openstack-nova21:48
*** slaweq has quit IRC21:50
*** gouthamr has quit IRC21:51
*** tlian has quit IRC22:09
*** dimtruck is now known as zz_dimtruck22:14
openstackgerritJohn L. Villalovos proposed openstack/nova: WIP/DNM: Testing  https://review.openstack.org/41464622:20
*** ducttape_ has quit IRC22:27
*** lpetrut has quit IRC22:31
*** nic has quit IRC22:33
*** tjones has quit IRC22:54
*** ociuhandu has quit IRC22:56
*** armax has quit IRC22:57
*** rwmjones|hols|af is now known as rwmjones22:57
*** rwmjones is now known as rwmjones|hols22:57
*** ociuhandu has joined #openstack-nova22:58
*** ijw_ has quit IRC23:10
*** fragatina has quit IRC23:15
*** ssurana has quit IRC23:17
*** siva_krish has quit IRC23:22
*** ducttape_ has joined #openstack-nova23:24
*** ducttape_ has quit IRC23:28
*** mvk has quit IRC23:35
*** salv-orl_ has quit IRC23:37
jlvillalNot sure if anyone around: Ironic gate blocked. Need this patch to fix: https://review.openstack.org/#/c/414214/23:37
*** dave-mccowan has joined #openstack-nova23:38
*** dave-mccowan has quit IRC23:46
sfinucanjlvillal: I am (barely), but I don't think anyone else is :(23:48
*** Zer0Byte__ has quit IRC23:48
*** dave-mccowan has joined #openstack-nova23:50
*** dave-mccowan has quit IRC23:51

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