Wednesday, 2015-04-29

*** ijw has quit IRC00:00
*** melwitt has quit IRC00:03
*** otter768 has joined #openstack-nova00:10
*** zzzeek has quit IRC00:11
*** otter768 has quit IRC00:14
*** claudiub has quit IRC00:15
*** iamjarvo has joined #openstack-nova00:16
*** salv-orlando has quit IRC00:16
*** tsekiyama has quit IRC00:18
*** mtanino_ has quit IRC00:21
*** ijw has joined #openstack-nova00:22
*** achanda has joined #openstack-nova00:23
*** iamjarvo has quit IRC00:24
*** armax has joined #openstack-nova00:29
*** jwcroppe_ has quit IRC00:30
*** jwcroppe has joined #openstack-nova00:30
*** david-lyle has quit IRC00:32
*** browne has quit IRC00:33
*** jwcroppe has quit IRC00:35
*** ijw has quit IRC00:35
*** yingjun has joined #openstack-nova00:39
*** yingjun_ has joined #openstack-nova00:40
*** IanGovett has quit IRC00:40
*** promulo_ has joined #openstack-nova00:42
*** Marga_ has quit IRC00:42
openstackgerritShilla Saebi proposed openstack/nova: minor change to aggregates.rst  https://review.openstack.org/17846200:43
openstackgerritEli Qiao proposed openstack/nova: API/Object: Remove _get_flavor in simple_tenant_usage api  https://review.openstack.org/17805600:43
*** yingjun has quit IRC00:44
*** promulo__ has quit IRC00:46
openstackgerritShilla Saebi proposed openstack/nova: minor edit to policy_enforcement.rst  https://review.openstack.org/17846300:49
*** asettle has joined #openstack-nova00:49
*** Nic has quit IRC00:50
*** asettle has quit IRC00:54
*** asettle has joined #openstack-nova00:56
*** asettle has quit IRC00:56
*** signed8bit has joined #openstack-nova00:58
*** asettle has joined #openstack-nova00:59
*** tjones1 has joined #openstack-nova01:01
openstackgerritLianhao Lu proposed openstack/nova: Convert bandwidth_usage related timestamp to UTC native datetime  https://review.openstack.org/17321301:03
*** xyang1 has quit IRC01:05
*** emagana has joined #openstack-nova01:06
*** Marga_ has joined #openstack-nova01:09
openstackgerritEli Qiao proposed openstack/nova-specs: Improve lock_server api return code  https://review.openstack.org/16327501:10
*** emagana has quit IRC01:10
*** emagana has joined #openstack-nova01:11
*** ijw has joined #openstack-nova01:11
*** yingjun_ has quit IRC01:13
*** ijw has quit IRC01:16
*** salv-orlando has joined #openstack-nova01:17
*** david-lyle has joined #openstack-nova01:20
*** tjones1 has quit IRC01:21
*** zhangtralon has joined #openstack-nova01:22
*** salv-orlando has quit IRC01:22
*** dsanders has quit IRC01:23
*** popw has joined #openstack-nova01:24
*** joefides has joined #openstack-nova01:24
*** joefides_ has quit IRC01:27
openstackgerritEli Qiao proposed openstack/nova: API: remove admin require from compute_node_get_all_by_* from db layer  https://review.openstack.org/17533401:31
openstackgerritEli Qiao proposed openstack/nova: API: Add policy enforcement test cases for pci API  https://review.openstack.org/17574501:31
openstackgerritEli Qiao proposed openstack/nova: API: remove instance_get_all_by_host hard-code admin check from db  https://review.openstack.org/17767301:31
openstackgerritEli Qiao proposed openstack/nova: API: remove admin require for compute_node(get_all/search_by_hyperviso) from db  https://review.openstack.org/17574401:31
openstackgerritEli Qiao proposed openstack/nova: API: remove admin require for compute_node_create/update/delete from db layer  https://review.openstack.org/17569301:31
*** Jeffrey4l has joined #openstack-nova01:36
*** markvoelker has quit IRC01:36
*** marun has quit IRC01:38
*** zhenguo has joined #openstack-nova01:38
*** annashen has joined #openstack-nova01:39
*** otter768 has joined #openstack-nova01:41
openstackgerritEli Qiao proposed openstack/nova: API: remove instance_get_all_by_host hard-code admin check from db  https://review.openstack.org/17767301:42
*** annashen has quit IRC01:42
openstackgerritAlex Xu proposed openstack/nova: API: remove instance_get_all_by_host hard-code admin check from db  https://review.openstack.org/17767301:46
openstackgerritAlex Xu proposed openstack/nova: API: remove instance_get_all_by_host hard-code admin check from db  https://review.openstack.org/17767301:47
openstackgerritAlex Xu proposed openstack/nova: API: remove instance_get_all_by_host hard-code admin check from db  https://review.openstack.org/17767301:47
*** yamahata has quit IRC01:48
*** signed8bit has quit IRC01:48
*** mwagner_lap has quit IRC01:49
*** baoli has joined #openstack-nova01:49
*** erkules_ has joined #openstack-nova01:50
*** erkules has quit IRC01:50
openstackgerritheijlong proposed openstack/nova: Merge servers-ips functional tests of v2 and v2.1  https://review.openstack.org/17423701:51
*** vilobhmm1 has quit IRC01:51
*** dims_ is now known as dims01:52
*** baoli has quit IRC01:53
*** vilobhmm1 has joined #openstack-nova01:53
*** baoli has joined #openstack-nova01:53
*** xuhanp has joined #openstack-nova01:53
*** xuhanp_ has joined #openstack-nova01:53
*** vilobhmm1 has quit IRC01:53
*** vilobhmm1 has joined #openstack-nova01:54
*** bnemec has quit IRC01:56
*** asettle has quit IRC01:57
openstackgerritmelanie witt proposed openstack/nova: WIP: send object to cells instance_update_at_top  https://review.openstack.org/17651801:57
*** bnemec has joined #openstack-nova01:59
*** browne has joined #openstack-nova02:00
*** harlowja is now known as harlowja_away02:03
*** vilobhmm1 has quit IRC02:04
*** mwagner_lap has joined #openstack-nova02:06
*** VW_ has joined #openstack-nova02:07
*** baoli has quit IRC02:11
*** baoli has joined #openstack-nova02:12
*** tjones1 has joined #openstack-nova02:20
*** tjones1 has quit IRC02:21
*** tjones1 has joined #openstack-nova02:21
*** tjones1 has left #openstack-nova02:22
*** unicell1 has quit IRC02:24
*** angdraug has quit IRC02:25
*** yingjun has joined #openstack-nova02:27
*** dave-mccowan has joined #openstack-nova02:28
*** david-lyle has quit IRC02:30
*** david-lyle has joined #openstack-nova02:31
*** patrickeast has quit IRC02:32
*** jwcroppe has joined #openstack-nova02:32
*** david-lyle has quit IRC02:36
*** tralon_ has joined #openstack-nova02:37
*** emagana has quit IRC02:40
*** markvoelker has joined #openstack-nova02:41
*** jwcroppe has quit IRC02:45
*** asettle has joined #openstack-nova02:51
*** ildikov has quit IRC02:51
*** gyee has quit IRC02:53
*** VW_ has quit IRC02:53
*** VW_ has joined #openstack-nova02:54
*** yingjun has quit IRC02:59
*** ildikov has joined #openstack-nova03:04
*** yingjun has joined #openstack-nova03:08
*** dave-mccowan has quit IRC03:12
*** salv-orlando has joined #openstack-nova03:14
*** salv-orlando has quit IRC03:25
*** tojuvone has joined #openstack-nova03:26
*** coolsvap|afk is now known as coolsvap03:27
*** yamahata has joined #openstack-nova03:27
*** david-lyle has joined #openstack-nova03:32
*** changbl has joined #openstack-nova03:33
*** baoli has quit IRC03:34
*** jwcroppe has joined #openstack-nova03:42
*** sudipto has joined #openstack-nova03:43
*** tralon_ has quit IRC03:54
*** yingjun has quit IRC04:01
*** sdake_ has joined #openstack-nova04:04
*** vilobhmm1 has joined #openstack-nova04:08
*** sdake has quit IRC04:08
*** armax has quit IRC04:09
*** sdake_ has quit IRC04:11
*** kiran-r has joined #openstack-nova04:13
*** VW_ has quit IRC04:18
*** rushiagr_away is now known as rushiagr04:21
*** garyk has quit IRC04:21
*** kiran-r has quit IRC04:22
*** jyuso1 has quit IRC04:23
*** krak has joined #openstack-nova04:28
krakHey Guys, seeing some errors. Folks from heat list have suggested its a bug between neutron and nova.04:29
krakcould someone please have a look at the logs and help me with that to confirm it.04:29
*** asettle has quit IRC04:30
krakIt usually occurs when deleting a vm. the server ends up in error state.04:30
krakI have the logs. kindly let me know if someone can have a look.04:30
*** asettle has joined #openstack-nova04:34
*** mwagner_lap has quit IRC04:36
*** ildikov has quit IRC04:44
*** rushiagr is now known as rushiagr_away04:45
*** zhangtralon has quit IRC04:46
*** asettle has quit IRC04:47
*** mwagner_lap has joined #openstack-nova04:52
*** unicell has joined #openstack-nova04:53
*** ildikov has joined #openstack-nova04:57
*** nkrinner has joined #openstack-nova04:57
*** unicell1 has joined #openstack-nova05:00
*** gtt116_ has joined #openstack-nova05:00
*** unicell has quit IRC05:01
*** gtt116 has quit IRC05:04
*** sudipto has quit IRC05:04
*** krak has quit IRC05:08
openstackgerritAlex Xu proposed openstack/nova: Add version bump guideline in microversion devref  https://review.openstack.org/17777805:08
*** Nikolay_St has quit IRC05:09
*** yingjun has joined #openstack-nova05:14
*** yingjun has quit IRC05:14
*** salv-orlando has joined #openstack-nova05:15
*** xuhanp_ has quit IRC05:16
*** xuhanp has quit IRC05:16
*** asettle has joined #openstack-nova05:19
*** asettle has quit IRC05:20
openstackgerritChris Friesen proposed openstack/nova: fix network setup on evacuate  https://review.openstack.org/16982705:21
*** irenab has joined #openstack-nova05:21
*** salv-orlando has quit IRC05:26
*** ildikov has quit IRC05:28
*** vilobhmm1 has quit IRC05:28
*** atuvenie has joined #openstack-nova05:30
*** Marga_ has quit IRC05:34
*** wanghao has quit IRC05:35
*** kiran-r has joined #openstack-nova05:35
*** ajayaa has joined #openstack-nova05:38
*** cfriesen has quit IRC05:40
*** josecastroleon has joined #openstack-nova05:40
*** jichen has joined #openstack-nova05:41
*** park_heijlong has joined #openstack-nova05:45
*** annashen has joined #openstack-nova05:45
*** mwagner_lap has quit IRC05:46
*** jecarey has quit IRC05:46
*** Murali has joined #openstack-nova05:49
openstackgerritMh Raies proposed openstack/nova: Enable tenant/user specific instance filtering  https://review.openstack.org/13902505:52
*** atuvenie has quit IRC05:53
*** yingjun has joined #openstack-nova05:55
*** annashen has quit IRC05:57
*** atuvenie has joined #openstack-nova05:59
*** Hosam has joined #openstack-nova06:00
*** doron_afk has joined #openstack-nova06:00
*** dulek has joined #openstack-nova06:02
*** romainh has joined #openstack-nova06:02
*** Longgeek has joined #openstack-nova06:03
*** mwagner_lap has joined #openstack-nova06:03
*** kiwik-chenrui has quit IRC06:03
*** kiwik-chenrui has joined #openstack-nova06:03
*** vilobhmm1 has joined #openstack-nova06:03
*** otter768 has quit IRC06:04
*** sudipto has joined #openstack-nova06:06
*** igordcard_ has quit IRC06:06
*** doude has quit IRC06:07
*** Longgeek has quit IRC06:08
*** exploreshaifali has joined #openstack-nova06:09
*** pcaruana has joined #openstack-nova06:09
*** park_heijlong has quit IRC06:11
*** leseb has quit IRC06:12
*** jlvillal has quit IRC06:14
*** igordcard_ has joined #openstack-nova06:14
*** jlvillal has joined #openstack-nova06:16
*** eliqiao has quit IRC06:16
*** zul has joined #openstack-nova06:17
*** leseb has joined #openstack-nova06:18
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex  https://review.openstack.org/17805506:18
*** kiwik-chenrui has quit IRC06:19
*** kiwik-chenrui has joined #openstack-nova06:19
*** pkoniszewski has joined #openstack-nova06:19
*** jyuso1 has joined #openstack-nova06:22
*** zul has quit IRC06:24
*** zul has joined #openstack-nova06:25
*** zhangtralon has joined #openstack-nova06:25
*** xuhanp has joined #openstack-nova06:25
*** xuhanp_ has joined #openstack-nova06:25
*** salv-orlando has joined #openstack-nova06:34
*** Kevin_Zheng has joined #openstack-nova06:37
*** ildikov has joined #openstack-nova06:40
*** salv-orlando has quit IRC06:42
*** sudipto has quit IRC06:45
*** dulek has quit IRC06:46
*** vilobhmm1 has quit IRC06:46
*** vilobhmm1 has joined #openstack-nova06:47
*** vilobhmm1 has quit IRC06:48
openstackgerritSergey Nikitin proposed openstack/nova: Added method is_exist to the Tag object  https://review.openstack.org/17810306:48
*** jcoufal has joined #openstack-nova06:49
*** sudipto has joined #openstack-nova06:50
*** garyk has joined #openstack-nova06:50
*** dulek has joined #openstack-nova06:52
*** Longgeek has joined #openstack-nova06:56
*** garyk1 has joined #openstack-nova07:00
*** tdurakov has quit IRC07:00
*** ajo has quit IRC07:01
*** Longgeek has quit IRC07:02
*** garyk has quit IRC07:02
*** tdurakov has joined #openstack-nova07:03
*** Longgeek has joined #openstack-nova07:06
*** sahid has joined #openstack-nova07:06
*** browne has quit IRC07:09
*** exploreshaifali has quit IRC07:09
openstackgerritTimofey Durakov proposed openstack/nova: Changed logic in _compare_result  api_samples_test_base  https://review.openstack.org/17819107:12
openstackgerritTimofey Durakov proposed openstack/nova: Removed 'PYTHONHASHSEED=0' from tox.ini  https://review.openstack.org/16387707:12
*** pask81 has joined #openstack-nova07:15
*** mpavone has joined #openstack-nova07:20
*** xuhanp_ has quit IRC07:21
*** xuhanp has quit IRC07:21
openstackgerritTimofey Durakov proposed openstack/nova: Changed logic in _compare_result  api_samples_test_base  https://review.openstack.org/17819107:22
openstackgerritTimofey Durakov proposed openstack/nova: Removed 'PYTHONHASHSEED=0' from tox.ini  https://review.openstack.org/16387707:22
*** garyk has joined #openstack-nova07:23
*** bfic has joined #openstack-nova07:25
*** garyk1 has quit IRC07:26
*** doron_afk is now known as doron07:26
*** mwagner_lap has quit IRC07:28
openstackgerritBartosz Fic proposed openstack/nova: Live migration failure in API doesn't set vm_state to Error  https://review.openstack.org/16891607:30
*** zul has quit IRC07:32
*** zul has joined #openstack-nova07:32
*** exploreshaifali has joined #openstack-nova07:38
*** kaisers has quit IRC07:39
openstackgerritTimofey Durakov proposed openstack/nova: Changed logic in _compare_result  api_samples_test_base  https://review.openstack.org/17819107:39
openstackgerritTimofey Durakov proposed openstack/nova: Removed 'PYTHONHASHSEED=0' from tox.ini  https://review.openstack.org/16387707:39
*** kaisers has joined #openstack-nova07:41
*** eliqiao has joined #openstack-nova07:42
*** jlanoux has joined #openstack-nova07:43
*** ildikov has quit IRC07:43
*** moshele has joined #openstack-nova07:44
*** jwcroppe has quit IRC07:44
*** ildikov has joined #openstack-nova07:44
*** dsanders has joined #openstack-nova07:45
*** jwcroppe has joined #openstack-nova07:45
*** kaisers has quit IRC07:45
*** mwagner_lap has joined #openstack-nova07:46
tdurakovgilliard: hi! thanks for comment^)07:46
*** doron is now known as doron_afk07:46
gilliardhello tdurakov :)07:47
*** kaisers has joined #openstack-nova07:47
*** achanda has quit IRC07:48
gilliardYou're welcome. Is that the last of those patches now?07:48
*** Nikolay_St has joined #openstack-nova07:49
*** jwcroppe has quit IRC07:49
*** ajo has joined #openstack-nova07:51
*** exploreshaifali has quit IRC07:51
*** achanda has joined #openstack-nova07:56
*** jeblair has quit IRC07:56
*** jeblair has joined #openstack-nova07:56
*** jistr has joined #openstack-nova07:58
openstackgerritYongli He proposed openstack/nova: resource tracker style pci resource management  https://review.openstack.org/13132108:00
*** derekh has joined #openstack-nova08:01
*** otter768 has joined #openstack-nova08:05
*** Longgeek has quit IRC08:05
*** salv-orlando has joined #openstack-nova08:08
*** zul has quit IRC08:09
*** otter768 has quit IRC08:10
*** dims has quit IRC08:10
openstackgerritTimofey Durakov proposed openstack/nova: Changed logic in _compare_result  api_samples_test_base  https://review.openstack.org/17819108:11
openstackgerritTimofey Durakov proposed openstack/nova: Removed 'PYTHONHASHSEED=0' from tox.ini  https://review.openstack.org/16387708:11
*** ndipanov has quit IRC08:12
*** ndipanov has joined #openstack-nova08:12
*** dsanders has quit IRC08:12
*** lucasagomes has joined #openstack-nova08:14
openstackgerritHans Lindgren proposed openstack/nova: Remove unused db.aggregate_metadata_get_by_metadata_key() call  https://review.openstack.org/15905308:15
openstackgerritAlex Xu proposed openstack/nova: Add the rule of separate plugin for Nova REST API in devref  https://review.openstack.org/16291308:18
*** yassine_ has joined #openstack-nova08:18
*** eglynn has joined #openstack-nova08:24
openstackgerritMarian Horban proposed openstack/nova: Disassociating of floating IPs corrected  https://review.openstack.org/14935808:24
*** zul has joined #openstack-nova08:25
*** yamahata has quit IRC08:25
*** zhangtralon has quit IRC08:27
openstackgerritNoel Nelson Dsouza proposed openstack/nova: Nova can stop instance when Fake/Actual compute driver used  https://review.openstack.org/17474608:29
*** achanda has quit IRC08:33
*** jeblair has quit IRC08:33
*** achanda has joined #openstack-nova08:33
*** achanda has quit IRC08:35
*** gszasz has joined #openstack-nova08:35
*** zz_johnthetubagu is now known as johnthetubaguy08:36
bauzasjohnthetubaguy: morning (hassling you quite early eh)08:40
bauzasjohnthetubaguy: I have some points to discuss with you about requestspec objecft08:40
bauzasjohnthetubaguy: first, it would be cool if you could just quickly review a spec modification https://review.openstack.org/#/c/176083/08:41
bauzasjohnthetubaguy: just because it's how it will be needed to be08:41
bauzasjohnthetubaguy: and that prevents me further work08:41
bauzasjohnthetubaguy: the second point is on the same spec, I should mention that I'll make use of an ImageMeta object08:42
*** yingjun has quit IRC08:42
bauzasjohnthetubaguy: https://review.openstack.org/#/c/76234/25 having a procedural -2, I think it would be ideal if you could remove that one just because my spec will need it (provided we merge the spec change), or just because it's part of the global objects effort08:43
*** yingjun has joined #openstack-nova08:43
*** johnthetubaguy is now known as zz_johnthetubagu08:44
*** zz_johnthetubagu is now known as johnthetubaguy08:45
*** jwcroppe has joined #openstack-nova08:46
*** ferest has joined #openstack-nova08:49
*** jeblair has joined #openstack-nova08:51
johnthetubaguybauzas: sorry, I was fighting IRC there, looking at your requests08:53
johnthetubaguybauzas: can't remove the -2 until the spec is merged and blueprint is approved08:54
*** bfic has quit IRC08:54
eliqiaohi johnthetubaguy: can you kindly help review this spec https://review.openstack.org/#/c/177671/ , it's about console-auth service HA support.08:54
*** ferest has quit IRC08:54
*** mwagner_lap has quit IRC08:55
bauzasjohnthetubaguy: makes sense08:55
johnthetubaguyeliqiao: its on my list, but we are about a week behind or so at the moment, hope to get to yours very soon08:55
openstackgerritSylvain Bauza proposed openstack/nova-specs: Amend RequestSpec blueprint by using an ImageMeta object  https://review.openstack.org/17856308:55
bauzasjohnthetubaguy: hence what's just coming in ^08:55
johnthetubaguybauzas: I usually remove the -2 when I approve the blueprint, assuming I understand the blueprint dashboard at the time I approve the blueprint08:56
eliqiao johnthetubaguy: thanks in advance :)08:56
*** doron_afk is now known as doron08:56
bauzasjohnthetubaguy: tbh it was overkill to mention a separate BP for https://review.openstack.org/#/c/76234/08:56
bauzasjohnthetubaguy: but I understand your point which is valid08:57
bauzasjohnthetubaguy: so I or dan will update the change by changing the commit msg to point to the RequestSpec bp08:57
*** annashen has joined #openstack-nova08:57
johnthetubaguybauzas: sorry, I am confused now08:58
bauzasjohnthetubaguy: okay, rewinding :)08:58
johnthetubaguybauzas: what blueprints do you want to track the work? which are approved right now?08:58
bauzasjohnthetubaguy: so, danpb worked on https://review.openstack.org/#/c/76234/ on his side separately08:58
bauzasjohnthetubaguy: because he wanted that for the virt drivers08:58
bauzasjohnthetubaguy: on my side, I'm working on implementing http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/request-spec-object.html08:59
bauzasjohnthetubaguy: but I discovered some last minute issues that need to amend the spec08:59
johnthetubaguybauzas: OK08:59
bauzasjohnthetubaguy: the first one is that we need to explode the Instance object into smaller fields, ie. https://review.openstack.org/#/c/176083/09:00
johnthetubaguybauzas: we need to add that blueprint into liberty, and get it approved, not sure it needs a spec: https://blueprints.launchpad.net/nova/+spec/convert-image-meta-into-nova-object09:00
bauzasjohnthetubaguy: the second one is that we can't accept a DictofStrings for the image info, hence https://review.openstack.org/17856309:00
bauzasso, https://review.openstack.org/#/c/176083/ and https://review.openstack.org/178563 are just there for making sure the spec is matching what will be implemented09:01
bauzasjohnthetubaguy: then, a new patchset for https://review.openstack.org/#/c/76234/ will be the implementation of https://review.openstack.org/178563 so I will change the commit msg to refer to request-spec-object instead of that convert-image-meta-into-nova-object BP09:02
*** annashen has quit IRC09:02
bauzasjohnthetubaguy: that would allow to consider https://review.openstack.org/#/c/76234/ as implementation of https://review.openstack.org/178563 which would be approved, so the -2 would no longer be needed09:02
*** gtt116 has joined #openstack-nova09:03
*** popw has quit IRC09:03
*** popw has joined #openstack-nova09:03
*** dulek_ has joined #openstack-nova09:03
*** dulek has quit IRC09:03
*** bfic has joined #openstack-nova09:04
*** bfic has quit IRC09:04
johnthetubaguybauzas: we just need the patch to point at an approved blueprint, added it to the meeting agenda to get approved09:05
johnthetubaguybauzas: changing the patch to point to an approved blueprint is also fine09:05
bauzasjohnthetubaguy: agreed09:05
*** gtt116_ has quit IRC09:05
*** Longgeek has joined #openstack-nova09:06
*** apuimedo has quit IRC09:09
openstackgerritNoel Nelson Dsouza proposed openstack/nova: Nova can stop instance when Fake/Actual compute driver used  https://review.openstack.org/17474609:10
*** doron is now known as doron_afk09:10
*** mwagner_lap has joined #openstack-nova09:11
*** Longgeek has quit IRC09:12
*** Longgeek has joined #openstack-nova09:13
*** yingjun has quit IRC09:13
*** yingjun_ has joined #openstack-nova09:13
*** jwcroppe has quit IRC09:14
*** jyuso1 has quit IRC09:14
*** jyuso1 has joined #openstack-nova09:14
*** zhangtralon has joined #openstack-nova09:17
*** jyuso1 has quit IRC09:17
*** zul has quit IRC09:27
openstackgerritSergey Nikitin proposed openstack/nova: Added method exists to the Tag object  https://review.openstack.org/17810309:30
*** josecastroleon has quit IRC09:33
*** yingjun_ has quit IRC09:35
*** sudipto has quit IRC09:36
*** sorrison has quit IRC09:38
openstackgerritIldiko Vancsa proposed openstack/nova: Add functional test for server group  https://review.openstack.org/14751409:40
*** aswadr has joined #openstack-nova09:44
*** sorrison has joined #openstack-nova09:46
johnthetubaguyalex_xu: hey, I attempted to update that spec for the API relax validation: https://review.openstack.org/#/c/173243/09:46
alex_xujohnthetubaguy: hi, what you want to update?09:47
*** atuvenie has quit IRC09:48
*** pixelb has joined #openstack-nova09:49
alex_xujohnthetubaguy: oops, I guess you mean you 'updated' the spec09:49
*** oro has joined #openstack-nova09:49
alex_xujohnthetubaguy: will review the spec today09:49
*** markmcclain has quit IRC09:49
*** zul has joined #openstack-nova09:51
*** dims has joined #openstack-nova09:51
*** sudipto has joined #openstack-nova09:52
*** redbeard has joined #openstack-nova09:52
*** yassine_ has quit IRC09:54
*** ajayaa has quit IRC09:57
*** Hosam has quit IRC09:57
*** markmcclain has joined #openstack-nova09:59
*** IanGovett has joined #openstack-nova10:00
*** bfic has joined #openstack-nova10:02
johnthetubaguyalex_xu: thank you :)10:02
*** jeblair has quit IRC10:03
alex_xujohnthetubaguy: np :)10:03
*** jeblair has joined #openstack-nova10:03
*** dulek_ has quit IRC10:06
*** otter768 has joined #openstack-nova10:06
*** mtreinish has quit IRC10:07
openstackgerritJohn Garbutt proposed openstack/nova: devref: update the nova architecture doc  https://review.openstack.org/17737410:07
sudiptojohnthetubaguy, quick question....Would you be able to tell me, why the scheduled_at field is not longer being updated in the nova instances table?10:08
*** doude has joined #openstack-nova10:09
*** ajayaa has joined #openstack-nova10:10
*** otter768 has quit IRC10:10
kashyapAnyone else see the "check-grenade-dsvm" job failing? -- http://logs.openstack.org/61/178061/1/check/check-grenade-dsvm/92458a9/console.html10:11
bauzassudipto: that reminds me a good explanation10:11
sudiptobauzas, ok :)10:11
bauzassudipto: sec, finding the link10:11
sudiptobauzas, it looks like this https://review.openstack.org/#/c/143725/2 removed it?10:12
johnthetubaguysudipto: bauzas: I didn't know that was the case, if I am honest10:12
sudiptobauzas, and if did remove the updating of the field - then having it defined in the models.py probably means we need to clean it up?10:12
bauzassudipto: johnthetubaguy: http://lists.openstack.org/pipermail/openstack-dev/2015-March/059603.html10:13
bauzassudipto: since the scheduler is no longer always involved in migration tasks, the scheduled_at field is unrelevant10:14
johnthetubaguybauzas: it just looks like a bug in the build_and_run_instance method we added, if I am being honest10:14
*** ociuhandu has joined #openstack-nova10:15
bauzasjohnthetubaguy: I think it's confusing to fix that10:15
johnthetubaguybauzas: granted we don't want the scheduler writing that in the DB, just not sure why the conductor does't do that10:15
bauzasjohnthetubaguy: since a nova evacuate vm host can just bypass the scheduler10:15
sudiptobauzas, johnthetubaguy the launched_at field is something that gains more significance in this case?10:15
johnthetubaguybauzas: I didn't think it worked for that either10:16
bauzasjohnthetubaguy: yeah, so the real metric is launched_at IMHO10:16
johnthetubaguybauzas: we need migration records for evacuate really, thats how you should track VMs moving around, be we haven't got those for everything yet10:16
bauzasjohnthetubaguy: because it's now decoupled between scheduling and booting10:16
bauzasjohnthetubaguy: as formerly, the scheduler was proxying the boot call10:16
johnthetubaguybauzas: thats an excuse not a reason, but anyways, its mostly a useless statistic10:17
bauzasjohnthetubaguy: launched_at is updated when migratingh10:17
bauzasjohnthetubaguy: as I said, what comes if a VM is scheduled yesterday to an host and then evacuated to a second host by specifying its destination ?10:18
jichenhi, do anyone know if cherrypick to old branch , reported cherry pick error,  what kind of actions should I take?10:18
bauzasjohnthetubaguy: then the current_time - scheduled_at would makes no sense10:18
bauzasjichen: I don't get it ? you have conflicts N10:18
bauzas?10:19
bauzasjichen: if so, git mergetool and git cherry-pick --continue10:19
*** doude has quit IRC10:19
*** zul has quit IRC10:19
jichenbauzas: ok, so I need to get the code locally and do it ?10:19
sudiptobauzas, johnthetubaguy Ok - from a stats per say - how would one be able to determine the uptime of a VM on a given host?10:19
johnthetubaguysudipto: right now you can't really, we should be able to use the migration records10:20
bauzasjichen: oh, just chekout your stable branch and cherry-pick on the master sha10:20
*** arif-ali has quit IRC10:20
johnthetubaguysudipto: but we only update migration records for migrates and resizes at this point10:20
bauzassudipto: current_time - launched_at as said by jaypipes10:20
jichenbauzas: ok, let me try it, thanks10:20
*** apuimedo has joined #openstack-nova10:20
johnthetubaguybauzas: hmm, not so sure about that, but OK…. does that work with evacuate etc?10:21
sudiptobauzas, the launched_at is updated for a operation like re-size as well...10:21
*** dikonoor has joined #openstack-nova10:21
johnthetubaguysudipto: for resize you can look at migrate records in the DB10:21
sudiptobauzas, if that's the case, then what Jay Pipes has suggested would only give me the time for that instance on a given host since the last operation.10:21
sudiptobauzas, ok.10:21
bauzasjohnthetubaguy: yeah, sounds like but I can double-check10:22
*** zhangtralon has quit IRC10:22
sudiptobauzas, johnthetubaguy IMHO this isn't a very direct way of figuring out the age of a VM on a host.10:22
bauzassudipto: then call the bus :)10:22
bauzassudipto: ie. look at your notifications10:23
bauzassudipto: querying a DB doesn't sound very handy IMHO10:23
bauzassudipto: that's the whole purpose of Ceilometer btw.10:23
bauzassudipto: but you don't need Ceilo, just subsribe to the RPC notifications10:23
johnthetubaguystachtach does that too, but that is another way10:23
sudiptoCeilo is also fine...10:24
bauzasjohnthetubaguy: agreed, as I said, it's just all about getting stats from the queue10:24
bauzasso any oslo.messing notification poller is good10:24
bauzasoh10:24
bauzass/messing/messaging10:24
* bauzas made a unvoluntary but funny pun10:25
bauzasdims: ^10:25
sudiptobauzas, johnthetubaguy atleast it sounds like I can work on a cleanup task to clear out the scheduled_at field from the models and subsequently from the DB?10:26
bauzasso yeah, IMHO getting stats from DB is just one way for operators to troubleshoot10:26
sudiptobauzas, johnthetubaguy but that may lead to some migration issues?10:26
bauzassudipto: I would tend to say yes, if the field is unused - but that needs to be triaged on how - just because we're working on online migrations10:26
*** wznoinsk has quit IRC10:27
*** doude has joined #openstack-nova10:27
sudiptobauzas, triaged on how - i guess you mean - a trace to confirm that it's completely unused?10:28
bauzassudipto: no no10:28
bauzassudipto: I mean, you can't just write a migration script which would remove that col10:29
bauzassudipto: so a spec should be needed anyway10:29
bauzassudipto: because that's a DB migration10:29
bauzasbut I leave that ^ to johnthetubaguy10:30
johnthetubaguyit needs a spec10:30
bauzasvoila10:30
bauzasand we need to discuss on how migrations should be managed in Liberty right ?10:30
sudiptobauzas, ok a relatively dumb question, do we support backward compatibility of the tables? as in, since this field is not being used in the newer releases, will we have the operator do migration to an older release?10:31
johnthetubaguysudipto: appreciate the appetite for tidying up the DB thought, awesome stuff :)10:31
*** ociuhandu has quit IRC10:31
johnthetubaguythe spec should be really short, which is nice10:31
sudiptojohnthetubaguy, If i have to write a spec, would you please help me with any existing ones that may have done such a thing?10:31
sudiptojohnthetubaguy, I relatively new to openstack contributions, hence asking :)10:32
bauzassudipto: that's what objects are used to10:32
bauzassudipto: in general, you write a compatibility layer in the object, then you do your DB change10:32
johnthetubaguysudipto: there is a data model impact section, you just need to say which column you drop, and at the top in the problem, just say its no longer used, and that should all you need10:32
sudiptobauzas, yeah - the sqlalchemy layer - models.py i guess?10:32
bauzassudipto: no no10:33
bauzassudipto: NovaObjects10:33
sudiptobauzas, Ah i see...10:33
sudiptojohnthetubaguy, ok thanks :)10:33
bauzassudipto: some NovaObjects can be persisted10:33
bauzassome are not10:33
bauzassudipto: so, when calling the .save() method, it writes the content to DB10:33
sudiptobauzas, gotcha!10:34
*** ociuhandu has joined #openstack-nova10:34
johnthetubaguybauzas: it will need to change both I suspect10:34
bauzassudipto: there is also a _from_db_object() method for hydrating the object from an SQLA model10:34
sudiptobauzas, ok10:34
bauzasjohnthetubaguy: yes, my point was just about the compat layer10:34
bauzasjohnthetubaguy: the compat layer is in the object, and the model and migrations are necessary too10:34
*** atuvenie has joined #openstack-nova10:35
sudiptobauzas, so it's not an easy job :'(10:35
bauzasnow the real 1M USD question is : are all the writes to the instances table being objectified ?10:35
bauzassudipto: well, it requires some ramp-up I agree10:36
bauzassudipto: but that's the price to pay for allowing Nova to live upgrade10:36
sudiptobauzas, hmm yeah! But it should be a good exercise I suppose.10:36
bauzassudipto: agreed10:36
sudiptobauzas, any other spec has done a similar thing that you are aware of?10:37
bauzassudipto: oh yes, plenty of10:37
bauzassudipto: I would just recommend you the one I wrote :)10:37
sudiptobauzas, that'd be awesome :) Please give me :)10:37
bauzassudipto: http://specs.openstack.org/openstack/nova-specs/specs/kilo/implemented/detach-service-from-computenode.html10:38
sudiptobauzas, atleast the ramp up would be taken care of via this I would hope...10:38
sudiptobauzas, thanks!10:38
bauzassudipto: well the detach-service-from-computenode patch series is a beast10:39
bauzassudipto: an ugly monster who can make nightmares to children10:39
sudiptobauzas, lol10:39
bauzassudipto: so you have to care about the level of abstraction you need to understand10:39
sudiptobauzas, I will take my time and get back to you with questions...10:40
bauzassudipto: I would rather recommend you to understand how writes are coming in to DB using the NovaObjects10:40
bauzassudipto: sure10:40
sudiptobauzas, Sure...10:40
bauzassudipto: and how the NovaObjects are mediators to DB10:40
sudiptobauzas, Ok...10:40
*** mwagner_lap has quit IRC10:41
*** mmedvede has quit IRC10:45
*** dulek has joined #openstack-nova10:47
openstackgerritjichenjc proposed openstack/nova-specs: New API for get service info from id  https://review.openstack.org/17241210:49
*** mmedvede has joined #openstack-nova10:53
*** bfic has joined #openstack-nova10:56
*** bfic has quit IRC10:57
*** bfic has joined #openstack-nova10:58
johnthetubaguyjogo: would be cool if you could take a quick look at this for me please: https://review.openstack.org/#/c/141108/510:59
*** mwagner_lap has joined #openstack-nova10:59
*** wznoinsk has joined #openstack-nova10:59
*** jwcroppe has joined #openstack-nova11:00
*** jichen has quit IRC11:00
*** yassine_ has joined #openstack-nova11:01
*** dulek_ has joined #openstack-nova11:02
*** dulek has quit IRC11:05
*** atuvenie has quit IRC11:07
*** jwcroppe has quit IRC11:08
*** atuvenie has joined #openstack-nova11:08
*** leseb has quit IRC11:09
kashyapHas anyone else noticed this CI job failing w/ SSH timeouts? - http://logs.openstack.org/61/178061/1/check/check-grenade-dsvm/92458a9/console.html11:13
*** ociuhandu has quit IRC11:13
*** Hosam has joined #openstack-nova11:14
kashyapHmm, seems like this is the offending test - TestVolumeBootPattern.test_volume_boot_pattern,11:14
kashyapwhere it's failing to invoke: '/bin/bash tools/pretty_tox.sh (?!.*\\[.*\\bslow\\b.*\\])((smoke)|(^tempest\\.scenario)) --concurrency=4'11:14
* kashyap looks for ER signatures11:14
*** dims has quit IRC11:18
*** sudipto has quit IRC11:20
*** leseb has joined #openstack-nova11:20
*** tojuvone has quit IRC11:21
*** igordcard has joined #openstack-nova11:23
*** overlayer has joined #openstack-nova11:23
*** Hosam has quit IRC11:23
kashyapJust about 11 hits in the last 48h though, logstash says.11:25
*** ajayaa has quit IRC11:30
*** dulek_ has quit IRC11:33
*** coolsvap is now known as coolsvap|afk11:33
openstackgerritMh Raies proposed openstack/nova: Enable tenant/user specific instance filtering  https://review.openstack.org/13902511:33
*** overlayer has quit IRC11:34
*** igordcard has quit IRC11:34
*** aix has joined #openstack-nova11:35
*** sudipto has joined #openstack-nova11:35
*** igordcard has joined #openstack-nova11:37
jaypipessudipto: what did I suggest? :)11:38
*** dulek has joined #openstack-nova11:39
sudiptojaypipes, Sorry my bad - I was just making sure we don't need this column anymore :)11:39
*** arif-ali has joined #openstack-nova11:40
*** bfic has quit IRC11:40
*** igordcard has quit IRC11:40
*** lucasagomes is now known as lucas-hungry11:40
*** ajayaa has joined #openstack-nova11:42
sudiptojaypipes, and you too would agree with a spec to drop it i suppose?11:43
openstackgerritJohn Garbutt proposed openstack/nova: devref: add information to clarify nova scope  https://review.openstack.org/17862311:43
bauzasjaypipes: it was re: http://lists.openstack.org/pipermail/openstack-dev/2015-March/059603.html11:45
*** mmedvede has quit IRC11:46
bauzasjohnthetubaguy: oh oh thanks for https://review.openstack.org/#/c/177374/3/doc/source/devref/architecture.rst,cm11:46
johnthetubaguybauzas: trying to go through and update that, and add bit missing bits that we want to talk about at the summit11:47
bauzasjohnthetubaguy: since a long time, I was thinking about updating it :)11:47
bauzasjohnthetubaguy: actually since 2 yrs I work with Nova :)11:47
bauzaswell, 1.5yo11:48
jaypipessudipto: gotcha. yeah, I don't really think it's worth having two columns...11:49
jaypipesis that bauzas' opinion as well?11:50
bauzasjaypipes: yep, I think scheduled_at is no longer needed11:50
*** baoli has joined #openstack-nova11:50
bauzasjaypipes: just because when migrating, the scheduler can't be checked11:50
jaypipesright.11:50
bauzasjaypipes: and also because the scheduler no longer proxies the compute boot11:51
jaypipesbauzas: right. it still grabs the destrination host, but the conductor does all the rest, along with the target and source compute nodes.11:51
*** haomaiwa_ has quit IRC11:52
bauzasjaypipes: yeah, as the conductor is also actually an RPC cast...11:52
bauzasjaypipes: so if we would like to know when the conductor is running for the instance, it should be conducted_at11:53
*** kiran-r has quit IRC11:53
*** baoli has quit IRC11:53
bauzasjaypipes: because then scheduler check is an RPC call so synced11:53
*** zul has joined #openstack-nova11:53
*** ferest has joined #openstack-nova11:53
bauzasjaypipes: and last point, I don't see why all of that is needed, just because we now have RPC notifications11:54
*** baoli has joined #openstack-nova11:54
jaypipesbauzas: right. this is one reason why moving to a task-record persistence system is appealing to me. having these timestamp fields on the instance record doesn't fit in many cases.11:54
bauzasjaypipes: so I'm like -1 on querying DB for knowing when an instance is running, and preferring more using the oslo.messaging notifications11:54
*** igordcard has joined #openstack-nova11:54
jaypipesbauzas: would be better to have a tasks table that recorded the start and end time of each task.11:54
*** mmedvede has joined #openstack-nova11:54
bauzasusing Ceilo, Stacktach or whatever else11:54
sudiptojaypipes, bauzas the task table sounds interesting...11:55
bauzasjaypipes: but in Nova, we don't have a right tasks management cycle11:55
jaypipesbauzas: well, yes, I know :)11:55
openstackgerritJohn Garbutt proposed openstack/nova: devref: add a description of flavors  https://review.openstack.org/17862811:55
bauzasI mean, we don't have a -- man, I don't remember the name11:55
jaypipesbauzas: and I agree with you that a notification that the VM was started (and its timestamp) is the most accurate way of determining when a VM was actually running.11:55
jaypipessudipto: so, have we sufficiently confused you yet? ;)11:56
*** dims has joined #openstack-nova11:56
sudiptojaypipes, nope nope, i followed through :)11:56
jaypipes:)11:56
jaypipescool11:56
sudiptojaypipes, I was actually coming from running an algorithm to figure out the oldest VM on a given host...and i think subscribing to the bus also is a good solution.11:57
bauzassudipto: I think so too, just because you don't need to hack DB11:58
jaypipessudipto: right (though of course that won't give you historical data... only from the time you start listening ;)11:58
bauzassudipto: just because the notification events are self-explanatory11:58
bauzasah ah, state diagram !11:58
bauzasman, it's just 10 mins I'm thinking about that without remembering the name...11:59
bauzasor state machine, even11:59
bauzasjaypipes: ^ I remember alaski's session in ATL about providing an API tasks module12:00
sudiptojaypipes, bauzas this is where i am a bit lost...12:00
bauzassounds like it was just stale ?12:00
bauzassudipto: about what ?12:00
sudiptobauzas, the comment for jaypipes about the subscription time and the state diagram stuffd12:00
bauzassudipto: we manage states of the VMs using a single module as an helper12:00
jaypipesalaski is indeed the man.12:01
bauzassudipto: so you know which states are referring to 'running'12:01
bauzassudipto: but we don't really manage the transitions12:01
jaypipesthat's right.12:01
bauzassudipto: it's just each time a new event is fired that we just update the state by going to a state that we know12:01
openstackgerritJohn Garbutt proposed openstack/nova: devref: discuss current state of error handling  https://review.openstack.org/17863512:01
*** bfic has joined #openstack-nova12:01
bauzassudipto: but we're not telling the direction, rather the target, which is not good for many reasons12:02
*** moshele has quit IRC12:02
*** dulek has quit IRC12:02
sudiptobauzas, okk12:02
bauzassudipto: so provided you would like to monitor the notifications, you would have to match with the semantics of what means "running" for Nova12:02
sudiptobauzas, ok..12:03
bauzassudipto: all of that is not provided as an external API information that you can synchronize12:03
bauzasor gather or whatever12:03
sudiptobauzas, got it...12:03
*** raildo has joined #openstack-nova12:03
sudiptobauzas, thanks! I will get back to you for doubts! Making a drive back home now...12:03
bauzassudipto: so, if someone is just creating a new state (like we did for the shelve actions), you have to figure out if that's running or not12:04
sudiptobauzas, ok..12:04
bauzassudipto: one last point12:04
bauzassudipto: for some reason, the documentation is up-to-date about that, which is excellent12:04
*** dprince has joined #openstack-nova12:04
bauzassudipto: http://docs.openstack.org/developer/nova/devref/vmstates.html12:04
sudiptobauzas, great. I should go through this...12:05
bauzasabove is the state transition diagram12:05
sudiptobauzas, and bug you back in sometime...12:05
*** igordcard has quit IRC12:05
bauzassudipto: eh eh12:05
*** tojuvone has joined #openstack-nova12:05
sudiptobauzas, :D12:05
*** bauzas is now known as bauzas_notme12:05
sudiptolol12:05
sudiptothanks for your time!12:06
bauzas_notmenp12:06
bauzas_notmeargh, faced12:06
*** moshele has joined #openstack-nova12:06
sudiptolol12:06
*** bauzas_notme is now known as bauzas12:06
*** igordcard has joined #openstack-nova12:06
*** openstackgerrit has quit IRC12:07
*** otter768 has joined #openstack-nova12:07
*** openstackgerrit has joined #openstack-nova12:07
*** thomasem has joined #openstack-nova12:08
*** Marga_ has joined #openstack-nova12:10
*** sudipto has quit IRC12:10
*** Marga_ has quit IRC12:10
*** Marga_ has joined #openstack-nova12:11
*** otter768 has quit IRC12:12
*** xylan_kong has joined #openstack-nova12:12
xylan_konghey, guys, would anyone please take a look at my patch https://review.openstack.org/163842, to fix the problem of wrong az property of instance. Sylvain and I have different opinions about the patch, so want to hear more suggestions. really appreciate your comments.12:15
*** atuvenie has quit IRC12:15
jaypipes_cerberus_: heya, https://review.openstack.org/#/c/178190/ patch series could use your re-review. thx in advance.12:18
openstackgerritDaniel Berrange proposed openstack/nova: objects: define the ImageMeta & ImageMetaProps objects  https://review.openstack.org/7623412:20
*** rfolco has joined #openstack-nova12:22
openstackgerritTakenori Yoshimatsu proposed openstack/nova: Avoid AttributeError at instance.info_cache.delete  https://review.openstack.org/16790512:23
*** igordcard has quit IRC12:25
*** igordcard has joined #openstack-nova12:25
*** bfic has quit IRC12:26
*** vladikr has joined #openstack-nova12:26
tdurakovjohnthetubaguy: Hi, John!Could you look at blueptint: https://blueprints.launchpad.net/nova/+spec/nova-api-hypervsor-cpu-info ?12:27
*** atuvenie has joined #openstack-nova12:31
*** doron_afk is now known as doron12:31
*** dave-mccowan has joined #openstack-nova12:31
*** joefides has quit IRC12:32
*** jwcroppe has joined #openstack-nova12:35
*** claudiub has joined #openstack-nova12:36
*** EmilienM is now known as EmilienM|afk12:41
openstackgerritSergey Nikitin proposed openstack/nova: Removed unused variables, methods and classes  https://review.openstack.org/16466612:41
*** ajayaa has quit IRC12:41
openstackgerritgaryk proposed openstack/nova: Scheduler: enhance debug messages for multitenancy aggregates  https://review.openstack.org/17865212:42
*** lifeless has quit IRC12:43
openstackgerritSergey Nikitin proposed openstack/nova: Removed twice declared variables  https://review.openstack.org/17865312:43
garykxylan_kong: i think that your proposal is valid12:43
garykxylan_kong: i think that in cases like this there are a number of options: i. wait for cores to get involved in the discussion (this may take a very very … very long time) II. write a mail to the list and let the discussion take place on the list iii. try and collaborate with him to come to a general concensus12:45
openstackgerritSergey Nikitin proposed openstack/nova: Removed variables used not in the scope that they are declared  https://review.openstack.org/17865412:45
*** lucas-hungry is now known as lucasagomes12:47
*** Jeffrey4l has quit IRC12:48
*** tojuvone has quit IRC12:49
*** neelashah has joined #openstack-nova12:49
*** mpaolino has joined #openstack-nova12:51
openstackgerritEli Qiao proposed openstack/nova: Set power state to None when evacuate fails due to recreate not supported  https://review.openstack.org/17315012:52
*** VW_ has joined #openstack-nova12:52
*** moshele has quit IRC12:52
*** lpetrut has joined #openstack-nova12:52
*** doron has quit IRC12:53
*** moshele has joined #openstack-nova12:55
*** jungleboyj has joined #openstack-nova12:55
*** Hosam has joined #openstack-nova12:55
jaypipesjohnthetubaguy: https://review.openstack.org/#/c/177112/ could use fast-tracking, if you wouldn't mind... it's been approved the last 3 cycles. just the API microversion isn't merged. snikitin has updated the spec to align with the API working group guidance on tagging.12:55
openstackgerritJoe Cropper proposed openstack/nova-specs: Proposal for Flexible Resize Scheduling  https://review.openstack.org/17511512:57
*** doron has joined #openstack-nova12:58
*** VW_ has quit IRC12:58
*** bfic has joined #openstack-nova12:59
*** Jeffrey4l has joined #openstack-nova12:59
*** sandywalsh has joined #openstack-nova13:00
*** ir2ivps3_ has quit IRC13:01
*** Alexandra_ has joined #openstack-nova13:02
openstackgerritPh. Marek proposed openstack/nova-specs: Connecting Nova to DRBD storage nodes directly.  https://review.openstack.org/13415313:02
johnthetubaguyjaypipes: its changed quite a lot since the previous version, so technically needs another +2, but it looks good to me now13:02
*** dulek has joined #openstack-nova13:03
jaypipesjohnthetubaguy: thx!13:03
*** annegentle has joined #openstack-nova13:03
johnthetubaguytdurakov: I think your suggest API change makes a lot of sense, but you will need to submit a spec for an API change I am afraid13:05
johnthetubaguytdurakov: we want a spec to make sure we can get API folks to check all changes, because we generally have to support any API change for ever, so we want to try and get it correct first time13:06
johnthetubaguytdurakov: I hope that helps13:06
tdurakovjohnthetubaguy, yep, thanks!13:06
tdurakovso, start with the spec:)13:06
*** doron is now known as doron_afk13:07
*** ZZelle has quit IRC13:08
*** ZZelle has joined #openstack-nova13:09
kashyapWould anyone else like to take a look this easy set of cleanups, it already has a +2 and Jenkins seems to behave just fine? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:cleanups_in_create_cow_image,n,z13:09
garykany chance that we can geta  +2 on https://review.openstack.org/#/c/174480/ - it is blocking all features, bug fixes with DB migrations …13:09
*** joefides has joined #openstack-nova13:12
*** ociuhandu has joined #openstack-nova13:12
*** Madkiss has quit IRC13:13
garykkashyap: the queue is pretty long for review requests …13:14
*** burt has joined #openstack-nova13:15
kashyapgaryk: Yep, I think the one you pointed out above sounds a bit of high prio13:15
kashyapMine can wait.13:15
*** atuvenie has quit IRC13:16
garykkashyap: i think that they are all equally important. i have reviewed yours and they look good. nice cleanup.13:18
kashyapThanks for taking a look.13:19
*** dulek has quit IRC13:21
*** dulek has joined #openstack-nova13:21
johnthetubaguydansmith: do we know what turbo hipster is waiting for with this one? https://review.openstack.org/#/c/17448013:22
*** atuvenie has joined #openstack-nova13:26
sdagueso I thought that mikal and jhesketh were going to fix the data sets for that13:26
sdagueI'm going to see if I can trigger another db ci run on it13:27
jheskethsdague: waiting on seeing if https://review.openstack.org/#/c/176574/ goes in13:28
jheskethonce that goes in it should pass13:28
jheskethsdague: re-triggering won't help, I have it not voting on failures from the gerrit stream breaking up earlier13:28
sdaguejhesketh: well... that had a -2 from jenkins13:28
jheskethI need to turn it back on, but I've been monitoring the failures myself13:29
sdagueI just rechecked13:29
jheskethsdague: right, so need to figure out if that was legit or not13:29
jhesketh(haven't looked at the logs yet)13:29
sdaguethe fail runs are 18s long13:29
sdaguewhich means there is no chance they are legit13:29
jheskethah, well noted13:29
sdaguethat's like super early gerrit setup13:29
jheskethokay, once that goes in I can get th to pass on the wedge migrations thingy13:30
jheskethbut that'll probably have to wait until tomorrow sorry13:30
johnthetubaguyah, thanks folks for the context, and keeping it going13:30
*** matrohon has joined #openstack-nova13:30
*** artom has joined #openstack-nova13:32
sdagueok, well, in future when we have a critical patch stack like this, it would be nice to be a bit more aggressive staying on top of them. Those fail results came back 14 hrs ago.13:32
*** ir2ivps3_ has joined #openstack-nova13:33
*** ajayaa has joined #openstack-nova13:34
*** jungleboyj has quit IRC13:34
*** Hosam has quit IRC13:35
*** EmilienM|afk is now known as EmilienM13:37
*** VW_ has joined #openstack-nova13:38
*** lifeless has joined #openstack-nova13:38
*** josecastroleon has joined #openstack-nova13:39
*** VW_ has quit IRC13:39
*** VW_ has joined #openstack-nova13:40
*** popw has quit IRC13:41
jheskethsdague: sorry, I didn't realise it was critical/urgent13:42
*** ferest has quit IRC13:44
*** thangp has joined #openstack-nova13:45
*** Jeffrey4l has quit IRC13:46
*** Alexandra_ has quit IRC13:46
dansmithjohnthetubaguy: yeah, not really blocked on t-h, but I don't think there's a rush to land it so waiting until we have a good ack makes sense to me13:46
mriedem_awayhttps://review.openstack.org/#/c/176574/ is back in the queue13:47
*** atuvenie has quit IRC13:48
sdaguemriedem_away: right, I just rechecked :)13:49
*** mriedem_away has quit IRC13:51
johnthetubaguyalex_xu: have you reviewed this spec, it seems similar to some you are suggesting: https://review.openstack.org/#/c/168982/13:52
*** mtreinish has joined #openstack-nova13:54
*** iamjarvo has joined #openstack-nova13:55
*** iamjarvo has quit IRC13:56
*** iamjarvo has joined #openstack-nova13:57
*** iamjarvo has quit IRC13:57
*** ujjain- is now known as ujjain13:57
*** iamjarvo has joined #openstack-nova13:57
*** iamjarvo has quit IRC13:58
*** iamjarvo has joined #openstack-nova13:58
*** iamjarvo has quit IRC13:58
*** iamjarvo has joined #openstack-nova13:59
*** Jeffrey4l has joined #openstack-nova13:59
*** iamjarvo has quit IRC13:59
*** cfriesen has joined #openstack-nova13:59
*** iamjarvo has joined #openstack-nova14:00
*** jecarey has joined #openstack-nova14:01
*** zz_jgrimm is now known as jgrimm14:01
*** jichen has joined #openstack-nova14:02
*** mriedem has joined #openstack-nova14:03
*** marun has joined #openstack-nova14:04
*** rushiagr_away has quit IRC14:05
openstackgerritjichenjc proposed openstack/nova-specs: New API for get service info from id  https://review.openstack.org/17241214:05
*** alexpilotti has joined #openstack-nova14:05
*** mtreinish has quit IRC14:05
*** joefides_ has joined #openstack-nova14:06
alex_xujohnthetubaguy: yes, I noticed after work out my spec. In the intial I just want to propose this https://review.openstack.org/#/c/170396, then I extended it to custom cpu model also. I hope can work with the author of https://review.openstack.org/#/c/168982 . I think the custom cpu feature and cpu model both depend on how we repor cpu model info14:06
-openstackstatus- NOTICE: gerrit has been restarted to clear a stuck events queue. any change events between 13:29-14:05 utc should be rechecked or have their approval votes reapplied to trigger jobs14:06
*** bfic has quit IRC14:06
*** otter768 has joined #openstack-nova14:08
*** joefides has quit IRC14:08
*** cbits has joined #openstack-nova14:08
*** exploreshaifali has joined #openstack-nova14:08
*** cbits has left #openstack-nova14:09
*** haomaiwang has joined #openstack-nova14:10
*** pkoniszewski has quit IRC14:11
*** apuimedo has quit IRC14:12
*** otter768 has quit IRC14:12
*** annegentle has quit IRC14:12
johnthetubaguyalex_xu: the new version of their spec seems to cover that now, if you could review their latest spec and add anything you think is missing, that would be awesome14:13
johnthetubaguyalex_xu: as a heads up, I am really worried about the "bad API" extra-specs and image properties are becoming, so I hope we can get this all validated soon-ish14:13
*** yamahata has joined #openstack-nova14:14
alex_xujohnthetubaguy: looking the spec now14:15
*** lpetrut has quit IRC14:15
johnthetubaguyalex_xu: awesome, no rush, but that would be great14:15
alex_xujohnthetubaguy: yea, that's worth to think for API14:15
*** rushiagr_away has joined #openstack-nova14:16
*** annegentle has joined #openstack-nova14:17
dansmithmriedem: before I respond, aren't restore-able instances in SOFT_DELETED state and not actually deleted=1 ?14:17
*** zhenguo has quit IRC14:17
mriedemdansmith: hrm, will have to look14:18
mriedembtw, we set the deleted value to the id of the row that we're deleting?14:18
mriedemnever dug into why deleted wasn't a boolean14:19
*** exploreshaifali has quit IRC14:20
dansmithmriedem: yes14:20
dansmithmriedem: it's something related to the constraints and indexing14:20
dansmithmriedem: we can surely add a --also-deleted flag that will let people migrate them if they want, or a --uuid= flag14:21
johnthetubaguydansmith: was it the unique constraint, so two deleted things can actually have all the same values, or something like that?14:21
dansmithmriedem: but blocking their upgrade because of deleted instances is going to be hard to justify14:21
dansmithjohnthetubaguy: yeah, probably14:21
mriedemdansmith: i'm not -114:21
mriedemjust asking questions14:21
dansmithmriedem: yep, and they're good14:21
mriedemi know some people hate asking questions in reviews :)14:21
dansmithmriedem: hah14:22
bauzasmriedem: -1 for your sentence14:22
mriedem*hate being asked questions14:22
dansmithI got it :)14:22
mriedemthat was for bauzas14:23
bauzasmriedem: eh, seriously, kidding :)14:23
bauzasbut we can just open the can of worms14:23
openstackgerritjichenjc proposed openstack/nova-specs: New API for get service info from id  https://review.openstack.org/17241214:23
*** Murali has quit IRC14:24
*** nkrinner has quit IRC14:24
PaulMurrayjohnthetubaguy, jaypipes I have a question about api versions - one of my outstanding patches for RT objects corrects api samples for pci14:25
PaulMurrayjohnthetubaguy, jaypipes does that need an api version bump?14:25
PaulMurraynote the samples were wrong - I haven't changed the api behavior14:25
bauzasPaulMurray: I tend to say no, that's examples14:26
*** asrangne has joined #openstack-nova14:26
bauzasPaulMurray: just because it doesn't need a dialog between the client and the API14:26
PaulMurraybauzas, johnthetubaguy ok - but I assume it would need a docimpact though - I think samples are included in docs - is that right?14:27
johnthetubaguyPaulMurray: if you don't change what the API is doing, seems like you don't need to bump the version, don't know about the docs, I would have to check14:27
jaypipesPaulMurray: if the *actual* API is correct, just the api samples tests are incorrect, then no. if the actual API changes, then yes, it would need a spec, a microversion, and approval.14:27
jaypipesor, what johnthetubaguy said, simply :)14:28
*** jungleboyj has joined #openstack-nova14:28
PaulMurray:)14:28
*** aswadr has quit IRC14:28
dansmithbauzas: api_samples are not just examples :)14:28
openstackgerritTomi Juvonen proposed openstack/nova-specs: New nova API call to mark nova-compute down  https://review.openstack.org/16983614:28
openstackgerritHans Lindgren proposed openstack/nova: Cleanup conductor unused methods  https://review.openstack.org/16885614:28
openstackgerritMarian Horban proposed openstack/nova: Disassociating of floating IPs corrected  https://review.openstack.org/14935814:29
PaulMurraydansmith, there is a test that passes in something like the sample and checks it - it only works because the dict data structure is not checked - the incorrect fields just pass through14:29
openstackgerritDan Smith proposed openstack/nova: Wedge DB migrations if flavor migrations are not complete  https://review.openstack.org/17448014:29
PaulMurraydansmith, when I put th ecompute node object in it checks the structure and blows up because its incorrect14:29
dansmithPaulMurray: right, so like jaypipes said, you need a bump if the api has to change, not if the tests were broken and you fix them14:30
bauzasdansmith: orly ? in which situation are they more than just docs ?14:30
bauzasdansmith: except tests of course14:30
dansmithbauzas: they're validating input and output from the api14:30
bauzasoh man...14:30
dansmithbauzas: they generate doc samples as a side effect :)14:30
bauzasdansmith: you mean v2 then ?14:30
dansmithbauzas: both14:31
bauzasdansmith: because I thought input validation was done other way for v2.114:31
dansmithbauzas: we have them for v2 and v2.1 to make sure they're the same, but we had v3 as well14:31
bauzasok, stop chatting, bauzas and RTFC14:31
dansmithbauzas: right, but these are actually testing it14:31
dansmiththey don't really test anything other than that the output for a command given a certain input matches what we expect14:32
dansmithbut they do catch some very subtle things sometimes14:32
bauzasI see14:32
dansmithand in general,14:32
bauzasI was previously working using Pecan/WSME, so I'm missing how input validation is done now in Nova14:32
dansmithany time someone *changes* one of the api samples without adding a version or extension, that's a pointer to them probably changing the api in a bad way :)14:32
bauzasdansmith: agreed14:33
bauzasgood to know14:33
mriedembauzas: jsonschema validation is used with the v2.1 stuff14:33
bauzasPaulMurray: so, FWIW, I think it needs more than just a simple answer14:33
* bauzas RTFC14:33
mriedemhttp://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/schemas/v314:33
mriedembauzas: e.g. http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/plugins/v3/migrate_server.py#n6414:34
mriedemlive migration calls into the schema validation for that14:34
bauzasmriedem: yeah, that's what I was reading for the validators14:34
mriedemwhich is this http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/schemas/v3/migrate_server.py#n2314:35
bauzasmriedem: gotcha14:35
bauzasmriedem: just reading now for v214:36
bauzasok, that's on purpose14:37
PaulMurraybauzas, I think in this case the api verson does not need to bump - just the test and sample needs correcting14:38
*** lpetrut has joined #openstack-nova14:38
PaulMurraybauzas, the reason is incorrect data is provided in the test and a passes through to be checked against incorrect sample output14:39
*** wznoinsk has quit IRC14:39
PaulMurraybauzas, if generated by the virt driver the inccroect data isn't given14:39
*** Marga_ has quit IRC14:39
PaulMurraybauzas, adding the compute node object makes the test fail because for the first time its checking that format of the data - and its incorrect14:39
PaulMurrayso I fix the test and the sample and we're done I think14:40
bauzasPaulMurray: I remember your change14:40
PaulMurraybut still not sure if I need a docimpact in the commit - I need to find the wiki page I think14:40
bauzasPaulMurray: yeah, by reading it, I was thinking it was not used as input validation, hence my "yes, it doesn't need a bump"14:40
bauzasPaulMurray: I should now ramp-up on the validation side before giving you a vote14:41
*** dims has quit IRC14:42
*** bkopilov has quit IRC14:42
PaulMurraybauzas, don't worry - I was updating the spec and realised I might need to mention this14:42
*** dims has joined #openstack-nova14:42
*** mtreinish has joined #openstack-nova14:45
*** bkopilov has joined #openstack-nova14:45
*** mriedem has quit IRC14:46
*** jwcroppe has quit IRC14:47
*** jwcroppe has joined #openstack-nova14:47
bauzasdansmith: so I looked at the code, and IIUC api_samples are used for functionally testing that both inputs and outputs match the requirements, right ?14:48
*** alexpilotti has quit IRC14:49
dansmithI dunno about "requirements" but they're used to make sure the input/output doesn't drift14:49
dansmithfor example,14:49
*** marcusvrn_ has joined #openstack-nova14:49
dansmithwe used to sometimes return an id of "1" and other times an id of 1, depending on where we got the data from14:49
*** tonytan4ever has joined #openstack-nova14:49
*** mtreinish has quit IRC14:49
bauzasdansmith: understood, I was more thinking of something decorating the API method saying "eh, this is the format I'm waiting"14:49
dansmiththis (helps to) ensure that we don't have that happen14:50
dansmithright, we have that14:50
dansmiththat's the schema14:50
dansmiththese are the tests14:50
bauzasdansmith: that, I can find it for v2.114:50
dansmithright14:50
bauzasI mean the jsonvalidation14:50
dansmithright14:50
bauzasbut that's not reading the api_samples IIUC14:50
dansmithright14:51
*** ajayaa has quit IRC14:52
sdaguePaulMurray: if the tests were wrong, fixing the tests is fine14:52
bauzasok, then that's what I was meaning to say when I said "api_samples are just used for docs and tests :)"14:52
*** mriedem has joined #openstack-nova14:52
bauzasbut that's fine14:52
dansmithbauzas: you said just docs :)14:52
*** jwcroppe has quit IRC14:52
*** claudiub has quit IRC14:52
dansmith<bauzas> PaulMurray: I tend to say no, that's examples14:52
bauzas(16:30:11) bauzas: dansmith: orly ? in which situation are they more than just docs ?14:52
bauzas(16:30:19) bauzas: dansmith: except tests of course14:52
bauzasbut I think I was unclear :)14:53
dansmithI didn't interpret that "except tests" to mean what you say now14:53
dansmithbut that's fine, you get it now14:53
bauzas:D14:53
*** pkoniszewski has joined #openstack-nova14:53
*** garyk has quit IRC14:53
bauzasyeah, excellent opportunity to consider how Nova is doing input/output validation14:53
bauzasand trust me, that's far simplier than WSME types :(14:54
*** tteggel_ has quit IRC14:54
*** garyk has joined #openstack-nova14:55
bauzasdansmith: thanks for having helped me to sort out that stuff :)14:55
*** emagana has joined #openstack-nova14:55
*** tteggel has joined #openstack-nova14:56
dansmithnp14:56
*** unicell has joined #openstack-nova14:57
*** garyk has quit IRC14:57
*** unicell1 has quit IRC14:57
jaypipessdague: Hey, trying to get one of your Gerrit queries workin...14:57
jaypipeshttps://review.openstack.org/#/q/status:open+-Verified-1+Verified%252B1+-CodeReview%252B2+-CodeReview%252B1+-CodeReview-1+-CodeReview-2+(project:openstack/nova+OR+project:openstack/python-novaclient)+branch:master,n,z14:57
*** sirushti has left #openstack-nova14:58
jaypipessdague: wondering why it doesn't filter out CodeReview properly.14:58
sdagueso.... column references changed from gerrit 2.4 -> 2.814:58
jaypipessdague: this is your "Lost Patches" query, BTW, from your blog post14:58
jaypipessdague: gotcha... do I need the Label: thing?14:58
sdagueinstead you need things like label:CodeReview>=-114:59
sdagueyeh14:59
sdaguethe gerrit dash creator has some examples14:59
jaypipessdague: got it. ty sir :)14:59
sdaguehttps://github.com/stackforge/gerrit-dash-creator/blob/master/dashboards/compute-program.dash14:59
sdagueoh, it's Code-Review as well14:59
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Add documentation for nova-cells and nova-idmapshift commands  https://review.openstack.org/16655314:59
*** patrickeast has joined #openstack-nova15:00
sdagueanyway, personally, when I'm building new queries like this I just make local dash files in gerrit dash creator and build the urls from tehre15:00
sdagueI find it easier than hacking urls15:00
jaypipesgotcha.15:01
jaypipesthx again, sean15:01
*** garyk has joined #openstack-nova15:01
sdagueno prob15:01
*** packet has joined #openstack-nova15:02
*** nelsnelson has joined #openstack-nova15:02
*** browne has joined #openstack-nova15:02
*** dulek has quit IRC15:02
cfriesendansmith: re https://review.openstack.org/#/c/169836/   Russel is saying that you can't evacuate _from_ disabled (but "up") compute services.15:03
openstackgerritZhenguo Niu proposed openstack/nova: Ironic hypervisor should not be counted when in maintenance status  https://review.openstack.org/17757515:03
cfriesens/Russel/Russell15:03
*** ganso__ has joined #openstack-nova15:04
*** ijw has joined #openstack-nova15:05
*** ajayaa has joined #openstack-nova15:05
*** ijw has quit IRC15:06
*** ijw has joined #openstack-nova15:06
*** erkules_ is now known as erkules15:08
*** erkules has joined #openstack-nova15:08
dansmithcfriesen: yeah15:08
bauzascfriesen: we already discussed that IIRC15:09
bauzascfriesen: I thought we were agreeing on seeing why the service_is_up() check was still needed, particularly if you have distributed computes15:10
cfriesensorry...I scrolled up a bit but must have missed it.15:10
bauzascfriesen: it was like a week before15:10
openstackgerritOleg Bondarev proposed openstack/nova: Log a warning if sec-groups and port id are provided on boot  https://review.openstack.org/15406815:10
cfriesenbauzas: ah, yes.  thought you were referring to something recent. ?)15:11
cfriesen:)15:11
cfriesencan't type today15:11
*** alexpilotti has joined #openstack-nova15:11
*** claudiub has joined #openstack-nova15:13
*** erkules has left #openstack-nova15:13
openstackgerritSergey Nikitin proposed openstack/nova-specs: Implement server instance tagging  https://review.openstack.org/17711215:14
*** Jeffrey4l has quit IRC15:14
*** dprince has quit IRC15:16
claudiubjaypipes: hello15:16
jaypipesclaudiub: ciao amico15:17
jogojohnthetubaguy: re: https://review.openstack.org/#/c/141108/515:18
jogowhat do the neutron folks think?15:18
claudiubjaypipes: :)) bonjour monsieur. Do you have a little bit of time to take a look at 2 very small commits? One of them has a +2: https://review.openstack.org/#/c/170433/ and https://review.openstack.org/#/c/176817/15:18
*** tjones2 has joined #openstack-nova15:18
jaypipesclaudiub: certo!15:18
claudiubjaypipes: merci! :D15:19
*** joefides_ has quit IRC15:19
jaypipesclaudiub: you are not Italian? :)15:19
johnthetubaguyjogo: we have a rep to ask now, I should activate them somehow15:19
bauzasje peux parler Francais moi aussi hein15:19
*** joefides has joined #openstack-nova15:19
claudiubjaypipes: actually, no. :D alexpilotti is. I'm just romanian. :P15:19
jaypipesha! ok then!15:20
*** tjones2 has left #openstack-nova15:20
claudiubjaypipes: but you are partly corect, since we have the same roots. :D15:20
jaypipesclaudiub: Credeam că ești italian15:20
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Fix disconnect_volume issue when find_multipath_device returns None  https://review.openstack.org/15962615:20
claudiubjaypipes: :O sunt surprins să văd romănă pe irc. :))15:21
jogojohnthetubaguy: I see a few neutron folks are signed up on the spec itself15:21
jogojohnthetubaguy: so +Wed15:21
jaypipesclaudiub: actually, we have a couple Romanians at Mirantis :) Including our CEO :)15:21
*** heyongli has quit IRC15:21
claudiubjaypipes: I've actually heard that, when we were at the paris summit. I was shocked when I heard. :))15:22
*** heyongli has joined #openstack-nova15:22
johnthetubaguyjogo: its a spec thats probably best reviewed in code anyway really, if it looks like a dogs dinner, then we need to try something else15:22
jaypipesclaudiub: actually, I should have known... if you were Italian, I think your name would be "Claudio", not "Claudiu", right?15:22
claudiubjaypipes: yeah, that's right. :)15:23
*** Nikolay_St has quit IRC15:23
* jaypipes laments his lame and mono-syllabic names.15:23
*** eharney has quit IRC15:23
*** jwcroppe has joined #openstack-nova15:23
jogojohnthetubaguy: yeah15:24
claudiubjaypipes: but you don't have to worry about people butchering your name, like I do. :D15:24
jaypipesheh, true enough :)15:24
dansmithjaypipes: at least you have an interesting last name15:25
jaypipesdansmith: :)15:25
jaypipesclaudiub: k, https://review.openstack.org/#/c/176817/ approved.15:25
claudiubjaypipes: awesome, mulțumesc. :D15:26
jaypipesclaudiub: nici o problema15:27
*** moshele has quit IRC15:28
mriedemjogo: i'm -1 on that spec15:28
openstackgerritSergey Nikitin proposed openstack/nova-specs: Implement server instance tagging  https://review.openstack.org/17711215:28
* dims needs google auto translation now on slack AND irc :)15:28
mriedemjogo: johnthetubaguy: -1 on https://review.openstack.org/#/c/141108/ but it's already merged15:29
mriedemwas just leaving comments15:29
johnthetubaguymriedem: sorry, I totally missed the nonsense in the testing section15:30
* mriedem storms out15:31
johnthetubaguymriedem: my take is, "I want to see the code" before we can tell if this works properly, so I wasn't really keeping the bar that high15:31
mriedemyeah we'll see i guess15:32
mriedemwe could have had wip code up for 6 months now15:32
*** Jeffrey4l has joined #openstack-nova15:32
*** sdake has joined #openstack-nova15:34
johnthetubaguymriedem: yes, that would have made it easier, honestly, I wonder about just deleting the spec and calling it a spec-less blueprint15:35
mriedemmeh15:36
*** iamjarvo has quit IRC15:36
mriedemthe spec is pretty nebulous, it has some goals but doesn't have details15:36
mriedemso yeah, the spec is kind of pointless15:36
*** VW__ has joined #openstack-nova15:36
mriedemcode talks15:36
mriedemplus i'm just grumpy15:36
*** redbeard has quit IRC15:36
*** moshele has joined #openstack-nova15:37
*** tjones2 has joined #openstack-nova15:37
*** VW_ has quit IRC15:37
*** mtanino has joined #openstack-nova15:38
johnthetubaguymriedem: yeah, I kinda read too many specs, it just doesn't seem as bad as the others, I appreciate the kick :)15:38
jaypipesmriedem: https://review.openstack.org/#/c/170433/ looks to be familiar to you...15:39
*** kiran-r has joined #openstack-nova15:39
openstackgerritDaniel Berrange proposed openstack/nova: objects: define the ImageMeta & ImageMetaProps objects  https://review.openstack.org/7623415:39
mriedemjaypipes: yeah15:39
*** VW__ has quit IRC15:41
bauzasalaski: morning, I guess you would be interested in https://review.openstack.org/#/c/178563/15:41
bauzasjaypipes: ^ too15:41
*** dulek has joined #openstack-nova15:41
*** ildikov has quit IRC15:41
*** atuvenie has joined #openstack-nova15:42
jaypipesbauzas: hells yeah. +115:42
bauzasjaypipes: there is also https://review.openstack.org/#/c/176083/ that you could care about15:43
alaskibauzas: that technically adds a dependency to the spec.  but +1 on the change15:43
dansmithsahid: left some comments on your console objects spec, looking for some more detail15:43
bauzasalaski: the spec already needs a couple of new objects to be created :(15:43
* artom tries asking here, since some people are bound to be familiar with Neutron. I can't figure out how to "trigger" pagination in Neutron. The _pagination method in the client seems to indicate it's supported for all resources, but I haven't been able to obtain a response with the link elements..15:43
jaypipesbauzas: also an easy +1. thx.15:43
*** mtreinish has joined #openstack-nova15:44
dansmithartom: try #openstack-neutron15:44
dansmithoh, you did15:44
sahiddansmith: cool i will see that, thanks15:44
dansmithheh15:44
artomdansmith, it's kinda dead in there :P15:45
*** armax has joined #openstack-nova15:45
alaskibauzas: hmm, there should probably be another update to add those all as dependencies15:45
*** mhorban has quit IRC15:45
*** jcoufal has quit IRC15:45
bauzasalaski: all of those nested objects will each one be a separate change, for sure15:45
*** rushiagr_away is now known as rushiagr15:46
bauzasalaski: but that fortunately doesn't require a spec for each of those :D15:46
*** browne has quit IRC15:46
*** jwcroppe has quit IRC15:47
*** sdake_ has joined #openstack-nova15:47
*** garyk1 has joined #openstack-nova15:48
*** yassine_ has quit IRC15:48
*** tjones2 has quit IRC15:48
*** dsanders has joined #openstack-nova15:48
*** igordcard has quit IRC15:48
dansmithmriedem: seen this? http://logs.openstack.org/80/174480/15/check/check-tempest-dsvm-full/27b3bb4/console.html15:49
dansmithdon't see an e-r bug for it15:49
*** ferest has joined #openstack-nova15:50
mriedemnope15:50
*** garyk has quit IRC15:50
mriedemit's very descriptive though15:50
mriedemhttp://goo.gl/ivd4Zk15:51
mriedemmtreinish: ^15:51
dansmithdammit15:51
dansmithmy logstash fu is too slow15:51
*** sdake has quit IRC15:51
mriedemdansmith: http://logs.openstack.org/80/174480/15/check/check-tempest-dsvm-full/27b3bb4/logs/tempest.txt.gz?level=TRACE#_2015-04-29_15_22_12_23415:51
dansmithwe've seen this 20-30 times a day in a couple cases15:52
mriedemthat's the real error15:52
dansmithweird15:52
*** tjones2 has joined #openstack-nova15:52
mtreinishugh, I hate boto15:52
mriedemi'll open a bug15:52
mriedemand dibs on the e-r query15:52
dansmithlet me know the bug number so I can recheck against it15:52
*** annegentle has quit IRC15:53
*** asrangne has quit IRC15:53
dansmithjhesketh: your force patch landed, so can you get t-h on board?15:54
*** ferest has quit IRC15:54
mriedemdansmith: https://bugs.launchpad.net/tempest/+bug/145011015:55
openstackLaunchpad bug 1450110 in tempest "boto tests randomly fail in resource_cleanup" [Undecided,New]15:55
dansmithmriedem: thanks15:55
*** Sukhdev has joined #openstack-nova15:55
johnthetubaguyjaypipes: has the API group spoken about using 409 conflict in this way, I can see good arguments both sides really: https://review.openstack.org/#/c/163275/515:56
mriedemalso this http://logs.openstack.org/40/164340/4/gate/gate-tempest-dsvm-postgres-full/4ef4e88/logs/tempest.txt.gz?level=TRACE#_2015-04-24_17_27_55_96315:56
mriedemmtreinish: ^15:57
mtreinishmriedem: oh so it's nova fault then :)15:57
mriedemhey, let them eat cake15:58
*** tjones2 has quit IRC15:58
jaypipesjohnthetubaguy: I'll have a looksie.15:59
*** dulek has quit IRC15:59
mriedemcool, so there are multiple boto teardown timeout failures16:00
mriedemmtreinish: heh, nova problem? http://logs.openstack.org/40/164340/4/gate/gate-tempest-dsvm-postgres-full/4ef4e88/logs/screen-c-vol.txt.gz?level=TRACE16:00
mtreinishmriedem: heh, I guess I can't read today. That tempest stack trace totally says volume errored too16:02
johnthetubaguyjaypipes: thanks16:02
*** igordcard has joined #openstack-nova16:02
*** dprince has joined #openstack-nova16:03
openstackgerritdave-mcnally proposed openstack/nova: Create a reserved aggregate filter  https://review.openstack.org/17875016:03
*** dulek_home has joined #openstack-nova16:03
*** VW_ has joined #openstack-nova16:04
*** igordcard has quit IRC16:05
*** jlanoux has quit IRC16:05
*** vilobhmm1 has joined #openstack-nova16:06
*** dulek_home is now known as dulek16:06
*** hemnafk is now known as hemna16:06
*** ildikov has joined #openstack-nova16:08
*** otter768 has joined #openstack-nova16:09
*** mriedem is now known as mriedem_lunch16:09
*** eharney has joined #openstack-nova16:09
*** moshele has quit IRC16:11
*** sudipto has joined #openstack-nova16:11
*** jistr has quit IRC16:11
*** Nic has joined #openstack-nova16:13
*** yuntong has quit IRC16:13
*** otter768 has quit IRC16:13
*** yuntong has joined #openstack-nova16:14
openstackgerritAndrew Laski proposed openstack/nova: Reschedules sometimes do not allocate networks  https://review.openstack.org/17747016:14
*** zul has quit IRC16:15
*** mpaolino has quit IRC16:15
*** oro has quit IRC16:17
*** josecastroleon has quit IRC16:17
*** unicell has quit IRC16:19
openstackgerritDaniel Berrange proposed openstack/nova: objects: define the ImageMeta & ImageMetaProps objects  https://review.openstack.org/7623416:19
*** ociuhandu has quit IRC16:20
*** alexpilotti has quit IRC16:20
*** pcaruana has quit IRC16:22
*** mpaolino has joined #openstack-nova16:28
*** atuvenie has quit IRC16:28
*** dikonoor has quit IRC16:31
*** dulek_home_ has joined #openstack-nova16:34
*** dulek has quit IRC16:35
*** dulek_home_ is now known as dulek16:35
openstackgerritsahid proposed openstack/nova: libvirt: remove dependance with libvirt  https://review.openstack.org/17722316:35
openstackgerritsahid proposed openstack/nova: monitor: fix tests to use fakelibvirt  https://review.openstack.org/17815416:35
openstackgerritsahid proposed openstack/nova: libvirt: add method to get hardware info to Host  https://review.openstack.org/17722216:35
openstackgerritsahid proposed openstack/nova: libvirt: add method to compare cpu to Host  https://review.openstack.org/17733116:35
openstackgerritsahid proposed openstack/nova: libvirt: add method to list pci devices to Host  https://review.openstack.org/17733016:35
openstackgerritsahid proposed openstack/nova: libvirt: add method to get device by name to Host  https://review.openstack.org/17732916:35
openstackgerritsahid proposed openstack/nova: libvirt: avoid call of listDefinedDomains when post live migration  https://review.openstack.org/17732816:35
openstackgerritsahid proposed openstack/nova: libvirt: add method to define instance to host  https://review.openstack.org/17431416:35
openstackgerritsahid proposed openstack/nova: libvirt: introduces a Guest to wrap arround virConnect  https://review.openstack.org/17431316:35
openstackgerritsahid proposed openstack/nova: libvirt: add todo note to avoid call to libvirt from the driver  https://review.openstack.org/17733216:35
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation  https://review.openstack.org/13825516:35
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004516:35
*** Jeffrey4l has quit IRC16:37
*** VW__ has joined #openstack-nova16:38
*** VW__ has quit IRC16:38
*** doron_afk has quit IRC16:39
*** VW__ has joined #openstack-nova16:39
*** garyk1 has quit IRC16:39
*** VW_ has quit IRC16:40
*** gszasz has quit IRC16:41
openstackgerritsahid proposed openstack/nova: libvirt: add method to get cpu stats to host  https://review.openstack.org/17722316:42
openstackgerritsahid proposed openstack/nova: monitor: remove dependance with libvirt  https://review.openstack.org/17815416:42
openstackgerritsahid proposed openstack/nova: libvirt: add method to compare cpu to Host  https://review.openstack.org/17733116:42
openstackgerritsahid proposed openstack/nova: libvirt: add method to list pci devices to Host  https://review.openstack.org/17733016:42
openstackgerritsahid proposed openstack/nova: libvirt: add method to get device by name to Host  https://review.openstack.org/17732916:42
openstackgerritsahid proposed openstack/nova: libvirt: add method to define instance to host  https://review.openstack.org/17431416:42
openstackgerritsahid proposed openstack/nova: libvirt: introduces a Guest to wrap arround virConnect  https://review.openstack.org/17431316:42
openstackgerritsahid proposed openstack/nova: libvirt: add todo note to avoid call to libvirt from the driver  https://review.openstack.org/17733216:42
*** sahid has quit IRC16:43
*** achanda has joined #openstack-nova16:43
*** mriedem_lunch is now known as mriedem16:44
*** zzzeek has joined #openstack-nova16:44
*** sudipto has quit IRC16:44
*** pask81 has quit IRC16:44
*** armax has quit IRC16:46
*** armax has joined #openstack-nova16:48
*** Jeffrey4l has joined #openstack-nova16:49
*** matrohon has quit IRC16:50
*** unicell has joined #openstack-nova16:50
*** unicell1 has joined #openstack-nova16:51
*** unicell has quit IRC16:51
openstackgerritMathieu Gagné proposed openstack/nova: Add ability to override network templace from InstanceMetadata  https://review.openstack.org/17834016:53
*** _gryf is now known as _gryf|afk16:54
*** ganso__ is now known as ganso_16:54
*** annegentle has joined #openstack-nova16:54
*** marcusvrn_ is now known as marcusvrn16:55
*** romainh has left #openstack-nova16:56
*** ndipanov has quit IRC16:56
*** ndipanov has joined #openstack-nova16:56
*** gilliard has quit IRC16:57
*** gilliard has joined #openstack-nova16:58
*** derekh has quit IRC16:59
*** sdake_ has quit IRC16:59
*** sdake has joined #openstack-nova16:59
*** patrickeast has quit IRC17:00
*** annegentle has quit IRC17:01
*** pkoniszewski has quit IRC17:01
*** sdake has quit IRC17:02
*** signed8bit has joined #openstack-nova17:03
*** sdake has joined #openstack-nova17:03
*** annegentle has joined #openstack-nova17:03
*** mpavone has quit IRC17:04
*** tjones2 has joined #openstack-nova17:05
*** unicell1 has quit IRC17:06
*** tonytan4ever has quit IRC17:06
*** unicell has joined #openstack-nova17:06
*** igordcard has joined #openstack-nova17:07
*** sdake has quit IRC17:07
*** Nic has quit IRC17:07
*** aix has quit IRC17:10
*** yamahata has quit IRC17:11
*** browne has joined #openstack-nova17:12
*** igordcard has quit IRC17:12
openstackgerritPaul Murray proposed openstack/nova-specs: (Re)Propose: make resource tracker use objects  https://review.openstack.org/17876817:13
*** igordcard has joined #openstack-nova17:13
*** igordcard has joined #openstack-nova17:14
PaulMurrayjohnthetubaguy, jaypipes ^^^ re-submitted RT objects spec17:14
*** penick has joined #openstack-nova17:14
johnthetubaguyPaulMurray: I think I already approved that one for liberty, without a spec, my bad, I assumed it hadn't been approved17:16
*** eglynn has quit IRC17:16
*** tellesnobrega_ has joined #openstack-nova17:16
johnthetubaguyPaulMurray: I can fast approve it though I supose.17:16
*** neelashah has quit IRC17:16
jaypipesPaulMurray: read, and +1 from me.17:16
*** igordcard has quit IRC17:17
*** VW__ has quit IRC17:17
johnthetubaguyPaulMurray: I think you are all approved on this now for liberty17:18
*** marcusvrn has quit IRC17:19
*** melwitt has joined #openstack-nova17:19
*** tellesnobrega_ has quit IRC17:21
*** IanGovett has quit IRC17:21
openstackgerritEd Leafe proposed openstack/nova: Scheduler: Introduce a FilterRequestSpec class  https://review.openstack.org/15824517:23
PaulMurrayjohnthetubaguy, jaypipes thank you very much gentlemen17:24
jaypipesPaulMurray: your welcome, m'lady.17:24
*** sdake has joined #openstack-nova17:24
jaypipesor even you're welcome. where are my grammanners.17:24
openstackgerritDaniel Berrange proposed openstack/nova: objects: remove tracked changes in NUMA objects  https://review.openstack.org/17877217:24
*** patrickeast has joined #openstack-nova17:25
*** tonytan4ever has joined #openstack-nova17:26
*** dikonoor has joined #openstack-nova17:26
PaulMurrayjaypipes, I knew I recognised you - your lady Penelope's  chauffeur from thunderbirds17:27
alaskidansmith: if you have a bit to look at https://review.openstack.org/#/c/177470/4 you could tell me how terrible that is and that there's a better way to do it17:27
jaypipesPaulMurray: lol :)17:27
openstackgerritDaniel Berrange proposed openstack/nova: docs: fixup libvirt NUMA testing docs to match reality  https://review.openstack.org/17877317:28
alaskidansmith: besides ripping out the async networking stuff, which I would like to do17:28
*** iamjarvo has joined #openstack-nova17:30
*** iamjarvo has quit IRC17:30
*** dprince has quit IRC17:30
*** dprince has joined #openstack-nova17:30
*** iamjarvo has joined #openstack-nova17:31
*** iamjarvo has quit IRC17:31
*** moshele has joined #openstack-nova17:31
*** yamahata has joined #openstack-nova17:31
*** iamjarvo has joined #openstack-nova17:31
*** iamjarvo has quit IRC17:32
*** iamjarvo has joined #openstack-nova17:32
*** moshele has quit IRC17:33
*** neelashah has joined #openstack-nova17:33
*** emagana has quit IRC17:33
*** neelashah1 has joined #openstack-nova17:34
*** neelashah has quit IRC17:37
*** mpaolino has quit IRC17:38
mriedemjogo: checked out sahid's comment here https://review.openstack.org/#/c/169520/17:40
mriedemjogo: i don't think that's a problem though, they aren't asynchronous operations17:40
mriedemre: the calls to driver.unfilter_instance17:40
mriedemand driver.cleanup (which calls unfilter_instance)17:40
mriedemjogo: so i'm going to just rebase my retry change17:42
jogomriedem: sounds goo17:43
jogod17:43
jogoI was wondering about that myself17:43
jogoI thought maybe libvirt is async in the background magically17:43
jogoeither way17:43
mriedemi'll add some debug logging in the compute manager though17:43
*** tjones2 has quit IRC17:43
mriedemjogo: maybe it is17:43
mriedemidk17:44
mriedemif libvirt's nwfilter.undefine() was asynch, then yeah that would be a problem17:44
mriedembut we still couldn't fix it with a lock17:44
jogodebugging info will be useful17:44
jogoright17:44
jogoso a retry is needed even in that case17:44
mriedemyeah17:44
*** ganso_ has quit IRC17:47
*** iamjarvo has quit IRC17:48
dansmithalaski: I took a peek earlier17:50
dansmithalaski: it seems icky and fragile if we don't have every such path protected17:50
dansmithalaski: I was thinking, could we do a clone of the instance object when we fire off the async helper, and let it use that for its update? That way we can ensure that it's clean before, and the only thing it will do is  update sysmeta17:51
alaskidansmith: hmm, that could work.  there is an instance.refresh in the conductor on a reschedule so it should pick up the change17:53
alaskiwe could also look at moving the sys_meta update to right before a reschedule, outside of the alloc thread17:54
*** Sukhdev has quit IRC17:55
*** iamjarvo has joined #openstack-nova17:55
*** iamjarvo has quit IRC17:55
*** iamjarvo has joined #openstack-nova17:56
dansmithalaski: true, jsut setting it on the object is all you need to do, right? you don't need to actually save it?17:58
dansmithif the thread waiting for async just makes sure to do a save for some reason once that completes..17:59
dansmiththat'll save a db hit too17:59
dansmithwhen that async code was designed, we didn't have a way to avoid that17:59
*** harlowja_away is now known as harlowja18:00
*** emagana has joined #openstack-nova18:01
alaskidansmith: ahh, good point18:01
alaskiyeah, that's probably the easiest solution here18:02
dansmithif so, it's easy and gainful18:02
alaskiJoshNang: ^18:03
JoshNangoh. wow, yeah that should work18:04
*** beagles is now known as beagles_brb18:04
*** davidlenwell has quit IRC18:05
*** davidlenwell has joined #openstack-nova18:05
*** dhellmann has quit IRC18:05
JoshNangalaski: i'll write up that change18:05
*** lpetrut has quit IRC18:06
alaskiJoshNang: cool18:07
artomPagination in Nova happens at the database level, yes?18:08
artomAnd it seems to not be universally applied to all resources? Specifically, for floating ips I can't find anything.18:08
*** otter768 has joined #openstack-nova18:09
*** claudiub has quit IRC18:10
*** moshele has joined #openstack-nova18:12
*** otter768 has quit IRC18:14
*** harlowja has quit IRC18:16
*** harlowja has joined #openstack-nova18:16
*** harlowja has quit IRC18:17
*** harlowja has joined #openstack-nova18:20
dansmithartom: I think those statements are truthy18:21
mriedemartom: yeah, i only see paging for instances and flavors18:21
mriedemhttp://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/api.py#n207218:21
*** beagles_brb is now known as beagles18:21
mriedemhttp://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/api.py#n467518:21
*** vilobhmm1 has quit IRC18:22
*** ijw has quit IRC18:23
artomdansmith, mriedem, cool, thanks! It's always nice to have someone with more experience confirm what I think is happening ;)18:27
artomAt least there's paginate_query in oslo.db18:27
*** VW_ has joined #openstack-nova18:27
melwittjogo: would you mind taking a quick look at this when you get a chance, small novaclient functional test refactor https://review.openstack.org/#/c/175854/18:28
*** markvoelker has quit IRC18:28
jogomelwitt: heading to lunch, but will look at it this afternoon18:29
melwittcool thanks18:29
*** harlowja has quit IRC18:29
*** sdake has quit IRC18:29
*** harlowja has joined #openstack-nova18:29
mriedemjaypipes: you know this guy? https://review.openstack.org/#/c/177824/18:30
*** IanGovett has joined #openstack-nova18:31
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: retry to undefine network filters during _post_live_migration  https://review.openstack.org/16952018:31
*** Nikolay_St has joined #openstack-nova18:31
*** thangp has quit IRC18:35
*** thangp has joined #openstack-nova18:35
*** oro has joined #openstack-nova18:35
*** annegentle has quit IRC18:36
*** emagana has quit IRC18:36
*** sdake has joined #openstack-nova18:37
*** marun has quit IRC18:39
bauzasmriedem: woah, was just wondering why I was CC'd on https://review.openstack.org/#/c/177824/ with all the gurus18:42
mriedembauzas: i added you since $scheduler18:43
*** emagana has joined #openstack-nova18:43
mriedembefore i noticed that he was just copying the content from the other docs18:43
mriedemwhich led to me barfing on duplicate docs18:43
bauzasmriedem: eh, was just kidding :)18:43
bauzassounds like someone wants an ATC discount code... :D18:44
artomhttps://github.com/openstack/nova/blob/master/nova/api/openstack/common.py#L19918:45
artom"If 'limit' is not specified, 0, or > max_limit, we default to max_limit"18:45
artomif 'limit' in request.GET:18:45
artom        params['limit'] = _get_int_param(request, 'limit')18:45
artomSo this is a fun contradiction :D18:45
artomparams['limit'] is only set if 'limit' is in the GET request.18:46
artomNone of this 'we use max_limit if it's not set' business :P18:46
mriedemartom: there is here: https://github.com/openstack/nova/blob/master/nova/api/openstack/common.py#L26818:47
mriedemand here: https://github.com/openstack/nova/blob/master/nova/api/openstack/common.py#L27618:47
mriedemand here https://github.com/openstack/nova/blob/master/nova/api/openstack/common.py#L49118:47
mriedemget_pagination_params is creating the dict which is used https://github.com/openstack/nova/blob/master/nova/api/openstack/common.py#L275 which does the min18:48
*** matrohon has joined #openstack-nova18:48
*** kaufer has joined #openstack-nova18:48
mriedemartom: so the docstring is misleading18:48
mriedembut doesn't look like a gaping bug18:48
mriedemthe code was probably just refactored over time18:48
mriedemand the docstring should move to get_limit_and_marker18:48
mriedemartom: patches welcome :)18:49
artommriedem, haha, for sure :)18:49
mriedemkaufer: go back to your day job, crisis averted18:49
artomEven if it was a gaping bug, it would have been expected behaviour, and the docstring should have been changed to match it, not the other way around.18:49
*** annegentle has joined #openstack-nova18:49
mriedemartom: not necessarily18:49
mriedemsometimes docs are wrong18:49
artommriedem, the thing is, get_pagination_params is actually called as is from other places in the code18:50
artomWithout going through get_limit_and_marker18:50
mriedemok, so i guess the docstring there is telling the callers what to do? idk, that's pretty weak if it is.18:50
artomThe docstring needs fixing to match what the code is doing, for sure.18:51
artomNot as confident about what to do with code that calls get_pagination_params without going through get_limit_and_marker18:51
*** annegentle has quit IRC18:52
*** annegentle has joined #openstack-nova18:53
*** dikonoor has quit IRC18:53
*** annashen has joined #openstack-nova18:55
*** Marga_ has joined #openstack-nova18:56
*** redbeard has joined #openstack-nova18:57
dimsmelwitt: +1 to https://review.openstack.org/#/c/159626/18:58
dims(your comment i mean)18:58
melwittdims: got it, thanks :)18:58
*** erlon has joined #openstack-nova18:58
dansmithmelwitt: FYI you can fix my tests whenever you want, no permission required :P18:58
melwittdansmith: haha :P18:59
mriedemif you want something to do,19:00
mriedemi've had on my board for awhile to organize test_volume.py into classes per volume type19:01
*** emagana has quit IRC19:01
mriedemright now it's just a big gorp19:01
*** annashen has quit IRC19:01
mriedemnoticed that when i was helping move the s390x fibre channel changes19:01
*** vilobhmm1 has joined #openstack-nova19:01
*** annashen has joined #openstack-nova19:02
* dansmith just threw up in his mouth a little19:02
mriedemswallow it back down19:02
dimsdansmith: that's very nice of you! (fix my tests)19:03
mriedemhttps://www.youtube.com/watch?v=CuQO33FM60419:03
dansmithdims: I'm very gracious about letting other people fix my crap19:03
*** Nikolay_St has quit IRC19:03
*** emagana has joined #openstack-nova19:04
dansmithmriedem: gross19:04
mriedemyou're welcome19:04
*** dhellmann has joined #openstack-nova19:06
*** annashen has quit IRC19:06
*** jaypipes has quit IRC19:06
*** Marga_ has quit IRC19:07
artomTop score for control though.19:07
* lbragstad shakes head at mriedem19:07
*** pixelb has quit IRC19:08
*** VW_ has quit IRC19:10
bauzasmriedem: FYI if you want to comment on... http://lists.openstack.org/pipermail/openstack-dev/2015-April/062964.html19:10
*** vilobhmm1 has quit IRC19:13
*** rushiagr is now known as rushiagr_away19:15
*** vilobhmm1 has joined #openstack-nova19:17
*** salv-orl_ has joined #openstack-nova19:18
*** unicell has quit IRC19:18
openstackgerritIldiko Vancsa proposed openstack/nova: Add functional test for server group  https://review.openstack.org/14751419:19
openstackgerritIldiko Vancsa proposed openstack/nova: scheduler:refactor group info in filter_properties  https://review.openstack.org/14827519:19
openstackgerritIldiko Vancsa proposed openstack/nova: api: add soft-affinity policies for server groups  https://review.openstack.org/14752019:19
openstackgerritIldiko Vancsa proposed openstack/nova: Add functional tests to soft affinity capabilities  https://review.openstack.org/14752119:19
openstackgerritIldiko Vancsa proposed openstack/nova: scheduler: add num of instance per host in a group  https://review.openstack.org/14827719:19
openstackgerritIldiko Vancsa proposed openstack/nova: object:add count_members_by_hosts to InstanceGroup  https://review.openstack.org/14751719:19
openstackgerritIldiko Vancsa proposed openstack/nova: scheduler: add soft-(anti-)affinity weighers  https://review.openstack.org/14751619:19
*** vilobhmm1 has quit IRC19:20
mriedembauzas: thanks, will comment19:21
bauzasmriedem: by looking at the reference docs, it seems they are only provided each time a new release is coming in...19:21
bauzasmriedem: they don't have a master running branch19:22
*** salv-orlando has quit IRC19:22
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/17684419:22
bauzasmriedem: so it sounds little price to pay to take the existing devref and copy that to the reference docs, either manually or by a magic tool19:22
mriedembauzas: i'd prefer to keep the 'golden' docs on scheduler filters in the nova devref19:23
*** vilobhmm1 has joined #openstack-nova19:23
*** jaypipes has joined #openstack-nova19:23
bauzasmriedem: but I can't hardly accept a change which woudn't provide a good doc19:23
mriedemsince they are versioned and reviewed at the same time as changes to the scheduler filters are changed or added, like you said19:23
*** ajayaa has quit IRC19:23
bauzasmriedem: yeah, that's my point19:23
mriedemi don't think nova devs are doing much in the way of reviews in other docs repos19:23
bauzasmriedem: exactly19:23
bauzasmriedem: and that would create some confusion19:23
bauzasmriedem: so yes, it's like the API docs19:24
bauzasmriedem: that's up to the developer to make sure he updates the in-tree docs19:24
*** isd has joined #openstack-nova19:24
openstackgerritJosh Gachnang proposed openstack/nova: Reschedules sometimes do not allocate networks  https://review.openstack.org/17747019:25
*** vilobhmm1 has quit IRC19:26
*** lucasagomes is now known as lucas-dinner19:26
*** luv_ has joined #openstack-nova19:27
*** luv_ has quit IRC19:27
*** luv_ has joined #openstack-nova19:28
*** VW_ has joined #openstack-nova19:28
*** nkrinner has joined #openstack-nova19:29
*** luv_ is now known as luv19:29
dansmithalaski: JoshNang: that patch looks good if it really passes your tests, aside from the small complaint I added there19:30
*** neelashah1 has quit IRC19:30
*** Nikolay_St has joined #openstack-nova19:31
alaskidansmith: they'll be testing it in a real environment in a bit19:31
dansmithalaski: okay cool19:31
*** lucas-dinner has quit IRC19:31
dansmithit's just about the best outcome for a patch,19:32
*** Mike_D_laptop has joined #openstack-nova19:32
dansmithremoving a db hit, fixing a bug, with nothing but code removal :D19:32
dansmithactually,19:32
dansmithdoing all that with just removing a single line of code19:32
dansmithimpressive19:32
alaskiyeah, it's an excellent solution if it works19:32
JoshNangheh :D i've got a meeting in a sec, but plan to get it into preprod and hammer on it this afternoon19:32
JoshNangand ++ on the test bit19:33
melwittI notice the comment mentions _spawn shares a reference to the instance, and recently I found _spawn is the deprecated code path that's part of run_instance. this anyway works in the current code path (build_and_run_instance) I assume?19:36
*** Nikolay_St has quit IRC19:36
alaskimelwitt: always with the sharp eyes19:37
dansmithmelwitt: it's a generic thing19:37
alaskiit works in the current path too19:37
dansmithyeah, he means the thread doing the driver._spawn19:37
dansmithor whatever it is19:37
alaskiit probably came from the test_default_access_ip test which was failing on the previous solution, and tests the old run_instance path19:38
melwittokay, cool. I figured it must be working but when I saw "_spawn" I felt alerted19:38
alaskiwe spent a lot of time looking at that path19:38
dansmithif this works, it was totally my idea to drop that save()19:40
dansmithand if it leads to lots of failures in lemming because of flaky saves, then I dunno, I think I heard alaski say it19:40
mriedemthe channel isn't logged so sure19:41
alaskidansmith: hah19:41
alaskiit is logged now19:41
alaskiI came looking for advice and was totally tricked into that idea19:42
*** unicell has joined #openstack-nova19:43
*** Nic has joined #openstack-nova19:43
*** liusheng has quit IRC19:43
*** david-lyle has quit IRC19:44
*** liusheng has joined #openstack-nova19:44
*** david-lyle has joined #openstack-nova19:44
mriedemoh no http://eavesdrop.openstack.org/irclogs/%23openstack-nova/19:47
bauzascall me stupid or os-services is not presented in http://developer.openstack.org/api-ref-compute-v2.1.html ?19:47
mtreinishmriedem: you didn't realize that :)19:47
*** ajayaa has joined #openstack-nova19:48
mriedemmtreinish: nope19:48
mriedembauzas: don't see it either19:49
*** FL1SK has quit IRC19:49
bauzasmriedem: okay, will ask alex_xu tomorrow morning CET19:49
openstackgerritRoman Podoliaka proposed openstack/nova: Fix rebuild of an instance with a volume attached  https://review.openstack.org/17689119:49
openstackgerritRoman Podoliaka proposed openstack/nova: rebuild: make sure server is shut down before volumes are detached  https://review.openstack.org/17689219:49
bauzasbecause I'm just facing the extension in front of my IDE :)19:50
*** Mike_D_laptop has quit IRC19:52
*** dprince has quit IRC19:52
*** Nikolay_St has joined #openstack-nova19:54
mriedemssshhhh19:55
mriedemdon't let them know you use an IDE19:55
*** arif-ali has quit IRC19:56
artomIntegrated Development Emacs.19:56
artomAka just emacs.19:56
*** doude has quit IRC20:01
*** doude has joined #openstack-nova20:01
bauzasmriedem: well, Sublime Text is technically not an IDE20:04
bauzasit was just a shortcut :)20:04
* bauzas would love LimeText to be as nice as ST3... :(20:05
bauzasbut we're not on Friday, right?20:05
*** eharney has quit IRC20:07
*** otter768 has joined #openstack-nova20:10
*** emagana has quit IRC20:11
openstackgerritMatt Riedemann proposed openstack/nova: Add DB2 support  https://review.openstack.org/6904720:11
*** jungleboyj has quit IRC20:11
*** emagana has joined #openstack-nova20:14
*** otter768 has quit IRC20:15
*** annegentle has quit IRC20:15
*** kiran-r has quit IRC20:19
*** gtt116_ has joined #openstack-nova20:19
*** tonytan4ever has quit IRC20:21
*** gtt116 has quit IRC20:22
*** annegentle has joined #openstack-nova20:23
*** jungleboyj has joined #openstack-nova20:24
*** marun has joined #openstack-nova20:26
bauzasalaski: buzz me when the meeting is starting, I should miss the start20:28
bauzasalaski: thanks20:29
*** jungleboyj has quit IRC20:35
*** eharney has joined #openstack-nova20:41
*** artom has quit IRC20:43
*** jcoufal has joined #openstack-nova20:45
*** mtreinish_ has joined #openstack-nova20:45
*** liusheng has quit IRC20:46
*** liusheng has joined #openstack-nova20:46
*** oro has quit IRC20:48
*** matrohon has quit IRC20:48
*** mtreinish has quit IRC20:51
*** mtreinish_ is now known as mtreinish20:51
*** ajo has quit IRC20:51
*** dulek has quit IRC20:52
*** belmoreira has joined #openstack-nova20:55
*** raildo has quit IRC20:58
*** iamjarvo has quit IRC20:58
*** VW_ has quit IRC21:01
*** tonytan4ever has joined #openstack-nova21:01
*** jcoufal has quit IRC21:03
*** iamjarvo has joined #openstack-nova21:06
*** iamjarvo has quit IRC21:06
*** iamjarvo has joined #openstack-nova21:07
*** marun has quit IRC21:08
*** ajayaa has quit IRC21:13
*** iamjarvo has quit IRC21:14
*** Sukhdev has joined #openstack-nova21:14
openstackgerritjichenjc proposed openstack/nova: Code clean up db.instance_get_all_by_host()  https://review.openstack.org/16413221:15
*** mriedem has quit IRC21:17
openstackgerritjichenjc proposed openstack/nova: Code clean up db.instance_get_all_by_host()  https://review.openstack.org/16413221:17
*** arif-ali has joined #openstack-nova21:18
*** VW_ has joined #openstack-nova21:20
*** annashen has joined #openstack-nova21:21
*** achanda has quit IRC21:22
*** pixelb has joined #openstack-nova21:24
*** nkrinner has quit IRC21:25
*** Longgeek has quit IRC21:29
*** flwang1 has joined #openstack-nova21:29
*** marun has joined #openstack-nova21:29
*** annashen has quit IRC21:33
*** vilobhmm1 has joined #openstack-nova21:33
*** joefides has quit IRC21:33
*** vilobhmm1 has quit IRC21:34
*** vilobhmm1 has joined #openstack-nova21:34
*** IanGovett has quit IRC21:34
*** angdraug has joined #openstack-nova21:35
*** marun has quit IRC21:35
*** achanda has joined #openstack-nova21:36
*** harlowja is now known as harlowja_away21:36
*** harlowja_away is now known as harlowja21:37
*** joefides has joined #openstack-nova21:38
*** Jeffrey4l has quit IRC21:38
*** vilobhmm1 has quit IRC21:39
*** unicell has quit IRC21:40
*** unicell has joined #openstack-nova21:40
*** pixelb has quit IRC21:42
*** thangp has quit IRC21:42
*** marun has joined #openstack-nova21:44
*** isd has quit IRC21:44
*** jichen has quit IRC21:44
*** FL1SK has joined #openstack-nova21:48
*** Jeffrey4l has joined #openstack-nova21:51
*** annegentle has quit IRC21:51
*** bnemec has quit IRC21:52
*** vilobhmm1 has joined #openstack-nova21:53
*** tonytan4ever has quit IRC21:54
flwang1dansmith: ping re patch https://review.openstack.org/#/c/164494/21:58
*** Jeffrey4l has quit IRC21:58
dansmithin a meeting, but go ahead21:58
*** kaufer has quit IRC21:58
flwang1dansmith: cool, thanks, you can replay at your most convenience21:59
*** joefides has quit IRC21:59
flwang11. so at https://review.openstack.org/#/c/164494/12/nova/compute/manager.py  do we still need a new version, like 4.1?21:59
flwang12. based on you and matt's latest comments, seems we're backing to the ps6 solution, adding a new parameter22:00
flwang1so which will cause same change for cell's rpcapi, the question is do we still need a version change for cell's rpcapi?22:00
*** tjones1 has joined #openstack-nova22:01
*** jecarey has quit IRC22:01
*** bnemec has joined #openstack-nova22:02
*** melwitt has quit IRC22:02
*** melwitt has joined #openstack-nova22:02
*** VW_ has quit IRC22:03
*** patrickeast_ has joined #openstack-nova22:04
dansmith1. yes 2. Since mriedem said it explicitly, yes22:04
dansmithcells, if we're changing the call, then yes22:05
*** patrickeast has quit IRC22:05
*** patrickeast_ is now known as patrickeast22:05
dansmithflwang1: ^22:05
flwang1dansmith: but when i changed the version of https://review.openstack.org/#/c/164494/12/nova/compute/manager.py  to 4.1 it will cause some other unrelated test failures22:06
*** VW_ has joined #openstack-nova22:06
melwittdansmith: I was curious about the difference between instance_info_cache and instance objects save method https://github.com/openstack/nova/blob/master/nova/objects/instance_info_cache.py#L88-L98 here we don't do self._from_db_object(rv), does it not really matter? I'm looking at passing an info_cache object here22:07
melwittcurrently it passes rv, so I thought is it wrong to pass self instead if self hasn't been updated as rv?22:08
dansmithflwang1: anything that depends on the rpc version being 4.0 will need to change, yeah22:08
*** VW_ has quit IRC22:09
*** VW_ has joined #openstack-nova22:09
dansmithmelwitt: meaning we don't refresh the info_cache object on save?22:09
melwittdansmith: I mean updating our "self" with the result from the db22:10
dansmithmelwitt: that looks like a terrifically horrible bug/oversight, and may be why everyone wants to refresh that thing all the damn time22:10
dansmithmelwitt: I would recommend fixing that, let's put it into the tree and, uh, see how things go22:11
melwittdansmith: okay, I'll cook it :)22:11
dansmithmelwitt: I'm suddenly terrified that that may be related to the never-ending "SSH timeout" bugs22:11
*** Jeffrey4l has joined #openstack-nova22:11
dansmithif so, you're an instant legend22:11
*** otter768 has joined #openstack-nova22:11
flwang1dansmith: ok, thanks, i will dig it22:15
*** VW_ has quit IRC22:16
*** otter768 has quit IRC22:16
openstackgerritPraveen Yalagandula proposed openstack/nova: neutron api: removing an obsolete comment to avoid confusion  https://review.openstack.org/16601922:17
dansmithmelwitt: it's probably because there is only one thing, and we just saved it, so it doesn't seem like it should have changed22:17
dansmithmelwitt: so probably not as severe as I was thinking, but definitely not right22:18
melwittdansmith: yeah, I was thinking that22:18
*** belmoreira has quit IRC22:18
dansmithmelwitt: i.e. good for fixing22:18
*** nelsnelson has quit IRC22:18
*** salv-orlando has joined #openstack-nova22:19
*** salv-orl_ has quit IRC22:23
*** dsanders1 has joined #openstack-nova22:24
*** dsanders has quit IRC22:27
dansmithalaski: I'm very unforgiving with my definitions of things like "decent" as you can tell22:27
* dansmith takes some xanex22:27
alaskidansmith: I am now keenly aware22:27
dansmith:)22:27
openstackgerritAndrew Laski proposed openstack/nova-specs: RequestSpec object split instance_properties  https://review.openstack.org/17608322:28
alaskidansmith: it was a good discussion though and some of it should probaly get transcribed into the spec22:29
dansmithalaski: got it22:29
*** fazal_ has joined #openstack-nova22:29
dansmithalaski: I guess I thought I had made it clear enough22:29
alaskidansmith: thanks22:29
*** Nic has quit IRC22:30
dansmithalaski: I'll add some more scary words22:30
dansmithwhen I fix my typos22:30
fazal_I have one routable public IP which I have assigned to my control node to access the dashboard and also the novnc consoles of the VMs. I can access the dashboard on the Public IP but not the novnc consoles. when i open the novnc of any VM, it gets directed to my <Public IP>:6080/vnc_auto.html but that doesn't come up. What should I do?22:30
dansmithfazal_: please see topic22:31
fazal_sorry. didn't see it.22:32
*** Jeffrey4l has quit IRC22:33
alaskidansmith: scary words are good.  what I took away from our parley was that there are unintentional things that could cause it, which isn't really called out22:34
alaskilike even if you're careful you could have bad things happen22:34
dansmithalaski: yeah, that's the main point, so clearly I need clarity22:34
dansmithyeah22:34
dansmithI also wrote that in the middle of the big discussion we had on it, so I probably assumed the reader would have the context22:35
alaskiyeah.  it's scary, but the uninformed reader should need to change their pants afterwards22:37
*** dsanders1 has quit IRC22:37
alaskiand +1 for robustify22:37
*** ZZelle_ has joined #openstack-nova22:37
dansmithalaski: "swap disks and reboot -> delete everything" should be pants-soiling22:38
dansmithheh22:38
*** dsanders has joined #openstack-nova22:38
mgagneI'm reading this spec: http://specs.openstack.org/openstack/nova-specs/specs/kilo/approved/metadata-service-network-info.html and its proposed implementation: https://review.openstack.org/#/c/102649/2022:40
mgagneIs the network id supposed to be unique across all networks irregardless of IP version? Or is the actual network identifier a composite of network id and network type?22:40
*** melwitt has quit IRC22:41
*** melwitt has joined #openstack-nova22:41
*** redbeard has quit IRC22:42
*** Jeffrey4l has joined #openstack-nova22:45
*** josecastroleon has joined #openstack-nova22:46
*** josecastroleon has quit IRC22:48
*** joefides has joined #openstack-nova22:49
*** dboik has quit IRC22:51
*** dboik has joined #openstack-nova22:51
*** ZZelle_ has quit IRC22:53
*** ndipanov has quit IRC22:54
*** Raildo has joined #openstack-nova22:57
*** penick has quit IRC22:59
*** Raildo is now known as raildo23:00
*** patrickeast has quit IRC23:03
*** signed8bit has quit IRC23:05
*** ildikov has quit IRC23:08
*** patrickeast has joined #openstack-nova23:09
*** joefides_ has joined #openstack-nova23:09
jogodansmith: any reason not to +W this https://review.openstack.org/#/c/174480/23:09
dansmithjogo: yeah, hoping to get a clean run from t-h when jhesketh wakes up23:10
*** bkopilov has quit IRC23:10
jogodansmith: ahh added a comment23:10
dansmithjogo: thanks23:11
jheskethdansmith, jogo: I'll take a look in a minute, it's high on my list :-)23:11
dansmithjhesketh: thanks23:11
*** joefides has quit IRC23:12
*** Sukhdev has quit IRC23:14
*** kmartin_ has quit IRC23:17
*** kmartin has joined #openstack-nova23:17
*** baoli has quit IRC23:20
*** penick has joined #openstack-nova23:21
*** Raildo_ has joined #openstack-nova23:21
*** ildikov has joined #openstack-nova23:22
*** bkopilov has joined #openstack-nova23:25
*** raildo has quit IRC23:25
*** packet has quit IRC23:25
*** Raildo_ has quit IRC23:27
*** penick has quit IRC23:30
*** melwitt has quit IRC23:33
*** marun has quit IRC23:39
*** emagana has quit IRC23:40
*** dsanders has quit IRC23:47
*** dims has quit IRC23:50
*** annashen has joined #openstack-nova23:52
*** moshele has quit IRC23:57

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