Monday, 2016-05-16

« Sunday, 2016-05-15 Index Tuesday, 2016-05-17 »
*** saneax is now known as saneax_AFK00:17
openstackgerritMatthew J Black proposed openstack/puppet-neutron: Consolidate service_providers into neutron server  https://review.openstack.org/31362100:37
openstackgerritzhongshengping proposed openstack/puppet-keystone: Deprecated PKI signing related parameter  https://review.openstack.org/31644601:10
openstackgerritzhongshengping proposed openstack/puppet-keystone: Add unit test for keystone::config  https://review.openstack.org/31653101:20
mfischEmilienM: done01:32
mfischa bit earlier but didnt mention here01:32
*** ilbot3 has quit IRC01:45
*** ilbot3 has joined #puppet-openstack01:48
openstackgerritliaonanhai proposed openstack/puppet-manila: Add support for db_max_retries param  https://review.openstack.org/31529701:52
*** xiaohua has joined #puppet-openstack02:02
openstackgerritzhongshengping proposed openstack/puppet-openstack-integration: Increased integration testing instruction for mistral  https://review.openstack.org/31653602:18
xiaohuahi,help.the path pass unite testing,but Patch in merge conflict.How can I do to solve the problem？ url:https://review.openstack.org/#/c/305591/02:34
*** guessi has joined #puppet-openstack02:37
*** ducttape_ has joined #puppet-openstack03:38
*** furlongm has quit IRC03:40
*** foxhunt_away is now known as foxhunt7203:46
*** ducttape_ has quit IRC04:02
*** ducttape_ has joined #puppet-openstack04:02
*** foxhunt72 is now known as foxhunt_away04:06
*** chandankumar has joined #puppet-openstack04:06
*** ducttape_ has quit IRC04:07
*** v1k0d3n has quit IRC04:31
openstackgerritMatthew J Black proposed openstack/puppet-neutron: Consolidate service_providers into neutron server  https://review.openstack.org/31362104:34
*** xiaohua has quit IRC04:47
*** xiaohua has joined #puppet-openstack04:48
*** v1k0d3n has joined #puppet-openstack05:01
*** rcernin has joined #puppet-openstack05:16
*** rcernin has quit IRC05:21
*** saneax_AFK is now known as saneax05:22
*** xiaohua has quit IRC05:26
*** xiaohua has joined #puppet-openstack05:26
*** rcernin has joined #puppet-openstack05:38
*** xiaohua has quit IRC05:39
*** xiaohua has joined #puppet-openstack05:40
*** xiaohua has quit IRC05:49
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Create keystone endpoints in keystone profile  https://review.openstack.org/30412505:49
*** xiaohua has joined #puppet-openstack05:56
*** xiaohua has quit IRC06:04
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/31658306:05
*** chandankumar has quit IRC06:11
*** chandankumar has joined #puppet-openstack06:11
*** xiaohua has joined #puppet-openstack06:12
*** jaosorior has joined #puppet-openstack06:18
*** xiaohua has quit IRC06:18
*** xiaohua has joined #puppet-openstack06:26
*** xiaohua has quit IRC06:27
*** xiaohua has joined #puppet-openstack06:27
*** jmikolajczak has joined #puppet-openstack06:30
*** paramite has joined #puppet-openstack06:45
*** jaosorior has quit IRC06:55
*** jaosorior_ has joined #puppet-openstack06:56
EmilienMw00t, proposal bot works!06:57
*** jaosorior_ is now known as jaosorior07:00
*** v1k0d3n has quit IRC07:06
openstackgerritBabu Shanmugam proposed openstack/puppet-tripleo: Add OVN support for neutron profile  https://review.openstack.org/31487507:19
*** jtomasek has joined #puppet-openstack07:26
*** igajsin has quit IRC07:27
EmilienMdegorenko: if you have time, I have some patches for review, let me know if you're arround07:31
*** v1k0d3n has joined #puppet-openstack07:33
*** igajsin has joined #puppet-openstack07:33
*** igajsin has left #puppet-openstack07:34
*** shardy has joined #puppet-openstack07:40
*** aarefiev has quit IRC07:43
*** ddmitriev has quit IRC07:43
*** mkarpin has quit IRC07:43
*** aarefiev has joined #puppet-openstack07:44
*** aarefiev has quit IRC07:44
*** aarefiev has joined #puppet-openstack07:44
*** mkarpin has joined #puppet-openstack07:44
openstackgerritMichal Adamczyk proposed openstack/puppet-magnum: Various fixes  https://review.openstack.org/31329307:49
EmilienMwho is ^ ?07:49
EmilienMjust fyi, magnum looks broken in master, I noticed errors in conductor service (see logs)07:50
*** ddmitriev has joined #puppet-openstack07:52
*** hjensas has joined #puppet-openstack07:54
*** hjensas has quit IRC07:54
*** hjensas has joined #puppet-openstack07:54
openstackgerritzhongshengping proposed openstack/puppet-magnum: Add spec tests for magnum::db::sync  https://review.openstack.org/31152407:58
*** jaosorior has quit IRC08:03
*** jaosorior has joined #puppet-openstack08:03
*** markvoelker has joined #puppet-openstack08:15
*** markvoelker has quit IRC08:23
openstackgerritzhongshengping proposed openstack/puppet-magnum: Add spec tests for magnum::db::sync  https://review.openstack.org/31152408:24
*** markvoelker has joined #puppet-openstack08:24
*** jistr has joined #puppet-openstack08:30
*** dmk0202 has joined #puppet-openstack08:32
*** derekh has joined #puppet-openstack08:45
*** pma has joined #puppet-openstack08:46
*** danpawlik has quit IRC08:49
*** danpawlik has joined #puppet-openstack08:49
*** paramite has quit IRC08:51
*** paramite has joined #puppet-openstack08:53
*** mkarpin has quit IRC09:08
*** mkarpin has joined #puppet-openstack09:09
mattymohi all! Can I get some reviews?09:11
mattymohttps://review.openstack.org/#/c/309103/09:11
*** evidex has joined #puppet-openstack09:21
openstackgerritzhongshengping proposed openstack/puppet-manila: Switch manila to oslo  https://review.openstack.org/31634409:30
openstackgerritzhongshengping proposed openstack/puppet-manila: Switch manila to oslo  https://review.openstack.org/31634409:30
*** rmart04 has joined #puppet-openstack09:47
*** dtantsur is now known as dtantsur|brb09:53
openstackgerritSharat Sharma proposed openstack/puppet-designate: Added the options to be configured for designate-central service  https://review.openstack.org/31669109:55
*** johnmilton has quit IRC10:01
*** xb_ has joined #puppet-openstack10:03
*** dgurtner has joined #puppet-openstack10:04
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Adds the puppet-tripleo manifests for manila  https://review.openstack.org/31352710:04
*** mgould has joined #puppet-openstack10:23
openstackgerritSharat Sharma proposed openstack/puppet-designate: Added rabbit_ha_queues option  https://review.openstack.org/31670710:27
*** paramite is now known as paramite|afk10:33
openstackgerritSharat Sharma proposed openstack/puppet-designate: Added the options to be configured for designate-central service  https://review.openstack.org/31669110:33
*** spredzy has quit IRC10:36
openstackgerritSharat Sharma proposed openstack/puppet-designate: Added the options to be configured for designate-central service  https://review.openstack.org/31669110:51
openstackgerritSharat Sharma proposed openstack/puppet-designate: Added the options to be configured for designate-central service  https://review.openstack.org/31669110:52
*** dgurtner has quit IRC10:54
degorenkoEmilienM, ping :) hey11:01
*** markvoelker has quit IRC11:06
*** spredzy has joined #puppet-openstack11:07
*** _stowa has quit IRC11:13
*** jtomasek_ has joined #puppet-openstack11:14
*** johnmilton has joined #puppet-openstack11:26
*** dtantsur|brb is now known as dtantsur11:28
*** johnmilton has quit IRC11:34
*** jtomasek_ has quit IRC11:38
*** johnmilton has joined #puppet-openstack11:48
*** xiaohua has quit IRC11:54
*** dprince has joined #puppet-openstack11:59
*** danpawlik has quit IRC12:01
*** markvoelker has joined #puppet-openstack12:08
*** jtomasek_ has joined #puppet-openstack12:10
*** ducttape_ has joined #puppet-openstack12:11
*** jayg|g0n` is now known as jayg12:12
*** markvoelker has quit IRC12:13
*** saneax is now known as saneax_AFK12:14
*** paramite|afk is now known as paramite12:16
*** severion has joined #puppet-openstack12:21
*** v1k0d3n has quit IRC12:24
*** ducttape_ has quit IRC12:29
*** iurygregory has joined #puppet-openstack12:29
iurygregorymorning12:29
*** trown|brb is now known as trown12:31
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Adds the puppet-tripleo manifests for manila  https://review.openstack.org/31352712:39
*** morazi has joined #puppet-openstack12:46
EmilienMdegorenko: hey12:50
EmilienMwe need to deprecate verbose in all modules12:51
degorenkoEmilienM, well, i can find someone to help with it :)12:51
iurygregoryo/12:52
iurygregoryi can help =)12:53
EmilienMwe need to do like my patch in puppet-oslo12:54
EmilienMhttps://review.openstack.org/31652512:54
EmilienMbut also add a release note for all modules that already were released one day12:55
degorenkoEmilienM, ack12:55
EmilienMiurygregory: can you start with puppet-keystone please? I'll review it before we continue12:55
degorenkoEmilienM, iurygregory let's create a trello card for this12:55
degorenkoiurygregory, can you do this?12:55
EmilienMdegorenko: +112:55
iurygregorysure12:55
iurygregory=)12:55
degorenkoand please add all our modules with check list12:56
iurygregoryok12:56
degorenkothen we can simply check what's done and what's need to be prepared12:56
iurygregorymake sense =)12:56
degorenkothanks :)12:56
iurygregorynp guys =)12:56
degorenkoand let's use same topic12:56
degorenkoiurygregory, take it from EmilienM patch12:57
EmilienMiurygregory: I think most of the modules are using oslo::log. For those who won't, it's almost same patch.12:57
degorenkoyes12:57
EmilienMyeah, same logic everywhere.12:57
iurygregorynice to know =)12:57
degorenkoyou can simply check it :)12:57
EmilienMI did not add a release note in puppet-oslo because module was never released12:57
*** igajsin has joined #puppet-openstack12:57
degorenkoyeah, i saw12:57
EmilienMdegorenko: can you check why fuel CI is failing?12:57
degorenkoEmilienM, on which patch?12:57
*** igajsin has left #puppet-openstack12:58
degorenkoщрб щыдщ12:58
degorenkoops12:58
degorenkooslo patch, got it12:58
degorenkoEmilienM, failed all modules with oslo::log on tests expected that the catalogue would contain Aodh_config[DEFAULT/verbose] - it's ok, i'll file bug for this12:59
degorenkoand will upload fix12:59
degorenkoEmilienM, btw, fix for https://review.openstack.org/#/c/316334/4 is already prepared :)12:59
*** markvoelker has joined #puppet-openstack13:00
iurygregoryEmilienM, degorenko https://trello.com/c/C0mt7dKp/84-deprecate-verbose-option-in-logging-option-verbose-from-group-default-is-deprecated-for-removal i'm adding the modules right now13:03
degorenkoiurygregory, thanks :)13:04
*** markvoelker has quit IRC13:04
*** markvoelker has joined #puppet-openstack13:04
EmilienMdegorenko, iurygregory: good13:05
iurygregorydo we need puppet-pacemaker?13:06
*** NachoDuck has joined #puppet-openstack13:08
EmilienMwhat?13:09
EmilienMiurygregory: for verbose?13:09
iurygregoryyep13:09
EmilienMonly openstack services13:09
EmilienMthat use Oslo13:09
degorenkoright13:09
iurygregoryok13:09
iurygregoryi will start the work and after i'll update the list in trello13:10
*** degorenko is now known as _degorenko|afk13:10
openstackgerritEyal proposed openstack/puppet-vitrage: vitrage puppet installation  https://review.openstack.org/31644113:20
EmilienMiurygregory: thanks13:24
EmilienMmwhahaha, _degorenko|afk: please look https://review.openstack.org/#/c/316425/ when you can13:27
*** markvoelker has quit IRC13:28
iurygregoryEmilienM, i'm updating the examples too ok?13:34
EmilienMeverything13:34
EmilienMdrop verbose!13:34
iurygregoryverbose was in README lol i changed to debug =P13:35
EmilienM:)13:35
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Revert "Revert "allow to configure my_ip parameter""  https://review.openstack.org/31611213:38
iurygregoryLOL13:38
iurygregoryinception13:38
openstackgerritliaonanhai proposed openstack/puppet-manila: Add support for db_max_retries param  https://review.openstack.org/31529713:45
vinshEmilienM: Good marnin'!   I want to add the feature release not you suggest in 316270... is there a proper way to do this?  I don't know how to use the current release note structure in there.13:47
*** ducttape_ has joined #puppet-openstack13:49
iberezovskiyvinsh, http://docs.openstack.org/developer/reno/usage.html13:49
vinshah-hah. Thank you.13:50
*** chandankumar has quit IRC13:53
*** dfisher has joined #puppet-openstack13:54
EmilienMvinsh: thanks!14:00
*** _degorenko|afk is now known as degorenko14:00
vinsh@EmilienM I figure I will add the feature note in 316270.. and we can make sure 316080 is merged first to cover that as well.14:01
EmilienMvinsh: can I +A https://review.openstack.org/#/c/316080/ ?14:03
vinshYes14:03
EmilienMboom14:03
openstackgerritEyal proposed openstack/puppet-vitrage: vitrage puppet installation  https://review.openstack.org/31644114:03
degorenko\o/14:03
degorenko:)14:03
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Support xenial with swift-init service provider  https://review.openstack.org/31627014:08
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Deprecate verbose option in logging  https://review.openstack.org/31686714:09
iurygregoryEmilienM, ^ enjoy14:09
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Support xenial with swift-init service provider  https://review.openstack.org/31627014:09
EmilienMiurygregory: commented, we're almost there14:12
iurygregory\o/ nice14:13
openstackgerritMerged openstack/puppet-ceilometer: acceptance: remove scenario without apache  https://review.openstack.org/31642514:19
mjblackmorning14:20
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Composable role for RabbitMQ  https://review.openstack.org/31184914:21
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Deprecate verbose option in logging  https://review.openstack.org/31686714:23
*** markvoelker has joined #puppet-openstack14:25
EmilienMiurygregory: +2 thanks14:28
mjblackdegorenko: your comment on https://review.openstack.org/#/c/316439/1/manifests/init.pp, are we no longer have the comment about deprecated params?14:28
* mjblack needs more caffiene 14:29
mjblacknevermind14:29
EmilienMlol14:29
degorenko:)14:29
iurygregorytks EmilienM14:29
*** markvoelker has quit IRC14:30
degorenkoiurygregory, +2 from me too :)14:31
iurygregoryI can change the other modules tomorrow, after returning from my trip =) i'll not be able to attend the meeting14:31
degorenkoyou can do it when you wnat :)14:32
degorenkowant*14:32
EmilienMiurygregory: yeah, when you can14:32
EmilienMthe most important was in puppet-oslo14:32
EmilienMso we can promote our CI to latest master14:32
iurygregorynice =)14:32
EmilienMwe still need my patch to land in nova https://review.openstack.org/#/c/316610/14:33
degorenkoEmilienM, i've already prepared fix for library, waiting for CI and we will merge my and your changes14:33
EmilienMdegorenko: ack14:33
mjblackdegorenko: to address your comments, I'm going to fix up the depre sections and see if appropriate warnings had been added14:37
degorenkook, but you also can check it with horizon code probably14:37
mjblackdegorenko: I'm doing a bit more14:38
*** mdorman has joined #puppet-openstack14:39
degorenkook :)14:39
mjblackEmilienM, degorenko: so it looks like can_set_mount_point in ::horizon was deprecated some time ago14:39
mjblackdoesnt appear to do anything anymore14:40
mjblackI'm going to remove it14:40
*** dprince has quit IRC14:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: WIP: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/30853014:45
EmilienMdegorenko: rebased & updated ^14:45
EmilienMbut it will still fail14:45
EmilienMany progress on failures? libvirt, ssl, etc?14:45
degorenkoEmilienM, not yet, but i'll continue working on this today14:46
openstackgerritMatthew J Black proposed openstack/puppet-horizon: Updated deprecated section and removed parameter  https://review.openstack.org/31643914:46
mjblackdegorenko: ^ thats the update to your comment :P14:46
degorenkomjblack, ok :)14:46
degorenkomjblack, you missed my comments here: https://review.openstack.org/#/c/313621/214:47
EmilienMdegorenko: excellent thx14:47
*** hjensas has quit IRC14:47
mjblackdegorenko: I did but only because I was trying to get the ci testing to pass :D14:49
degorenkomjblack, ok ;) then fix them :D14:50
mjblackEmilienM: with regards to the multiline provider, you are ok with me changing the unit test to expect an array instead of a string?14:51
EmilienMmjblack: what is actually expected?14:51
mjblackEmilienM: thats a matter of perspective :D14:52
iurygregoryO.o14:52
mjblacka single element array is just the same as a string, so long as the single element matches :D14:52
mjblackat least in this instance14:52
iurygregoryif you let array you just need to ensure the length?14:53
mjblackiurygregory: the issue is that I'm switching, say neutron_config, from ini provider to ruby14:54
mjblackso we can do multiline keys14:54
iurygregoryok14:54
mjblackit treats every setting as having an array of values14:54
iurygregorydo you have a patch?14:55
mjblackyeah14:55
*** jaosorior has quit IRC14:55
mjblackiurygregory: https://review.openstack.org/#/c/31362114:55
iurygregorytks14:55
mjblackbut to further go into detail14:55
*** jaosorior has joined #puppet-openstack14:56
mjblackso every value will be an array14:56
mjblackeven if it only contains one element14:56
openstackgerritCody Herriges proposed openstack/puppet-neutron: Change MidoNet Plugin package name  https://review.openstack.org/29448614:56
iurygregoryto me make sense, i'll try to take a look tomorrow =) i need to study for a test XD14:58
openstackgerritCody Herriges proposed openstack/puppet-neutron: Change MidoNet Plugin package name  https://review.openstack.org/29448615:00
*** jaosorior has quit IRC15:01
*** jaosorior has joined #puppet-openstack15:02
mjblackiurygregory: heh, no worries I dont think its ready yet, it's failing beaker tests so I have to figure that out15:02
openstackgerritMerged openstack/puppet-swift: Do not use any specific service provider for ubuntu  https://review.openstack.org/31608015:03
openstackgerritMerged openstack/puppet-ceilometer: Add support for db_max_retries param  https://review.openstack.org/31412815:03
openstackgerritMerged openstack/puppet-ironic: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/31524215:03
*** ntpttr- has joined #puppet-openstack15:05
openstackgerritMatthew J Black proposed openstack/puppet-neutron: Consolidate service_providers into neutron server  https://review.openstack.org/31362115:06
vinshHave others hit this/is this known from memcached and xinetd modules? "Error: /Stage[main]/Memcached/Service[memcached]: Could not evaluate: private method `getpid' called for #<Puppet::Type::Service::ProviderDebian:0x00000003ba9f98>"15:08
*** haplo37 has joined #puppet-openstack15:11
*** xarses has joined #puppet-openstack15:15
EmilienMo_O15:15
vinshits seems like I hit it when I downgraded from puppet 4.4.2 to anything less15:17
vinshProbably some dependency/library crud mixed up.15:18
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Cisco nexus config manifest - obsolete parameter (switch_replay_count).  https://review.openstack.org/31691115:20
*** richm has joined #puppet-openstack15:24
mjblackvinsh: why did you downgrade from puppet 4.4.2?15:24
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: Add ensure_absent_val behavior to Nuage neutron plugin type  https://review.openstack.org/31691515:26
vinshmjblack: I'm having issues with 4.4.2 with using the file resource from a custom provider.  If no file is present.. it fails to write a new file claiming "Error: File written to disk did not match checksum; discarding changes (24 vs {md5}d41d8cd98f00b204e9800998ecf8427e)"15:27
vinshThis doesn't happen in the CI gate tests though.. I suspect it's something with the vm and my install to it.15:27
mjblackvinsh: hmm yeah you might have some left over libs that are getting in the way15:28
vinshWhere do you suggest I clean up?15:28
mjblackvinsh: how did you install puppet 4? gem?15:32
*** jmikolajczak has quit IRC15:33
vinshYes, gem15:34
vinshusing the bundle install from the puppet-swift git repo15:34
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Swift-ring-builder skip line 'Ring up-to-date..'  https://review.openstack.org/31627415:35
*** chandankumar has joined #puppet-openstack15:38
vinshxenial gate experimental tests failing due to permissions issue EmilienM.15:39
vinshknown issue?15:40
EmilienMshow logs, URL,15:40
vinshhttp://logs.openstack.org/70/316270/5/experimental/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-xenial/532502f/console.html15:40
EmilienMvinsh: can you give URL with the timestamp? It's really helpful15:40
vinshhttp://logs.openstack.org/70/316270/5/experimental/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-xenial/532502f/console.html#_2016-05-16_14_24_51_54315:40
vinshand all the lines following that15:41
mjblackdegorenko: to answer your comment, I dont think we do it that much to have seen it elsewhere15:41
degorenkomjblack, on which change ? :D15:41
mjblackonly other place I see this is with the keystone module15:41
mjblackdegorenko: the provider require statement15:41
mjblackin cinder_type15:41
EmilienMvinsh: this error is because tempest didn't run15:42
EmilienMvinsh: the actual error is not here15:42
EmilienMerror is here: http://logs.openstack.org/70/316270/5/experimental/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-xenial/532502f/console.html#_2016-05-16_14_24_50_56815:42
degorenkomjblack, well, we can ask _ody for this :)15:43
*** dmk0202 has quit IRC15:44
_odyo.O15:44
degorenko_ody, hey :)15:44
degorenkowhat do you think about https://review.openstack.org/#/c/316384/1 ?15:44
* _ody actually gets IRC notificaiton now adays...15:44
vinshEmilienM: oh well.. I guess temptest only runs sometimes then?15:44
EmilienMvinsh: tempest is run at the end of the job15:45
EmilienMvinsh: and your job don't even run puppet15:45
EmilienMdon't even install puppet15:45
*** rmart04 has quit IRC15:45
EmilienMso don't look at this error15:45
EmilienMit's a JJB publisher error, where jenkins is not able to copy tempest.conf because it does not exist15:45
EmilienMbecause puppet didnt create it15:46
vinshI was looking to see how my change worked on xenail.. I thought that was what the experimental jobs did15:46
vinshI don't really understand what you mean.15:47
_odydegorenko: Just like mjblack's reference to glance_image, I've seen this outside OpenStack too.  I'll hunt down the actual technical reasoning because I can't remember right now if it is all Ruby's fault or Puppet's code autoloader.15:47
EmilienMvinsh: you need https://review.openstack.org/#/c/308530/ to test xenial15:48
EmilienMbut I already update https://review.openstack.org/#/c/308530/ with your patch in Depends-On15:48
EmilienMand I already ran check experimental15:49
degorenko_ody, so, you are ok with this workaround?15:49
_odyyeah.15:49
EmilienMvinsh: ^15:49
degorenkook15:49
EmilienMvinsh: but jobs are still runnning15:49
mjblack_ody: I looked at keystone and that would need the same change15:49
mjblackjust to be consistent15:49
vinshOH OK.  +1 EmilienM15:49
vinsha lot going on there.15:49
_odymjblack: Ok.  I'll get a better answer to the why today.15:50
mjblack_ody: i think I found the puppet labs ticket pertaining to this15:54
mjblackhttps://review.openstack.org/#/c/316384/115:54
mjblackoops15:54
mjblackhttps://tickets.puppetlabs.com/browse/PUP-4450?focusedCommentId=241609&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-24160915:54
mjblackand https://tickets.puppetlabs.com/browse/PUP-4450?focusedCommentId=241595&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-24159515:56
mjblackdegorenko: ^ thats why15:58
EmilienMdegorenko, mwhahaha: could we fix Fuel CI for https://review.openstack.org/#/c/316241/ ?16:01
EmilienMit's also blocking us to bump, 2.0 api has been dropped16:01
degorenkomjblack, wow, thanks for info16:02
degorenkoEmilienM, i will look16:02
EmilienMthx16:02
mjblackdegorenko: after reading those two comments, it looks like why it's broken is because puppetlabs is trying to address the problem with code leaking between environments16:03
degorenkoyeah16:03
degorenkomjblack, there is also some workaround https://tickets.puppetlabs.com/browse/SERVER-571?focusedCommentId=166876&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-166876 but it is not production case :D16:04
openstackgerritMatt Fischer proposed openstack/puppet-openstack-integration: Rename metering_secret to telemetry_secret  https://review.openstack.org/31584816:05
mjblackdegorenko: lol well we can troll people and tell them to run masterless puppet to get around this too :D16:06
degorenkomjblack, indeed!16:07
degorenko:D16:07
mjblackdegorenko: so about that -1... :P16:07
degorenkomjblack,  well, we can't handle this config with our modules, it should be done at least on orchestration layer. You don't have what you can -1 :D16:09
* xarses yawns and stretches after a week of vacation =)16:10
mjblackdegorenko: uhh...well we can update the require statements to work with puppet 4 :D16:10
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: CI test - never merge  https://review.openstack.org/31697016:11
_odymjblack: Could you quickly look at what your ruby-load-path setting in /etc/puppetlabs/puppetserver/conf.d/puppet-server.conf is set to on the master you were seeing the load error on?16:13
*** foxhunt_away is now known as foxhunt7216:13
degorenko_ody, we already want to have it as workaround :D to change this conf i mean :)16:14
mjblack_ody: ruby-load-path: [/opt/puppetlabs/puppet/lib/ruby/vendor_ruby]16:14
_odyok.  That is congruent with my findings after looking through that ticket you linked.16:15
mjblackdegorenko: the problem with modifying the conf is it means that users will not be able to have puppet servers with a shared environment16:16
_odyBasically that autoloader had never been able to load code like we had it in the providers, it was just covered up by the fact that the master was using the same ruby load path as the agent.16:16
_odySo once an agent on the master ran and did a plugin sync, it all worked.16:16
_odyIn puppetserver they, by default do not share that same load path and so no longer polute each other.16:17
_odyIn PE I just noticed...16:17
_odyhttp://paste.openstack.org/show/497240/16:18
_odyThe last line is where the agent puts pluginsync code in puppet 4.16:18
*** spredzy has quit IRC16:19
_odyWithout changes to the way the autoloader works, the change in question is probably the "correct" way.16:20
_odyI need to hit the shower, still in my pajamas after an early morning meeting fell through.16:21
mjblack_ody: that might be how PE works but the community version is missing that load path line16:22
_odymjblack: Yeah.  That is why I agree that you patch is the better way to handle the issue.16:22
mjblackhonestly, my thoughts are to fix the providers instead of telling the user to futz with their config16:23
degorenkoabout changing config file - it was just joke :D16:24
*** spredzy has joined #puppet-openstack16:24
_odyBy referencing PE I was just validating why I am yet to see this issue in production either.16:24
*** agrebennikov has joined #puppet-openstack16:26
*** derekh has quit IRC16:28
*** evidex has quit IRC16:29
mattymoEmilienM, feeling ambitious today?16:29
mattymohttps://review.openstack.org/#/c/309103/16:29
degorenkoEmilienM, i've realized problem in Fuel CI here: https://review.openstack.org/#/c/316241/ - we are using v3 endpoint. That's why all jobs are red.16:31
* degorenko going to file bug for Fuel CI...again...sigh...16:31
mjblackdegorenko: heh I know but it is a possible solution to the issue :D16:31
degorenkomjblack, yes, it is solutionfor home's production not for production at all :)16:33
*** mgould has quit IRC16:37
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Fixed require statements for puppet 4  https://review.openstack.org/31698116:38
mjblack_ody,degorenko: ^16:41
*** trown is now known as trown|lunch16:45
openstackgerritMerged openstack/puppet-ceilometer: Correct service_credentials section  https://review.openstack.org/31633416:47
degorenkocrinkle, why you merged this https://review.openstack.org/#/c/316334/ ? :(16:48
degorenkowe were waiting for fix for fuel ci :( it is already on review...i mentioned in comments about this :(16:49
crinkledegorenko: oh, i'm sorry :(16:49
degorenkomwhahaha, can you please merge this https://review.openstack.org/#/c/316765/16:49
degorenkoit is already have +216:49
mwhahahak16:49
degorenkocrinkle, in this current case - that's not a big problem. but can you please do not merge changes if fuel ci is red and patch has some comments about this? :) or just do fuel: recheck16:51
crinkledegorenko: sure thing16:51
degorenkocrinkle, thanks :)16:52
openstackgerritMatthew J Black proposed openstack/puppet-openstacklib: Fixed require statements for puppet 4  https://review.openstack.org/31699016:53
*** mgould has joined #puppet-openstack16:57
*** jistr has quit IRC17:02
*** foxhunt72 is now known as foxhunt_away17:06
openstackgerritMatt Fischer proposed openstack/puppet-ceilometer: Correct service_credentials section  https://review.openstack.org/31633817:07
EmilienMmattymo: +A17:07
openstackgerritAndy Smith proposed openstack/puppet-ceilometer: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/31468817:07
degorenkomwhahaha, what do you think about: https://review.openstack.org/#/c/316999/ ?17:17
mwhahahai'd like to know the backstory on how it's now v2.1 which is to be used instead of v3? :o17:18
dmsimardEmilienM: so fwiw I'm trying out stable/mitaka17:19
dmsimardwith fernet enable17:19
dmsimardand I'm getting those permission denied errors I mentioned earlier17:19
degorenkoEmilienM, ^ can you answer on mwhahaha question?17:19
mwhahahai'm sure it's some sort of upstream decision, but version numbers mean nothing i guess17:19
EmilienMmwhahaha, degorenko: ask nova folks17:20
EmilienMbut they told me that17:20
degorenko:D nice17:20
mwhahahasome what related, we need to get the ec2api stuff in a good spot17:21
degorenkoec2api module is not ready yet17:21
mwhahahayea i know, that's kinda problem17:22
EmilienMmwhahaha: are we too slow?17:23
mwhahahawell the issue is that we're removing functionality with no equivalent way to configure the thing that replaces it17:24
openstackgerritMerged openstack/puppet-cinder: Fixed cinder_type provider require line  https://review.openstack.org/31638417:28
openstackgerritMatthew J Black proposed openstack/puppet-cinder: Fixed cinder_type provider require line  https://review.openstack.org/31701617:28
mjblack^ cherry-pick to stable/mitaka17:29
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Composable role for RabbitMQ  https://review.openstack.org/31184917:29
degorenkomwhahaha, cherry-pick for mitaka for mfisch patch https://review.openstack.org/31700417:29
degorenkoso many fixes17:30
mfischtyvm17:30
EmilienMeod for me, back later tonight probably17:33
iurygregoryeod = end of day ?17:34
degorenkolooks like :)17:35
iurygregorytks XD17:36
*** trown|lunch is now known as trown17:46
*** rcernin has quit IRC17:47
*** dhollinger has joined #puppet-openstack17:54
*** dtantsur is now known as dtantsur|afk18:06
*** delattec has joined #puppet-openstack18:09
*** mgould has quit IRC18:10
*** cdelatte has quit IRC18:12
*** v1k0d3n has joined #puppet-openstack18:14
*** delatte has joined #puppet-openstack18:14
*** delattec has quit IRC18:17
*** jtomasek has quit IRC18:19
*** shardy has quit IRC18:24
vinshdegorenko: I was close to a regex then went with quick n'easy ;p18:25
degorenkovinsh, well, is not obvious :018:26
degorenko:)18:26
degorenkoit is*18:26
*** chandankumar has quit IRC18:26
*** foxhunt_away is now known as foxhunt7218:39
*** mattymo` has joined #puppet-openstack18:43
mattymo`EmilienM: https://review.openstack.org/#/c/309103/ can you remove WF+1 and repeat? It got stuck in gate jobs18:43
degorenkomattymo`, it has -2 from gate jobs :)18:46
degorenkoalready rechecked18:46
*** degorenko is now known as _degorenko|afk18:48
*** v1k0d3n has quit IRC18:54
*** v1k0d3n has joined #puppet-openstack18:55
*** morazi has quit IRC18:55
*** johnmilton has quit IRC19:05
iurygregorymwhahaha, if you have time can you look at https://review.openstack.org/#/c/316867/ problem with fuel noop, i'm doing recheck because scenario02/03 didn't pass Build timed out19:05
mwhahahaiurygregory: i think we've got a change to remove the verbose option in fuel-library for that change19:06
mwhahahait's cause we're checking for it19:06
iurygregorymwhahaha, oh tks =)19:07
mwhahahaoh wait no that failure is cause of the change that get merged earlier around the auth stuff19:07
iurygregorywow19:07
*** xb_ has quit IRC19:08
mwhahahahttps://review.openstack.org/#/c/316765/19:08
mwhahahashould be ok now19:08
iurygregoryok =) i'll do recheck because scenarios cant finish i think "Build timed out" =(19:08
mwhahahak19:08
*** foxhunt72 is now known as foxhunt_away19:09
openstackgerritMerged openstack/puppet-swift: Support xenial with swift-init service provider  https://review.openstack.org/31627019:22
*** imcsk8 is now known as imcsk8|afk19:40
*** rcernin has joined #puppet-openstack19:42
*** mattymo` has quit IRC19:54
*** jaosorior has quit IRC19:56
openstackgerritAlex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache.  https://review.openstack.org/29734220:06
*** paramite has quit IRC20:09
*** mjblack has quit IRC20:13
*** dmk0202 has joined #puppet-openstack20:14
EmilienMmhh I'm seeing random sahara failures in stable/mitaka http://logs.openstack.org/15/316915/1/check/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-trusty/e52f329/console.html#_2016-05-16_19_39_30_87120:14
openstackgerritAlex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache.  https://review.openstack.org/29734220:15
EmilienMhttp://logs.openstack.org/15/316915/1/check/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-trusty/e52f329/logs/sahara/sahara-api.txt.gz#_2016-05-16_19_34_56_96620:16
*** mjblack has joined #puppet-openstack20:18
EmilienMmwhahaha: can we approve https://review.openstack.org/#/c/316911/ please?20:19
*** imcsk8|afk is now known as imcsk820:19
mwhahahak20:19
EmilienMthanks20:20
*** jayg is now known as jayg|g0n320:25
EmilienMcolonwq: do you have progress on puppet-ceph / jewel?20:25
EmilienMmwhahaha: are we still waiting on something to land https://review.openstack.org/#/c/316525/ ?20:28
mwhahahayea the library fix20:29
mwhahahasec let me look20:29
EmilienMkk20:29
EmilienMit's on my list to promote to recent newton20:29
EmilienMthey dropped verbose, we need this patch20:29
*** hjensas has joined #puppet-openstack20:30
*** hjensas has joined #puppet-openstack20:30
*** iurygregory has quit IRC20:30
*** ericksonsantos has quit IRC20:30
EmilienMoh and same question for https://review.openstack.org/#/c/316241/20:32
*** jmikolajczak has joined #puppet-openstack20:33
*** iurygregory has joined #puppet-openstack20:34
*** ericksonsantos has joined #puppet-openstack20:34
iurygregorymwhahaha, recheck didnt help  https://ci.fuel-infra.org/job/master.puppet-openstack.fuel.noop/1094/ : FAILURE20:40
mwhahahawell now it's only 5 failures, thought it was 8 prior20:41
mwhahahalet me look20:41
iurygregoryahsuahsuash each recheck removed 1 maybe XD20:41
mwhahahaEmilienM: 316241 has a fuel patch, that won't likely land today20:41
*** ericksonsantos has quit IRC20:41
mwhahaha316241 is more breaking than the verbose thing20:41
EmilienMso can I go sleep?20:42
mwhahahayea20:42
iurygregoryof course you can20:42
EmilienMon behalf of my bed: "thank you"20:42
*** ericksonsantos has joined #puppet-openstack20:42
mwhahahai'll merge the verbose thing once the  fuel patch lands (currently waiting on gate)20:43
mwhahahai'll try and get the other fuel patch in a spot to  land tomorrow europe time20:43
iurygregorygood luck =)20:43
EmilienMthanks20:44
*** rmart04 has joined #puppet-openstack20:44
*** rmart04 has quit IRC20:46
*** jmikolajczak has quit IRC20:58
openstackgerritMerged openstack/puppet-tripleo: Remove Nova EC2 HAproxy endpoint  https://review.openstack.org/31636121:00
openstackgerritCody Herriges proposed openstack/puppet-openstack_extras: Backport to reflect the move of Kilo to CentOS SIG  https://review.openstack.org/31708621:02
*** trown is now known as trown|outtypewww21:03
*** haplo37 has quit IRC21:11
mwhahahaargh it looks like noop is broken, seems like we missed some other spots21:14
iurygregory:o21:15
iurygregory+1 from jenkis \o/ https://review.openstack.org/#/c/316867/ just the problem with noop =)21:18
mwhahahawhich is unrelated to that change21:18
mwhahahaso i'll just merge it21:18
iurygregorytks mwhahaha =D21:18
mwhahaha_degorenko|afk: https://review.openstack.org/317096 you missed the ceilometer controller task :/21:24
openstackgerritCody Herriges proposed openstack/puppet-openstack_extras: Backport to reflect the move of Kilo to CentOS SIG  https://review.openstack.org/31708621:52
*** markvoelker has joined #puppet-openstack21:57
*** markvoelker has quit IRC22:01
*** markvoelker has joined #puppet-openstack22:01
*** mdorman has quit IRC22:08
*** dmk0202 has quit IRC22:10
*** dhollinger has quit IRC22:11
mwhahahait seems nothing wants to merge today22:18
*** ducttape_ has quit IRC22:22
*** xarses has quit IRC22:39
*** johnmilton has joined #puppet-openstack22:41
*** johnmilton has quit IRC22:45
openstackgerritMerged openstack/puppet-ceilometer: metering_secret was renamed to telemetry_secret  https://review.openstack.org/31584622:50
*** johnmilton has joined #puppet-openstack22:58
*** markvoelker has quit IRC23:06
*** rcernin has quit IRC23:08
*** dfisher has quit IRC23:10
*** saneax_AFK is now known as saneax23:15
*** johnmilton has quit IRC23:21
colonwqEmilienM, Yes. There needs to be rgw.pp changes. 14.04 requires a diff set of settings. See: http://docs.ceph.com/docs/master/radosgw/config/#add-a-gateway-configuration-to-ceph23:31
colonwqEmilienM, I am locally testing the changes.23:32
*** xarses has joined #puppet-openstack23:39
*** mjblack has quit IRC23:39
*** mjblack has joined #puppet-openstack23:43
« Sunday, 2016-05-15 Index Tuesday, 2016-05-17 »

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