Friday, 2016-05-13

openstackgerritMerged openstack/puppet-magnum: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/31302400:00
openstackgerritzhongshengping proposed openstack/puppet-neutron: Add the capability to configure neutron_vpnaas.conf  https://review.openstack.org/31535300:19
*** ayoung has joined #puppet-openstack00:20
*** rcernin has quit IRC00:24
*** ntpttr_ has quit IRC00:25
*** markvoelker has joined #puppet-openstack00:27
*** saneax is now known as saneax_AFK00:28
openstackgerritzhongshengping proposed openstack/puppet-aodh: Add spec test for aodh::config  https://review.openstack.org/31581900:31
*** markvoelker has quit IRC00:34
*** grimlock86 has quit IRC00:36
*** ntpttr_ has joined #puppet-openstack00:37
*** johnmilton has quit IRC00:52
*** ntpttr_ has quit IRC01:01
openstackgerritliaonanhai proposed openstack/puppet-manila: Add support for db_max_retries param  https://review.openstack.org/31529701:01
*** openstackgerrit has quit IRC01:02
*** openstackgerrit has joined #puppet-openstack01:03
openstackgerritzhongshengping proposed openstack/puppet-mistral: Define db related parameters in class ::mistral  https://review.openstack.org/31583001:08
openstackgerritzhongshengping proposed openstack/puppet-mistral: Define db related parameters in class ::mistral  https://review.openstack.org/31583001:12
*** ntpttr_ has joined #puppet-openstack01:15
openstackgerritzhongshengping proposed openstack/puppet-mistral: Define db related parameters in class ::mistral  https://review.openstack.org/31583001:21
*** xiaohua has joined #puppet-openstack01:21
*** fedexo has joined #puppet-openstack01:23
openstackgerritzhongshengping proposed openstack/puppet-mistral: Define db related parameters in class ::mistral  https://review.openstack.org/31583001:25
openstackgerritzhongshengping proposed openstack/puppet-neutron: Add the capability to configure neutron_vpnaas.conf  https://review.openstack.org/31535301:29
*** markvoelker has joined #puppet-openstack01:29
openstackgerritzhongshengping proposed openstack/puppet-manila: Fix markdown format typo  https://review.openstack.org/31583301:30
*** foxhunt_away is now known as foxhunt7201:32
*** markvoelker has quit IRC01:34
*** zhongshengping has quit IRC01:41
*** saneax_AFK is now known as saneax01:43
*** ducttape_ has joined #puppet-openstack01:53
*** foxhunt72 is now known as foxhunt_away01:58
*** iurygregory has quit IRC02:13
*** imcsk8 has quit IRC02:15
*** imcsk8 has joined #puppet-openstack02:15
*** xiaohua_ has joined #puppet-openstack02:15
*** foxhunt_away is now known as foxhunt7202:16
openstackgerritzhongshengping proposed openstack/puppet-cinder: Add spec test for cinder::config  https://review.openstack.org/31584102:16
*** xiaohua has quit IRC02:18
openstackgerritzhongshengping proposed openstack/puppet-aodh: Add spec test for aodh::config  https://review.openstack.org/31581902:18
*** cwolferh has quit IRC02:21
*** ducttape_ has quit IRC02:25
*** ducttape_ has joined #puppet-openstack02:26
*** ducttape_ has quit IRC02:27
*** ducttape_ has joined #puppet-openstack02:28
*** foxhunt72 is now known as foxhunt_away02:29
*** markvoelker has joined #puppet-openstack02:30
*** xiaohua has joined #puppet-openstack02:31
*** xiaohua_ has quit IRC02:31
openstackgerritMatt Fischer proposed openstack/puppet-ceilometer: metering_secret was renamed to telemetry_secret  https://review.openstack.org/31584602:33
*** xiaohua_ has joined #puppet-openstack02:33
*** markvoelker has quit IRC02:34
*** xiaohua has quit IRC02:36
mfischEmilienM: see I love ceilometer02:37
mfisch^02:37
openstackgerritMatt Fischer proposed openstack/puppet-ceilometer: metering_secret was renamed to telemetry_secret  https://review.openstack.org/31584602:38
*** _ducttape_ has joined #puppet-openstack02:39
openstackgerritMatt Fischer proposed openstack/puppet-openstack-integration: Rename metering_secret to telemetry_secret  https://review.openstack.org/31584802:40
*** ducttape_ has quit IRC02:43
mfischcrinkle: 3 min02:44
crinklemfisch: k02:46
*** xiaohua_ has quit IRC02:51
*** _ducttape_ has quit IRC02:54
*** xiaohua has joined #puppet-openstack02:57
*** xiaohua_ has joined #puppet-openstack02:59
*** xiaohua has quit IRC02:59
*** agrebennikov has quit IRC03:20
*** saneax is now known as saneax_AFK03:22
*** ducttape_ has joined #puppet-openstack03:24
*** ducttape_ has quit IRC03:24
*** ducttape_ has joined #puppet-openstack03:24
*** markvoelker has joined #puppet-openstack03:31
*** markvoelker has quit IRC03:35
openstackgerritMerged openstack/puppet-keystone: Add some kombu options  https://review.openstack.org/31516903:40
*** chandankumar has joined #puppet-openstack03:50
*** ducttape_ has quit IRC04:00
*** ducttape_ has joined #puppet-openstack04:01
*** ducttape_ has quit IRC04:01
*** ayoung has quit IRC04:07
*** fedexo has quit IRC04:24
*** markvoelker has joined #puppet-openstack04:31
*** markvoelker has quit IRC04:37
*** xiaohua_ has quit IRC04:47
*** xiaohua has joined #puppet-openstack04:48
*** chandankumar has quit IRC05:02
*** jaosorior has joined #puppet-openstack05:21
*** xiaohua has quit IRC05:26
*** xiaohua has joined #puppet-openstack05:26
*** markvoelker has joined #puppet-openstack05:32
*** markvoelker has quit IRC05:37
*** saneax_AFK is now known as saneax05:45
*** rcernin has joined #puppet-openstack05:50
*** fangzhen has quit IRC05:55
*** fangzhen has joined #puppet-openstack05:57
*** fedexo has joined #puppet-openstack05:59
*** fedexo has quit IRC06:04
*** fedexo has joined #puppet-openstack06:04
*** jmikolajczak has joined #puppet-openstack06:06
*** leanderthal|afk has quit IRC06:14
*** foxhunt_away is now known as foxhunt7206:20
*** chandankumar has joined #puppet-openstack06:24
EmilienMmfisch: so glad06:25
EmilienMgood morning06:26
*** fedexo has quit IRC06:28
*** hjensas has quit IRC06:31
*** markvoelker has joined #puppet-openstack06:33
*** markvoelker has quit IRC06:38
*** xiaohua has quit IRC06:48
*** xiaohua has joined #puppet-openstack06:49
*** Mosibi has quit IRC06:50
*** xiaohua has quit IRC06:50
*** Mosibi has joined #puppet-openstack06:51
*** xiaohua has joined #puppet-openstack06:51
EmilienMa lot of beaker jobs are failing, I think we need to land https://review.openstack.org/315770 asap06:52
*** Mosibi has quit IRC06:55
*** tesseract has joined #puppet-openstack06:56
*** Mosibi has joined #puppet-openstack06:58
*** foxhunt72 is now known as foxhunt_away07:02
*** fawadkhaliq has joined #puppet-openstack07:20
*** paramite has joined #puppet-openstack07:20
openstackgerritXiaohua Yuan proposed openstack/puppet-trove: Add Users/DBs related parameters to guestagent class  https://review.openstack.org/30571707:27
*** xiaohua_ has joined #puppet-openstack07:27
openstackgerritXiaohua Yuan proposed openstack/puppet-trove: Add datastore_registry_ext option to trove-guestagent.conf  https://review.openstack.org/30559107:31
*** xiaohua has quit IRC07:31
*** xiaohua_ has quit IRC07:31
*** xiaohua has joined #puppet-openstack07:31
*** amoralej|off is now known as amoralej07:31
*** markvoelker has joined #puppet-openstack07:34
*** hjensas has joined #puppet-openstack07:34
openstackgerritliaonanhai proposed openstack/puppet-manila: Add support for db_max_retries param and  https://review.openstack.org/31529707:35
*** shardy has joined #puppet-openstack07:36
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: deploy keystone at step 3 and 4  https://review.openstack.org/31594307:37
*** danpawlik has joined #puppet-openstack07:37
*** markvoelker has quit IRC07:39
*** karimb has joined #puppet-openstack07:40
*** xiaohua has quit IRC08:09
*** xiaohua has joined #puppet-openstack08:10
*** evidex has joined #puppet-openstack08:13
*** xiaohua has quit IRC08:13
*** xiaohua has joined #puppet-openstack08:13
openstackgerritJavier Peña proposed openstack/puppet-tempest: Rename deprecated configuration options  https://review.openstack.org/31459408:17
*** rmart04 has joined #puppet-openstack08:22
*** gfidente has joined #puppet-openstack08:22
*** leanderthal|afk has joined #puppet-openstack08:25
*** markvoelker has joined #puppet-openstack08:35
openstackgerritMichal Adamczyk proposed openstack/puppet-magnum: Various fixes  https://review.openstack.org/31329308:35
*** markvoelker has quit IRC08:39
*** timrc has quit IRC08:43
*** jtomasek_ has joined #puppet-openstack08:46
*** timrc has joined #puppet-openstack08:46
*** jistr has joined #puppet-openstack08:46
*** leanderthal|afk is now known as leanderthal08:48
*** jtomasek_ has quit IRC08:51
*** jpena|off is now known as jpena08:52
*** dgurtner has joined #puppet-openstack09:05
*** dgurtner has quit IRC09:05
*** dgurtner has joined #puppet-openstack09:05
*** dmk0202 has joined #puppet-openstack09:09
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Deprecate prevent_arp_spoofin option  https://review.openstack.org/31598009:18
*** markvoelker has joined #puppet-openstack09:36
*** markvoelker has quit IRC09:40
openstackgerritBabu Shanmugam proposed openstack/puppet-tripleo: Add OVN support for neutron profile  https://review.openstack.org/31487509:43
*** mkarpin has quit IRC09:44
*** chem has joined #puppet-openstack09:45
openstackgerritBabu Shanmugam proposed openstack/puppet-tripleo: Add OVN support for neutron profile  https://review.openstack.org/31487509:45
*** mkarpin has joined #puppet-openstack09:49
openstackgerritMerged openstack/puppet-ironic: Implement ironic::drivers::deploy class  https://review.openstack.org/31523310:04
openstackgerritMerged openstack/puppet-cinder: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/31519510:05
*** leanderthal has quit IRC10:19
*** beddari has quit IRC10:28
*** jtomasek_ has joined #puppet-openstack10:33
*** markvoelker has joined #puppet-openstack10:36
*** markvoelker has quit IRC10:41
*** jtomasek_ has quit IRC10:42
*** dgurtner has quit IRC10:43
*** dgurtner has joined #puppet-openstack10:45
*** beddari has joined #puppet-openstack10:49
openstackgerritXiaohua Yuan proposed openstack/puppet-trove: Add Users/DBs related parameters to guestagent class  https://review.openstack.org/30571711:02
*** xiaohua_ has joined #puppet-openstack11:02
openstackgerritXiaohua Yuan proposed openstack/puppet-trove: Add datastore_registry_ext option to trove-guestagent.conf  https://review.openstack.org/30559111:02
*** xiaohua__ has joined #puppet-openstack11:03
*** xiaohua_ has quit IRC11:03
*** xiaohua has quit IRC11:03
*** leanderthal has joined #puppet-openstack11:03
openstackgerritzhongshengping proposed openstack/puppet-cinder: Add spec test for cinder::config  https://review.openstack.org/31584111:07
openstackgerritMerged openstack/puppet-neutron: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/31519211:08
*** johnmilton has joined #puppet-openstack11:10
openstackgerritzhongshengping proposed openstack/puppet-neutron: Add the capability to configure neutron_vpnaas.conf  https://review.openstack.org/31535311:13
*** mgould has joined #puppet-openstack11:14
openstackgerritMichal Adamczyk proposed openstack/puppet-magnum: Various fixes  https://review.openstack.org/31329311:14
openstackgerritzhongshengping proposed openstack/puppet-nova: Deprecate listen parameter  https://review.openstack.org/31552411:14
jaosoriorEmilienM: Still online?11:18
*** fangzhen_ has joined #puppet-openstack11:19
*** fangzhen has quit IRC11:23
*** dgurtner has quit IRC11:29
*** dprince has quit IRC11:29
*** jaosorior has quit IRC11:30
openstackgerritJames Slagle proposed openstack/puppet-ironic: Revert "allow to configure my_ip parameter"  https://review.openstack.org/31603911:30
*** jaosorior has joined #puppet-openstack11:30
*** dgurtner has joined #puppet-openstack11:31
openstackgerritJames Slagle proposed openstack/puppet-ironic: Revert "allow to configure my_ip parameter"  https://review.openstack.org/31603911:31
*** morazi has joined #puppet-openstack11:32
*** markvoelker has joined #puppet-openstack11:37
openstackgerritMatthew Mosesohn proposed openstack/puppet-nova: Add nova_service type for purging nova services from offline hosts  https://review.openstack.org/30910311:40
*** markvoelker has quit IRC11:42
*** derekh has joined #puppet-openstack11:42
*** leanderthal has quit IRC11:45
EmilienMjaosorior: yes11:45
*** ansmith has joined #puppet-openstack11:46
EmilienMjaosorior: we reverted the puppet-ironic patch11:48
EmilienMmwhahaha, degorenko: fyi11:48
*** xb_ has joined #puppet-openstack11:52
openstackgerritJiri Stransky proposed openstack/puppet-tripleo: Add a service manifest  https://review.openstack.org/31354312:05
openstackgerritJiri Stransky proposed openstack/puppet-tripleo: Add a service manifest  https://review.openstack.org/31354312:10
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Heat profiles  https://review.openstack.org/31301912:12
openstackgerritAndy Smith proposed openstack/puppet-ironic: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/31524212:13
*** ducttape_ has joined #puppet-openstack12:14
*** leanderthal has joined #puppet-openstack12:16
mwhahahaK12:17
degorenkook12:17
*** jayg|g0n3 is now known as jayg12:19
*** chem has quit IRC12:19
*** aarefiev has quit IRC12:20
*** aarefiev has joined #puppet-openstack12:21
*** chem has joined #puppet-openstack12:23
*** ducttape_ has quit IRC12:24
*** leanderthal has quit IRC12:25
*** shardy has quit IRC12:28
*** gfidente has quit IRC12:30
EmilienMmwhahaha: can we get https://review.openstack.org/#/c/315770/ please?12:33
EmilienMI'm afraid a lot of beaker jobs are failing without it12:33
*** dprince has joined #puppet-openstack12:34
*** markvoelker has joined #puppet-openstack12:38
*** jpena is now known as jpena|lunch12:39
*** iurygregory has joined #puppet-openstack12:41
*** rcernin_ has joined #puppet-openstack12:42
*** rcernin has quit IRC12:42
*** markvoelker has quit IRC12:43
iurygregorymorning12:43
mwhahahaOK +A12:45
*** xiaohua__ has quit IRC12:54
*** ayoung has joined #puppet-openstack13:00
openstackgerritDenis Egorenko proposed openstack/puppet-swift: Add systemd service provider for Xenial  https://review.openstack.org/31608013:02
degorenkovinsh, ping and take a look ^13:02
openstackgerritMerged openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/26346113:03
*** fedexo has joined #puppet-openstack13:06
*** dprince has quit IRC13:11
*** dprince has joined #puppet-openstack13:11
*** rcernin_ has quit IRC13:16
*** ducttape_ has joined #puppet-openstack13:16
*** fawadkhaliq has quit IRC13:20
*** fawadkhaliq has joined #puppet-openstack13:20
*** fedexo has quit IRC13:23
*** fawadkhaliq has quit IRC13:25
*** rcernin has joined #puppet-openstack13:27
* mjblack yawns 13:27
openstackgerritMerged openstack/puppet-keystone: stop managing eventlet service on RedHat  https://review.openstack.org/31577013:28
EmilienMok if you still see some beaker jobs errors on centos7 because of keystone service, please ping me13:28
openstackgerritDenis Egorenko proposed openstack/puppet-openstack-integration: WIP: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/30853013:37
*** markvoelker has joined #puppet-openstack13:39
*** xiaohua has joined #puppet-openstack13:39
openstackgerritPradeep Kilambi proposed openstack/puppet-tripleo: loadbalancer: add Aodh API support  https://review.openstack.org/28812213:43
*** openstack has joined #puppet-openstack13:46
EmilienMhttps://review.openstack.org/#/c/312770/13:47
vinshIt worked for you.... just that you needed to update all the spec tests to match that I think13:47
degorenkovinsh, do not use any hardcoded providers at all?13:47
vinshright13:47
vinshpuppet will choose upstart for 14.0413:47
EmilienMyes but AFIK there is a bug _ody found13:47
vinshoh https://github.com/puppetlabs/puppet/blob/master/lib/puppet/provider/service/upstart.rb13:47
EmilienMhttps://review.openstack.org/#/c/312284/13:47
*** jpena|lunch is now known as jpena13:47
EmilienMhttps://tickets.puppetlabs.com/browse/PUP-535313:48
EmilienMI'm not sure it's related13:48
*** openstack has joined #puppet-openstack13:54
*** markvoelker has joined #puppet-openstack13:55
*** derekh has quit IRC13:59
*** chandankumar has quit IRC14:03
*** ducttape_ has quit IRC14:05
*** ducttape_ has joined #puppet-openstack14:07
clayton_ody: did you guys ever get a vmpooler equivalent for openstack?14:07
openstackgerritAde Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/26346214:08
openstackgerritMerged openstack/puppet-ironic: Revert "allow to configure my_ip parameter"  https://review.openstack.org/31603914:08
aleedegorenko, good catch on the missing manifest.  I was still trying to wrack my brains and figure out why it wasn't failing for me locally.14:10
degorenkoalee, it happens :) no problem14:11
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Revert "Revert "allow to configure my_ip parameter""  https://review.openstack.org/31611214:12
mwhahahaoh the good ol' revert revert14:12
degorenkoha ha14:13
iurygregoryrevert can always solve a problem =P14:13
iurygregorybut you can't revert your life =( ahsuahsuahsuash14:13
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Revert "Revert "allow to configure my_ip parameter""  https://review.openstack.org/31611214:13
degorenkoiurygregory, what does mean 'ahsuahsuahsuash' ?14:14
iurygregorydegorenko, ''ahsuahsuahsuash'' == laugh in Brazil XD sorry14:15
degorenkoah, ok :)14:16
EmilienMor just say 'lol'14:16
iurygregorylol its common in game i think XD sorry14:18
vinshIt's fun to try and laugh saying "ahsuahsuahsuash" :p14:18
degorenko:D14:18
*** jaosorior has quit IRC14:19
iurygregoryi can do this \o/ people say that when i laugh it's funny14:19
vinshdegorenko: Looks like CI timeout/failure in 316080 scenario00214:23
degorenkorecheck is coming14:24
mfischEmilienM: is there a way to undef a parameter in rspec?14:26
mfischlike I want all the defaults but want to remove 114:26
iurygregorynil will work i think14:28
mfischlemme try14:30
iurygregorygood luck o/14:30
*** dgurtner has quit IRC14:34
mfischadding a test so I get to pick the string14:34
mfisch'broncos' it is14:34
openstackgerritMatt Fischer proposed openstack/puppet-ceilometer: metering_secret was renamed to telemetry_secret  https://review.openstack.org/31584614:35
mfischdegorenko: can you look at https://review.openstack.org/#/c/314408/ today14:35
degorenkomfisch, ack14:36
mfischspasiba14:36
degorenkolooks good, +A14:37
mfischdegorenko: I updated this hot mess too14:37
mfischhttps://review.openstack.org/#/c/315846/14:37
mfischceilometer devs /me shakes fist14:38
mfischoh dang whitespace, lemme fix14:38
openstackgerritMatt Fischer proposed openstack/puppet-ceilometer: metering_secret was renamed to telemetry_secret  https://review.openstack.org/31584614:38
*** shardy has joined #puppet-openstack14:39
openstackgerritzhongshengping proposed openstack/puppet-aodh: Add spec test for aodh::config  https://review.openstack.org/31581914:43
openstackgerritzhongshengping proposed openstack/puppet-cinder: Add spec test for cinder::config  https://review.openstack.org/31584114:43
*** alee has left #puppet-openstack14:44
*** alee has joined #puppet-openstack14:44
aleedegorenko, EmilienM, mwhahaha  -- jenkins is happy again -- https://review.openstack.org/#/c/263462/14:45
degorenkoalee, may be then you can also back listening with('tcp') ? :)14:47
*** imcsk8 is now known as imcsk8|afk14:47
aleedegorenko, I ended up taking that out before because the test failed with tcp14:47
*** gfidente has joined #puppet-openstack14:48
aleedegorenko, I dont think that was related to the missing manifest14:48
aleedegorenko, I'll try locally again - but I believe it fails in this case.14:48
openstackgerritzhongshengping proposed openstack/puppet-barbican: Add unit test for barbican::config  https://review.openstack.org/31613414:50
*** mattymo` has joined #puppet-openstack14:58
mattymo`looking for reviewers: https://review.openstack.org/#/c/309103/ finally passes CI14:58
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Adds the puppet-tripleo manifests for manila  https://review.openstack.org/31352715:01
*** yo_ has joined #puppet-openstack15:07
openstackgerritBrad P. Crochet proposed openstack/puppet-tripleo: Add Heat profiles  https://review.openstack.org/31301915:07
*** alee is now known as alee_afk15:08
degorenkoalee_afk, ok15:10
vinshdegorenko: Crazy.. when I apply your patch 316080.. almost all of the spec tests fail for latest puppet with "     Evaluation Error: Error while evaluating a Function Call, 'versioncmp' parameter 'a' expects a String value, got Undef at /root/puppet-swift/spec/fixtures/modules/swift/manifests/params.pp:30:12 on node puppet-xenial.openstacklocal"15:12
degorenkovinsh, what?15:13
degorenkohow?15:13
vinshsee http://logs.openstack.org/80/316080/1/check/gate-puppet-swift-puppet-unit-latest-centos-7/926256a/console.html15:13
*** karimb has quit IRC15:13
*** dmk0202 has quit IRC15:13
*** saneax is now known as saneax_AFK15:13
*** karimb has joined #puppet-openstack15:14
*** haplo37 has joined #puppet-openstack15:14
openstackgerritDmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params  https://review.openstack.org/31459615:15
openstackgerritMerged openstack/puppet-trove: default swift_url to SERVICE_DEFAULT  https://review.openstack.org/31440815:16
*** openstackgerrit has quit IRC15:17
*** openstackgerrit has joined #puppet-openstack15:18
vinshdegorenko: Ah, maybe centos doesn't have a fact for operatingsystemmajrelease15:20
degorenkovinsh, it is case for debian based os15:21
vinshBut, I am running it on 16.04 and when I debug operatingsystemmajrelease indeed is "16.04"15:21
vinshoh yeah15:21
degorenkowhen i tested it on my 16.04 - everything was fine15:21
vinshSo the value is populated.. but needs to be stubbed out for the tests I think15:21
degorenkoprobably yes15:21
degorenkoit is not in tests15:21
degorenkoyou're right15:21
degorenkobut then - its strange that it passed ci15:22
degorenkooh, of course it is passed on 14.0415:22
vinshwell.. it's the unit tests that fail with puppet >4.015:22
vinshSomething about latest puppet and that fact15:23
degorenkoso, fact is missed in tests15:23
degorenkohow is puppet4 here?15:23
vinshhttp://logs.openstack.org/80/316080/1/check/gate-puppet-swift-puppet-unit-latest-centos-7/926256a/15:23
vinshruns latest puppet.. like I have on my test node15:23
vinshpuppet apply shows a value for that fact.. but running spec tests it fails15:24
degorenkogot undef - right15:24
degorenkobecause tests dont know about this :D15:24
vinshwhy does it pass with puppet 4.0 then?15:24
vinshgate-puppet-swift-puppet-unit-4.0-centos-7  PASS15:25
degorenkogood question15:25
degorenkomay be here some changes in versioncmp function?15:25
degorenkobetween 4.0 and latest?15:25
vinshYeah.. that sounds right15:25
degorenkoanyway, we need update tests as well, because it will fail on xenial - i mean unit tests15:26
degorenkoand on centos15:26
vinshfound it15:26
degorenkoand? :) where is problem?15:26
vinshneed quotes15:26
vinsh       if versioncmp('$::operatingsystemmajrelease', '16') >= 0 {15:26
iurygregoryO.o15:26
vinshotherwise it doesn't evaluate15:26
iurygregorylol15:26
mwhahahathe fact is missing15:26
degorenkoquotes15:26
mwhahahain the test15:26
mwhahahathat's the fix15:26
vinshno. the fact is there15:27
degorenkoreally?15:27
vinshJust versioncomp with puppet 4.4 is different15:27
degorenkoah15:27
mwhahahano, it's missing15:27
mwhahahahttp://logs.openstack.org/80/316080/1/check/gate-puppet-swift-puppet-unit-latest-centos-7/926256a/console.html#_2016-05-13_13_20_05_97115:27
mwhahahaUndef15:28
vinshthat's because its missing quotes15:28
mwhahahano15:28
mwhahahathat's missing15:28
mwhahahaquoting it just turns it to an empty string15:28
mwhahahathe problem is that the fact is not there15:28
degorenko^ yes, not undef15:28
degorenkobut fact is missing15:28
mwhahahathe correct fix is to provide the fact15:28
degorenkopuppet 4.4 cant work with undef15:28
mwhahahawrapping in quotes hides it15:28
*** ducttape_ has quit IRC15:28
vinshok.. so puppet 4.4 alerts on this when 4.0 would let an empty string pass.15:28
degorenkoand older version undef == ''15:29
degorenkosounds like that15:29
mwhahahayea we've hit this oodles of times15:29
mwhahahaif you switch to rspec-puppet-facts this info gets provided15:29
vinshGood to know, thanks for the detail15:29
mwhahahaother wise you need to provide the fact15:29
vinshis that a module-wide switch that needs to be done?15:30
mwhahahano you can do it on a per test basis15:30
degorenkoso, for debian based for now we can hardcode 14.04 version i guess15:30
degorenkoor i can implement for xenial too15:30
vinshwell. centos needs a version too15:31
degorenkono15:31
degorenkoits only for debian15:31
*** chandankumar has joined #puppet-openstack15:31
degorenkoat least case for debian :D15:32
*** ducttape_ has joined #puppet-openstack15:32
mwhahahaunfortunately because you put that in params, it needs to be defined for all tests that will consume params15:32
vinshYes. sorry15:32
mwhahahaso you'd have to provide the fact in all of the swift tests probably15:32
degorenkooh, crap15:32
*** ericksonsantos has joined #puppet-openstack15:33
mwhahahaor wrap it in quotes with a TODO to remove once the tests get migrated to rspec-puppet-facts15:33
mwhahahaaka dirty hack15:33
vinshFor debian.. I think we can remove any hard code of provider in params15:33
degorenkoi think we can ask _ody about bug in service providers15:34
vinshWhen I look at the nova case EmilienM provided from _ody.. it was only problems with puppet latest on redhat based15:34
vinshhttps://review.openstack.org/#/c/312284/4/manifests/params.pp15:34
degorenkoyeah, i saw15:35
degorenkomay be here some another issue for debian15:35
vinshI guess that would apply to 16.04 though..15:35
vinsh16.04 + puppet 4.0-4.4 maybe15:36
degorenkowell, undef for service provider for debian15:36
degorenkolooks like it can work15:36
degorenkovinsh, i can reupload my patch without specific provider name15:36
vinshYeah, will need to clean the unit tests too to match I think.15:37
degorenkook, i ll do that15:37
vinshcool.. I believe that is a better design for this part.15:38
*** mgould has quit IRC15:38
vinshI'm fixing the swift-init provider now to handle 16.0415:39
vinshFolks should just use that provider anyway for all OS's and swift :) says me.15:39
vinshIt turns out it works pretty well with docker+puppet for swift also.15:40
degorenkosounds good :)15:40
openstackgerritDmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes  https://review.openstack.org/31459715:40
vinshI have a change soon to review on the os_docker hooks for puppet-swift. very similar to the nova/keystone/desginate stuff.15:40
openstackgerritDmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes  https://review.openstack.org/31459715:41
openstackgerritDmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params  https://review.openstack.org/31459615:41
*** cwolferh has joined #puppet-openstack15:42
*** jmikolajczak has quit IRC15:42
*** ntpttr- has quit IRC15:42
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for logging  https://review.openstack.org/31554815:43
*** haplo37 has quit IRC15:47
*** aimon has quit IRC15:48
openstackgerritDmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes  https://review.openstack.org/31459815:48
*** mgould has joined #puppet-openstack15:49
*** ifarkas has quit IRC15:51
*** aimon has joined #puppet-openstack15:55
openstackgerritJames Slagle proposed openstack/puppet-keystone: Revert "stop managing eventlet service on RedHat"  https://review.openstack.org/31616715:55
openstackgerritJames Slagle proposed openstack/puppet-keystone: Revert "stop managing eventlet service on RedHat"  https://review.openstack.org/31616715:55
*** tesseract has quit IRC15:56
iurygregory wow O.o15:56
*** karimb has quit IRC15:56
*** rmart04 has quit IRC15:57
mwhahahahmm seems like tripleo should setup that dependency differently15:57
mwhahahaas you shouldn't be requiring Service['keystone'] when running under httpd15:58
openstackgerritDenis Egorenko proposed openstack/puppet-swift: Do not use any specific service provider for ubuntu  https://review.openstack.org/31608016:01
degorenkovinsh, that should be fine now ^16:01
*** yo_ has quit IRC16:01
*** degorenko is now known as _degorenko|afk16:02
*** rcernin has quit IRC16:02
vinshVery good.  be back in an hour. LUNCH TIME.16:02
*** alee_afk is now known as alee16:06
*** hjensas has quit IRC16:07
*** pma_ has quit IRC16:11
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Update keystone service name for signing keys  https://review.openstack.org/31617716:11
iurygregorythis make more sense i think ^16:12
mwhahaha:D16:12
iurygregorynice catch mwhahaha o/16:13
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/31617916:16
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/31617916:18
*** markvoelker has quit IRC16:18
*** agrebennikov has joined #puppet-openstack16:22
*** markvoelker has joined #puppet-openstack16:22
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/30478016:23
*** ducttape_ has quit IRC16:27
iurygregoryEmilienM, https://review.openstack.org/31617716:29
EmilienMyeah I saw it16:29
iurygregorymaybe will help i think =)16:29
EmilienMand yeah nice catch alex16:29
*** shardy has quit IRC16:32
EmilienMmfisch: yeah, nil16:33
EmilienMsorry for lag16:33
EmilienMvinsh: yeah, we had issues with redhat, nevermind about this one16:34
*** ducttape_ has joined #puppet-openstack16:41
mfischEmilienM: merci16:42
*** jistr has quit IRC16:42
*** aimon has quit IRC16:44
*** imcsk8|afk is now known as imcsk816:46
*** aimon has joined #puppet-openstack16:53
*** ducttape_ has quit IRC16:54
*** mgould has quit IRC16:56
*** kgalanov has quit IRC16:56
*** holser has quit IRC16:56
*** igorbelikov has quit IRC16:56
*** jpena is now known as jpena1off17:07
*** evidex has quit IRC17:07
alee_degorenko|afk, yeah - it still fails with tcp - not sure why17:09
alee_degorenko|afk, but it passes as is now .. :p17:10
aleemwhahaha, thanks!17:10
aleeEmilienM, finally!  all the patches are in.  time to start thinking about doing tempest tests.17:11
alee(and focusing on the triple-o patches)17:12
EmilienMalee: congrats17:12
EmilienMalee: and thanks for your hard work on that17:12
EmilienMalee: I'll help you for the integration17:12
EmilienMalee: I was thinking at using a scneario where cinder is running so we can test cinder volume encryption17:13
EmilienMand also later, maybe a new scenario with magnum so we test magnum + barbican + lbaas (they depend each others)17:13
aleeEmilienM, cool - I need to head out to the garage right now - but I'll check back in with you later today17:13
aleeEmilienM, but yeah encrypted volumes for sure, maybe magnum, and swift just did encrypted objects17:14
*** alee is now known as alee_garage17:15
EmilienMmwhahaha: in https://review.openstack.org/#/c/316177/1/manifests/profile/base/keystone.pp -- should we include ::apache::params?17:15
EmilienMwell, maybe it's not needed17:15
mwhahahait's possible17:16
EmilienMI'm watching CI now I'll let you know17:16
mwhahahabut i think that's included in ::keystone::wsgi::apache no?17:16
EmilienMyeah17:16
mwhahahai also noticed the file permissions17:16
mwhahahasince they are set to keystone17:17
mwhahahabut since i've never actually used tripleo, i have no idea what i'm doing :)17:17
*** holser has joined #puppet-openstack17:17
*** alee_garage has quit IRC17:20
*** kgalanov has joined #puppet-openstack17:20
openstackgerritAndy Smith proposed openstack/puppet-sahara: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/31518617:21
iurygregorycongrats mwhahaha jenkis +1 XD17:21
mwhahahawell that's cause there are no actual tests for those ;)17:21
mwhahahaat least not in rspec17:22
mwhahahaso +1 for not creating a syntax error17:22
iurygregoryonly syntax? O.o17:22
*** mattymo` has quit IRC17:22
openstackgerritMerged openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/26346217:23
*** dh64 has quit IRC17:28
*** ansmith has quit IRC17:31
*** igorbelikov has joined #puppet-openstack17:36
*** chandankumar has quit IRC17:36
*** ansmith has joined #puppet-openstack17:46
EmilienMmwhahaha: jobs are still running, finger crosses17:48
EmilienMcrossed*17:48
*** alee has joined #puppet-openstack17:52
*** ducttape_ has joined #puppet-openstack17:55
*** ducttape_ has quit IRC17:56
*** ducttape_ has joined #puppet-openstack17:56
*** xiaohua has joined #puppet-openstack18:04
*** xiaohua has quit IRC18:09
*** cwolferh has quit IRC18:20
*** ansmith has quit IRC18:26
*** ansmith has joined #puppet-openstack18:27
*** ducttape_ has quit IRC18:30
openstackgerritMerged openstack/puppet-nova: Adding a purge_config option for nova_config  https://review.openstack.org/31273418:31
EmilienMmwhahaha: it worked!18:32
mwhahahaYay18:33
EmilienMmwhahaha: you fixed tripleo CI, but we won't tell anyone.18:33
openstackgerritMerged openstack/puppet-tripleo: Update keystone service name for signing keys  https://review.openstack.org/31617718:36
*** jtomasek_ has joined #puppet-openstack18:36
*** cwolferh has joined #puppet-openstack18:37
*** jtomasek_ has quit IRC18:38
*** jtomasek_ has joined #puppet-openstack18:39
*** ayoung has quit IRC18:42
*** morazi has quit IRC18:44
*** alee has quit IRC18:48
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: deploy keystone at step 3 and 4  https://review.openstack.org/31594318:48
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Revert "Revert "allow to configure my_ip parameter""  https://review.openstack.org/31611218:48
*** gfidente has quit IRC18:52
*** ducttape_ has joined #puppet-openstack18:54
*** darkxploit has joined #puppet-openstack18:55
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Swift defaults Nova endpoints to 2.1  https://review.openstack.org/31624118:58
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Swift defaults Nova endpoints to 2.1  https://review.openstack.org/31624119:04
*** xb_ has quit IRC19:06
openstackgerritNathan Taylor proposed openstack/puppet-keystone: Adding a purge_config option for keystone_config  https://review.openstack.org/31624519:13
*** jtomasek_ has quit IRC19:27
*** ayoung has joined #puppet-openstack19:30
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Switch API to 2.1 and deprecate 3  https://review.openstack.org/31624119:47
*** dfisher has quit IRC19:48
EmilienMwe need to be super careful with ^19:48
mjblackEmilienM: could you take a quick look at https://review.openstack.org/#/c/313621/? I'm having trouble with the unit tests with going from ini setting to openstackconfig, its expecting a string but getting back an array.19:49
mjblackI'm at a lost of where the issue is19:50
EmilienMwell, beaker jobs are failing too19:50
pabelangerEmilienM: http://mirror.dfw.rax.openstack.org/centos/7/cloud/ online now, so you can updated your yum repolist to use it19:50
EmilienMmjblack: error is pretty obvious http://logs.openstack.org/21/313621/2/check/gate-puppet-neutron-puppet-unit-4.0-centos-7/6bcf936/console.html.gz#_2016-05-06_17_03_27_60519:51
EmilienMit expect a string and it got an array19:51
EmilienMpabelanger: that is really awesome, I'm going to prepare that right now19:51
mjblackEmilienM: right but I'm not sure where to fix it, unless you're saying update it to expect an array :D19:51
pabelangerEmilienM: Ya, you can see how we do it in https://github.com/openstack-infra/project-config/blob/master/nodepool/scripts/configure_mirror.sh for CentOS-Base.repo19:52
EmilienMpabelanger: I do it with puppet :)19:54
EmilienMpatch is coming19:55
pabelangerEmilienM: right, /etc/nodepool/provider will give you the nodepool cloud details19:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: switch centos mirror to use OpenStack mirror  https://review.openstack.org/31626620:02
EmilienMpabelanger: ^ for master (for stable branches will come later)20:02
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: Allow to change CentOS mirror URL  https://review.openstack.org/31626720:02
EmilienMpabelanger: and that will be for stable branches ^20:02
*** dmk0202 has joined #puppet-openstack20:03
EmilienMmjblack: https://review.openstack.org/#/c/313621/2/lib/puppet/type/neutron_config.rb20:03
pabelangerEmilienM: that won't work well, as each cloud provider has a local mirror: EG: http://mirror.nyj01.internap.openstack.org/centos/7/20:03
EmilienMit sounds like you're returning an array, isn't?20:03
pabelangerEmilienM: you'll have to check which cloud / region you are in and configure accordingly20:04
EmilienMo_O20:04
EmilienMwe can't do it with puppet20:04
EmilienMwell, actually we might be able with some tricks20:05
EmilienMpabelanger: can you show me /etc/nodepool/provider example?20:05
pabelangerEmilienM: http://docs.openstack.org/infra/nodepool/scripts.html#ready-script20:06
pabelangerthe configure_mirror.sh script I liked before will show how to build up the URL20:06
EmilienMoh ok so we need to use it20:07
pabelangerlinked*20:07
EmilienMpabelanger: configure_mirror.sh is automatically executed ?20:07
pabelangerEmilienM: yes, each time we launch a node for the first time20:08
pabelangerEmilienM: but we won't add the cloud repo into it, since no other job uses it20:08
EmilienMmhh ok but I need to find how configure the cloud repo20:08
EmilienMyeah, and it's versionned20:08
EmilienMmitaka/liberty, etc20:08
EmilienMI guess for now, I can use http://mirror.dfw.rax.openstack.org/centos/7/cloud/20:09
EmilienMit will probably be a good start, isn't?20:09
pabelangergive me a few minutes to think about it20:09
*** paramite has quit IRC20:09
EmilienMI'm going to sleep20:10
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP swift-init provider xenial support  https://review.openstack.org/31627020:14
*** jayg is now known as jayg|g0n320:15
EmilienMpabelanger: if you have some idea, please let me know if Gerrit, thanks!20:17
pabelangerEmilienM: will do20:18
EmilienMthanks! have a nice week-end folks!20:18
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Swift-ring-builder skip line 'Ring up-to-date..'  https://review.openstack.org/31627420:20
*** johnmilton has quit IRC20:21
*** dmk0202 has quit IRC20:22
*** morazi has joined #puppet-openstack20:24
*** xiaohua has joined #puppet-openstack20:31
*** ansmith has quit IRC20:35
*** xiaohua has quit IRC20:36
*** beddari has quit IRC20:36
*** chem` has joined #puppet-openstack20:47
*** chem has quit IRC20:49
openstackgerritMerged openstack/puppet-trove: Use oslo module for logging  https://review.openstack.org/31554820:59
*** johnmilton has joined #puppet-openstack21:02
iurygregorybye people, have a good weekend o/21:07
*** iurygregory has quit IRC21:09
*** johnmilton has quit IRC21:12
*** ducttape_ has quit IRC21:15
vinshPuppet 4.4.2... when applying file resources.. for a file that doesn't exist.  Causes an md5sum error.21:17
vinshit makes me sad.21:17
vinsh*when applying a file resource from within a provider21:18
vinshI think this feature introduced some bug here https://tickets.puppetlabs.com/browse/PUP-569521:20
*** ducttape_ has joined #puppet-openstack21:23
vinshonly hitting this when running puppet apply locally.  Not seeing the same error in gate jobs.21:24
vinshhrm.21:24
*** johnmilton has joined #puppet-openstack21:24
*** markvoelker has quit IRC21:26
openstackgerritCody Herriges proposed openstack/puppet-neutron: Change MidoNet Plugin package name  https://review.openstack.org/29448621:31
*** ansmith has joined #puppet-openstack21:36
*** ducttape_ has quit IRC21:36
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Support xenial with swift-init service provider  https://review.openstack.org/31627021:38
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Support xenial with swift-init service provider  https://review.openstack.org/31627021:40
*** dprince has quit IRC21:45
*** dmk0202 has joined #puppet-openstack21:53
*** dmk0202 has quit IRC22:06
*** markvoelker has joined #puppet-openstack22:27
*** markvoelker has quit IRC22:31
*** ducttape_ has joined #puppet-openstack22:36
mfischEmilienM: je t'aime ceilometer: https://bugs.launchpad.net/puppet-ceilometer/+bug/158169122:42
openstackLaunchpad bug 1581691 in puppet-ceilometer "ceilometer config options for agents are broken, polling agents are broken" [Undecided,New]22:42
*** agrebennikov has quit IRC22:42
mfischI will work on that this weekend22:43
crinkleeverything is broken \o/22:43
mfischwe inherited that mess22:43
mfischits not puppet's fault22:44
mfischcrinkle: https://twitter.com/openmfisch/status/73059162300484812822:44
crinklehahaha22:44
*** xiaohua has joined #puppet-openstack22:59
*** xiaohua has quit IRC23:03
*** cwolferh has quit IRC23:17
*** ducttape_ has quit IRC23:20
*** markvoelker has joined #puppet-openstack23:28
*** markvoelker has quit IRC23:32
*** cwolferh has joined #puppet-openstack23:33
*** saneax_AFK is now known as saneax23:42
*** johnmilton has quit IRC23:45

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