Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-09-24

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:03 gildub joined #puppet-openstack
00:14 richm gildub: 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 more
00:15 EmilienM I invite you to read https://docs.google.com/forms/d/1Z6IGeJRNmX7x​x0Ggmr5Pmpzq7BudphDkZE-3t4Q5G1k/viewanalytics
00:15 EmilienM richm, gildub in particular
00:15 EmilienM it's interesting data
00:17 richm EmilienM: yes
00:18 richm gildub: 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 simpler
00:18 richm then, rebase the 'get rid of indirection' patch on top of the 'change self.instances to be simpler' patch
00:19 richm then, if chem comes back and says "we have to use 'name::domain'" then add back the deprecation warnings
00:19 richm as a separate patch
00:21 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: manifest/db: fix lint WARNINGS  https://review.openstack.org/225127
00:35 xarses joined #puppet-openstack
00:36 xarses joined #puppet-openstack
00:41 linkedinyou joined #puppet-openstack
00:44 openstackgerrit Merged openstack/puppet-openstacklib: Allow to path custom fragment to vhost  https://review.openstack.org/219279
00:44 openstackgerrit Merged openstack/puppet-neutron: Filename change for config-dir for ml2 cisco nexus mech driver.  https://review.openstack.org/220376
00:44 openstackgerrit Merged openstack/puppet-swift: provide means to disable log_requests in config templates  https://review.openstack.org/220621
00:46 openstackgerrit Merged openstack/puppet-ironic: Allow customization of db sync command line  https://review.openstack.org/221245
00:48 openstackgerrit Merged openstack/puppet-horizon: add api_versions parameter  https://review.openstack.org/211650
00:49 openstackgerrit Merged openstack/puppet-neutron: Allow greater flexibility in the kind of parameter passed to ml2.pp  https://review.openstack.org/221318
00:52 zhangjn joined #puppet-openstack
00:53 partner joined #puppet-openstack
00:54 tiswanso joined #puppet-openstack
00:54 tiswanso joined #puppet-openstack
00:56 tiswanso_ joined #puppet-openstack
00:58 gildub richm, I agree with the split then indirection on top
00:58 openstackgerrit Merged openstack/puppet-horizon: add new parameters for multidomain support  https://review.openstack.org/211704
00:58 gildub EmilienM, richm, re stats, very interesting, thanks
00:58 openstackgerrit Merged openstack/puppet-neutron: Add support for rpc_response_timeout option  https://review.openstack.org/223974
00:59 gildub EmilienM, well v2/v3 almost on par, and more need for V3 everywhere, getting there :)
00:59 openstackgerrit Merged openstack/puppet-neutron: Restart Neutron agents if packages are updated  https://review.openstack.org/199337
01:02 openstackgerrit Merged openstack/puppet-neutron: Introduce neutron::db class  https://review.openstack.org/208415
01:18 jfluhmann joined #puppet-openstack
01:28 xarses joined #puppet-openstack
01:29 xarses joined #puppet-openstack
01:51 openstackgerrit Merged openstack/puppet-vswitch: Support for FreeBSD  https://review.openstack.org/191523
01:52 openstackgerrit Merged openstack/puppet-nova: Handle libvirt/cpu_model option  https://review.openstack.org/214542
01:53 openstackgerrit Merged openstack/puppet-manila: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/189872
01:56 openstackgerrit Merged openstack/puppet-cinder: make it possible to have multiple type_sets with the same value  https://review.openstack.org/191498
02:03 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Cleanup configure_delegated_roles deprecated parameter  https://review.openstack.org/227083
02:12 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable telemetry service  https://review.openstack.org/226490
02:12 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/227084
02:13 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable telemetry service  https://review.openstack.org/226490
02:13 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/227084
02:21 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/227084
02:22 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/227084
02:25 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
02:43 EmilienM integ/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:44 EmilienM integ/orchestration is WIP: https://review.openstack.org/#/c/227084/ (almost all test pass)
02:44 EmilienM integ/horizon is stalled > https://bugs.launchpad.net/tempest/+bug/1498297
02:44 openstack Launchpad bug 1498297 in tempest "dashboard scenario does not handle 30x HTTP code" [Undecided,In progress] - Assigned to Mehdi Abaakouk (sileht)
02:45 openstackgerrit Mike Dorman proposed openstack/puppet-glance: Add S3 backend configuration for glance  https://review.openstack.org/226015
02:45 EmilienM the next is integ/swift - but I'm don't know if we can deploy it in one single puppet run
02:53 iurygregory joined #puppet-openstack
02:57 sergmelikyan joined #puppet-openstack
03:01 ferest joined #puppet-openstack
03:02 xarses joined #puppet-openstack
03:11 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Cleanup configure_delegated_roles deprecated parameter  https://review.openstack.org/227083
03:11 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Update rpc_backend default parameter  https://review.openstack.org/227092
03:28 EmilienM gildub: can we use keystone_user_role to assign an a role to an user in a domain (and not in a tenant) ?
03:32 EmilienM gildub: I might have found a bug in fact
03:32 EmilienM in lib/puppet/provider/keysto​ne_user_role/openstack.rb
03:32 EmilienM http://logs.openstack.org/84/227084/4/check/g​ate-puppet-openstack-integration-dsvm-trusty/​ccbbb57/console.html#_2015-09-24_02_32_30_113
03:38 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/227084
03:39 EmilienM gildub: I go to bed now but I feel like the user is not assigned to a role in the domain
03:44 gildub EmilienM, there are issues in user when the tenant doesn't have the domain assigned and when different then default domain
03:44 EmilienM ah
03:44 EmilienM I might hit it now
03:45 EmilienM so I should assign the tenant to the domain, right?
03:45 EmilienM and I'm not using the default domain
03:46 gildub EmilienM, if it's the default domain everywhere that should work but if not then there are definitely issues unless using the ::domain everywhere
03:46 EmilienM gildub: is it WIP or?
03:46 gildub EmilienM, yes, my priority
03:46 EmilienM oh nice
03:47 gildub EmilienM, 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 that
03:49 EmilienM gildub: what is the name of the default domain again ? 'Default' ?
03:50 EmilienM gildub: have you a LP # for this thing?
03:52 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/227084
03:53 gildub EmilienM, default/Default id/name
03:53 gildub EmilienM, LP?
03:56 EmilienM gildub: launchpad
03:57 EmilienM gildub: a bug number
03:58 EmilienM gildub: I'm deploying heat / trusts in our CI > https://review.openstack.org/227084
04:03 gildub EmilienM, no LP
04:04 EmilienM gildub: ok
04:04 EmilienM 2015-09-24 04:02:13.873 | Notice: /Stage[main]/Heat::Keystone::Domain/Keysto​ne_user_role[heat_admin@::Default]/ensure: created
04:05 EmilienM not sure it worked ^
04:05 gildub EmilienM, but basically I'm taking few things out of keystoneV3/default_domain_support (219127, 218044 and 218059)
04:06 gildub EmilienM, without the deprecation warnings, which avoids de-facto domains
04:15 EmilienM gildub: ok... it still does not work for me, I'll continue tomorrow
04:15 EmilienM gildub: cheers!
04:39 chandankumar joined #puppet-openstack
04:40 sanjayu joined #puppet-openstack
04:50 gildub EmilienM, bonne nuit
04:50 khappone joined #puppet-openstack
04:56 alokj joined #puppet-openstack
05:13 openstackgerrit Ben Kero proposed openstack/puppet-swift: Warn that object storage parameter mount_check changes next release  https://review.openstack.org/201755
05:18 bkero moar patches
05:21 xarses_ joined #puppet-openstack
05:26 wabu joined #puppet-openstack
05:27 finchd joined #puppet-openstack
05:32 dgurtner joined #puppet-openstack
05:57 smakar_ joined #puppet-openstack
05:57 openstackgerrit Kuo-Le.Mei proposed openstack/puppet-nova: Fixed value `rbd.keyring` not update  https://review.openstack.org/203909
05:58 openstackgerrit Kuo-Le.Mei proposed openstack/puppet-nova: Fixed value `rbd.keyring` not update  https://review.openstack.org/203909
06:26 nihilifer joined #puppet-openstack
06:30 leanderthal joined #puppet-openstack
06:34 chandankumar joined #puppet-openstack
06:48 smakar_ joined #puppet-openstack
06:53 dgurtner joined #puppet-openstack
06:53 dgurtner joined #puppet-openstack
07:19 alokj_ joined #puppet-openstack
07:28 markvoelker joined #puppet-openstack
07:31 jpena joined #puppet-openstack
07:48 arnaud_orange joined #puppet-openstack
07:57 openstackgerrit Yanis Guenane proposed openstack/puppet-cinder: Introduce cinder::db class  https://review.openstack.org/208354
07:58 duritong olé for merging https://review.openstack.org/191498
08:03 paramite joined #puppet-openstack
08:15 zeus joined #puppet-openstack
08:17 mjb joined #puppet-openstack
08:26 mihgen joined #puppet-openstack
08:37 rochaporto joined #puppet-openstack
08:37 openstackgerrit Yanis Guenane proposed openstack/puppet-ceilometer: Update ceilometer::db class to match other module pattern  https://review.openstack.org/227179
08:38 zhangjn joined #puppet-openstack
08:39 openstackgerrit Yanis Guenane proposed openstack/puppet-ceilometer: Update ceilometer::db class to match other module pattern  https://review.openstack.org/227179
08:51 derekh joined #puppet-openstack
09:02 gfidente joined #puppet-openstack
09:05 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Introduce heat::db class  https://review.openstack.org/227187
09:08 openstackgerrit Yanis Guenane proposed openstack/puppet-heat: Introduce heat::db class  https://review.openstack.org/227187
09:29 markvoelker joined #puppet-openstack
09:47 openstackgerrit Yanis Guenane proposed openstack/puppet-nova: Update nova::db class to match other module pattern  https://review.openstack.org/227196
10:05 smakar_ joined #puppet-openstack
10:09 openstackgerrit Yanis Guenane proposed openstack/puppet-tuskar: Introduce tuskar::db class  https://review.openstack.org/227202
10:15 aderyugin Hi, folks! Review https://review.openstack.org/#/c/216682/ please
10:16 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/227207
10:16 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/227208
10:17 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/227209
10:33 sergmelikyan joined #puppet-openstack
10:35 openstackgerrit Yanis Guenane proposed openstack/puppet-tuskar: Introduce tuskar::db class  https://review.openstack.org/227202
10:46 openstackgerrit Michael Polenchuk proposed openstack/puppet-nova: Add kombu_reconnect_delay option  https://review.openstack.org/213102
10:48 openstackgerrit Yanis Guenane proposed openstack/puppet-designate: Update designate::db class to match other module pattern  https://review.openstack.org/227217
10:51 openstackgerrit Yanis Guenane proposed openstack/puppet-designate: Update designate::db class to match other module pattern  https://review.openstack.org/227217
11:19 amaksimov joined #puppet-openstack
11:20 gildub joined #puppet-openstack
11:27 zhangjn joined #puppet-openstack
11:28 cdelatte joined #puppet-openstack
11:30 markvoelker joined #puppet-openstack
11:36 EmilienM good morning
11:36 iurygregory good morning o/
11:39 iurygregory hey aderyugin, i'll give +1 if you change the commit message "Implementaion" is wrong ^^
11:42 degorenko morning o/
11:45 iurygregory thanks for the review degorenko o/
11:46 degorenko iurygregory, np o/
11:47 jtomasek joined #puppet-openstack
11:48 gfidente joined #puppet-openstack
11:48 gfidente joined #puppet-openstack
11:53 degorenko iurygregory, i left some comments more :D take a look please
11:53 iurygregory sure
11:54 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Cleanup configure_delegated_roles deprecated parameter  https://review.openstack.org/227083
11:55 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Update rpc_backend default parameter  https://review.openstack.org/227092
12:07 chandankumar joined #puppet-openstack
12:09 dprince joined #puppet-openstack
12:14 ducttape_ joined #puppet-openstack
12:28 bpiotrowski joined #puppet-openstack
12:30 ducttape_ joined #puppet-openstack
12:31 markvoelker joined #puppet-openstack
12:32 openstackgerrit Yanis Guenane proposed openstack/puppet-cinder: Put all the logging related parameters to the logging class  https://review.openstack.org/227265
12:39 openstackgerrit Emilien Macchi proposed openstack/puppet-heat: Update rpc_backend default parameter  https://review.openstack.org/227092
12:45 degorenko spredzy, hey o/ are you going to create logging class for all projects or for cinder only for now?
12:45 spredzy degorenko, 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 class
12:46 degorenko spredzy, really? wow, i didn't saw this class before :(
12:47 degorenko i'll do the same for sahara then
12:48 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: compute agent: do not try to configure nova.conf  https://review.openstack.org/226512
12:48 spredzy degorenko, for heat for example https://github.com/openstack/puppet-h​eat/blob/master/manifests/logging.pp
12:48 degorenko yeah, cool!
12:48 spredzy but I know not every module has it yet
12:49 degorenko spredzy, what about creating separate class for rpc?
12:49 spredzy degorenko, 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 unit
12:49 degorenko urrrhhhh, gerrit so slow today
12:50 spredzy not a big blob file
12:50 spredzy yep reallyyyy slow
12:50 EmilienM yes, this is a good topic for the summit
12:50 degorenko i'll add this then :)
12:50 EmilienM I think spredzy already did
12:50 spredzy EmilienM, degorenko I already wrote it
12:51 degorenko spredzy, it is contain in 'Reshaping of our current init.pp' probably?
12:51 degorenko contains*
12:51 spredzy degorenko, yep
12:51 degorenko spredzy, ok! for me it is good point too
12:53 degorenko spredzy, as i understand, we don't want 'include logging' in main init class?
12:58 openstackgerrit Merged openstack/puppet-openstack-integration: Puppetfile: bump qpid to 1.0.3  https://review.openstack.org/226927
12:59 spredzy degorenko, 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 logging
13:00 degorenko spredzy, yes, but you use pick function, so, what are problems actually?
13:01 spredzy degorenko, we need to remain backward compatible
13:01 spredzy and still support non hiera deployment
13:01 degorenko spredzy,  but if we will use include with pick - backward compatibility will not broken
13:02 iurygregory hey people can anyone review https://review.openstack.org/#/c/223777/ ?
13:03 obbs joined #puppet-openstack
13:03 cdelatte joined #puppet-openstack
13:05 khushbu_ joined #puppet-openstack
13:06 richm joined #puppet-openstack
13:12 ducttape_ joined #puppet-openstack
13:16 morazi joined #puppet-openstack
13:19 tiswanso joined #puppet-openstack
13:37 pradk joined #puppet-openstack
13:37 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: rspec: run tests for ::ceilometer::agent::auth  https://review.openstack.org/227306
13:37 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: agent/auth: bring consistent how we manage empty parameters  https://review.openstack.org/227307
13:37 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement auth_endpoint_type parameter  https://review.openstack.org/227308
13:45 zhangjn joined #puppet-openstack
13:47 zhangjn joined #puppet-openstack
13:47 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Introduce Sahara extended logging class  https://review.openstack.org/227310
13:48 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Service Provider  https://review.openstack.org/216821
13:48 zhangjn joined #puppet-openstack
13:49 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Introduce Sahara extended logging class  https://review.openstack.org/227310
13:50 openstackgerrit Zhao Lei proposed openstack/puppet-neutron: Fix a spelling typo in config's help  https://review.openstack.org/227314
13:54 dfisher joined #puppet-openstack
13:55 khushbu_ joined #puppet-openstack
13:59 zhangjn joined #puppet-openstack
14:17 ducttape_ joined #puppet-openstack
14:20 bpiotrowski joined #puppet-openstack
14:20 bpiotrowski joined #puppet-openstack
14:29 cdelatte joined #puppet-openstack
14:40 aderyugin Hi, EmilienM, please take a look at https://review.openstack.org/#/c/216682/
14:40 zhangjn joined #puppet-openstack
14:41 kindjal joined #puppet-openstack
14:41 zhangjn joined #puppet-openstack
14:42 zhangjn joined #puppet-openstack
14:42 zhangjn joined #puppet-openstack
14:43 social joined #puppet-openstack
14:44 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/226830
14:44 EmilienM chem: ^
14:44 chem ack, but it should be good, shouldn't it :)
14:44 zhangjn joined #puppet-openstack
14:50 degorenko EmilienM, i have a question about a new prepare fixtures. You removed .fixtures.yaml, how it will know which modules are required? I don't understand
14:50 EmilienM degorenko: have you looked at https://review.openstack.org/226830 ?
14:50 degorenko EmilienM, yep, i saw this
14:51 EmilienM https://review.openstack.o​rg/#/c/226830/18/Rakefile,cm
14:51 degorenko i understand how will be cloned openstack-integration
14:51 EmilienM have you read install_modules.sh script?
14:51 degorenko yes
14:51 EmilienM so?
14:51 EmilienM it uses r10k to install modules in a dir you want
14:52 EmilienM that's what our beaker + integration jobs is using
14:52 degorenko yes, but modules names taken here: https://github.com/openstack/puppet-openstack-​integration/blob/master/install_modules.sh#L23 ?
14:52 EmilienM rspec will use it to clone modules in spec/fixtures/modules
14:52 EmilienM degorenko: yes
14:53 degorenko EmilienM, ok, so now will be installed all modules, right?
14:53 EmilienM right
14:53 degorenko ah, i see
14:53 degorenko thanks
14:53 EmilienM it will clone all modules
14:53 EmilienM it's overkill but consistent
14:53 degorenko yep, just missed PuppetFile from this repo
14:53 EmilienM cool
14:54 degorenko i saw your mail, just want to know, how it works exactly
14:54 degorenko now i know :) thanks again
14:54 EmilienM degorenko: cool :)
14:55 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/226830
14:55 ducttape_ joined #puppet-openstack
14:56 cdelatte joined #puppet-openstack
15:00 EmilienM something is broking our CI
15:00 EmilienM on trusty, Rabbitmq
15:00 EmilienM http://logs.openstack.org/92/227092/3/check/ga​te-puppet-heat-puppet-beaker-rspec-dsvm-trusty​/04b0498/console.html#_2015-09-24_13_51_39_218
15:00 degorenko yes, i saw
15:00 EmilienM degorenko: any idea?
15:01 * degorenko looking at logs
15:01 EmilienM it seems like something in RabbitMQ version
15:01 degorenko 2015-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 operator
15:01 degorenko something missed?
15:02 degorenko host
15:02 EmilienM 3.5.4-1 -> 3.5.5-1
15:02 EmilienM rabbitmq upgrade
15:02 EmilienM the problem was not in 3.5.4-1
15:02 * EmilienM reading diff
15:03 EmilienM Hunner: ^
15:03 degorenko EmilienM, i don't see such tags in puppetlabs-rabbitmq
15:03 EmilienM degorenko: not in puppet
15:03 degorenko where i should check?
15:04 EmilienM in the package.
15:04 degorenko ah
15:04 EmilienM I'm looking
15:04 EmilienM https://github.com/rabbitmq/rabbitmq​-server/releases/tag/rabbitmq_v3_5_5
15:04 degorenko thanks
15:06 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: DNM - CI test  https://review.openstack.org/227357
15:07 EmilienM Hunner: RabbiMQ 3.5.5-1 was released 3 hours ago and breaks puppetlabs-rabbitmq
15:07 EmilienM Could not evaluate: cannot parse line from list_user_permissions:/usr/sbin/rabbitmqctl: 19: [: Linux: unexpected operator
15:09 zhangjn_ joined #puppet-openstack
15:12 EmilienM something in https://github.com/rabbitmq/rabbitmq-server​/compare/rabbitmq_v3_5_4...rabbitmq_v3_5_5
15:12 degorenko EmilienM, yeah i looking at it too
15:26 zhangjn joined #puppet-openstack
15:27 degorenko there are some changes in scripts/rabbitmq-server script
15:28 degorenko but extra arguments don't change
15:28 zhangjn joined #puppet-openstack
15:28 zhangjn joined #puppet-openstack
15:29 angdraug joined #puppet-openstack
15:30 dgurtner joined #puppet-openstack
15:35 zhangjn_ joined #puppet-openstack
15:37 EmilienM degorenko: I'm reproducing it in my dev env
15:37 EmilienM still investigating
15:37 IvanBerezovskiy joined #puppet-openstack
15:38 EmilienM root@em-vag-u1404-dev:~# /usr/sbin/rabbitmqctl -q list_user_permissions cinder
15:38 EmilienM /usr/sbin/rabbitmqctl: 19: [: Linux: unexpected operator
15:38 EmilienM \o/
15:39 degorenko EmilienM, me too
15:39 degorenko trying to install 3.54.
15:39 degorenko 3.5.4
15:39 degorenko EmilienM, you can run just  /usr/sbin/rabbitmqctl -q list_users
15:39 degorenko root@puppet-test-vm:/home/ubuntu#  /usr/sbin/rabbitmqctl -q list_users
15:39 degorenko /usr/sbin/rabbitmqctl: 19: [: Linux: unexpected operator
15:39 degorenko guest[administrator]
15:40 EmilienM I found it
15:40 EmilienM \o/
15:40 degorenko EmilienM, what is it?
15:40 EmilienM degorenko: if you replace sh by bash in L1, it works
15:40 dfisher it secretly installed Solaris when you weren't looking
15:40 dfisher :ssh:
15:41 degorenko EmilienM, you're right
15:41 degorenko it works for me too
15:42 degorenko but in 3.5.4 - sh too
15:43 degorenko EmilienM, https://github.com/rabbitmq/rabbitmq-server/co​mmit/b82a9fdb0fbc0f675facccdbcdf8dd8d753f2899
15:43 EmilienM oh
15:44 EmilienM nice shot
15:44 degorenko :)
15:44 EmilienM degorenko: I do a PR, or you?
15:44 EmilienM I'm ready
15:44 degorenko EmilienM, go ahead :)
15:44 EmilienM cool
15:44 EmilienM our CI is broken anyway
15:44 degorenko yea
15:45 degorenko Can you put please the link on review here? :)
15:46 EmilienM degorenko: sure
15:46 zhangjn joined #puppet-openstack
15:48 EmilienM degorenko: https://github.com/rabbitmq​/rabbitmq-server/issues/321
15:49 degorenko EmilienM, hey, there are change this: https://github.com/rabbitmq/rabbitmq-serve​r/compare/rabbitmq_v3_5_4...rabbitmq_v3_5_​5#diff-0dd46c0d3d8b9bb6b2b9dcf5701cb049R25
15:49 degorenko but anyway - it doens't work properly
15:50 degorenko EmilienM, may be we also should add output from cmd? I'll add comment
15:50 EmilienM oh it's fixed already
15:51 EmilienM 6964e8ac1559cf76b9245394b46aaec5822f4178
15:51 degorenko i put output from simple list_users
15:51 degorenko yes
15:51 degorenko it is
15:51 degorenko but it is not working :D
15:52 EmilienM I'm contacting ubuntu packages
15:52 degorenko oh, you mean that we have broken tag, right?
15:57 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Changes in enabling federation spec  https://review.openstack.org/223777
15:57 degorenko btw, it was fixed 3 days ago
15:58 degorenko how they test tags before release? lol
16:01 degorenko joined #puppet-openstack
16:02 EmilienM degorenko: I found a workaround
16:02 EmilienM degorenko: yeah...
16:02 EmilienM I wonder if they have CI
16:02 chem joined #puppet-openstack
16:02 EmilienM in rabbimt puppet module there is manifests/init.pp +173
16:03 EmilienM it does not take the version from ::params
16:03 degorenko EmilienM, my PC has had a reboot ...
16:03 degorenko so what is workaround?
16:03 EmilienM it happens :P
16:03 degorenko yeah :(
16:03 EmilienM I'm preparing a patch
16:03 * degorenko waiting
16:03 EmilienM the $version seems not used
16:03 degorenko some kind pin rabbitmq package version?
16:04 EmilienM probably, yes
16:04 degorenko one more hack for package :) nice
16:05 EmilienM degorenko: brb
16:06 degorenko :D
16:08 cdelatte joined #puppet-openstack
16:08 degorenko EmilienM, btw, https://github.com/rabbitmq/rabbitmq-server/co​mmit/6964e8ac1559cf76b9245394b46aaec5822f4178 - is not working
16:08 degorenko ha ha
16:08 degorenko 3.5.5 contains this fix
16:09 openstackgerrit Michael Polenchuk proposed stackforge/puppet-openstack: Add default domain into openrc  https://review.openstack.org/227400
16:09 EmilienM it's 6964e8ac1559cf76b9245394b46aaec5822f4178 right ?
16:09 degorenko yes
16:09 degorenko let me a few minutes
16:09 degorenko i'll try to use old way
16:10 EmilienM maybe in the pacakge
16:12 degorenko lol
16:12 degorenko with old
16:13 degorenko rabbitmqctl -q list_users
16: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 file
16:13 EmilienM what?
16:13 degorenko two rabbits - old and new
16:13 degorenko copy-paste
16:13 degorenko result above
16:13 EmilienM wow
16:13 degorenko lol
16:13 degorenko how?
16:14 Hunner Lovely
16:14 EmilienM Hunner: tl;dr: our CI is broken
16:14 EmilienM degorenko: how come it worked before
16:14 Hunner Oh, not rabbitmq 3.5.5?
16:14 EmilienM we can diff packages
16:14 degorenko i don't know how
16:15 * degorenko trying ti understand
16:15 degorenko to*
16:15 EmilienM I'm diffing
16:15 degorenko ok, only one error. Don't use bash :)
16:15 degorenko rabbitmqctl -q list_users
16:15 degorenko /usr/sbin/rabbitmqctl: 47: /usr/sbin/rabbitmqctl: Syntax error: end of file unexpected (expecting ")")
16:15 EmilienM https://www.diffchecker.com/wd2vfkru
16:16 EmilienM at the left, when our CI was working
16:16 EmilienM at the right, when it broke
16:16 EmilienM it's pretty obvious :)
16:17 EmilienM degorenko: are you sure it's broken in 3.5.4 ?
16:17 EmilienM did you use the same VM ?
16:17 degorenko need to scroll down - i'm looking on the top of page  and think -erm, what is it? :D
16:18 degorenko no
16:18 degorenko i use two
16:18 degorenko but with the same repos
16:18 EmilienM mhh
16:18 degorenko trusty 14.04
16:18 EmilienM dpkg -l
16:18 EmilienM and diff
16:18 EmilienM see if we are missing anything
16:19 EmilienM https://www.diffchecker.com/wd2vfkru is showing it's this package though.
16:19 EmilienM have you tried 3.5.4-1 ?
16:19 degorenko yes
16:19 degorenko it works
16:20 EmilienM degorenko: 3.5.5-1 has ==
16:20 EmilienM so it's broken
16:20 degorenko oh, first vm has many software for diskimage-builder...
16:20 degorenko i forgot about it
16:20 degorenko need create new vm
16:20 degorenko but actually
16:21 degorenko it will have diff only for rabbit
16:21 markvoelker_ joined #puppet-openstack
16:21 degorenko i'm sure
16:21 EmilienM I confirm 3.5.4-1 does not have ==
16:21 EmilienM so 3.5.4-1 should wor
16:21 EmilienM work*
16:21 degorenko what do you mean by ==?
16:21 EmilienM if [ $(uname -s) == "Linux" ]; then
16:21 EmilienM that error ^
16:21 degorenko yes
16:21 EmilienM is not in 3.5.4
16:21 degorenko it is right
16:21 EmilienM so it should work in the previous
16:22 degorenko but i've removed this!
16:22 EmilienM but you say 3.5.4 is broken
16:22 degorenko and got error
16:22 degorenko no no no
16:22 degorenko :)
16:22 EmilienM ahhhhhh
16:22 EmilienM so let's pin 3.5.4
16:22 degorenko 3.5.5 - broken
16:22 EmilienM ok
16:22 degorenko with sh from 3.5.4
16:22 EmilienM good
16:22 degorenko i don't know why
16:22 degorenko we can pin
16:22 degorenko yes
16:22 EmilienM I'm preparing a patch
16:23 degorenko so interesting package debugging :)
16:23 cdelatte joined #puppet-openstack
16:24 EmilienM degorenko: yeah
16:24 iraw- joined #puppet-openstack
16:24 EmilienM our CI is really on the edge lol
16:24 aderyugin joined #puppet-openstack
16:24 degorenko :D
16:24 joshsamuelson joined #puppet-openstack
16:24 kbyrne joined #puppet-openstack
16:25 duritong joined #puppet-openstack
16:25 EmilienM we're testing the world
16:25 degorenko \o/
16:25 bkero joined #puppet-openstack
16:27 ducttape_ joined #puppet-openstack
16:31 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Pin RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/227417
16:31 EmilienM degorenko: please review ^
16:31 * degorenko looking
16:32 EmilienM degorenko: they replied, lol they are fixing it!
16:32 degorenko EmilienM, they replied - wow
16:32 EmilienM yeah
16:32 EmilienM at least they are responsive
16:32 degorenko yes :)
16:33 EmilienM degorenko: should we tell them to have a CI ? lol
16:33 degorenko because i want already will try to fix myself :D
16:33 degorenko no
16:33 EmilienM degorenko: was joking
16:33 degorenko don't disappoint them
16:33 EmilienM maybe
16:34 EmilienM if my patch ^ works
16:34 EmilienM I'll push it to all modules
16:34 degorenko to acceptance tests, yes?
16:34 EmilienM degorenko: yeah
16:35 EmilienM except if they fix the package
16:35 EmilienM probably we won't need to fix all our repos
16:35 EmilienM if they re-build the package today
16:35 EmilienM I'll wait until this afternoon
16:35 degorenko are you sure that they will update package in a mirror today? :)
16:35 EmilienM we need to monitor http://www.rabbitmq.com/rele​ases/rabbitmq-server/v3.5.5/
16:36 EmilienM "We are recreating the RPM and Debian packages with the script fixed"
16:36 degorenko yea
16:36 degorenko i saw
16:36 degorenko but who is guaranty that it will be done today?
16:36 mdorman joined #puppet-openstack
16:37 EmilienM degorenko: nothing, I'm waiting a bit and then we will have to unblock our CI
16:37 EmilienM it could be great to avoid patching all our modules again
16:42 EmilienM degorenko: my fix does not work: https://jenkins07.openstack.org/job/gate-puppet-o​penstack-integration-dsvm-trusty/116/consoleFull
16:42 EmilienM grep "Error:"
16:42 EmilienM not sure my param worked
16:42 EmilienM I found out
16:42 EmilienM a typo
16:43 serg_melikyan joined #puppet-openstack
16:43 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Pin RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/227417
16:43 degorenko where?
16:43 EmilienM https://review.openstack.org/#/c/227417/1..2/fixtures/scenario001.pp,cm
16:43 degorenko oh
16:43 degorenko ensure
16:43 EmilienM my bad
16:43 EmilienM degorenko: I go eat my lunch, I'm back later o/
16:44 degorenko EmilienM, ok, have a nice lunch o/
16:45 degorenko EmilienM, lol, i find what the problem actually.... Epic lol
16:46 degorenko and they already fixed it in master
16:47 mdorman_ joined #puppet-openstack
16:48 degorenko EmilienM, https://github.com/rabbitmq/rabbitmq-server/co​mmit/67c24aa1ccf5209660d54ad49033a1d1e3cc0502
16:48 degorenko it is in master
16:51 imcsk8 EmilienM: 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 providers
16:56 EmilienM degorenko: could you ask murano maintainers to modify https://launchpad.net/puppet-murano so puppet group could manage it
16:56 EmilienM degorenko: take example from https://launchpad.net/puppet-nova
16:57 degorenko EmilienM, yes, sure :) did you saw my message about rabbit?
16:57 EmilienM no
16:58 EmilienM ah
16:58 EmilienM in github, yes
16:58 degorenko nice? :) do they know sh? or don't
16:58 degorenko and, how they tested it?
16:58 degorenko i don't any idea
16:59 zhangjn_ joined #puppet-openstack
17:02 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/223690
17:07 EmilienM degorenko: E: Version '3.5.4-1' for 'rabbitmq-server' was not found
17:07 degorenko lol
17:08 * degorenko checking rabbitmq version
17:08 zhangjn joined #puppet-openstack
17:08 degorenko yeah
17:08 degorenko thats true
17:09 degorenko EmilienM, 3.5.4-1~cloud0.
17:09 ducttape_ joined #puppet-openstack
17:09 degorenko without '.'
17:09 EmilienM oh
17:09 degorenko we need ~cloud0
17:09 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Pin RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/227417
17:10 EmilienM I go lunch for real
17:11 sergmelikyan joined #puppet-openstack
17:13 mwhahaha wouldn't you just require 3.5.4? with no -1~cloud0?
17:13 degorenko mwhahaha, take a look on patch set 2
17:13 degorenko mwhahaha, E: Version '3.5.4-1' for 'rabbitmq-server' was not found
17:14 mwhahaha right i'm saying isn't it 3.5.4
17:14 degorenko E: Version '3.5.4' for 'rabbitmq-server' was not found
17:14 mwhahaha not 3.5.4-1
17:14 degorenko the same
17:14 mwhahaha that's annoying
17:15 degorenko it happens :)
17:15 jfluhmann joined #puppet-openstack
17:18 openstackgerrit Alex Schultz proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/216654
17:24 EmilienM I could also trick something else to not have to patch our modules
17:24 EmilienM in install_modules.sh
17:24 EmilienM since puppetlabs-rabbitmq is just checking the package is 'installed' by default (when running yum & apt package providers)
17:25 EmilienM we could install the .deb manually before the puppet run
17:25 degorenko in each test?
17:25 EmilienM no
17:25 EmilienM let me show you
17:26 * degorenko looking
17:27 patrickeast hi, 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:28 patrickeast or that i only check that there is a cinder::backend::pure and don't validate the variables
17:29 mwhahaha patrickeast: 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 resources
17:29 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Install RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/227430
17:29 EmilienM degorenko: ^
17:30 ducttape_ joined #puppet-openstack
17:31 patrickeast mwhahaha: ok so just a single check for the backend
17:31 degorenko looking
17:31 degorenko EmilienM, oh! of course!
17:31 EmilienM degorenko: I like #2 more than #1
17:32 EmilienM so we don't have to patch all our modules
17:32 degorenko yeah
17:32 EmilienM but it fails
17:32 degorenko what aboud dependencies?
17:32 degorenko about*
17:34 EmilienM yeah
17:34 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Install RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/227430
17:35 degorenko EmilienM, also we can use '-d' flag
17:35 EmilienM damn it fails
17:35 degorenko in this case we will have all dependencies
17:35 degorenko it is in apt-get
17:36 degorenko EmilienM, apt-get install -d rabbitmq-server and then our rabbitmq
17:36 EmilienM yeah, we can also install the one from ubuntu
17:36 EmilienM good idea
17:37 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Install RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/227430
17:38 degorenko erm
17:38 degorenko i don't think that will work
17:38 degorenko we already have repo with broken rabbit
17:38 degorenko or not?
17:38 EmilienM https://jenkins06.openstack.org/job/gate-puppet​-openstack-integration-dsvm-trusty/118/console
17:39 EmilienM 3.2.4-1
17:39 EmilienM it works
17:39 EmilienM well, we have an old version but we can live with that now
17:39 openstackgerrit Patrick East proposed openstack/puppet-cinder: Add support for Pure Volume Drivers  https://review.openstack.org/225368
17:40 degorenko EmilienM, hm, yea
17:40 cdelatte joined #puppet-openstack
17:42 mdorman_ joined #puppet-openstack
17:46 EmilienM degorenko: my second patch works
17:47 degorenko \o/
17:47 degorenko cool
17:48 degorenko EmilienM, can we run tests for projects?
17:49 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: Add ironic-inspector support  https://review.openstack.org/223690
17:49 chem joined #puppet-openstack
18:04 EmilienM degorenko: with depends-on
18:04 EmilienM crinkle, mgagne, sbadia, clayton, mfisch: please look https://review.openstack.org/#/c/227430/ - our CI is blocked right now
18:05 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Introduce Sahara extended logging class  https://review.openstack.org/227310
18:05 tiswanso joined #puppet-openstack
18:05 degorenko EmilienM, ^^ i've added depends-on
18:06 mgagne done
18:07 EmilienM well, they just updated https://github.com/rabbitmq​/rabbitmq-server/issues/321 to say a new package is coming
18:12 degorenko "We really need to improve the testing of the Debian and RPM packages " - oh, really?
18:13 EmilienM lol
18:13 EmilienM s/improve/start
18:13 degorenko lol
18:13 dmsimard :(
18:14 cdelatte joined #puppet-openstack
18:18 EmilienM degorenko: I would like your help on integration work
18:18 EmilienM degorenko: to bring sahara support
18:18 degorenko EmilienM, sure :)
18:18 EmilienM see https://review.openstack.org/226490 for example
18:18 EmilienM you need to add the class, and enable it in puppet-tempest
18:19 openstackgerrit Merged openstack/puppet-openstack-integration: Install RabbitMQ 3.5.4 on Ubuntu systems  https://review.openstack.org/227430
18:19 EmilienM so we could add this module to the party
18:20 degorenko EmilienM, yep :) can i do it tomorrow? :D because 9 pm at Moscow zone :)
18:21 EmilienM degorenko: lol I was just asking if you were interested to help
18:21 EmilienM not tonight !!
18:22 degorenko i'm :) i will do it tomorrow
18:22 EmilienM degorenko: I need to warn you :
18:22 EmilienM 1/ you'll need to patch puppet-tempest, so use Depends-On on the integ patch you'll do
18:22 EmilienM 2/ the puppet run needs to be idempotent
18:22 degorenko btw, i have a good progress with auth based on openstack client for now :)
18:23 degorenko got it !
18:23 EmilienM 3/ please follow the parameters values (for passwords, etc) to be consistent
18:23 EmilienM awesome
18:23 degorenko for nova*
18:23 degorenko sorry, typo
18:23 EmilienM degorenko: do we need swift for running sahara?
18:23 degorenko no
18:23 EmilienM good
18:23 EmilienM because we don't have swift
18:23 EmilienM yet
18:23 degorenko we can use swift, but optional
18:25 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Use OpenstackClient for nova auth  https://review.openstack.org/226862
18:26 degorenko EmilienM, it should work for nova_aggregate provider now ^^
18:26 degorenko take a look quickly please, if you have time
18:26 degorenko may be some ruby issues
18:26 EmilienM we have ruby experts here :)
18:27 degorenko i'll add all tests and another providers later :)
18:30 ducttape_ joined #puppet-openstack
18:35 degorenko EmilienM, i go home, if you will have any questions, send personal message :) i'll answer tomorrow
18:36 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable telemetry service  https://review.openstack.org/226490
18:36 EmilienM degorenko: good night
18:36 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable orchestration service  https://review.openstack.org/227084
18:36 degorenko EmilienM, buy o/
18:36 degorenko bye* :D
18:38 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: compute agent: do not try to configure nova.conf  https://review.openstack.org/226512
18:39 xarses joined #puppet-openstack
18:44 ducttape_ joined #puppet-openstack
18:50 IvanBerezovskiy left #puppet-openstack
19:05 EmilienM Hunner, _ody: do you have thoughts about the fixtures thread on openstack ML ?
19:07 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: rspec: run tests for ::ceilometer::agent::auth  https://review.openstack.org/227306
19:07 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: agent/auth: bring consistent how we manage empty parameters  https://review.openstack.org/227307
19:07 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement auth_endpoint_type parameter  https://review.openstack.org/227308
19:12 ducttape_ joined #puppet-openstack
19:15 openstackgerrit Merged openstack/puppet-nova: rabbitmq: do not manage rabbitmq service anymore  https://review.openstack.org/227007
19:17 iurygregory EmilienM, 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 EmilienM iurygregory: cool
19:18 iurygregory thanks for the review =D
19:19 EmilienM lol they fixed rabbitmq
19:20 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Revert "Install RabbitMQ 3.5.4 on Ubuntu systems"  https://review.openstack.org/227495
19:28 aglarendil joined #puppet-openstack
19:35 openstackgerrit Merged openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/227207
19:35 openstackgerrit Merged openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/227208
19:35 openstackgerrit Merged openstack/puppet-nova: Remove dead nova providers  https://review.openstack.org/227209
19:50 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: [WIP] Support for Keystone as Identity Provider  https://review.openstack.org/208054
19:51 mdorman joined #puppet-openstack
19:51 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/226830
19:52 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Identity Provider  https://review.openstack.org/208054
19:52 EmilienM crinkle: for https://review.openstack.org/219275 - don't you want to wait we sort out the SERVICE_DEFAULT thing ?
19:52 EmilienM crinkle: we are waiting for _ody proposal on Gerrit I guess
19:54 crinkle EmilienM: oh I thought using the string was spredzy's original plan and anything else was just optimizing
19:55 EmilienM crinkle: I'm hope we can find a concensus quickly, I like _ody's proposal which is to use openstacklib
20:03 openstackgerrit Merged openstack/puppet-cinder: Add support for Pure Volume Drivers  https://review.openstack.org/225368
20:08 sergmelikyan joined #puppet-openstack
20:09 openstackgerrit Merged openstack/puppet-designate: add designate_rootwrap_config in designate::config  https://review.openstack.org/208178
20:13 openstackgerrit Merged openstack/puppet-nova: Add kombu_reconnect_delay option  https://review.openstack.org/213102
20:13 openstackgerrit Merged openstack/puppet-neutron: Accommodate for VPNaaS package name change in RDO  https://review.openstack.org/202065
20:15 EmilienM crinkle: can you +a https://review.openstack.org/#/c/227495/ please ?
20:17 EmilienM crinkle: 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 yet
20:17 openstackgerrit Merged openstack/puppet-swift: Warn that object storage parameter mount_check changes next release  https://review.openstack.org/201755
20:19 openstackgerrit Merged openstack/puppet-openstack-integration: Revert "Install RabbitMQ 3.5.4 on Ubuntu systems"  https://review.openstack.org/227495
20:20 EmilienM actually, it's more than ceilometer
20:20 EmilienM it seems like a lot of regressions on their side, nothing we can really do now
20:21 EmilienM dmsimard: http://logs.openstack.org/90/226490/13​/check/gate-puppet-openstack-integrati​on-dsvm-centos7/5fdd4d2/console.html
20:21 EmilienM dmsimard: it's weird, but some days ago scenarios were all working well
20:21 EmilienM I got some Got server fault
20:22 EmilienM looks like in cinder
20:23 EmilienM http://logs.openstack.org/90/226490/13/check/gate​-puppet-openstack-integration-dsvm-centos7/5fdd4d​2/logs/cinder/api.txt.gz#_2015-09-24_19_37_11_123
20:23 EmilienM dmsimard: maybe something in taskflow, any recent package?
20:24 bnemec joined #puppet-openstack
20:28 EmilienM it's in python-taskflow
20:31 openstackgerrit Merged openstack/puppet-nova: remove openstackocci installation feature  https://review.openstack.org/194977
20:36 EmilienM mwhahaha: reading you reply, I still don't understand what you actually propose
20:36 mwhahaha don't make it manditory
20:36 EmilienM well, your feedback is valuable
20:36 mwhahaha leave fixtures, allow for zuul version to be overriden
20:36 EmilienM if you read the rakefile, we actually allow to not run zuul cloner on the machine but git
20:36 EmilienM well, technically r10k
20:37 mwhahaha let me pull up the change again
20:37 chem joined #puppet-openstack
20:37 mwhahaha my concern is that this is an openstack CI specific thing
20:37 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/226830
20:38 EmilienM mwhahaha: I would rather say "it's an openstack specific thing"
20:38 mwhahaha but we're solving a CI problem
20:38 EmilienM I see some people who complain because we have module dependencies
20:38 EmilienM the question is "ok, how do you want to do?"
20:38 EmilienM it's not that easy to run without dependencies
20:39 EmilienM mwhahaha: we're solving a code issue also, let me give you an example
20:39 mwhahaha right and it's only because we're dealing with what 20+ modules?
20:39 EmilienM mwhahaha: 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 master
20:39 EmilienM so we had to update the code with the fix
20:40 EmilienM mwhahaha: 25
20:40 EmilienM https://wiki.openstack.org/wiki/Puppet/releases
20:40 EmilienM I would like to see a serious proposal
20:41 mwhahaha serious proposal to solve what
20:41 EmilienM the use case I just mentionned
20:41 EmilienM puppet-nova was unit tested against puppet-qpid X while functionaly tested against Y
20:41 EmilienM the manifest was not good because of this
20:42 EmilienM mwhahaha: do you know openstack/requirements ?
20:42 mwhahaha So the issue is trying to manage puppet dependencies that are shared between 25 modules
20:42 mwhahaha right?
20:42 EmilienM https://github.com/openstack/requ​irements#why-global-requirements
20:42 EmilienM right
20:42 EmilienM keep consistent our testing
20:43 EmilienM reading https://github.com/openstack/requ​irements#why-global-requirements - this is exactly our issue at this time.
20:43 EmilienM we're not running tox, but rake though.
20:44 mwhahaha basically we have to re-invent what they have for the python side in puppet
20:45 EmilienM mwhahaha: right, because fixtures is not enough for us
20:45 EmilienM proposal #2: bring zuul cloner support in rspec
20:45 mwhahaha so no it may be, it depends on what you are trying to fix.
20:45 mwhahaha so there are two issues
20:45 EmilienM but I personally don't have the bandwith
20:46 EmilienM well, it's written in my email, but I can explain again.
20:46 mwhahaha no it's ok
20:46 mwhahaha what i'm trying to get at is there are multiple problems
20:46 EmilienM 1/ dependencies are not consistent across our .fixtures.yaml in modules
20:46 mwhahaha 1) the management of versions
20:46 mwhahaha 2) change dependency inclusion for testing
20:47 EmilienM 3/ what is testing in unit is not consistent with functional
20:47 mwhahaha So i want to make it so that _you_ don't have to be the one to solve it
20:48 EmilienM how?
20:48 mwhahaha if we followed other openstack processes this should be a blueprint explain the problem and a spec proposal, etc
20:48 mwhahaha we kinda skipped over the talking about the problems and what we need to solve
20:49 mwhahaha and the side-effects
20:51 EmilienM mwhahaha: 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 ML
20:51 mwhahaha :(
20:52 EmilienM it's called "lazy consensus" :)
20:52 EmilienM seriously, I understand your concern, that I share. I'm just wondering: do you have a better solution?
20:52 mwhahaha I will take some time to see if i can wedge it something to address my concerns
20:53 mwhahaha tonight/tomorrow
20:53 EmilienM mwhahaha: no problem - we won't merge that thing yet. I just got the feedback we need this feature
20:54 mwhahaha Ultimately, 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 EmilienM mwhahaha: though openstack/requirements is a good example of "it already exits"
20:54 mwhahaha and by us i mean the fuel ci stuff we've built
20:54 EmilienM mwhahaha: I can propose you something
20:54 mwhahaha i know i've caught flack for the librarian items because we have the requirement to be able to provide an offline solution
20:55 EmilienM in the rakefile, we add a test that check if /path/Puppetfile exist
20:55 EmilienM if yes, use it
20:55 EmilienM if not, use Puppetfile from us
20:55 EmilienM so people are free to test what they like
20:55 mwhahaha That i think would at least provide an option
20:55 EmilienM and not a bad option
20:56 EmilienM you told us you're using librarian
20:56 EmilienM this option would work for you
20:56 EmilienM put your Puppetfile somewhere and do you business
20:56 mwhahaha yes
20:56 mwhahaha that would allow us to use our versions of the modules
20:56 EmilienM yeah
20:56 EmilienM great
20:57 EmilienM I'm 1/ doing the code 2/ replying to ML with this new change
20:57 EmilienM so people can follow-up with all we said ^^^^^
20:57 mwhahaha sounds good
20:57 EmilienM mwhahaha: awesome, thx a lot
21:00 mwhahaha EmilienM: 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 EmilienM well, I don't know if we should convert all the things
21:00 EmilienM I was waiting for _ody proposal
21:02 mwhahaha I'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 week
21:06 EmilienM mwhahaha: no prob
21:21 derekh joined #puppet-openstack
21:40 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/226830
21:40 openstackgerrit Aleksandr Didenko proposed openstack/puppet-swift: Add support for DLO configuration.  https://review.openstack.org/172021
21:42 iraw- joined #puppet-openstack
21:49 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: DNM - CI test  https://review.openstack.org/227565
22:09 zhangjn joined #puppet-openstack
22:12 zhangjn joined #puppet-openstack
22:13 zhangjn joined #puppet-openstack
22:15 EmilienM crinkle: https://review.openstack.org/#/c/226945/ pass \o/
22:15 EmilienM wait
22:16 EmilienM looking at logs to be sure
22:16 EmilienM http://logs.openstack.org/65/227565/1/check/ga​te-puppet-nova-puppet-beaker-rspec-dsvm-trusty​/48dd4f4/console.html#_2015-09-24_21_54_35_993
22:16 EmilienM puppetlabs-apt (v2.1.1)
22:16 EmilienM \o/
22:28 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/226830
22:36 linkedinyou joined #puppet-openstack
22:36 zhangjn joined #puppet-openstack
22:38 zhangjn joined #puppet-openstack
22:57 zhangjn_ joined #puppet-openstack
23:18 mdorman joined #puppet-openstack

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary