Wednesday, 2015-12-09

*** jerrygb has quit IRC00:01
*** baker has quit IRC00:03
*** zhangjn has joined #puppet-openstack00:05
*** zhangjn has quit IRC00:06
*** zhangjn has joined #puppet-openstack00:06
*** dfisher has quit IRC00:11
*** nlahouti has quit IRC00:13
*** psanchez has quit IRC00:21
*** migi has quit IRC00:22
*** migi has joined #puppet-openstack00:23
*** psanchez has joined #puppet-openstack00:23
*** zhangjn has quit IRC00:24
*** toMeloos has quit IRC00:26
*** aimon has quit IRC00:36
*** ducttape_ has joined #puppet-openstack00:38
*** RichardRaseley has quit IRC00:45
*** openstackstatus has quit IRC00:50
*** openstack has joined #puppet-openstack00:54
*** zhangjn has joined #puppet-openstack00:55
*** aimon has joined #puppet-openstack01:03
*** zhangjn has quit IRC01:04
*** rahulait has quit IRC01:06
*** zhangjn has joined #puppet-openstack01:06
*** xarses has joined #puppet-openstack01:14
*** xarses has quit IRC01:14
*** xarses has joined #puppet-openstack01:15
*** xarses has quit IRC01:21
*** ducttape_ has joined #puppet-openstack01:23
*** saneax is now known as saneax_AFK01:27
*** zhangjn_ has joined #puppet-openstack01:36
*** zhangjn has quit IRC01:40
*** ducttape_ has quit IRC01:44
*** ferest has joined #puppet-openstack01:47
*** ducttape_ has joined #puppet-openstack01:47
*** jesusaurus has joined #puppet-openstack01:50
*** ferest has quit IRC01:53
*** xarses has joined #puppet-openstack02:04
*** ferest has joined #puppet-openstack02:12
*** ferest has quit IRC02:12
*** ferest has joined #puppet-openstack02:13
*** ducttape_ has quit IRC02:16
*** ducttape_ has joined #puppet-openstack02:17
*** ducttape_ has quit IRC02:18
*** ferest has quit IRC02:22
*** ducttape_ has joined #puppet-openstack02:23
*** xarses has quit IRC02:37
*** xarses has joined #puppet-openstack02:37
*** ducttape_ has quit IRC02:43
*** xarses_ has joined #puppet-openstack02:44
*** ducttape_ has joined #puppet-openstack02:46
*** xarses has quit IRC02:47
*** baker has joined #puppet-openstack02:56
*** ducttape_ has quit IRC02:57
*** jerrygb has joined #puppet-openstack02:57
*** ducttape_ has joined #puppet-openstack03:01
*** jerrygb has quit IRC03:02
*** ducttape_ has quit IRC03:09
*** ducttape_ has joined #puppet-openstack03:14
*** vpati011_ has joined #puppet-openstack03:15
*** morazi has quit IRC03:17
*** jerrygb has joined #puppet-openstack03:18
*** vpati011_ has quit IRC03:28
*** saneax_AFK is now known as saneax03:37
*** ducttape_ has quit IRC03:42
*** jerrygb has quit IRC03:48
*** jerrygb has joined #puppet-openstack03:48
*** jerrygb has quit IRC03:52
*** zhangjn_ has quit IRC03:58
*** jerrygb has joined #puppet-openstack04:03
*** saneax is now known as saneax_AFK04:05
*** mattymo has quit IRC04:05
*** mattymo has joined #puppet-openstack04:11
*** vpati011_ has joined #puppet-openstack04:13
*** vpati011_ has quit IRC04:22
*** zhangjn has joined #puppet-openstack04:29
*** baker has quit IRC04:30
*** jerrygb has quit IRC04:49
*** jerrygb has joined #puppet-openstack04:51
*** jerrygb has quit IRC04:51
*** jerrygb has joined #puppet-openstack04:51
*** zhangjn has quit IRC04:52
*** zhangjn has joined #puppet-openstack05:15
*** saneax_AFK is now known as saneax05:18
*** zhangjn has quit IRC05:28
*** zhangjn has joined #puppet-openstack05:28
*** jerrygb has quit IRC06:17
*** jerrygb has joined #puppet-openstack06:17
*** jerrygb has quit IRC06:21
*** logan- has quit IRC06:24
*** gildub has quit IRC06:53
*** jerrygb has joined #puppet-openstack07:00
*** jtomasek has joined #puppet-openstack07:03
*** chem has joined #puppet-openstack07:04
*** gildub has joined #puppet-openstack07:06
*** jtomasek has quit IRC07:06
*** jerrygb has quit IRC07:06
*** jaosorior has joined #puppet-openstack07:20
*** rcernin has joined #puppet-openstack07:20
*** jtomasek has joined #puppet-openstack07:21
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Remove all 'validate_array' statements  https://review.openstack.org/25076307:35
*** zhangjn has quit IRC07:36
*** chem has quit IRC07:42
*** chem has joined #puppet-openstack07:58
*** zhangjn has joined #puppet-openstack08:00
*** sleinen-AS559 has joined #puppet-openstack08:14
*** foxhunt_away is now known as foxhunt7208:16
*** toMeloos has joined #puppet-openstack08:18
*** sleinen-AS559 has quit IRC08:19
*** chem has quit IRC08:23
*** michchap_ has quit IRC08:32
*** michchap has joined #puppet-openstack08:33
*** chem has joined #puppet-openstack08:34
*** dgurtner has joined #puppet-openstack08:34
*** dgurtner has quit IRC08:34
*** dgurtner has joined #puppet-openstack08:34
*** chem has quit IRC08:36
*** paramite has joined #puppet-openstack08:36
*** jpena has joined #puppet-openstack08:39
*** chem has joined #puppet-openstack08:40
*** dgurtner has quit IRC08:44
*** dgurtner has joined #puppet-openstack08:44
*** dgurtner has quit IRC08:44
*** dgurtner has joined #puppet-openstack08:44
*** gfidente has joined #puppet-openstack08:52
*** gfidente has quit IRC08:52
*** gfidente has joined #puppet-openstack08:52
*** jaosorior has quit IRC08:55
*** karimb has joined #puppet-openstack08:58
*** skolekonov has quit IRC08:58
*** chem has left #puppet-openstack09:02
*** jerrygb has joined #puppet-openstack09:02
*** _degorenko|afk is now known as degorenko09:04
*** openstackstatus has quit IRC09:05
*** openstack has joined #puppet-openstack09:09
openstackgerritJavier Peña proposed openstack/puppet-manila: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24572109:15
*** skolekonov has joined #puppet-openstack09:15
*** rochaporto has joined #puppet-openstack09:15
openstackgerritGael Chamoulaud proposed openstack/puppet-trove: Put all the logging related parameters to the logging class  https://review.openstack.org/24239509:16
*** jaosorior has joined #puppet-openstack09:16
*** karimb_ has joined #puppet-openstack09:27
*** karimb has quit IRC09:30
*** IvanBerezovskiy has joined #puppet-openstack09:31
*** openstackgerrit has quit IRC09:32
*** openstackgerrit has joined #puppet-openstack09:33
openstackgerritMykyta Karpin proposed openstack/puppet-glance: add more settings to glance api, registry, rabbitmq  https://review.openstack.org/23703509:40
*** zhangjn has quit IRC09:45
*** davideagle has joined #puppet-openstack09:48
openstackgerritMykyta Karpin proposed openstack/puppet-glance: add more settings to glance api, registry, rabbitmq  https://review.openstack.org/23703509:51
openstackgerritGael Chamoulaud proposed openstack/puppet-gnocchi: Switch gnocchi to use os_service_default fact  https://review.openstack.org/24003409:57
*** zhangjn has joined #puppet-openstack09:57
*** openstackgerrit has quit IRC10:02
*** openstackgerrit has joined #puppet-openstack10:03
*** jpena has quit IRC10:08
*** jpena has joined #puppet-openstack10:08
openstackgerritMartin Mágr proposed openstack/puppet-keystone: Preparation for default endpoint names rename  https://review.openstack.org/23479810:24
*** chem has joined #puppet-openstack10:29
*** davideagle has quit IRC10:31
*** jistr has joined #puppet-openstack10:39
*** davideagle has joined #puppet-openstack10:42
*** jerrygb has joined #puppet-openstack11:04
*** jerrygb has quit IRC11:09
*** zhangjn has quit IRC11:23
openstackgerritGael Chamoulaud proposed openstack/puppet-gnocchi: Fix ::gnocchi::statsd class parameters documentation  https://review.openstack.org/25521211:33
*** saneax is now known as saneax_AFK11:36
*** saneax_AFK is now known as saneax11:39
*** mgrosz has joined #puppet-openstack11:49
*** zhangjn has joined #puppet-openstack11:49
mgroszHello.... Can I am trying to initiate puppet-vitrage module based on https://wiki.openstack.org/wiki/Puppet/New_module. There is a bootstarp.sh script that leads me to non-existed directory11:50
mgroszThe generated puppet-vitrage repo is placed under /tmp/puppet-vitrage/cookiecutter/puppet-vitrage. I would like to execute git review but I don't understand in what directories do I need to do so.11:52
mgroszThere is this puppet-modulesync-configs repo. Do I need also to git review it?11:52
mgroszDoes anyone can please send me to some docs that describe the steps?11:53
*** cdelatte has quit IRC12:05
*** mgrosz has quit IRC12:09
EmilienMgood morning folks!12:19
mkarpinHi!12:19
*** davideagle has quit IRC12:20
*** gildub has quit IRC12:34
openstackgerritClayton O'Neill proposed openstack/puppet-nova: Fix typo in hooks patch  https://review.openstack.org/25523512:37
claytonEmilienM, degorenko: we need to get that patch merged ^^^12:40
claytonfixes a typo in the hooks patch that breaks the ceph backend12:40
*** ducttape_ has joined #puppet-openstack12:40
*** jaosorior has quit IRC12:42
*** jaosorior has joined #puppet-openstack12:42
EmilienMclayton: ok12:46
EmilienMclayton: done12:46
degorenkoclayton, why tests didn't catched it?12:46
EmilienMwe need a CI with ceph!12:46
EmilienMdegorenko: we don't test RBD12:47
EmilienMit's weird puppet syntax did not catch it12:47
degorenkoEmilienM, we don't have spec tests forthis class?12:47
EmilienMwe do have12:47
*** cdelatte has joined #puppet-openstack12:48
degorenkoEmilienM, clayton this change wasn't cover in spec tests :( https://github.com/openstack/puppet-nova/blob/master/spec/classes/nova_compute_rbd_spec.rb#L8612:49
EmilienMdegorenko: :)12:49
degorenkothat's why ci passed12:50
*** jerrygb has joined #puppet-openstack12:53
*** davideagle has joined #puppet-openstack12:58
*** ducttape_ has quit IRC12:58
*** dprince has joined #puppet-openstack13:01
*** morazi has joined #puppet-openstack13:08
*** jayg|g0n3 is now known as jayg13:14
*** vpati011_ has joined #puppet-openstack13:15
openstackgerritMartin Mágr proposed openstack/puppet-glance: Preparation for default endpoint names rename  https://review.openstack.org/23479113:24
*** vpati011_ has quit IRC13:26
openstackgerritDenis Egorenko proposed openstack/puppet-ironic: Fix typo in CHANGELOG file  https://review.openstack.org/25525513:27
degorenkoEmilienM, clayton ^^^13:27
*** saneax is now known as saneax_AFK13:28
openstackgerritMartin Mágr proposed openstack/puppet-glance: Preparation for default endpoint names rename  https://review.openstack.org/23479113:29
*** karimb_ has quit IRC13:29
EmilienMdegorenko: +2 +A13:29
degorenko:)13:30
EmilienMdegorenko: we need reno.  lol13:30
degorenkoEmilienM, can i help you somehow?13:30
EmilienMdegorenko: https://review.openstack.org/254924 is all I haev13:31
degorenkoEmilienM, not much :)13:32
EmilienMI'm so boring13:33
openstackgerritMerged openstack/puppet-nova: Fix typo in hooks patch  https://review.openstack.org/25523513:33
degorenkoEmilienM, do you have some short list of thing which should be prepared?13:34
EmilienMdegorenko: could you look cody's patches about puppet4? I'm even not sure it's ready yet13:37
EmilienMI was interested by having a multi node CI also but I haven't taken time to work on it13:37
*** psanchez has quit IRC13:37
EmilienMif we haev Fuel + tripleo jobs running that's good enough IMHO13:37
openstackgerritMartin Mágr proposed openstack/puppet-nova: Preparation for default endpoint names rename  https://review.openstack.org/22212013:38
degorenkoEmilienM, we have fuel weekly meeting today. I remember about Fuel jobs for upstream :)13:39
EmilienMdegorenko: nice, I'm looking forward to seeing it13:39
degorenko_ody, hey, do you have patches about puppet4? Can you paste links here, if you have, of course :)13:40
degorenkoEmilienM, i will ping you of course13:40
EmilienMdegorenko: nice, ping me anytime13:40
EmilienMdegorenko: for _ody, look at his gerrit13:41
degorenkoEmilienM, two patches, right?13:42
degorenkohttps://review.openstack.org/#/c/248316/13:42
*** sleinen-AS559 has joined #puppet-openstack13:44
EmilienMdegorenko: I haven't checked13:45
*** sleinen-AS5591 has joined #puppet-openstack13:46
*** sleinen-AS559 has quit IRC13:49
*** ducttape_ has joined #puppet-openstack13:54
*** chem has quit IRC13:55
openstackgerritMerged openstack/puppet-ironic: Fix typo in CHANGELOG file  https://review.openstack.org/25525513:55
*** baker has joined #puppet-openstack13:57
*** jpena is now known as jpena|lunch13:57
*** karimb has joined #puppet-openstack13:58
*** logan- has joined #puppet-openstack14:01
*** sleinen-AS5591 has quit IRC14:07
*** ducttape_ has quit IRC14:08
*** jerrygb_ has joined #puppet-openstack14:08
*** jerrygb has quit IRC14:11
*** chem has joined #puppet-openstack14:28
*** bklei has joined #puppet-openstack14:28
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Fix service and package tags  https://review.openstack.org/25528314:30
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Run db sync only from main script  https://review.openstack.org/25480914:33
EmilienMdegorenko: I would love seeing tests for murano on https://review.openstack.org/#/c/250436/14:33
mkarpindegorenko,EmilienM, i resolved all comments to  https://review.openstack.org/#/c/237035/ and https://review.openstack.org/#/c/249711/ can you please take a look? thank you14:33
EmilienMmkarpin: I'll, today14:34
degorenkomkarpin, will look :)14:34
mkarpintnx!14:34
degorenkoaderyugin, please fix merge conflict https://review.openstack.org/#/c/250436/14:34
EmilienMVinsh: jistr told me your swift patch broke OOO CI14:35
EmilienMI'll let jistr explain :-)14:35
VinshOh boy14:35
*** slagle has joined #puppet-openstack14:35
jistrVinsh: haha, hello :)14:35
VinshHi :) how can I help?14:36
EmilienMVinsh: YOU BREAK THE WORLD14:36
EmilienMagain.14:36
VinshListen, I write pure golden code.14:36
EmilienMyeah of course14:36
*** davideagle has quit IRC14:36
EmilienMlook https://github.com/openstack/puppet-swift/commit/5a7d18975f891c628cdfce798c5c426046203fcb14:36
slaglehttps://review.openstack.org/#/c/203220/ is breaking tripleo14:36
jistrVinsh: the service provider patch seems to prevent us from doing this https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/manifests/overcloud_controller_pacemaker.pp#L806-L82014:37
*** mgrosz has joined #puppet-openstack14:37
EmilienMin  manifests/storage/server.pp14:37
jistrVinsh: as in storage/server.pp the swift::storage::${type} changed from include to resource-style14:37
VinshYes. I struggled with that part. I couldn't figure out how to pass in anything to an include14:38
Vinshso I went for resource-style14:38
EmilienMVinsh: gold, uh?14:39
EmilienMthe only alternative I see is passing all parameters14:39
EmilienMbut that's ugly14:39
mgrosz@EmilienM: Did you have a chance to look on my question regarding init of the puppet-vitrage module?14:40
jistrEmilienM, Vinsh: that's what i kinda have in the works. Not all params, but adding service_ensure and enable to traverse the hierarchy14:40
EmilienMmgrosz: yes but I was sleeping, I'm EST14:40
EmilienMmgrosz: spredzy and sbadia created the tools, maybe they can hel14:40
jistrVinsh: that would solve tripleo's problem, but not the general problem of not being able to override other params anymore14:40
EmilienMhelp*14:40
mgroszAre they online?14:40
EmilienMmgrosz: though last time I used it out of the box14:40
EmilienMspredzy, sbadia: hey, do you have 5 min to help on bootstraping a new module?14:41
mgrosz@EmilienM: thanks.14:41
Vinshjistr: Not being able to override other params where exactly?14:41
jistrVinsh: on ::swift::storage::* classes14:41
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: scenario002: enable application catalog service  https://review.openstack.org/25043614:42
jistrVinsh: one could use hiera in theory, but we cannot because https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/manifests/overcloud_controller_pacemaker.pp#L3814:42
jistr($step doesn't come from hiera yaml files, it's fed by Heat)14:42
mgroszspredzy, sbadia: Are you online?14:43
jistror more precisely, it does come via hiera but isn't static, it changes throughout the deployment, so we need that condition there14:43
openstackgerritVishal Patil proposed openstack/puppet-neutron: Fix dependencies for Nuage plugin configuration  https://review.openstack.org/24723914:44
*** vpati011_ has joined #puppet-openstack14:44
sbadiamgrosz: no :-p14:45
sbadiahi here14:45
sbadiamgrosz: sorry :-) can I help?14:45
mgrosz@sbadia: I used the instructions here https://wiki.openstack.org/wiki/Puppet/New_module to create a new module puppet-vitrage. I don't understand in what directories I need to execute "git review"... the script sends me to a directory…14:48
mgroszto a non existed directory14:48
*** davideagle has joined #puppet-openstack14:48
openstackgerritJiri Stransky proposed openstack/puppet-swift: Allow setting manage_service and enable again on storage services  https://review.openstack.org/25529514:49
jistrVinsh: do you think something like this could work? ^^14:49
Vinshjistr: How about if: we move swift::storage::${type} back to how it was and just require an operator specify the provider type direcly to object/account/container in their manifest?14:49
sbadiamgrosz: can you paste the execution log? (on paste.openstack.org)14:49
mgrosz@sbadia: can I sent it to you by email?14:50
jistrVinsh: yeah that would be great, can you submit that?14:50
EmilienMthat would work14:50
jistrVinsh: i think it's better than what i just posted, because what i posted is focused specifically on TripleO problem14:50
sbadiamgrosz: shure, (my nickname @redhat.com)14:50
jistrthere might be different deployments which want to set different params than tripleo14:50
EmilienMjistr: can you run a tripleo job with dependson your patch in p-swift?14:50
VinshOk, thanks for bringing this up.  I'll fire that patch up right now.14:50
claytonhas anyone else seen issues with puppet-keystone and the _member_ role not getting created?14:50
jistrEmilienM: i think what Vinsh suggests is better though14:51
claytonit appears that keystone (at least on master) only creates the role when adding a user with a default tenant14:51
claytonand we recently removed all the default tenant code14:51
EmilienMclayton: I would poke chem -- he worked a lot on this module lately14:51
Vinshjistr: Yes. and allowing the operator to set it per storage type provides the feature I wanted while not breaking an existing use case.14:51
jistrVinsh: cool, thanks!14:51
*** ntpttr has joined #puppet-openstack14:51
EmilienMjistr: +1 with Vinsh gold's proposal14:51
Vinsh\0_o/14:52
chemclayton: this code has been removed some time ago14:52
claytonyeah, but I'm updating to the latest and having issues :)14:52
mgrosz@sbadia: I sent you by email.14:52
chemclayton: I believe there was a like a depreciation warning.  You have to explicitely use the user_role resource to create membership.  The _member_ creation was just an artifact of the previous openstack cli not being able to create an user without a role14:53
*** ducttape_ has joined #puppet-openstack14:54
claytonyou're saying _member_ is not necessary anymore?14:54
chemclayton: yes14:54
claytonhow do I add a user to a project then?14:54
chemclayton: user_role resource14:54
claytonsure, which depends on the _member_ role existing, right?14:55
chemclayton: keystone_user_role14:55
chemclayton: no14:55
claytoncan you point me at an example?14:55
chemclayton: yep 2s14:55
*** xarses_ has quit IRC14:57
chemclayton: https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L51-L5414:57
sbadiamgrosz: ok I take a look, thx!14:57
claytonchem: and what does that look like if I dont' want the user to be an admin?14:57
chemclayton: keystone_user_role { 'foo@bar': ensure => present, role ['blah', 'bli']}14:58
claytonI feel like I'm missing something14:58
claytonI just want them to be a member of the project, how do I do that if it's not adding _member_ ?14:58
*** dfisher has joined #puppet-openstack14:59
chemclayton: will create add the existing role blah, bli to user foo in domain Default for project bar in domain Default14:59
openstackgerritGael Chamoulaud proposed openstack/puppet-gnocchi: Switch gnocchi to use os_service_default fact  https://review.openstack.org/24003415:00
*** jpena|lunch is now known as jpena15:00
chemclayton: I would say that you can create a role with keystone_role and associate it using this role15:01
claytonso you're saying having any role name on the project is the same as _member_ ?15:04
chemclayton: it depends on your platform15:05
chemclayton: http://docs.openstack.org/user-guide-admin/manage_projects_users_and_roles.html15:05
chemclayton: it boils down to what you have defined policy.json i would say15:06
claytonit appears that at least some of the _member_ code is hard coded inside of keystone15:06
claytonwell, _member_ isn't, but member_role_id and member_role_name15:06
claytonand _member_ is the default value for member_role_name15:06
chemclayton: ack, so if it exists already, no need to create and if you define it won't be re-created.15:07
claytonmy point is that _member_ used to be automatically created and it appears that it no longer is15:07
claytonspecifically this removed code that used to automatically add _member_ if it didn't exist - https://review.openstack.org/#/c/228744/15:08
claytonand that code was merged in sept15:08
*** cdelatte has quit IRC15:09
chemclayton: yes.  As I said, it not required anymore for user creation as it used to be and the readme make it clear why it has been removed, no ?15:09
*** cdelatte has joined #puppet-openstack15:10
jistrVinsh: i reported the bug so that we can track it and so that i can refer to it when pinning TripleO to an older puppet-swift version temporarily https://bugs.launchpad.net/tripleo/+bug/152438115:10
openstackLaunchpad bug 1524381 in tripleo "Duplicate declaration when defining swift::storage::* outside swift::storage::server" [Critical,Triaged] - Assigned to Jiří Stránský (jistr)15:10
richm1right - keystone upstream wants you to explicitly create roles instead because the _member_ role is deprecated15:10
*** richm1 is now known as richm15:10
Vinshjistr: Got it. I'll add that to my commit message. review up shortly15:10
claytonI'm missing the note about _member_ being deprecated and I don't know what the replacement is15:10
jistrVinsh: perfect, thanks!15:11
richmclayton: the replacement is that you create your own role15:11
richmclayton: so you can just do keystone_role {"_member_"} if you need _membeR_15:11
richm_member_15:11
claytonso there is still a need for a role that serves that function, it just may not be named _member_15:12
claytonis that correct?15:12
*** delattec has joined #puppet-openstack15:12
richmclayton: _member_ is mostly meaningless - it is much better to define a real role that is meaningful to your organization15:12
claytonmeaningful how?15:12
richm_member_ was a hack to allow you to add users to tenants/projects without having to first define a role15:12
*** xarses_ has joined #puppet-openstack15:13
richmI'm not sure what "meaningful" means - but what does "_member_" mean?  What does it imply that at user can do within the tenant/project/domain where it has that role?15:14
richms/that at/that a/15:14
*** cdelatte has quit IRC15:15
degorenkoEmilienM, ping :)15:17
EmilienMdegorenko: yo15:18
claytonrichm: what determines if they can get a token for a given project?  Having any role at all on the project?15:18
*** derekh has joined #puppet-openstack15:18
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Use include for swift::storage::${type}  https://review.openstack.org/25531515:18
degorenkoEmilienM, so, i've asked our guys about jobs for our modules, and actually work is already started. But progress is slow. I will push our devops and other fuel guys, who is working at this :)15:19
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Use include for swift::storage::${type}  https://review.openstack.org/25531515:19
EmilienMdegorenko: thanks for the followup!15:20
EmilienMdegorenko: let me know if there is something we can help with15:20
degorenkoEmilienM, sure :)15:20
*** dgurtner has quit IRC15:21
richmclayton: yes15:21
claytonok15:22
claytonI think I got it now.  Thanks for bearing with me.15:23
openstackgerritAdam Vinsh proposed openstack/puppet-swift: Use include for swift::storage::${type}  https://review.openstack.org/25531515:25
claytonI'd still argue this is a backwards incompat change that's not documented15:26
claytonbut everyone else here would probably say that if it bothers me enough I'll put up a patch for the release notes ;)15:28
openstackgerritDan Prince proposed openstack/puppet-mistral: Add mistral::engine class  https://review.openstack.org/24841215:28
openstackgerritDan Prince proposed openstack/puppet-mistral: Add mistral::executor class  https://review.openstack.org/24841315:29
openstackgerritDan Prince proposed openstack/puppet-mistral: Add mistral::api class  https://review.openstack.org/24841015:29
openstackgerritDan Prince proposed openstack/puppet-mistral: Update mistral::db::sync to run 'update head'  https://review.openstack.org/24840415:29
*** davideagle has quit IRC15:30
*** dprince has quit IRC15:33
*** aimon has quit IRC15:34
EmilienMclayton: sorry for that, I missed that information :(15:34
claytonwell, it's hidden pretty well, I'm not sure it's really intentional that it worked at all :)15:36
richmayoung can explain this much better than I can15:37
*** imcsk8 is now known as imcsk8|gym15:44
*** paramite has quit IRC15:45
claytonI think I've got it now15:45
degorenkoEmilienM, can you please review: https://review.openstack.org/#/q/topic:fix_db_sync,n,z15:47
EmilienMdegorenko: no15:47
degorenkoi find this problem during fixing fuel bug :)15:47
degorenko:'( oh15:47
EmilienMdegorenko: :)15:48
EmilienMoh wait15:48
EmilienMdegorenko: why do you import ::murano?15:48
EmilienMin api.pp15:48
degorenkoEmilienM, in case of using api as stand alone service15:48
degorenkoor it shoud be done on higher layer?15:49
EmilienMyes15:49
EmilienMlike in other modules15:49
degorenkook, i will update it then15:49
degorenkooops :D15:49
EmilienM:)15:49
degorenkothen i should fix it for sahara too15:49
EmilienM+2 on the other one15:49
degorenkofor sahara it was used because of broken package15:49
*** vpati011_ has quit IRC15:50
degorenkobut now packages for sahara are ok, so it should be removed, you're right15:50
EmilienMok15:50
*** zhangjn has quit IRC15:52
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Run db sync only from main script  https://review.openstack.org/25480915:56
*** dhollinger has joined #puppet-openstack16:01
colonwqEmilienM, Can  you help with this patch: https://review.openstack.org/#/c/254548/ I pass all but two tests and have documented the reason the centos test fails.16:01
*** aimon has joined #puppet-openstack16:02
EmilienMcolonwq: I'm not the best guy about puppet-ceph, I don't know their CI status16:03
EmilienMxarses_ or dmsimard know though16:04
degorenkodmsimard, ^^^16:04
colonwqEmilienM, thanks. Trying to get my first patch in.16:05
openstackgerritDenis Egorenko proposed openstack/puppet-murano: Run db sync only from main script  https://review.openstack.org/25480916:05
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add hiera calls to allow management of ports and protocols  https://review.openstack.org/24020816:08
dmsimardcolonwq: heya16:16
dmsimardcolonwq: yeah, CI is a bit of a pain right now. We're trying to fix it but we don't have as much time as we'd like to spend on it.16:16
dmsimardThere's a thread about it on openstack-dev here http://lists.openstack.org/pipermail/openstack-dev/2015-November/080275.html16:17
dmsimardthe thread forked here: http://lists.openstack.org/pipermail/openstack-dev/2015-December/081066.html16:18
*** jerrygb_ has quit IRC16:18
colonwqdmsimard, thanks for the info.16:27
colonwqdmsimard, How does the patch I sumitted look?16:28
dmsimardcolonwq: I added myself as a reviewer, I'll have a look when I have a chance16:29
colonwqdmsimard, great thanks.16:29
dmsimardI'm not familiar with civetweb but I'll give an opinion on the code16:29
colonwqIt is built into the radosgw binary.16:30
dmsimardcolonwq: yeah but I've always run it with Apache :(16:32
*** foxhunt72 is now known as foxhunt_away16:32
*** jaosorior has quit IRC16:32
colonwqdmsimard, I can find the ceph version civetweb was add. Surprisingly it 'just works' without extra packages (apache, fcgi)16:34
colonwqI can change the structure of the code as needed.16:35
*** toMeloos has quit IRC16:36
colonwqdmsimard, firefly added civetweb support. https://ceph.com/planet/replace-apache-by-civetweb-on-the-radosgw/16:41
*** xarses_ is now known as Xarses16:51
XarsesEmilienM: colonwq at first look there appears to be a deps issue in the epel repo http://logs.openstack.org/95/254095/1/check/gate-puppet-ceph-puppet-beaker-rspec-dsvm-centos7/ac9f226/console.html#_2015-12-08_05_29_33_22516:52
EmilienMdmsimard can help here16:53
colonwqXarses, right. I added a coment about that. It is a centos 7.1 isue16:53
colonwqissue16:53
Xarseswe'll have to figure out how to deal with it in order to merge anything16:54
XarsesEmilienM: dmsimard is there someone you can talk to about the missing dep ?16:56
colonwqXarses, I manually added the libunwind package before running beaker locally. I do not know how jenkins can be handled16:56
colonwq7.2 should fix it.16:56
VinshEmilienM: https://review.openstack.org/#/c/255315/2  tripleo swift fix ready to go.  jistr16:56
EmilienMVinsh: gold or silver?16:56
VinshPLATINUM!16:56
EmilienMlol16:56
Vinshlimited time only.16:56
Xarsescolonwq: we can force the package to install if there is a usable version on the interweb somewhere16:57
EmilienMjistr, slagle: can I test https://review.openstack.org/#/c/255315/ in tripleo ? or did you pin puppet-swift?16:57
*** aimon has quit IRC16:57
jistrVinsh: lgtm, thanks16:57
Xarsescolonwq: is 7.2 out yet?16:57
colonwqXarses, it is not out yet...e ventually .. sometime.16:57
colonwqXarses, here is where I installed libunwind from : https://wiki.centos.org/AdditionalResources/Repositories/CR16:57
jistrEmilienM: we have a patch for the pin, waiting for CI https://review.openstack.org/#/c/255314/1  and we also have a patch to test the fix https://review.openstack.org/#/c/255320/16:58
dmsimardXarses: I can ask around for help16:58
jistr(as it only breaks the HA job which doesn't run on puppet-swift patches)16:58
dmsimardXarses: (For that broken dep)16:58
Xarsesdmsimard: should we just enable this CR repo? or grab the missing package from it and run?16:59
EmilienMjistr: don't you have to change the interface in Hiera?16:59
dmsimardXarses: as a temporary workaround ?16:59
Xarsesdmsimard: yes16:59
dmsimardworth a try17:00
jistrEmilienM: not sure what you mean? IIUC the fix would preserve the original behavior17:00
EmilienMjistr: good17:00
Xarsesdmsimard: I'll whip something up17:00
*** imcsk8|gym is now known as imcsk817:01
Vinshjistr: You could opt-in to using the swiftinit provider :)17:01
jistrVinsh: if that's the recommended thing to use going forward, then we'll probably do so17:02
EmilienM++17:03
VinshI'll work with you on an update for tripleo for that in the future.17:03
jistrok17:03
*** yottatsa has joined #puppet-openstack17:04
dmsimardXarses: ok, meanwhile I poked the mothership to see if anyone can help17:06
*** yottatsa has quit IRC17:08
*** dprince has joined #puppet-openstack17:10
*** yottatsa has joined #puppet-openstack17:10
*** psanchez has joined #puppet-openstack17:11
*** rcernin has quit IRC17:11
colonwqEmilienM, I fixed those issues. Too many years doing perl and C. Thanks.17:13
*** jistr has quit IRC17:19
dmsimardXarses, colonwq: looks like the broken dependency is a known issue https://bugzilla.redhat.com/show_bug.cgi?id=128907317:26
openstackbugzilla.redhat.com bug 1289073 in libunwind "Cannot install nginx from epel" [High,New] - Assigned to tcallawa17:26
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Remove require on main init class  https://review.openstack.org/25539317:27
*** degorenko is now known as _degorenko|afk17:27
colonwqdmsimard, That reads pretty much like what I found on a mailing list.17:28
* Xarses rolls his eys17:29
Xarsesyay, fun with package deps17:29
dmsimardlooks like that broke a couple things, nginx .. ceph, probably others too.17:30
*** colonwq is now known as colonwq_afk17:36
*** aimon has joined #puppet-openstack17:37
ntpttrwould someone mind taking a look at what degorenko and I were talking about on logging variables here https://review.openstack.org/#/c/253216/6/manifests/logging.pp ? wondering what the correct approach is17:39
Xarseserm, vagrant gives me a ssh error as soon as the node comes up, but I can `vagrant ssh` into the node just fine17:42
*** nlahouti has joined #puppet-openstack17:43
*** karimb has quit IRC17:45
*** rcola has joined #puppet-openstack17:46
*** rochaporto has quit IRC17:54
Xarsesahh, I broke it17:57
Xarsesmakes sense17:57
*** yottatsa has quit IRC17:58
openstackgerritNate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/25321618:00
*** nlahouti has quit IRC18:03
*** nlahouti has joined #puppet-openstack18:05
*** imcsk8 has quit IRC18:11
*** imcsk8 has joined #puppet-openstack18:11
*** markvoelker has quit IRC18:18
*** yottatsa has joined #puppet-openstack18:20
*** markvoelker has joined #puppet-openstack18:21
Xarsescolonwq_afk: so while I'18:23
Xarsescolonwq_afk: so while I'm trying to fix up the gate, I've gone over your review18:24
openstackgerritAndrew Woodward proposed openstack/puppet-ceph: Work around missing libunwind dep (CentOS)  https://review.openstack.org/25542118:24
Xarsesdmsimard: colonwq_afk ^18:24
*** ducttape_ has quit IRC18:26
*** yottatsa_ has joined #puppet-openstack18:28
*** yottatsa has quit IRC18:30
*** yottatsa has joined #puppet-openstack18:32
*** yottatsa_ has quit IRC18:32
*** yottatsa has quit IRC18:33
openstackgerritMaty Grosz proposed openstack/puppet-vitrage: puppet-vitrage: Initial commit  https://review.openstack.org/25543118:33
mgroszHey, please review puppet-vitrage: Initial commit  https://review.openstack.org/255431. Thanks18:38
rcolaHi, I am trying to set up Keystone with the OpenStack/Keystone puppet module.  I get this error "Error: Could not prefetch keystone_tenant provider 'openstack' ..."18:47
chemrcola: could be a lot of things.  Could you paste somewhere (http://paste.openstack.org/) the complete error log ?18:48
rcolathanks chem I'll do that18:49
*** richm has quit IRC18:49
*** ducttape_ has joined #puppet-openstack18:50
*** jpena is now known as jpena|away18:51
rcolaHere's is my paste ... http://paste.openstack.org/show/481360/18:53
mgrosz@EmilienM: online?18:54
chemrcola: don't forget to ping me, if not, I'm not looking :)18:55
rcolaok thanks chem18:55
*** ducttape_ has quit IRC19:00
*** ducttape_ has joined #puppet-openstack19:01
*** dprince has quit IRC19:03
chemrcola: well, you should remake you manifest to look more like this: https://github.com/openstack/puppet-keystone/blob/stable/kilo/spec/acceptance/keystone_wsgi_apache_spec.rb#L20-L7119:04
chemrcola: you shoudn't need to setup all that apache stuff yourself and I believe they may be some missing requirement (puppet's require), as I don't them being executed in the log.19:06
*** mdorman has joined #puppet-openstack19:07
chemrcola: the code there works and will avoid you to have those "exec" as well (which are lacking the dependencies as well).19:08
chemrcola: hope it helps19:08
rcolachem: thanks I'll try that out19:09
mdormanhey guys i’m struggling a little bit with the latest keystone module.  it’s using v3 api, but the default domain info isn’t getting set in the credentials object, so some calls to keystone with openstackclient are failing19:10
*** derekh has quit IRC19:10
mdormani’m having trouble understanding where the domain config info is supposed to come from.  so i need an openrc file?19:11
mgrosz@EmilienM: please review puppet-vitrage: Initial commit  https://review.openstack.org/255431. Thanks19:12
*** gfidente is now known as gfidente|afk19:15
dmsimardXarses: it looks like this might be fixed (when it propagates through the mirrors) https://bugzilla.redhat.com/show_bug.cgi?id=1289073#c1419:17
openstackbugzilla.redhat.com bug 1289073 in libunwind "Cannot install nginx from epel" [High,New] - Assigned to tcallawa19:17
*** mgrosz has quit IRC19:18
ntpttrmdorman: I'vve run into similar issues, and I did need to create an openrc file to fix them personally19:28
ntpttrmdorman: There could be other ways around it but I'm not sure what they are19:28
mdorman@ntpttr   thanks.   i’m going down another path right now, i may just do that.19:29
mdormank19:29
*** _ducttape_ has joined #puppet-openstack19:34
*** ducttape_ has quit IRC19:38
EmilienMmdorman: you don't need openrc afik19:55
EmilienMmdorman: can you give more details of what is failing? Any example of manifest?19:56
*** dprince has joined #puppet-openstack19:58
ntpttrEmilienM: A little bit ago I tried deploying Keystone using this manifest http://paste.openstack.org/show/481371/ and it worked for the actual keystone installation but when it tried to create the admin role and the keystone endpoint it failed because it was using openstack-client commands that required v3 I think, so after I sourced an openrc file the commands worked20:00
*** davideagle has joined #puppet-openstack20:00
ntpttrEmilienM: I don't have the output of the failure any more, but it was something about incorrect options being passed in to the openstack commands20:01
EmilienMmdorman, ntpttr: file a bug I would recommend20:03
ntpttrEmilienM: sure20:04
dmsimardcolonwq_afk: it looks like the libunwind issue is fixed in centos upstream, when you submit your new patchset we should no longer encounter that error20:07
*** rmsilva has joined #puppet-openstack20:09
ntpttrEmilienM: It seems something has changed in the last two weeks, because I just tried to recreate the error and it worked20:16
*** gildub has joined #puppet-openstack20:16
*** davideagle has quit IRC20:17
EmilienMntpttr: cool! the bug is fixed - NEXT20:17
openstackgerritDan Prince proposed openstack/puppet-ironic: Add ironic::keystone::auth_inspector  https://review.openstack.org/25349320:20
nibalizerEmilienM: you sent me a link to review and I lost it, can I have it again?20:22
EmilienMnibalizer: https://review.openstack.org/25493320:22
EmilienMcrinkle: should look also20:22
*** colonwq_afk is now known as colonwq20:22
colonwqXarses, thanks for the review and the updated spec file.20:23
openstackgerritNate Potter proposed openstack/puppet-keystone: Remove references to puppet-openstack from README  https://review.openstack.org/24838220:33
*** karimb has joined #puppet-openstack20:34
openstackgerritNate Potter proposed openstack/puppet-keystone: Remove references to puppet-openstack from README  https://review.openstack.org/24838220:34
*** RichardRaseley has joined #puppet-openstack20:35
*** derekh has joined #puppet-openstack20:45
*** derekh has quit IRC20:45
mdormanEmilienM: ntpttr:  yeah i’m still working through it on my end.  i’ll definitely post a bug/review if that turns out to be it.  but it’s most likely some dumb thing on my end20:52
*** nlahouti has quit IRC20:53
*** rmsilva has quit IRC20:55
*** rmsilva has joined #puppet-openstack20:55
*** rcola has quit IRC20:58
*** bklei_ has joined #puppet-openstack20:58
*** jayg is now known as jayg|g0n321:00
*** bklei has quit IRC21:02
*** bklei_ has quit IRC21:15
*** bklei has joined #puppet-openstack21:15
*** dprince has quit IRC21:19
*** _ducttape_ has quit IRC21:19
*** ducttape_ has joined #puppet-openstack21:19
openstackgerritNate Potter proposed openstack/puppet-nova: Fix scopre warnings in nova::vncproxy::common  https://review.openstack.org/25552921:21
openstackgerritNate Potter proposed openstack/puppet-nova: Fix scope warnings in nova::vncproxy::common  https://review.openstack.org/25552921:28
*** jpena|away has quit IRC21:34
*** trown|BOS is now known as trown|outttypeww21:35
*** richm1 has joined #puppet-openstack21:49
*** cwolferh has quit IRC22:03
*** rmsilva has quit IRC22:04
*** nlahouti has joined #puppet-openstack22:08
*** richm1 has quit IRC22:09
*** karimb has quit IRC22:27
*** bklei has quit IRC22:28
*** bklei has joined #puppet-openstack22:29
*** bklei has quit IRC22:33
*** aglarendil has quit IRC22:36
*** patrickeast has quit IRC22:40
openstackgerritNate Potter proposed openstack/puppet-openstack-cookiecutter: Replace ceilometer with correct project name  https://review.openstack.org/25557522:40
openstackgerritNate Potter proposed openstack/puppet-openstack-cookiecutter: Fix manifests::db $database_connection variable  https://review.openstack.org/25557722:48
*** ducttape_ has quit IRC22:59
*** morazi has quit IRC23:00
*** baker has quit IRC23:03
*** baker has joined #puppet-openstack23:06
*** baker has quit IRC23:09
*** dhollinger has quit IRC23:15
*** saneax_AFK is now known as saneax23:24
*** toMeloos has joined #puppet-openstack23:29
*** mdorman has quit IRC23:34
*** ntpttr has left #puppet-openstack23:34
*** rcernin has joined #puppet-openstack23:37
*** cwolferh has joined #puppet-openstack23:38
*** gfidente|afk is now known as gfidente23:42
*** aglarendil has joined #puppet-openstack23:47
*** toMeloos has quit IRC23:50
*** patrickeast has joined #puppet-openstack23:51
*** bklei has joined #puppet-openstack23:53
Xarsesdmsimard: ok, so gitbuilder.ceph.com isn't building fastcgi any more so thats what's wrong with ubuntu23:53

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