Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:11 zhangjn joined #puppet-openstack
00:18 guessi joined #puppet-openstack
00:38 xavpaice joined #puppet-openstack
00:56 zhangjn joined #puppet-openstack
00:57 zhangjn joined #puppet-openstack
01:02 zhangjn_ joined #puppet-openstack
01:10 xavpaice left #puppet-openstack
01:15 nlahouti joined #puppet-openstack
01:26 aimon joined #puppet-openstack
01:32 vinsh joined #puppet-openstack
01:34 gildub joined #puppet-openstack
01:55 ducttape_ joined #puppet-openstack
02:00 nlahouti joined #puppet-openstack
02:03 pleia2 joined #puppet-openstack
02:17 ducttape_ joined #puppet-openstack
02:24 ducttape_ joined #puppet-openstack
02:31 richm joined #puppet-openstack
02:41 vinsh joined #puppet-openstack
02:57 ducttape_ joined #puppet-openstack
03:04 fedexo joined #puppet-openstack
03:16 gomarivera__ joined #puppet-openstack
03:17 xingchao joined #puppet-openstack
03:17 ducttape_ joined #puppet-openstack
03:28 ducttape_ joined #puppet-openstack
03:44 richm joined #puppet-openstack
03:58 zhangjn joined #puppet-openstack
03:58 gomarivera__ left #puppet-openstack
04:00 zhangjn joined #puppet-openstack
04:03 xingchao joined #puppet-openstack
04:05 ducttape_ joined #puppet-openstack
04:12 ducttape_ joined #puppet-openstack
04:35 zhangjn joined #puppet-openstack
04:37 nlahouti joined #puppet-openstack
05:27 xingchao joined #puppet-openstack
05:46 sanjayu joined #puppet-openstack
06:01 zhangjn_ joined #puppet-openstack
06:26 zhangjn joined #puppet-openstack
06:35 zhangjn joined #puppet-openstack
06:44 zhangjn joined #puppet-openstack
07:12 xingchao joined #puppet-openstack
07:18 zhangjn joined #puppet-openstack
07:19 leanderthal joined #puppet-openstack
07:26 zhangjn joined #puppet-openstack
07:46 zerda joined #puppet-openstack
07:53 zhangjn joined #puppet-openstack
08:03 zhangjn joined #puppet-openstack
08:19 zigo joined #puppet-openstack
08:32 dgurtner joined #puppet-openstack
08:32 dgurtner joined #puppet-openstack
08:33 zhangjn joined #puppet-openstack
08:36 gildub joined #puppet-openstack
08:37 zhangjn joined #puppet-openstack
08:47 paramite joined #puppet-openstack
09:06 xarses joined #puppet-openstack
09:08 saneax joined #puppet-openstack
09:12 gfidente joined #puppet-openstack
09:20 openstack joined #puppet-openstack
09:27 openstackgerrit Gael Chamoulaud proposed openstack/puppet-trove: Put all the logging related parameters to the logging class  https://review.openstack.org/242395
09:29 openstackgerrit Merged openstack/puppet-modulesync-configs: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/242105
09:29 sbadia morning
09:32 degorenko joined #puppet-openstack
09:33 mkarpin Good morning!
09:34 mkarpin can somebody please review https://review.openstack.org/#/c/241614/ ? thank you!
09:39 openstackgerrit Sebastien Badia proposed openstack/puppet-openstack-cookiecutter: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/243009
09:41 jistr joined #puppet-openstack
09:49 bogdando joined #puppet-openstack
09:50 IvanBerezovskiy joined #puppet-openstack
09:53 ddmitriev joined #puppet-openstack
09:56 openstackgerrit Gael Chamoulaud proposed openstack/puppet-gnocchi: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/243014
09:59 gchamoul joined #puppet-openstack
10:02 derekh joined #puppet-openstack
10:08 openstackgerrit Merged openstack/puppet-murano: Introduce Murano extended logging class  https://review.openstack.org/239427
10:27 gchamoul joined #puppet-openstack
10:31 gchamoul- joined #puppet-openstack
10:40 gchamoul joined #puppet-openstack
10:55 openstackgerrit Mykyta Karpin proposed openstack/puppet-keystone: Add ability to configure paste ini options using config class  https://review.openstack.org/243030
10:58 openstackgerrit Martin Mágr proposed openstack/puppet-heat: Fix unit test  https://review.openstack.org/242585
11:01 openstackgerrit Martin Mágr proposed openstack/puppet-heat: Improve heat::keystone::domain  https://review.openstack.org/239680
11:10 igajsin joined #puppet-openstack
11:14 igajsin left #puppet-openstack
11:32 sergmelikyan joined #puppet-openstack
11:35 iurygregory morning people
11:36 zhangjn joined #puppet-openstack
11:36 zhangjn joined #puppet-openstack
11:43 weihan joined #puppet-openstack
11:43 sergmelikyan joined #puppet-openstack
11:48 degorenko hey o/
11:52 xingchao joined #puppet-openstack
12:05 iurygregory hey degorenko o/ if you have time: http://logs.openstack.org/21/216821/34/check/gate​-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/​2a97d5e/console.html.gz#_2015-11-06_22_03_00_385
12:08 degorenko iurygregory, hey, will look after my lunch :)
12:09 iurygregory no problem man o/. Have a nice lunch
12:38 mkarpin Hey all! can you please review https://review.openstack.org/#/c/237035/ ? thank you!
12:42 _nick joined #puppet-openstack
12:45 openstackgerrit Merged openstack/puppet-gnocchi: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/243014
12:55 xarses joined #puppet-openstack
12:59 degorenko iurygregory, can you provide link to patch?
12:59 iurygregory https://review.openstack.org/#/c/216821/
13:04 samuelBartel joined #puppet-openstack
13:05 moshele1 joined #puppet-openstack
13:10 degorenko iurygregory, i think, you should use 3 concat's: 1 - declaring of concat resource on some file; 2 - you read existing; 3 - add new part
13:10 degorenko now you're trying to mix 2 and 3
13:10 degorenko may be it is your problem
13:10 degorenko and why order 27?
13:13 iurygregory justa random number =X
13:13 iurygregory just a *
13:13 degorenko it's wrong
13:13 degorenko use order 1 for 2) and order 2 for 2)
13:13 degorenko * 1 for 1)
13:13 degorenko :D
13:13 iurygregory oh ok ^^
13:13 iurygregory i'll send another patch right now
13:24 khushbu_ joined #puppet-openstack
13:26 dprince joined #puppet-openstack
13:40 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
13:55 myatsenko spredzy:  crinkle:  EmilienM:   Hello Guys!  I met with some CI problem, I hope you can help me with it.
13:55 myatsenko I have this patch - https://review.openstack.org/#/c/233011/,  it uses "netaddr" module.  To pass unit tests in this patch I have added "netaddr" module to Gemfile and everything was OK.
13:55 myatsenko But after adding "acceptance" tests I got an error like this:
13:55 myatsenko "Error: Could not autoload puppet/provider/nova_floating/nova_manage: cannot load such file -- netaddr"
13:55 myatsenko (http://logs.openstack.org/11/233011/32/c​heck/gate-puppet-nova-puppet-beaker-rspe​c-dsvm-centos7/6590602/console.html.gz)
13:55 myatsenko To investigate this error I run infrastructure tests (bundle exec rspec spec/acceptance) locally. To run this tests locally was asked to install "vagrant", it creates virtual test environment (run VM in my case).
13:55 myatsenko After these changes and after running "acceptance" tests I got the same error, smth like:
13:55 myatsenko "Error: Could not autoload puppet/provider/nova_floating/nova_manage: cannot load such file -- netaddr"
13:55 myatsenko To solve this error I had to install "netaddr" module in VM that was run by "vagrant". After "netaddr" module installation mentioned issue was solved.
13:55 myatsenko How I can solve the proble in Openstack CI?
14:08 gchamoul_ joined #puppet-openstack
14:09 gchamoul_ joined #puppet-openstack
14:13 jerrygb joined #puppet-openstack
14:15 gchamoul_ joined #puppet-openstack
14:15 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
14:15 morazi joined #puppet-openstack
14:22 richm joined #puppet-openstack
14:28 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/216654
14:35 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
14:43 aderyugin Hi, all! Please take a look: https://review.openstack.org/#/c/218333/
14:44 aderyugin spredzy, mfisch, sbadia, EmilienM, crinkle : ^^
14:49 sergmelikyan joined #puppet-openstack
14:52 degorenko joined #puppet-openstack
14:52 sbadia aderyugin: thanks for the patch, I've just a minor comment about tags, otherwise it's ok
14:53 dfisher joined #puppet-openstack
14:57 aderyugin sbadia: thanks, i'll take a look
14:57 ducttape_ joined #puppet-openstack
14:58 dhollinger joined #puppet-openstack
15:00 ducttape_ joined #puppet-openstack
15:04 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation  https://review.openstack.org/218333
15:08 _ducttape_ joined #puppet-openstack
15:11 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
15:14 ntpttr joined #puppet-openstack
15:17 openstackgerrit joined #puppet-openstack
15:32 ducttape_ joined #puppet-openstack
15:43 khushbu_ joined #puppet-openstack
15:43 xingchao joined #puppet-openstack
15:46 sergmelikyan joined #puppet-openstack
15:50 rgowrishankar joined #puppet-openstack
15:55 rgowrishankar EmilienM: can you check out 214798 today if possible?
15:57 skippy joined #puppet-openstack
15:57 skippy hello!  I don't know how to do what is being asked of me in this comment: https://review.openstack.org/#/c/238565/
15:58 mwhahaha rgowrishankar: he's on vacation this week
15:58 mwhahaha skippy: the ask is to combine the two reviews into one
15:58 skippy rather, I know how to squash commits, but not how to do so on the original commit I made.
15:59 mwhahaha skippy: as long as when you squash you end up with the original change-id (I7f5dff32083795e0b362e6f6a176fe0a54115e19), it'll go to the original review
16:02 skippy mwhahaha: I can't `git rebase -i` from a headless checkout.  Please advise.
16:04 rgowrishankar mwhahaha: can any of the other core members look at it, emilien had given +2 subject to removal of a comment
16:06 gchamoul sbadia, spredzy and degorenko: could you please take a look at https://review.openstack.org/#/c/214798/ ? ^^^^
16:07 degorenko rgowrishankar, EmilienM is on PTO
16:09 mdorman joined #puppet-openstack
16:11 sanjayu joined #puppet-openstack
16:12 sergmelikyan joined #puppet-openstack
16:15 degorenko rgowrishankar, 214798 looks good - one more recheck, to be sure, that scenario tests for trusty works after that i'll +2
16:15 rgowrishankar Ok
16:17 degorenko skippy, you can simply mix two commit in one - just add all changes from last patch to first one and upload new patch
16:22 ntpttr When anyone has time, would you mind checking out my initial commit of puppet-magnum at https://review.openstack.org/242164? It's mostly just generated by the cookiecutter with a few small changes to get it to pass the Jenkins checks.
16:25 sergmelikyan joined #puppet-openstack
16:26 vinsh joined #puppet-openstack
16:29 degorenko joined #puppet-openstack
16:43 skippy degorenko: sorry, this workflow is new to me.  i have checked out the `checkout` link from the review page.  Do I push to this, or do I re-run `git review`?
16:44 mwhahaha skippy: http://docs.openstack.org/i​nfra/manual/developers.html
16:45 degorenko skippy, so, you have your first patch - add new changes then -> git add <your changed files> -> git commit --amend -> git review
16:46 skippy degorenko: ok.  It lists three commits to be posted, and tells me I should consider squashing.  But `git rebase -i` demands a branch.
16:48 degorenko skippy, three? why? it should only one - 238521
16:48 skippy i don't know why.  I did this:  git fetch https://review.openstack.org/openstack/puppet-ceph refs/changes/26/238526/1 && git checkout FETCH_HEAD
16:49 skippy then added the spec file from the newer commit (https://review.openstack.org/#/c/238565/)
16:49 ntpttr joined #puppet-openstack
16:49 mwhahaha so you should be using git commit --amend
16:50 skippy ah, ok.  I htink I get it now.
16:50 openstackgerrit skippy proposed openstack/puppet-ceph: Add Apache 2.4 proxy_fastcgi support for radosgw  https://review.openstack.org/238526
16:51 mwhahaha with gerrit, always use git commit --amend when updating something
16:51 mwhahaha it tracks the changes with the Change-Id from the commit message
16:51 skippy thank you!
16:51 mwhahaha so you could also just redo a commit and as long as you include the original change-id, it should end up in the same spot
16:52 degorenko skippy, yes :) +1 to mwhahaha, btw, you can abandon second patch :)
16:52 skippy how do I abandon it?
16:52 degorenko skippy, on UI button - abandon :)
16:53 skippy oh, i wasn't logged in.  didnt see the button.
16:53 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
16:53 dmsimard degorenko: fyi same error for the stackforge rename http://logs.openstack.org/21/238521/4/​check/gate-puppet-ceph-puppet-beaker-r​spec-dsvm-trusty/da5719f/console.html
16:53 dmsimard there might be a dependency that isn't setup properly
16:54 degorenko dmsimard, strange :(
16:55 degorenko because i have not any code changes
16:56 dmsimard degorenko: it might not be puppet-ceph's fault
16:56 dmsimard keystone is pinned quite liberally http://logs.openstack.org/21/238521/4/check/ga​te-puppet-ceph-puppet-beaker-rspec-dsvm-trusty​/da5719f/console.html#_2015-11-09_09_58_58_874
16:57 degorenko dmsimard, keystone is on 6.1.0 now, and it works https://github.com/openstack/puppet-k​eystone/blob/master/metadata.json#L3
16:58 dmsimard We install juno though
16:58 dmsimard so there might be something broken, I dunno - but this is where I would dig
16:58 degorenko puppet-ceph uses juno?
16:59 dmsimard degorenko: https://github.com/openstack/puppet-ceph/blob/m​aster/spec/acceptance/ceph_rgw_spec.rb#L47-L75
16:59 degorenko dmsimard, oh
17:00 degorenko dmsimard, then https://review.openstack.org/#/c/23​8521/4/spec/spec_helper_acceptance.rb is wrong (in line 48)
17:00 degorenko i'm about keystone versions
17:00 degorenko probably i need back to >=5.1.0 <6.0.0
17:00 degorenko my bad :)
17:01 degorenko just i didn't know about juno code for ceph
17:01 dmsimard degorenko: I vaguely remember dgurtner telling me he wasn't able to get it to work with kilo at the time so we settled on using juno
17:01 dmsimard we should definitely test with liberty at this point if we can get it to work
17:03 openstackgerrit Denis Egorenko proposed openstack/puppet-ceph: Replace all StackForge names to OpenStack  https://review.openstack.org/238521
17:03 degorenko dmsimard, ^ will see what jenkins thinks now
17:05 dmsimard degorenko: oh I see, you bumped keystone at the same time
17:05 degorenko yeah
17:05 degorenko :)
17:06 dmsimard yeah that might explain it, was openstackclient even in juno ?
17:06 degorenko actually it was
17:06 dmsimard probably changed drastically between juno and liberty though
17:06 degorenko but i'm not sure about dependency for openstack client in juno and now :) they are different
17:07 degorenko how it works on CentOS then? :D
17:07 * dmsimard shrugs
17:08 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation  https://review.openstack.org/218333
17:09 rgowrishankar degoernko: Do I need to do anything for the recheck?
17:10 degorenko rgowrishankar, typo in my nick :)
17:10 degorenko no you don't
17:10 degorenko it is almost done
17:10 rgowrishankar degorenko: Yes, I realized it just after I hit enter.. thanks :)
17:10 degorenko http://status.openstack.org/zuul/
17:10 degorenko your patch already passed trusty scenario 2
17:11 degorenko rgowrishankar, ^
17:11 degorenko oh
17:11 degorenko now all scenarios is voting
17:11 degorenko centos7 is failed because of lack openstack-trove packages
17:16 sergmelikyan joined #puppet-openstack
17:34 IvanBerezovskiy left #puppet-openstack
17:38 skippy left #puppet-openstack
17:40 gchamoul degorenko: weird because rdo repos have openstack-trove-* packages ...
17:40 degorenko gchamoul, yes, but they are probably not in stable
17:41 degorenko gchamoul, https://review.openstack.org/#/c/239272/ because of this patch
17:42 degorenko puppet jobs failed i mean
17:43 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-cookiecutter: Replace StackForge namespace on Openstack in metadata.json  https://review.openstack.org/243230
17:44 degorenko sbadia, spredzy crinkle https://review.openstack.org/#/c/243230/
17:47 gchamoul degorenko: duplicate with this one https://review.openstack.org/#/c/243009/
17:48 degorenko oh crap :)
17:48 gchamoul degorenko: and you already put +2 ;-)
17:48 degorenko yep... :D
17:48 degorenko forgot about this patch)
17:49 degorenko so many patches were today :)
17:53 openstackgerrit Nate Potter proposed openstack/puppet-magnum: puppet-magnum: Initial commit  https://review.openstack.org/242164
17:53 rahulait joined #puppet-openstack
17:56 sergmelikyan joined #puppet-openstack
17:57 iurygregory hey people, any tips on how to solve this http://logs.openstack.org/21/216821/39/check/gat​e-puppet-keystone-puppet-beaker-rspec-dsvm-trust​y/bac1a77/console.html#_2015-11-09_17_35_17_406 / https://review.openstack.org/#/c/216821/
18:04 degorenko dmsimard, http://logs.openstack.org/21/238521/5/check/ga​te-puppet-ceph-puppet-beaker-rspec-dsvm-trusty​/8c1bbbd/console.html#_2015-11-09_17_52_16_670
18:04 degorenko any thoughts?
18:04 degorenko again - how it works for centos? \o/
18:07 degorenko oh, crap
18:07 degorenko there is only 6.0.0 and 6.1.0 versions
18:07 degorenko in https://forgeapi.puppetlabs​.com/#!/release/getReleases
18:07 degorenko dmsimard, https://forgeapi.puppetlabs.com/v3/r​eleases?module=openstack%2Fkeystone
18:07 dmsimard before that it was probably under puppetlabs
18:08 dmsimard i.e, puppetlabs/keystone
18:08 degorenko yes
18:08 degorenko yuor're right
18:08 degorenko https://forgeapi.puppetlabs.com/v3/r​eleases?module=puppetlabs%2Fkeystone
18:09 xingchao joined #puppet-openstack
18:09 degorenko also, as i see - there are no actual tests for centos, that's why it passed
18:10 dmsimard oh man, you're right, a skip for them was just merged
18:10 dmsimard because there was a completely unrelated issue (which we still haven't figured out)
18:10 dmsimard mwhahaha and dgurtner can explain that better than I can
18:11 degorenko so, i guess, i need back again to stackforge for spec_helpers
18:11 degorenko because it probably will not work on 6.0.0 keystone
18:11 dmsimard It looks like we'll need to work on getting the rgw tests fixed entirely -- and up to juno
18:11 dmsimard matter of fact, infernalis is out now so we're due to bump versions anyway
18:12 degorenko dmsimard, may be up to kilo at least? :D
18:13 degorenko you are on juno right now, ist'n it?
18:16 dmsimard degorenko: well the acceptance tests run off of juno yeah
18:18 degorenko dmsimard, so, for my patch, what i should to do? back stackforge for spec_helpers and put some comment or what?
18:22 dmsimard degorenko: yeah, can't touch keystone module in the acceptance tests yet so just leave it be and see if that works
18:23 dmsimard it'd be great if I had the time to work out some liberty/infernalis tests
18:24 openstackgerrit Denis Egorenko proposed openstack/puppet-ceph: Replace all StackForge names to OpenStack  https://review.openstack.org/238521
18:25 degorenko dmsimard, ^^
18:25 degorenko i hope - it will passed tests :)
18:25 dmsimard degorenko: let's wait and find out
18:25 degorenko yep ;)
18:26 * degorenko afk
18:45 Hunner joined #puppet-openstack
18:45 Hunner joined #puppet-openstack
19:03 dhollinger joined #puppet-openstack
19:13 rahulait joined #puppet-openstack
19:15 ntpttr1 joined #puppet-openstack
19:15 xingchao joined #puppet-openstack
19:26 ducttape_ joined #puppet-openstack
19:44 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
19:44 angdraug joined #puppet-openstack
19:51 dprince joined #puppet-openstack
20:22 ntpttr1 left #puppet-openstack
20:27 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
20:36 sanjayu joined #puppet-openstack
20:43 ducttape_ joined #puppet-openstack
20:44 myatsenko spredzy: crinkle: EmilienM:  Guys plz help me with this...
20:44 myatsenko I met with some CI problem, I hope you can help me with it.
20:44 myatsenko I have this patch - https://review.openstack.org/#/c/233011/,
20:44 myatsenko it uses "netaddr" module.
20:44 myatsenko To pass unit tests in this patch I have added "netaddr" module to Gemfile and
20:44 myatsenko everything was OK.
20:44 myatsenko But after adding "acceptance" tests I got an error like this:
20:44 myatsenko "Error: Could not autoload puppet/provider/nova_floating/nova_manage: cannot load such file -- netaddr"
20:44 myatsenko (http://logs.openstack.org/11/233011/32/c​heck/gate-puppet-nova-puppet-beaker-rspe​c-dsvm-centos7/6590602/console.html.gz)
20:44 myatsenko To investigate this error I run infrastructure tests (bundle exec rspec spec/acceptance) locally.
20:44 myatsenko To run this tests locally was asked to install "vagrant", it creates virtual test environment (run VM in my case).
20:44 myatsenko After these changes and after running "acceptance" tests I got the same error, smth like:
20:44 myatsenko "Error: Could not autoload puppet/provider/nova_floating/nova_manage: cannot load such file -- netaddr"
20:44 myatsenko To solve this error I had to install "netaddr" module in VM that was run by "vagrant". After "netaddr" module
20:47 iurygregory myatsenko, emilien is on vacation this week
20:47 crinkle myatsenko: you can't add an external gem as a dependency for a provider, you need to find a way to do it with built-in ruby modules or with libaries within puppet
20:51 Hunner Usually I wrap the `require` in a rescue block in the provider and print a helpful message if the load fails
20:51 Hunner And document it in the readme
20:52 iurygregory hey crinkle, can you help me with a problem that i'm facing using concat?
21:00 Hunner iurygregory: What's up?
21:00 iurygregory hey Hunner =)
21:00 iurygregory Well, i need to modify the keystone virtual host to add some extra configurations to use Federation...
21:01 iurygregory The thing is, the extra configuration (location and locationMatch tags) should be after </virtualhost>
21:02 iurygregory so i'm trying to use concat and concat::fragment but i'm facing some issues
21:02 iurygregory This is the code: https://review.openstack.org/#/c/216821/
21:03 iurygregory this is what i'm trying to do to the virtual host http://paste.openstack.org/compare/473858/473857/
21:03 iurygregory lines 30 to 39
21:05 Hunner looking
21:05 iurygregory tks
21:10 Hunner iurygregory: Well, the error is because the concat resource declares a file resource (it's wrapped in concat/manifest/init.pp line 195 which is the line the error is print for)
21:10 Hunner So you probably don't want to redeclare the base concat resource; just have your concat::fragment resources add to the end of the existing one
21:11 iurygregory oh so i don't need line 59 to 67?
21:12 Hunner ooo
21:12 Hunner in keystone/manifests/wsgi/apache.pp it actually declares keystone_wsgi_admin as a file... still poking through things
21:13 iurygregory yeah =/
21:13 Hunner ah, then declares an apache::vhost resource, which is where the concat actually comes in...
21:15 Hunner iurygregory: https://github.com/puppetlabs/puppetlabs-a​pache/blob/master/manifests/vhost.pp#L973 is where </VirtualHost> comes from
21:15 Hunner So if you make your orders > 999 then they would get added after that I think
21:15 Hunner concat::fragment orders *
21:16 iurygregory oh ok Hunner, so i'll try to use just concat::fragment with what i need to add and order greater than 999 right?
21:16 xingchao joined #puppet-openstack
21:17 Hunner I'd give that a go
21:17 iurygregory i'll try :D
21:18 iurygregory like this right? http://paste.openstack.org/show/478393/
21:19 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
21:22 Hunner I'd have thought target would be "${keystone::wsgi::apache::prio​rity}-keystone_wsgi_admin.conf"
21:23 ducttape_ joined #puppet-openstack
21:23 iurygregory oh ok =) if fails i'll try with this
21:23 iurygregory thank you \o/
21:24 Hunner apache::vhost is not very accessible :(. There needs to be more ways to add custom content
21:24 Hunner without having to make explicit concat::fragment resources
21:25 iurygregory yeah =/ should be a variable that we can use XD
21:25 Hunner apache::vhost::extra_content
21:25 Hunner or something
21:25 Hunner PRs welcome XD
21:26 iurygregory PRs? O.o
21:26 Hunner also apache::vhost::custom_content
21:26 Hunner apache's workflow is github based, so PRs instead of patches
21:27 iurygregory oh thanks for the explanation
21:42 iurygregory Hunner, CI fails but now is a different thing  https://jenkins04.openstack.org/job​/gate-puppet-keystone-puppet-beaker​-rspec-dsvm-trusty/261/consoleFull
21:42 iurygregory Could not find dependent Exec[concat_/etc/apache2/sites-avai​lable/10-keystone_wsgi_admin.conf] for File[/var/lib/puppet/concat/_etc_apache2​_sites-available_10-keystone_wsgi_admin.​conf/fragments/1000_keystone_federation] at /etc/puppet/modules/concat​/manifests/fragment.pp:132
21:48 Hunner iurygregory: you should try fixing your target :)
21:48 iurygregory ok ;)
21:49 dmsimard degorenko: heyyyyyyy success !
21:49 Hunner The exec's title of "concat_/etc..." makes it sound like your absolute path target doesn't match an existing concat resource
21:50 iurygregory I'm switching to target  => "${keystone::wsgi::apache::prior​ity}-keystone_wsgi_admin.conf",
21:50 iurygregory shoul i switch the concat::fragment name?
21:51 iurygregory should*
21:51 Hunner the name doesn't matter, as long as it is unique
21:51 iurygregory ok
21:54 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/216821
21:54 rahulait joined #puppet-openstack
22:02 nlahouti joined #puppet-openstack
22:35 angdraug joined #puppet-openstack
22:50 social joined #puppet-openstack
23:02 jerrygb joined #puppet-openstack
23:21 cdelatte joined #puppet-openstack

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