Tuesday, 2016-07-26

« Monday, 2016-07-25 Index Wednesday, 2016-07-27 »
*** ducttape_ has quit IRC00:00
*** ducttape_ has joined #puppet-openstack00:01
*** ducttape_ has quit IRC00:04
*** furlongm has quit IRC00:05
*** furlongm has joined #puppet-openstack00:05
*** ducttape_ has joined #puppet-openstack00:15
*** ducttape_ has quit IRC00:19
*** ayoung has joined #puppet-openstack00:45
*** ayoung has quit IRC01:00
*** ducttape_ has joined #puppet-openstack01:03
*** ducttape_ has quit IRC01:09
*** ducttape_ has joined #puppet-openstack01:13
*** ducttape_ has quit IRC01:16
*** cdelatte has quit IRC01:29
*** ducttape_ has joined #puppet-openstack01:33
*** ducttape_ has quit IRC01:42
*** rstarmer has quit IRC01:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-cookiecutter: Add CI scripts  https://review.openstack.org/34491102:18
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: Add CI scripts  https://review.openstack.org/34491602:20
EmilienMmwhahaha: we have CI jobs for ^ !02:21
EmilienMI'll work on it tomorrow to make sure the script pass02:21
*** jistr has quit IRC02:32
*** jistr has joined #puppet-openstack02:33
mwhahahaOk02:36
*** ebalduf has quit IRC02:37
*** richm has quit IRC02:56
*** ducttape_ has joined #puppet-openstack03:01
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-barbican: WIP - Move barbican to authtoken  https://review.openstack.org/34710503:22
*** ebalduf has joined #puppet-openstack03:41
*** ducttape_ has quit IRC03:42
*** jtomasek has quit IRC04:39
*** jtomasek has joined #puppet-openstack04:58
*** ntpttr- has quit IRC04:59
*** ntpttr- has joined #puppet-openstack05:03
*** ntpttr- has quit IRC05:15
*** ntpttr- has joined #puppet-openstack05:17
*** rstarmer has joined #puppet-openstack05:23
openstackgerritMerged openstack/puppet-openstacklib: Increase command_timeout for openstackclient commands  https://review.openstack.org/34699405:28
*** jtomasek has quit IRC05:38
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34713406:11
*** rstarmer has quit IRC06:13
*** leanderthal|afk is now known as leanderthal06:18
*** ntpttr- has quit IRC06:24
*** ntpttr- has joined #puppet-openstack06:26
*** rstarmer has joined #puppet-openstack06:31
*** ntpttr- has quit IRC06:37
*** rstarmer has quit IRC06:38
*** rstarmer has joined #puppet-openstack06:39
*** tesseract- has joined #puppet-openstack06:39
*** ntpttr- has joined #puppet-openstack06:39
*** shardy has joined #puppet-openstack06:52
*** rstarmer has quit IRC06:52
*** rstarmer has joined #puppet-openstack07:01
*** atom_ has joined #puppet-openstack07:02
*** dgurtner has joined #puppet-openstack07:06
*** dgurtner has quit IRC07:06
*** dgurtner has joined #puppet-openstack07:06
*** rcernin has joined #puppet-openstack07:06
*** rstarmer has quit IRC07:09
*** rstarmer has joined #puppet-openstack07:12
*** alee has quit IRC07:15
openstackgerritCarlos Camacho proposed openstack/puppet-tripleo: Add pacemaker mistral profile  https://review.openstack.org/34334607:18
*** rstarmer has quit IRC07:18
*** ebalduf has quit IRC07:27
*** ifarkas_afk is now known as ifarkas07:28
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Configure keystone authtoken options  https://review.openstack.org/34668507:33
*** toMeloos has joined #puppet-openstack07:33
*** rstarmer has joined #puppet-openstack07:46
*** rstarmer has quit IRC07:56
*** rstarmer has joined #puppet-openstack08:03
*** rstarmer has quit IRC08:08
*** alee has joined #puppet-openstack08:09
openstackgerritSu Zhengwei proposed openstack/puppet-glance: Fix inconsistent between puppet and install guide  https://review.openstack.org/33413708:17
*** derekh has joined #puppet-openstack08:29
*** paramite has joined #puppet-openstack08:32
*** jtomasek has joined #puppet-openstack08:33
openstackgerritAde Lee proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980608:45
*** saneax_AFK is now known as saneax09:05
*** gfidente has joined #puppet-openstack09:12
*** gfidente has quit IRC09:12
*** gfidente has joined #puppet-openstack09:12
*** xarses_ has joined #puppet-openstack09:48
*** xarses_ has quit IRC09:48
*** xarses_ has joined #puppet-openstack09:49
*** xarses_ has quit IRC09:49
*** xarses_ has joined #puppet-openstack09:50
*** rstarmer has joined #puppet-openstack09:50
*** xarses has quit IRC09:50
*** rstarmer has quit IRC09:55
openstackgerritIvan Berezovskiy proposed openstack/puppet-mistral: Prepare 8.2.0 (mitaka)  https://review.openstack.org/34619510:02
*** rstarmer has joined #puppet-openstack10:04
*** aernhart has joined #puppet-openstack10:14
openstackgerritMerged openstack/puppet-octavia: Adding health_worker.pp  https://review.openstack.org/34304410:17
*** _degorenko|afk is now known as degorenko10:18
*** rstarmer has quit IRC10:22
*** rstarmer has joined #puppet-openstack10:24
*** ntpttr- has quit IRC10:30
*** ntpttr- has joined #puppet-openstack10:35
*** Ahharu has joined #puppet-openstack10:36
Ahharuhello10:36
AhharuI am getting hit since this morning with this from puppet-keystone module10:36
Ahharu       Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Error creating type specialization of an Apache::Listen-Type, Cannot use Integer where a resource title String is expected at /home/ahharu/puppet-testing/puppet-midonet_openstack/spec/fixtures/modules/apache/manifests/vhost.pp:411:39 at /home/ahharu/puppet-testing/puppet-midonet_openstack/spec/fixtures/modules/keystone/manifests/wsgi/apach10:36
Ahharuany idea?10:36
*** johnmilton has quit IRC10:45
openstackgerritMerged openstack/puppet-cinder: SolidFire backend deployment should use $::os_service_default  https://review.openstack.org/34563210:45
openstackgerritMerged openstack/puppet-mistral: Add missing tag on python-mistralclient package.  https://review.openstack.org/34680810:45
*** rstarmer has quit IRC10:53
*** rstarmer has joined #puppet-openstack10:58
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Added Contrail puppet interfaces for tripleo  https://review.openstack.org/34725011:01
openstackgerritMerged openstack/puppet-cinder: Removing deprecated cluster_id db var  https://review.openstack.org/34674511:06
*** trown|outtypewww is now known as trown11:17
*** rstarmer has quit IRC11:19
*** rstarmer has joined #puppet-openstack11:28
*** rstarmer has quit IRC11:29
*** johnmilton has joined #puppet-openstack11:31
*** rstarmer has joined #puppet-openstack11:35
*** richm has joined #puppet-openstack11:54
*** iberezovskiy|off is now known as iberezovskiy11:55
*** paramite is now known as paramite|afk11:55
*** cdelatte has joined #puppet-openstack11:56
*** rstarmer has quit IRC11:57
iberezovskiyAhharu, what's you apache and keystone module versions?11:58
*** ducttape_ has joined #puppet-openstack12:00
*** shardy is now known as shardy_lunch12:01
AhharuI seem to have found the culprit @iberezovskiy12:07
Ahharuits that keystone module defaults ports to integers12:07
Ahharuthus this fails in puppet 3 when it check for the ports being defined12:07
AhharuI added this to my keystone::wsgi::apache12:07
Ahharu    public_port      => '5000',     admin_port       => '35357',12:07
degorenkochem, hey, can you take a look on https://review.openstack.org/#/c/330623/ ?12:09
chemdegorenko: sure12:11
degorenkothanks12:11
*** ducttape_ has quit IRC12:15
chemdegorenko: I have serevals idea to stub this stuff, let me do some tests to find out which is the best12:15
degorenkochem, feel free to re-upload12:15
chemdegorenko: ack12:15
EmilienMhello12:22
EmilienMmjblack: can you  look https://review.openstack.org/#/c/329991/ when you around?12:23
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Move nova constraints from controller manifest to each service  https://review.openstack.org/34730912:24
iurygregorymorning12:25
*** ericksonsantos has joined #puppet-openstack12:27
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Create role for the fake openstack-core resource  https://review.openstack.org/34731512:30
iurygregoryalee, i forgot to update the acceptance sorry12:34
aleeiurygregory, yeah - I figured it was something like that12:35
iurygregoryi've update let's see if is gonna work =D12:36
*** moravec has quit IRC12:36
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-barbican: WIP - Move barbican to authtoken  https://review.openstack.org/34710512:36
EmilienMalee: progress on testing?12:39
aleeEmilienM, yes in fact12:39
aleeEmilienM, so the problem I had yesterday was that the user needed to have admin role to delete a secret12:40
EmilienMall is red https://review.openstack.org/#/c/339806/12:40
aleeEmilienM, when I added admin role to tempest roles, I was abe to delete the secret successfully12:40
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Next generation HA architecture work  https://review.openstack.org/34265012:41
aleeEmilienM, but -- there are some other network visibility tests that fail - perhaps because the user is not supposed to be an admin12:41
aleeEmilienM, which is why all is red I think12:41
aleeEmilienM, but yesterday I pushed a change to barbican which allows a secret creator to delete their own secret12:42
aleeEmilienM, so when that newest barbican package goes through, the new admin role will be unneccessary12:42
EmilienMalee: which new package?12:43
EmilienMhow old?12:43
aleeEmilienM, well - the change probably merged last night sometime .. looking ..12:43
EmilienMwe are promoting right now https://review.openstack.org/#/c/347134/12:43
aleeEmilienM, https://review.openstack.org/#/c/311830/12:44
EmilienMk12:44
EmilienMI'm glad our CI started to do real tests12:44
aleeEmilienM, merged 9 hours ago ..12:44
aleenot sure when the latest build was done.12:45
EmilienMalee: we'll have the patch in ~30 min12:45
aleeEmilienM, cool12:45
EmilienMalee: it's easy to check yourself12:45
EmilienMalee: https://trunk.rdoproject.org/centos7-master/report.html12:45
EmilienMalee: and https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/repos.pp#L3412:46
EmilienMthat's all;12:46
aleeEmilienM, once the admin role was added, I encountered another problem with attaching the volume .. which I need to follow on with cinder devs12:46
aleeEmilienM, cool ..12:46
aleeEmilienM, http://logs.openstack.org/06/339806/35/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/422a18a/logs/nova/nova-compute.txt.gz#_2016-07-26_09_17_19_16712:47
aleeEmilienM, I think its related to http://logs.openstack.org/06/339806/35/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/422a18a/logs/nova/nova-compute.txt.gz#_2016-07-26_09_17_16_92412:47
aleebut need to check with cinder guys12:48
aleeEmilienM, anyways barbican is doing the right thing - the question is why attaching the volume is failing here.12:48
aleeEmilienM, did we ever run volume encryption tests in our scenarios (without barbican) before?12:49
EmilienMalee: no12:49
*** paramite|afk is now known as paramite12:51
aleeEmilienM, ok -- the exception thatis being thrown is here : https://github.com/openstack/nova/blob/master/nova/volume/encryptors/cryptsetup.py#L4612:51
*** morazi has joined #puppet-openstack12:51
aleeEmilienM, notice the note about libvirt drivers ..12:51
EmilienMin scenario001, we use rbd, not in scenario00212:52
aleeEmilienM, yeah - so maybe it needs to be moved to scenario00212:53
EmilienMalee: so we can't encrypt volumes with ceph?12:53
aleeEmilienM, I'm not sure -- I'm going to check with eharney when he gets online12:54
*** dmsimard is now known as dmsimard|afk12:58
EmilienMalee: isn't it any doc somewhere?12:59
EmilienMhttps://ask.openstack.org/en/question/80306/does-cinder-in-kilo-supports-encryption-with-ceph/13:00
*** severion has joined #puppet-openstack13:00
EmilienMhttps://bugs.launchpad.net/cinder/+bug/144022713:00
openstackLaunchpad bug 1440227 in Cinder "Not all volume drivers set the 'encrypted' key in connection_info" [Medium,Fix released] - Assigned to Matt Riedemann (mriedem)13:00
EmilienMsounds it's supported, no?13:01
aleeEmilienM, sounds like it is ..13:02
aleeEmilienM, I had not heard of it not being supported before13:02
aleebut the cinder guys should be able to say whats going on13:02
*** bklei has joined #puppet-openstack13:09
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-openstacklib: Introduce openstack_resource_id_getter function  https://review.openstack.org/33062313:11
chemdegorenko: just an idea, didn't test in real deployment, let me know if that works for you ^13:12
*** shardy_lunch is now known as shardy13:15
degorenkochem, ok, thanks, let's wait for ci13:18
degorenkochem, it works on my local tests13:23
chemdegorenko: great, so that will do it :)13:24
degorenkochem, btw, what do you think about this function? Is it good to mix function with provider stuff?13:24
chemdegorenko: it looks to me like a code smell, there is no other exemple of function checking the local state on the node13:25
degorenkochem, didnt get :) can you rephrase please13:26
chemdegorenko: well, I check in all other openstack modules and dependend module and puppetlab function and no one run a command on the server (local state) in a function13:28
chemdegorenko: this should be the job of a provider I think, hence the "code smell"13:29
degorenkochem, ah, right, but there is no way to get output from provider to some var, right?13:29
degorenkochem, also, my doubts about this command are: 1) it will be useful only when all services are upped; 2) we can't use this function on deployment stage13:30
chemdegorenko: not that I know of.   The dogme is that you puppet manifest should not be calculated in any way with data that will appear later ... or something like that.  Well puppet ain't chef :)13:31
*** moravec has joined #puppet-openstack13:32
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Corrected syntax mistakes  https://review.openstack.org/34738513:32
chemdegorenko: the right aproche could be for the provider that handles the configuration with the uuid to calculate the good one I think.  but I like background on the use case13:33
degorenkochem, that make sense, but also may be some situations when we can get calculated data only on some deployment stage :) and then we can use only execs :D13:33
degorenkochem, i guess, in our cause - use case for such function - is set some id to config13:34
chemdegorenko: uuid that we get at run time ... I get it now13:35
degorenkochem, yeah :)13:35
chemdegorenko: so we should have a *_config_to_uuid provider I think13:36
chemdegorenko: should be easy to implement as a descendant class of *_config provider13:36
degorenkochem, probably some other config provider right, which will take as argument config name and command (in openstack cli format? ) and will call config provider inside13:36
chemdegorenko: yeap my though exactly13:37
degorenkochem, ok, i think we can raise this question on review13:37
degorenkochem, probably, there is also other way for using uuid - not only for configs13:37
degorenkoi don't know, let's find out!13:37
chemdegorenko: so wdyt, we put that on discussion for tomorrow meeting ?13:38
degorenkochem, today ;)13:39
chemdegorenko: oups ... yes, today ... It's very hot where I live and my brain is melting away13:39
degorenkochem, and where are you from? :)13:43
chemdegorenko: south of France near Montpellier, current temperature is 34 degree under shadow.13:44
degorenkochem, ah, that's like in Russia, on south and Volga based cities13:46
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Cleanup  https://review.openstack.org/34739513:46
chemdegorenko: each time I read Volga I think about Michel Strogoff :)13:47
*** ayoung has joined #puppet-openstack13:48
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Cleanup2  https://review.openstack.org/34739613:48
degorenkochem, hm, action of this novel is in Siberia :D13:49
chemdegorenko: I think it follow the volga at some point13:49
chems/it/he/13:50
degorenkochem, not sure, but i didn't read that :) Volga river located on european side of Russia, before Ural mountains and Irkuts13:51
chemdegorenko: oki, so now I will think about nothing when I read Volga :) (but I will check out on a map later ...)13:53
iurygregoryalee, it work, tonight i'll add tests and release notes :D13:53
aleeiurygregory, great13:54
degorenkochem, you can think about the biggest river in europe :D13:54
degorenkochem, i've added topic for meeting btw13:55
chemdegorenko: volga passes through nijni-novgrod, as Michel does, so it does appear in the book (http://verne.garmtdevries.nl/pics/maps/mi_1_full.jpg)  And yeah ... this is a hell of a river!13:56
degorenkochem, well :) main actions i guess are in Siberia, but oki. And Nijni-Novgorod much more northern then my Saratov :D13:58
*** xarses_ has quit IRC14:00
*** dfisher has joined #puppet-openstack14:08
degorenkodfisher, hey14:10
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-cookiecutter: Add CI scripts  https://review.openstack.org/34491114:10
*** atom_ has quit IRC14:10
dfisherdegorenko: morning14:10
dfisherstill reading mails.14:10
degorenkodfisher, chem updated patch with opesntack_id_getter :) but we have another solution for this also14:10
dfisheroh sweet.  thanks chem!14:10
degorenkodfisher, this function is for setting some uuid to config right?14:11
dfisheryes14:11
dfisherspecifically the router_id in l3_agent.ini14:11
degorenkodfisher, so we have another solution then14:11
* dfisher all ears14:11
degorenkodfisher, take a look on my comment to function patch :)14:12
dfisherok, the withenv block?14:12
degorenkodfisher, no, with new provider + config provider14:13
* dfisher wince14:13
dfisherso, it won't work as a provider14:13
dfisherat least, as far as I understand it14:13
degorenkodfisher, but if our new provider will write to config - it will :)14:13
openstackgerritMatthew J Black proposed openstack/puppet-gnocchi: WIP Keystone authoken resource changes  https://review.openstack.org/34741714:14
dfishersec, getting you an example14:14
degorenkoi guess we can discuss it in weekly meeting today14:14
*** emccormick has joined #puppet-openstack14:17
dfisherthe problem is that if you have this:14:17
dfisherhttp://paste.openstack.org/show/542109/14:17
degorenkodfisher, not exactly14:18
dfisherthe problem is the neutron:agents::l3 class14:18
dfishernot the new provider14:18
dfisherit will set router_id =14:18
dfisherand then try to start the service14:18
dfisherand puke (because use_namespaces = false and router_id is not set to anything)14:18
dfisherthen the new provider will set it.  and re-enable the service14:19
dfisherthen, 20 minutes later puppet agent will run again and redo that entire dance14:19
dfisherwhere agents::l3 unsets it14:20
dfisherand restarts the service14:20
dfishercausing it to puke14:20
dfisherthen the new provider sets it and fixes everything14:20
dfisherand over and over14:20
degorenkodfisher, we can add some new deps anchors for this14:20
dfisherbut we'd have to add new anchors in *every* module14:21
dfishernot just neutron14:21
degorenkodfisher, we have to add in case we need this provider14:21
degorenkoonce we need it - we add new anchor14:21
dfisheri'm not sure I understand why we'd do all this churn when a function works more or less just fine.14:22
degorenkodfisher, mixing provider stuff with function - not best solution :)14:22
degorenkoalso we have to execute this code on master14:22
dfisherright.14:22
*** ebalduf has joined #puppet-openstack14:23
degorenkodfisher, that's just a suggestion - it is not like, current implementation is crap lets do it in other way - no14:23
chemdfisher: degorenko who is currently writing the router_id thing14:23
dfisherright, i knwo :)14:23
dfisherchem: i am14:23
chemdfisher: no, I mean, how it currently works ?14:24
dfisheri have to go afk for about 5 minutes14:24
dfisherbe right back14:24
degorenkochem, didn't get your question. how works what?14:24
mjblackEmilienM: I provided feedback on that one change14:25
EmilienMthx14:25
chemdegorenko: the router_id uuid should be set when by which module, do you have a code example that is currently working with your patch ?14:25
EmilienMdanpawlik: ^14:25
degorenkochem, ah, got it now, well, then, dfisher i guess has answer on this question14:26
danpawlikEmilienM, mjblack: checking14:26
degorenkochem, i saw also couple of ironic patches14:26
degorenkochem, like that https://review.openstack.org/#/c/328921 - but it is not ready i think14:27
chemdegorenko: oki, so we should start from there to find the best way, with real code that would work with the function and then backtrack to see if there is a provider that could do the job without messing up every thing14:27
toMeloosJust tried to use puppetlabs-haproxy 1.5 with puppetlabs-concat 2.2 and it seems broken. Anybody else tried haproxy with concat 2 yet?14:29
chemdegorenko: oki, so if in any *_config provider we see a key that end by _uuid then we do the convertion14:29
degorenkochem, btw, we already have some custom providers https://github.com/openstack/puppet-tempest/blob/master/lib/puppet/provider/tempest_neutron_net_id_setter/openstack.rb14:30
degorenkoi guess then we can vote for one global provider for such goals14:31
dfisherchem: yes.  one sec14:31
degorenkodfisher, fyi ^14:31
dfisherright, EmilienM pointed me at those already14:32
dfisherand as I said, i've tried the provider route14:32
dfisherand agents::l3 is the problem14:32
degorenkodfisher, but we can use your function, in case of first deploy14:32
degorenkowhen all services are not ready14:33
degorenkocan't*14:33
dfisherchem: http://paste.us.oracle.com/2552114:33
degorenkodfisher, 40414:33
dfisheroh dammit14:34
dfisherhttp://paste.openstack.org/show/542113/14:34
dfishersorry.14:34
mwhahahaline #15 is bad in that paste btw14:34
dfisherworks fine for me ;)14:35
degorenkodfisher, so, in this case, in first run it will not be executed14:35
mwhahahayea it works, but you don't need to quote it ;)14:35
dfisherk14:35
degorenkoif you use one catalog for all services, like in our p-o-i jobs14:35
chemdfisher: I cannot see how it works, as it's evaluated at compile time when, for a first install, there is no resource created at all14:35
dfisherthat's correct.  it doesn't work on the first pass.14:36
dfisherit never will14:36
*** haplo37__ has joined #puppet-openstack14:36
degorenkoor you need to create another catalog (another puppet file and run) for neutron services and for keystone14:36
degorenkothen it will14:36
dfisheri create a second manifest for just l314:37
degorenkobut it is not run time solution :)14:37
dfishernope.  it sure isn't.14:37
EmilienMdegorenko: so we need a meeting for the 2 points in the etherpad?14:37
EmilienMhttps://etherpad.openstack.org/p/puppet-openstack-weekly-meeting-2016072614:37
degorenkoEmilienM, well, we can discuss it here. It just for information14:37
chemdfisher: this is set by neutron_l3_agent_config, can we assume that at the time this is run the router_id is created thanks to all existing dependency ?14:39
EmilienMwell, we can do a meeting if you want14:39
EmilienMbut I already see discussion here so...14:39
dfisherchem: yes.  I've already created a router with the neutron_router provider14:39
chemso if neutron_l3_agent_config is smart enought to get the uuid from the name then it's done all in run/manifest, no ?14:40
dfisherneutron_l3_agent_config is, sadly, not smart enough14:41
dfisher(neither is neutron itself for that matter)14:41
chemdfisher: I know, but I think that the code you've created should belong to this provider, in some way14:42
dfisherhow do you reconcile the issues with agents::l3 unsetting router_id?14:42
chemdfisher: I don't know about this issue, I just looked at the past you've provided14:43
* dfisher nods14:43
*** ntpttr__ has joined #puppet-openstack14:43
*** xarses_ has joined #puppet-openstack14:44
chemdfisher: why would it unset the value, I don't get this ?14:44
dfisherchem: http://paste.openstack.org/show/542109/14:44
dfisherso in neutron::agents::l314:44
dfisherif use_namespaces is false14:44
chemdfisher: oki I got it14:44
dfisherit sets router_id to whatever's passed in14:45
dfisherin this case, undef / ""14:45
dfisherthen agents::l3 enables the service14:45
dfisherwhich pukes because router_id isn't set correctly14:45
degorenkoEmilienM, we can drop meeting today. What do you think about first point in ehterpad?14:45
chemdfisher: but what I'm saying is something like this http://paste.openstack.org/show/542116/14:46
EmilienMi'm currently in tripleo meeting14:46
chemin agents::l314:47
EmilienMI'll look after14:47
dfisherchem:  that would totally work.  I don't know how to modify the provider to account for that, however14:47
degorenkochem, +1 for your solution14:47
dfisher^14:47
degorenkodfisher, there is nothing hard to implement that :)14:48
chemdfisher: well that shouldn't be a big deal, but I'm so short on time right now ...14:48
dfisherit's Ruby.  everything's hard (for me) :)14:48
chemhehe14:48
dfisherchem: if you have an example somewhere, I can copy and paste like a champ14:48
degorenkoonly one problem from my vision - how to detect what we should ask from openstack client in chem way14:48
chemdegorenko: we hardcode it in the code based on the key name ?14:49
degorenkochem, but then we need to hardcode some request every time?14:50
*** ducttape_ has joined #puppet-openstack14:50
chemdegorenko: there is no other way.  It would have been one function per uuid anyway.  If there is some common code appearing then we refactor.14:51
degorenkowell, ok14:52
*** aderyugin has quit IRC14:52
*** atom_ has joined #puppet-openstack14:53
*** aderyugin has joined #puppet-openstack14:53
*** aderyugin has quit IRC14:54
*** aderyugin has joined #puppet-openstack14:54
chemdegorenko: dfisher I see a way to implement this.  Will try for the next 15 min and see how it goes14:55
*** aderyugin has quit IRC14:55
dfisherif you have a rough mock-up, I can take it from there and try it14:55
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Corrected syntax mistakes  https://review.openstack.org/34738514:56
EmilienMmeeting in 3 min14:57
*** xarses_ is now known as xarses14:57
degorenkoEmilienM, we have some conclusion on 2 point. Do we need meeting for 1 point? :)14:58
degorenkobecause as i see it now - we putting bunch of new parameters to some classes. May be can go through way that use logging and db classes?14:58
degorenkomwhahaha, ^14:58
dfisheri can't even find documentation on 'value => $router_name, from_name => true;'  (the from_name => true) piece14:58
degorenkodfisher, it is not exist yet14:59
dfisheraaaand that's why14:59
degorenkoit was chem idea how to do this trick with uuid14:59
dfisherok.  will wait for chem.15:00
*** aderyugin has joined #puppet-openstack15:00
EmilienMyeah, let's do a quick meeting15:01
dfisherwhat's the room again?15:01
*** rcernin has quit IRC15:05
danpawlikopenstack-meeting-415:08
danpawlikdfisher: openstack-meeting-4 :)15:08
* dfisher nods15:08
dfisherthanks!15:08
openstackgerritClayton O'Neill proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446415:10
iurygregoryEmilienM, doing the pacth for meeting15:12
EmilienMack15:12
*** danpawlik has quit IRC15:13
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Update meetings #88  https://review.openstack.org/34744515:17
* iurygregory lunch time brb15:17
*** Ahharu has quit IRC15:21
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Allow to configure [ssh] section in ironic config  https://review.openstack.org/34744915:22
EmilienMtrown: ^15:22
*** bklei has quit IRC15:22
EmilienMtrown: doing instack patch now15:23
trownEmilienM: thanks I think... note quickstart has to support mitaka and liberty15:23
*** Guest51132 is now known as med_15:23
*** med_ has quit IRC15:24
*** med_ has joined #puppet-openstack15:24
trownseems like a lot of work for something only tripleo-quickstart will ever use15:24
*** aderyugin has quit IRC15:24
trownand that is not broken atm...15:24
*** aderyugin has joined #puppet-openstack15:24
EmilienMtrown: ok, I'm trying to help with master atm15:26
trownEmilienM: ya I think the puppet-ironic patch is good regardless... just saying I wouldnt patch anything into instack-undercloud15:27
trownEmilienM: as master has generic hiera override anyways15:27
trownEmilienM: and quickstart can switch to it unless we backported everything to liberty which seems like a bad idea given what we have works15:28
trowns/can/cant/15:28
*** saneax is now known as saneax_AFK15:29
*** leanderthal is now known as leanderthal|afk15:30
*** vinsh has joined #puppet-openstack15:31
openstackgerritMerged openstack/puppet-openstack-integration: Enable ironic tests back for Ubuntu  https://review.openstack.org/34671315:34
EmilienMdamn I see that http://logs.openstack.org/95/346195/3/check/gate-puppet-mistral-puppet-beaker-rspec-ubuntu-trusty/c807ccc/console.html#_2016-07-26_14_13_10_38505315:39
EmilienMiberezovskiy: cool work ^15:39
mwhahahai think that pure json error has been there for a while i believe15:40
*** ifarkas is now known as ifarkas_afk15:40
*** tesseract- has quit IRC15:43
EmilienMmwhahaha: it worked at 6:41 AM15:43
EmilienMtoday15:43
mwhahahaoh it's trusty15:44
mwhahahaprobably need a pin for mitaka/liberty then15:44
EmilienMagain?15:45
EmilienMbut why did it work thing morning15:45
EmilienMI'm finishing a task and I'll look15:45
mwhahahasomething got released with a bumped requirement for json_pure i guess15:45
mwhahahahard to say unless we have the old logs15:45
*** ntpttr__ has quit IRC15:47
*** atom_ has quit IRC15:49
EmilienMold logs: http://logs.openstack.org/95/346195/3/check/gate-puppet-mistral-puppet-beaker-rspec-ubuntu-trusty/0278949/console.html15:49
EmilienMI'll look in a bit15:49
mwhahaha2.0.2 - July 26, 2016 (136 KB)15:53
mwhahahaonce again15:53
mwhahaha2.0.2 includes a ~> 2 where 2.0.1 was a >= 015:53
mwhahahaso getting screwed on a dep we probably don't even use :/15:54
*** ntpttr__ has joined #puppet-openstack15:54
EmilienMdamn15:57
EmilienMmwhahaha: did you see which dep?15:57
mwhahahajson_pure itself15:57
EmilienMoh15:57
EmilienMlet's pin it?15:57
mwhahahayea15:58
EmilienMok, I'm doing it15:58
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/34746315:59
*** trown is now known as trown|lunch15:59
openstackgerritSaravanan KR proposed openstack/puppet-vswitch: Add DPDK driver modprobe for loading modules on reboot  https://review.openstack.org/34746415:59
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: Pin json_pure to 2.0.1  https://review.openstack.org/34746516:00
EmilienMmwhahaha: looks good? ^16:00
dfisherchem: ah ha.  i see now16:00
mwhahahayea should be16:00
EmilienMmwhahaha: damn, we'll need https://review.openstack.org/#/c/344913/ first and its backport16:01
EmilienMmwhahaha: do you know this error? http://logs.openstack.org/13/344913/1/check/gate-puppet-openstack-libraries-puppet-syntax-3-centos-7/53fbc09/console.html#_2016-07-26_14_08_10_67343016:01
mwhahahano i can look into it though16:01
EmilienMi'm trying to reproduce it16:02
EmilienMok I have it on my laptop16:03
*** johnmilton has quit IRC16:03
aleeEmilienM, ok confirmed -- https://bugs.launchpad.net/cinder/+bug/146352516:04
openstackLaunchpad bug 1463525 in Cinder "There is no volume encryption support for rbd-backed volumes" [Undecided,Triaged]16:04
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/34746816:05
chemdfisher: degorenko so I've done some prototyping of the idea in above two patch16:05
chemdfisher: degorenko completly untested, just code coming from my brain to gerrit16:05
* dfisher nods16:05
dfisheri see them both.16:06
EmilienMalee: ok, so move it to scenario00216:06
degorenkochem, will look16:06
degorenkoone for openstacklib and other neutron right?16:06
degorenkochem, ^16:06
aleeEmilienM, ok - will do16:06
chemdegorenko: yep16:06
degorenkoack16:06
aleeEmilienM, I should be able to pick up the latest barbican package , right?16:06
chemdegorenko: dfisher feel free to push patch on it if you like it, I don't have much bandwidth currently16:06
EmilienMalee: I showed you how you can check yourself, isn't?16:06
*** ntpttr__ has quit IRC16:07
EmilienMalee: https://review.openstack.org/#/c/347134/ is still in gate, so no.16:07
*** ntpttr__ has joined #puppet-openstack16:07
aleeEmilienM, ok16:08
openstackgerritMerged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34713416:12
dfisherdo WIP git reviews have jenkins run on them?16:13
EmilienMalee: now we have it ^16:13
EmilienMdfisher: yes.16:13
aleeEmilienM, cool beans thanks16:14
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Create role for the fake openstack-core resource  https://review.openstack.org/34731516:14
degorenkochem, should not it be transform_to here? https://review.openstack.org/#/c/347468/1/lib/puppet/provider/neutron_l3_agent_config/ini_setting.rb16:14
EmilienMmwhahaha: when I run rake manually it works16:15
mwhahahak i'll take a look in a few16:15
EmilienMwith FUTURE_PARSER=yes it fails16:15
EmilienMbut with FUTURE_PARSER=no it works16:16
*** richm has quit IRC16:16
*** richm has joined #puppet-openstack16:16
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: run_syntax_tests: add logic to find which puppet version to test  https://review.openstack.org/34491316:18
EmilienMmwhahaha: in the meantime, I disable it16:18
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Remove global openstack-core resource  https://review.openstack.org/34747216:18
dfisherdegorenko: are you currently working on either of chem's two patches?16:28
dfisherif not, i'll start poking at them.,16:28
*** derekh has quit IRC16:30
*** rwsu has joined #puppet-openstack16:37
*** chem` has joined #puppet-openstack16:37
*** chem has quit IRC16:37
*** dmsimard|afk is now known as dmsimard16:47
*** bklei has joined #puppet-openstack16:52
*** dhollinger has quit IRC16:54
*** foxhunt_away is now known as foxhunt7216:55
degorenkodfisher, go ahead, was afk :)16:57
dfisherno problem.16:57
dfisherwill start poking at it soon.16:57
*** ntpttr__ has quit IRC17:01
*** johnmilton has joined #puppet-openstack17:01
*** ntpttr__ has joined #puppet-openstack17:04
openstackgerritcraig delatte proposed openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446417:05
*** paramite has quit IRC17:06
*** johnmilton has quit IRC17:07
degorenkodfisher, i have only one doubt - see my comment above17:09
degorenkoi think neutron patch has typo error17:09
openstackgerritAde Lee proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980617:09
degorenkotransform -> trasnform_to17:09
dfisherright17:09
dfisherwas going to mess with that.17:09
degorenkoah, ok17:09
*** jtomasek has quit IRC17:10
degorenkodfisher, can you also upload test patch may be?17:10
dfisheryep17:10
dfisherwill do all of the above17:10
dfisherjust need to get done with what I'm whacking on right now17:11
dfisher235672- things at once, per usual17:11
EmilienMalee: thx for helping on it17:12
*** shardy has quit IRC17:13
aleeEmilienM, no worries -  going to grab dinner now -- hopefully it'll pass ..17:13
aleeEmilienM, will check on it later tonight17:13
EmilienMalee: ok17:13
aleeEmilienM, if I've made a typo or something obvious , feel free to correct if you notice before I do17:14
*** trown|lunch is now known as trown17:14
*** alee is now known as alee_dinner17:14
EmilienMok17:14
*** alee_dinner has quit IRC17:16
*** johnmilton has joined #puppet-openstack17:19
mwhahahaEmilienM: that error is because it's running puppet4 i think17:20
mwhahahahttp://logs.openstack.org/13/344913/1/check/gate-puppet-openstack-libraries-puppet-syntax-3-centos-7/53fbc09/console.html#_2016-07-26_14_07_42_53468717:21
EmilienMdang17:21
*** dhollinger has joined #puppet-openstack17:21
EmilienMI missed export PUPPET_GEM_VERSION='~> {puppet_version}'17:22
EmilienMdouble dang17:22
EmilienMmwhahaha: nice catch17:23
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: run_syntax_tests: add logic to find which puppet version to test  https://review.openstack.org/34491317:23
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-cookiecutter: Add CI scripts  https://review.openstack.org/34491117:28
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: Add CI scripts  https://review.openstack.org/34491617:28
EmilienMchem`: fyi^ we'll have more CI jobs for cookiecutter and modulesync17:29
*** ebalduf has quit IRC17:29
iurygregorynice :D17:29
*** chem` is now known as chem17:29
chemEmilienM: nice, will have a look17:29
EmilienMchem, iurygregory: we'll have lint, syntax, unit and beaker17:32
iurygregoryEmilienM, awesome :D17:32
*** foxhunt72 is now known as foxhunt_away17:34
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly  https://review.openstack.org/32834717:41
*** gfidente is now known as gfidente|afk17:44
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class properly  https://review.openstack.org/32834717:47
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: profile/base/nova: declare nova class and configure cache correctly.  https://review.openstack.org/32834717:48
*** ntpttr__ has quit IRC17:48
*** _stowa has joined #puppet-openstack17:51
*** iberezovskiy is now known as iberezovskiy|off17:55
*** _stowa has quit IRC17:55
*** degorenko is now known as _degorenko|afk17:55
*** ntpttr__ has joined #puppet-openstack17:56
*** priteau has joined #puppet-openstack17:57
*** moravec has quit IRC17:58
*** moravec has joined #puppet-openstack17:59
*** leanderthal|afk has quit IRC18:07
EmilienMmwhahaha: can you look https://review.openstack.org/#/c/347465/ please?18:08
openstackgerritMichael  Henkel proposed openstack/puppet-tripleo: Corrected syntax mistakes  https://review.openstack.org/34738518:32
openstackgerritMerged openstack/puppet-openstack-cookiecutter: Add CI scripts  https://review.openstack.org/34491118:34
*** danpawlik has joined #puppet-openstack18:36
*** delattec has joined #puppet-openstack18:37
*** delattec has quit IRC18:39
*** delattec has joined #puppet-openstack18:41
*** cdelatte has quit IRC18:41
EmilienMchem: https://review.openstack.org/#/c/344913/3/run_syntax_tests.sh18:45
EmilienMI don't understand your comment18:45
EmilienMwe test if puppet < 4 in the first conditional and set parser to yes18:45
EmilienMchem: I think I replied18:47
EmilienMchem: no, puppet syntax will ignore the variable :18:47
EmilienMhttps://github.com/gds-operations/puppet-syntax/blob/master/lib/puppet-syntax/tasks/puppet-syntax.rb#L3318:47
mwhahahaif you ran that manually with PUPPET_VERSION='3.8' you won't get the future parser18:48
mwhahahaie. manually18:48
chemEmilienM: no you put is to ${FUTURE_PARSER:-yes}, which mean that if FUTURE_PARSER is set then you don't get yes.  But it's more the else part that bother me (a little, not strong minded on this, he) where you can have puppet > 4 and future_parse set to yes (from the env)18:49
*** hjensas has quit IRC18:49
EmilienMchem: not a problem18:49
EmilienMsee https://github.com/gds-operations/puppet-syntax/blob/master/lib/puppet-syntax/tasks/puppet-syntax.rb#L3318:49
EmilienMit's just ignored !18:49
chemEmilienM: I've seen a unicorn!18:49
chemEmilienM: ack, thanks for the clarifacation18:49
chemEmilienM: but my first comment as mwhahaha point out is kinda still valid18:50
EmilienMmwhahaha: right18:50
EmilienMI guess we can address that later18:50
dfisherchem: seeing errors with your two patches and I'm not sure how to reconcile:  http://paste.openstack.org/show/542163/18:51
chemEmilienM: you can add this : puppet_major_version = `echo $PUPPET_VERSION | cut -d. -1`18:51
chemEmilienM: and be done with it :)18:51
EmilienMchem: ok I'm doing it thanks18:51
dfisherdoes neutron_l3_agent_config need to inherit from the openstacklib openstack_config provider?18:52
dfisherbecause right now, it's got:18:52
dfisherrequire File.join(File.dirname(__FILE__), '..','..','..',18:52
dfisher                  'puppet/provider/neutron')18:52
dfisherPuppet::Type.type(:neutron_l3_agent_config).provide(18:52
dfisher  :ini_setting,18:52
dfisher  :parent => Puppet::Type.type(:openstack_config).provider(:ini_setting)18:52
dfisher) do18:52
dfisherchange that to … uh … :openstack_setting instead of :ini_setting ?18:53
danpawlikHello, to create new relates notes page for puppet-nova module, I just need to type tox -e venv -- reno new <NEW FEATURE> or there is other way?18:53
dfisherwait, no.18:53
dfisherit already is.18:53
* dfisher duh18:53
*** danpawlik has left #puppet-openstack18:56
chemdfisher: the inheritance chain is correct.  let me check something about the syntax and I get back to you18:56
*** danpawlik has joined #puppet-openstack18:56
dfisherno problem.18:56
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-cookiecutter: run_syntax: use PUPPET_MAJOR_VERSION  https://review.openstack.org/34751918:56
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-cookiecutter: run_syntax: use PUPPET_MAJOR_VERSION  https://review.openstack.org/34751918:58
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: Add CI scripts  https://review.openstack.org/34491618:58
chemdfisher: in the paste you put ini_settings.rb instead of ini_setting.rb, just a copy/paste error ?18:58
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: run_syntax_tests: add logic to find which puppet version to test  https://review.openstack.org/34491318:59
EmilienMchem: done  ^ can you review it please?18:59
chemEmilienM: yep18:59
dfisherchem: yes.  jsut a typo18:59
dfisherchem: i'm trying your patches by just making the changes to the installed modules18:59
chemdfisher: that's correct, I'm doing the same :)19:00
*** ntpttr__ has quit IRC19:00
dfisher:)19:01
danpawlikif I add new feature like MultiStrOpt for parameter (https://review.openstack.org/#/c/329991/) in release notes I should add description into upgrade section, or ?19:04
*** dgurtner has quit IRC19:05
*** haplo37__ has quit IRC19:06
openstackgerritMerged openstack/puppet-openstack_spec_helper: Pin json_pure to 2.0.1  https://review.openstack.org/34746519:06
EmilienMwe need it in liberty too19:07
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: Pin json_pure to 2.0.1  https://review.openstack.org/34752419:08
*** rain has joined #puppet-openstack19:12
*** rain is now known as Guest9013319:13
openstackgerritDaniel Pawlik proposed openstack/puppet-nova: Add MultiStrOpt for scheduler_available_filters  https://review.openstack.org/32999119:13
openstackgerritBrent Eagles proposed openstack/puppet-octavia: Adding housekeeping.pp  https://review.openstack.org/34752719:16
*** Guest90133 has quit IRC19:18
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Allow to configure [ssh] section in ironic config  https://review.openstack.org/34744919:24
EmilienMmwhahaha, chem: good ? https://review.openstack.org/#/c/347519/19:28
mwhahahaya19:29
EmilienMok19:29
EmilienMI think we can release a new Mitaka tomorrow then19:29
*** Guest90133 has joined #puppet-openstack19:30
chemdfisher: oki, to sum up, type inheritence is not implemented (may or may not be working, have to check https://groups.google.com/forum/#!topic/puppet-dev/67WvuAe-CBs).  So for now you can just add newparam(:transform_to) to neutron/lib/puppet/type/neutron_l3_agent_config.rb19:31
dfisherah, ok.  will do19:31
chemdfisher: it fails with another error though :)19:31
dfisherhot damn19:32
* dfisher excited19:32
chemdfisher: I have to leave now, it's quite late here, but ...19:32
dfisherno worries.19:32
dfisherwe can keep plugging away at it19:32
chemdfisher: be aware that the value needs to be untransformed in some way to preserve idempotency when fetch back from the system, so "def value" need some tweeking as well19:33
dfisherchem: Error: /Stage[main]/Main/Neutron_l3_agent_config[DEFAULT/router_id]: Could not evaluate: undefined method `empty' for "uuid":String19:34
dfisherthat one?19:34
chemdfisher: yep, must be something stupid19:34
dfisherok.  will poke.19:34
dfisherthanks again for the help19:34
chemdfisher: if you have some input just add code to the review, I will have another (fresh) look tomorrow19:34
dfisheri will19:34
dfisherg'night!19:34
chemthanks, bye19:35
*** chem is now known as chem|off19:35
*** dgurtner has joined #puppet-openstack19:36
*** dgurtner has quit IRC19:36
*** dgurtner has joined #puppet-openstack19:36
*** danpawlik has left #puppet-openstack19:36
chem|offEmilienM: +119:37
EmilienMchem|off: go to bed !19:37
EmilienMchem|off: thanks :)19:37
openstackgerritDaniel Pawlik proposed openstack/puppet-watcher: Added planer sections  https://review.openstack.org/34178819:40
*** dgurtner has quit IRC19:42
openstackgerritMerged openstack/puppet-openstack_spec_helper: Pin json_pure to 2.0.1  https://review.openstack.org/34752419:42
dfishercan puppet providers inherit from multiple providers?19:47
openstackgerritMerged openstack/puppet-openstack-cookiecutter: run_syntax: use PUPPET_MAJOR_VERSION  https://review.openstack.org/34751919:48
*** alee_dinner has joined #puppet-openstack19:49
EmilienMalee_dinner: you didn't copy scenario001 to scenario002 correctly ...19:53
EmilienMyou missed attach_encrypted_volume19:53
alee_dinnerEmilienM, looking .. yeah -- I was about to look through to see what I missed19:55
EmilienMalee_dinner: and your patch breaks Swift tests...19:55
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980619:55
EmilienMalee_dinner: I'll let you figure why it breaks swift tests..19:55
EmilienMprobabily some keystone thing with roles19:55
EmilienMyou said, we don't need role?19:55
EmilienMwe don't need specific role?19:55
alee_dinnerI took out the admin role19:56
alee_dinnerwe still need creator role19:56
EmilienMah19:56
EmilienMit breaks swift i think.19:56
alee_dinner but thats barbican specific19:56
alee_dinner(I thought)19:56
EmilienMalee_dinner: http://logs.openstack.org/06/339806/36/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/4961870/console.html#_2016-07-26_17_54_50_79216919:56
alee_dinnerEmilienM, ok - tempest_roles I think has "Member" as default19:59
alee_dinnerEmilienM, so if we add creator it needs to be "Member,creator"19:59
alee_dinnerand that should fix swift19:59
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980620:01
EmilienMalee_dinner: lets try20:01
alee_dinnerEmilienM, I think it needs to be "Member,creator" and not ['Member', 'creator']20:03
alee_dinnerEmilienM, I tried the other way and it did not work before20:03
EmilienMalee_dinner: why?20:03
EmilienMit's because juan did a mistake in puppet-tempest20:03
EmilienMthe parameter is an array20:04
alee_dinnerEmilienM, could be - but I did read similar things in tempest.conf file ..20:05
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: Ensure tempest_roles is an array  https://review.openstack.org/34754120:08
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980620:08
EmilienMalee_dinner: ^20:08
EmilienMnow it will be an array20:08
alee_dinnercool :)20:09
EmilienMalee_dinner: please never do "a,b" as a string20:09
EmilienMuse arrays...20:09
alee_dinnerEmilienM, oh defineitely20:15
dfisherwhat's the git magic to pull a WIP patch?  I want to take chem's https://review.openstack.org/#/c/347463/ and add to it20:17
*** delattec has quit IRC20:18
iurygregorydfisher, you want to push a new patch set right?20:21
dfisheryes20:21
dfisherisn't there a git clone <mumble> refs/mumble/bumble thing?20:21
iurygregoryyou have puppet-openstacklib in your machine?20:21
iurygregorycd puppet-openstacklib20:22
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: Add CI scripts  https://review.openstack.org/34491620:22
dfisherfrom github or the package?20:22
iurygregorydfisher, git ;)20:22
iurygregorygit review -d 34746320:22
dfisherawesome.20:22
iurygregorywhen you do your changes just do git add and git commit --amend20:23
iurygregoryand git review =)20:23
EmilienMiurygregory: so was it you or denis to push the first authtoken.pp class?20:23
dfisheriurygregory: thanks!20:23
iurygregoryEmilienM, i work on the poc tonight, do you think i can do for barbican? or do you have other module in mind?20:24
iurygregorydfisher, nope o/20:24
openstackgerritDrew Fisher proposed openstack/puppet-openstacklib: WIP: add a way for provider to transform value in config.  https://review.openstack.org/34746320:27
EmilienMiurygregory: you can start by whatever you like20:27
EmilienMstart simple and iterate20:27
iurygregoryEmilienM, ack ;)20:27
openstackgerritDrew Fisher proposed openstack/puppet-neutron: WIP: transform name to uuid.  https://review.openstack.org/34746820:33
openstackgerritMerged openstack/puppet-openstack_spec_helper: run_syntax_tests: add logic to find which puppet version to test  https://review.openstack.org/34491320:38
*** jtomasek has joined #puppet-openstack20:40
*** jtomasek has quit IRC20:41
*** jtomasek has joined #puppet-openstack20:41
alee_dinnerEmilienM, something else broken -- all tests failing ..20:47
*** cdelatte has joined #puppet-openstack20:50
openstackgerritMerged openstack/puppet-openstack-guide: Update meetings #88  https://review.openstack.org/34744520:58
*** trown is now known as trown|outtypewww21:02
*** cdelatte has quit IRC21:15
*** ntpttr__ has joined #puppet-openstack21:23
*** mjb has quit IRC21:25
*** mjb has joined #puppet-openstack21:28
*** toMeloos has quit IRC21:29
*** dgurtner has joined #puppet-openstack21:32
*** dgurtner has quit IRC21:32
*** dgurtner has joined #puppet-openstack21:32
openstackgerritEdward Balduf proposed openstack/puppet-cinder: SolidFire backend deployment should use $::os_service_default  https://review.openstack.org/34706621:36
*** dgurtner has quit IRC21:37
*** ayoung has quit IRC21:45
*** ntpttr__ has quit IRC21:54
*** ntpttr__ has joined #puppet-openstack21:55
*** openstackgerrit has quit IRC22:03
*** openstackgerrit has joined #puppet-openstack22:04
*** moravec has quit IRC22:22
*** priteau has quit IRC22:23
*** ntpttr__ has quit IRC22:24
*** bklei has quit IRC22:34
*** bklei has joined #puppet-openstack22:35
*** bklei has quit IRC22:40
*** openstackgerrit has quit IRC22:48
*** openstackgerrit has joined #puppet-openstack22:48
*** ducttape_ has quit IRC22:53
*** morazi has quit IRC22:55
*** vinsh has quit IRC23:06
*** bklei has joined #puppet-openstack23:13
*** xarses has quit IRC23:38
*** dfisher has quit IRC23:57
*** ducttape_ has joined #puppet-openstack23:59
« Monday, 2016-07-25 Index Wednesday, 2016-07-27 »

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