Thursday, 2015-09-24

*** gildub has quit IRC00:03
*** gildub has joined #puppet-openstack00:03
*** zhangjn has quit IRC00:04
*** jfluhmann has quit IRC00:06
*** angdraug has quit IRC00:11
richmgildub: as far as the forcing the use of ::domain, chem hasn't finished his investigation yet, to see if it will actually work - he had some doubts and wanted to investigate some more00:14
EmilienMI invite you to read https://docs.google.com/forms/d/1Z6IGeJRNmX7xx0Ggmr5Pmpzq7BudphDkZE-3t4Q5G1k/viewanalytics00:15
EmilienMrichm, gildub in particular00:15
EmilienMit's interesting data00:15
richmEmilienM: yes00:17
*** sanjayu has quit IRC00:17
richmgildub: so I guess the best approach for now is to split off the patch to separate the deprecation warnings about missing '::domain' from the parts about changing self.instances to be simpler00:18
richmthen, rebase the 'get rid of indirection' patch on top of the 'change self.instances to be simpler' patch00:18
richmthen, if chem comes back and says "we have to use 'name::domain'" then add back the deprecation warnings00:19
richmas a separate patch00:19
openstackgerritMerged openstack/puppet-openstack-cookiecutter: manifest/db: fix lint WARNINGS  https://review.openstack.org/22512700:21
*** xarses has joined #puppet-openstack00:35
*** xarses has quit IRC00:35
*** xarses has joined #puppet-openstack00:36
*** linkedinyou has joined #puppet-openstack00:41
*** xarses has quit IRC00:43
openstackgerritMerged openstack/puppet-openstacklib: Allow to path custom fragment to vhost  https://review.openstack.org/21927900:44
openstackgerritMerged openstack/puppet-neutron: Filename change for config-dir for ml2 cisco nexus mech driver.  https://review.openstack.org/22037600:44
openstackgerritMerged openstack/puppet-swift: provide means to disable log_requests in config templates  https://review.openstack.org/22062100:44
*** richm has quit IRC00:45
openstackgerritMerged openstack/puppet-ironic: Allow customization of db sync command line  https://review.openstack.org/22124500:46
openstackgerritMerged openstack/puppet-horizon: add api_versions parameter  https://review.openstack.org/21165000:48
openstackgerritMerged openstack/puppet-neutron: Allow greater flexibility in the kind of parameter passed to ml2.pp  https://review.openstack.org/22131800:49
*** zhangjn has joined #puppet-openstack00:52
*** partner has quit IRC00:53
*** partner has joined #puppet-openstack00:54
*** tiswanso has joined #puppet-openstack00:54
*** tiswanso has quit IRC00:54
*** tiswanso has joined #puppet-openstack00:54
*** tiswanso_ has joined #puppet-openstack00:56
gildubrichm, I agree with the split then indirection on top00:58
*** tiswanso has quit IRC00:58
openstackgerritMerged openstack/puppet-horizon: add new parameters for multidomain support  https://review.openstack.org/21170400:58
gildubEmilienM, richm, re stats, very interesting, thanks00:58
openstackgerritMerged openstack/puppet-neutron: Add support for rpc_response_timeout option  https://review.openstack.org/22397400:58
gildubEmilienM, well v2/v3 almost on par, and more need for V3 everywhere, getting there :)00:59
openstackgerritMerged openstack/puppet-neutron: Restart Neutron agents if packages are updated  https://review.openstack.org/19933700:59
openstackgerritMerged openstack/puppet-neutron: Introduce neutron::db class  https://review.openstack.org/20841501:02
*** jfluhmann has joined #puppet-openstack01:18
*** xarses has joined #puppet-openstack01:28
*** xarses has quit IRC01:28
*** xarses has joined #puppet-openstack01:29
openstackgerritMerged openstack/puppet-vswitch: Support for FreeBSD  https://review.openstack.org/19152301:51
openstackgerritMerged openstack/puppet-nova: Handle libvirt/cpu_model option  https://review.openstack.org/21454201:52
openstackgerritMerged openstack/puppet-manila: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987201:53
openstackgerritMerged openstack/puppet-cinder: make it possible to have multiple type_sets with the same value  https://review.openstack.org/19149801:56
*** xarses has quit IRC01:58
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Cleanup configure_delegated_roles deprecated parameter  https://review.openstack.org/22708302:03
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable telemetry service  https://review.openstack.org/22649002:12
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/22708402:12
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable telemetry service  https://review.openstack.org/22649002:13
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/22708402:13
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/22708402:21
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/22708402:22
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/20322002:25
*** EmilienM sets mode: +o mfisch02:37
EmilienMinteg/telemetry is ready I think https://review.openstack.org/#/c/226490/ (centos is red because polling agent package is currently being fixed in RDO)02:43
EmilienMinteg/orchestration is WIP: https://review.openstack.org/#/c/227084/ (almost all test pass)02:44
EmilienMinteg/horizon is stalled > https://bugs.launchpad.net/tempest/+bug/149829702:44
openstackLaunchpad bug 1498297 in tempest "dashboard scenario does not handle 30x HTTP code" [Undecided,In progress] - Assigned to Mehdi Abaakouk (sileht)02:44
openstackgerritMike Dorman proposed openstack/puppet-glance: Add S3 backend configuration for glance  https://review.openstack.org/22601502:45
EmilienMthe next is integ/swift - but I'm don't know if we can deploy it in one single puppet run02:45
*** iurygregory has quit IRC02:47
*** iurygregory has joined #puppet-openstack02:53
*** sergmelikyan has joined #puppet-openstack02:57
*** ferest has joined #puppet-openstack03:01
*** xarses has joined #puppet-openstack03:02
*** linkedinyou has quit IRC03:03
*** sergmelikyan has quit IRC03:05
*** ferest has quit IRC03:05
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Cleanup configure_delegated_roles deprecated parameter  https://review.openstack.org/22708303:11
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Update rpc_backend default parameter  https://review.openstack.org/22709203:11
*** tiswanso_ has quit IRC03:27
EmilienMgildub: can we use keystone_user_role to assign an a role to an user in a domain (and not in a tenant) ?03:28
EmilienMgildub: I might have found a bug in fact03:32
EmilienMin lib/puppet/provider/keystone_user_role/openstack.rb03:32
EmilienMhttp://logs.openstack.org/84/227084/4/check/gate-puppet-openstack-integration-dsvm-trusty/ccbbb57/console.html#_2015-09-24_02_32_30_11303:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/22708403:38
EmilienMgildub: I go to bed now but I feel like the user is not assigned to a role in the domain03:39
gildubEmilienM, there are issues in user when the tenant doesn't have the domain assigned and when different then default domain03:44
EmilienMah03:44
EmilienMI might hit it now03:44
EmilienMso I should assign the tenant to the domain, right?03:45
EmilienMand I'm not using the default domain03:45
gildubEmilienM, if it's the default domain everywhere that should work but if not then there are definitely issues unless using the ::domain everywhere03:46
EmilienMgildub: is it WIP or?03:46
gildubEmilienM, yes, my priority03:46
EmilienMoh nice03:46
gildubEmilienM, basically removing the non default domain to not be mentioned, then the indirection patch. Once there we should be able to address other issues around that03:48
EmilienMgildub: what is the name of the default domain again ? 'Default' ?03:49
EmilienMgildub: have you a LP # for this thing?03:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/22708403:52
gildubEmilienM, default/Default id/name03:53
gildubEmilienM, LP?03:53
EmilienMgildub: launchpad03:56
EmilienMgildub: a bug number03:57
EmilienMgildub: I'm deploying heat / trusts in our CI > https://review.openstack.org/22708403:58
gildubEmilienM, no LP04:03
EmilienMgildub: ok04:04
EmilienM2015-09-24 04:02:13.873 | Notice: /Stage[main]/Heat::Keystone::Domain/Keystone_user_role[heat_admin@::Default]/ensure: created04:04
EmilienMnot sure it worked ^04:05
gildubEmilienM, but basically I'm taking few things out of keystoneV3/default_domain_support (219127, 218044 and 218059)04:05
gildubEmilienM, without the deprecation warnings, which avoids de-facto domains04:06
EmilienMgildub: ok... it still does not work for me, I'll continue tomorrow04:15
EmilienMgildub: cheers!04:15
*** chandankumar has joined #puppet-openstack04:39
*** sanjayu has joined #puppet-openstack04:40
gildubEmilienM, bonne nuit04:50
*** khappone has joined #puppet-openstack04:50
*** alokj has joined #puppet-openstack04:56
*** alokj is now known as Guest5233404:57
openstackgerritBen Kero proposed openstack/puppet-swift: Warn that object storage parameter mount_check changes next release  https://review.openstack.org/20175505:13
bkeromoar patches05:18
*** xarses_ has joined #puppet-openstack05:21
*** xarses has quit IRC05:24
*** finchd has quit IRC05:24
*** wabu has quit IRC05:25
*** wabu has joined #puppet-openstack05:26
*** finchd has joined #puppet-openstack05:27
*** dgurtner has joined #puppet-openstack05:32
*** Guest52334 is now known as alokj_05:41
*** smakar_ has joined #puppet-openstack05:57
openstackgerritKuo-Le.Mei proposed openstack/puppet-nova: Fixed value `rbd.keyring` not update  https://review.openstack.org/20390905:57
openstackgerritKuo-Le.Mei proposed openstack/puppet-nova: Fixed value `rbd.keyring` not update  https://review.openstack.org/20390905:58
*** chandankumar has quit IRC06:24
*** smakar_ has quit IRC06:26
*** nihilifer1 has quit IRC06:26
*** nihilifer has joined #puppet-openstack06:26
*** markvoelker has quit IRC06:27
*** leanderthal has joined #puppet-openstack06:30
*** dgurtner has quit IRC06:32
*** chandankumar has joined #puppet-openstack06:34
*** smakar_ has joined #puppet-openstack06:48
*** dgurtner has joined #puppet-openstack06:53
*** alokj_ has quit IRC07:06
*** alokj_ has joined #puppet-openstack07:19
*** markvoelker has joined #puppet-openstack07:28
*** jpena has joined #puppet-openstack07:31
*** markvoelker has quit IRC07:33
*** arnaud_orange has joined #puppet-openstack07:48
openstackgerritYanis Guenane proposed openstack/puppet-cinder: Introduce cinder::db class  https://review.openstack.org/20835407:57
duritongolé for merging https://review.openstack.org/19149807:58
*** paramite has joined #puppet-openstack08:03
*** mihgen has quit IRC08:14
*** zeus has quit IRC08:14
*** mjb has quit IRC08:14
*** zeus has joined #puppet-openstack08:15
*** zeus is now known as Guest687908:15
*** mjb has joined #puppet-openstack08:17
*** mihgen has joined #puppet-openstack08:26
*** zhangjn has quit IRC08:35
*** rochaporto has joined #puppet-openstack08:37
openstackgerritYanis Guenane proposed openstack/puppet-ceilometer: Update ceilometer::db class to match other module pattern  https://review.openstack.org/22717908:38
*** alokj_ is now known as alokj08:38
*** zhangjn has joined #puppet-openstack08:38
openstackgerritYanis Guenane proposed openstack/puppet-ceilometer: Update ceilometer::db class to match other module pattern  https://review.openstack.org/22717908:39
*** paramite is now known as paramite|afk08:50
*** derekh has joined #puppet-openstack08:51
*** paramite|afk is now known as paramite09:01
*** gfidente has joined #puppet-openstack09:02
*** smakar_ has quit IRC09:02
openstackgerritYanis Guenane proposed openstack/puppet-heat: Introduce heat::db class  https://review.openstack.org/22718709:05
openstackgerritYanis Guenane proposed openstack/puppet-heat: Introduce heat::db class  https://review.openstack.org/22718709:08
*** markvoelker has joined #puppet-openstack09:29
*** markvoelker has quit IRC09:34
openstackgerritYanis Guenane proposed openstack/puppet-nova: Update nova::db class to match other module pattern  https://review.openstack.org/22719609:47
*** smakar_ has joined #puppet-openstack10:05
*** paramite is now known as paramite|afk10:07
openstackgerritYanis Guenane proposed openstack/puppet-tuskar: Introduce tuskar::db class  https://review.openstack.org/22720210:09
aderyuginHi, folks! Review https://review.openstack.org/#/c/216682/ please10:15
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/22720710:16
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/22720810:16
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/22720910:17
*** zhangjn has quit IRC10:21
*** sergmelikyan has joined #puppet-openstack10:33
openstackgerritYanis Guenane proposed openstack/puppet-tuskar: Introduce tuskar::db class  https://review.openstack.org/22720210:35
openstackgerritMichael Polenchuk proposed openstack/puppet-nova: Add kombu_reconnect_delay option  https://review.openstack.org/21310210:46
openstackgerritYanis Guenane proposed openstack/puppet-designate: Update designate::db class to match other module pattern  https://review.openstack.org/22721710:49
openstackgerritYanis Guenane proposed openstack/puppet-designate: Update designate::db class to match other module pattern  https://review.openstack.org/22721710:51
*** gildub has quit IRC11:03
*** gfidente has quit IRC11:08
*** maximov has quit IRC11:18
*** amaksimov has joined #puppet-openstack11:19
*** gildub has joined #puppet-openstack11:20
*** zhangjn has joined #puppet-openstack11:27
*** cdelatte has joined #puppet-openstack11:28
*** chandankumar has quit IRC11:28
*** markvoelker has joined #puppet-openstack11:30
*** markvoelker has quit IRC11:34
*** paramite|afk is now known as paramite11:35
EmilienMgood morning11:36
iurygregorygood morning o/11:36
iurygregoryhey aderyugin, i'll give +1 if you change the commit message "Implementaion" is wrong ^^11:39
degorenkomorning o/11:42
iurygregorythanks for the review degorenko o/11:45
*** jtomasek has quit IRC11:46
degorenkoiurygregory, np o/11:46
*** jtomasek has joined #puppet-openstack11:47
*** gfidente has joined #puppet-openstack11:48
*** gfidente has quit IRC11:48
*** gfidente has joined #puppet-openstack11:48
*** jpena is now known as jpena|lunch11:48
degorenkoiurygregory, i left some comments more :D take a look please11:53
iurygregorysure11:53
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Cleanup configure_delegated_roles deprecated parameter  https://review.openstack.org/22708311:54
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Update rpc_backend default parameter  https://review.openstack.org/22709211:55
*** sergmelikyan has quit IRC12:01
*** jayg|g0n3 is now known as jayg12:02
*** alokj has quit IRC12:06
*** chandankumar has joined #puppet-openstack12:07
*** dprince has joined #puppet-openstack12:09
*** ducttape_ has joined #puppet-openstack12:14
*** ducttape_ has quit IRC12:28
*** bpiotrowski has joined #puppet-openstack12:28
*** ducttape_ has joined #puppet-openstack12:30
*** markvoelker has joined #puppet-openstack12:31
*** cdelatte has quit IRC12:31
openstackgerritYanis Guenane proposed openstack/puppet-cinder: Put all the logging related parameters to the logging class  https://review.openstack.org/22726512:32
*** sanjayu has quit IRC12:37
openstackgerritEmilien Macchi proposed openstack/puppet-heat: Update rpc_backend default parameter  https://review.openstack.org/22709212:39
degorenkospredzy, hey o/ are you going to create logging class for all projects or for cinder only for now?12:45
spredzydegorenko, logging class is already in most projects, what I am trying to achieve is centralized the logging parameters in the logging class and not in the init class12:46
*** zhangjn has quit IRC12:46
degorenkospredzy, really? wow, i didn't saw this class before :(12:46
*** ikkeT has quit IRC12:47
degorenkoi'll do the same for sahara then12:47
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: compute agent: do not try to configure nova.conf  https://review.openstack.org/22651212:48
spredzydegorenko, for heat for example https://github.com/openstack/puppet-heat/blob/master/manifests/logging.pp12:48
degorenkoyeah, cool!12:48
spredzybut I know not every module has it yet12:48
degorenkospredzy, what about creating separate class for rpc?12:49
spredzydegorenko, ahaha to me it makes sense :) I like the idea of specialized class, this way when troubleshooting something or writing tests for something it is in logical unit12:49
degorenkourrrhhhh, gerrit so slow today12:50
spredzynot a big blob file12:50
spredzyyep reallyyyy slow12:50
EmilienMyes, this is a good topic for the summit12:50
degorenkoi'll add this then :)12:50
EmilienMI think spredzy already did12:50
spredzyEmilienM, degorenko I already wrote it12:50
*** ducttape_ has quit IRC12:50
*** obbs has quit IRC12:50
degorenkospredzy, it is contain in 'Reshaping of our current init.pp' probably?12:51
degorenkocontains*12:51
spredzydegorenko, yep12:51
degorenkospredzy, ok! for me it is good point too12:51
degorenkospredzy, as i understand, we don't want 'include logging' in main init class?12:53
openstackgerritMerged openstack/puppet-openstack-integration: Puppetfile: bump qpid to 1.0.3  https://review.openstack.org/22692712:58
spredzydegorenko, we do want the include in the init, what I'd like to avoid is some logging parameters set in init and other set in logging12:59
degorenkospredzy, yes, but you use pick function, so, what are problems actually?13:00
*** jpena|lunch is now known as jpena13:01
spredzydegorenko, we need to remain backward compatible13:01
spredzyand still support non hiera deployment13:01
degorenkospredzy,  but if we will use include with pick - backward compatibility will not broken13:02
iurygregoryhey people can anyone review https://review.openstack.org/#/c/223777/ ?13:02
*** obbs has joined #puppet-openstack13:03
*** cdelatte has joined #puppet-openstack13:03
*** khushbu_ has joined #puppet-openstack13:05
*** richm has joined #puppet-openstack13:06
*** larsks has quit IRC13:07
*** ducttape_ has joined #puppet-openstack13:12
*** gildub has quit IRC13:16
*** morazi has joined #puppet-openstack13:16
*** pradk has quit IRC13:17
*** tiswanso has joined #puppet-openstack13:19
*** ducttape_ has quit IRC13:26
*** xarses_ is now known as xarses13:33
*** pradk has joined #puppet-openstack13:37
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: rspec: run tests for ::ceilometer::agent::auth  https://review.openstack.org/22730613:37
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: agent/auth: bring consistent how we manage empty parameters  https://review.openstack.org/22730713:37
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement auth_endpoint_type parameter  https://review.openstack.org/22730813:37
*** smakar_ has quit IRC13:44
*** zhangjn has joined #puppet-openstack13:45
*** zhangjn has quit IRC13:46
*** zhangjn has joined #puppet-openstack13:47
*** zhangjn has quit IRC13:47
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Introduce Sahara extended logging class  https://review.openstack.org/22731013:47
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Service Provider  https://review.openstack.org/21682113:48
*** zhangjn has joined #puppet-openstack13:48
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Introduce Sahara extended logging class  https://review.openstack.org/22731013:49
*** zhangjn has quit IRC13:49
openstackgerritZhao Lei proposed openstack/puppet-neutron: Fix a spelling typo in config's help  https://review.openstack.org/22731413:50
*** social has quit IRC13:51
*** khushbu_ has quit IRC13:51
*** cdelatte has quit IRC13:54
*** dfisher has joined #puppet-openstack13:55
*** khushbu_ has joined #puppet-openstack13:55
*** chandankumar has quit IRC13:55
*** zhangjn has joined #puppet-openstack13:59
*** khushbu_ has quit IRC14:01
*** ducttape_ has joined #puppet-openstack14:17
*** bpiotrowski has quit IRC14:20
*** bpiotrowski has joined #puppet-openstack14:20
*** bpiotrowski has joined #puppet-openstack14:20
*** obbs has quit IRC14:23
*** gfidente has quit IRC14:25
*** cdelatte has joined #puppet-openstack14:29
*** zhangjn has quit IRC14:39
aderyuginHi, EmilienM, please take a look at https://review.openstack.org/#/c/216682/14:40
*** zhangjn has joined #puppet-openstack14:40
*** zhangjn has quit IRC14:40
*** kindjal has joined #puppet-openstack14:41
*** zhangjn has joined #puppet-openstack14:41
*** zhangjn has quit IRC14:41
*** zhangjn has joined #puppet-openstack14:42
*** zhangjn has quit IRC14:42
*** zhangjn has joined #puppet-openstack14:43
*** zhangjn has quit IRC14:43
*** social has joined #puppet-openstack14:43
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683014:44
EmilienMchem: ^14:44
chemack, but it should be good, shouldn't it :)14:44
*** zhangjn has joined #puppet-openstack14:44
*** dgurtner has quit IRC14:46
*** ducttape_ has quit IRC14:46
degorenkoEmilienM, i have a question about a new prepare fixtures. You removed .fixtures.yaml, how it will know which modules are required? I don't understand14:50
EmilienMdegorenko: have you looked at https://review.openstack.org/226830 ?14:50
degorenkoEmilienM, yep, i saw this14:50
EmilienMhttps://review.openstack.org/#/c/226830/18/Rakefile,cm14:51
degorenkoi understand how will be cloned openstack-integration14:51
*** cdelatte has quit IRC14:51
EmilienMhave you read install_modules.sh script?14:51
degorenkoyes14:51
EmilienMso?14:51
EmilienMit uses r10k to install modules in a dir you want14:51
EmilienMthat's what our beaker + integration jobs is using14:52
degorenkoyes, but modules names taken here: https://github.com/openstack/puppet-openstack-integration/blob/master/install_modules.sh#L23 ?14:52
EmilienMrspec will use it to clone modules in spec/fixtures/modules14:52
*** apuimedo has quit IRC14:52
EmilienMdegorenko: yes14:52
degorenkoEmilienM, ok, so now will be installed all modules, right?14:53
EmilienMright14:53
degorenkoah, i see14:53
degorenkothanks14:53
EmilienMit will clone all modules14:53
EmilienMit's overkill but consistent14:53
degorenkoyep, just missed PuppetFile from this repo14:53
EmilienMcool14:53
degorenkoi saw your mail, just want to know, how it works exactly14:54
degorenkonow i know :) thanks again14:54
EmilienMdegorenko: cool :)14:54
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683014:55
*** ducttape_ has joined #puppet-openstack14:55
*** cdelatte has joined #puppet-openstack14:56
EmilienMsomething is broking our CI15:00
EmilienMon trusty, Rabbitmq15:00
EmilienMhttp://logs.openstack.org/92/227092/3/check/gate-puppet-heat-puppet-beaker-rspec-dsvm-trusty/04b0498/console.html#_2015-09-24_13_51_39_21815:00
degorenkoyes, i saw15:00
EmilienMdegorenko: any idea?15:00
* degorenko looking at logs15:01
EmilienMit seems like something in RabbitMQ version15:01
degorenko2015-09-24 13:59:32.691 | Error: /Stage[main]/Main/Rabbitmq_user_permissions[sahara@/]: Could not evaluate: cannot parse line from list_user_permissions:/usr/sbin/rabbitmqctl: 19: [: Linux: unexpected operator15:01
degorenkosomething missed?15:01
degorenkohost15:02
EmilienM3.5.4-1 -> 3.5.5-115:02
EmilienMrabbitmq upgrade15:02
EmilienMthe problem was not in 3.5.4-115:02
* EmilienM reading diff15:02
*** ducttape_ has quit IRC15:03
EmilienMHunner: ^15:03
degorenkoEmilienM, i don't see such tags in puppetlabs-rabbitmq15:03
EmilienMdegorenko: not in puppet15:03
degorenkowhere i should check?15:03
EmilienMin the package.15:04
degorenkoah15:04
EmilienMI'm looking15:04
EmilienMhttps://github.com/rabbitmq/rabbitmq-server/releases/tag/rabbitmq_v3_5_515:04
degorenkothanks15:04
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: DNM - CI test  https://review.openstack.org/22735715:06
EmilienMHunner: RabbiMQ 3.5.5-1 was released 3 hours ago and breaks puppetlabs-rabbitmq15:07
EmilienMCould not evaluate: cannot parse line from list_user_permissions:/usr/sbin/rabbitmqctl: 19: [: Linux: unexpected operator15:07
*** zhangjn_ has joined #puppet-openstack15:09
*** cdelatte has quit IRC15:11
EmilienMsomething in https://github.com/rabbitmq/rabbitmq-server/compare/rabbitmq_v3_5_4...rabbitmq_v3_5_515:12
degorenkoEmilienM, yeah i looking at it too15:12
*** zhangjn has quit IRC15:13
*** arnaud_orange has quit IRC15:16
*** zhangjn_ has quit IRC15:26
*** zhangjn has joined #puppet-openstack15:27
*** zhangjn has quit IRC15:27
degorenkothere are some changes in scripts/rabbitmq-server script15:27
degorenkobut extra arguments don't change15:28
*** zhangjn has joined #puppet-openstack15:28
*** zhangjn has quit IRC15:28
*** zhangjn has joined #puppet-openstack15:28
*** angdraug has joined #puppet-openstack15:29
*** dgurtner has joined #puppet-openstack15:30
*** LimorStotland has quit IRC15:31
*** zhangjn_ has joined #puppet-openstack15:35
EmilienMdegorenko: I'm reproducing it in my dev env15:37
*** zhangjn has quit IRC15:37
EmilienMstill investigating15:37
*** IvanBerezovskiy has joined #puppet-openstack15:37
EmilienMroot@em-vag-u1404-dev:~# /usr/sbin/rabbitmqctl -q list_user_permissions cinder15:38
EmilienM/usr/sbin/rabbitmqctl: 19: [: Linux: unexpected operator15:38
EmilienM\o/15:38
degorenkoEmilienM, me too15:39
degorenkotrying to install 3.54.15:39
degorenko3.5.415:39
degorenkoEmilienM, you can run just  /usr/sbin/rabbitmqctl -q list_users15:39
degorenkoroot@puppet-test-vm:/home/ubuntu#  /usr/sbin/rabbitmqctl -q list_users15:39
degorenko/usr/sbin/rabbitmqctl: 19: [: Linux: unexpected operator15:39
degorenkoguest[administrator]15:39
*** dgurtner has quit IRC15:39
EmilienMI found it15:40
EmilienM\o/15:40
degorenkoEmilienM, what is it?15:40
EmilienMdegorenko: if you replace sh by bash in L1, it works15:40
dfisherit secretly installed Solaris when you weren't looking15:40
dfisher:ssh:15:40
degorenkoEmilienM, you're right15:41
degorenkoit works for me too15:41
degorenkobut in 3.5.4 - sh too15:42
degorenkoEmilienM, https://github.com/rabbitmq/rabbitmq-server/commit/b82a9fdb0fbc0f675facccdbcdf8dd8d753f289915:43
EmilienMoh15:43
EmilienMnice shot15:44
degorenko:)15:44
EmilienMdegorenko: I do a PR, or you?15:44
EmilienMI'm ready15:44
degorenkoEmilienM, go ahead :)15:44
EmilienMcool15:44
EmilienMour CI is broken anyway15:44
degorenkoyea15:44
degorenkoCan you put please the link on review here? :)15:45
EmilienMdegorenko: sure15:46
*** zhangjn has joined #puppet-openstack15:46
*** skolekonov is now known as _skolekonov15:47
EmilienMdegorenko: https://github.com/rabbitmq/rabbitmq-server/issues/32115:48
*** chem has quit IRC15:48
degorenkoEmilienM, hey, there are change this: https://github.com/rabbitmq/rabbitmq-server/compare/rabbitmq_v3_5_4...rabbitmq_v3_5_5#diff-0dd46c0d3d8b9bb6b2b9dcf5701cb049R2515:49
*** zhangjn_ has quit IRC15:49
degorenkobut anyway - it doens't work properly15:49
degorenkoEmilienM, may be we also should add output from cmd? I'll add comment15:50
EmilienMoh it's fixed already15:50
EmilienM6964e8ac1559cf76b9245394b46aaec5822f417815:51
degorenkoi put output from simple list_users15:51
degorenkoyes15:51
degorenkoit is15:51
degorenkobut it is not working :D15:51
EmilienMI'm contacting ubuntu packages15:52
degorenkooh, you mean that we have broken tag, right?15:52
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Changes in enabling federation spec  https://review.openstack.org/22377715:57
degorenkobtw, it was fixed 3 days ago15:57
degorenkohow they test tags before release? lol15:58
*** degorenko has quit IRC16:00
*** degorenko has joined #puppet-openstack16:01
EmilienMdegorenko: I found a workaround16:02
EmilienMdegorenko: yeah...16:02
EmilienMI wonder if they have CI16:02
*** chem has joined #puppet-openstack16:02
EmilienMin rabbimt puppet module there is manifests/init.pp +17316:02
EmilienMit does not take the version from ::params16:03
degorenkoEmilienM, my PC has had a reboot ...16:03
degorenkoso what is workaround?16:03
EmilienMit happens :P16:03
degorenkoyeah :(16:03
EmilienMI'm preparing a patch16:03
* degorenko waiting16:03
EmilienMthe $version seems not used16:03
degorenkosome kind pin rabbitmq package version?16:03
EmilienMprobably, yes16:04
degorenkoone more hack for package :) nice16:04
EmilienMdegorenko: brb16:05
degorenko:D16:06
*** jfluhmann has quit IRC16:06
*** cdelatte has joined #puppet-openstack16:08
degorenkoEmilienM, btw, https://github.com/rabbitmq/rabbitmq-server/commit/6964e8ac1559cf76b9245394b46aaec5822f4178 - is not working16:08
degorenkoha ha16:08
degorenko3.5.5 contains this fix16:08
openstackgerritMichael Polenchuk proposed stackforge/puppet-openstack: Add default domain into openrc  https://review.openstack.org/22740016:09
*** derekh has quit IRC16:09
EmilienMit's 6964e8ac1559cf76b9245394b46aaec5822f4178 right ?16:09
degorenkoyes16:09
degorenkolet me a few minutes16:09
degorenkoi'll try to use old way16:09
EmilienMmaybe in the pacakge16:10
degorenkolol16:12
degorenkowith old16:12
degorenkorabbitmqctl -q list_users16:13
degorenko/usr/sbin/rabbitmqctl: rabbitmqctl: line 19: unexpected EOF while looking for matching `)'16:13
degorenko/usr/sbin/rabbitmqctl: rabbitmqctl: line 47: syntax error: unexpected end of file16:13
EmilienMwhat?16:13
degorenkotwo rabbits - old and new16:13
degorenkocopy-paste16:13
degorenkoresult above16:13
EmilienMwow16:13
degorenkolol16:13
degorenkohow?16:13
HunnerLovely16:14
EmilienMHunner: tl;dr: our CI is broken16:14
EmilienMdegorenko: how come it worked before16:14
HunnerOh, not rabbitmq 3.5.5?16:14
EmilienMwe can diff packages16:14
degorenkoi don't know how16:14
* degorenko trying ti understand16:15
degorenkoto*16:15
EmilienMI'm diffing16:15
degorenkook, only one error. Don't use bash :)16:15
degorenkorabbitmqctl -q list_users16:15
degorenko/usr/sbin/rabbitmqctl: 47: /usr/sbin/rabbitmqctl: Syntax error: end of file unexpected (expecting ")")16:15
EmilienMhttps://www.diffchecker.com/wd2vfkru16:15
EmilienMat the left, when our CI was working16:16
EmilienMat the right, when it broke16:16
EmilienMit's pretty obvious :)16:16
*** cdelatte has quit IRC16:17
EmilienMdegorenko: are you sure it's broken in 3.5.4 ?16:17
EmilienMdid you use the same VM ?16:17
degorenkoneed to scroll down - i'm looking on the top of page  and think -erm, what is it? :D16:17
degorenkono16:18
degorenkoi use two16:18
degorenkobut with the same repos16:18
EmilienMmhh16:18
degorenkotrusty 14.0416:18
EmilienMdpkg -l16:18
EmilienMand diff16:18
EmilienMsee if we are missing anything16:18
EmilienMhttps://www.diffchecker.com/wd2vfkru is showing it's this package though.16:19
EmilienMhave you tried 3.5.4-1 ?16:19
degorenkoyes16:19
degorenkoit works16:19
EmilienMdegorenko: 3.5.5-1 has ==16:20
EmilienMso it's broken16:20
degorenkooh, first vm has many software for diskimage-builder...16:20
degorenkoi forgot about it16:20
degorenkoneed create new vm16:20
degorenkobut actually16:20
degorenkoit will have diff only for rabbit16:21
*** markvoelker_ has joined #puppet-openstack16:21
degorenkoi'm sure16:21
EmilienMI confirm 3.5.4-1 does not have ==16:21
EmilienMso 3.5.4-1 should wor16:21
EmilienMwork*16:21
degorenkowhat do you mean by ==?16:21
EmilienM if [ $(uname -s) == "Linux" ]; then16:21
EmilienMthat error ^16:21
degorenkoyes16:21
EmilienMis not in 3.5.416:21
degorenkoit is right16:21
EmilienMso it should work in the previous16:21
degorenkobut i've removed this!16:22
EmilienMbut you say 3.5.4 is broken16:22
degorenkoand got error16:22
degorenkono no no16:22
degorenko:)16:22
EmilienMahhhhhh16:22
EmilienMso let's pin 3.5.416:22
degorenko3.5.5 - broken16:22
EmilienMok16:22
degorenkowith sh from 3.5.416:22
EmilienMgood16:22
degorenkoi don't know why16:22
degorenkowe can pin16:22
degorenkoyes16:22
EmilienMI'm preparing a patch16:22
degorenkoso interesting package debugging :)16:23
*** cdelatte has joined #puppet-openstack16:23
EmilienMdegorenko: yeah16:24
*** ddmitriev1 has quit IRC16:24
*** markvoelker has quit IRC16:24
*** aderyugin has quit IRC16:24
*** kbyrne has quit IRC16:24
*** bkero has quit IRC16:24
*** jprs has quit IRC16:24
*** afranc202 has quit IRC16:24
*** iraw- has joined #puppet-openstack16:24
EmilienMour CI is really on the edge lol16:24
*** aderyugin has joined #puppet-openstack16:24
degorenko:D16:24
*** joshsamuelson has joined #puppet-openstack16:24
*** kbyrne has joined #puppet-openstack16:24
*** duritong has quit IRC16:25
*** duritong has joined #puppet-openstack16:25
EmilienMwe're testing the world16:25
degorenko\o/16:25
*** bkero has joined #puppet-openstack16:25
*** rochaporto has quit IRC16:26
*** ducttape_ has joined #puppet-openstack16:27
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Pin RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/22741716:31
EmilienMdegorenko: please review ^16:31
* degorenko looking16:31
EmilienMdegorenko: they replied, lol they are fixing it!16:32
degorenkoEmilienM, they replied - wow16:32
EmilienMyeah16:32
EmilienMat least they are responsive16:32
degorenkoyes :)16:32
EmilienMdegorenko: should we tell them to have a CI ? lol16:33
degorenkobecause i want already will try to fix myself :D16:33
degorenkono16:33
EmilienMdegorenko: was joking16:33
degorenkodon't disappoint them16:33
EmilienMmaybe16:33
EmilienMif my patch ^ works16:34
EmilienMI'll push it to all modules16:34
degorenkoto acceptance tests, yes?16:34
EmilienMdegorenko: yeah16:34
EmilienMexcept if they fix the package16:35
EmilienMprobably we won't need to fix all our repos16:35
EmilienMif they re-build the package today16:35
EmilienMI'll wait until this afternoon16:35
degorenkoare you sure that they will update package in a mirror today? :)16:35
EmilienMwe need to monitor http://www.rabbitmq.com/releases/rabbitmq-server/v3.5.5/16:35
EmilienM"We are recreating the RPM and Debian packages with the script fixed"16:36
degorenkoyea16:36
degorenkoi saw16:36
degorenkobut who is guaranty that it will be done today?16:36
*** mdorman has joined #puppet-openstack16:36
EmilienMdegorenko: nothing, I'm waiting a bit and then we will have to unblock our CI16:37
EmilienMit could be great to avoid patching all our modules again16:37
*** jpena has quit IRC16:38
EmilienMdegorenko: my fix does not work: https://jenkins07.openstack.org/job/gate-puppet-openstack-integration-dsvm-trusty/116/consoleFull16:42
EmilienMgrep "Error:"16:42
EmilienMnot sure my param worked16:42
EmilienMI found out16:42
EmilienMa typo16:42
*** serg_melikyan has joined #puppet-openstack16:43
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Pin RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/22741716:43
degorenkowhere?16:43
EmilienMhttps://review.openstack.org/#/c/227417/1..2/fixtures/scenario001.pp,cm16:43
degorenkooh16:43
degorenkoensure16:43
EmilienMmy bad16:43
EmilienMdegorenko: I go eat my lunch, I'm back later o/16:43
degorenkoEmilienM, ok, have a nice lunch o/16:44
degorenkoEmilienM, lol, i find what the problem actually.... Epic lol16:45
*** paramite has quit IRC16:45
degorenkoand they already fixed it in master16:46
*** mdorman_ has joined #puppet-openstack16:47
*** serg_melikyan has quit IRC16:47
degorenkoEmilienM, https://github.com/rabbitmq/rabbitmq-server/commit/67c24aa1ccf5209660d54ad49033a1d1e3cc050216:48
degorenkoit is in master16:48
*** mdorman has quit IRC16:48
*** mdorman_ is now known as mdorman16:48
imcsk8EmilienM: i was looking at your n1k patch and i remembered that the image puppetlabs/ubuntu-14.04-64-nocm does not have a libvirt provider, i think it might be appropiate to use a image with more providers16:51
*** chem has quit IRC16:54
EmilienMdegorenko: could you ask murano maintainers to modify https://launchpad.net/puppet-murano so puppet group could manage it16:56
EmilienMdegorenko: take example from https://launchpad.net/puppet-nova16:56
*** pradk has quit IRC16:57
degorenkoEmilienM, yes, sure :) did you saw my message about rabbit?16:57
EmilienMno16:57
EmilienMah16:58
EmilienMin github, yes16:58
degorenkonice? :) do they know sh? or don't16:58
degorenkoand, how they tested it?16:58
degorenkoi don't any idea16:58
*** leanderthal has quit IRC16:59
*** zhangjn_ has joined #puppet-openstack16:59
*** zhangjn has quit IRC16:59
*** ducttape_ has quit IRC16:59
openstackgerritJohn Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/22369017:02
EmilienMdegorenko: E: Version '3.5.4-1' for 'rabbitmq-server' was not found17:07
degorenkolol17:07
* degorenko checking rabbitmq version17:08
*** zhangjn has joined #puppet-openstack17:08
degorenkoyeah17:08
degorenkothats true17:08
*** zhangjn_ has quit IRC17:09
degorenkoEmilienM, 3.5.4-1~cloud0.17:09
*** ducttape_ has joined #puppet-openstack17:09
degorenkowithout '.'17:09
EmilienMoh17:09
degorenkowe need ~cloud017:09
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Pin RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/22741717:09
EmilienMI go lunch for real17:10
*** sergmelikyan has joined #puppet-openstack17:11
mwhahahawouldn't you just require 3.5.4? with no -1~cloud0?17:13
degorenkomwhahaha, take a look on patch set 217:13
degorenkomwhahaha, E: Version '3.5.4-1' for 'rabbitmq-server' was not found17:13
mwhahaharight i'm saying isn't it 3.5.417:14
degorenkoE: Version '3.5.4' for 'rabbitmq-server' was not found17:14
mwhahahanot 3.5.4-117:14
degorenkothe same17:14
mwhahahathat's annoying17:14
degorenkoit happens :)17:15
*** jfluhmann has joined #puppet-openstack17:15
openstackgerritAlex Schultz proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/21665417:18
*** zhangjn has quit IRC17:23
EmilienMI could also trick something else to not have to patch our modules17:24
EmilienMin install_modules.sh17:24
EmilienMsince puppetlabs-rabbitmq is just checking the package is 'installed' by default (when running yum & apt package providers)17:24
EmilienMwe could install the .deb manually before the puppet run17:25
*** ducttape_ has quit IRC17:25
degorenkoin each test?17:25
*** cdelatte has quit IRC17:25
EmilienMno17:25
EmilienMlet me show you17:25
* degorenko looking17:26
patrickeasthi, quick question about the comment on https://review.openstack.org/#/c/225368/3/spec/classes/cinder_volume_pure_spec.rb,cm is the recommendation that i switch each of the "is_expected.to contain_cinder_config" checks to more like the "should contain_cinder__backend__pure" ones?17:27
patrickeastor that i only check that there is a cinder::backend::pure and don't validate the variables17:28
mwhahahapatrickeast: i think since that class is just leveraging cinder::backend::pure resource you just need to check that. and in the tests for the cinder::backend::pure you would check for the cinder_config resources17:29
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Install RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/22743017:29
EmilienMdegorenko: ^17:29
*** ducttape_ has joined #puppet-openstack17:30
patrickeastmwhahaha: ok so just a single check for the backend17:31
degorenkolooking17:31
degorenkoEmilienM, oh! of course!17:31
EmilienMdegorenko: I like #2 more than #117:31
EmilienMso we don't have to patch all our modules17:32
degorenkoyeah17:32
EmilienMbut it fails17:32
degorenkowhat aboud dependencies?17:32
degorenkoabout*17:32
EmilienMyeah17:34
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Install RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/22743017:34
degorenkoEmilienM, also we can use '-d' flag17:35
EmilienMdamn it fails17:35
degorenkoin this case we will have all dependencies17:35
degorenkoit is in apt-get17:35
degorenkoEmilienM, apt-get install -d rabbitmq-server and then our rabbitmq17:36
EmilienMyeah, we can also install the one from ubuntu17:36
EmilienMgood idea17:36
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Install RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/22743017:37
degorenkoerm17:38
degorenkoi don't think that will work17:38
degorenkowe already have repo with broken rabbit17:38
degorenkoor not?17:38
EmilienMhttps://jenkins06.openstack.org/job/gate-puppet-openstack-integration-dsvm-trusty/118/console17:38
EmilienM3.2.4-117:39
EmilienMit works17:39
EmilienMwell, we have an old version but we can live with that now17:39
openstackgerritPatrick East proposed openstack/puppet-cinder: Add support for Pure Volume Drivers  https://review.openstack.org/22536817:39
degorenkoEmilienM, hm, yea17:40
*** cdelatte has joined #puppet-openstack17:40
*** mdorman_ has joined #puppet-openstack17:42
*** mdorman has quit IRC17:43
*** mdorman_ is now known as mdorman17:43
*** mdorman has quit IRC17:44
EmilienMdegorenko: my second patch works17:46
degorenko\o/17:47
degorenkocool17:47
degorenkoEmilienM, can we run tests for projects?17:48
openstackgerritJohn Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/22369017:49
*** chem has joined #puppet-openstack17:49
*** ducttape_ has quit IRC17:58
EmilienMdegorenko: with depends-on18:04
EmilienMcrinkle, mgagne, sbadia, clayton, mfisch: please look https://review.openstack.org/#/c/227430/ - our CI is blocked right now18:04
*** tiswanso has quit IRC18:05
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Introduce Sahara extended logging class  https://review.openstack.org/22731018:05
*** tiswanso has joined #puppet-openstack18:05
degorenkoEmilienM, ^^ i've added depends-on18:05
mgagnedone18:06
EmilienMwell, they just updated https://github.com/rabbitmq/rabbitmq-server/issues/321 to say a new package is coming18:07
*** cdelatte has quit IRC18:11
degorenko"We really need to improve the testing of the Debian and RPM packages " - oh, really?18:12
EmilienMlol18:13
EmilienMs/improve/start18:13
degorenkolol18:13
dmsimard:(18:13
*** cdelatte has joined #puppet-openstack18:14
EmilienMdegorenko: I would like your help on integration work18:18
EmilienMdegorenko: to bring sahara support18:18
degorenkoEmilienM, sure :)18:18
EmilienMsee https://review.openstack.org/226490 for example18:18
EmilienMyou need to add the class, and enable it in puppet-tempest18:18
*** xarses has quit IRC18:19
openstackgerritMerged openstack/puppet-openstack-integration: Install RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/22743018:19
EmilienMso we could add this module to the party18:19
degorenkoEmilienM, yep :) can i do it tomorrow? :D because 9 pm at Moscow zone :)18:20
EmilienMdegorenko: lol I was just asking if you were interested to help18:21
EmilienMnot tonight !!18:21
degorenkoi'm :) i will do it tomorrow18:22
EmilienMdegorenko: I need to warn you :18:22
EmilienM1/ you'll need to patch puppet-tempest, so use Depends-On on the integ patch you'll do18:22
EmilienM2/ the puppet run needs to be idempotent18:22
degorenkobtw, i have a good progress with auth based on openstack client for now :)18:22
degorenkogot it !18:23
EmilienM3/ please follow the parameters values (for passwords, etc) to be consistent18:23
EmilienMawesome18:23
degorenkofor nova*18:23
degorenkosorry, typo18:23
EmilienMdegorenko: do we need swift for running sahara?18:23
degorenkono18:23
EmilienMgood18:23
EmilienMbecause we don't have swift18:23
EmilienMyet18:23
degorenkowe can use swift, but optional18:23
*** social has quit IRC18:24
openstackgerritDenis Egorenko proposed openstack/puppet-nova: Use OpenstackClient for nova auth  https://review.openstack.org/22686218:25
degorenkoEmilienM, it should work for nova_aggregate provider now ^^18:26
degorenkotake a look quickly please, if you have time18:26
degorenkomay be some ruby issues18:26
EmilienMwe have ruby experts here :)18:27
degorenkoi'll add all tests and another providers later :)18:27
*** ducttape_ has joined #puppet-openstack18:30
degorenkoEmilienM, i go home, if you will have any questions, send personal message :) i'll answer tomorrow18:35
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable telemetry service  https://review.openstack.org/22649018:36
EmilienMdegorenko: good night18:36
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/22708418:36
degorenkoEmilienM, buy o/18:36
degorenkobye* :D18:36
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: compute agent: do not try to configure nova.conf  https://review.openstack.org/22651218:38
*** xarses has joined #puppet-openstack18:39
*** ducttape_ has quit IRC18:39
*** ducttape_ has joined #puppet-openstack18:44
*** sergmelikyan has quit IRC18:47
*** cdelatte has quit IRC18:48
*** IvanBerezovskiy has left #puppet-openstack18:50
*** ducttape_ has quit IRC18:50
EmilienMHunner, _ody: do you have thoughts about the fixtures thread on openstack ML ?19:05
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: rspec: run tests for ::ceilometer::agent::auth  https://review.openstack.org/22730619:07
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: agent/auth: bring consistent how we manage empty parameters  https://review.openstack.org/22730719:07
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement auth_endpoint_type parameter  https://review.openstack.org/22730819:07
*** dprince has quit IRC19:10
*** ducttape_ has joined #puppet-openstack19:12
*** ducttape_ has quit IRC19:14
openstackgerritMerged openstack/puppet-nova: rabbitmq: do not manage rabbitmq service anymore  https://review.openstack.org/22700719:15
iurygregoryEmilienM, sorry i've forgotten to track the acceptance in the patch =/  but the patch number 13 have the acceptance. I'll upload another with the acceptance ^^19:17
EmilienMiurygregory: cool19:17
iurygregorythanks for the review =D19:18
EmilienMlol they fixed rabbitmq19:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Revert "Install RabbitMQ 3.5.4 on Ubuntu systems"  https://review.openstack.org/22749519:20
*** aglarendil has joined #puppet-openstack19:28
*** bnemec has quit IRC19:34
openstackgerritMerged openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/22720719:35
openstackgerritMerged openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/22720819:35
openstackgerritMerged openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/22720919:35
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Identity Provider  https://review.openstack.org/20805419:50
*** mdorman has joined #puppet-openstack19:51
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683019:51
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Identity Provider  https://review.openstack.org/20805419:52
EmilienMcrinkle: for https://review.openstack.org/219275 - don't you want to wait we sort out the SERVICE_DEFAULT thing ?19:52
EmilienMcrinkle: we are waiting for _ody proposal on Gerrit I guess19:52
crinkleEmilienM: oh I thought using the string was spredzy's original plan and anything else was just optimizing19:54
EmilienMcrinkle: I'm hope we can find a concensus quickly, I like _ody's proposal which is to use openstacklib19:55
*** jayg is now known as jayg|g0n319:58
openstackgerritMerged openstack/puppet-cinder: Add support for Pure Volume Drivers  https://review.openstack.org/22536820:03
*** chem has quit IRC20:05
*** sergmelikyan has joined #puppet-openstack20:08
openstackgerritMerged openstack/puppet-designate: add designate_rootwrap_config in designate::config  https://review.openstack.org/20817820:09
openstackgerritMerged openstack/puppet-nova: Add kombu_reconnect_delay option  https://review.openstack.org/21310220:13
openstackgerritMerged openstack/puppet-neutron: Accommodate for VPNaaS package name change in RDO  https://review.openstack.org/20206520:13
EmilienMcrinkle: can you +a https://review.openstack.org/#/c/227495/ please ?20:15
EmilienMcrinkle: are you good to also merge https://review.openstack.org/#/c/226490/ even if centos is not passing? RDO team is working on Ceilometer Polling agent packaging, but the package is not in the repo we are using yet20:17
openstackgerritMerged openstack/puppet-swift: Warn that object storage parameter mount_check changes next release  https://review.openstack.org/20175520:17
openstackgerritMerged openstack/puppet-openstack-integration: Revert "Install RabbitMQ 3.5.4 on Ubuntu systems"  https://review.openstack.org/22749520:19
EmilienMactually, it's more than ceilometer20:20
EmilienMit seems like a lot of regressions on their side, nothing we can really do now20:20
EmilienMdmsimard: http://logs.openstack.org/90/226490/13/check/gate-puppet-openstack-integration-dsvm-centos7/5fdd4d2/console.html20:21
EmilienMdmsimard: it's weird, but some days ago scenarios were all working well20:21
EmilienMI got some Got server fault20:21
EmilienMlooks like in cinder20:22
EmilienMhttp://logs.openstack.org/90/226490/13/check/gate-puppet-openstack-integration-dsvm-centos7/5fdd4d2/logs/cinder/api.txt.gz#_2015-09-24_19_37_11_12320:23
EmilienMdmsimard: maybe something in taskflow, any recent package?20:23
*** bnemec has joined #puppet-openstack20:24
EmilienMit's in python-taskflow20:28
openstackgerritMerged openstack/puppet-nova: remove openstackocci installation feature  https://review.openstack.org/19497720:31
*** tiswanso has quit IRC20:32
EmilienMmwhahaha: reading you reply, I still don't understand what you actually propose20:36
mwhahahadon't make it manditory20:36
EmilienMwell, your feedback is valuable20:36
mwhahahaleave fixtures, allow for zuul version to be overriden20:36
EmilienMif you read the rakefile, we actually allow to not run zuul cloner on the machine but git20:36
EmilienMwell, technically r10k20:36
mwhahahalet me pull up the change again20:37
*** chem has joined #puppet-openstack20:37
mwhahahamy concern is that this is an openstack CI specific thing20:37
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683020:37
EmilienMmwhahaha: I would rather say "it's an openstack specific thing"20:38
mwhahahabut we're solving a CI problem20:38
EmilienMI see some people who complain because we have module dependencies20:38
EmilienMthe question is "ok, how do you want to do?"20:38
EmilienMit's not that easy to run without dependencies20:38
EmilienMmwhahaha: we're solving a code issue also, let me give you an example20:39
mwhahaharight and it's only because we're dealing with what 20+ modules?20:39
*** jfluhmann has quit IRC20:39
EmilienMmwhahaha: yesterday I was doing this patch and by doing this I realized puppet-qpid was broken because fixtures used a very old release, while our functional testing was running master20:39
EmilienMso we had to update the code with the fix20:39
EmilienMmwhahaha: 2520:40
EmilienMhttps://wiki.openstack.org/wiki/Puppet/releases20:40
EmilienMI would like to see a serious proposal20:40
mwhahahaserious proposal to solve what20:41
EmilienMthe use case I just mentionned20:41
EmilienMpuppet-nova was unit tested against puppet-qpid X while functionaly tested against Y20:41
EmilienMthe manifest was not good because of this20:41
EmilienMmwhahaha: do you know openstack/requirements ?20:42
mwhahahaSo the issue is trying to manage puppet dependencies that are shared between 25 modules20:42
mwhahaharight?20:42
EmilienMhttps://github.com/openstack/requirements#why-global-requirements20:42
EmilienMright20:42
EmilienMkeep consistent our testing20:42
EmilienMreading https://github.com/openstack/requirements#why-global-requirements - this is exactly our issue at this time.20:43
EmilienMwe're not running tox, but rake though.20:43
mwhahahabasically we have to re-invent what they have for the python side in puppet20:44
EmilienMmwhahaha: right, because fixtures is not enough for us20:45
EmilienMproposal #2: bring zuul cloner support in rspec20:45
mwhahahaso no it may be, it depends on what you are trying to fix.20:45
mwhahahaso there are two issues20:45
EmilienMbut I personally don't have the bandwith20:45
EmilienMwell, it's written in my email, but I can explain again.20:46
mwhahahano it's ok20:46
mwhahahawhat i'm trying to get at is there are multiple problems20:46
EmilienM1/ dependencies are not consistent across our .fixtures.yaml in modules20:46
mwhahaha1) the management of versions20:46
mwhahaha2) change dependency inclusion for testing20:46
EmilienM3/ what is testing in unit is not consistent with functional20:47
mwhahahaSo i want to make it so that _you_ don't have to be the one to solve it20:47
EmilienMhow?20:48
mwhahahaif we followed other openstack processes this should be a blueprint explain the problem and a spec proposal, etc20:48
mwhahahawe kinda skipped over the talking about the problems and what we need to solve20:48
mwhahahaand the side-effects20:49
EmilienMmwhahaha: I don't see much feedback except you on the ML, I'm wondering if 1/ someone cares 2/ they agree 3/ they don't read ML20:51
mwhahaha:(20:51
EmilienMit's called "lazy consensus" :)20:52
EmilienMseriously, I understand your concern, that I share. I'm just wondering: do you have a better solution?20:52
mwhahahaI will take some time to see if i can wedge it something to address my concerns20:52
mwhahahatonight/tomorrow20:53
EmilienMmwhahaha: no problem - we won't merge that thing yet. I just got the feedback we need this feature20:53
mwhahahaUltimately, you're right in that it's a lazy consensus and we'll probably just go with what we have. I'm not going to block it. I just want to point out some issues so when it blows up for us I can say see i tried but no one else said anything :)20:54
EmilienMmwhahaha: though openstack/requirements is a good example of "it already exits"20:54
mwhahahaand by us i mean the fuel ci stuff we've built20:54
EmilienMmwhahaha: I can propose you something20:54
mwhahahai know i've caught flack for the librarian items because we have the requirement to be able to provide an offline solution20:54
EmilienMin the rakefile, we add a test that check if /path/Puppetfile exist20:55
EmilienMif yes, use it20:55
EmilienMif not, use Puppetfile from us20:55
EmilienMso people are free to test what they like20:55
mwhahahaThat i think would at least provide an option20:55
EmilienMand not a bad option20:55
EmilienMyou told us you're using librarian20:56
EmilienMthis option would work for you20:56
EmilienMput your Puppetfile somewhere and do you business20:56
mwhahahayes20:56
mwhahahathat would allow us to use our versions of the modules20:56
EmilienMyeah20:56
EmilienMgreat20:56
EmilienMI'm 1/ doing the code 2/ replying to ML with this new change20:57
EmilienMso people can follow-up with all we said ^^^^^20:57
mwhahahasounds good20:57
EmilienMmwhahaha: awesome, thx a lot20:57
mwhahahaEmilienM: https://review.openstack.org/#/c/219275 there are already '<SERVICE DEFAULT>'s in the code, do you want us just to go back to undef for that change?21:00
EmilienMwell, I don't know if we should convert all the things21:00
EmilienMI was waiting for _ody proposal21:00
mwhahahaI'll just leave it for now, if degorenko fixes it then so be it. i'll add it to the list of things to revisit next week21:02
EmilienMmwhahaha: no prob21:06
*** mdorman has quit IRC21:15
*** derekh has joined #puppet-openstack21:21
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683021:40
openstackgerritAleksandr Didenko proposed openstack/puppet-swift: Add support for DLO configuration.  https://review.openstack.org/17202121:40
*** iraw- has quit IRC21:41
*** iraw- has joined #puppet-openstack21:42
*** derekh has quit IRC21:43
*** sergmelikyan has quit IRC21:47
openstackgerritEmilien Macchi proposed openstack/puppet-nova: DNM - CI test  https://review.openstack.org/22756521:49
*** zhangjn has joined #puppet-openstack22:09
*** zhangjn has quit IRC22:11
*** zhangjn has joined #puppet-openstack22:12
*** zhangjn has quit IRC22:12
*** zhangjn has joined #puppet-openstack22:13
EmilienMcrinkle: https://review.openstack.org/#/c/226945/ pass \o/22:15
EmilienMwait22:15
EmilienMlooking at logs to be sure22:16
EmilienMhttp://logs.openstack.org/65/227565/1/check/gate-puppet-nova-puppet-beaker-rspec-dsvm-trusty/48dd4f4/console.html#_2015-09-24_21_54_35_99322:16
EmilienMpuppetlabs-apt (v2.1.1)22:16
EmilienM\o/22:16
*** zhangjn has quit IRC22:17
*** kindjal has quit IRC22:20
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/22683022:28
*** linkedinyou has joined #puppet-openstack22:36
*** zhangjn has joined #puppet-openstack22:36
*** zhangjn has quit IRC22:37
*** zhangjn has joined #puppet-openstack22:38
*** dfisher has quit IRC22:50
*** chem has quit IRC22:54
*** zhangjn_ has joined #puppet-openstack22:57
*** zhangjn has quit IRC23:01
*** markvoelker_ has quit IRC23:04
*** morazi has quit IRC23:05
*** mdorman has joined #puppet-openstack23:18
*** mdorman has quit IRC23:19
*** xarses has quit IRC23:27
*** mihgen has quit IRC23:56
*** mjb has quit IRC23:56

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