Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:05 zhangjn joined #puppet-openstack
00:06 zhangjn joined #puppet-openstack
00:23 migi joined #puppet-openstack
00:23 psanchez joined #puppet-openstack
00:38 ducttape_ joined #puppet-openstack
00:53 openstackstatus joined #puppet-openstack
00:54 openstack joined #puppet-openstack
00:55 zhangjn joined #puppet-openstack
01:03 aimon joined #puppet-openstack
01:06 zhangjn joined #puppet-openstack
01:14 xarses joined #puppet-openstack
01:15 xarses joined #puppet-openstack
01:23 ducttape_ joined #puppet-openstack
01:36 zhangjn_ joined #puppet-openstack
01:47 ferest joined #puppet-openstack
01:47 ducttape_ joined #puppet-openstack
01:50 jesusaurus joined #puppet-openstack
02:04 xarses joined #puppet-openstack
02:12 ferest joined #puppet-openstack
02:13 ferest joined #puppet-openstack
02:17 ducttape_ joined #puppet-openstack
02:23 ducttape_ joined #puppet-openstack
02:37 xarses joined #puppet-openstack
02:45 xarses_ joined #puppet-openstack
02:46 ducttape_ joined #puppet-openstack
02:56 baker joined #puppet-openstack
02:57 jerrygb joined #puppet-openstack
03:01 ducttape_ joined #puppet-openstack
03:14 ducttape_ joined #puppet-openstack
03:15 vpati011_ joined #puppet-openstack
03:18 jerrygb joined #puppet-openstack
03:48 jerrygb joined #puppet-openstack
04:03 jerrygb joined #puppet-openstack
04:11 mattymo joined #puppet-openstack
04:13 vpati011_ joined #puppet-openstack
04:29 zhangjn joined #puppet-openstack
04:51 jerrygb joined #puppet-openstack
04:51 jerrygb joined #puppet-openstack
05:15 zhangjn joined #puppet-openstack
05:28 zhangjn joined #puppet-openstack
06:17 jerrygb joined #puppet-openstack
07:00 jerrygb joined #puppet-openstack
07:03 jtomasek joined #puppet-openstack
07:04 chem joined #puppet-openstack
07:06 gildub joined #puppet-openstack
07:20 jaosorior joined #puppet-openstack
07:20 rcernin joined #puppet-openstack
07:21 jtomasek joined #puppet-openstack
07:35 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: Remove all 'validate_array' statements  https://review.openstack.org/250763
07:58 chem joined #puppet-openstack
08:00 zhangjn joined #puppet-openstack
08:14 sleinen-AS559 joined #puppet-openstack
08:18 toMeloos joined #puppet-openstack
08:33 michchap joined #puppet-openstack
08:34 chem joined #puppet-openstack
08:34 dgurtner joined #puppet-openstack
08:34 dgurtner joined #puppet-openstack
08:36 paramite joined #puppet-openstack
08:39 jpena joined #puppet-openstack
08:40 chem joined #puppet-openstack
08:44 dgurtner joined #puppet-openstack
08:44 dgurtner joined #puppet-openstack
08:52 gfidente joined #puppet-openstack
08:52 gfidente joined #puppet-openstack
08:58 karimb joined #puppet-openstack
09:02 chem left #puppet-openstack
09:02 jerrygb joined #puppet-openstack
09:08 openstackstatus joined #puppet-openstack
09:09 openstack joined #puppet-openstack
09:15 openstackgerrit Javier Peña proposed openstack/puppet-manila: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/245721
09:15 skolekonov joined #puppet-openstack
09:15 rochaporto joined #puppet-openstack
09:16 openstackgerrit Gael Chamoulaud proposed openstack/puppet-trove: Put all the logging related parameters to the logging class  https://review.openstack.org/242395
09:16 jaosorior joined #puppet-openstack
09:27 karimb_ joined #puppet-openstack
09:31 IvanBerezovskiy joined #puppet-openstack
09:33 openstackgerrit joined #puppet-openstack
09:40 openstackgerrit Mykyta Karpin proposed openstack/puppet-glance: add more settings to glance api, registry, rabbitmq  https://review.openstack.org/237035
09:48 davideagle joined #puppet-openstack
09:51 openstackgerrit Mykyta Karpin proposed openstack/puppet-glance: add more settings to glance api, registry, rabbitmq  https://review.openstack.org/237035
09:57 openstackgerrit Gael Chamoulaud proposed openstack/puppet-gnocchi: Switch gnocchi to use os_service_default fact  https://review.openstack.org/240034
09:57 zhangjn joined #puppet-openstack
10:03 openstackgerrit joined #puppet-openstack
10:08 jpena joined #puppet-openstack
10:24 openstackgerrit Martin Mágr proposed openstack/puppet-keystone: Preparation for default endpoint names rename  https://review.openstack.org/234798
10:29 chem joined #puppet-openstack
10:39 jistr joined #puppet-openstack
10:42 davideagle joined #puppet-openstack
11:04 jerrygb joined #puppet-openstack
11:33 openstackgerrit Gael Chamoulaud proposed openstack/puppet-gnocchi: Fix ::gnocchi::statsd class parameters documentation  https://review.openstack.org/255212
11:49 mgrosz joined #puppet-openstack
11:49 zhangjn joined #puppet-openstack
11:50 mgrosz Hello.... 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 directory
11:52 mgrosz The 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 mgrosz There is this puppet-modulesync-configs repo. Do I need also to git review it?
11:53 mgrosz Does anyone can please send me to some docs that describe the steps?
12:19 EmilienM good morning folks!
12:19 mkarpin Hi!
12:37 openstackgerrit Clayton O'Neill proposed openstack/puppet-nova: Fix typo in hooks patch  https://review.openstack.org/255235
12:40 clayton EmilienM, degorenko: we need to get that patch merged ^^^
12:40 clayton fixes a typo in the hooks patch that breaks the ceph backend
12:40 ducttape_ joined #puppet-openstack
12:42 jaosorior joined #puppet-openstack
12:46 EmilienM clayton: ok
12:46 EmilienM clayton: done
12:46 degorenko clayton, why tests didn't catched it?
12:46 EmilienM we need a CI with ceph!
12:47 EmilienM degorenko: we don't test RBD
12:47 EmilienM it's weird puppet syntax did not catch it
12:47 degorenko EmilienM, we don't have spec tests forthis class?
12:47 EmilienM we do have
12:48 cdelatte joined #puppet-openstack
12:49 degorenko EmilienM, clayton this change wasn't cover in spec tests :( https://github.com/openstack/puppet-nova/blob/master/spec/classes/nova_compute_rbd_spec.rb#L86
12:49 EmilienM degorenko: :)
12:50 degorenko that's why ci passed
12:53 jerrygb joined #puppet-openstack
12:58 davideagle joined #puppet-openstack
13:01 dprince joined #puppet-openstack
13:08 morazi joined #puppet-openstack
13:15 vpati011_ joined #puppet-openstack
13:24 openstackgerrit Martin Mágr proposed openstack/puppet-glance: Preparation for default endpoint names rename  https://review.openstack.org/234791
13:27 openstackgerrit Denis Egorenko proposed openstack/puppet-ironic: Fix typo in CHANGELOG file  https://review.openstack.org/255255
13:27 degorenko EmilienM, clayton ^^^
13:29 openstackgerrit Martin Mágr proposed openstack/puppet-glance: Preparation for default endpoint names rename  https://review.openstack.org/234791
13:29 EmilienM degorenko: +2 +A
13:30 degorenko :)
13:30 EmilienM degorenko: we need reno.  lol
13:30 degorenko EmilienM, can i help you somehow?
13:31 EmilienM degorenko: https://review.openstack.org/254924 is all I haev
13:32 degorenko EmilienM, not much :)
13:33 EmilienM I'm so boring
13:33 openstackgerrit Merged openstack/puppet-nova: Fix typo in hooks patch  https://review.openstack.org/255235
13:34 degorenko EmilienM, do you have some short list of thing which should be prepared?
13:37 EmilienM degorenko: could you look cody's patches about puppet4? I'm even not sure it's ready yet
13:37 EmilienM I was interested by having a multi node CI also but I haven't taken time to work on it
13:37 EmilienM if we haev Fuel + tripleo jobs running that's good enough IMHO
13:38 openstackgerrit Martin Mágr proposed openstack/puppet-nova: Preparation for default endpoint names rename  https://review.openstack.org/222120
13:39 degorenko EmilienM, we have fuel weekly meeting today. I remember about Fuel jobs for upstream :)
13:39 EmilienM degorenko: nice, I'm looking forward to seeing it
13:40 degorenko _ody, hey, do you have patches about puppet4? Can you paste links here, if you have, of course :)
13:40 degorenko EmilienM, i will ping you of course
13:40 EmilienM degorenko: nice, ping me anytime
13:40 EmilienM degorenko: for _ody, look at his gerrit
13:42 degorenko EmilienM, two patches, right?
13:42 degorenko https://review.openstack.org/#/c/248316/
13:44 sleinen-AS559 joined #puppet-openstack
13:45 EmilienM degorenko: I haven't checked
13:46 sleinen-AS5591 joined #puppet-openstack
13:54 ducttape_ joined #puppet-openstack
13:55 openstackgerrit Merged openstack/puppet-ironic: Fix typo in CHANGELOG file  https://review.openstack.org/255255
13:57 baker joined #puppet-openstack
13:58 karimb joined #puppet-openstack
14:01 logan- joined #puppet-openstack
14:08 jerrygb_ joined #puppet-openstack
14:28 chem joined #puppet-openstack
14:28 bklei joined #puppet-openstack
14:30 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Fix service and package tags  https://review.openstack.org/255283
14:33 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Run db sync only from main script  https://review.openstack.org/254809
14:33 EmilienM degorenko: I would love seeing tests for murano on https://review.openstack.org/#/c/250436/
14:33 mkarpin degorenko,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 you
14:34 EmilienM mkarpin: I'll, today
14:34 degorenko mkarpin, will look :)
14:34 mkarpin tnx!
14:34 degorenko aderyugin, please fix merge conflict https://review.openstack.org/#/c/250436/
14:35 EmilienM Vinsh: jistr told me your swift patch broke OOO CI
14:35 EmilienM I'll let jistr explain :-)
14:35 Vinsh Oh boy
14:35 slagle joined #puppet-openstack
14:35 jistr Vinsh: haha, hello :)
14:36 Vinsh Hi :) how can I help?
14:36 EmilienM Vinsh: YOU BREAK THE WORLD
14:36 EmilienM again.
14:36 Vinsh Listen, I write pure golden code.
14:36 EmilienM yeah of course
14:36 EmilienM look https://github.com/openstack/puppet-swift/commit/5a7d18975f891c628cdfce798c5c426046203fcb
14:36 slagle https://review.openstack.org/#/c/203220/ is breaking tripleo
14:37 jistr Vinsh: 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-L820
14:37 mgrosz joined #puppet-openstack
14:37 EmilienM in  manifests/storage/server.pp
14:37 jistr Vinsh: as in storage/server.pp the swift::storage::${type} changed from include to resource-style
14:38 Vinsh Yes. I struggled with that part. I couldn't figure out how to pass in anything to an include
14:38 Vinsh so I went for resource-style
14:39 EmilienM Vinsh: gold, uh?
14:39 EmilienM the only alternative I see is passing all parameters
14:39 EmilienM but that's ugly
14:40 mgrosz @EmilienM: Did you have a chance to look on my question regarding init of the puppet-vitrage module?
14:40 jistr EmilienM, Vinsh: that's what i kinda have in the works. Not all params, but adding service_ensure and enable to traverse the hierarchy
14:40 EmilienM mgrosz: yes but I was sleeping, I'm EST
14:40 EmilienM mgrosz: spredzy and sbadia created the tools, maybe they can hel
14:40 jistr Vinsh: that would solve tripleo's problem, but not the general problem of not being able to override other params anymore
14:40 EmilienM help*
14:40 mgrosz Are they online?
14:40 EmilienM mgrosz: though last time I used it out of the box
14:41 EmilienM spredzy, sbadia: hey, do you have 5 min to help on bootstraping a new module?
14:41 mgrosz @EmilienM: thanks.
14:41 Vinsh jistr: Not being able to override other params where exactly?
14:41 jistr Vinsh: on ::swift::storage::* classes
14:42 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: scenario002: enable application catalog service  https://review.openstack.org/250436
14:42 jistr Vinsh: 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#L38
14:42 jistr ($step doesn't come from hiera yaml files, it's fed by Heat)
14:43 mgrosz spredzy, sbadia: Are you online?
14:43 jistr or more precisely, it does come via hiera but isn't static, it changes throughout the deployment, so we need that condition there
14:44 openstackgerrit Vishal Patil proposed openstack/puppet-neutron: Fix dependencies for Nuage plugin configuration  https://review.openstack.org/247239
14:44 vpati011_ joined #puppet-openstack
14:45 sbadia mgrosz: no :-p
14:45 sbadia hi here
14:45 sbadia mgrosz: sorry :-) can I help?
14:48 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 mgrosz to a non existed directory
14:48 davideagle joined #puppet-openstack
14:49 openstackgerrit Jiri Stransky proposed openstack/puppet-swift: Allow setting manage_service and enable again on storage services  https://review.openstack.org/255295
14:49 jistr Vinsh: do you think something like this could work? ^^
14:49 Vinsh jistr: 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 sbadia mgrosz: can you paste the execution log? (on paste.openstack.org)
14:50 mgrosz @sbadia: can I sent it to you by email?
14:50 jistr Vinsh: yeah that would be great, can you submit that?
14:50 EmilienM that would work
14:50 jistr Vinsh: i think it's better than what i just posted, because what i posted is focused specifically on TripleO problem
14:50 sbadia mgrosz: shure, (my nickname @redhat.com)
14:50 jistr there might be different deployments which want to set different params than tripleo
14:50 EmilienM jistr: can you run a tripleo job with dependson your patch in p-swift?
14:50 Vinsh Ok, thanks for bringing this up.  I'll fire that patch up right now.
14:50 clayton has anyone else seen issues with puppet-keystone and the _member_ role not getting created?
14:51 jistr EmilienM: i think what Vinsh suggests is better though
14:51 clayton it appears that keystone (at least on master) only creates the role when adding a user with a default tenant
14:51 clayton and we recently removed all the default tenant code
14:51 EmilienM clayton: I would poke chem -- he worked a lot on this module lately
14:51 Vinsh jistr: 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 jistr Vinsh: cool, thanks!
14:51 ntpttr joined #puppet-openstack
14:51 EmilienM jistr: +1 with Vinsh gold's proposal
14:52 Vinsh \0_o/
14:52 chem clayton: this code has been removed some time ago
14:52 clayton yeah, but I'm updating to the latest and having issues :)
14:52 mgrosz @sbadia: I sent you by email.
14:53 chem clayton: 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 role
14:54 ducttape_ joined #puppet-openstack
14:54 clayton you're saying _member_ is not necessary anymore?
14:54 chem clayton: yes
14:54 clayton how do I add a user to a project then?
14:54 chem clayton: user_role resource
14:55 clayton sure, which depends on the _member_ role existing, right?
14:55 chem clayton: keystone_user_role
14:55 chem clayton: no
14:55 clayton can you point me at an example?
14:55 chem clayton: yep 2s
14:57 chem clayton: https://github.com/openstack/puppet-keystone/blob/master/spec/acceptance/keystone_wsgi_apache_spec.rb#L51-L54
14:57 sbadia mgrosz: ok I take a look, thx!
14:57 clayton chem: and what does that look like if I dont' want the user to be an admin?
14:58 chem clayton: keystone_user_role { 'foo@bar': ensure => present, role ['blah', 'bli']}
14:58 clayton I feel like I'm missing something
14:58 clayton I just want them to be a member of the project, how do I do that if it's not adding _member_ ?
14:59 dfisher joined #puppet-openstack
14:59 chem clayton: will create add the existing role blah, bli to user foo in domain Default for project bar in domain Default
15:00 openstackgerrit Gael Chamoulaud proposed openstack/puppet-gnocchi: Switch gnocchi to use os_service_default fact  https://review.openstack.org/240034
15:01 chem clayton: I would say that you can create a role with keystone_role and associate it using this role
15:04 clayton so you're saying having any role name on the project is the same as _member_ ?
15:05 chem clayton: it depends on your platform
15:05 chem clayton: http://docs.openstack.org/user-guide-admin/manage_projects_users_and_roles.html
15:06 chem clayton: it boils down to what you have defined policy.json i would say
15:06 clayton it appears that at least some of the _member_ code is hard coded inside of keystone
15:06 clayton well, _member_ isn't, but member_role_id and member_role_name
15:06 clayton and _member_ is the default value for member_role_name
15:07 chem clayton: ack, so if it exists already, no need to create and if you define it won't be re-created.
15:07 clayton my point is that _member_ used to be automatically created and it appears that it no longer is
15:08 clayton specifically this removed code that used to automatically add _member_ if it didn't exist - https://review.openstack.org/#/c/228744/
15:08 clayton and that code was merged in sept
15:09 chem clayton: 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:10 cdelatte joined #puppet-openstack
15:10 jistr Vinsh: 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/1524381
15:10 richm1 right - keystone upstream wants you to explicitly create roles instead because the _member_ role is deprecated
15:10 openstack Launchpad bug 1524381 in tripleo "Duplicate declaration when defining swift::storage::* outside swift::storage::server" [Critical,Triaged] - Assigned to Jiří Stránský (jistr)
15:10 Vinsh jistr: Got it. I'll add that to my commit message. review up shortly
15:10 clayton I'm missing the note about _member_ being deprecated and I don't know what the replacement is
15:10 jistr Vinsh: perfect, thanks!
15:11 richm clayton: the replacement is that you create your own role
15:11 richm clayton: so you can just do keystone_role {"_member_"} if you need _membeR_
15:11 richm _member_
15:11 clayton so there is still a need for a role that serves that function, it just may not be named _member_
15:12 clayton is that correct?
15:12 delattec joined #puppet-openstack
15:12 richm clayton: _member_ is mostly meaningless - it is much better to define a real role that is meaningful to your organization
15:12 clayton meaningful how?
15:12 richm _member_ was a hack to allow you to add users to tenants/projects without having to first define a role
15:13 xarses_ joined #puppet-openstack
15:14 richm I'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 richm s/that at/that a/
15:17 degorenko EmilienM, ping :)
15:18 EmilienM degorenko: yo
15:18 clayton richm: what determines if they can get a token for a given project?  Having any role at all on the project?
15:18 derekh joined #puppet-openstack
15:18 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Use include for swift::storage::${type}  https://review.openstack.org/255315
15:18 degorenko EmilienM, 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 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Use include for swift::storage::${type}  https://review.openstack.org/255315
15:20 EmilienM degorenko: thanks for the followup!
15:20 EmilienM degorenko: let me know if there is something we can help with
15:20 degorenko EmilienM, sure :)
15:21 richm clayton: yes
15:21 clayton ok
15:23 clayton I think I got it now.  Thanks for bearing with me.
15:25 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Use include for swift::storage::${type}  https://review.openstack.org/255315
15:26 clayton I'd still argue this is a backwards incompat change that's not documented
15:28 clayton but everyone else here would probably say that if it bothers me enough I'll put up a patch for the release notes ;)
15:28 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Add mistral::engine class  https://review.openstack.org/248412
15:28 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Add mistral::executor class  https://review.openstack.org/248413
15:28 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Add mistral::api class  https://review.openstack.org/248410
15:29 openstackgerrit Dan Prince proposed openstack/puppet-mistral: Update mistral::db::sync to run 'update head'  https://review.openstack.org/248404
15:34 EmilienM clayton: sorry for that, I missed that information :(
15:36 clayton well, it's hidden pretty well, I'm not sure it's really intentional that it worked at all :)
15:37 richm ayoung can explain this much better than I can
15:45 clayton I think I've got it now
15:47 degorenko EmilienM, can you please review: https://review.openstack.org/#/q/topic:fix_db_sync,n,z
15:47 EmilienM degorenko: no
15:47 degorenko i find this problem during fixing fuel bug :)
15:47 degorenko :'( oh
15:48 EmilienM degorenko: :)
15:48 EmilienM oh wait
15:48 EmilienM degorenko: why do you import ::murano?
15:48 EmilienM in api.pp
15:48 degorenko EmilienM, in case of using api as stand alone service
15:49 degorenko or it shoud be done on higher layer?
15:49 EmilienM yes
15:49 EmilienM like in other modules
15:49 degorenko ok, i will update it then
15:49 degorenko oops :D
15:49 EmilienM :)
15:49 degorenko then i should fix it for sahara too
15:49 EmilienM +2 on the other one
15:49 degorenko for sahara it was used because of broken package
15:50 degorenko but now packages for sahara are ok, so it should be removed, you're right
15:50 EmilienM ok
15:56 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Run db sync only from main script  https://review.openstack.org/254809
16:01 dhollinger joined #puppet-openstack
16:01 colonwq EmilienM, 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:02 aimon joined #puppet-openstack
16:03 EmilienM colonwq: I'm not the best guy about puppet-ceph, I don't know their CI status
16:04 EmilienM xarses_ or dmsimard know though
16:04 degorenko dmsimard, ^^^
16:05 colonwq EmilienM, thanks. Trying to get my first patch in.
16:05 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Run db sync only from main script  https://review.openstack.org/254809
16:08 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add hiera calls to allow management of ports and protocols  https://review.openstack.org/240208
16:16 dmsimard colonwq: heya
16:16 dmsimard colonwq: 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:17 dmsimard There's a thread about it on openstack-dev here http://lists.openstack.org/pipermail/openstack-dev/2015-November/080275.html
16:18 dmsimard the thread forked here: http://lists.openstack.org/pipermail/openstack-dev/2015-December/081066.html
16:27 colonwq dmsimard, thanks for the info.
16:28 colonwq dmsimard, How does the patch I sumitted look?
16:29 dmsimard colonwq: I added myself as a reviewer, I'll have a look when I have a chance
16:29 colonwq dmsimard, great thanks.
16:29 dmsimard I'm not familiar with civetweb but I'll give an opinion on the code
16:30 colonwq It is built into the radosgw binary.
16:32 dmsimard colonwq: yeah but I've always run it with Apache :(
16:34 colonwq dmsimard, I can find the ceph version civetweb was add. Surprisingly it 'just works' without extra packages (apache, fcgi)
16:35 colonwq I can change the structure of the code as needed.
16:41 colonwq dmsimard, firefly added civetweb support. https://ceph.com/planet/replace-apache-by-civetweb-on-the-radosgw/
16:52 Xarses EmilienM: 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_225
16:53 EmilienM dmsimard can help here
16:53 colonwq Xarses, right. I added a coment about that. It is a centos 7.1 isue
16:53 colonwq issue
16:54 Xarses we'll have to figure out how to deal with it in order to merge anything
16:56 Xarses EmilienM: dmsimard is there someone you can talk to about the missing dep ?
16:56 colonwq Xarses, I manually added the libunwind package before running beaker locally. I do not know how jenkins can be handled
16:56 colonwq 7.2 should fix it.
16:56 Vinsh EmilienM: https://review.openstack.org/#/c/255315/2  tripleo swift fix ready to go.  jistr
16:56 EmilienM Vinsh: gold or silver?
16:56 Vinsh PLATINUM!
16:56 EmilienM lol
16:56 Vinsh limited time only.
16:57 Xarses colonwq: we can force the package to install if there is a usable version on the interweb somewhere
16:57 EmilienM jistr, slagle: can I test https://review.openstack.org/#/c/255315/ in tripleo ? or did you pin puppet-swift?
16:57 jistr Vinsh: lgtm, thanks
16:57 Xarses colonwq: is 7.2 out yet?
16:57 colonwq Xarses, it is not out yet...e ventually .. sometime.
16:57 colonwq Xarses, here is where I installed libunwind from : https://wiki.centos.org/AdditionalResources/Repositories/CR
16:58 jistr EmilienM: 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 dmsimard Xarses: I can ask around for help
16:58 jistr (as it only breaks the HA job which doesn't run on puppet-swift patches)
16:58 dmsimard Xarses: (For that broken dep)
16:59 Xarses dmsimard: should we just enable this CR repo? or grab the missing package from it and run?
16:59 EmilienM jistr: don't you have to change the interface in Hiera?
16:59 dmsimard Xarses: as a temporary workaround ?
16:59 Xarses dmsimard: yes
17:00 dmsimard worth a try
17:00 jistr EmilienM: not sure what you mean? IIUC the fix would preserve the original behavior
17:00 EmilienM jistr: good
17:00 Xarses dmsimard: I'll whip something up
17:01 Vinsh jistr: You could opt-in to using the swiftinit provider :)
17:02 jistr Vinsh: if that's the recommended thing to use going forward, then we'll probably do so
17:02 EmilienM ++
17:03 Vinsh I'll work with you on an update for tripleo for that in the future.
17:03 jistr ok
17:04 yottatsa joined #puppet-openstack
17:06 dmsimard Xarses: ok, meanwhile I poked the mothership to see if anyone can help
17:10 dprince joined #puppet-openstack
17:10 yottatsa joined #puppet-openstack
17:11 psanchez joined #puppet-openstack
17:13 colonwq EmilienM, I fixed those issues. Too many years doing perl and C. Thanks.
17:26 dmsimard Xarses, colonwq: looks like the broken dependency is a known issue https://bugzilla.redhat.com/show_bug.cgi?id=1289073
17:26 openstack bugzilla.redhat.com bug 1289073 in libunwind "Cannot install nginx from epel" [High,New] - Assigned to tcallawa
17:27 openstackgerrit Denis Egorenko proposed openstack/puppet-sahara: Remove require on main init class  https://review.openstack.org/255393
17:28 colonwq dmsimard, That reads pretty much like what I found on a mailing list.
17:29 * Xarses rolls his eys
17:29 Xarses yay, fun with package deps
17:30 dmsimard looks like that broke a couple things, nginx .. ceph, probably others too.
17:37 aimon joined #puppet-openstack
17:39 ntpttr would 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 is
17:42 Xarses erm, vagrant gives me a ssh error as soon as the node comes up, but I can `vagrant ssh` into the node just fine
17:43 nlahouti joined #puppet-openstack
17:46 rcola joined #puppet-openstack
17:57 Xarses ahh, I broke it
17:57 Xarses makes sense
18:00 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/253216
18:05 nlahouti joined #puppet-openstack
18:11 imcsk8 joined #puppet-openstack
18:20 yottatsa joined #puppet-openstack
18:21 markvoelker joined #puppet-openstack
18:23 Xarses colonwq_afk: so while I'
18:24 Xarses colonwq_afk: so while I'm trying to fix up the gate, I've gone over your review
18:24 openstackgerrit Andrew Woodward proposed openstack/puppet-ceph: Work around missing libunwind dep (CentOS)  https://review.openstack.org/255421
18:24 Xarses dmsimard: colonwq_afk ^
18:28 yottatsa_ joined #puppet-openstack
18:32 yottatsa joined #puppet-openstack
18:33 openstackgerrit Maty Grosz proposed openstack/puppet-vitrage: puppet-vitrage: Initial commit  https://review.openstack.org/255431
18:38 mgrosz Hey, please review puppet-vitrage: Initial commit  https://review.openstack.org/255431. Thanks
18:47 rcola Hi, 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:48 chem rcola: could be a lot of things.  Could you paste somewhere (http://paste.openstack.org/) the complete error log ?
18:49 rcola thanks chem I'll do that
18:50 ducttape_ joined #puppet-openstack
18:53 rcola Here's is my paste ... http://paste.openstack.org/show/481360/
18:54 mgrosz @EmilienM: online?
18:55 chem rcola: don't forget to ping me, if not, I'm not looking :)
18:55 rcola ok thanks chem
19:01 ducttape_ joined #puppet-openstack
19:04 chem rcola: 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-L71
19:06 chem rcola: 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:07 mdorman joined #puppet-openstack
19:08 chem rcola: the code there works and will avoid you to have those "exec" as well (which are lacking the dependencies as well).
19:08 chem rcola: hope it helps
19:08 rcola chem: thanks I'll try that out
19:10 mdorman hey 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 failing
19:11 mdorman i’m having trouble understanding where the domain config info is supposed to come from.  so i need an openrc file?
19:12 mgrosz @EmilienM: please review puppet-vitrage: Initial commit  https://review.openstack.org/255431. Thanks
19:17 dmsimard Xarses: it looks like this might be fixed (when it propagates through the mirrors) https://bugzilla.redhat.com/show_bug.cgi?id=1289073#c14
19:17 openstack bugzilla.redhat.com bug 1289073 in libunwind "Cannot install nginx from epel" [High,New] - Assigned to tcallawa
19:28 ntpttr mdorman: I'vve run into similar issues, and I did need to create an openrc file to fix them personally
19:28 ntpttr mdorman: There could be other ways around it but I'm not sure what they are
19:29 mdorman @ntpttr   thanks.   i’m going down another path right now, i may just do that.
19:29 mdorman k
19:34 _ducttape_ joined #puppet-openstack
19:55 EmilienM mdorman: you don't need openrc afik
19:56 EmilienM mdorman: can you give more details of what is failing? Any example of manifest?
19:58 dprince joined #puppet-openstack
20:00 ntpttr EmilienM: 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 worked
20:00 davideagle joined #puppet-openstack
20:01 ntpttr EmilienM: I don't have the output of the failure any more, but it was something about incorrect options being passed in to the openstack commands
20:03 EmilienM mdorman, ntpttr: file a bug I would recommend
20:04 ntpttr EmilienM: sure
20:07 dmsimard colonwq_afk: it looks like the libunwind issue is fixed in centos upstream, when you submit your new patchset we should no longer encounter that error
20:09 rmsilva joined #puppet-openstack
20:16 ntpttr EmilienM: It seems something has changed in the last two weeks, because I just tried to recreate the error and it worked
20:16 gildub joined #puppet-openstack
20:17 EmilienM ntpttr: cool! the bug is fixed - NEXT
20:20 openstackgerrit Dan Prince proposed openstack/puppet-ironic: Add ironic::keystone::auth_inspector  https://review.openstack.org/253493
20:22 nibalizer EmilienM: you sent me a link to review and I lost it, can I have it again?
20:22 EmilienM nibalizer: https://review.openstack.org/254933
20:22 EmilienM crinkle: should look also
20:23 colonwq Xarses, thanks for the review and the updated spec file.
20:33 openstackgerrit Nate Potter proposed openstack/puppet-keystone: Remove references to puppet-openstack from README  https://review.openstack.org/248382
20:34 karimb joined #puppet-openstack
20:34 openstackgerrit Nate Potter proposed openstack/puppet-keystone: Remove references to puppet-openstack from README  https://review.openstack.org/248382
20:35 RichardRaseley joined #puppet-openstack
20:44 derekh joined #puppet-openstack
20:52 mdorman EmilienM: 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 end
20:55 rmsilva joined #puppet-openstack
20:58 bklei_ joined #puppet-openstack
21:15 bklei joined #puppet-openstack
21:19 ducttape_ joined #puppet-openstack
21:21 openstackgerrit Nate Potter proposed openstack/puppet-nova: Fix scopre warnings in nova::vncproxy::common  https://review.openstack.org/255529
21:28 openstackgerrit Nate Potter proposed openstack/puppet-nova: Fix scope warnings in nova::vncproxy::common  https://review.openstack.org/255529
21:49 richm1 joined #puppet-openstack
22:08 nlahouti joined #puppet-openstack
22:29 bklei joined #puppet-openstack
22:40 openstackgerrit Nate Potter proposed openstack/puppet-openstack-cookiecutter: Replace ceilometer with correct project name  https://review.openstack.org/255575
22:48 openstackgerrit Nate Potter proposed openstack/puppet-openstack-cookiecutter: Fix manifests::db $database_connection variable  https://review.openstack.org/255577
23:06 baker joined #puppet-openstack
23:29 toMeloos joined #puppet-openstack
23:34 ntpttr left #puppet-openstack
23:37 rcernin joined #puppet-openstack
23:38 cwolferh joined #puppet-openstack
23:47 aglarendil joined #puppet-openstack
23:51 patrickeast joined #puppet-openstack
23:53 bklei joined #puppet-openstack
23:53 Xarses dmsimard: ok, so gitbuilder.ceph.com isn't building fastcgi any more so thats what's wrong with ubuntu

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