Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-12-01

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

All times shown according to UTC.

Time Nick Message
00:04 jerrygb_ joined #puppet-openstack
00:08 zhangjn joined #puppet-openstack
00:09 zhangjn joined #puppet-openstack
00:26 openstackstatus joined #puppet-openstack
00:49 spredzy joined #puppet-openstack
00:56 nlahouti joined #puppet-openstack
00:56 francois joined #puppet-openstack
00:56 openstackgerrit Merged openstack/puppet-trove: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/249145
00:59 zhangjn joined #puppet-openstack
00:59 rahulait joined #puppet-openstack
01:04 openstackgerrit Merged openstack/puppet-tuskar: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/249111
01:08 evanistheking left #puppet-openstack
01:10 openstackgerrit Merged openstack/puppet-nova: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/246857
01:22 openstackgerrit joined #puppet-openstack
01:26 xiaohua joined #puppet-openstack
01:33 ducttape_ joined #puppet-openstack
01:39 zhangjn joined #puppet-openstack
01:40 aimon joined #puppet-openstack
01:52 jerrygb joined #puppet-openstack
02:13 xingchao joined #puppet-openstack
02:56 xarses joined #puppet-openstack
03:02 xingchao joined #puppet-openstack
03:20 zhangjn joined #puppet-openstack
03:36 baker joined #puppet-openstack
03:44 xingchao joined #puppet-openstack
03:54 xiaohua joined #puppet-openstack
04:10 cwolferh joined #puppet-openstack
04:13 xiaohua joined #puppet-openstack
04:58 ducttape_ joined #puppet-openstack
05:32 zhangjn joined #puppet-openstack
05:33 xiaohua joined #puppet-openstack
05:35 jerrygb joined #puppet-openstack
05:39 rahulait joined #puppet-openstack
05:46 gildub joined #puppet-openstack
06:07 nlahouti joined #puppet-openstack
06:07 xingchao joined #puppet-openstack
06:15 xiaohua joined #puppet-openstack
06:51 xiaohua joined #puppet-openstack
06:53 ducttape_ joined #puppet-openstack
06:57 zhangjn joined #puppet-openstack
07:03 ducttape_ joined #puppet-openstack
07:17 cwolferh joined #puppet-openstack
07:30 nlahouti joined #puppet-openstack
07:32 nlahouti1 joined #puppet-openstack
07:36 tdb joined #puppet-openstack
07:41 tdb joined #puppet-openstack
07:46 zhangjn joined #puppet-openstack
07:49 mpryc joined #puppet-openstack
07:56 mflobo joined #puppet-openstack
07:56 beddari joined #puppet-openstack
07:59 xiaohua joined #puppet-openstack
08:00 dgurtner joined #puppet-openstack
08:00 dgurtner joined #puppet-openstack
08:21 ducttape_ joined #puppet-openstack
08:23 xiaohua joined #puppet-openstack
08:28 zhangjn joined #puppet-openstack
08:36 jpena joined #puppet-openstack
08:37 rcernin joined #puppet-openstack
08:48 karimb joined #puppet-openstack
08:49 paramite joined #puppet-openstack
08:52 jerrygb joined #puppet-openstack
08:55 zhangjn joined #puppet-openstack
08:55 mflobo Hello everybody
08:56 mflobo I would like some help to propose a new Puppet module to OpenStack
08:57 xiaohua joined #puppet-openstack
09:02 derekh joined #puppet-openstack
09:05 karimb_ joined #puppet-openstack
09:08 openstackgerrit Javier Peña proposed openstack/puppet-ironic: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245892
09:16 gfidente joined #puppet-openstack
09:16 gfidente joined #puppet-openstack
09:21 ducttape_ joined #puppet-openstack
09:44 rochaporto joined #puppet-openstack
09:46 ferest joined #puppet-openstack
09:48 ferest_ joined #puppet-openstack
09:52 jistr joined #puppet-openstack
09:57 yottatsa joined #puppet-openstack
10:11 xiaohua joined #puppet-openstack
10:11 zhangjn joined #puppet-openstack
10:22 zhangjn joined #puppet-openstack
10:22 ducttape_ joined #puppet-openstack
10:27 zhangjn joined #puppet-openstack
10:27 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Remove `validate_ip_address` validator.  https://review.openstack.org/251767
10:28 toMeloos joined #puppet-openstack
10:33 yottatsa joined #puppet-openstack
10:34 ferest joined #puppet-openstack
10:40 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Introduce glance::db class  https://review.openstack.org/237970
10:44 mkarpin joined #puppet-openstack
10:44 artem_panchenko_ joined #puppet-openstack
10:48 mdavidson joined #puppet-openstack
10:49 timrc joined #puppet-openstack
10:51 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/216654
10:53 skolekonov joined #puppet-openstack
10:57 skolekonov EmilienM, hi! Sorry, I was not available yesterday. I've rebased https://review.openstack.org/#/c/216654, tests are in progress
11:01 xiaohua joined #puppet-openstack
11:02 openstackgerrit Merged openstack/puppet-cinder: Add backup_swift_auth_url to cinder::backup::swift  https://review.openstack.org/248141
11:03 zhangjn joined #puppet-openstack
11:04 openstackgerrit Merged openstack/puppet-openstack-integration: Do a regular clone of tempest if we're running outside the gate  https://review.openstack.org/249933
11:11 openstackgerrit Kyrylo Galanov proposed openstack/puppet-swift: Create ring builders under swift user  https://review.openstack.org/248150
11:14 openstackgerrit Merged openstack/puppet-sahara: Allow customization of db sync command line  https://review.openstack.org/241290
11:15 openstackgerrit Mykyta Karpin proposed openstack/puppet-keystone: Add policy driver option for keystone  https://review.openstack.org/240946
11:15 openstackgerrit Merged openstack/puppet-murano: acceptance: cleanup useless orchestration  https://review.openstack.org/251042
11:15 openstackgerrit Merged openstack/puppet-neutron: Move Neutron to $::os_service_default - part 1  https://review.openstack.org/240929
11:16 jerrygb joined #puppet-openstack
11:20 sanjayu joined #puppet-openstack
11:22 iurygregory_ Good morning people =)
11:25 iurygregory_ hey mflobo, have a take a look in https://wiki.openstack.org/wiki/Puppet/New_module ? ^^
11:26 iurygregory_ mflobo, some steps you should follow 1/ move the module to OpenStack namespace 2/ submit the patch in Governance to add the new repo 3/ Adapt your module with our Wiki page documentation, and the tool we use.
11:32 zhangjn joined #puppet-openstack
11:37 ferest_ joined #puppet-openstack
12:01 ducttape_ joined #puppet-openstack
12:13 clayton joined #puppet-openstack
12:18 francois hi there
12:18 francois I'm wondering why the puppet-sahara depends on duritong/sysctl, any idea?
12:18 francois https://github.com/openstack/puppet-​sahara/blob/master/metadata.json#L30
12:19 francois $ grep -r sysctl .
12:19 francois ./metadata.json:    { "name": "duritong/sysctl", "version_requirement": ">=0.0.1 <1.0.0" },
12:19 francois $
12:19 francois git history isn't particulary helpful here
12:20 jerrygb joined #puppet-openstack
12:30 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-nova: Switch Nova to $::os_service_default - Part1  https://review.openstack.org/248901
12:37 zhangjn joined #puppet-openstack
12:39 mkarpin guys can you please review https://review.openstack.org/#/c/240946/ ? thank you
12:39 figless joined #puppet-openstack
12:45 mflobo iurygregory, hello
12:46 iurygregory hi o/
12:46 mflobo iurygregory, yes, I check that web page, but still I have questions, maybe I'm missing something
12:46 iurygregory go ahead
12:46 mflobo iurygregory, I want to move this puppet module https://github.com/cernops/puppet-ec2api to OpenStack world
12:47 mflobo iurygregory, so, my question
12:47 iurygregory EmilienM, take a look ^
12:48 mflobo iurygregory, Step "Submit a review to openstack-infra/project-config to create the project" I guess in this situation that step dos not make sense, right?
12:49 iurygregory why not?  i think you need this because this repo need to be in openstack namespace
12:50 mflobo iurygregory, oh, is that he meaning of that sentence? ok, so, where (url) can I do that?
12:50 mflobo s/he/the
12:51 iurygregory mflobo, https://review.openstack.org/#/c/240303/ this patchs was to create the puppet-magnum you can take a look
12:51 iurygregory this patch*
12:51 iurygregory You can follow this steps =)
12:52 mflobo iurygregory, ah, that's great example, thanks!
12:52 iurygregory you're welcome ^^
12:53 iurygregory after this you can create the initial commit for the module (https://review.openstack.org/#/c/242164/)
12:54 iurygregory we have a meeting today 15 UTC #openstack-meeting-4
12:56 dprince joined #puppet-openstack
12:58 mflobo iurygregory, I'll be there
13:00 jerrygb joined #puppet-openstack
13:10 ducttape_ joined #puppet-openstack
13:11 mflobo iurygregory, I guess I'll able to create the initial commit for the module when the patch for repository creation is merge, right?
13:11 EmilienM good morning
13:11 iurygregory i think soo mflobo
13:12 iurygregory morning EmilienM o/
13:12 mflobo good morning
13:12 chem morning
13:19 xiaohua joined #puppet-openstack
13:27 xiaohua joined #puppet-openstack
13:28 ferest joined #puppet-openstack
13:41 vinsh EmilienM: https://review.openstack.org/#/c/203220/  +A
13:43 EmilienM vinsh: good morning
13:43 EmilienM looking
13:44 vinsh Good Morning good sir.
13:44 EmilienM vinsh: why does it fail on centos?
13:44 EmilienM have you looked at logs?
13:44 vinsh yeah
13:44 vinsh there is another path to fix that. thats a new puppet issue
13:44 richm joined #puppet-openstack
13:44 vinsh https://review.openstack.org/#/c/250233/
13:44 EmilienM the patch is red for tripleo & centos
13:45 vinsh Yes, due to ^^
13:45 EmilienM vinsh: nope, not unit tests
13:45 EmilienM I mean about functional tests
13:45 vinsh which?
13:45 vinsh gate-puppet-swift-puppet-unit-latest-dsvm-centos7 ?
13:46 EmilienM vinsh: no, that's unit tests
13:46 iurygregory hey EmilienM , if you have time can you look https://review.openstack.org/#/c/248901/ https://review.openstack.org/#/c/249377/ =)
13:46 EmilienM vinsh: a sec
13:47 EmilienM vinsh: have you looked how works the CI ?
13:47 EmilienM https://wiki.openstack.org/wiki/Puppet/CI
13:48 EmilienM in your patch, gate-puppet-openstack-integra​tion-scenario002-dsvm-centos7 and gate-tripleo-ci-f21-nonha are not passing
13:48 EmilienM if you look https://github.com/openstack/puppet​-openstack-integration#description
13:48 EmilienM swift is on scenario002
13:48 morazi joined #puppet-openstack
13:48 EmilienM so I suspect your patch breaks swift deployments on centos OR there is a random issue
13:49 EmilienM the answer is in http://logs.openstack.org/20/203220/58/chec​k/gate-puppet-openstack-integration-scenari​o002-dsvm-centos7/57fad8c/console.html.gz
13:49 markvoelker_ joined #puppet-openstack
13:50 EmilienM looking at the logs, it seems the second puppet run is not idempotent on centos7
13:50 EmilienM got it http://logs.openstack.org/20/203220/58​/check/gate-puppet-openstack-integrati​on-scenario002-dsvm-centos7/57fad8c/co​nsole.html.gz#_2015-11-25_21_43_34_729
13:51 vinsh owner change eh?
13:52 tristanC joined #puppet-openstack
13:53 vinsh how is that a failure?
13:53 EmilienM vinsh: the second puppet run is not idempotent
13:53 EmilienM our CI test that
13:54 EmilienM something in https://review.openstack.org/#/c/203220/​58/lib/puppet/provider/service/swiftinit.rb,cm
13:54 EmilienM I'm looking
13:55 EmilienM puppet creates the files as 'swift' but packaging set 'root'
13:56 vinsh Indeed, but I havn't modified this behavior in my patch
13:58 EmilienM vinsh: I think https://review.openstack.org/#/c/203220/​58/lib/puppet/provider/service/swiftinit.rb,cm did
13:58 EmilienM vinsh: let me try without your patch
13:58 openstackgerrit Emilien Macchi proposed openstack/puppet-swift: DNM - CI test  https://review.openstack.org/251875
13:59 yottatsa joined #puppet-openstack
14:11 mdorman joined #puppet-openstack
14:15 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/250770
14:18 EmilienM feel free to have topics on our meeting agenda: https://etherpad.openstack.org/p/pupp​et-openstack-weekly-meeting-20151201
14:20 figless left #puppet-openstack
14:22 yottatsa joined #puppet-openstack
14:22 vinsh EmilienM: swiftinit.rb does not modify anything in /etc/swift .  If anything its a timing issue related to: https://review.openstack.org/#/c/2​03220/58/manifests/storage/server.pp  line 139?
14:23 EmilienM vinsh: yes, eventually
14:24 vinsh This same test passed for this patch a week ago, I suspect ordering can change in the runs.  I wonder if I need to keep root as owner of the conf file directory?
14:27 EmilienM running the CI without your patch https://jenkins05.openstack.org/job/​gate-puppet-openstack-integration-sc​enario002-dsvm-centos7/189/console
14:27 EmilienM it worked
14:27 EmilienM tempest is running now
14:27 EmilienM vinsh: so there is something with your patch
14:30 vinsh something new-ish that changed after I developed it.
14:30 EmilienM clayton: good thought on https://review.openstack.org/#/c/250546/
14:31 clayton EmilienM if we can get the external hooks stuff in place for the keystone module then my objection mostly goes away
14:31 clayton I have the patch for that mostly done for nova, keystone is next on my list
14:31 clayton but there is also discussion on the ops list about how several people (us included) are excited about apache being the only choice
14:32 clayton s/are/aren't/
14:32 EmilienM clayton: ack - and no worries we are not going to merge it as it is now. I just sent a patch to initiate the work but you're right, maybe the patch is a bit tough
14:32 IvanBerezovskiy1 joined #puppet-openstack
14:33 darkxploit joined #puppet-openstack
14:34 yottatsa joined #puppet-openstack
14:34 markvoelker joined #puppet-openstack
14:39 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Remove all 'validate_array' statements  https://review.openstack.org/250763
14:39 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Adding MidoNet LoadBalancing options  https://review.openstack.org/250777
14:39 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/251319
14:44 degorenko joined #puppet-openstack
14:44 EmilienM can anyone merge this CI patch ? https://review.openstack.org/#/c/239293/
14:45 degorenko EmilienM, hey o/ I can :)
14:46 EmilienM aderyugin: ping murano testing https://review.openstack.org/#/c/250436/
14:49 EmilienM we have our weekly meeting in 10 minutes!
14:49 davideagle joined #puppet-openstack
14:50 ducttape_ joined #puppet-openstack
14:52 degorenko guys, please https://review.openstack.org/250741 :) it is backport from master
14:58 openstackgerrit Merged openstack/puppet-openstack-integration: scenario001: deploy & test Aodh  https://review.openstack.org/239293
14:58 yottatsa joined #puppet-openstack
15:02 angdraug joined #puppet-openstack
15:07 openstackgerrit Merged openstack/puppet-ceilometer: Ceilometer $::os_service_default for db and log  https://review.openstack.org/249377
15:13 morazi joined #puppet-openstack
15:19 openstackgerrit Merged openstack/puppet-barbican: Barbican $::os_service_default for db and logging  https://review.openstack.org/249401
15:20 baker joined #puppet-openstack
15:20 xiaohua joined #puppet-openstack
15:22 EmilienM degorenko: I think I understand what do you mean
15:22 EmilienM we want to test the value of the *_config is <SERVICE DEFAULT>
15:23 EmilienM but I was wondering if we should not rather test that the value is absent
15:26 degorenko EmilienM, because in puppet class we have value parameter - hence we should test it. When we have ensure parameter - we will test ensure.
15:26 degorenko we can't test provider's work in spec tests
15:27 EmilienM degorenko: ack, it makes sense
15:27 EmilienM thanks
15:28 markvoelker_ joined #puppet-openstack
15:29 degorenko you're welcome :)
15:30 IvanBerezovskiy1 left #puppet-openstack
15:31 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
15:33 mdorman joined #puppet-openstack
15:35 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/250770
15:35 EmilienM degorenko, sbadia ^ ready now - thanks for your reviews :)
15:44 openstackgerrit Merged openstack/puppet-mistral: Correct Default spelling in init.pp  https://review.openstack.org/247268
15:45 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: scenario002: enable application catalog service  https://review.openstack.org/250436
15:47 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/250770
15:51 ntpttr joined #puppet-openstack
15:52 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/250770
15:54 openstackgerrit Nate Potter proposed openstack/puppet-glance: Implement multiple store configuration  https://review.openstack.org/242718
15:59 vishpat joined #puppet-openstack
16:00 markvoelker joined #puppet-openstack
16:02 dhollinger joined #puppet-openstack
16:08 rochaporto joined #puppet-openstack
16:08 openstackgerrit Merged openstack/puppet-manila: Manila $::os_service_default for db and log  https://review.openstack.org/249394
16:12 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Introduce glance::db class  https://review.openstack.org/237970
16:13 degorenko iurygregory, ^^^ thanks for review :)
16:15 iurygregory hey degorenko you're welcome =)
16:15 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
16:21 mkarpin Guys i today updated https://review.openstack.org/#/c/240946/
16:21 mkarpin can you please review it
16:21 mkarpin it is small :)
16:23 mkarpin degorenko, EmilienM, crinkle, mfisch, sbadia, spredzy : ^^
16:29 asilenkov joined #puppet-openstack
16:43 dprince joined #puppet-openstack
16:45 vishpat joined #puppet-openstack
16:47 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/250770
16:53 openstackgerrit Nate Potter proposed openstack/puppet-glance: Implement multiple store configuration  https://review.openstack.org/242718
16:54 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Clean up api and persister users/passwords, separate from dbadmin  https://review.openstack.org/249974
16:59 skolekonov joined #puppet-openstack
17:14 nlahouti joined #puppet-openstack
17:28 nlahouti joined #puppet-openstack
17:34 openstackgerrit Merged openstack/puppet-heat: Switch Heat to $::os_service_default - part 1  https://review.openstack.org/249119
17:38 haplo37 joined #puppet-openstack
17:39 mpryc left #puppet-openstack
17:40 xarses joined #puppet-openstack
17:40 baker joined #puppet-openstack
17:43 dhollinger joined #puppet-openstack
17:55 openstackgerrit Javier Peña proposed openstack/puppet-heat: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245715
17:57 openstackgerrit Javier Peña proposed openstack/puppet-heat: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245715
17:58 openstackgerrit Javier Peña proposed openstack/puppet-manila: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245721
18:03 openstackgerrit Merged openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/250770
18:04 xiaohua joined #puppet-openstack
18:05 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/252003
18:37 rcernin joined #puppet-openstack
18:43 ntpttr joined #puppet-openstack
18:45 openstackgerrit Merged openstack/puppet-nova: Switch Nova to $::os_service_default - Part1  https://review.openstack.org/248901
18:49 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: loadbalancer: add Gnocchi API support  https://review.openstack.org/252015
18:58 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Add support for configuring rgw using civetweb  https://review.openstack.org/252020
19:00 nlahouti joined #puppet-openstack
19:00 baker joined #puppet-openstack
19:15 openstackgerrit Emilien Macchi proposed openstack/puppet-gnocchi: Implement ::gnocchi::client  https://review.openstack.org/252026
19:21 vishpat joined #puppet-openstack
19:23 aimon joined #puppet-openstack
19:30 EmilienM _ody, Hunner: have you read the discussion in our ML about Config support for oslo.config.cfg.MultiStrOpt ?
19:31 nibalizer Hunner: is in australia, not very much around
19:31 EmilienM nibalizer: thanks for the info
19:37 openstackgerrit Emilien Macchi proposed openstack/puppet-gnocchi: Implement ::gnocchi::client  https://review.openstack.org/252026
19:42 bklei joined #puppet-openstack
19:44 cdelatte joined #puppet-openstack
19:50 cdelatte joined #puppet-openstack
19:53 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Add support for configuring rgw with civetweb  https://review.openstack.org/252031
19:58 EmilienM I found the new challenge: backport the patches from master (that have SERVICE DEFAULT) to stable/liberty (that does not have SERVICE_DEFAULT)
19:58 iurygregory wow
19:59 EmilienM I wanted to backport https://review.openstack.org/#/c/252003/
19:59 EmilienM and it failed to pass CI, which is normal
20:00 EmilienM hopefully the fact is in stable
20:01 iurygregory you should add all  dependencies to service default to stable/liberty i think
20:02 iurygregory like the @default_facts in rb files
20:03 EmilienM iurygregory: probably
20:03 iurygregory try to send the files i've update in ceilometer
20:03 iurygregory https://review.openstack.org/#/c/249377/
20:19 dgurtner joined #puppet-openstack
20:19 dgurtner joined #puppet-openstack
20:24 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/252003
20:27 vishpat joined #puppet-openstack
20:28 vishpat Hi folks, I have review open at https://review.openstack.org/#/c/247239 and one of the review comment says that I need to update the unit test for this change as well.
20:29 vishpat However these changes do not change any of the plugin functionality and just fix the order in which the configuration steps are run and hence does not need any change to the existing unit test
20:29 vishpat Hence would like someone to let me know what am I missing
20:29 iurygregory vishpat, do you have acceptance tests?
20:30 vishpat https://github.com/openstack/puppet-neutron/blob/​master/spec/classes/neutron_plugins_nuage_spec.rb
20:30 vishpat Is this what do you mean ?
20:30 iurygregory nops
20:31 vishpat Please note I am new to this process, so pardon my ignorance
20:31 iurygregory rlx man ;)
20:32 EmilienM iurygregory, vishpat: no need of acceptance tests for neutron plugins, since we don't have the SDN installed
20:32 iurygregory well i agree with denis you need to add tests, you have add a require.. you cant put this in your unit test (spec/classes...)
20:32 iurygregory tks EmilienM  ;)
20:34 vishpat Ok it makes sense now
20:34 vishpat I will make the appropriate changes
20:34 iurygregory simple things =)
20:36 vishpat Thanks
20:36 iurygregory you're welcome ^^
20:38 vishpat joined #puppet-openstack
20:41 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/252003
20:45 foxhunt_away joined #puppet-openstack
20:45 xiaohua joined #puppet-openstack
20:46 davideagle joined #puppet-openstack
20:47 openstackgerrit Merged openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/216654
21:08 asilenkov joined #puppet-openstack
21:15 _ody EmilienM: Got time to go over what my proposed CI changes are?
21:15 EmilienM _ody: sure
21:15 EmilienM _ody: I need to get the context again
21:16 EmilienM _ody: let me read your e-mail again
21:18 EmilienM _ody: ok - now looking patches
21:18 nlahouti joined #puppet-openstack
21:19 EmilienM _ody: wow, that creates a lot of jobs for puppet-cinder
21:19 EmilienM 4 more jobs?
21:21 _ody EmilienM: Yeah, I want to make sure they pass before I just delete gate-puppet-*-puppet-beaker-rspec-dsvm-*
21:21 EmilienM ok
21:22 _ody I suspect everyone still wants to test against 3.8.x so I don't see the way around having 4 beaker jobs.
21:23 _ody That is kinda up to the community as a whole though.  I don't think we have future parser on in our 3.8.x jobs and it is the only parser in 4.x.
21:23 cwolferh joined #puppet-openstack
21:23 _ody (we get most of our parser testing done in rspec though)
21:24 ferest joined #puppet-openstack
21:24 EmilienM _ody: I think you could trick our beaker jobs to not having to patch project-config
21:25 EmilienM you could work on openstack/puppet-openstack-integration and do what you want
21:26 EmilienM your patches look good to me, but they don't pass CI
21:26 EmilienM one thing could be to patch openstack/puppet-openstack-integration to install puppet4 with your method and run the tests
21:27 EmilienM that would avoid changing openstack infra jobs
21:27 EmilienM that's the fastest option
21:27 _ody To test they work you mean?
21:27 _ody I can easily make openstack/puppet-openstack-integration default to puppet4 but then puppet3 isn't going to be tested.
21:28 EmilienM _ody: yes but it's just for testing
21:28 EmilienM I'm ok with having more jobs
21:28 EmilienM just for puppet-cinder
21:28 EmilienM but your patch is not passing CI
21:28 rahulait joined #puppet-openstack
21:29 _ody EmilienM: I was doing puppet-cinder just as an example, it would had these jobs to everything eventully.
21:29 EmilienM _ody: that's too much
21:29 EmilienM that's a lot of resources
21:30 _ody Then I'd simply vote to beaker test against puppet4.
21:31 EmilienM yeah
21:31 _ody ok.  I'll make that happen then.
21:31 EmilienM it will break all our CI though probably
21:31 EmilienM but that's fine, we'll fix it :)
21:32 _ody cinder passed cleanly, fwiw.
21:33 _ody Well the beaker test did on my laptop in a VM, I'll look into while the submitted stuff failes.
21:34 _ody ok, I see it already.
21:35 vinsh joined #puppet-openstack
21:37 imcsk8_ joined #puppet-openstack
21:40 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: (WIP) Add support for different module paths  https://review.openstack.org/248316
21:41 openstackgerrit Tim Rozet proposed openstack/puppet-neutron: Adds configuration support for OpenDaylight SDN Controller  https://review.openstack.org/249390
21:42 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/249409
21:45 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/249409
21:47 davideagle_ joined #puppet-openstack
21:47 gildub joined #puppet-openstack
21:50 openstackgerrit Emilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/252003
21:50 _ody EmilienM: I did also research some docker related testing.  It isn't feasible for beaker testing right now because some services fail but if we want to reduce our nodepool usage, it is very realistic for rspec-puppet.
21:50 _ody We could do the entire suite on a single nodepool instance.
21:56 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/252077
21:59 EmilienM _ody: I think we already talked about that with nibalizer
22:00 EmilienM it came out having multiple jobs was easier to debug and faster
22:00 EmilienM I have to go now, I'm back later
22:05 baker joined #puppet-openstack
22:16 toMeloos Small heads-up: turns out Liberty breaks the puppet-keystone wsgi example.....
22:19 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/249409
22:19 toMeloos The apache_with_paths.pp example that is, because two wsgiscipts in one vhost won't work anymove
22:24 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/249409
22:26 openstackgerrit Vishal Patil proposed openstack/puppet-neutron: Fix dependencies for Nuage plugin configuration  https://review.openstack.org/247239
22:32 jerrygb joined #puppet-openstack
22:38 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: (WIP) Add support for different module paths  https://review.openstack.org/248316
22:42 imcsk8 joined #puppet-openstack
22:56 nlahouti joined #puppet-openstack
23:19 ntpttr left #puppet-openstack
23:26 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/249409
23:54 openstackgerrit Evan Giles proposed openstack/puppet-gnocchi: Add Influxdb as a possible backend storage for Gnocchi  https://review.openstack.org/252107

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