Tuesday, 2015-12-01

*** jerrygb_ has joined #puppet-openstack00:04
*** jerrygb has quit IRC00:05
*** zhangjn has joined #puppet-openstack00:08
*** zhangjn has quit IRC00:08
*** zhangjn has joined #puppet-openstack00:09
*** zhangjn has quit IRC00:22
*** openstackstatus has quit IRC00:24
*** openstackstatus has joined #puppet-openstack00:26
*** ChanServ sets mode: +v openstackstatus00:26
*** ducttape_ has quit IRC00:34
*** spredzy has quit IRC00:48
*** nlahouti has quit IRC00:49
*** spredzy has joined #puppet-openstack00:49
*** xarses has quit IRC00:54
*** nlahouti has joined #puppet-openstack00:56
*** francois_ has quit IRC00:56
*** francois has joined #puppet-openstack00:56
openstackgerritMerged openstack/puppet-trove: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24914500:56
*** RichardRaseley has quit IRC00:56
*** morazi has quit IRC00:57
*** zhangjn has joined #puppet-openstack00:59
*** rahulait has joined #puppet-openstack00:59
*** dgurtner has quit IRC01:04
openstackgerritMerged openstack/puppet-tuskar: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24911101:04
*** aimon has quit IRC01:05
*** evanistheking has left #puppet-openstack01:08
openstackgerritMerged openstack/puppet-nova: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24685701:10
*** richm has quit IRC01:16
*** karimb has quit IRC01:20
*** openstackgerrit has quit IRC01:22
*** openstackgerrit has joined #puppet-openstack01:22
*** xiaohua has joined #puppet-openstack01:26
*** nlahouti has quit IRC01:27
*** ducttape_ has joined #puppet-openstack01:33
*** zhangjn has quit IRC01:36
*** zhangjn has joined #puppet-openstack01:39
*** aimon has joined #puppet-openstack01:40
*** xiaohua has quit IRC01:42
*** markvoelker has quit IRC01:46
*** jerrygb_ has quit IRC01:51
*** jerrygb has joined #puppet-openstack01:52
*** jerrygb has quit IRC01:55
*** rahulait has quit IRC02:05
*** xingchao has joined #puppet-openstack02:13
*** zhangjn has quit IRC02:28
*** cwolferh has quit IRC02:41
*** xarses has joined #puppet-openstack02:56
*** xingchao has quit IRC03:01
*** xingchao has joined #puppet-openstack03:02
*** zhangjn has joined #puppet-openstack03:20
*** baker has joined #puppet-openstack03:36
*** zhangjn has quit IRC03:42
*** xingchao has quit IRC03:43
*** xingchao has joined #puppet-openstack03:44
*** baker has quit IRC03:51
*** xiaohua has joined #puppet-openstack03:54
*** xiaohua has quit IRC03:58
*** cwolferh has joined #puppet-openstack04:10
*** xiaohua has joined #puppet-openstack04:13
*** xingchao has quit IRC04:15
*** xiaohua has quit IRC04:45
*** ducttape_ has quit IRC04:49
*** ducttape_ has joined #puppet-openstack04:58
*** saneax_AFK is now known as saneax05:28
*** zhangjn has joined #puppet-openstack05:32
*** xiaohua has joined #puppet-openstack05:33
*** jerrygb has joined #puppet-openstack05:35
*** rahulait has joined #puppet-openstack05:39
*** gildub has joined #puppet-openstack05:46
*** ducttape_ has quit IRC05:52
*** jerrygb has quit IRC05:52
*** xiaohua has quit IRC05:54
*** nlahouti has joined #puppet-openstack06:07
*** xingchao has joined #puppet-openstack06:07
*** xiaohua has joined #puppet-openstack06:15
*** zhangjn has quit IRC06:25
*** xiaohua has quit IRC06:34
*** cwolferh has quit IRC06:48
*** xiaohua has joined #puppet-openstack06:51
*** ducttape_ has joined #puppet-openstack06:53
*** zhangjn has joined #puppet-openstack06:57
*** zhangjn has quit IRC06:58
*** ducttape_ has quit IRC06:59
*** ducttape_ has joined #puppet-openstack07:03
*** gildub has quit IRC07:13
*** cwolferh has joined #puppet-openstack07:17
*** nlahouti has quit IRC07:19
*** ducttape_ has quit IRC07:20
*** nlahouti has joined #puppet-openstack07:30
*** nlahouti1 has joined #puppet-openstack07:32
*** nlahouti has quit IRC07:34
*** tdb has quit IRC07:34
*** tdb has joined #puppet-openstack07:36
*** tdb has quit IRC07:38
*** xiaohua has quit IRC07:40
*** tdb has joined #puppet-openstack07:41
*** zhangjn has joined #puppet-openstack07:46
*** nlahouti1 has quit IRC07:48
*** mpryc has joined #puppet-openstack07:49
*** beddari has quit IRC07:53
*** mflobo has joined #puppet-openstack07:56
*** beddari has joined #puppet-openstack07:56
*** xiaohua has joined #puppet-openstack07:59
*** dgurtner has joined #puppet-openstack08:00
*** dgurtner has quit IRC08:00
*** dgurtner has joined #puppet-openstack08:00
*** foxhunt_away is now known as foxhunt7208:06
*** zhangjn has quit IRC08:17
*** ducttape_ has joined #puppet-openstack08:21
*** xiaohua has quit IRC08:21
*** xiaohua has joined #puppet-openstack08:23
*** ducttape_ has quit IRC08:25
*** zhangjn has joined #puppet-openstack08:28
*** jpena has joined #puppet-openstack08:36
*** rcernin has joined #puppet-openstack08:37
*** xiaohua has quit IRC08:41
*** zhangjn has quit IRC08:44
*** karimb has joined #puppet-openstack08:48
*** paramite has joined #puppet-openstack08:49
*** jerrygb has joined #puppet-openstack08:52
*** zhangjn has joined #puppet-openstack08:55
mfloboHello everybody08:55
mfloboI would like some help to propose a new Puppet module to OpenStack08:56
*** xiaohua has joined #puppet-openstack08:57
*** derekh has joined #puppet-openstack09:02
*** karimb_ has joined #puppet-openstack09:05
*** karimb has quit IRC09:07
openstackgerritJavier Peña proposed openstack/puppet-ironic: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24589209:08
*** paramite is now known as paramite|afk09:09
*** gfidente has joined #puppet-openstack09:16
*** gfidente has quit IRC09:16
*** gfidente has joined #puppet-openstack09:16
*** ducttape_ has joined #puppet-openstack09:21
*** zhangjn has quit IRC09:24
*** ducttape_ has quit IRC09:26
*** rochaporto has joined #puppet-openstack09:44
*** ferest has joined #puppet-openstack09:46
*** ferest_ has joined #puppet-openstack09:48
*** ferest has quit IRC09:48
*** jistr has joined #puppet-openstack09:52
*** paramite|afk is now known as paramite09:53
*** yottatsa has joined #puppet-openstack09:57
*** xiaohua has quit IRC10:08
*** xiaohua has joined #puppet-openstack10:11
*** zhangjn has joined #puppet-openstack10:11
*** yottatsa has quit IRC10:11
*** zhangjn has quit IRC10:12
*** rahulait has quit IRC10:15
*** zhangjn has joined #puppet-openstack10:22
*** ducttape_ has joined #puppet-openstack10:22
*** zhangjn has quit IRC10:23
*** zhangjn has joined #puppet-openstack10:27
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Remove `validate_ip_address` validator.  https://review.openstack.org/25176710:27
*** ducttape_ has quit IRC10:28
*** toMeloos has joined #puppet-openstack10:28
*** jerrygb has quit IRC10:31
*** yottatsa has joined #puppet-openstack10:33
*** ferest_ has quit IRC10:33
*** ferest has joined #puppet-openstack10:34
*** paramite is now known as paramite|afk10:38
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Introduce glance::db class  https://review.openstack.org/23797010:40
*** paramite|afk is now known as paramite10:40
*** mkarpin has quit IRC10:41
*** xingchao has quit IRC10:42
*** mkarpin has joined #puppet-openstack10:44
*** artem_panchenko_ has joined #puppet-openstack10:44
*** artem_panchenko_ has quit IRC10:45
*** timrc has quit IRC10:47
*** mdavidson has joined #puppet-openstack10:48
*** timrc has joined #puppet-openstack10:49
*** zhangjn has quit IRC10:51
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/21665410:51
*** skolekonov has joined #puppet-openstack10:53
*** xiaohua has quit IRC10:53
*** yottatsa has quit IRC10:53
skolekonovEmilienM, hi! Sorry, I was not available yesterday. I've rebased https://review.openstack.org/#/c/216654, tests are in progress10:57
*** paramite is now known as paramite|afk11:01
*** xiaohua has joined #puppet-openstack11:01
openstackgerritMerged openstack/puppet-cinder: Add backup_swift_auth_url to cinder::backup::swift  https://review.openstack.org/24814111:02
*** zhangjn has joined #puppet-openstack11:03
openstackgerritMerged openstack/puppet-openstack-integration: Do a regular clone of tempest if we're running outside the gate  https://review.openstack.org/24993311:04
*** zhangjn has quit IRC11:04
*** xiaohua has quit IRC11:08
*** saneax is now known as saneax_AFK11:10
openstackgerritKyrylo Galanov proposed openstack/puppet-swift: Create ring builders under swift user  https://review.openstack.org/24815011:11
openstackgerritMerged openstack/puppet-sahara: Allow customization of db sync command line  https://review.openstack.org/24129011:14
openstackgerritMykyta Karpin proposed openstack/puppet-keystone: Add policy driver option for keystone  https://review.openstack.org/24094611:15
openstackgerritMerged openstack/puppet-murano: acceptance: cleanup useless orchestration  https://review.openstack.org/25104211:15
openstackgerritMerged openstack/puppet-neutron: Move Neutron to $::os_service_default - part 1  https://review.openstack.org/24092911:15
*** jerrygb has joined #puppet-openstack11:16
*** jerrygb has quit IRC11:19
*** sanjayu has quit IRC11:20
*** sanjayu has joined #puppet-openstack11:20
iurygregory_Good morning people =)11:22
iurygregory_hey mflobo, have a take a look in https://wiki.openstack.org/wiki/Puppet/New_module ? ^^11:25
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:26
*** iurygregory_ is now known as iurygregory11:27
*** zhangjn has joined #puppet-openstack11:32
*** ferest_ has joined #puppet-openstack11:37
*** ferest has quit IRC11:37
*** zhangjn has quit IRC12:00
*** ducttape_ has joined #puppet-openstack12:01
*** ducttape_ has quit IRC12:06
*** clayton has quit IRC12:06
*** paramite|afk is now known as paramite12:09
*** clayton has joined #puppet-openstack12:13
francoishi there12:18
francoisI'm wondering why the puppet-sahara depends on duritong/sysctl, any idea?12:18
francoishttps://github.com/openstack/puppet-sahara/blob/master/metadata.json#L3012:18
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
francoisgit history isn't particulary helpful here12:19
*** jerrygb has joined #puppet-openstack12:20
*** jerrygb has quit IRC12:27
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-nova: Switch Nova to $::os_service_default - Part1  https://review.openstack.org/24890112:30
*** zhangjn has joined #puppet-openstack12:37
mkarpinguys can you please review https://review.openstack.org/#/c/240946/ ? thank you12:39
*** figless has joined #puppet-openstack12:39
mfloboiurygregory, hello12:45
iurygregoryhi o/12:46
mfloboiurygregory, yes, I check that web page, but still I have questions, maybe I'm missing something12:46
iurygregorygo ahead12:46
mfloboiurygregory, I want to move this puppet module https://github.com/cernops/puppet-ec2api to OpenStack world12:46
mfloboiurygregory, so, my question12:47
iurygregoryEmilienM, take a look ^12:47
mfloboiurygregory, 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:48
iurygregorywhy not?  i think you need this because this repo need to be in openstack namespace12:49
mfloboiurygregory, oh, is that he meaning of that sentence? ok, so, where (url) can I do that?12:50
mflobos/he/the12:50
iurygregorymflobo, https://review.openstack.org/#/c/240303/ this patchs was to create the puppet-magnum you can take a look12:51
iurygregorythis patch*12:51
iurygregoryYou can follow this steps =)12:51
mfloboiurygregory, ah, that's great example, thanks!12:52
iurygregoryyou're welcome ^^12:52
iurygregoryafter this you can create the initial commit for the module (https://review.openstack.org/#/c/242164/)12:53
iurygregorywe have a meeting today 15 UTC #openstack-meeting-412:54
*** dprince has joined #puppet-openstack12:56
mfloboiurygregory, I'll be there12:58
*** jerrygb has joined #puppet-openstack13:01
*** ducttape_ has joined #puppet-openstack13:10
mfloboiurygregory, I guess I'll able to create the initial commit for the module when the patch for repository creation is merge, right?13:11
EmilienMgood morning13:11
iurygregoryi think soo mflobo13:11
iurygregorymorning EmilienM o/13:12
mflobogood morning13:12
chemmorning13:12
*** jayg|g0n3 is now known as jayg13:13
*** xiaohua has joined #puppet-openstack13:19
*** xiaohua has quit IRC13:24
*** xiaohua has joined #puppet-openstack13:27
*** trown|outttypeww is now known as trown13:27
*** ferest has joined #puppet-openstack13:28
*** jpena is now known as jpena|lunch13:28
*** ferest_ has quit IRC13:29
*** paramite is now known as paramite|afk13:30
*** xiaohua has quit IRC13:31
*** ducttape_ has quit IRC13:33
*** paramite|afk is now known as paramite13:33
vinshEmilienM: https://review.openstack.org/#/c/203220/  +A13:41
EmilienMvinsh: good morning13:43
EmilienMlooking13:43
vinshGood Morning good sir.13:44
EmilienMvinsh: why does it fail on centos?13:44
EmilienMhave you looked at logs?13:44
vinshyeah13:44
vinshthere is another path to fix that. thats a new puppet issue13:44
*** richm has joined #puppet-openstack13:44
vinshhttps://review.openstack.org/#/c/250233/13:44
EmilienMthe patch is red for tripleo & centos13:44
vinshYes, due to ^^13:45
EmilienMvinsh: nope, not unit tests13:45
EmilienMI mean about functional tests13:45
vinshwhich?13:45
vinshgate-puppet-swift-puppet-unit-latest-dsvm-centos7 ?13:45
EmilienMvinsh: no, that's unit tests13:46
iurygregoryhey EmilienM , if you have time can you look https://review.openstack.org/#/c/248901/ https://review.openstack.org/#/c/249377/ =)13:46
EmilienMvinsh: a sec13:46
EmilienMvinsh: have you looked how works the CI ?13:47
EmilienMhttps://wiki.openstack.org/wiki/Puppet/CI13:47
EmilienMin your patch, gate-puppet-openstack-integration-scenario002-dsvm-centos7 and gate-tripleo-ci-f21-nonha are not passing13:48
EmilienMif you look https://github.com/openstack/puppet-openstack-integration#description13:48
EmilienMswift is on scenario00213:48
*** morazi has joined #puppet-openstack13:48
EmilienMso I suspect your patch breaks swift deployments on centos OR there is a random issue13:48
EmilienMthe answer is in http://logs.openstack.org/20/203220/58/check/gate-puppet-openstack-integration-scenario002-dsvm-centos7/57fad8c/console.html.gz13:49
*** markvoelker_ has joined #puppet-openstack13:49
EmilienMlooking at the logs, it seems the second puppet run is not idempotent on centos713:50
EmilienMgot it http://logs.openstack.org/20/203220/58/check/gate-puppet-openstack-integration-scenario002-dsvm-centos7/57fad8c/console.html.gz#_2015-11-25_21_43_34_72913:50
vinshowner change eh?13:51
*** tristanC has joined #puppet-openstack13:52
vinshhow is that a failure?13:53
EmilienMvinsh: the second puppet run is not idempotent13:53
EmilienMour CI test that13:53
EmilienMsomething in https://review.openstack.org/#/c/203220/58/lib/puppet/provider/service/swiftinit.rb,cm13:54
EmilienMI'm looking13:54
EmilienMpuppet creates the files as 'swift' but packaging set 'root'13:55
vinshIndeed, but I havn't modified this behavior in my patch13:56
EmilienMvinsh: I think https://review.openstack.org/#/c/203220/58/lib/puppet/provider/service/swiftinit.rb,cm did13:58
EmilienMvinsh: let me try without your patch13:58
openstackgerritEmilien Macchi proposed openstack/puppet-swift: DNM - CI test  https://review.openstack.org/25187513:58
*** yottatsa has joined #puppet-openstack13:59
*** darkxploit has quit IRC14:03
*** paramite is now known as paramite|afk14:06
*** paramite|afk is now known as paramite14:08
*** mdorman has joined #puppet-openstack14:11
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25077014:15
*** EmilienM sets mode: +o crinkle14:15
*** EmilienM sets mode: +o spredzy14:15
*** EmilienM sets mode: +o clayton14:15
EmilienMfeel free to have topics on our meeting agenda: https://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2015120114:18
*** figless has left #puppet-openstack14:20
*** yottatsa has quit IRC14:21
*** yottatsa has joined #puppet-openstack14:22
vinshEmilienM: swiftinit.rb does not modify anything in /etc/swift .  If anything its a timing issue related to: https://review.openstack.org/#/c/203220/58/manifests/storage/server.pp  line 139?14:22
EmilienMvinsh: yes, eventually14:23
vinshThis 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:24
EmilienMrunning the CI without your patch https://jenkins05.openstack.org/job/gate-puppet-openstack-integration-scenario002-dsvm-centos7/189/console14:27
EmilienMit worked14:27
EmilienMtempest is running now14:27
EmilienMvinsh: so there is something with your patch14:27
vinshsomething new-ish that changed after I developed it.14:30
EmilienMclayton: good thought on https://review.openstack.org/#/c/250546/14:30
claytonEmilienM if we can get the external hooks stuff in place for the keystone module then my objection mostly goes away14:31
claytonI have the patch for that mostly done for nova, keystone is next on my list14:31
claytonbut there is also discussion on the ops list about how several people (us included) are excited about apache being the only choice14:31
claytons/are/aren't/14:32
EmilienMclayton: 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 tough14:32
*** IvanBerezovskiy1 has joined #puppet-openstack14:32
*** yottatsa has quit IRC14:32
*** darkxploit has joined #puppet-openstack14:33
*** yottatsa has joined #puppet-openstack14:34
*** markvoelker has joined #puppet-openstack14:34
*** markvoelker_ has quit IRC14:35
*** jpena|lunch is now known as jpena14:37
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Remove all 'validate_array' statements  https://review.openstack.org/25076314:39
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Adding MidoNet LoadBalancing options  https://review.openstack.org/25077714:39
*** davideagle_ has quit IRC14:39
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/25131914:39
*** degorenko has joined #puppet-openstack14:44
EmilienMcan anyone merge this CI patch ? https://review.openstack.org/#/c/239293/14:44
degorenkoEmilienM, hey o/ I can :)14:45
EmilienMaderyugin: ping murano testing https://review.openstack.org/#/c/250436/14:46
*** mdorman has quit IRC14:46
*** ferest has quit IRC14:48
EmilienMwe have our weekly meeting in 10 minutes!14:49
*** davideagle has joined #puppet-openstack14:49
*** ducttape_ has joined #puppet-openstack14:50
*** paramite is now known as paramite|afk14:52
degorenkoguys, please https://review.openstack.org/250741 :) it is backport from master14:52
*** paramite|afk is now known as paramite14:54
openstackgerritMerged openstack/puppet-openstack-integration: scenario001: deploy & test Aodh  https://review.openstack.org/23929314:58
*** yottatsa has quit IRC14:58
*** yottatsa has joined #puppet-openstack14:58
*** angdraug has joined #puppet-openstack15:02
*** morazi has quit IRC15:05
openstackgerritMerged openstack/puppet-ceilometer: Ceilometer $::os_service_default for db and log  https://review.openstack.org/24937715:07
*** morazi has joined #puppet-openstack15:13
*** saneax_AFK is now known as saneax15:19
openstackgerritMerged openstack/puppet-barbican: Barbican $::os_service_default for db and logging  https://review.openstack.org/24940115:19
*** baker has joined #puppet-openstack15:20
*** xiaohua has joined #puppet-openstack15:20
EmilienMdegorenko: I think I understand what do you mean15:22
EmilienMwe want to test the value of the *_config is <SERVICE DEFAULT>15:22
EmilienMbut I was wondering if we should not rather test that the value is absent15:23
*** xiaohua has quit IRC15:25
degorenkoEmilienM, because in puppet class we have value parameter - hence we should test it. When we have ensure parameter - we will test ensure.15:26
degorenkowe can't test provider's work in spec tests15:26
*** skolekonov has quit IRC15:27
EmilienMdegorenko: ack, it makes sense15:27
EmilienMthanks15:27
*** paramite is now known as paramite|afk15:28
*** markvoelker_ has joined #puppet-openstack15:28
degorenkoyou're welcome :)15:29
*** IvanBerezovskiy1 has left #puppet-openstack15:30
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/20322015:31
*** markvoelker has quit IRC15:32
*** mdorman has joined #puppet-openstack15:33
*** markvoelker_ has quit IRC15:33
*** paramite|afk is now known as paramite15:33
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25077015:35
EmilienMdegorenko, sbadia ^ ready now - thanks for your reviews :)15:35
*** zhangjn has quit IRC15:35
*** mdorman has quit IRC15:35
openstackgerritMerged openstack/puppet-mistral: Correct Default spelling in init.pp  https://review.openstack.org/24726815:44
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: scenario002: enable application catalog service  https://review.openstack.org/25043615:45
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25077015:47
*** ntpttr has joined #puppet-openstack15:51
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25077015:52
openstackgerritNate Potter proposed openstack/puppet-glance: Implement multiple store configuration  https://review.openstack.org/24271815:54
*** rochaporto has quit IRC15:59
*** vishpat has joined #puppet-openstack15:59
*** markvoelker has joined #puppet-openstack16:00
*** dhollinger has joined #puppet-openstack16:03
*** rcernin has quit IRC16:04
*** paramite has quit IRC16:05
*** yottatsa has quit IRC16:05
*** vishpat has quit IRC16:07
*** rochaporto has joined #puppet-openstack16:08
openstackgerritMerged openstack/puppet-manila: Manila $::os_service_default for db and log  https://review.openstack.org/24939416:08
*** asilenkov has quit IRC16:10
openstackgerritDenis Egorenko proposed openstack/puppet-glance: Introduce glance::db class  https://review.openstack.org/23797016:12
degorenkoiurygregory, ^^^ thanks for review :)16:13
iurygregoryhey degorenko you're welcome =)16:15
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/20322016:15
mkarpinGuys i today updated https://review.openstack.org/#/c/240946/16:21
mkarpincan you please review it16:21
mkarpinit is small :)16:22
mkarpindegorenko, EmilienM, crinkle, mfisch, sbadia, spredzy : ^^16:23
*** dprince has quit IRC16:25
*** asilenkov has joined #puppet-openstack16:29
*** foxhunt72 is now known as foxhunt_away16:34
*** darkxploit has quit IRC16:38
*** dprince has joined #puppet-openstack16:43
*** vishpat has joined #puppet-openstack16:45
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25077016:47
openstackgerritNate Potter proposed openstack/puppet-glance: Implement multiple store configuration  https://review.openstack.org/24271816:53
openstackgerritBradley Klein proposed openstack/puppet-monasca: Clean up api and persister users/passwords, separate from dbadmin  https://review.openstack.org/24997416:54
*** imcsk8_ is now known as imcsk816:56
*** trown is now known as trown|lunch16:57
*** skolekonov has joined #puppet-openstack16:59
*** xarses has quit IRC17:13
*** nlahouti has joined #puppet-openstack17:14
*** nlahouti has quit IRC17:15
*** nlahouti has joined #puppet-openstack17:28
*** baker has quit IRC17:31
openstackgerritMerged openstack/puppet-heat: Switch Heat to $::os_service_default - part 1  https://review.openstack.org/24911917:34
*** karimb_ has quit IRC17:37
*** haplo37 has joined #puppet-openstack17:38
*** mpryc has left #puppet-openstack17:39
*** dhollinger has quit IRC17:39
*** xarses has joined #puppet-openstack17:40
*** baker has joined #puppet-openstack17:40
*** dhollinger has joined #puppet-openstack17:43
*** degorenko has quit IRC17:43
*** jistr has quit IRC17:54
openstackgerritJavier Peña proposed openstack/puppet-heat: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24571517:55
openstackgerritJavier Peña proposed openstack/puppet-heat: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24571517:57
openstackgerritJavier Peña proposed openstack/puppet-manila: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24572117:58
*** dmsimard is now known as dmsimard|afk18:00
openstackgerritMerged openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25077018:03
*** jpena has quit IRC18:03
*** xiaohua has joined #puppet-openstack18:04
*** derekh has quit IRC18:05
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25200318:05
*** xiaohua has quit IRC18:08
*** aimon has quit IRC18:09
*** vishpat has quit IRC18:17
*** dgurtner has quit IRC18:31
*** trown|lunch is now known as trown18:32
*** ntpttr has quit IRC18:37
*** rcernin has joined #puppet-openstack18:37
*** baker has quit IRC18:42
*** ntpttr has joined #puppet-openstack18:43
*** nlahouti has quit IRC18:44
openstackgerritMerged openstack/puppet-nova: Switch Nova to $::os_service_default - Part1  https://review.openstack.org/24890118:45
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: loadbalancer: add Gnocchi API support  https://review.openstack.org/25201518:49
*** vinsh has quit IRC18:53
*** skolekonov has quit IRC18:56
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for configuring rgw using civetweb  https://review.openstack.org/25202018:58
*** nlahouti has joined #puppet-openstack19:00
*** baker has joined #puppet-openstack19:00
*** rochaporto has quit IRC19:11
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Implement ::gnocchi::client  https://review.openstack.org/25202619:16
*** vishpat has joined #puppet-openstack19:21
*** aimon has joined #puppet-openstack19:23
*** vishpat has quit IRC19:24
*** davideagle has quit IRC19:25
EmilienM_ody, Hunner: have you read the discussion in our ML about Config support for oslo.config.cfg.MultiStrOpt ?19:30
nibalizerHunner: is in australia, not very much around19:31
EmilienMnibalizer: thanks for the info19:31
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Implement ::gnocchi::client  https://review.openstack.org/25202619:37
*** bklei has joined #puppet-openstack19:42
*** baker has quit IRC19:42
*** cdelatte has quit IRC19:44
*** cdelatte has joined #puppet-openstack19:45
*** dmsimard|afk is now known as dmsimard19:45
*** cdelatte has quit IRC19:49
*** cdelatte has joined #puppet-openstack19:50
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for configuring rgw with civetweb  https://review.openstack.org/25203119:53
EmilienMI found the new challenge: backport the patches from master (that have SERVICE DEFAULT) to stable/liberty (that does not have SERVICE_DEFAULT)19:58
iurygregorywow19:58
EmilienMI wanted to backport https://review.openstack.org/#/c/252003/19:59
EmilienMand it failed to pass CI, which is normal19:59
EmilienMhopefully the fact is in stable20:00
iurygregoryyou should add all  dependencies to service default to stable/liberty i think20:01
iurygregorylike the @default_facts in rb files20:02
EmilienMiurygregory: probably20:03
iurygregorytry to send the files i've update in ceilometer20:03
iurygregoryhttps://review.openstack.org/#/c/249377/20:04
*** nlahouti has quit IRC20:12
*** dgurtner has joined #puppet-openstack20:19
*** dgurtner has quit IRC20:19
*** dgurtner has joined #puppet-openstack20:19
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25200320:24
*** vishpat has joined #puppet-openstack20:27
vishpatHi 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:28
vishpatHowever 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 test20:29
vishpatHence would like someone to let me know what am I missing20:29
iurygregoryvishpat, do you have acceptance tests?20:29
vishpathttps://github.com/openstack/puppet-neutron/blob/master/spec/classes/neutron_plugins_nuage_spec.rb20:30
vishpatIs this what do you mean ?20:30
iurygregorynops20:30
vishpatPlease note I am new to this process, so pardon my ignorance20:31
iurygregoryrlx man ;)20:31
EmilienMiurygregory, vishpat: no need of acceptance tests for neutron plugins, since we don't have the SDN installed20:32
iurygregorywell 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
iurygregorytks EmilienM  ;)20:32
vishpatOk it makes sense now20:34
*** cwolferh has quit IRC20:34
vishpatI will make the appropriate changes20:34
iurygregorysimple things =)20:34
vishpatThanks20:36
iurygregoryyou're welcome ^^20:36
*** vishpat has quit IRC20:37
*** vishpat has joined #puppet-openstack20:38
*** bklei has quit IRC20:38
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25200320:41
*** foxhunt_away has quit IRC20:44
*** foxhunt_away has joined #puppet-openstack20:45
*** xiaohua has joined #puppet-openstack20:45
*** davideagle has joined #puppet-openstack20:46
openstackgerritMerged openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/21665420:47
*** xiaohua has quit IRC20:49
*** gfidente has quit IRC21:00
*** asilenkov has quit IRC21:04
*** asilenkov has joined #puppet-openstack21:08
*** dprince has quit IRC21:12
*** vishpat has quit IRC21:12
_odyEmilienM: Got time to go over what my proposed CI changes are?21:15
EmilienM_ody: sure21:15
EmilienM_ody: I need to get the context again21:15
EmilienM_ody: let me read your e-mail again21:16
*** jayg is now known as jayg|g0n321:17
EmilienM_ody: ok - now looking patches21:18
*** nlahouti has joined #puppet-openstack21:19
EmilienM_ody: wow, that creates a lot of jobs for puppet-cinder21:19
EmilienM4 more jobs?21:19
_odyEmilienM: Yeah, I want to make sure they pass before I just delete gate-puppet-*-puppet-beaker-rspec-dsvm-*21:21
EmilienMok21:21
_odyI suspect everyone still wants to test against 3.8.x so I don't see the way around having 4 beaker jobs.21:22
_odyThat 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 has joined #puppet-openstack21:23
_ody(we get most of our parser testing done in rspec though)21:23
*** ferest has joined #puppet-openstack21:24
EmilienM_ody: I think you could trick our beaker jobs to not having to patch project-config21:24
EmilienMyou could work on openstack/puppet-openstack-integration and do what you want21:25
EmilienMyour patches look good to me, but they don't pass CI21:26
EmilienMone thing could be to patch openstack/puppet-openstack-integration to install puppet4 with your method and run the tests21:26
EmilienMthat would avoid changing openstack infra jobs21:27
EmilienMthat's the fastest option21:27
_odyTo test they work you mean?21:27
_odyI can easily make openstack/puppet-openstack-integration default to puppet4 but then puppet3 isn't going to be tested.21:27
EmilienM_ody: yes but it's just for testing21:28
EmilienMI'm ok with having more jobs21:28
EmilienMjust for puppet-cinder21:28
EmilienMbut your patch is not passing CI21:28
*** rahulait has joined #puppet-openstack21:28
_odyEmilienM: I was doing puppet-cinder just as an example, it would had these jobs to everything eventully.21:29
*** ferest has quit IRC21:29
EmilienM_ody: that's too much21:29
EmilienMthat's a lot of resources21:29
_odyThen I'd simply vote to beaker test against puppet4.21:30
EmilienMyeah21:31
*** davideagle has quit IRC21:31
_odyok.  I'll make that happen then.21:31
EmilienMit will break all our CI though probably21:31
EmilienMbut that's fine, we'll fix it :)21:31
*** rcernin has quit IRC21:31
_odycinder passed cleanly, fwiw.21:32
*** jerrygb has quit IRC21:32
_odyWell the beaker test did on my laptop in a VM, I'll look into while the submitted stuff failes.21:33
_odyok, I see it already.21:34
*** vinsh has joined #puppet-openstack21:35
*** imcsk8_ has joined #puppet-openstack21:37
*** imcsk8 has quit IRC21:39
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: (WIP) Add support for different module paths  https://review.openstack.org/24831621:40
openstackgerritTim Rozet proposed openstack/puppet-neutron: Adds configuration support for OpenDaylight SDN Controller  https://review.openstack.org/24939021:41
openstackgerritCody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/24940921:42
openstackgerritCody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/24940921:45
*** davideagle_ has joined #puppet-openstack21:47
*** gildub has joined #puppet-openstack21:47
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement Gnocchi dispatcher  https://review.openstack.org/25200321:50
*** rahulait has quit IRC21:50
_odyEmilienM: 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
_odyWe could do the entire suite on a single nodepool instance.21:50
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/25207721:56
*** trown is now known as trown|outttypeww21:59
EmilienM_ody: I think we already talked about that with nibalizer21:59
EmilienMit came out having multiple jobs was easier to debug and faster22:00
EmilienMI have to go now, I'm back later22:00
*** baker has joined #puppet-openstack22:06
toMeloosSmall heads-up: turns out Liberty breaks the puppet-keystone wsgi example.....22:16
openstackgerritCody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/24940922:19
toMeloosThe apache_with_paths.pp example that is, because two wsgiscipts in one vhost won't work anymove22:19
*** dgurtner has quit IRC22:19
*** fc___ is now known as fc__22:20
openstackgerritCody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/24940922:24
openstackgerritVishal Patil proposed openstack/puppet-neutron: Fix dependencies for Nuage plugin configuration  https://review.openstack.org/24723922:26
*** jerrygb has joined #puppet-openstack22:32
*** jerrygb has quit IRC22:37
openstackgerritCody Herriges proposed openstack/puppet-openstack-integration: (WIP) Add support for different module paths  https://review.openstack.org/24831622:38
*** imcsk8_ has quit IRC22:42
*** imcsk8 has joined #puppet-openstack22:42
*** nlahouti has quit IRC22:45
*** haplo37 has quit IRC22:46
*** nlahouti has joined #puppet-openstack22:56
*** ntpttr has left #puppet-openstack23:19
openstackgerritCody Herriges proposed openstack/puppet-cinder: (WIP) Prototype the usage of different base paths  https://review.openstack.org/24940923:26
*** dhollinger has quit IRC23:29
*** baker has quit IRC23:34
openstackgerritEvan Giles proposed openstack/puppet-gnocchi: Add Influxdb as a possible backend storage for Gnocchi  https://review.openstack.org/25210723:54

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