Thursday, 2015-12-03

openstackgerritDavid Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/25266400:03
*** zhangjn has joined #puppet-openstack00:03
*** dfisher has quit IRC00:06
*** nlahouti has quit IRC00:08
*** RichardRaseley has joined #puppet-openstack00:09
*** zhangjn has quit IRC00:13
*** nlahouti has joined #puppet-openstack00:14
*** ducttape_ has quit IRC00:15
*** baker has joined #puppet-openstack00:21
*** xingchao has joined #puppet-openstack00:22
openstackgerritDavid Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/25266400:26
*** xingchao has quit IRC00:26
*** xarses has quit IRC00:36
*** rcernin has quit IRC00:37
openstackgerritDavid Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/25266400:44
*** dgurtner has quit IRC00:48
*** zhangjn has joined #puppet-openstack00:55
*** aimon has quit IRC00:57
openstackgerritClayton O'Neill proposed openstack/puppet-nova: Add hooks for external install & svc management  https://review.openstack.org/25269300:58
*** RichardRaseley has quit IRC01:01
*** xingchao has joined #puppet-openstack01:02
*** xarses has joined #puppet-openstack01:03
*** xarses has quit IRC01:03
*** xarses has joined #puppet-openstack01:03
*** xarses has quit IRC01:20
*** markvoelker has quit IRC01:23
*** rahulait has joined #puppet-openstack01:24
*** aimon has joined #puppet-openstack01:29
*** xingchao has quit IRC01:31
*** davideagle_ has quit IRC01:35
*** nlahouti has quit IRC01:35
openstackgerritClayton O'Neill proposed openstack/puppet-nova: Add hooks for external install & svc management  https://review.openstack.org/25269301:35
*** xingchao has joined #puppet-openstack01:36
*** xarses has joined #puppet-openstack01:42
*** nlahouti has joined #puppet-openstack02:07
*** nlahouti has quit IRC02:07
*** mdorman has quit IRC02:10
*** mdorman has joined #puppet-openstack02:38
*** zhangjn has quit IRC02:51
openstackgerritClayton O'Neill proposed openstack/puppet-nova: Add hooks for external install & svc management  https://review.openstack.org/25269302:54
openstackgerritClayton O'Neill proposed openstack/puppet-nova: Only require netaddr when needed  https://review.openstack.org/25272002:54
*** zhangjn has joined #puppet-openstack02:55
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: (WIP) Add support for different module paths  https://review.openstack.org/24831603:01
*** mdorman has quit IRC03:04
openstackgerritCody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/24940903:04
*** vinsh has quit IRC03:17
*** jerrygb has quit IRC03:19
*** ducttape_ has joined #puppet-openstack03:29
*** rahulait has quit IRC03:36
*** xarses_ has joined #puppet-openstack03:36
*** xarses has quit IRC03:38
*** baker has quit IRC03:38
*** morazi has quit IRC03:42
openstackgerritClayton O'Neill proposed openstack/puppet-openstack-specs: Enhance inifile option deprecations spec  https://review.openstack.org/24051504:00
*** xingchao_ has joined #puppet-openstack04:09
*** xingchao has quit IRC04:09
*** zhangjn has quit IRC04:12
*** ducttape_ has quit IRC04:12
*** jerrygb has joined #puppet-openstack04:20
*** jerrygb has quit IRC04:26
*** gildub has joined #puppet-openstack04:34
*** zhangjn has joined #puppet-openstack04:45
*** gildub has quit IRC04:46
*** zhangjn has quit IRC04:57
*** jamespage has quit IRC05:00
*** jamespage has joined #puppet-openstack05:00
*** mflobo has quit IRC05:18
*** markvoelker has joined #puppet-openstack05:25
*** zhangjn has joined #puppet-openstack05:26
*** zhangjn has quit IRC05:26
*** zhangjn has joined #puppet-openstack05:27
*** markvoelker_ has joined #puppet-openstack05:28
*** markvoelker has quit IRC05:30
*** markvoelker has joined #puppet-openstack05:31
*** markvoelker_ has quit IRC05:34
*** mflobo has joined #puppet-openstack05:37
*** markvoelker_ has joined #puppet-openstack05:38
*** markvoelker has quit IRC05:38
*** mflobo has quit IRC05:42
*** markvoelker_ has quit IRC05:51
*** saneax_AFK is now known as saneax05:58
*** jerrygb has joined #puppet-openstack06:22
*** jerrygb has quit IRC06:26
*** karmix has quit IRC06:30
*** karmix has joined #puppet-openstack06:31
*** nlahouti has joined #puppet-openstack06:34
*** nlahouti has quit IRC06:50
*** markvoelker has joined #puppet-openstack06:50
*** mflobo has joined #puppet-openstack06:51
*** xingchao has joined #puppet-openstack06:51
*** xingchao_ has quit IRC06:53
*** xarses_ has quit IRC06:55
*** markvoelker has quit IRC06:55
*** xarses has joined #puppet-openstack06:57
*** xarses has quit IRC06:57
*** nlahouti has joined #puppet-openstack06:57
*** xarses has joined #puppet-openstack06:58
*** xarses has quit IRC06:58
*** xarses has joined #puppet-openstack06:59
*** xarses_ has joined #puppet-openstack07:00
*** bkero has quit IRC07:01
*** bkero has joined #puppet-openstack07:02
*** xarses has quit IRC07:03
*** bkero has quit IRC07:10
*** bkero has joined #puppet-openstack07:10
*** nlahouti1 has joined #puppet-openstack07:10
*** nlahouti has quit IRC07:12
*** rcernin has joined #puppet-openstack07:19
*** rcernin_ has joined #puppet-openstack07:24
*** nlahouti1 has quit IRC07:31
*** nlahouti has joined #puppet-openstack07:34
*** nlahouti has quit IRC07:37
*** nlahouti has joined #puppet-openstack07:53
*** migi has quit IRC07:54
*** migi has joined #puppet-openstack07:55
*** nlahouti has quit IRC07:55
*** zhangjn has quit IRC07:57
*** beddari has quit IRC07:57
*** beddari has joined #puppet-openstack08:01
*** zhangjn has joined #puppet-openstack08:14
*** mgrosz has joined #puppet-openstack08:15
mgroszHello, can someone please review new puppet-vitrage init project patch https://review.openstack.org/#/c/252214/? Thanks08:15
mgroszSomeone is online?08:21
*** markvoelker has joined #puppet-openstack08:25
*** jpena has joined #puppet-openstack08:27
*** foxhunt_away is now known as foxhunt7208:28
*** markvoelker has quit IRC08:30
openstackgerritMatt Popow proposed openstack/puppet-cinder: Add a delimiter for cinder type-list grep check. Hyphens will be matched as a word boundry, so add a space to avoid false positives. For example, with an exisitng "demo-type1" we can't create a new "type1".  https://review.openstack.org/25282108:38
openstackgerritMatt Popow proposed openstack/puppet-cinder: Add a delimiter for cinder type-list grep check.  https://review.openstack.org/25282108:41
openstackgerritMatt Popow proposed openstack/puppet-cinder: Add a delimiter for cinder type-list grep check.  https://review.openstack.org/25282108:43
*** dgurtner has joined #puppet-openstack08:49
*** dgurtner has quit IRC08:49
*** dgurtner has joined #puppet-openstack08:49
*** rcernin has quit IRC08:50
*** rcernin_ has quit IRC08:51
*** mgrosz has quit IRC08:51
*** mgrosz has joined #puppet-openstack08:53
openstackgerritDavid Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/25266408:53
openstackgerritJavier Peña proposed openstack/puppet-neutron: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24522909:08
openstackgerritJavier Peña proposed openstack/puppet-neutron: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24522909:10
openstackgerritJavier Peña proposed openstack/puppet-heat: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24571509:11
*** mgrosz has quit IRC09:12
openstackgerritJavier Peña proposed openstack/puppet-manila: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24572109:13
*** mgrosz has joined #puppet-openstack09:14
openstackgerritJavier Peña proposed openstack/puppet-ironic: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24589209:14
*** paramite has joined #puppet-openstack09:41
*** zhangjn has quit IRC09:45
*** mgrosz has quit IRC09:57
*** degorenko has joined #puppet-openstack09:59
*** rcernin has joined #puppet-openstack09:59
*** zhangjn has joined #puppet-openstack09:59
*** IvanBerezovskiy has joined #puppet-openstack10:03
openstackgerritIvan Berezovskiy proposed openstack/puppet-glance: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24692610:05
openstackgerritIvan Berezovskiy proposed openstack/puppet-ceilometer: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24746710:07
*** jistr has joined #puppet-openstack10:09
*** derekh has joined #puppet-openstack10:10
openstackgerritMerged openstack/puppet-murano: Add ability to deploy Murano CloudFoundry service broker API  https://review.openstack.org/24504310:13
*** mgrosz has joined #puppet-openstack10:17
*** mgrosz has quit IRC10:18
*** markvoelker has joined #puppet-openstack10:26
*** markvoelker has quit IRC10:31
openstackgerritDavid Gurtner proposed openstack/puppet-ceph: Split the Rados GW tests into parts  https://review.openstack.org/25266410:33
*** yottatsa has joined #puppet-openstack10:34
*** derekh has quit IRC10:40
socialEmilienM: can you also have look at https://review.openstack.org/#/c/221273/ ?10:43
*** mgrosz has joined #puppet-openstack10:43
*** dgurtner has quit IRC10:47
*** dgurtner has joined #puppet-openstack10:48
*** xingchao has quit IRC10:59
*** derekh has joined #puppet-openstack11:03
*** jayg|g0n3 has quit IRC11:03
*** mgrosz has quit IRC11:04
*** gfidente has joined #puppet-openstack11:04
*** gfidente has quit IRC11:04
*** gfidente has joined #puppet-openstack11:04
*** jayg|g0n3 has joined #puppet-openstack11:06
*** yottatsa has quit IRC11:08
*** rochaporto has joined #puppet-openstack11:09
*** yottatsa has joined #puppet-openstack11:16
*** kylo has quit IRC11:17
*** kylo has joined #puppet-openstack11:18
*** mgrosz has joined #puppet-openstack11:25
iurygregorygood morning =)11:27
*** mgrosz has quit IRC11:31
mkarpinHey!11:35
*** xingchao has joined #puppet-openstack11:49
*** xingchao has quit IRC11:53
openstackgerritMerged openstack/puppet-nova: Extend nova quota support  https://review.openstack.org/24967111:55
degorenkoclayton, hey o/ can you please merge https://review.openstack.org/250741 and https://review.openstack.org/250288 ?12:04
degorenkocrinkle, spredzy sbadia &&& guys, please :)12:04
*** derekh has quit IRC12:04
degorenkoand this one: https://review.openstack.org/22989512:05
iurygregoryhttps://review.openstack.org/#/c/252536/ this one too XD12:05
*** cwolferh has quit IRC12:06
claytonlooking, you can review https://review.openstack.org/#/c/252720/ for me12:06
*** clayton sets mode: +o degorenko12:07
degorenkoclayton, nice catch!12:08
claytonwell, it broke a patch of mine :)12:08
degorenkoi guess, next patch with hooks? :)12:09
claytonyeap12:09
degorenkolooking12:09
claytonyou're welcome to review that one too, but it's huge12:09
*** derekh has joined #puppet-openstack12:17
degorenkoclayton, few comments after first quick review :)12:23
degorenkodo not forget please about this cherry pick please :) https://review.openstack.org/25074112:24
*** paramite is now known as paramite|afk12:24
*** degorenko is now known as degorenko|afk12:24
*** paramite|afk is now known as paramite12:26
*** markvoelker has joined #puppet-openstack12:27
openstackgerritClayton O'Neill proposed openstack/puppet-nova: Add hooks for external install & svc management  https://review.openstack.org/25269312:28
openstackgerritMerged openstack/puppet-horizon: Fix wrong use of os_package_type  https://review.openstack.org/25028812:30
*** zhangjn has quit IRC12:30
*** paramite is now known as paramite|afk12:30
*** markvoelker has quit IRC12:32
*** openstackgerrit has quit IRC12:32
*** openstackgerrit has joined #puppet-openstack12:33
*** zhangjn has joined #puppet-openstack12:36
openstackgerritMerged openstack/puppet-glance: Update Glance Swift backend configuration  https://review.openstack.org/25074112:41
EmilienMgood morning12:44
*** yottatsa has quit IRC12:44
EmilienMsocial: sure12:44
*** dprince has joined #puppet-openstack12:44
openstackgerritMerged openstack/puppet-openstacklib: Catch HTTP 403 response  https://review.openstack.org/25147712:44
*** iurygregory has left #puppet-openstack12:45
*** iurygregory has joined #puppet-openstack12:45
openstackgerritMerged openstack/puppet-openstack_extras: Fix installation instruction  https://review.openstack.org/24837512:45
mkarpinneed core review here https://review.openstack.org/#/c/237035/ please! thanks12:46
mkarpindegorenko,crinkle, mfisch, sbadia, spredzy : ^^12:47
*** ducttape_ has joined #puppet-openstack12:47
openstackgerritMerged openstack/puppet-glance: Stop collecting glance services by name  https://review.openstack.org/24063012:51
openstackgerritMerged openstack/puppet-horizon: Remove installation of lesscpy package  https://review.openstack.org/24816212:52
openstackgerritMerged openstack/puppet-nova: Remove docs for unused cluster_id parameter  https://review.openstack.org/24389612:56
*** degorenko|afk is now known as degorenko12:58
degorenkoEmilienM, morning :)12:58
iurygregoryhey denis can you take a look https://review.openstack.org/#/c/252536/12:59
iurygregory=)12:59
degorenkomkarpin, what about second part of $::os_service_default for heat? :)12:59
degorenkoiurygregory, will look13:00
mkarpindegorenko: its ready to be reviewed also :)13:00
degorenkomkarpin, link :)13:01
mkarpinhttps://review.openstack.org/#/c/249711/13:01
*** trown|outttypeww is now known as trown13:05
degorenkomkarpin, -1 for 23703513:05
degorenkotake a look13:05
degorenkomkarpin, for heat os_service_default also -1 :) you forgot about init.pp class13:09
degorenkowe should have two patches :) as it was discussed on summit :) first for ensure => absent and second for all another13:09
degorenkoit can be one common, if module is not so big13:09
iurygregorydenis https://github.com/openstack/puppet-heat/blob/master/manifests/init.pp init already have $::os_service_default13:10
mkarpindegorenko: i switched all  what i thought can be switched to os_service_default in init.pp in patch part113:10
degorenkonot all parameters13:10
iurygregoryoh ^^13:11
mkarpinall other i think is different from default values in docs13:11
*** saneax is now known as saneax_AFK13:11
mkarpinbut i will double check13:11
degorenkoyes please, if all parameters are ok - then i will +2A13:13
degorenkoiurygregory, why you didn't change logging and db classes for magnum?13:14
*** ducttape_ has quit IRC13:14
iurygregorysure i'll update =)13:16
*** openstackgerrit has quit IRC13:17
*** openstackgerrit has joined #puppet-openstack13:18
*** jayg|g0n3 is now known as jayg13:20
*** jerrygb has joined #puppet-openstack13:21
mkarpindegorenko: ok, will do, thanks for review!13:24
*** markvoelker has joined #puppet-openstack13:28
*** zhangjn has quit IRC13:28
*** zhangjn has joined #puppet-openstack13:30
*** markvoelker has quit IRC13:32
*** zhangjn has quit IRC13:33
openstackgerritClayton O'Neill proposed openstack/puppet-monasca: Remove obsolete tenant parameter  https://review.openstack.org/25295413:34
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-magnum: add @default_facts to magnum  https://review.openstack.org/25253613:38
*** morazi has joined #puppet-openstack13:39
*** jpena is now known as jpena|lunch13:48
*** markvoelker has joined #puppet-openstack13:50
*** markvoelker_ has joined #puppet-openstack13:52
openstackgerritMerged openstack/puppet-nova: Only require netaddr when needed  https://review.openstack.org/25272013:54
*** markvoelker has quit IRC13:54
*** bdossant has joined #puppet-openstack13:56
bdossantHi! I'm trying to run unit tests locally with "bundle exec rake spec" but i get /root/puppet-nova/openstack/puppet-openstack-integration/functions: line 51: /usr/local/share/gems/bin/r10k: No such file or directory13:57
bdossantcan anyone point me in the right direction?13:57
*** zhangjn has joined #puppet-openstack13:57
iurygregorywhen i get this i remove my directory with gems and do bundle install again13:57
bdossant"bundle exec rspec spec/acceptance" requires a vagrant node, can i run this locally?13:57
*** zhangjn has quit IRC13:57
iurygregoryif you have everything you can =)13:58
bdossantok13:58
*** zhangjn has joined #puppet-openstack13:58
iurygregoryi only run bundle exec rspec spec13:58
bdossantso remove "/usr/local/share/gems/bin" ?13:58
iurygregoryno XD13:59
bdossantbut do you have vagrant on your box?13:59
iurygregoryin you puppet-<module>13:59
iurygregorydirectory13:59
iurygregoryyou have .bundle_gems ?13:59
*** zhangjn has quit IRC13:59
iurygregorybdossant, http://my1.fr/blog/puppet-module-functional-testing-with-vagrant-openstack-and-beaker/ take a look at the shell script here =)14:00
*** saneax_AFK is now known as saneax14:00
*** zhangjn has joined #puppet-openstack14:00
*** zhangjn has quit IRC14:00
*** zhangjn has joined #puppet-openstack14:01
bdossantthnaks14:02
*** zhangjn has quit IRC14:02
bdossanti thought i could run this locally14:02
iurygregoryi run locally on my virtual machine o.o14:02
bdossantah14:03
bdossantso i just need the second script14:04
bdossantif i install vagrant in my vm14:04
iurygregoryi just use the shell script in my virtual machine and it works =( but you can set up vagrant very easily14:05
*** RichardRaseley has joined #puppet-openstack14:09
*** paramite|afk is now known as paramite14:09
*** zhangjn has joined #puppet-openstack14:10
*** yottatsa has joined #puppet-openstack14:11
socialIvanBerezovskiy: thanks, looks like I should take some typing lessons14:14
*** rochaporto has quit IRC14:28
*** bdossant_ has joined #puppet-openstack14:30
*** bdossant has quit IRC14:31
*** zhangjn has quit IRC14:36
*** saneax is now known as saneax_AFK14:38
*** zhangjn has joined #puppet-openstack14:44
*** xingchao has joined #puppet-openstack14:47
*** derekh has quit IRC14:50
*** bdossant_ has quit IRC14:53
*** derekh has joined #puppet-openstack14:53
*** bdossant has joined #puppet-openstack14:54
*** dfisher has joined #puppet-openstack14:57
*** ducttape_ has joined #puppet-openstack15:00
*** jpena|lunch is now known as jpena15:00
*** vinsh has joined #puppet-openstack15:00
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: scenario002: enable application catalog service  https://review.openstack.org/25043615:02
*** xarses_ has quit IRC15:03
*** dhollinger has joined #puppet-openstack15:04
openstackgerritLukas Bezdicka proposed openstack/puppet-trove: Remove trove ubuntu package hack  https://review.openstack.org/22127315:05
iurygregorydegorenko, it's done ;) https://review.openstack.org/#/c/252536/15:07
EmilienMaderyugin: I dropped a comment on https://review.openstack.org/#/c/250436/ - I'm happy to see more testing !!!15:10
*** karimb has joined #puppet-openstack15:11
*** bdossant has quit IRC15:16
*** bdossant has joined #puppet-openstack15:18
*** ntpttr has joined #puppet-openstack15:20
degorenkoiurygregory, just a little comment - update please commit message :)15:22
degorenkoand title15:23
iurygregoryoh sorry XD15:23
degorenkonow your patch is fully switch os_service_defualt :)15:23
degorenkoEmilienM, spredzy https://review.openstack.org/#/c/237970/ take a look please :) and this one https://review.openstack.org/24698815:24
degorenkovinsh, https://review.openstack.org/246988 i've updated patch :) take a look please15:24
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-magnum: Switch magnum to $::os_service_default  https://review.openstack.org/25253615:24
EmilienMdegorenko: ok15:24
iurygregorydegorenko, done15:25
*** markvoelker_ has quit IRC15:25
degorenkoiurygregory, +2 :)15:25
EmilienMdegorenko: glance::api::db ?15:25
EmilienMit should be glance::db15:25
EmilienMwhy separating it?15:25
degorenkoEmilienM, we have separated options15:26
degorenkofor glance_api15:26
degorenkoand for glance_registry15:26
degorenkohttps://github.com/openstack/puppet-glance/blob/master/manifests/api.pp#L268-L28315:26
degorenkohttps://github.com/openstack/puppet-glance/blob/master/manifests/registry.pp#L191-L20615:27
degorenkoEmilienM, ^15:27
EmilienMdegorenko: ok15:27
EmilienMI'll review them today15:27
openstackgerritMerged openstack/puppet-nova: Create networks only if nova-network is enabled  https://review.openstack.org/22989515:30
trownEmilienM: the selinux module just returns nil if there is no selinux support https://github.com/puppetlabs/puppet/blob/master/lib/puppet/util/selinux.rb#L8515:32
EmilienMtrown: mhh15:32
EmilienMso I guess we can go ahead with your patch15:32
trownEmilienM: so I think we do not need a conditional to use seltype ... and a conditional inside of file is pretty gross15:32
EmilienMok15:32
EmilienMtrown: +215:33
trownEmilienM: awesome. thanks!15:33
mkarpindegorenko: I answered on your comment https://review.openstack.org/#/c/237035/ , so what do you think?15:35
openstackgerritAlexey Deryugin proposed openstack/puppet-tempest: Allow to activate Murano Service Broker tests  https://review.openstack.org/25301915:35
degorenkomkarpin, looks like hardcode :)15:36
vinshdegorenko: Will do15:36
degorenkowe finally need to switch glance to os_service_default15:37
degorenkospredzy, do you working at this? ^^^15:37
*** derekh has quit IRC15:39
*** xarses_ has joined #puppet-openstack15:39
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: scenario002: enable application catalog service  https://review.openstack.org/25043615:41
*** baker has joined #puppet-openstack15:41
EmilienMaderyugin: maybe we can align on https://review.openstack.org/#/c/253019/1/manifests/init.pp,cm ?15:42
degorenkoEmilienM, i have news for you :) oslo.messaging dropped support of qpid :)15:42
EmilienMdegorenko: yeah?15:42
degorenkoi will create a card for it and update our manifests15:42
degorenkoEmilienM, yeah :)15:43
EmilienMdegorenko: that means openstack does not support qpid at all?15:43
degorenkoi guess yes15:43
degorenkofrom mitaka15:43
*** chem has joined #puppet-openstack15:43
*** chem`` has quit IRC15:44
degorenkoEmilienM, https://github.com/openstack/oslo.messaging/commit/925eb734a9d3cb46bb89a89ec1a78281d2d7afe915:44
*** chem has left #puppet-openstack15:45
EmilienMdegorenko: thanks15:45
*** derekh has joined #puppet-openstack15:45
degorenkoi will upload changes :)15:45
*** xarses_ is now known as xarses15:45
degorenkoEmilienM, fyi, progress will be here: https://trello.com/c/nQe9qrkt/76-remove-support-of-qpid15:46
EmilienMdegorenko: I suggest we send a warning if QPID options are set, and backport it in stable/liberty15:46
EmilienMso people using stable/liberty have 5 months to move to something else but qpid15:46
degorenkoEmilienM, of course, i will deprecated all qpid for mitaka15:46
degorenkoand liberty15:46
EmilienMwe need also to deprecate parameters15:46
EmilienMdegorenko: you awesome15:46
degorenko:D15:46
degorenkoEmilienM, as i said on proposing me to core: will do my best ;)15:47
EmilienM:)15:48
claytonso is keystone v2 just not supported by puppet-keystone anymore?15:53
claytonI'm trying to upgrade to 6.1.0 of the module, and everything appears to be hard coded for Puppet::Provider::Openstack::CredentialsV3 now15:54
*** markvoelker has joined #puppet-openstack15:55
*** RichardRaseley has quit IRC15:59
*** skolekonov is now known as _skolekonov16:02
claytonit is always setting OS_IDENTITY_API_VERSION=3 and there doesn't appear to be a way to override that.16:06
EmilienMgood question, you should be able to use v2 iirc16:07
claytonI can't figure out how16:07
EmilienMchem and richm are not here ...16:08
claytonversion is set using this "self.class.to_s.sub(/.*V/,'').sub('_','.')"16:08
claytonand it's hardcoded to use CredentialsV316:08
*** xingchao has quit IRC16:08
*** rcernin has quit IRC16:12
EmilienMclayton: I think you can still use v2 somehow, iirc our functional tests used to test both16:13
claytonwell, I'd love to be wrong, this has me dead in the water on upgrading16:13
EmilienMhttps://github.com/openstack/puppet-keystone/blob/stable/kilo/spec/acceptance/keystone_wsgi_apache_spec.rb#L14216:13
EmilienMclayton: this shows we run tests against v2 too16:14
clayton@EmilienM this is specifically for keystone_user16:14
EmilienMyou need to specifcy --os-identity-api-version 216:14
claytonsorry, I should have been more clear16:14
EmilienMok16:14
EmilienMsorry I missed that16:14
EmilienMI'll ping richm to make sure16:14
openstackgerritDenis Egorenko proposed openstack/puppet-aodh: Deprecate qpid messaging parameters  https://review.openstack.org/25304216:17
openstackgerritDenis Egorenko proposed openstack/puppet-ceilometer: Deprecate qpid messaging parameters  https://review.openstack.org/25304316:17
*** bdossant has quit IRC16:18
*** chem has joined #puppet-openstack16:21
iurygregoryEmilienM, https://review.openstack.org/252536 o/ sorry i've update the commit message16:27
EmilienMchem: you were offline but clayton mentionned he can't use keystone v2 for keystone_user provider in 6.1.0 (kilo release) - could you check that?16:27
EmilienMlogs: http://irclog.perlgeek.de/puppet-openstack/16:28
chemEmilienM: thanks :)16:28
*** paramite has quit IRC16:35
*** foxhunt72 is now known as foxhunt_away16:36
claytonmy issue specifically is that I had a /root/openrc that existed, but had auth_url set to the v2 endpoint16:37
*** zhangjn has quit IRC16:38
claytonbut the keystone_user provider was explicitly setting OS_API_VERSION to 316:38
claytonwhich caused openstackclient to barf16:38
openstackgerritNate Potter proposed openstack/puppet-cinder: Remove references to puppet-openstack from README  https://review.openstack.org/24837916:38
chemclayton: I'm having a look16:39
openstackgerritTim Rozet proposed openstack/puppet-neutron: Adds configuration support for OpenDaylight SDN Controller  https://review.openstack.org/24939016:39
*** aglarendil has quit IRC16:40
*** aglarendil has joined #puppet-openstack16:40
*** ddmitriev has joined #puppet-openstack16:43
openstackgerritDenis Egorenko proposed openstack/puppet-heat: Deprecate qpid messaging parameters  https://review.openstack.org/25306216:50
*** rcernin has joined #puppet-openstack16:53
chemclayton: there is nothing specify about the keystone_user provider 6.1.0, all the other keystone provider (tenant, user_role, ...) are using CredentialsV3.  Did I misunderstand something here /16:53
chem?16:53
claytonthe keystone_user provider requires v3 api be used, right?16:54
chemclayton: all of them use CredentialsV316:54
chemclayton: but they also parse the $HOME/openrc for auth info16:54
claytonthat was a change in 6.0.0 that wasn't mentioned in the release notes16:54
chemclayton: I think the problem may be comming from your /root/openrc in a case that was not foreseen16:56
EmilienMchem: clayton wants to use v2, that's the point.16:56
chemEmilienM: I got this :)16:56
EmilienMsorry :)16:56
claytonwell, I'm going to just remove the openrc and move on, just kind of annoyed the failure mode is so obscure and the change wasn't documented16:56
openstackgerritDenis Egorenko proposed openstack/puppet-ironic: Deprecate qpid messaging parameters  https://review.openstack.org/25306616:57
chemclayton: as I said, it was not foreseen that an openrc would have the v2 info.  I could debug the stuff and try to backport the change if I find the source of the problem.  The module is supposed to be backward comptatible, that's just something that has been missed.16:58
*** nlahouti has joined #puppet-openstack16:58
chemclayton: could you paste me the openrc (without the real creadential) so I invistage further16:58
chem?16:58
claytonpersonally I've never really understood why it reads from /root/openrc16:58
*** cwolferh has joined #puppet-openstack16:59
EmilienMclayton: I wrote this release note I think - so that's my fault probably :(16:59
claytonEmilienM have you been following the stuff for reno?16:59
EmilienMyou're right, it's not really clear in https://github.com/openstack/puppet-keystone/blob/stable/kilo/CHANGELOG.md16:59
EmilienMclayton: no17:00
EmilienMah yes I did17:00
EmilienMin fact17:00
claytonreno is the new tool all the core openstack projects are switching to for release notes in mitaka17:00
*** holser_ has quit IRC17:00
claytonseems like a good approach, or at least a big improvement17:00
*** nlahouti has quit IRC17:00
claytonEmilienM it's just hard to do release notes after the fact17:00
*** holser_ has joined #puppet-openstack17:00
iurygregoryEmilienM it's not only your fault is our fault too because we reviewed =)17:01
EmilienMclayton: I'll look at reno and see if our project can use it17:01
*** baker has quit IRC17:02
EmilienMclayton: https://trello.com/c/upZVtMps/77-look-at-reno-for-release-notes17:02
chemIf it has happened to clayton that may be happening to others.  I'm going to debug (openrcv2 and puppet-keystone-6.1.0) this and check what we can do.  That is, I think, a bug.17:03
*** xingchao has joined #puppet-openstack17:04
claytonchem: https://gist.github.com/dvorak/32a85fa6ab0d6cbbe56b17:05
openstackgerritDenis Egorenko proposed openstack/puppet-manila: Deprecate qpid messaging parameters  https://review.openstack.org/25307117:05
chemclayton: thanks17:05
*** nlahouti has joined #puppet-openstack17:05
*** xingchao has quit IRC17:08
EmilienMclayton: I think reno is really worth to use it for us17:10
EmilienMmwhahaha: could you reconsider https://review.openstack.org/252562 please?17:15
*** yottatsa has quit IRC17:16
openstackgerritDenis Egorenko proposed openstack/puppet-mistral: Deprecate qpid messaging parameters  https://review.openstack.org/25308517:17
*** RichardRaseley has joined #puppet-openstack17:19
*** baker has joined #puppet-openstack17:21
*** jistr has quit IRC17:21
*** yottatsa has joined #puppet-openstack17:25
openstackgerritDenis Egorenko proposed openstack/puppet-neutron: Deprecate qpid messaging parameters  https://review.openstack.org/25309017:26
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Deprecate qpid messaging parameters  https://review.openstack.org/25309717:31
*** trown is now known as trown|lunch17:31
*** jpena has quit IRC17:33
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Remove deprecated parameters since Liberty release  https://review.openstack.org/25309917:33
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Deprecate qpid messaging parameters  https://review.openstack.org/25310517:42
openstackgerritDenis Egorenko proposed openstack/puppet-trove: Deprecate qpid messaging parameters  https://review.openstack.org/25311617:57
*** degorenko is now known as _degorenko|afk17:59
*** yottatsa has quit IRC17:59
*** karimb has quit IRC18:02
*** derekh has quit IRC18:04
*** trown|lunch is now known as trown18:09
*** aimon has quit IRC18:11
chemclayton: Just two quick questions: are you upgrading from 5.2.0 (juno) or 6.0.0, and does  curl http://127.0.0.1:5000/v3 works on your setup ?  I'm trying to reproduce the problem and wouldn't want to make wrong assumption.18:12
claytonv3 does actually work for this18:13
claytonso removing the openrc resolved the issue for now18:13
chemclayton: oki, I could reproduce the bug.18:18
chemclayton: checking what's wrong with this now :)18:18
*** davideagle_ has joined #puppet-openstack18:21
chemclayton: Oki, I got the idea.  The keystone_user is special in that for checking the password the classic CredentialV3 class is bypassed and a direct 'token issue' is made.  That's where the problem is and that's why it happen only with keystone_user.  I'm filling the bug now and will fix it (hopefully) tomorrow.18:25
claytoncool, thanks for looking at it18:27
*** nlahouti has quit IRC18:30
iurygregoryclayton, can you look https://review.openstack.org/#/c/252536/ ? ^^18:30
*** holser_ has quit IRC18:31
*** holser_ has joined #puppet-openstack18:33
*** nlahouti has joined #puppet-openstack18:37
*** ericksonsantos has quit IRC18:38
*** iurygregory has quit IRC18:38
*** davideagle_ has quit IRC18:45
*** iurygregory has joined #puppet-openstack18:45
*** ericksonsantos has joined #puppet-openstack18:51
*** aimon has joined #puppet-openstack18:54
*** dmsimard is now known as dmsimard|afk19:01
*** davideagle has joined #puppet-openstack19:02
chemclayton: https://bugs.launchpad.net/puppet-keystone/+bug/152254119:02
openstackLaunchpad bug 1522541 in puppet-keystone "Having /root/openrc filled with v2 auth_url makes keystone_user fails." [Undecided,New]19:02
*** RichardRaseley has quit IRC19:03
openstackgerritMerged openstack/puppet-nova: Allow customization of db sync command line  https://review.openstack.org/24100919:04
*** derekh has joined #puppet-openstack19:09
openstackgerritMerged openstack/puppet-heat: Allow customization of db sync command line  https://review.openstack.org/24129919:11
*** derekh has quit IRC19:11
chemclayton: you're bumping into a double bug in your case as your auth_url ends with '/' which make the overall url http://127.0.0.1:5000/v2.0//v3 (do not happen anymore in 7.0.0)19:12
chemEmilienM: should we make another bug to this version only ^ ?19:12
EmilienMchem: if it's a different patch, yes I think19:13
chemEmilienM: ack19:13
*** bnemec has quit IRC19:16
*** dgurtner has quit IRC19:18
*** bnemec has joined #puppet-openstack19:22
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Keystone_user fails with v2 openrc.  https://review.openstack.org/25315719:29
chemclayton: if you remove the last "/" from your openrc this will fix the issue ^19:30
chemEmilienM: how do I do a patch for 6.1.0 only, should I checkout 6.1.0 and patch against it ?19:31
EmilienMnope19:31
EmilienMstable/kilo19:31
EmilienMchem: ^19:31
chemEmilienM: ack19:31
EmilienMmaybe we need to patch master first?19:31
chemEmilienM: the patch above is for master and should be backported after, but I have another bug https://bugs.launchpad.net/puppet-keystone/+bug/1522550 which is specific to 6.1.0 (stable/kilo)19:33
openstackLaunchpad bug 1522550 in puppet-keystone "If /root/openrc filled with v2 auth_url ending with "/" make keystone_user fails." [Undecided,New]19:33
chemyeah ^19:33
EmilienMchem: ok, so then only stable/kilo19:33
*** davideagle has quit IRC19:36
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: keystone_user fails if auth_url ends with '/' in Openrc  https://review.openstack.org/25317019:50
chemclayton: with the two patches above you're good to go even with /root/openrc19:50
*** davideagle_ has joined #puppet-openstack19:52
*** dmsimard|afk is now known as dmsimard19:52
*** davideagle_ has quit IRC19:55
*** chem has quit IRC20:18
*** chem has joined #puppet-openstack20:20
*** RichardRaseley has joined #puppet-openstack20:25
openstackgerritClayton O'Neill proposed openstack/puppet-keystone: Keystone_user fails with v2 openrc.  https://review.openstack.org/25318820:26
openstackgerritClayton O'Neill proposed openstack/puppet-keystone: Keystone_user fails with v2 openrc.  https://review.openstack.org/25318920:26
*** nlahouti has quit IRC20:27
EmilienMclayton: you need to wait the patch is merged in master before any backport20:27
EmilienMbecause now you're missing the "Cherry-picked from"20:27
EmilienMand the commit id20:27
claytonah.  that's annoying.  that's an easy way for it to get forgotten.20:28
*** foxhunt_away is now known as foxhunt7220:34
*** openstackgerrit has quit IRC20:47
*** openstackgerrit has joined #puppet-openstack20:48
openstackgerritDan Prince proposed openstack/puppet-mistral: Fix keystone configuration.  https://review.openstack.org/24727420:59
openstackgerritDan Prince proposed openstack/puppet-mistral: Add mistral::engine class  https://review.openstack.org/24841220:59
openstackgerritDan Prince proposed openstack/puppet-mistral: Add mistral::executor class  https://review.openstack.org/24841320:59
openstackgerritDan Prince proposed openstack/puppet-mistral: Add mistral::api class  https://review.openstack.org/24841020:59
openstackgerritDan Prince proposed openstack/puppet-mistral: Allow distro packages to install dependencies, etc  https://review.openstack.org/24727720:59
openstackgerritDan Prince proposed openstack/puppet-mistral: Autorequire mistral-common in mistral_config.  https://review.openstack.org/24840620:59
openstackgerritDan Prince proposed openstack/puppet-mistral: Update mistral::db::sync to run 'update head'  https://review.openstack.org/24840420:59
openstackgerritDan Prince proposed openstack/puppet-mistral: Default Rabbit and Qpid ports to 5672  https://review.openstack.org/24726920:59
*** rcernin has quit IRC21:05
*** delatte has quit IRC21:07
openstackgerritNate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/25321621:10
openstackgerritNate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/25321621:12
*** karimb has joined #puppet-openstack21:12
*** ianw has quit IRC21:13
*** ianw has joined #puppet-openstack21:18
*** gfidente has quit IRC21:19
openstackgerritNate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/25321621:21
*** saneax_AFK is now known as saneax21:21
*** nlahouti has joined #puppet-openstack21:23
openstackgerritDan Prince proposed openstack/puppet-mistral: Fix keystone configuration.  https://review.openstack.org/24727421:24
openstackgerritDan Prince proposed openstack/puppet-mistral: Add mistral::engine class  https://review.openstack.org/24841221:24
openstackgerritDan Prince proposed openstack/puppet-mistral: Add mistral::executor class  https://review.openstack.org/24841321:24
openstackgerritDan Prince proposed openstack/puppet-mistral: Add mistral::api class  https://review.openstack.org/24841021:24
openstackgerritDan Prince proposed openstack/puppet-mistral: Allow distro packages to install dependencies, etc  https://review.openstack.org/24727721:24
openstackgerritDan Prince proposed openstack/puppet-mistral: Autorequire mistral-common in mistral_config.  https://review.openstack.org/24840621:24
openstackgerritDan Prince proposed openstack/puppet-mistral: Update mistral::db::sync to run 'update head'  https://review.openstack.org/24840421:24
openstackgerritDan Prince proposed openstack/puppet-mistral: Default Rabbit and Qpid ports to 5672  https://review.openstack.org/24726921:24
*** aimon has quit IRC21:36
*** dprince has quit IRC21:36
*** cdelatte has joined #puppet-openstack21:38
*** ntpttr has quit IRC21:43
*** foxhunt72 is now known as foxhunt_away22:00
*** trown is now known as trown|outttypeww22:02
*** rahulait has joined #puppet-openstack22:14
*** nlahouti has quit IRC22:27
*** jayg is now known as jayg|g0n322:28
*** nlahouti has joined #puppet-openstack22:29
*** gildub has joined #puppet-openstack22:43
*** dhollinger has quit IRC22:46
*** saneax is now known as saneax_AFK22:47
*** baker has quit IRC23:04
*** baker has joined #puppet-openstack23:04
*** baker has quit IRC23:09
*** jerrygb has quit IRC23:12
*** aimon has joined #puppet-openstack23:28
*** bnemec has quit IRC23:33
*** bnemec has joined #puppet-openstack23:33
openstackgerritMerged openstack/puppet-keystone: keystone_user fails if auth_url ends with '/' in Openrc  https://review.openstack.org/25317023:38
EmilienMclayton: ^ merged23:43
*** rahulait has quit IRC23:47
openstackgerritMerged openstack/puppet-keystone: Keystone_user fails with v2 openrc.  https://review.openstack.org/25315723:50
*** ntpttr has joined #puppet-openstack23:53
*** ntpttr has left #puppet-openstack23:53
*** ducttape_ has quit IRC23:53
*** dfisher has quit IRC23:55

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