Monday, 2015-11-09

« Sunday, 2015-11-08 Index Tuesday, 2015-11-10 »
*** zhangjn has joined #puppet-openstack00:11
*** zhangjn has quit IRC00:15
*** guessi has joined #puppet-openstack00:18
*** xarses has quit IRC00:20
*** guessi has quit IRC00:28
*** xavpaice has joined #puppet-openstack00:38
*** zhangjn has joined #puppet-openstack00:56
*** zhangjn has quit IRC00:56
*** zhangjn has joined #puppet-openstack00:57
*** zhangjn_ has joined #puppet-openstack01:02
*** zhangjn has quit IRC01:04
*** xavpaice has left #puppet-openstack01:10
*** nlahouti has joined #puppet-openstack01:15
*** xingchao has quit IRC01:24
*** aimon has joined #puppet-openstack01:26
*** aimon_ has quit IRC01:29
*** vinsh has joined #puppet-openstack01:32
*** gildub has joined #puppet-openstack01:34
*** ducttape_ has joined #puppet-openstack01:55
*** nlahouti has quit IRC02:00
*** vinsh has quit IRC02:00
*** nlahouti has joined #puppet-openstack02:00
*** pleia2 has quit IRC02:03
*** pleia2 has joined #puppet-openstack02:03
*** ducttape_ has quit IRC02:07
*** ducttape_ has joined #puppet-openstack02:17
*** ducttape_ has quit IRC02:21
*** ducttape_ has joined #puppet-openstack02:24
*** richm has joined #puppet-openstack02:31
*** vinsh has joined #puppet-openstack02:41
*** richm has quit IRC02:45
*** ducttape_ has quit IRC02:51
*** ducttape_ has joined #puppet-openstack02:57
*** vinsh has quit IRC02:58
*** ducttape_ has quit IRC03:01
*** fedexo has joined #puppet-openstack03:04
*** gomarivera__ has joined #puppet-openstack03:16
*** xingchao has joined #puppet-openstack03:17
*** ducttape_ has joined #puppet-openstack03:17
*** ducttape_ has quit IRC03:27
*** ducttape_ has joined #puppet-openstack03:28
*** richm has joined #puppet-openstack03:44
*** ducttape_ has quit IRC03:55
*** zhangjn_ has quit IRC03:55
*** xingchao has quit IRC03:57
*** zhangjn has joined #puppet-openstack03:58
*** gomarivera__ has left #puppet-openstack03:58
*** zhangjn has quit IRC03:59
*** zhangjn has joined #puppet-openstack04:00
*** xingchao has joined #puppet-openstack04:03
*** ducttape_ has joined #puppet-openstack04:05
*** ducttape_ has quit IRC04:05
*** zhangjn has quit IRC04:07
*** richm has quit IRC04:09
*** ducttape_ has joined #puppet-openstack04:12
*** ducttape_ has quit IRC04:19
*** nlahouti has quit IRC04:34
*** zhangjn has joined #puppet-openstack04:35
*** nlahouti has joined #puppet-openstack04:37
*** xingchao has quit IRC04:37
*** xingchao has joined #puppet-openstack05:27
*** sanjayu has joined #puppet-openstack05:46
*** zhangjn_ has joined #puppet-openstack06:01
*** zhangjn has quit IRC06:04
*** zhangjn_ has quit IRC06:18
*** nlahouti has quit IRC06:22
*** fedexo has quit IRC06:22
*** zhangjn has joined #puppet-openstack06:26
*** zhangjn has quit IRC06:33
*** zhangjn has joined #puppet-openstack06:35
*** zhangjn has quit IRC06:38
*** zhangjn has joined #puppet-openstack06:44
*** zhangjn has quit IRC06:48
*** gildub has quit IRC06:57
*** xingchao has quit IRC07:11
*** xingchao has joined #puppet-openstack07:12
*** zhangjn has joined #puppet-openstack07:18
*** leanderthal has joined #puppet-openstack07:19
*** zhangjn has quit IRC07:24
*** zhangjn has joined #puppet-openstack07:26
*** zerda has joined #puppet-openstack07:46
*** zhangjn has quit IRC07:47
*** zhangjn has joined #puppet-openstack07:53
*** zhangjn has quit IRC07:54
*** zhangjn has joined #puppet-openstack08:03
*** zigo has quit IRC08:16
*** zigo has joined #puppet-openstack08:19
*** zhangjn has quit IRC08:27
*** dgurtner has joined #puppet-openstack08:32
*** dgurtner has quit IRC08:32
*** dgurtner has joined #puppet-openstack08:32
*** zhangjn has joined #puppet-openstack08:33
*** gildub has joined #puppet-openstack08:36
*** zhangjn has quit IRC08:36
*** zhangjn has joined #puppet-openstack08:37
*** paramite has joined #puppet-openstack08:47
*** xarses has joined #puppet-openstack09:06
*** saneax has quit IRC09:07
*** saneax has joined #puppet-openstack09:08
*** gfidente has joined #puppet-openstack09:13
*** openstack has joined #puppet-openstack09:20
openstackgerritGael Chamoulaud proposed openstack/puppet-trove: Put all the logging related parameters to the logging class  https://review.openstack.org/24239509:27
openstackgerritMerged openstack/puppet-modulesync-configs: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/24210509:29
sbadiamorning09:29
*** degorenko has joined #puppet-openstack09:32
mkarpinGood morning!09:33
mkarpincan somebody please review https://review.openstack.org/#/c/241614/ ? thank you!09:34
openstackgerritSebastien Badia proposed openstack/puppet-openstack-cookiecutter: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/24300909:39
*** jistr has joined #puppet-openstack09:41
*** bogdando has joined #puppet-openstack09:49
*** IvanBerezovskiy has joined #puppet-openstack09:50
*** ddmitriev has joined #puppet-openstack09:53
openstackgerritGael Chamoulaud proposed openstack/puppet-gnocchi: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/24301409:56
*** gchamoul has joined #puppet-openstack09:59
*** derekh has joined #puppet-openstack10:02
openstackgerritMerged openstack/puppet-murano: Introduce Murano extended logging class  https://review.openstack.org/23942710:08
*** gchamoul has quit IRC10:16
*** gchamoul has joined #puppet-openstack10:27
*** gchamoul- has joined #puppet-openstack10:32
*** gchamoul has quit IRC10:34
*** xarses has quit IRC10:37
*** gchamoul has joined #puppet-openstack10:40
openstackgerritMykyta Karpin proposed openstack/puppet-keystone: Add ability to configure paste ini options using config class  https://review.openstack.org/24303010:55
openstackgerritMartin Mágr proposed openstack/puppet-heat: Fix unit test  https://review.openstack.org/24258510:58
openstackgerritMartin Mágr proposed openstack/puppet-heat: Improve heat::keystone::domain  https://review.openstack.org/23968011:01
*** igajsin has joined #puppet-openstack11:10
*** zhangjn has quit IRC11:10
*** igajsin has left #puppet-openstack11:14
*** xingchao has quit IRC11:25
*** sergmelikyan has joined #puppet-openstack11:32
iurygregorymorning people11:35
*** zhangjn has joined #puppet-openstack11:36
*** zhangjn has quit IRC11:36
*** zhangjn has joined #puppet-openstack11:36
*** sergmelikyan has quit IRC11:37
*** weihan has joined #puppet-openstack11:43
*** sergmelikyan has joined #puppet-openstack11:43
*** sergmelikyan has quit IRC11:44
degorenkohey o/11:48
*** xingchao has joined #puppet-openstack11:52
iurygregoryhey 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_38512:05
*** zerda has quit IRC12:06
degorenkoiurygregory, hey, will look after my lunch :)12:08
iurygregoryno problem man o/. Have a nice lunch12:09
*** gildub has quit IRC12:19
*** gchamoul- has quit IRC12:23
mkarpinHey all! can you please review https://review.openstack.org/#/c/237035/ ? thank you!12:38
*** _nick has quit IRC12:41
*** _nick has joined #puppet-openstack12:42
openstackgerritMerged openstack/puppet-gnocchi: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/24301412:45
*** xarses has joined #puppet-openstack12:55
*** sanjayu has quit IRC12:55
*** weihan has quit IRC12:57
degorenkoiurygregory, can you provide link to patch?12:59
iurygregoryhttps://review.openstack.org/#/c/216821/12:59
*** samuelBartel has joined #puppet-openstack13:04
*** moshele1 has joined #puppet-openstack13:05
*** moshele1 has quit IRC13:05
*** jerrygb has quit IRC13:07
degorenkoiurygregory, i think, you should use 3 concat's: 1 - declaring of concat resource on some file; 2 - you read existing; 3 - add new part13:10
degorenkonow you're trying to mix 2 and 313:10
degorenkomay be it is your problem13:10
degorenkoand why order 27?13:10
iurygregoryjusta random number =X13:13
iurygregoryjust a *13:13
degorenkoit's wrong13:13
degorenkouse order 1 for 2) and order 2 for 2)13:13
degorenko* 1 for 1)13:13
degorenko:D13:13
iurygregoryoh ok ^^13:13
iurygregoryi'll send another patch right now13:13
*** jayg|g0n3 is now known as jayg13:15
*** khushbu_ has joined #puppet-openstack13:24
*** dprince has joined #puppet-openstack13:26
*** trown|outttypeww is now known as trown13:37
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/21682113:40
myatsenkospredzy:  crinkle:  EmilienM:   Hello Guys!  I met with some CI problem, I hope you can help me with it.13:55
myatsenkoI 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
myatsenkoBut 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/check/gate-puppet-nova-puppet-beaker-rspec-dsvm-centos7/6590602/console.html.gz)13:55
myatsenkoTo 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
myatsenkoAfter 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
myatsenkoTo 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
myatsenkoHow I can solve the proble in Openstack CI?13:55
*** gchamoul_ has joined #puppet-openstack14:08
*** gchamoul_ has quit IRC14:08
*** gchamoul_ has joined #puppet-openstack14:09
*** gchamoul_ has quit IRC14:09
*** jerrygb has joined #puppet-openstack14:13
*** gchamoul_ has joined #puppet-openstack14:15
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/21682114:15
*** morazi has joined #puppet-openstack14:15
*** richm has joined #puppet-openstack14:22
*** gchamoul_ has quit IRC14:26
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/21665414:28
*** zhangjn has quit IRC14:30
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/21682114:35
aderyuginHi, all! Please take a look: https://review.openstack.org/#/c/218333/14:43
aderyuginspredzy, mfisch, sbadia, EmilienM, crinkle : ^^14:44
*** sergmelikyan has joined #puppet-openstack14:49
*** degorenko has quit IRC14:50
*** degorenko has joined #puppet-openstack14:52
sbadiaaderyugin: thanks for the patch, I've just a minor comment about tags, otherwise it's ok14:52
*** dfisher has joined #puppet-openstack14:53
aderyuginsbadia: thanks, i'll take a look14:57
*** ducttape_ has joined #puppet-openstack14:57
*** dhollinger has joined #puppet-openstack14:58
*** ducttape_ has quit IRC15:00
*** ducttape_ has joined #puppet-openstack15:00
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation  https://review.openstack.org/21833315:04
*** _ducttape_ has joined #puppet-openstack15:09
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/21682115:11
*** ducttape_ has quit IRC15:11
*** ntpttr has quit IRC15:14
*** ntpttr has joined #puppet-openstack15:14
*** openstackgerrit has quit IRC15:17
*** openstackgerrit has joined #puppet-openstack15:17
*** gfidente is now known as gfidente|afk15:25
*** trown is now known as trown|brb15:26
*** trown|brb is now known as trown15:30
*** ducttape_ has joined #puppet-openstack15:32
*** khushbu_ has quit IRC15:34
*** _ducttape_ has quit IRC15:35
*** sergmelikyan has quit IRC15:39
*** paramite is now known as paramite|afk15:40
*** xingchao has quit IRC15:42
*** khushbu_ has joined #puppet-openstack15:43
*** xingchao has joined #puppet-openstack15:43
*** sergmelikyan has joined #puppet-openstack15:46
*** rgowrishankar has joined #puppet-openstack15:50
*** paramite|afk is now known as paramite15:51
rgowrishankarEmilienM: can you check out 214798 today if possible?15:55
*** skippy has joined #puppet-openstack15:57
skippyhello!  I don't know how to do what is being asked of me in this comment: https://review.openstack.org/#/c/238565/15:57
mwhahahargowrishankar: he's on vacation this week15:58
mwhahahaskippy: the ask is to combine the two reviews into one15:58
skippyrather, I know how to squash commits, but not how to do so on the original commit I made.15:58
mwhahahaskippy: as long as when you squash you end up with the original change-id (I7f5dff32083795e0b362e6f6a176fe0a54115e19), it'll go to the original review15:59
*** sergmelikyan has quit IRC16:00
skippymwhahaha: I can't `git rebase -i` from a headless checkout.  Please advise.16:02
rgowrishankarmwhahaha: can any of the other core members look at it, emilien had given +2 subject to removal of a comment16:04
gchamoulsbadia, spredzy and degorenko: could you please take a look at https://review.openstack.org/#/c/214798/ ? ^^^^16:07
degorenkorgowrishankar, EmilienM is on PTO16:07
*** mdorman has joined #puppet-openstack16:09
*** sanjayu has joined #puppet-openstack16:11
*** sergmelikyan has joined #puppet-openstack16:12
*** sergmelikyan has quit IRC16:12
degorenkorgowrishankar, 214798 looks good - one more recheck, to be sure, that scenario tests for trusty works after that i'll +216:15
rgowrishankarOk16:15
degorenkoskippy, you can simply mix two commit in one - just add all changes from last patch to first one and upload new patch16:17
ntpttrWhen 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:22
*** sergmelikyan has joined #puppet-openstack16:25
*** vinsh has joined #puppet-openstack16:26
*** degorenko has quit IRC16:28
*** degorenko has joined #puppet-openstack16:29
*** paramite is now known as paramite|afk16:29
*** khushbu_ has quit IRC16:34
skippydegorenko: 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:43
mwhahahaskippy: http://docs.openstack.org/infra/manual/developers.html16:44
degorenkoskippy, so, you have your first patch - add new changes then -> git add <your changed files> -> git commit --amend -> git review16:45
skippydegorenko: ok.  It lists three commits to be posted, and tells me I should consider squashing.  But `git rebase -i` demands a branch.16:46
degorenkoskippy, three? why? it should only one - 23852116:48
skippyi don't know why.  I did this:  git fetch https://review.openstack.org/openstack/puppet-ceph refs/changes/26/238526/1 && git checkout FETCH_HEAD16:48
*** ntpttr has quit IRC16:48
skippythen added the spec file from the newer commit (https://review.openstack.org/#/c/238565/)16:49
*** sergmelikyan has quit IRC16:49
*** ntpttr has joined #puppet-openstack16:49
mwhahahaso you should be using git commit --amend16:49
skippyah, ok.  I htink I get it now.16:50
openstackgerritskippy proposed openstack/puppet-ceph: Add Apache 2.4 proxy_fastcgi support for radosgw  https://review.openstack.org/23852616:50
mwhahahawith gerrit, always use git commit --amend when updating something16:51
mwhahahait tracks the changes with the Change-Id from the commit message16:51
skippythank you!16:51
mwhahahaso you could also just redo a commit and as long as you include the original change-id, it should end up in the same spot16:51
degorenkoskippy, yes :) +1 to mwhahaha, btw, you can abandon second patch :)16:52
skippyhow do I abandon it?16:52
degorenkoskippy, on UI button - abandon :)16:52
skippyoh, i wasn't logged in.  didnt see the button.16:53
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/21682116:53
dmsimarddegorenko: fyi same error for the stackforge rename http://logs.openstack.org/21/238521/4/check/gate-puppet-ceph-puppet-beaker-rspec-dsvm-trusty/da5719f/console.html16:53
dmsimardthere might be a dependency that isn't setup properly16:53
*** paramite|afk is now known as paramite16:54
degorenkodmsimard, strange :(16:54
degorenkobecause i have not any code changes16:55
dmsimarddegorenko: it might not be puppet-ceph's fault16:56
dmsimardkeystone is pinned quite liberally http://logs.openstack.org/21/238521/4/check/gate-puppet-ceph-puppet-beaker-rspec-dsvm-trusty/da5719f/console.html#_2015-11-09_09_58_58_87416:56
degorenkodmsimard, keystone is on 6.1.0 now, and it works https://github.com/openstack/puppet-keystone/blob/master/metadata.json#L316:57
dmsimardWe install juno though16:58
dmsimardso there might be something broken, I dunno - but this is where I would dig16:58
degorenkopuppet-ceph uses juno?16:58
dmsimarddegorenko: https://github.com/openstack/puppet-ceph/blob/master/spec/acceptance/ceph_rgw_spec.rb#L47-L7516:59
degorenkodmsimard, oh17:00
degorenkodmsimard, then https://review.openstack.org/#/c/238521/4/spec/spec_helper_acceptance.rb is wrong (in line 48)17:00
degorenkoi'm about keystone versions17:00
degorenkoprobably i need back to >=5.1.0 <6.0.017:00
degorenkomy bad :)17:00
degorenkojust i didn't know about juno code for ceph17:01
dmsimarddegorenko: 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 juno17:01
dmsimardwe should definitely test with liberty at this point if we can get it to work17:01
openstackgerritDenis Egorenko proposed openstack/puppet-ceph: Replace all StackForge names to OpenStack  https://review.openstack.org/23852117:03
degorenkodmsimard, ^ will see what jenkins thinks now17:03
*** paramite has quit IRC17:05
dmsimarddegorenko: oh I see, you bumped keystone at the same time17:05
degorenkoyeah17:05
degorenko:)17:05
dmsimardyeah that might explain it, was openstackclient even in juno ?17:06
degorenkoactually it was17:06
dmsimardprobably changed drastically between juno and liberty though17:06
degorenkobut i'm not sure about dependency for openstack client in juno and now :) they are different17:06
degorenkohow it works on CentOS then? :D17:07
* dmsimard shrugs17:07
openstackgerritAlexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation  https://review.openstack.org/21833317:08
*** xingchao has quit IRC17:08
rgowrishankardegoernko: Do I need to do anything for the recheck?17:09
degorenkorgowrishankar, typo in my nick :)17:10
degorenkono you don't17:10
degorenkoit is almost done17:10
rgowrishankardegorenko: Yes, I realized it just after I hit enter.. thanks :)17:10
degorenkohttp://status.openstack.org/zuul/17:10
degorenkoyour patch already passed trusty scenario 217:10
degorenkorgowrishankar, ^17:11
degorenkooh17:11
degorenkonow all scenarios is voting17:11
degorenkocentos7 is failed because of lack openstack-trove packages17:11
*** samuelBartel has quit IRC17:14
*** sergmelikyan has joined #puppet-openstack17:16
*** IvanBerezovskiy has left #puppet-openstack17:34
*** skippy has left #puppet-openstack17:38
gchamouldegorenko: weird because rdo repos have openstack-trove-* packages ...17:40
degorenkogchamoul, yes, but they are probably not in stable17:40
degorenkogchamoul, https://review.openstack.org/#/c/239272/ because of this patch17:41
degorenkopuppet jobs failed i mean17:42
*** dhollinger has quit IRC17:42
*** aimon has quit IRC17:43
openstackgerritDenis Egorenko proposed openstack/puppet-openstack-cookiecutter: Replace StackForge namespace on Openstack in metadata.json  https://review.openstack.org/24323017:44
*** sanjayu has quit IRC17:44
degorenkosbadia, spredzy crinkle https://review.openstack.org/#/c/243230/17:44
gchamouldegorenko: duplicate with this one https://review.openstack.org/#/c/243009/17:47
degorenkooh crap :)17:48
gchamouldegorenko: and you already put +2 ;-)17:48
degorenkoyep... :D17:48
degorenkoforgot about this patch)17:48
degorenkoso many patches were today :)17:49
*** sergmelikyan has quit IRC17:50
*** derekh has quit IRC17:52
openstackgerritNate Potter proposed openstack/puppet-magnum: puppet-magnum: Initial commit  https://review.openstack.org/24216417:53
*** rahulait has joined #puppet-openstack17:53
*** sergmelikyan has joined #puppet-openstack17:56
iurygregoryhey people, any tips on how to solve this http://logs.openstack.org/21/216821/39/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/bac1a77/console.html#_2015-11-09_17_35_17_406 / https://review.openstack.org/#/c/216821/17:57
*** jistr has quit IRC18:03
degorenkodmsimard, http://logs.openstack.org/21/238521/5/check/gate-puppet-ceph-puppet-beaker-rspec-dsvm-trusty/8c1bbbd/console.html#_2015-11-09_17_52_16_67018:04
degorenkoany thoughts?18:04
degorenkoagain - how it works for centos? \o/18:04
degorenkooh, crap18:07
degorenkothere is only 6.0.0 and 6.1.0 versions18:07
degorenkoin https://forgeapi.puppetlabs.com/#!/release/getReleases18:07
degorenkodmsimard, https://forgeapi.puppetlabs.com/v3/releases?module=openstack%2Fkeystone18:07
dmsimardbefore that it was probably under puppetlabs18:08
dmsimardi.e, puppetlabs/keystone18:08
degorenkoyes18:08
degorenkoyuor're right18:08
degorenkohttps://forgeapi.puppetlabs.com/v3/releases?module=puppetlabs%2Fkeystone18:08
*** xingchao has joined #puppet-openstack18:09
degorenkoalso, as i see - there are no actual tests for centos, that's why it passed18:09
dmsimardoh man, you're right, a skip for them was just merged18:10
dmsimardbecause there was a completely unrelated issue (which we still haven't figured out)18:10
dmsimardmwhahaha and dgurtner can explain that better than I can18:10
degorenkoso, i guess, i need back again to stackforge for spec_helpers18:11
degorenkobecause it probably will not work on 6.0.0 keystone18:11
dmsimardIt looks like we'll need to work on getting the rgw tests fixed entirely -- and up to juno18:11
dmsimardmatter of fact, infernalis is out now so we're due to bump versions anyway18:11
degorenkodmsimard, may be up to kilo at least? :D18:12
degorenkoyou are on juno right now, ist'n it?18:13
*** xingchao has quit IRC18:15
dmsimarddegorenko: well the acceptance tests run off of juno yeah18:16
degorenkodmsimard, so, for my patch, what i should to do? back stackforge for spec_helpers and put some comment or what?18:18
dmsimarddegorenko: yeah, can't touch keystone module in the acceptance tests yet so just leave it be and see if that works18:22
dmsimardit'd be great if I had the time to work out some liberty/infernalis tests18:23
openstackgerritDenis Egorenko proposed openstack/puppet-ceph: Replace all StackForge names to OpenStack  https://review.openstack.org/23852118:24
degorenkodmsimard, ^^18:25
degorenkoi hope - it will passed tests :)18:25
dmsimarddegorenko: let's wait and find out18:25
degorenkoyep ;)18:25
* degorenko afk18:26
*** trown is now known as trown|lunch18:39
*** Hunner has quit IRC18:43
*** Hunner has joined #puppet-openstack18:45
*** Hunner has quit IRC18:45
*** Hunner has joined #puppet-openstack18:45
*** sergmelikyan has quit IRC18:55
*** dhollinger has joined #puppet-openstack19:03
*** ducttape_ has quit IRC19:04
*** rahulait has quit IRC19:05
*** rahulait has joined #puppet-openstack19:13
*** ntpttr1 has joined #puppet-openstack19:15
*** xingchao has joined #puppet-openstack19:15
*** dprince has quit IRC19:18
*** xingchao has quit IRC19:19
*** trown|lunch is now known as trown19:22
*** ducttape_ has joined #puppet-openstack19:26
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/21682119:44
*** angdraug has joined #puppet-openstack19:44
*** dprince has joined #puppet-openstack19:51
*** ntpttr1 has left #puppet-openstack20:22
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/21682120:27
*** ducttape_ has quit IRC20:32
*** sanjayu has joined #puppet-openstack20:36
*** ducttape_ has joined #puppet-openstack20:43
myatsenkospredzy: crinkle: EmilienM:  Guys plz help me with this...20:44
myatsenkoI met with some CI problem, I hope you can help me with it.20:44
myatsenkoI have this patch - https://review.openstack.org/#/c/233011/,20:44
myatsenkoit uses "netaddr" module.20:44
myatsenkoTo pass unit tests in this patch I have added "netaddr" module to Gemfile and20:44
myatsenkoeverything was OK.20:44
myatsenkoBut 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/check/gate-puppet-nova-puppet-beaker-rspec-dsvm-centos7/6590602/console.html.gz)20:44
myatsenkoTo investigate this error I run infrastructure tests (bundle exec rspec spec/acceptance) locally.20:44
myatsenkoTo run this tests locally was asked to install "vagrant", it creates virtual test environment (run VM in my case).20:44
myatsenkoAfter 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
myatsenkoTo solve this error I had to install "netaddr" module in VM that was run by "vagrant". After "netaddr" module20:44
iurygregorymyatsenko, emilien is on vacation this week20:47
crinklemyatsenko: 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 puppet20:47
HunnerUsually I wrap the `require` in a rescue block in the provider and print a helpful message if the load fails20:51
HunnerAnd document it in the readme20:51
iurygregoryhey crinkle, can you help me with a problem that i'm facing using concat?20:52
Hunneriurygregory: What's up?21:00
iurygregoryhey Hunner =)21:00
iurygregoryWell, i need to modify the keystone virtual host to add some extra configurations to use Federation...21:00
iurygregoryThe thing is, the extra configuration (location and locationMatch tags) should be after </virtualhost>21:01
iurygregoryso i'm trying to use concat and concat::fragment but i'm facing some issues21:02
iurygregoryThis is the code: https://review.openstack.org/#/c/216821/21:02
*** sanjayu has quit IRC21:03
iurygregorythis is what i'm trying to do to the virtual host http://paste.openstack.org/compare/473858/473857/21:03
iurygregorylines 30 to 3921:03
*** rahulait has quit IRC21:04
Hunnerlooking21:05
iurygregorytks21:05
Hunneriurygregory: 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
HunnerSo you probably don't want to redeclare the base concat resource; just have your concat::fragment resources add to the end of the existing one21:10
*** ducttape_ has quit IRC21:10
iurygregoryoh so i don't need line 59 to 67?21:11
Hunnerooo21:12
Hunnerin keystone/manifests/wsgi/apache.pp it actually declares keystone_wsgi_admin as a file... still poking through things21:12
iurygregoryyeah =/21:13
Hunnerah, then declares an apache::vhost resource, which is where the concat actually comes in...21:13
Hunneriurygregory: https://github.com/puppetlabs/puppetlabs-apache/blob/master/manifests/vhost.pp#L973 is where </VirtualHost> comes from21:15
HunnerSo if you make your orders > 999 then they would get added after that I think21:15
Hunnerconcat::fragment orders *21:15
iurygregoryoh 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 has joined #puppet-openstack21:16
HunnerI'd give that a go21:17
iurygregoryi'll try :D21:17
iurygregorylike this right? http://paste.openstack.org/show/478393/21:18
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/21682121:19
*** xingchao has quit IRC21:21
*** gfidente|afk has quit IRC21:22
HunnerI'd have thought target would be "${keystone::wsgi::apache::priority}-keystone_wsgi_admin.conf"21:22
*** ducttape_ has joined #puppet-openstack21:23
iurygregoryoh ok =) if fails i'll try with this21:23
iurygregorythank you \o/21:23
Hunnerapache::vhost is not very accessible :(. There needs to be more ways to add custom content21:24
Hunnerwithout having to make explicit concat::fragment resources21:24
iurygregoryyeah =/ should be a variable that we can use XD21:25
Hunnerapache::vhost::extra_content21:25
Hunneror something21:25
HunnerPRs welcome XD21:25
iurygregoryPRs? O.o21:26
Hunneralso apache::vhost::custom_content21:26
Hunnerapache's workflow is github based, so PRs instead of patches21:26
iurygregoryoh thanks for the explanation21:27
iurygregoryHunner, CI fails but now is a different thing  https://jenkins04.openstack.org/job/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/261/consoleFull21:42
iurygregoryCould not find dependent Exec[concat_/etc/apache2/sites-available/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:13221:42
*** jayg is now known as jayg|g0n321:44
Hunneriurygregory: you should try fixing your target :)21:48
iurygregoryok ;)21:48
dmsimarddegorenko: heyyyyyyy success !21:49
HunnerThe exec's title of "concat_/etc..." makes it sound like your absolute path target doesn't match an existing concat resource21:49
iurygregoryI'm switching to target  => "${keystone::wsgi::apache::priority}-keystone_wsgi_admin.conf",21:50
iurygregoryshoul i switch the concat::fragment name?21:50
iurygregoryshould*21:51
Hunnerthe name doesn't matter, as long as it is unique21:51
iurygregoryok21:51
*** trown is now known as trown|outttypeww21:53
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider  https://review.openstack.org/21682121:54
*** rahulait has joined #puppet-openstack21:54
*** dprince has quit IRC22:01
*** nlahouti has joined #puppet-openstack22:02
*** angdraug has quit IRC22:12
*** rgowrishankar has quit IRC22:15
*** angdraug has joined #puppet-openstack22:35
*** dgurtner has quit IRC22:42
*** social has quit IRC22:46
*** delattec has quit IRC22:50
*** social has joined #puppet-openstack22:50
*** jerrygb has quit IRC22:59
*** dhollinger has quit IRC22:59
*** jerrygb has joined #puppet-openstack23:02
*** dfisher has quit IRC23:18
*** cdelatte has joined #puppet-openstack23:21
*** ducttape_ has quit IRC23:32
*** mdorman has quit IRC23:45
« Sunday, 2015-11-08 Index Tuesday, 2015-11-10 »

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