Tuesday, 2015-08-18

*** morazi has quit IRC00:05
*** sergmelikyan has quit IRC00:06
*** ducttape_ has joined #puppet-openstack00:07
*** xarses has joined #puppet-openstack00:13
*** sergmelikyan has joined #puppet-openstack00:15
gildubcrinkle, Hi, yesterday we talked about cherry picking across branches. What about if there are 2 changes (linked) that I would like to carry over? Is picking the second one is going to bring the first one in, I'd assume yes but the commit message is only the last one so I'm not sure. Or do I have to pick both separately, but how to link them then?00:15
crinklegildub: I'm not entirely sure how it works, I don't think cherrypicking just the second one will work00:16
gildubcrinkle, yeah. Ok I'll grab them separately then, I should be able to link them afterwards. Thanks00:17
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Purely cosmetic change on keystone provider  https://review.openstack.org/21393800:17
gildubcrinkle, well, I can't cherry pick the second one anyway, I get an error00:19
crinklegildub: yeah i think since it depends on the first one it can't be cherry-picked cleanly on its own00:20
crinklegildub: would have to wait for the first one to merge00:20
crinklegildub: i'm not sure if there's a better mechanism in gerrit to do that00:21
gildubcrinkle, that's fine, it makes sense00:21
*** aimon has joined #puppet-openstack00:21
gildubcrinkle, but the first one has beaker/upgrade issues, might be related to the liberty branch. The first one is only comestic so I shouldn't break anything00:22
gildubEmilienM, ^00:28
*** ducttape_ has quit IRC00:34
*** tiswanso has joined #puppet-openstack00:38
*** tiswanso has quit IRC00:40
*** ducttape_ has joined #puppet-openstack00:40
*** ducttape_ has quit IRC00:40
*** tiswanso has joined #puppet-openstack00:41
*** ducttape_ has joined #puppet-openstack00:41
*** sergmelikyan has quit IRC00:46
*** ducttape_ has quit IRC00:48
*** sergmelikyan has joined #puppet-openstack00:48
*** sergmelikyan has quit IRC00:48
*** sergmelikyan has joined #puppet-openstack01:02
*** sergmelikyan has quit IRC01:07
vinshI found my bug crinkle and EmilienM.  When I evaluate "$service_name = split($name, 'swift-')" on centos.. it's different then ubuntu.01:09
vinshin centos it yields "[, object-server]"01:09
*** zacdev has joined #puppet-openstack01:09
vinshwhen in ubuntu its just "object-server"01:09
vinshI setup a centos vm and ran the beaker test locally.  Big help :)01:09
*** ducttape_ has joined #puppet-openstack01:10
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/20322001:19
vinshcrinkle: EmilienM: I VOTE that we add --debug ontop of those puppet beaker runs... it's a lot of output.. but it would be a major help in debugging.01:28
vinshMaybe there is a reason we didn't?  I solved my problem by running it locally with debug.. I would have never seen the malformed string interpolation otherwise.01:28
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Endpoints are APIs version-less  https://review.openstack.org/21395701:35
*** imcsk8 has quit IRC01:38
*** imcsk8 has joined #puppet-openstack01:38
crinklevinsh: I think it just makes a ton of output, might make the log a lot harder to read01:38
crinklebut I know this isn't the first time --debug has come in handy for that01:38
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Endpoints without API version and refactored auth  https://review.openstack.org/21252301:39
vinshcrinkle: I suppose maybe the user could add --debug to their commit somehow to debug if needed.  Dunno.01:46
*** angdraug has quit IRC01:58
*** ducttape_ has quit IRC02:02
*** logan2 has quit IRC02:44
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/20322003:33
*** logan2 has joined #puppet-openstack03:34
*** tiswanso has quit IRC03:35
*** markvoelker has quit IRC03:42
*** xingchao has joined #puppet-openstack03:47
*** fedexo has joined #puppet-openstack03:47
vinshnailed it.03:48
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Authentication URLs and endpoint clarity re-factor  https://review.openstack.org/21360304:04
*** gildub has quit IRC04:06
*** fedexo has quit IRC04:22
openstackgerritMerged openstack/puppet-tuskar: Rely on autorequire for config resource ordering  https://review.openstack.org/21191704:29
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Resource keystone_trust for Keystone V3 API  https://review.openstack.org/20099604:33
openstackgerritMerged openstack/puppet-swift: Rely on autorequire for config resource ordering  https://review.openstack.org/21192904:37
*** markvoelker has joined #puppet-openstack04:42
*** markvoelker has quit IRC04:47
*** xingchao has quit IRC04:57
*** _nick has quit IRC05:01
*** _nick has joined #puppet-openstack05:06
*** xingchao has joined #puppet-openstack05:15
*** LimorStotland has joined #puppet-openstack05:40
LimorStotlandHi all05:40
LimorStotlandI need just one more +2 to finish with the puppet-mistral. can anyone form the core team can help? pleas05:41
*** aimon has quit IRC05:46
*** aimon has joined #puppet-openstack05:46
LimorStotlandhttps://review.openstack.org/#/c/208457/ if you want to review :-)05:47
*** sergmelikyan has joined #puppet-openstack06:03
*** serg_melikyan has joined #puppet-openstack06:05
*** sergmelikyan has quit IRC06:05
nibalizerglxgears06:13
nibalizerwhoops06:13
*** gildub has joined #puppet-openstack06:26
*** jtomasek has joined #puppet-openstack06:28
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Update parameters for Sahara  https://review.openstack.org/20598806:33
*** linkedinyou has quit IRC06:40
*** openstackgerrit_ has joined #puppet-openstack06:43
*** markvoelker has joined #puppet-openstack06:43
*** markvoelker has quit IRC06:48
openstackgerritYanis Guenane proposed openstack/puppet-manila: Rely on autorequire for config resource ordering  https://review.openstack.org/21190206:49
*** xingchao has quit IRC06:53
*** rochaporto has joined #puppet-openstack06:57
*** dgurtner has joined #puppet-openstack07:01
*** dgurtner has quit IRC07:01
*** dgurtner has joined #puppet-openstack07:01
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Authentication URLs and endpoint clarity re-factor  https://review.openstack.org/21360307:08
*** dgurtner has quit IRC07:09
*** jpena|away is now known as jpena07:09
openstackgerritGilles Dubreuil proposed openstack/puppet-keystone: Endpoints are APIs version-less  https://review.openstack.org/21395707:09
*** fedexo has joined #puppet-openstack07:22
*** fedexo has quit IRC07:38
michchapLimorStotland: are there no packages so we have to have the systemd unit files in puppet?07:40
openstackgerritYanis Guenane proposed openstack/puppet-openstacklib: Allow the use of an ensure_absent_val param  https://review.openstack.org/20257407:40
LimorStotlandI am not sure i understand your question (I am new at puppet)....there aren't any RPMs (or what ever) for mistral07:42
michchapLimorStotland: yep that was the question, I see in init.pp there is a package {'mistral'} and I would expect the unit files to come from there, not the module07:43
michchapLimorStotland: in fact won't this module fail because that package doesn't exist in rdo/uca?07:44
LimorStotlandmichchap  : I created for myself an rpm for installing the mistral07:44
michchapLimorStotland: are you at redhat or do you know if they have plans to release a mistral rpm in the near future?07:45
LimorStotlandI am not from redhat and as far as i know thy will not support mistral in L07:45
LimorStotlandcan we use the rpm i created for the testing?07:46
michchapLimorStotland: I ask because it's a little odd to have a module that can't be used without building your own rpms. At the very least we should state as such in the readme. I think most users would expect the module to work with RDO at least07:46
michchapLimorStotland: do you have a repo with the rpmspec or instructions for building?07:46
openstackgerritYanis Guenane proposed openstack/puppet-openstacklib: Allow the use of an ensure_absent_val param  https://review.openstack.org/20257407:47
openstackgerritYanis Guenane proposed openstack/puppet-openstacklib: Enable acceptance tests for openstack_config  https://review.openstack.org/21306007:47
michchapLimorStotland: I think you should link to that from the readme so users can try the module - otherwise there is a big assumption that they have already built an appropriate mistral package.07:47
LimorStotlandI have an instructions for building the rpm (i build it using python ./setup.py bdist_rpm) and i can add to the readme07:48
michchapLimorStotland: that would be great. thanks. I will +2 your initial patch and can you please add that as another patch?07:48
LimorStotlandsure Thanks07:49
openstackgerritMerged openstack/puppet-mistral: puppet-mistral: initial commit  https://review.openstack.org/20845707:54
*** paramite has joined #puppet-openstack07:55
*** jistr has joined #puppet-openstack07:55
*** linkedinyou has joined #puppet-openstack07:55
*** social has joined #puppet-openstack07:58
*** gfidente has joined #puppet-openstack08:01
openstackgerritMerged openstack/puppet-horizon: Make redirect type configurable  https://review.openstack.org/20846208:10
*** zacdev has quit IRC08:11
*** derekh has joined #puppet-openstack08:12
*** derekh has quit IRC08:12
*** derekh has joined #puppet-openstack08:13
openstackgerritMerged openstack/puppet-neutron: Update Cisco ML2 classes and templates  https://review.openstack.org/19718108:20
*** IBerezovskiy has joined #puppet-openstack08:25
openstackgerritIvan Berezovskiy proposed openstack/puppet-nova: Add nameservers (dns) parameters  https://review.openstack.org/20278808:25
openstackgerritLimor Stotland proposed openstack/puppet-mistral: Adding instruction for building RPM  https://review.openstack.org/21406908:27
LimorStotlandmichchap I add to the README can you see if this it what you meant https://review.openstack.org/#/c/214069/?Thanks08:28
michchapLimorStotland: thanks ;)08:28
openstackgerritMerged openstack/puppet-trove: Add api ratelimit options  https://review.openstack.org/20039708:33
openstackgerritJavier Peña proposed openstack/puppet-swift: Add support for swift-object-expirer service  https://review.openstack.org/21154908:34
*** markvoelker has joined #puppet-openstack08:45
*** markvoelker has quit IRC08:50
*** aimon_ has joined #puppet-openstack08:55
*** d0ugal has joined #puppet-openstack08:58
*** aimon has quit IRC08:59
*** dgurtner has joined #puppet-openstack09:01
*** sh1kel has quit IRC09:01
*** shikel has quit IRC09:01
*** saneax has joined #puppet-openstack09:02
*** sh1kel has joined #puppet-openstack09:06
*** paramite is now known as paramite|afk09:08
IBerezovskiyhello, everyone. Can you please review small change for nova nameservers https://review.openstack.org/#/c/202788/ ?09:08
*** saneax has quit IRC09:09
*** paramite|afk is now known as paramite09:13
*** fvollero has joined #puppet-openstack09:19
*** fvollero has quit IRC09:19
*** fvollero has joined #puppet-openstack09:20
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Be more specific with CentOS7 package exclude  https://review.openstack.org/21409309:42
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: use full length APT key  https://review.openstack.org/20998109:50
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Drop no longer supported releases  https://review.openstack.org/21410410:01
*** ToMiles has quit IRC10:18
*** saneax has joined #puppet-openstack10:31
openstackgerritMichael Polenchuk proposed openstack/puppet-nova: Activate fping api extension  https://review.openstack.org/21413210:38
*** saneax has quit IRC10:43
*** markvoelker has joined #puppet-openstack10:46
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Fix RGW Apache syslog creation  https://review.openstack.org/21413510:47
*** sh1kel has quit IRC10:48
*** markvoelker has quit IRC10:50
*** saneax has joined #puppet-openstack10:50
openstackgerritSebastien Badia proposed openstack/puppet-neutron: ml2: Fix typo with ml2_srvio/supported_pci_vendor_devs param.  https://review.openstack.org/21413811:01
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Fix RGW Apache syslog creation  https://review.openstack.org/21413511:04
*** saneax has quit IRC11:04
openstackgerritSebastien Badia proposed openstack/puppet-neutron: ml2: Fix typo with ml2_srvio/supported_pci_vendor_devs param.  https://review.openstack.org/21414111:05
*** delattec has quit IRC11:10
*** jpena is now known as jpena|lunch11:31
*** cdelatte has joined #puppet-openstack11:32
*** chem has quit IRC11:37
*** bogdando has quit IRC11:38
*** tchaypo has quit IRC11:39
*** bogdando has joined #puppet-openstack11:40
*** saneax has joined #puppet-openstack11:41
*** markvoelker has joined #puppet-openstack11:46
*** markvoelker has quit IRC11:51
*** serg_melikyan has quit IRC11:51
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Add generic local_mac_map function and ceph_osds class  https://review.openstack.org/21193511:51
*** sergmelikyan has joined #puppet-openstack11:51
*** vinsh has quit IRC12:01
*** vinsh has joined #puppet-openstack12:02
*** dprince has joined #puppet-openstack12:04
*** dprince has quit IRC12:07
*** dprince has joined #puppet-openstack12:07
EmilienMgood morning12:08
openstackgerritSebastien Badia proposed openstack/puppet-neutron: lint: Add and fix missing parameter documentation  https://review.openstack.org/21005912:10
openstackgerritSebastien Badia proposed openstack/puppet-neutron: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/20912212:10
*** jayg|g0n3 is now known as jayg12:11
*** gildub has quit IRC12:11
*** markvoelker has joined #puppet-openstack12:12
EmilienMvinsh: re- logs/swift: yes I thought about that when I did a serie of patches to add debug in beaker jobs - afik puppet-swift does not support logs yet?12:19
*** ducttape_ has joined #puppet-openstack12:20
iurygregorygood morning12:20
*** gildub has joined #puppet-openstack12:27
*** mjblack has quit IRC12:27
*** zacdev has joined #puppet-openstack12:27
*** mjblack has joined #puppet-openstack12:30
*** mjblack has quit IRC12:30
openstackgerritSebastien Badia proposed openstack/puppet-openstack_extras: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987612:31
openstackgerritSebastien Badia proposed openstack/puppet-nova: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987412:31
openstackgerritSebastien Badia proposed openstack/puppet-vswitch: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988312:31
openstackgerritSebastien Badia proposed openstack/puppet-tripleo: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988012:32
openstackgerritSebastien Badia proposed openstack/puppet-openstacklib: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987512:34
openstackgerritSebastien Badia proposed openstack/puppet-tuskar: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988212:36
*** jpena|lunch is now known as jpena12:37
*** ducttape_ has quit IRC12:37
openstackgerritSebastien Badia proposed openstack/puppet-heat: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986812:37
*** ducttape_ has joined #puppet-openstack12:38
*** ducttape_ has quit IRC12:38
openstackgerritSebastien Badia proposed openstack/puppet-glance: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986512:39
*** dgurtner_ has joined #puppet-openstack12:40
openstackgerritSebastien Badia proposed openstack/puppet-trove: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18988112:40
*** gildub has quit IRC12:40
*** mjblack has joined #puppet-openstack12:41
openstackgerritSebastien Badia proposed openstack/puppet-sahara: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987712:41
vinshHey there EmilienM.  Which logs do you mean? for puppet-swift ?12:42
EmilienMvinsh: yeah, when I enabled logs for beaker jobs, I skipped swift because I did not know (for now) how to enable logs12:42
vinshI was asking about the actual puppet beaker run.  Lack of debug there makes it readable.. but not debugable ;)12:42
vinshSo other puppet-modules are collecting openstack logs after beaker runs?12:43
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Add an ability to set up drop_flows_on_start option  https://review.openstack.org/21198612:43
*** dgurtner has quit IRC12:44
EmilienMvinsh: infra script does12:44
vinshCan you point me at an example for another service? I will work it out for swift.12:44
*** saneax has quit IRC12:44
*** dgurtner_ has quit IRC12:44
EmilienMvinsh: https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/copy_puppet_logs.sh12:45
vinshThank you.  I'll ping back when I have something. :)12:45
EmilienMvinsh: https://github.com/openstack/puppet-nova/blob/master/manifests/init.pp#L166-L17312:45
*** dgurtner has joined #puppet-openstack12:45
vinshPerfect, this will work.12:46
EmilienMvinsh: http://logs.openstack.org/12/213512/1/check/gate-puppet-trove-puppet-beaker-rspec-dsvm-centos7/dcdf1c5/logs/12:46
EmilienMhttp://logs.openstack.org/12/213512/1/check/gate-puppet-trove-puppet-beaker-rspec-dsvm-centos7/dcdf1c5/logs/keystone/keystone.txt.gz12:46
EmilienMvinsh: thx a lot12:46
vinshI wonder if we can log puppet debug output to log.. and normal to console12:47
EmilienMvinsh: are we talking about swift logs or puppet logs?12:47
vinshBoth right now.  My initial question yesterday was about puppet logs12:47
vinshbut I am happy to enable swift logs also12:47
EmilienMwell, beaker jobs already have logs12:47
vinshjust not with debug output... which makes failures hard to figure out unless you setup the respective vm locally to test12:48
EmilienMwhile swift nothing12:48
vinshwhich I did for centos12:48
openstackgerritSebastien Badia proposed openstack/puppet-manila: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987212:48
vinshYesterday you two had mentioned it looked like a service had failed to start.. and that was why my 3rd beaker puppet run found changes.. and posted failure.12:49
vinshI had to setup beaker on a centos vm and run puppet in debug there to even figure out the command puppet was running was invalid.12:49
EmilienMspredzy: dropped a question on https://review.openstack.org/#/c/213060/12:49
vinshWhich was a good and useful learning experience in the end :)12:49
openstackgerritSebastien Badia proposed openstack/puppet-gnocchi: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18986612:51
*** sergmelikyan has quit IRC12:56
spredzyEmilienM, looking at https://review.openstack.org/#/c/213060/6/spec/acceptance/openstacklib_config_provider_spec.rb,cm it seems that is what I did first12:57
EmilienMlooking12:58
spredzyI remember an issue with keystone trying to connect to mysql hence I did what I did but I can't refind it in the log ...12:58
spredzyI prob. got confused12:58
EmilienMspredzy: no, even without ::keystone12:58
EmilienMis it possible to just use keystone_config?12:58
EmilienMor do we have scope issue12:58
spredzyEmilienM, not possible we need the package to be installed of the directory  to be created so the _config provider can work12:59
spredzythe provider can create the file but not the directory it has to be in13:00
EmilienMspredzy: yeah the mysql error you got is because you use mysql :)13:00
EmilienMyou should use sqlite13:00
EmilienMspredzy: I suggest managing the directory manually13:00
EmilienMand drop ::keystone13:00
EmilienMit's way more lightweight13:00
EmilienMspredzy: what do you think?13:00
spredzyEmilienM, last week you posted a review telling me you didnt want that13:00
spredzybut to use the package instead13:00
EmilienMspredzy: sorry for that, I realize now we probably add useless tests here13:01
spredzy:(13:01
spredzylet me push it back again13:01
EmilienMspredzy: wait13:01
EmilienMspredzy: what do you think?13:01
EmilienMthe reason why I don't like having all these classes is because it's adding dependencies and complexity while we just want to test a configuration file13:01
spredzyEmilienM, https://review.openstack.org/#/c/209882/4/spec/acceptance/heat_config_spec.rb,cm13:02
spredzyso I should stick to the way it is there right ?13:02
EmilienMI know and I was wrong13:02
EmilienMyes13:02
spredzyok13:02
EmilienMthis is actually the right path I think13:02
spredzyit is the simpler path in fact as it just test the provider logic and nothing else13:03
EmilienMspredzy: even better, add a comment to explain why we do that13:03
spredzyand that is what it was intended for13:03
EmilienMspredzy: cool - thanks for this wor13:03
EmilienMwork*13:03
EmilienMiurygregory: ping - re federation13:04
iurygregoryhi EmilienM13:04
EmilienMiurygregory: I was reading the BP13:04
EmilienMiurygregory: I have a few questions, maybe we can discuss here13:05
iurygregoryok13:05
*** xingchao has joined #puppet-openstack13:05
EmilienMiurygregory: first of all, is keystone wsgi required to run federation?13:05
openstackgerritMerged openstack/puppet-trove: acceptance: enable debug & verbosity for OpenStack logs  https://review.openstack.org/21351213:05
iurygregoryyes ^^13:06
iurygregorywe'll consider just using apache13:06
EmilienMiurygregory: ok so please add it in the blueprint - this is an important information I suppose13:06
openstackgerritMerged openstack/puppet-neutron: acceptance: enable debug & verbosity for OpenStack logs  https://review.openstack.org/21351013:07
EmilienMoh it's mentioned aready13:07
EmilienMiurygregory: my second question is about packages13:07
openstackgerritMerged openstack/puppet-ironic: acceptance: enable debug & verbosity for OpenStack logs  https://review.openstack.org/21350813:08
openstackgerritMerged openstack/puppet-ironic: acceptance: install openstack-selinux on redhat plateforms  https://review.openstack.org/21334813:08
iurygregoryok  go ahead13:08
EmilienMwhen I configure keystone for federation but IDP is external (on a separate node) - do I still need extra packages?13:08
iurygregoryyou mean IDP as "Google" for example? or other Keystone as IDP?13:09
EmilienMiurygregory: yeah, like Google13:09
EmilienMthe actual OpenID server for example13:09
EmilienMnot IDP, but SP13:10
openstackgerritSebastien Badia proposed openstack/puppet-swift: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987813:10
iurygregoryyes o need to install the required packages to make it work, cause the packages aren't installed when you do "apt-get keystone"13:10
EmilienMiurygregory: ok but my question is - do I need the packages even though the openid server is on a separate node, right?13:11
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Add generic local_mac_map function and ceph_osds class  https://review.openstack.org/21193513:11
EmilienMlike libapache2-auth-openidc13:11
openstackgerritSebastien Badia proposed openstack/puppet-swift: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987813:12
iurygregorywait a sec, you mean you have a Keystone in "node2" and "node1" have the openid?13:12
EmilienMiurygregory: yes13:13
iurygregoryyou need to install in the "node2" the openid cause keystone use enviroment variables when doing federated login13:14
EmilienMiurygregory: sorry, it's not clear to me.13:15
EmilienMAre we going to taking care of packages dependencies to install the SP backend?13:16
EmilienMor just to make Keystone working13:16
iurygregorythe first option in my opinion13:16
EmilienMI don't think this is the purpose of puppet-keystone13:17
iurygregoryyou think we only need to add the settings related to the keystone?13:18
iurygregoryif we need to install and change the configuration for "openid" "saml" other module should do that?13:19
*** dfisher has joined #puppet-openstack13:20
*** tiswanso has joined #puppet-openstack13:23
EmilienMiurygregory: puppet-keystone installs & configure keystone service13:24
EmilienMthat means: managing /etc/keystone/*, managing & running keystone service13:24
EmilienMexcept for WSGI run, we manage keystone to work in apache by configuring apache vhost13:24
EmilienMexample: we use MySQL backend to store stuffs - we don't actually configure MySQL13:25
EmilienMso if Federation aims to delegate auth to something else, we don't configure the something else, we just prepare keystone to work with the something else13:25
EmilienMif someone is deploying openid as a service provider, good, we need to use puppet-openid to deploy openid13:25
EmilienM(if there is)13:26
EmilienMiurygregory: does that makes sense?13:26
openstackgerritMerged openstack/puppet-ceilometer: wsgi: fix typos in comments  https://review.openstack.org/21320713:26
iurygregoryyes EmilienM, the configuration for federation that resides in the openid or saml should be in the specific puppet module.13:27
EmilienMiurygregory: this is really important to make the distinction13:27
EmilienMiurygregory: because puppet modules should have a scope limit13:28
iurygregorymake sense EmilienM, i'll update and send other patch13:28
*** sergmelikyan has joined #puppet-openstack13:30
*** saneax has joined #puppet-openstack13:32
*** saikrishna has joined #puppet-openstack13:32
iurygregoryanything else EmilienM ?13:33
*** paramite is now known as paramite|afk13:33
EmilienMiurygregory: yes13:33
EmilienMiurygregory: I'm finisihing the review13:33
iurygregoryok ^^13:33
*** LimorStotland has quit IRC13:33
EmilienMiurygregory: done13:34
iurygregorythanks EmilienM13:34
EmilienMiurygregory: please let me know if something is not clear - we can discuss here anytime13:34
iurygregoryok13:34
*** prad_ has joined #puppet-openstack13:39
openstackgerritSebastien Badia proposed openstack/puppet-tempest: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987913:39
*** ducttape_ has joined #puppet-openstack13:40
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: acceptance: bump to Liberty release  https://review.openstack.org/20929213:46
openstackgerritSebastien Badia proposed openstack/puppet-swift: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987813:47
openstackgerritEmilien Macchi proposed openstack/puppet-heat: acceptance: bump to Liberty release  https://review.openstack.org/20929013:49
dfisherEmilienM: can I ask you to look at a review and help me decide what to do with a comment?13:51
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: acceptance: enable debug & verbosity for OpenStack logs  https://review.openstack.org/21350413:51
openstackgerritYanis Guenane proposed openstack/puppet-openstacklib: Enable acceptance tests for openstack_config  https://review.openstack.org/21306013:51
EmilienMdfisher: shoot13:51
dfisherhttps://review.openstack.org/#/c/211593/13:51
dfisherin patch set 113:51
spredzyEmilienM, ^ (earlier conversation)13:52
dfisherI'm trying to figure out if I should put some of the 'file' entries back in for glance-registry13:52
EmilienMspredzy: ack13:52
dfisherfor db_sync13:52
EmilienMdfisher: oh I see the problem13:53
EmilienMIMHO we should not run db_sync at each File change13:53
EmilienMrather at each package upgrade13:54
dfisheri would agree with you.13:54
EmilienMdfisher: replied13:54
EmilienMspredzy: oh, I asked if we could put a comment to explain why we do that13:55
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Add generic local_mac_map function and ceph_osds class  https://review.openstack.org/21193513:55
dfisherok, so at this point, I think the review should be in good shape then.13:55
dfisherif you want to look at it, I would appreciate it.13:55
EmilienMdfisher: yeah13:55
EmilienMdfisher: after my daily scrum13:55
* dfisher hattip13:55
dfishertake your time :)13:56
*** xingchao has quit IRC13:56
*** sergmelikyan has quit IRC13:56
*** tiswanso has quit IRC14:00
*** sergmelikyan has joined #puppet-openstack14:00
*** tiswanso has joined #puppet-openstack14:01
*** tchaypo has joined #puppet-openstack14:02
*** xingchao has joined #puppet-openstack14:02
openstackgerritYanis Guenane proposed openstack/puppet-openstacklib: Enable acceptance tests for openstack_config  https://review.openstack.org/21306014:10
spredzyEmilienM, forgot about that. here ^14:10
*** paramite|afk is now known as paramite14:12
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: do not create tempest.conf from .sample file  https://review.openstack.org/21078414:13
*** saikrishna has quit IRC14:15
EmilienMspredzy: logs are very interesting14:23
EmilienMhttp://logs.openstack.org/04/213504/3/check/gate-puppet-ceilometer-puppet-beaker-rspec-dsvm-centos7/e2989e5/logs/ceilometer/collector.txt.gz#_2015-08-18_14_07_28_82914:24
EmilienMthis kind of feedback will help us to update parameters14:24
spredzyEmilienM, indeed14:25
spredzyneat14:25
*** prad_ is now known as pradk14:26
EmilienMspredzy: look the manila issue: http://logs.openstack.org/09/213509/2/check/gate-puppet-manila-puppet-beaker-rspec-dsvm-centos7/79a771c/logs/manila/api.txt.gz#_2015-08-18_14_03_59_00514:29
EmilienMspredzy: so I was wrong when I tried to fix orchestration14:30
EmilienMit's really packagin ghere14:30
*** saneax has quit IRC14:31
spredzy:(14:34
EmilienMclayton, mfisch, mgagne, sbadia: any core around can review https://review.openstack.org/#/c/209294/ ?14:39
*** saneax has joined #puppet-openstack14:39
*** saikrishna has joined #puppet-openstack14:42
EmilienMgentle reminder: our weekly meeting will start in 15 min on #openstack-meeting-414:47
*** paramite is now known as paramite|afk14:50
*** angdraug has joined #puppet-openstack14:53
*** saikrishna has quit IRC14:57
iurygregorysorry EmilienM =X if the configuration for "openid" is set in the keystone virtual host file we can use the puppet-keystone right14:57
*** jpena_ has joined #puppet-openstack14:57
*** michchap_ has joined #puppet-openstack14:58
EmilienMiurygregory: right !14:58
iurygregoryok o/14:58
*** dprince has quit IRC14:59
*** jpena has quit IRC14:59
*** bogdando has quit IRC14:59
*** fvollero has quit IRC14:59
*** jtomasek has quit IRC14:59
*** zino has quit IRC14:59
*** colby_ has quit IRC14:59
*** michchap has quit IRC14:59
*** finchd has quit IRC14:59
*** myatsenko has quit IRC14:59
*** saneax has quit IRC14:59
*** bogdando has joined #puppet-openstack15:00
*** zino has joined #puppet-openstack15:00
*** skolekonov has joined #puppet-openstack15:02
*** fvollero has joined #puppet-openstack15:03
*** dprince has joined #puppet-openstack15:12
*** jtomasek has joined #puppet-openstack15:12
*** finchd has joined #puppet-openstack15:12
*** myatsenko has joined #puppet-openstack15:12
*** colby_ has joined #puppet-openstack15:12
*** saikrishna has joined #puppet-openstack15:13
*** dprince has quit IRC15:16
*** paramite|afk is now known as paramite15:25
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/20322015:34
*** tchaypo has quit IRC15:36
openstackgerritMerged openstack/puppet-nova: Add nameservers (dns) parameters  https://review.openstack.org/20278815:38
openstackgerritMerged openstack/puppet-neutron: Allow deployment when DVR is enabled and L2 pop and tunneling are disabled  https://review.openstack.org/20254215:40
openstackgerritMerged openstack/puppet-openstacklib: Allow the use of an ensure_absent_val param  https://review.openstack.org/20257415:43
openstackgerritMerged openstack/puppet-ceilometer: WSGI: make it work, and test it with acceptance  https://review.openstack.org/21314615:43
openstackgerritMerged openstack/puppet-tempest: do not create tempest.conf from .sample file  https://review.openstack.org/21078415:43
openstackgerritSebastien Badia proposed openstack/puppet-tempest: Initial msync run for all Puppet OpenStack modules  https://review.openstack.org/18987915:44
*** sergmelikyan has quit IRC15:51
*** pabelanger has quit IRC15:51
*** rgowrishankar has joined #puppet-openstack15:52
*** saikrishna has quit IRC15:52
*** sergmelikyan has joined #puppet-openstack15:56
rgowrishankarHello, long time lurker, trying to be first time committer.. I work for Nuage Networks and we have puppet configuration for the neutron plugin and puppet configuration for nuage agents. The files will go under puppet-neutron/manifests/plugins/ and puppet-neutron/manifests/agents. It will follow the existing patterns of vendor code (new folder named nuage16:02
rgowrishankarsimilar to the cisco, nvp folders). Do I need a blue print for this or can I create a gerrit review directly or do I need to start a thread on the mailing list?16:02
*** angdraug has quit IRC16:02
EmilienMrgowrishankar: https://github.com/openstack/puppet-neutron/commit/183541d3890cd212db5a581470bdf98c21c40ad7 or https://github.com/openstack/puppet-neutron/commit/4ff68894b3ad537f52c5ec860e8d1a5c7549d19b are good examples16:06
EmilienMrgowrishankar: is nuage ml2 friendly?16:07
*** jistr has quit IRC16:08
*** tchaypo has joined #puppet-openstack16:11
*** aimon_ has quit IRC16:16
*** xingchao has quit IRC16:19
*** jpena_ is now known as jpena|away16:29
*** ilbot3 has quit IRC16:32
*** ilbot3 has joined #puppet-openstack16:33
*** dgurtner has quit IRC16:34
*** dprince has joined #puppet-openstack16:40
*** saikrishna has joined #puppet-openstack16:50
*** aimon has joined #puppet-openstack16:57
*** sergmelikyan has quit IRC17:00
*** saikrishna has quit IRC17:01
*** v1k0d3n has joined #puppet-openstack17:05
*** skolekonov has quit IRC17:10
*** rochaporto has quit IRC17:12
rgowrishankarEmilienM: Nuage is still on a monolithic plugin but there are plans to add ml2 mechanism driver in the near future..17:13
rgowrishankarEmilienM: Thanks for the links, I have followed the existing patterns, will look at these code samples also17:14
*** aimon has quit IRC17:19
*** aimon has joined #puppet-openstack17:20
*** aimon has quit IRC17:20
*** aimon has joined #puppet-openstack17:21
*** angdraug has joined #puppet-openstack17:30
*** sergmelikyan has joined #puppet-openstack17:32
*** jpena|away has quit IRC17:37
*** social has quit IRC17:39
*** xarses has quit IRC17:42
*** dfisher has quit IRC17:47
*** jpena|away has joined #puppet-openstack17:53
vinshIn and rspec beaker upgrade test... what might change on a centos vm.. to cause openstack services to be installed in /bin instead of /usr/bin ?17:56
*** tiswanso has quit IRC18:02
crinkledifferences between kilo and liberty packages?18:03
*** tiswanso has joined #puppet-openstack18:03
vinshI thought so, but other changes have merged in this project without that failure.  So its got to be something silly i'm doing.18:03
vinshIt may have to do with the manner in which I call the init script.  calling it by name vs its path/name18:04
vinshIt might infer /bin if not called from /usr/bin18:04
vinshtesting.18:04
*** xarses has joined #puppet-openstack18:05
EmilienMcrinkle: I know you're traveling, but have you 5 min to talk about integ work with pabelanger?18:05
crinkleEmilienM: i'm at ops midcycle, so in and out18:05
*** gfidente has quit IRC18:05
crinkleEmilienM: i can respond to comments on gerrit?18:06
EmilienMcrinkle: ok np18:06
EmilienMcrinkle: pabelanger is concerned about integration progress, so do I18:06
*** pabelanger has joined #puppet-openstack18:06
vinshcrinkle: Cool, you can see a presentation of all of the 'fun' we had with the kilo upgrade :)18:06
EmilienMcrinkle: pabelanger is mentioning an eventual design error about cloning all modules when testing keystone18:07
*** paramite is now known as paramite|afk18:07
crinkleEmilienM: i don't see any new comments from pabelanger on those patches, pabelanger maybe you can elaborate here?18:07
pabelangerEmilienM: going to pastebin our private conversation, cool?18:08
pabelangersave me some typing18:08
EmilienMpabelanger: sure18:08
*** social has joined #puppet-openstack18:09
pabelangerhttp://paste.openstack.org/show/420816/18:09
pabelangersome initial comments I just did with EmilienM. I was / am planning on composing a ML post for this18:09
pabelangerTL;DR, feel we need a (better?)spec for the puppet-openstack-integration module since we are boncing back and forth on a few points18:10
crinkleso this design that involves installing all the modules on integration tests was discussed at length in -infra, i can try to find logs and the etherpad18:11
*** openstackgerrit_ has quit IRC18:11
EmilienMI probably miss that information but I don't see how it's a blocker for us18:12
EmilienMto clone all modules18:12
pabelangerwell, it is not a blocker, it just create a circular deps on both beaker / integration jobs.18:13
pabelangerbut, perhaps standing back further. Maybe somebody can comment on the status of msync for puppet module. Is the plan to use it or not?  Because a lot of things I am thinking about, really depend on that process.18:14
EmilienMsome people (sbadia, _ody, spredzy) use it but in a manual way18:14
crinkleetherpad where we talked about how to get beaker working with zuul cloner https://etherpad.openstack.org/p/puppet-git-dependencies18:15
EmilienMpabelanger: puppet-openstack-integration should contains some manifests that run in OpenStack infra, end I guess the best way is crinkle's proposal #318:15
EmilienMhaving fixtures/scenario001.pp18:16
EmilienMand just run it18:16
EmilienMcrinkle: thx for the url18:16
pabelangerI'm still not clear on msync. If we are running it manually, does that mean as a project we are good to use it?18:17
EmilienMpabelanger: we have https://github.com/openstack/puppet-modulesync-configs18:17
*** zacdev has quit IRC18:18
pabelangercrinkle: right, I am not talking about breaking zuul-clone support. I am talking but reducing the module depends for each beaker job.  I believe everybody agreed to a Puppetfile for this18:18
pabelangeralong with r10k?18:18
EmilienMwe did18:19
*** chem has joined #puppet-openstack18:19
pabelangerso, if we are using msync, as a project. Why not define each module with a specific Puppetfile to define the dependencies. Breaking the need for beaker jobs to sync with puppet-openstack-integration.18:20
pabelangerwhich then breaks to need to create new beaker gate jobs for puppet-openstack-integration18:20
pabelangersince it is currently failing today, work needs to be done to make it pass18:20
EmilienMit's not failing because of Puppetfile, isn't?18:21
pabelangerno18:21
*** zacdev has joined #puppet-openstack18:21
pabelangerit is failing because puppet-openstack-integration does not have a spec/acceptance file that beaker needs.18:21
pabelangerand the gate-puppet-keystone jobs, excepts puppet-keystone to exists in the local file system.  Which it is not18:22
EmilienMpabelanger: I expected the job to run acceptance from puppet-keystone repo18:22
pabelangerno, that is the issue18:22
pabelangerpuppet-keystone is no where on the filesystem18:22
pabelangernothing is checking that repo out18:22
EmilienMyeah18:22
pabelangerSo, a new gate needs to be added to do that18:23
EmilienMbut what is the relationship with the Puppetfile?18:23
pabelangerthere is none, the gate would need to be updated to then use the local puppet-openstack-integration report to install from18:23
pabelangerwhich, means now updating puppet-keystone to handle this18:23
pabelangerIt is all doable, but we start getting more complex dependencies IMO18:24
EmilienMI just want to run puppetk-keystone beaker job by using ZUUL_BRANCH from the puppet-openstack-integration patchset18:24
pabelangerEmilienM: yes, but how do you install puppet-keystone?18:25
EmilienMpabelanger: good question18:28
pabelangerI know how to fix it.18:29
*** ducttape_ has quit IRC18:29
EmilienM\o/18:29
pabelangeryes and no18:29
pabelangerwill fix this issue, but expose another18:29
crinklepabelanger: as I said last time this came up, it would be good to discuss this in #openstack-infra or at least rope in jeblair and nibalizer since the design for this was worked out with the infra team and they are mimicing our design18:30
pabelangereither way, before going down the path. I feel we need a better spec in place, to make sure everybody understand both sides of it. I'm hoping we can do that at the mid-cycle.18:30
nibalizerI'm about to pop out to lunch18:31
crinkleI was feeling like we were pretty close to having something working, I'd like to continue on that front18:31
nibalizerand I am not at the mid cycle18:31
pabelangercrinkle: right, this is the main reason I'm working _upstream_ now. Because I want -infra to roll the process too18:31
nibalizerbut I will read when I get back18:31
pabelangerlike I said, a lot revolves around msync18:33
pabelangerif we are using it, we can extend our configs to handle some of this dependency18:33
pabelangerhttps://review.openstack.org/#/c/210517/ and https://review.openstack.org/#/c/210528/ are 2 reviews if people don't know18:34
*** chem has quit IRC18:35
crinkleiirc the main reason we moved toward installing from a global list of modules was that we didn't want to hard-code dependencies into the tests and we didn't have a good way of programmatically reading dependencies since they're not all in the metadata.json18:38
crinkleI'm not entirely opposed to hardcoding in dependencies but I would like to work out a way that's compatible with how infra needs to do it18:38
*** sanjayu_ has quit IRC18:39
crinkleI don't think what we're doing now is so broken that we need to call a state of emergency to change it18:39
pabelangerI agree, I don't think hardcording is the way either.18:39
pabelangerI'm just asking for somebody to explain the reason for why puppet-keystone beaker jobs needs puppet-nova to be installed?18:40
*** sanjayu has joined #puppet-openstack18:40
crinkleit doesn't18:40
crinklebut automating the knowledge that it doesn't need puppet-nova was hard18:40
EmilienMit just allows consistency when testing modules18:41
pabelangerNow, this could be my misunderstanding on how msync works. But if we do / can use it, we could setup a template for each module with shared dependencies like mysql and such18:41
EmilienMI don't think this is a blocker to make progress on actual testing jobs18:41
crinklemy general feeling on msync btw http://lists.openstack.org/pipermail/openstack-dev/2015-July/071036.html18:41
crinklei would generally prefer to centralize over copy18:42
nibalizerok im back18:42
*** chem has joined #puppet-openstack18:43
EmilienMpabelanger: other than this issue, wdyt about https://review.openstack.org/#/c/201298/ ?18:43
*** ducttape_ has joined #puppet-openstack18:47
*** chem has quit IRC18:48
nibalizerpabelanger: so I'm having a hard time understanding what the issue is, and what your proposed solution is, can you try to explain it to me?18:48
pabelangerEmilienM: I cannot speak to option 3, as I don't have much experience with that.  For me option 2 makes sense. I agree droping the testing namespace is fine. But my whole understanding of the puppet-openstack-integration module, was it was a puppet module / project (like system-config) that would be used by people to stand up a cloud environment for the purpose of testing / development (using puppet18:49
pabelangermodules [not git]). Again, this goes back to the first set of code I pushed up, which had some transaction layer making things generic and reusable. So, right now, the main issue I have, 6 weeks in of development (which is not much right now) and we still haven't actually committed anything into the project yet. We are still having healty discussion about how this is going to work, but still18:49
pabelangerconfusing. At least on my side18:49
pabelangernibalizer: break beaker-jobs depending on puppet-openstack-integration is the most basic thing.  Either move install_modules.sh out into another module, and both sync / clone from it18:50
pabelangerother wise, we have a circular dependency18:50
pabelangerwhich, when we did our first commit. broke puppet-beaker jobs18:50
EmilienMpabelanger: we never mentioned puppet-openstack-integration was a Puppet module18:50
EmilienMour initial discussions was about putting basic manifest that deploys OpenStack18:51
*** paramite|afk is now known as paramite18:52
nibalizerthe integration repo seems like the correct place to keep shared tests, scripts to help (install_modules.sh), and the cannonical list of dependencies18:52
EmilienMpabelanger: the '6 weeks' things is only because we are not yet agree. We could have our basic structure in place18:53
crinklepabelanger: it is specifically not a puppet module, we do not want people to use as such18:53
EmilienMif people wants to run tests, they use beaker tests on their laptop18:53
*** ducttape_ has quit IRC18:54
nibalizerso install_modules.sh is just a script, I'm not sure why we would move that18:54
EmilienMI would be in favor or doing #3 now, have the structure in place - and get the feedback at usage (and adapt the design accordingly)18:54
nibalizerand It sound slike you want a Puppetfile per module, which I don't think is a good idea18:54
*** ducttape_ has joined #puppet-openstack18:55
pabelangercan you explain why it is not a good idea?18:56
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/20322018:56
crinklepabelanger: isn't that the same as hardcoding dependencies, which you said above is not what you want?18:57
pabelangerI don't understand what you mean hardcodeding dependencies.18:57
*** tiswanso has quit IRC18:57
pabelangerbecause the dependency exists in the puppet module, not out side?18:57
nibalizerpabelanger: https://github.com/puppet-community/puppet-module-puppetboard/pull/26 is what I ran into18:58
nibalizerlibrarian-puppet recurses puppetfiles I guess? and everything gets pretty broken18:58
pabelangerRight, in the past I used librarian-puppet-simplke18:59
pabelangeror what ever it was called18:59
* crinkle afk for lunch18:59
nibalizerso at a /really/ high level: puppet is shit at dependencies18:59
*** angdraug has quit IRC18:59
nibalizerthere aren't any tools that do it well, the standards for experessing dependncies are bad18:59
nibalizerthere isn't runtime inspection18:59
nibalizerso simply writing down a list of what you need, in one place, is about as good as we are going to do19:00
*** tiswanso has joined #puppet-openstack19:00
pabelangerI agree, I am just suggesting we move that location to msync19:00
nibalizerwhen users come to this channel like 'hey does X version of mysql work?' we can point them to one file in puppet-integration and that's the end of it19:00
pabelangerwe then either sync all projects against it19:00
*** zacdev has quit IRC19:00
pabelangeror update clones to pull from it19:00
pabelangereither way, it breaks our beaker-jobs / integration dependency19:01
pabelangerlook at it this way19:01
pabelanger_what_ if we needed to update the acceptance today to change how install_modules works19:01
pabelangerhow are we expecting to do that with out breaking our gates19:02
nibalizer?19:02
nibalizerinstall_moudles.sh is in puppet-openstack-integration so we would change it there19:02
pabelangerwe moved it into tools19:02
pabelangerfor example19:02
pabelangeror changed the file name19:03
nibalizeryou'd have to change spec_helper_acceptance in every module19:03
nibalizerwhich would be like 40 reviews19:03
nibalizerwe've discussed moving the blob in s_h_a to a centralized repo or ruby gem19:04
pabelangerwe couldn't even do that today. If puppet-keystone beaker-job was a gate for puppet-openstack-integration19:04
pabelangerthat is the circular dep19:04
pabelangerit is the same issue we have migrating to puppet-httpd and puppet apply jobs19:04
nibalizerright and thats the one time you're alloowed to ask infra for help unwedging you19:04
nibalizerif you have an asymetric but wedged gate19:04
pabelangerRight19:05
nibalizerbut install_modules.sh is 40 lines19:05
pabelangerbut again, this all goes back to msync19:05
nibalizerand I don't really expect to have to change it ever19:05
nibalizermaybe a little bit19:05
pabelangerif we are using it, there is a way to get around the wedge19:05
nibalizerbut not commonly19:05
nibalizerI think you're conflating 'growing pains' with 'fundamental flaw'19:05
nibalizeryes getting off the ground is hard19:06
nibalizerpeople need to learn the new system, it has some unexpected behaviors19:06
nibalizerbut it can work most of the time once it gets going19:06
nibalizerand msying every repo every time just creates review churn and slows the project down19:06
EmilienMthe msync thing just does not scale when having tons of modules I would say19:07
pabelangerI'm not saying it is a fundamental flaw, I am saying there might be a better way to do some of this. But more to the point, it looks like I'm not on the same page as most people here. Which is fine, but causing me fustrations due to things slowing down more then I am used too19:08
pabelangerTo me, it is a clear plan in my head how it works. Take devstack, rewrite it using puppet.19:09
pabelangersomething I basically did already 18 months ago. So, I am clinging to something in the past. Re-writing it to work different is a hard thing for me to grasp right now. Mostly because I don't know how other people see all this working19:11
pabelangerthe main reason I signed up to attend mid-cycle19:11
*** ianw has quit IRC19:11
nibalizerthe mid-cycle is now virtual ya?19:11
EmilienMnibalizer: y19:12
pabelangeris, but MTL is 90mins away19:12
EmilienMfrom 2 to 419:12
EmilienMnibalizer: paul and I will meet in MTL19:12
nibalizerok19:13
*** ianw has joined #puppet-openstack19:13
nibalizerpabelanger: I believe the initial impetus for puppet-openstack-integration was only to centralize the Puppetfile and install_modules.sh19:15
EmilienMand then a basic manifest that is run to validate the whole stack19:16
EmilienMand then run tempest19:16
*** jpena|away has quit IRC19:18
nibalizeruh sure19:19
pabelangerwell, that is part of the confusion. I don't think everybody understand what the repo will do19:19
nibalizerya19:20
pabelangerso, I think for now, I'm going to step aside on work until the mid-cycle then see if I can get some spec in place.  Because, for the most part, I am just spining my wheels right now.19:20
nibalizerso a commit adding a README.md to that repo could be a thing19:20
nibalizeror a spec19:20
nibalizerpabelanger: ok19:20
nibalizerright now I am not seeing the keystone tests gating puppet-openstackci19:21
nibalizerer puppet-openstack-integration, sorry19:21
nibalizerhttps://review.openstack.org/#/c/207070/ for example19:21
*** paramite has quit IRC19:21
pabelangernibalizer: https://review.openstack.org/#/c/213502/19:22
nibalizerso thats a misconfigure19:24
nibalizerright so its called the keystone job19:24
nibalizerbut its just trying to run bundle exec rspec spec/acceptance in the puppet-openstack-integration repo19:24
nibalizerwhich won't work19:24
pabelangercorrect19:24
nibalizerbasically the beaker jobs would have to get smarter19:25
pabelangerwhat they want it to do, is checkout puppet-keystone, then have it run keystone acceptance19:25
nibalizeror a new job could be created that is smart enough to run the spec tests that it is assigned19:25
nibalizeris there a patch up to project-config to pull out the keystone test?19:27
EmilienMno19:27
nibalizerokay I will spin that up then19:27
nibalizerits a good idea to do that I think19:27
EmilienMI just don't know how to do that19:27
nibalizerbut it simply can't ever work19:27
nibalizeras written19:27
EmilienMnibalizer: yeah I did that patchn you +1'ed and it failed19:27
nibalizerya19:28
nibalizergo us19:28
nibalizer:)19:28
EmilienM:P19:28
EmilienMahah19:28
nibalizerI didn't realize what I now know19:28
nibalizerwhich is that test is more adaptive than descriptive19:28
nibalizercan you propose the patch to undo it, or propose a revert19:28
EmilienMnibalizer: ok19:29
EmilienMnibalizer: https://review.openstack.org/#/c/213582/19:30
EmilienMnibalizer: look how I'm fast, I came back in the past and did it19:30
nibalizeroh okay it was already done19:30
nibalizersweet19:30
EmilienMnibalizer: my time machine is up!19:31
EmilienMnibalizer: what kind of patch do you think?19:32
*** angdraug has joined #puppet-openstack19:32
nibalizerEmilienM: ?19:32
EmilienMpabelanger: do that mean we won't push code in integ repo before midcycle??19:32
EmilienMnibalizer: for project-config19:32
EmilienM"I will spin that up then"19:33
nibalizerI was going to spin up 21358219:34
nibalizerbut you alraedy did it19:34
pabelangerEmilienM: Ya, until I have a better understanding what people want to use the repo for.19:35
EmilienMpabelanger: I think and hope we can solve this issue *before* the midcycle, honestly19:36
EmilienMwe were so close to have something19:36
EmilienMtempest is running etc19:36
* crinkle is back19:37
crinkleI agree, I thought we were generally on the right track, I'm not sure what postponing will do19:37
pabelangerSo, if we merge the code today19:39
pabelangerwhat are we going to do with the repo?19:39
EmilienMrun the integ jobs in puppet-keystone as non voting19:41
EmilienMthis is an idea^19:41
EmilienMsee how it behaves and iterate19:41
EmilienMcrinkle, pabelanger, nibalizer ^19:43
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Enabling-federation  https://review.openstack.org/19036119:44
pabelangerright, but currently we have coverage from beaker-jobs, since it is a copy and paste for the most part19:46
EmilienMpabelanger: for now19:46
EmilienMpabelanger: once we have this merged, I'll push glance support19:46
EmilienMand run tempest glance19:47
EmilienMand add glance to puppe-glance19:47
EmilienMant so on19:47
EmilienMat the end, we have a full OpenStack cloud19:47
pabelangerEmilienM: well, I think we should address the apt-get problems we are having first19:47
pabelangersince the jobs are bascially bouncing19:47
EmilienMthe apt-get problem?19:47
pabelangererror 100 from apt-get?19:47
EmilienMwhich error sorry, I feel dumb here19:47
pabelangerhttp://logs.openstack.org/70/207070/12/check/gate-puppet-openstack-integration-dsvm-trusty/f5bf4b6/console.html.gz#_2015-08-14_19_06_42_89219:48
pabelangerright now, it is unstable19:48
EmilienMhttp://logs.openstack.org/70/207070/13/check/gate-puppet-openstack-integration-dsvm-trusty/8221fda/console.html#_2015-08-17_16_06_17_85719:49
EmilienMwe're using this puppet code everywhere in our acceptance19:49
EmilienMand it never failing like this19:49
EmilienMI think this is a random, rare issue19:49
pabelangerstill needs addressed no?19:50
EmilienMsure but not a blocker for us19:50
EmilienMI closely watch beaker jobs and I saw that once19:50
EmilienMit's less problematic than EPEL downtimes19:50
EmilienMthat happen every day by the way19:50
pabelangerYa, so if somebody want to take over my patchsets for integration, I'll stand aside. I have some other work with -infra I need to finish up in the next 2 weeks.19:56
pabelangerOnly other questions are for msync. If we are _not_ going to use it, then I'll drop https://review.openstack.org/#/c/210517/19:57
pabelangerhowever, I would not be in favor of people manually running it them19:57
pabelangersince the patch set just automates the process19:57
EmilienMpabelanger: me neither, I would like automation too19:59
*** linkedinyou has quit IRC20:02
EmilienMpabelanger: I'll take over patches20:11
iurygregorythanks EmilienM \o/20:11
EmilienMiurygregory: cool, thx - I think it's good now, I might have missed something20:11
iurygregoryNo problem ^^20:12
iurygregorycrinkle, if you have time you can review https://review.openstack.org/190361 ?20:12
pabelangerEmilienM: crinkle: I am currently looking at: https://review.openstack.org/#/c/207070/13/modules/openstack_integration/manifests/testing/scenario001.pp20:15
iurygregoryThere is a way to verify if keystone is running over apache?20:15
pabelangerif we keep this as a simple manifest (option 3). How would we move this common logic out into scenario002, if we wanted?20:15
EmilienMpabelanger: copy/paste for now I guess20:16
pabelangers/into/share with20:16
pabelangerEmilienM: okay, other then copy and paste20:16
EmilienMpabelanger: common.pp + include?20:16
EmilienMiurygregory: I think yeah, you can check the vhost resource in the catalog20:17
iurygregoryok i'll keep this in mind =)20:17
pabelangerEmilienM: would puppet know how to resolve common.pp if we moved it into fixtures/ ?20:18
EmilienMfor now, no20:18
pabelangerI'd be in favor of condensing to manifest folder for this purpose20:19
crinklethat wouldn't help, it would still need to be in a modulepath or add .. to the modulepath20:21
crinkleso that is a reasonable argument for having that directory layout20:22
EmilienMcrinkle: having fixtures is reasonable?20:22
pabelangerright. I don't know any other way, other then keeping puppet module structure, to use the include functionality20:23
crinkleEmilienM: no I'm saying pabelanger's directory layout is reasonable20:23
crinklethat was something I hadn't considered20:23
crinkleI asked for the reason on the review, this is a good answer to me20:24
EmilienMpabelanger, crinkle: so do we agree to have manifests/scenario001.pp and manifests/common.pp ?20:25
EmilienMerr, modules/openstack_integration/manifests/scenario001.pp and common.pp20:26
pabelangerWhat ever layout allows use to do include ::common from scenario001.pp20:26
pabelangerthe current was just 1 example way to do it20:26
EmilienMpabelanger: are you git pushing it?20:27
pabelangerEmilienM: no, in meeting right now20:27
EmilienMpabelanger: ok20:27
crinkleso if we do it that way we still need some kind of common manifest that includes the class, because just defining the class isn't enough20:28
*** jayg is now known as jayg|g0n320:28
EmilienMcrinkle: having modules/openstack_integration/manifests/common.pp and modules/openstack_integration/manifests/scenario001.pp that includes it is not enough?20:28
crinkleEmilienM: we could puppet apply -e "include scenario001" if we don't anticipate having any parameters20:30
crinkleEmilienM: but puppet apply modules/openstack_integration/manifests/scenario001.pp will have no effect20:30
pabelangerYa, this was another reason for using --certname for the node, and including the manifest20:31
openstackgerritMerged openstack/puppet-ceilometer: acceptance: enable debug & verbosity for OpenStack logs  https://review.openstack.org/21350420:31
EmilienMpabelanger, crinkle: honestly, I don't think we should block on that now. If we have a second scenario later, we will just change the folder layout if needed20:37
EmilienMI don't see major issue here20:37
EmilienMwe might want a first scenario up and running and see how it works20:38
vinshafter hours of riding the struggle buss.  I find that centos sets /bin before /usr/bin in its path.. thus busting my code.20:38
EmilienMthat's why we have iteration after all20:38
crinkleEmilienM: sounds good to me, we can always fix later20:38
EmilienMdesign is not really critical here20:38
EmilienMit's not like designing an API or...20:38
EmilienMvinsh: stay with us20:38
vinsh;) deep breath20:39
EmilienMpabelanger: wdyt?20:39
*** angdraug has quit IRC20:41
*** angdraug has joined #puppet-openstack20:41
EmilienMI vote for having fixtures/scenario001.pp now, make it work, run tempest, merge it, gate puppet-keystone with that as non voting and see how it behaves20:42
EmilienMthen, during midcycle, pabelanger and I will see how we can scale this out and solve our issues20:43
EmilienMbut at least we will have great feedback in the meantime20:43
EmilienMI don't want to wait until september to have a job workin20:43
pabelangerEmilienM: rather fix it now, other then kicking it down the road. But that is just me. Like I said, going on sidelines until summit, so feel free to continue on your changes.20:45
*** dgurtner has joined #puppet-openstack20:49
*** pabelanger has quit IRC20:51
*** derekh has quit IRC20:52
*** cdelatte has quit IRC20:57
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Create puppet folder layout  https://review.openstack.org/20129820:58
EmilienMcrinkle: ^20:58
vinshI need to find a way to set the PATH variable used in a puppet run.  Anyone messed with something like that before?21:11
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI)  https://review.openstack.org/20707021:16
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Create puppet folder layout  https://review.openstack.org/20129821:17
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI)  https://review.openstack.org/20707021:18
*** nigelk has quit IRC21:20
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/20322021:20
*** tiswanso has quit IRC21:25
*** ducttape_ has quit IRC21:26
*** ducttape_ has joined #puppet-openstack21:27
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Create puppet folder layout  https://review.openstack.org/20129821:41
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI)  https://review.openstack.org/20707021:42
openstackgerritMerged openstack/puppet-neutron: Prepare Liberty release  https://review.openstack.org/20929421:44
*** aimon has quit IRC21:48
*** tiswanso has joined #puppet-openstack21:50
*** dprince has quit IRC22:02
*** tiswanso has quit IRC22:03
*** tiswanso has joined #puppet-openstack22:05
*** tiswanso has quit IRC22:10
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: configure and run tempest  https://review.openstack.org/20707822:10
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: configure and run tempest  https://review.openstack.org/20707822:11
*** dgurtner has quit IRC22:11
*** angdraug has quit IRC22:14
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: deploy MySQL and Keystone (WSGI)  https://review.openstack.org/20707022:15
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: configure and run tempest  https://review.openstack.org/20707822:16
*** aimon has joined #puppet-openstack22:19
*** pradk has quit IRC22:24
*** chem has joined #puppet-openstack22:34
*** ducttape_ has quit IRC22:50
*** tiswanso has joined #puppet-openstack23:06
*** tiswanso has quit IRC23:18
*** zacdev has joined #puppet-openstack23:19
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP: Introduce swift_service class  https://review.openstack.org/20322023:28
*** xingchao has joined #puppet-openstack23:32
*** xarses has quit IRC23:48
*** xingchao has quit IRC23:49
*** sergmelikyan has quit IRC23:55
*** sergmelikyan has joined #puppet-openstack23:59

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