Monday, 2015-06-22

*** ducttape_ has joined #puppet-openstack00:00
*** markvoelker has joined #puppet-openstack00:01
*** gildub has joined #puppet-openstack00:02
*** markvoelker has quit IRC00:05
*** ducttape_ has quit IRC00:10
*** ducttape_ has joined #puppet-openstack00:13
*** ianw has quit IRC00:23
*** ianw has joined #puppet-openstack00:24
*** ferest has quit IRC00:39
*** ducttape_ has quit IRC00:40
*** gildub has quit IRC00:42
*** ducttape_ has joined #puppet-openstack00:52
*** ducttape_ has quit IRC00:52
*** gildub has joined #puppet-openstack01:07
*** fedexo has joined #puppet-openstack01:49
*** markvoelker has joined #puppet-openstack01:50
*** markvoelker has quit IRC01:55
*** stevemar has joined #puppet-openstack01:57
*** linkedinyou has quit IRC02:06
*** ducttape_ has joined #puppet-openstack02:07
*** ducttape_ has quit IRC02:12
*** ducttape_ has joined #puppet-openstack03:08
*** ducttape_ has quit IRC03:13
*** xingchao has joined #puppet-openstack03:21
*** ducttape_ has joined #puppet-openstack03:22
*** xingchao has quit IRC03:27
*** markvoelker has joined #puppet-openstack03:38
*** ducttape_ has quit IRC03:41
*** markvoelker has quit IRC03:43
gildubrichm, hi - do you know if beaker issue with user_role has been resolved, I can't reproduce it. I'm using 174999/2504:44
*** chandankumar has joined #puppet-openstack04:52
gildubrichm, never mind, now reading the channel logs04:55
*** stevemar has quit IRC05:19
*** markvoelker has joined #puppet-openstack05:27
*** rushiagr_away is now known as rushiagr05:27
*** markvoelker has quit IRC05:32
*** richm has quit IRC05:35
*** ducttape_ has joined #puppet-openstack05:43
*** ducttape_ has quit IRC05:48
*** magicboiz has quit IRC06:17
*** arnaud_orange has joined #puppet-openstack06:44
*** chandankumar_ has joined #puppet-openstack06:47
*** chandankumar has quit IRC06:47
*** fedexo has quit IRC06:51
*** chandankumar_ is now known as chandankumar06:53
*** jpena has joined #puppet-openstack07:01
*** fedexo has joined #puppet-openstack07:03
*** markvoelker has joined #puppet-openstack07:16
*** saneax has joined #puppet-openstack07:17
*** markvoelker has quit IRC07:21
*** ChanServ sets mode: +o EmilienM07:29
*** chandankumar has quit IRC07:31
*** ducttape_ has joined #puppet-openstack07:32
*** xingchao has joined #puppet-openstack07:33
*** dgurtner has joined #puppet-openstack07:35
*** ducttape_ has quit IRC07:36
*** chandankumar has joined #puppet-openstack07:40
*** gfidente has joined #puppet-openstack07:41
*** gfidente has quit IRC07:41
*** gfidente has joined #puppet-openstack07:41
*** fedexo has quit IRC07:44
*** cwolferh has quit IRC07:45
*** saneax has quit IRC07:46
*** gildub has quit IRC07:48
*** xingchao has quit IRC07:52
*** cwolferh has joined #puppet-openstack07:52
*** tshtilma has joined #puppet-openstack08:07
*** wabu has quit IRC08:13
*** wabu has joined #puppet-openstack08:14
*** paramite has joined #puppet-openstack08:15
*** saneax has joined #puppet-openstack08:33
openstackgerritDavid Gurtner proposed stackforge/puppet-ceph: Fix dependency version  https://review.openstack.org/19386908:38
*** __nick is now known as _nick08:43
*** derekh has joined #puppet-openstack08:44
*** dgurtner has quit IRC09:02
*** linkedinyou has joined #puppet-openstack09:02
*** markvoelker has joined #puppet-openstack09:05
*** markvoelker has quit IRC09:09
*** ducttape_ has joined #puppet-openstack09:20
*** dgurtner has joined #puppet-openstack09:22
*** dgurtner has quit IRC09:22
*** dgurtner has joined #puppet-openstack09:22
*** ducttape_ has quit IRC09:25
*** richm has joined #puppet-openstack10:03
*** chandankumar has quit IRC10:09
*** cdent has joined #puppet-openstack10:11
*** chandankumar has joined #puppet-openstack10:15
*** Guest11697 has quit IRC10:43
*** dweaver has quit IRC10:43
*** zigo has quit IRC10:43
*** khappone has quit IRC10:43
*** partner has quit IRC10:43
*** dweaver has joined #puppet-openstack10:43
*** mgagne has joined #puppet-openstack10:43
*** partner has joined #puppet-openstack10:43
*** mgagne is now known as Guest8120210:43
*** zigo has joined #puppet-openstack10:44
*** khappone has joined #puppet-openstack10:44
*** markvoelker has joined #puppet-openstack10:53
*** markvoelker has quit IRC10:58
openstackgerritYanis Guenane proposed openstack/puppet-neutron: Opencontrail: Create neutron::plugins::opencontrail  https://review.openstack.org/18798611:03
*** linkedinyou has quit IRC11:04
*** linkedinyou has joined #puppet-openstack11:06
*** ducttape_ has joined #puppet-openstack11:09
*** gildub has joined #puppet-openstack11:13
*** ducttape_ has quit IRC11:14
openstackgerritYanis Guenane proposed openstack/puppet-neutron: Opencontrail: Create neutron::plugins::opencontrail  https://review.openstack.org/18798611:22
*** cdelatte has joined #puppet-openstack11:23
*** xingchao has joined #puppet-openstack11:28
*** ducttape_ has joined #puppet-openstack11:32
*** xingchao has quit IRC11:35
*** xingchao has joined #puppet-openstack11:51
*** ducttape_ has quit IRC11:53
*** markvoelker has joined #puppet-openstack11:54
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Fix support for puppet4  https://review.openstack.org/19409711:55
*** rcallawa has quit IRC11:57
*** markvoelker has quit IRC11:59
*** dprince has joined #puppet-openstack12:00
*** markvoelker has joined #puppet-openstack12:03
*** todin has left #puppet-openstack12:05
EmilienMgood morning!12:10
*** xingchao has quit IRC12:19
*** jayg|g0n3 is now known as jayg12:24
*** ducttape_ has joined #puppet-openstack12:25
*** ducttape_ has quit IRC12:27
*** iurygregory has joined #puppet-openstack12:31
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-specs: Enabling-k2k-federation  https://review.openstack.org/19036112:35
*** rcallawa has joined #puppet-openstack12:44
*** jpena is now known as jpena|lunch12:46
*** morazi has quit IRC12:47
*** gildub has quit IRC13:00
*** rushiagr is now known as rushiagr_away13:10
*** xingchao has joined #puppet-openstack13:19
*** pradk_ is now known as prad13:24
trashIt's not just you! http://review.openstack.org looks down from here.13:24
trash:-(13:24
openstackgerritMerged openstack/puppet-nova: Better handling of package dependencies in nova generic_service  https://review.openstack.org/19200913:26
*** morazi has joined #puppet-openstack13:27
*** saneax has quit IRC13:37
*** zigo has quit IRC13:47
*** xingchao has quit IRC13:48
*** rushiagr_away is now known as rushiagr13:49
*** jpena|lunch is now known as jpena13:56
*** magicboiz has joined #puppet-openstack13:57
openstackgerritStefan Andres proposed openstack/puppet-glance: Support identity_uri  https://review.openstack.org/19415014:00
*** stevemar has joined #puppet-openstack14:04
*** zigo has joined #puppet-openstack14:05
*** dfisher has joined #puppet-openstack14:09
*** chandankumar has quit IRC14:10
openstackgerritGiulio Fidente proposed openstack/puppet-nova: Add parameter to control use of rbd for the ephemeral storage  https://review.openstack.org/11909314:12
dfisherEmilienM:  thanks for the explanation on the revert for my cset for puppet-keystone.  Working now on puppet-cinder14:16
EmilienMdfisher: cool14:16
dfisherapologies for not catching that one in keystone14:17
dfisherlooks like cinder is clear after cleaning up db/sync.pp:14:17
dfisherdfisher at rawr-3 in /export/home/dfisher/puppet-cinder-1458915/manifests14:17
dfisher$ grep cinder_conf **/*(.) | grep -v cinder_config14:17
dfisherdfisher at rawr-3 in /export/home/dfisher/puppet-cinder-1458915/manifests14:17
openstackgerritDrew Fisher proposed openstack/puppet-cinder: Remove POSIX users, groups, and file modes.  https://review.openstack.org/19318914:18
*** chem has joined #puppet-openstack14:28
*** sergmelikyan has joined #puppet-openstack14:31
*** ferest has joined #puppet-openstack14:34
*** dtalmage has joined #puppet-openstack14:34
*** mfisch` is now known as mfisch14:36
*** mfisch is now known as Guest8229014:36
*** xingchao has joined #puppet-openstack14:38
*** ferest has quit IRC14:39
*** sergmelikyan has quit IRC14:40
*** sergmelikyan has joined #puppet-openstack14:41
*** sergmelikyan has quit IRC14:42
*** sergmelikyan has joined #puppet-openstack14:43
*** tshtilma has quit IRC14:46
*** sergmelikyan has quit IRC14:47
*** paramite is now known as paramite|lung14:47
*** paramite|lung is now known as paramite|lunch14:47
*** ducttape_ has joined #puppet-openstack14:49
openstackgerritJason Guiditta proposed openstack/puppet-neutron: Add manage_service parameter to all agents::ml2 classes that did not yet have it.  https://review.openstack.org/19417714:53
*** IBerezovskiy has quit IRC14:58
*** xingchao has quit IRC15:04
dmsimardgfidente: Let's add the rbd ephemeral patchset to tomorrow's meeting if it hasn't merged by then :)15:06
gfidentedmsimard, fine for me, meeting is at same time of now, tomorrow right?15:07
dmsimardYeah, just about15:07
dmsimardFWIW we're working with that patchset in our environment - works well and passes integration tests15:08
*** paramite|lunch is now known as paramite15:09
*** rcallawa has quit IRC15:11
*** mdorman has joined #puppet-openstack15:12
*** sergmelikyan has joined #puppet-openstack15:13
*** paramite has quit IRC15:15
*** morazi has quit IRC15:15
*** morazi has joined #puppet-openstack15:21
*** paramite has joined #puppet-openstack15:21
*** magicboiz has quit IRC15:30
openstackgerritMike Dorman proposed openstack/puppet-nova: Add scheduler_driver parameter to nova::scheduler class  https://review.openstack.org/19360915:33
*** sergmelikyan has quit IRC15:33
*** bklei has joined #puppet-openstack15:35
*** Guest81202 is now known as mgagne15:36
*** mgagne has joined #puppet-openstack15:36
*** chandankumar has joined #puppet-openstack15:36
openstackgerritMike Dorman proposed openstack/puppet-nova: Add api max limit and instance naming parameters  https://review.openstack.org/19332815:39
*** linkedinyou has quit IRC15:39
*** _ryanv_ has joined #puppet-openstack15:46
openstackgerritBradley Klein proposed openstack/puppet-monasca: Update vertica projections for better performance  https://review.openstack.org/19317515:56
dfisherso I updated to the 5.1.0 commits and now I'm starting to see:15:56
dfisherNotice: Puppet::Provider::Openstack: project service is unavailable.15:56
dfisherWill retry for up to 10 seconds.15:56
dfisherError: /Stage[main]/Keystone_users/Keystone_user[glance]: Could not15:56
dfisherevaluate: Execution of '/usr/bin/openstack token issue --format value'15:56
dfisherreturned 1: ERROR: openstack 'tenant_id'15:56
dfisherI'm clearly missing a module … but I'm not sure which one15:56
*** rcallawa has joined #puppet-openstack15:57
dfisheri think I see it in openstacklib .. right?16:01
dfisherbut I have the 5.1.0 commit of that module too16:01
richmdfisher: hmm - not sure where tenant_id is coming from16:03
richmdfisher: it should be calling token issue with OS_AUTH_URL, OS_PASSWORD, OS_USERNAME, and OS_PROJECT_NAME16:04
*** morazi has quit IRC16:05
richmdfisher: you could try moving /usr/bin/openstack to /usr/bin/openstack.orig, and create a shell script wrapper /usr/bin/openstack that logs the arguments, the env, and the output of calling openstack.org16:06
mdormando you have a kilo version of openstackclient ?16:06
richmat least 1.0.2 or later16:06
richmpreferably 1.0.316:06
dfisheri grabbed the 5.1.0 tag for openstacklib16:06
dfisherand16:06
dfisher$ pkg list openstackclient16:06
dfisherNAME (PUBLISHER)                                  VERSION                    IFO16:06
dfisherlibrary/python/openstackclient                    1.0.1-5.12.0.0.0.77.0      i-16:06
dfisherso, that might be it16:06
*** paramite has quit IRC16:06
dfisherbut is 1.0.2/1.0.3 suitable for Juno?16:07
dfisherfrom16:07
dfisherhttps://github.com/openstack/requirements/blob/stable/juno/global-requirements.txt16:07
dfisherpython-openstackclient>=0.3.0,<=1.0.116:07
dfisherdoesn't look like it ..16:08
dfisherhttps://launchpad.net/python-openstackclient/+series doesn't mention juno or 1.0.216:10
*** Guest82290 is now known as mfisch16:10
*** mfisch has quit IRC16:10
*** mfisch has joined #puppet-openstack16:10
*** richm has quit IRC16:13
*** rcallawa has quit IRC16:14
*** aimon has quit IRC16:16
*** chandankumar has quit IRC16:16
*** pleia2_ is now known as pleia216:20
*** jpena has quit IRC16:21
*** xarses_ has quit IRC16:21
dfisherok, 1.0.2 fails in the same way.16:25
EmilienMdmsimard: I don't see any agenda topic for that, can I help with that?16:27
dmsimardEmilienM: I'll add it to the agenda if it isn't merged by tomorrow :)16:27
EmilienMdmsimard: very agile!16:27
*** _ryanv_ has quit IRC16:28
*** rcallawa has joined #puppet-openstack16:28
*** richm has joined #puppet-openstack16:28
*** chandankumar has joined #puppet-openstack16:31
openstackgerritBradley Klein proposed openstack/puppet-monasca: Update vertica projections for better performance  https://review.openstack.org/19317516:35
EmilienMmfisch: would you mind to help me in adding puppet-manager-core to puppet-monasca module?16:36
*** dgurtner has quit IRC16:38
mfischsure16:39
mfischwhats needed?16:39
richmdfisher: I don't see why you couldn't use > 1.0.1 with juno, unless there is some weird bug that I'm not aware of, or unless the output is subtly different, enough to cause problems16:40
richmThere's also the question of dependencies - python-openstackclient has a lot of dependencies16:40
dfisher1.0.2 works.  1.0.3 adds kilo deps16:40
dfisherwell16:40
dfisher1.0.2 works with juno.  not with puppet-keystone16:40
dfisheri'm reverting back to 1.0.1 for now16:41
EmilienMmfisch: permissions?16:41
dfisherand will try to debug puppet-keystone16:41
mfischEmilienM: do I do it in jenkins UI or is it a code submit?16:41
*** RichardRaseley has joined #puppet-openstack16:41
richm1.0.2 doesn't work with puppet-keystone ?16:42
EmilienMmfisch: mh, we have the same situation like in puppet-murano16:42
EmilienMmfisch: it's not fair (I think) to switch to puppet-manager-core because we did not write the module16:42
EmilienMmfisch: but I would like to see both groups able to vote for these modules16:42
mfischyes agreed16:42
dfisherrichm: for me, no16:43
dfisheri'm trying to figure out if this is something on my end (99% likely) or something in the module itself16:43
richmdfisher: ok - I can help - if you want me to take a look at something - paste.openstack.org16:44
dfisherthanks16:45
dfisheri'll let you know.16:45
*** rwsu has joined #puppet-openstack16:45
mfischEmilienM: here internally I'd just change it in jenkins but I think its a code review needed now16:46
*** xingchao has joined #puppet-openstack16:52
*** xarses_ has joined #puppet-openstack16:53
*** xarses_ is now known as xarses16:53
*** derekh has quit IRC16:55
*** xingchao has quit IRC16:59
*** sergmelikyan has joined #puppet-openstack17:00
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Update Sahara module  https://review.openstack.org/19272117:04
*** aimon has joined #puppet-openstack17:04
RichardRaseleyrichm: Are you around?17:06
richmRichardRaseley: yes17:08
RichardRaseleyrichm: This is perhaps a trivial observation on your 'How to uniquely name Keystone v3 resources in puppet?', but I was wondering about the use of the '::' as a separator for domain and user. In my mind this creates some conflation between Puppet namespaces. We already use '/' to support name-spacing in INI configs, perhaps we could use this (e.g. DOMAIN/user)?17:13
RichardRaseleyrichm: (Acknowledging that this may be 100% personal preference)17:14
richmRichardRaseley: / may used in user names - e.g. if using kerberos, a valid username might be admin/foo.example.com@EXAMPLE.COM17:14
richmlikewise \, if using AD17:14
RichardRaseleyrichm: Ah, you're too correct.17:15
RichardRaseleyrichm: Thank you for helping me better understand.17:15
richmI know - it is a pita finding some sort of delimiter character that is intuitive, yet not already used . . .17:15
*** morazi has joined #puppet-openstack17:15
dfisherbust out the unicode.   module<pile_of_poo>class17:15
RichardRaseley+1 =]17:16
richmI'm open to suggestions if "::" is too similar to ruby/puppet/perl namespacing17:16
* richm stares at the top row of keys on the keyboard . . .17:17
RichardRaseleyrichm: OK. I'll gather some feedback internally here to see if anyone has any bring ideas.17:17
degorenkoEmilienM, hi. I don't know why tests (for init.pp) are failed. Can you help me? https://review.openstack.org/#/c/192721/ For 'rabbit' rpc - ok, but for zmq and qpid - fail. I spent many time - but i really don't know why it's failed. Leave comments please.17:18
*** rwsu has quit IRC17:19
dfisherclass  openstack💩keystone17:20
dfisherit took me WAY too long to figure out how to do that.17:20
bkleilooking for help here with a puppet-monasca build failure -- puppet 4.0 unit related, seems to pass locally for me, but jenkins no bueno for https://review.openstack.org/#/c/193175/17:21
bkleithe specific failure is var/lib/gems/1.9.1/gems/puppetlabs_spec_helper-0.10.3/lib/puppetlabs_spec_helper/module_spec_helper.rb:23:in `block in <top (required)>': undefined method `environmentpath=' for #<RSpec::Core::Configuration:0x000000018008d0> (NoMethodError)17:21
mfischI thought we had the 4.0 stuff working?17:21
mfischbklei: maybe we need to update your fixtures17:22
mfischI mean gemfile17:22
bkleii think it did at one point -- but started failing at some point, i think it was non-voting so we ignored it for a bit17:22
mfischyeah thats it17:23
mfischyou need rspec 2.1.017:23
*** rcallawa has quit IRC17:23
mfischbklei: I'll push a review17:23
bkleii see that in the puppet-nova Gemfile, but hate flailing if i can't reproduce locally17:25
bklei:(17:25
mfischyou probably have rspec 2.1.0 installed on your mac17:25
mfischhmm gitreview is giving me troubl17:26
mfischtrouble17:26
bkleii had to reclone with 'openstack' instead of 'stackforge' to get gitreview working17:26
mfischit keeps trying to hit stackforge, wonder where that info is17:26
mfischyeah I did a reclone17:26
openstackgerritMatt Fischer proposed openstack/puppet-monasca: Sync Gemfile  https://review.openstack.org/19424417:27
mfischbklei: ^17:27
mfischI manually added the remote17:27
bkleigracias mfisch, will see if jenkins blesses ^^ and rebase17:27
*** Vivek_ is now known as Vivek17:28
crinkleyou need https://review.openstack.org/#/c/191296/ to merge for git review to work17:28
*** Vivek is now known as Guest5003117:28
*** Guest50031 is now known as Vivek_V_C17:31
*** rcallawa has joined #puppet-openstack17:31
mfischbklei: you have +2 on that17:31
mfischmerge it17:31
mfischcrinkle: what is needed to change the puppet-monasca owner group so that puppet-core can vote on the reviews?17:32
crinkleEmilienM: ^17:32
mfischhe's asked me to help but I dont know the process17:32
crinklei think buttons need to be clicked in the gerrit ui17:33
mfischok17:33
mfischsame as here17:33
mfischI can do that17:33
EmilienMmfisch: thx17:33
mfischI thought it needed a review17:33
*** iurygregory has quit IRC17:33
mfischpuppet-manager core right?17:34
*** gfidente has quit IRC17:35
EmilienMmfisch: yes17:35
*** Vivek_V_C is now known as Vivek17:35
mfischdone17:35
mfischcrinkle: https://review.openstack.org/#/c/194254/17:35
*** Vivek has quit IRC17:35
*** Vivek has joined #puppet-openstack17:35
mfischwth its already in merge conflict?17:35
*** rwsu has joined #puppet-openstack17:36
mfischcrinkle: did you do it too or something?17:36
crinklemfisch: i did nothing17:37
mfischcrinkle: I need to go ask infra17:42
*** reed has quit IRC17:44
*** reed has joined #puppet-openstack17:44
*** aimon has quit IRC17:46
EmilienMrichm: could we add gilles's patches in the Gerrit dependencies? I think it makes sense for people who test the patches and also for the Beaker patch so we can validate all the workflow.17:49
*** tshtilma has joined #puppet-openstack17:50
*** aimon has joined #puppet-openstack17:51
richmEmilienM: The keystone v3 patches?17:51
EmilienMrichm: yes17:51
richmyes17:51
*** iurygregory has joined #puppet-openstack17:52
EmilienMrichm: I tested to deploy master, run beaker, apply all patches, run beaker and it fails now. I'm digging17:52
richmand this one too https://review.openstack.org/18976417:52
EmilienMrichm: all the patches we need, need to be configured with dependencies17:52
mfischEmilienM: this is what you wanted for puppet-monasca: https://review.openstack.org/#/c/194263/17:53
mfischI will also ask bklei for a +117:53
richmEmilienM: do you mean an explicit Depends-On: Ixxxxx ?17:53
EmilienMmfisch: yeah. Thanks. I hope it works17:53
EmilienMrichm: or classic rebase17:53
EmilienMI prefer rebase because at least we can test them with beaker17:54
EmilienMcrinkle is working on the capability to use zuul cloner but it"s WIP afik17:54
EmilienMrichm: so if we could use git rebase that would be awesome17:54
richmafaik all of the patches have been rebased17:55
richmWhat patches need a rebase?17:56
richmFor example - https://review.openstack.org/#/c/189764/ only depends on https://review.openstack.org/187654 which has been merged17:57
EmilienMrichm: ok17:58
EmilienMrichm: maybe there is no patch to rebase then. I'm testing upgrade now17:59
mfischEmilienM: did you ever setup a bug tracker for puppet-monasca?18:00
*** cdent has quit IRC18:01
EmilienMmfisch: of course Sir (note the 'Sir') > https://bugs.launchpad.net/puppet-monasca18:01
openstackgerritMatt Fischer proposed openstack/puppet-monasca: Sync Gemfile  https://review.openstack.org/19424418:02
mfischty18:03
mfischbklei: the state of lint in monasca is bad, so I will comment stuff out in that Gemfile to get it passing, I filed a bug with the issues18:03
mfischEmilienM: https://bugs.launchpad.net/puppet-monasca/+bug/1467625 if you have an intern ;)18:04
openstackLaunchpad bug 1467625 in puppet-monasca "puppet-monasca has lots and lots of lint issues" [Undecided,New]18:04
EmilienMmfisch: lol18:05
bkleiperfect, thx mfisch18:08
mdormanmfisch/ clayton (or anybody), have you done any work to implement the oslo_messaging_rabbit/heartbeat_timeout_threshold and heartbeat_rate settings in the puppet modules?   in kilo you have to explicitly enable that stuff (default is off).   thinking about going and doing that work, but if there is stuff in flight already, i don’t want to dupe18:09
mfischwe have not18:10
mdormankk18:10
mfischwould love to have support18:10
mdormani assume that is something you will want18:10
mdormank18:10
openstackgerritMatt Fischer proposed openstack/puppet-monasca: Sync Gemfile  https://review.openstack.org/19424418:11
crinkleEmilienM: https://review.openstack.org/19428718:22
*** tshtilma has quit IRC18:24
EmilienMcrinkle: you can also create the governance patch if you like18:36
crinkleEmilienM: it looks like puppet-openstack-cookiecutter and puppet-modulesync-configs aren't in governance, what is supposed to be there and what is not?18:40
openstackgerritMerged openstack/puppet-monasca: Sync Gemfile  https://review.openstack.org/19424418:41
EmilienMcrinkle: good point. I suggest we import the repos in governance in one shot18:41
EmilienMspredzy, sbadia ^18:41
EmilienMcrinkle, richm: I've tested upgrade from puppet-keystone/master to v3 patches and everything looks okay when you don't change your manifests18:42
*** rcallawa has quit IRC18:51
*** sergmelikyan has quit IRC18:53
*** rcallawa has joined #puppet-openstack18:54
*** gsilvis_ is now known as gsilvis18:55
_odyEmilienM: https://review.openstack.org/#/c/194097...reasoning behind no running ruby code through spec tests?  Since we only run rspec-puppet I completely forgot to even look at providers.19:06
EmilienMsocial: ^19:06
*** rushiagr is now known as rushiagr_away19:08
richmEmilienM: ok19:08
_ody...no...we do according to the rake task.19:10
social_ody: this is kind of internal puppet issue, if you have idea how to test this I'm all ears19:11
chemrichm: are you working on the 'absent' user for Keystone_user_role which make the final beaker test fail. I think I could help here.19:12
_odysocial: Yeah.  Reading thorugh tests now.  Just surprised the test didn't bomb out with a similar no method found error.19:13
*** rwsu has quit IRC19:15
richmchem: yes19:15
richmchem: The problem is deeper than I thought - re: my recent email to openstack-dev19:15
richmchem: The real problem is - how does self.instances create the right resource name?19:16
richmchem: How does it know when the resource name needs to be 'name::domain' or just plain 'name'?19:16
richmWhen I am writing a manifest, how do I know that the username needs to be 'glance::services' rather than just keystone_user { 'glance': domain => 'services' }?19:17
richmI mean, yeah, we could bang out a solution to that particular problem and fix the acceptance test19:17
chemrichm: yes, I think I see.  I'm going over the mailing list to dig on the details, thanks19:18
richmI'm working on a poc for keystone_tenant to see if I can do something that works with existing manifests, still allows you to do a simple name of 'glance' even if 'glance' is in the 'services' domain, and allows full support for multi-domain v3 installations19:19
_odysocial: Oh.  We just don't set tenant_name in the tests and the code is only triggered if it is.19:19
*** rcallawa has quit IRC19:23
*** chandankumar has quit IRC19:26
*** rcallawa has joined #puppet-openstack19:27
*** rcallawa has joined #puppet-openstack19:28
spredzyEmilienM, crinkle puppet-openstack-cookiecutter is under review https://review.openstack.org/#/c/192636/ for governance19:32
crinklespredzy: oh I see19:33
EmilienMspredzy: and I +1... :)19:34
spredzycrinkle, EmilienM if you want me to squash puppet-modulesync-config in it I don't mind. If you prefer another review I don't mind either way :)19:34
EmilienMit's okay19:34
*** jtomasek has quit IRC19:37
*** rwsu has joined #puppet-openstack19:38
*** rwsu has quit IRC19:40
*** rwsu has joined #puppet-openstack19:40
openstackgerritMike Dorman proposed openstack/puppet-nova: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19431519:41
*** rcallawa has quit IRC19:45
richmchem: EmilienM: something like this - http://paste.openstack.org/show/314143/19:47
richmI have to run for an appt. - back later19:47
*** mgarza has joined #puppet-openstack19:47
openstackgerritBradley Klein proposed openstack/puppet-monasca: Update vertica projections for better performance  https://review.openstack.org/19317519:50
openstackgerritBradley Klein proposed openstack/puppet-monasca: Update vertica projections for better performance  https://review.openstack.org/19317519:55
openstackgerritBradley Klein proposed openstack/puppet-monasca: Update vertica projections for better performance  https://review.openstack.org/19317519:56
chemrichm: looking into it ...20:01
openstackgerritMike Dorman proposed openstack/puppet-ceilometer: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19433620:03
chemrichm: the first comment is weird regarding the code "# Use long name for resource if not unique and not in default domain" and you have == default_domain_id just above (L7). Am I missing something ?20:08
*** xarses has quit IRC20:08
*** rwsu has quit IRC20:10
*** iurygregory has quit IRC20:14
chemrichm: oki, I got the hang of it.  Your talking about the new resource...20:15
openstackgerritMike Dorman proposed openstack/puppet-cinder: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19434120:16
openstackgerritDan Prince proposed openstack/puppet-tripleo: Add a function to write package names  https://review.openstack.org/19434220:17
openstackgerritMike Dorman proposed openstack/puppet-cinder: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19434320:19
*** bklei has quit IRC20:22
*** dgurtner has joined #puppet-openstack20:25
*** dgurtner has quit IRC20:30
*** iurygregory has joined #puppet-openstack20:31
openstackgerritMike Dorman proposed openstack/puppet-glance: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19435420:32
chemrichm: so I get this right this is solution 1) of your post on the mailing list, isn't ?20:35
*** iurygregory has quit IRC20:37
*** dprince has quit IRC20:38
openstackgerritColleen Murphy proposed openstack/puppet-openstack_extras: Fix kilo repo installation for RedHat  https://review.openstack.org/19376120:39
chemrichm: with this code you cannot predict what will be the name of the resource from one run to another, is that ok ?20:42
*** iurygregory has joined #puppet-openstack20:44
*** iurygregory has quit IRC20:44
openstackgerritMike Dorman proposed openstack/puppet-heat: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19436920:49
openstackgerritMike Dorman proposed openstack/puppet-glance: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19435420:50
openstackgerritMike Dorman proposed openstack/puppet-cinder: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19434320:52
openstackgerritMike Dorman proposed openstack/puppet-nova: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19431520:52
openstackgerritMike Dorman proposed openstack/puppet-ceilometer: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19433620:53
*** rcallawa has joined #puppet-openstack21:01
*** rcallawa_ has joined #puppet-openstack21:03
*** rcallawa_ has quit IRC21:04
*** rcallawa_ has joined #puppet-openstack21:04
*** jayg is now known as jayg|g0n321:06
*** rcallawa has quit IRC21:06
openstackgerritMike Dorman proposed openstack/puppet-neutron: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19438121:07
*** rwsu has joined #puppet-openstack21:10
richmchem: That's one way to describe the problem - I cannot predict what the resource name will be in the manifest21:18
chemrichm: that I get, but I was talking about this paste http://paste.openstack.org/show/314143/21:23
richmchem: This paste attempts to use the short name when it is unique.  If the short name is not unique, it uses the short name if it is in the default domain.  Otherwise, it uses the long name.21:24
chemrichm: well, exactly, so if we take your example in the ML, we can end up with admin { domain => dom1} and admin::dom2 or the opposite depending on how the hash is build21:26
richmchem: right - it is non-deterministic21:27
richmchem: so how can we make it deterministic so that we have 'admin' and 'admin::dom2' every time?21:28
richmif dom1 is the default domain, it will be 'admin' and 'admin::dom2' every time21:28
chemrichm: yeap21:28
richmbut if neither dom1 nor dom2 is the default domain, we don't have a way for an operator to say "ok - prefer dom1 for shortnames"21:29
chemrichm: but I think that solution 2) in the mailing list is the way to go21:29
chemshort name only for default domain21:29
richmchem: So that means we will always have 'admin::dom1' and 'admin::dom2' and never just plain 'admin'21:29
richmchem: I think that will cause problems in this case21:30
richmchem: Consider the case where you want to put regular users in the default domain backed by the enterprise LDAP identity backend, and put the service accounts in the domain 'services'21:30
chemrichm: if the user is not using domain then it will work as expected.  If the user is using domain then he/she uses full domain name21:30
richmThat means you must use 'glance::services', 'cinder::services', etc. everywhere21:31
richmwhich will break a lot of existing manifests21:31
chemhum ...21:31
* EmilienM cries21:32
richmIf there were a way for operators to tell puppet "Use short names for resources in these domains: Default, services, etc."21:32
richmI mean, the easy way to do it would be "ok - everyone must use long names everywhere" which would of course break the world, but maybe it is worth the one time pain?21:33
chemrichm: with a default namespace without the long name.  The services domain is mostly internal openstack machinery and could be solved (I've got a  idea to test the name of the resource)21:34
EmilienMrichm: can't we do 'if just plain admin', then use default domain elseif admin::domain_admin then domain=domain_admin21:35
EmilienM?21:35
chemrichm: then when you don't use don't care about domain, everything work as expected and when you use them .. well you have to type more character ^^21:35
richmEmilienM: Are you talking about converting the output of openstack user list to a puppet resource, or taking a puppet resource name and figuring out which actual user it refers to?21:35
EmilienMrichm: #221:36
chemEmilienM: I have the feeling that it's risky ... like unsecure21:36
chemEmilienM: but it's just a feeling ..21:36
richmchem: EmilienM: If we can guarantee that no other puppet module refers to a keystone_user, keystone_tenant, or keystone_user_role (or the forthcoming keystone_group, etc.) resource directly - that is, we "hide" those resources from other puppet modules, then it might work21:37
richmWe already mostly do that - thanks to the keystone::resource::service_identity and keystone::roles::admin21:37
richmBut there are a lot of places that refer to keystone_user_role21:38
richmand if someone is using openstack directly or the REST API directly . . .21:38
richmand heat is doing something weird with v3 and domains already21:38
richmditto neutron21:38
richmIf we did have some sort of global setting like "Use the short name for resources in these domains: Default, services, etc." - where/how would such a setting be done?  facter?  env. var.?21:40
openstackgerritMike Dorman proposed openstack/puppet-keystone: Add support for RabbitMQ connection heartbeat  https://review.openstack.org/19439921:41
*** zigo has quit IRC21:43
chemrichm: IMHO, the fact that we cannot predict the name of the resource to reference is the problem here.  So with solution 1)  we add to the turmoil, so I would rather go with 2) and refactore the current puppet module using keystone name directly.  For the user the rules would be simple to use and adopt.  But I may be missing the big picture here.21:44
chemrichm: ^^ facter is not very user friendly.  Not you would have to do that in the keystone class entry point, the only thing that is unique in the system, I think.21:45
richmchem: Yeah, I agree, 2) is simple and deterministic.  But it means that if I want to put service accounts in their own separate domains, we have to make sure all existing manifests that refer to 'glance' change to use 'glance::services' instead - same with 'cinder', 'nova', 'neutron', 'swift', 'heat', etc. etc.21:46
richmWhich may cause an operator revolt . . .21:47
*** zigo has joined #puppet-openstack21:49
chemrichm: well I see your point ...21:49
richmOn the other hand, if we can encapsulate all references to domain scoped keystone resources, so that puppet-glance, puppet-cinder, etc., and composition layers, and orchestration layers, don't have to know or care what the resource names are, then this would be a good solution21:50
chemrichm: That definitively the way to go :)  Is that doable ?21:50
chemrichm: are you talking about the auth.pp file in all those modules ?21:51
chemrichm: or is there other cases ?21:51
richmchem: I don't know if there are other cases, but that's the main one - the references to keystone_user_role21:52
richmwhich implicitly refer to the keystone_user and keystone_tenant names21:52
*** dalees has quit IRC21:53
richmwe could get rid of that code - have e.g. glance::keystone::auth pass in a list of service names to keystone::resource::service_identity21:53
*** blkperl_ has quit IRC21:54
richmand just have keystone::resource::service_identity set up the user role ~> Service[name] notifications21:54
*** wabu has quit IRC21:54
*** xavpaice has quit IRC21:55
*** morazi has quit IRC21:55
*** xavpaice has joined #puppet-openstack21:55
chemrichm: well, encapsulation, determinism, retro-compatibility, happy user. It looks like a good idea21:57
richmswift::keystone::dispersion references keystone_user directly21:57
richmso does neutron::server::notifications21:59
richmthat looks like it as far as the standard openstack puppet modules shipped with rdo22:00
*** wabu has joined #puppet-openstack22:01
chemrichm: same finding here that's the only module that uses elsewhere that in the auth.pp modules (there is the  auth_cfn.pp in heat, but that's nearly a auth.pp also)22:02
richmok22:02
chemI think I can handle this part if you like22:02
*** dalees has joined #puppet-openstack22:03
richmIs self.instances used for anything except to do the search for self.prefetch, and for the puppet resource command?22:03
*** mgarza has quit IRC22:04
*** xarses has joined #puppet-openstack22:04
crinklethose are the two uses that i know of22:04
richmdoes it matter that puppet resource may show different resource names than are in the manifest?22:06
chemrichm: ^^ I'm looking22:07
richmthat is, puppet resource could show keystone_user { 'admin::dom1': domain => 'dom1' }, but I have keystone_user { 'admin': domain => 'dom1' } in my manifest, and they both refer to the same keystone_user?22:07
crinklerichm: it's okay that puppet resource shows an id property when the manifest doesn't, so by that rationalization i think it's fine if puppet resource shows something slightly different than the manifest22:08
*** blkperl has joined #puppet-openstack22:08
richmok22:11
richmso then maybe it is ok if puppet resource always shows the fully qualified name22:12
crinklei think that makes sense22:12
richmthen it's just a little extra work for self.prefetch to match the resource from the manifest with the output of of self.instances22:12
richmcrinkle that also means that "puppet resource keystone_user admin" will show nothing22:13
*** rushiagr_away has quit IRC22:13
richmthat may surprise a lot of operators who use puppet resource22:14
dfishererm.  keystone_service is really unhappy with openstackclient 1.0.1 …22:14
richmdfisher: what do you get?22:15
*** zigo has quit IRC22:15
dfisherah, crap.   that's what I get for stashing things in /tmp  ...22:16
dfisheri rebooted22:16
dfisheri had to do this:22:16
dfisher   def create22:16
dfisher-    properties = ['--name']22:16
dfisher-    properties << resource[:name]22:16
dfisher+    properties = []22:16
dfisher     if resource[:description]22:16
dfisher       properties << '--description'22:16
dfisher       properties << resource[:description]22:16
dfisher     end22:16
dfisher     raise(Puppet::Error, 'The service type is mandatory') unless resource[:type]22:16
dfisher+    properties << '--type'22:16
dfisher     properties << resource[:type]22:16
dfisher+    properties << resource[:name]22:16
dfisher     self.class.request('service', 'create', properties)22:16
dfisher     @property_hash[:ensure] = :present22:16
dfisherhttp://paste.openstack.org/show/314508/22:16
dfisherthere's no —name flag22:16
dfisherand no —type flag in the cmd construciton22:17
*** bklei has joined #puppet-openstack22:17
*** zigo has joined #puppet-openstack22:17
crinklei thought we depended on >=1.0.3?22:19
dfisherjuno's cap is 1.0.122:21
richmWhen you are using OS_IDENTITY_API_VERSION=3, you have to do service create --name $name $type22:22
dfisherstill using v222:23
richmbut I thought with API v2 you used --type $type $name22:23
dfisher^ exactly what I posted22:23
richmI'm looking at the keystone_service/openstack.rb code22:24
richmfor v2, it uses --name $name $type22:24
richmI think that's wrong22:24
dfisherit is :)22:25
richmcrap22:25
dfisheryou see my paste?22:25
richmyes22:25
dfisherok, good.22:25
dfisheri got yelled at by freenode for flood :)22:25
dfisherdidn't know if you saw it.22:25
richmlol22:25
openstackgerritMerged openstack/puppet-monasca: Update vertica projections for better performance  https://review.openstack.org/19317522:27
dfisheri'm not against switching everything to v322:27
dfisherjust … not sure yet how :)22:27
richmdfisher: neither are we :P\22:27
dfisherhahahhaha22:27
dfisher^ how openstack works.22:28
richmdfisher: no, the problem is that the v3 code snuck into the v2 implementation22:28
*** rushiagr_away has joined #puppet-openstack22:28
richmthat needs to be reverted back to the v2 style22:28
richmyour patch is correct22:29
* dfisher nods22:29
* richm considers waiting for gildub to fix this . . . 22:31
dfisheryou could if you want.  i have a local patch for it22:31
richmdfisher: ok - then go ahead and submit a review22:32
dfisherdo I need a launchpad bug?22:32
dfisherno problems if I do ...22:32
richmdfisher: it falls under the auth_restructure bp22:32
dfishererm.22:33
dfisheruh.22:33
dfisherthreeve.22:33
dfisher:)22:33
richmlooks like gildub put some v3 code into the auth_restructure patch22:33
*** stevemar has quit IRC22:37
*** xingchao has joined #puppet-openstack22:38
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: beaker: test to list keystone resources  https://review.openstack.org/19442522:40
*** xingchao has quit IRC22:43
*** aimon has quit IRC22:45
*** jasondotstar has joined #puppet-openstack22:47
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: beaker: test to list keystone resources  https://review.openstack.org/19442522:49
*** saneax has joined #puppet-openstack22:49
openstackgerritDrew Fisher proposed openstack/puppet-keystone: Restore Keystone v2 method for service create  https://review.openstack.org/19442822:50
dfisherrichm: posted22:50
dfishernot sure I did the commit message right.   i think I did, but I've gotten used to being wrong :)22:51
richmdfisher: ack22:52
dfisherhey, so, good-ish news:  i have solaris multi-node puppet working.  one controller node, one volume node, one compute node - all deployed via puppet.22:54
richmnice!22:58
dfisherman, when puppet starts clicking … it's so amazing23:00
*** rwsu has quit IRC23:00
nibalizerwe're gonna put that on the box23:00
nibalizer:)23:01
dfishernice.23:01
*** ducttape_ has quit IRC23:02
EmilienMdfisher: congrats23:06
dfisherthanks :)23:07
dfisheronce I get this stuff working a little bit more solidly23:08
dfisheri'll start working on upstream23:08
dfishercan puppet metadata.json files use optional dependencies?23:10
dfisherfor example, puppet-swift wants to depend on xinetd.  Solaris doesn't have xinetd23:11
dfisherand neutron::puppet-vswitch.23:11
dfisheretc.23:11
RichardRaseley_ody: ^^ RE: Solaris23:11
RichardRaseleydfisher: +1 =]23:11
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: auth_file: allow to change the path  https://review.openstack.org/19443123:12
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: beaker: test to list keystone resources  https://review.openstack.org/19442523:13
dfisherok, well, i'll have to dig up an answer to my dependency thing tomorrow.  gotta split....23:15
*** dfisher has quit IRC23:16
*** markvoelker has quit IRC23:16
*** aimon has joined #puppet-openstack23:17
*** aimon_ has joined #puppet-openstack23:19
*** prad has quit IRC23:20
*** aimon has quit IRC23:22
*** linkedinyou has joined #puppet-openstack23:22
*** xingchao has joined #puppet-openstack23:24
*** jasondotstar has quit IRC23:29
*** RichardRaseley has quit IRC23:29
*** mdorman has quit IRC23:33
*** jasondotstar has joined #puppet-openstack23:54

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