Thursday, 2016-01-07

*** ducttape_ has quit IRC00:11
*** samuelBartel has quit IRC00:36
*** zhangjn has joined #puppet-openstack00:43
*** bklei has joined #puppet-openstack00:53
*** zhangjn has quit IRC00:55
*** zhangjn has joined #puppet-openstack01:06
*** bklei has quit IRC01:09
*** ducttape_ has joined #puppet-openstack01:12
*** ducttape_ has quit IRC01:17
*** Guest73286 is now known as saneax_AFK01:19
*** bklei has joined #puppet-openstack01:20
*** angdraug has quit IRC01:32
*** angdraug has joined #puppet-openstack01:33
*** angdraug has quit IRC01:36
*** bklei has quit IRC01:43
*** llua has quit IRC01:50
*** zhangjn has quit IRC02:19
*** ducttape_ has joined #puppet-openstack02:20
*** bklei has joined #puppet-openstack02:23
*** zhangjn has joined #puppet-openstack02:38
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP swift dedicated replication network  https://review.openstack.org/26452402:42
*** ilbot3 has quit IRC02:45
*** ilbot3 has joined #puppet-openstack02:47
*** bklei has quit IRC02:58
*** bklei has joined #puppet-openstack02:59
*** bklei has quit IRC03:01
*** bklei has joined #puppet-openstack03:02
*** bklei_ has joined #puppet-openstack03:03
*** bklei has quit IRC03:06
*** bklei_ has quit IRC03:21
*** bklei has joined #puppet-openstack03:22
*** bklei has quit IRC03:27
*** zhangjn has quit IRC03:40
*** richm has quit IRC03:48
*** ducttape_ has quit IRC03:49
openstackgerritXingchao Yu proposed openstack/puppet-trove: Add rabbit_ha_queues option  https://review.openstack.org/21445803:49
*** v1k0d3n has joined #puppet-openstack03:54
*** baker has joined #puppet-openstack04:03
openstackgerritXingchao Yu proposed openstack/puppet-trove: Introduce amqp_durable_queues option  https://review.openstack.org/21447704:04
*** baker has quit IRC04:07
*** baker has joined #puppet-openstack04:08
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Use rspec-puppet-facts for fact management  https://review.openstack.org/24233004:09
*** llua^_^AznSmile- has joined #puppet-openstack04:10
*** baker_ has joined #puppet-openstack04:10
*** baker has quit IRC04:12
*** llua^_^AznSmile- is now known as llua04:18
*** llua has joined #puppet-openstack04:18
*** bklei has joined #puppet-openstack04:18
*** bklei has quit IRC04:23
openstackgerritAlex Schultz proposed openstack/puppet-aodh: Update to pull in rspec-puppet-facts  https://review.openstack.org/26453804:27
openstackgerritAlex Schultz proposed openstack/puppet-barbican: Update to pull in rspec-puppet-facts  https://review.openstack.org/26453904:27
openstackgerritAlex Schultz proposed openstack/puppet-ceilometer: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454004:28
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454104:28
openstackgerritAlex Schultz proposed openstack/puppet-designate: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454204:28
openstackgerritAlex Schultz proposed openstack/puppet-glance: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454304:28
openstackgerritAlex Schultz proposed openstack/puppet-gnocchi: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454404:28
openstackgerritAlex Schultz proposed openstack/puppet-heat: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454504:28
openstackgerritAlex Schultz proposed openstack/puppet-horizon: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454604:28
openstackgerritAlex Schultz proposed openstack/puppet-ironic: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454704:28
openstackgerritAlex Schultz proposed openstack/puppet-keystone: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454804:28
openstackgerritAlex Schultz proposed openstack/puppet-magnum: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454904:28
openstackgerritAlex Schultz proposed openstack/puppet-manila: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455004:28
openstackgerritAlex Schultz proposed openstack/puppet-mistral: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455104:28
openstackgerritAlex Schultz proposed openstack/puppet-murano: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455204:28
openstackgerritAlex Schultz proposed openstack/puppet-neutron: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455304:28
openstackgerritAlex Schultz proposed openstack/puppet-nova: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455404:28
openstackgerritAlex Schultz proposed openstack/puppet-openstacklib: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455504:28
openstackgerritAlex Schultz proposed openstack/puppet-openstack_extras: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455604:28
openstackgerritAlex Schultz proposed openstack/puppet-rally: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455704:28
openstackgerritAlex Schultz proposed openstack/puppet-sahara: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455804:28
openstackgerritAlex Schultz proposed openstack/puppet-swift: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455904:28
openstackgerritAlex Schultz proposed openstack/puppet-tempest: Update to pull in rspec-puppet-facts  https://review.openstack.org/26456004:28
openstackgerritAlex Schultz proposed openstack/puppet-tripleo: Update to pull in rspec-puppet-facts  https://review.openstack.org/26456104:28
openstackgerritAlex Schultz proposed openstack/puppet-trove: Update to pull in rspec-puppet-facts  https://review.openstack.org/26456204:28
openstackgerritAlex Schultz proposed openstack/puppet-tuskar: Update to pull in rspec-puppet-facts  https://review.openstack.org/26456304:28
openstackgerritAlex Schultz proposed openstack/puppet-vitrage: Update to pull in rspec-puppet-facts  https://review.openstack.org/26456404:29
openstackgerritAlex Schultz proposed openstack/puppet-vswitch: Update to pull in rspec-puppet-facts  https://review.openstack.org/26456504:29
openstackgerritAlex Schultz proposed openstack/puppet-zaqar: Update to pull in rspec-puppet-facts  https://review.openstack.org/26456604:29
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Update tests to leverage rspec-puppet-facts  https://review.openstack.org/24233004:35
openstackgerritMerged openstack/puppet-swift: Add support to change meta entry for ring devices  https://review.openstack.org/25867704:35
mwhahahaoops probably shouldn't have run that msync yet :/ my bad04:37
*** jaosorior has joined #puppet-openstack04:45
openstackgerritMerged openstack/puppet-designate: Update designate::db class to match other module pattern  https://review.openstack.org/22721704:45
*** baker_ has quit IRC04:54
*** baker has joined #puppet-openstack04:54
openstackgerritMerged openstack/puppet-sahara: Do not show warning on sahara/notify  https://review.openstack.org/26148104:58
*** baker has quit IRC04:58
openstackgerritAlex Schultz proposed openstack/puppet-designate: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26347505:03
openstackgerritAlex Schultz proposed openstack/puppet-designate: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26347505:21
*** zhangjn has joined #puppet-openstack05:23
*** zhangjn_ has joined #puppet-openstack05:34
*** zhangjn has quit IRC05:36
*** zhangjn_ has quit IRC05:38
*** zhangjn has joined #puppet-openstack05:45
*** zhangjn has quit IRC05:51
*** zhangjn has joined #puppet-openstack05:54
*** zhangjn has quit IRC05:55
*** zhangjn has joined #puppet-openstack06:00
*** zhangjn has quit IRC06:01
*** zhangjn has joined #puppet-openstack06:02
*** bklei has joined #puppet-openstack06:06
openstackgerritAlex Schultz proposed openstack/puppet-cinder: Update tests to leverage rspec-puppet-facts  https://review.openstack.org/24233006:07
*** bklei has quit IRC06:10
*** markvoelker has quit IRC06:11
openstackgerritAlex Schultz proposed openstack/puppet-designate: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26347506:11
*** zhangjn has quit IRC06:25
*** zhangjn has joined #puppet-openstack06:35
*** zhangjn has quit IRC06:36
*** dgurtner has joined #puppet-openstack06:36
*** dgurtner has quit IRC06:36
*** dgurtner has joined #puppet-openstack06:36
*** zhangjn has joined #puppet-openstack06:37
*** saneax_AFK is now known as saneax06:41
*** saneax is now known as Guest5588806:41
*** zhangjn has quit IRC06:44
*** itsuugo has quit IRC06:54
*** itsuugo has joined #puppet-openstack06:55
*** dgurtner has quit IRC06:57
*** dgurtner has joined #puppet-openstack06:58
*** zhangjn has joined #puppet-openstack07:04
*** markvoelker has joined #puppet-openstack07:11
*** markvoelker has quit IRC07:16
*** dgurtner has quit IRC07:20
*** dgurtner has joined #puppet-openstack07:21
*** dgurtner has quit IRC07:21
*** dgurtner has joined #puppet-openstack07:21
*** dgurtner has quit IRC07:33
*** jtomasek has joined #puppet-openstack07:53
*** bklei has joined #puppet-openstack07:54
*** dgurtner has joined #puppet-openstack07:55
*** dgurtner has quit IRC07:55
*** dgurtner has joined #puppet-openstack07:55
*** bklei has quit IRC07:58
*** zhangjn has quit IRC08:03
*** zhangjn has joined #puppet-openstack08:05
*** foxhunt_away is now known as foxhunt7208:07
*** jpena has joined #puppet-openstack08:21
*** jtomasek has quit IRC08:29
*** jtomasek has joined #puppet-openstack08:30
*** gfidente has joined #puppet-openstack08:35
*** gfidente has quit IRC08:35
*** gfidente has joined #puppet-openstack08:35
*** tdb has quit IRC08:39
*** samuelBartel has joined #puppet-openstack08:39
*** tdb has joined #puppet-openstack08:39
*** samuelBartel has quit IRC08:45
*** mkoderer_cloud has joined #puppet-openstack08:48
*** tdb_ has joined #puppet-openstack08:56
*** tdb has quit IRC08:56
openstackgerritXingchao Yu proposed openstack/puppet-glance: Add rabbit_ha_queues option  https://review.openstack.org/21550008:57
*** samuelBartel has joined #puppet-openstack08:58
*** markvoelker has joined #puppet-openstack09:12
*** markvoelker has quit IRC09:17
*** jaosorior has quit IRC09:17
*** jaosorior has joined #puppet-openstack09:32
*** derekh has joined #puppet-openstack09:35
*** rochaporto has joined #puppet-openstack09:40
*** jistr has joined #puppet-openstack09:41
*** bklei has joined #puppet-openstack09:42
*** zhangjn has quit IRC09:42
*** bklei has quit IRC09:46
*** zhangjn has joined #puppet-openstack09:59
*** paramite has joined #puppet-openstack10:07
openstackgerritGael Chamoulaud proposed openstack/puppet-trove: Use identity_uri and auth_uri in api module  https://review.openstack.org/23322910:22
*** jaosorior has quit IRC10:34
*** jaosorior has joined #puppet-openstack10:35
openstackgerritMartin Mágr proposed openstack/puppet-openstacklib: Fix handling of ensure_absent_val  https://review.openstack.org/26471010:54
openstackgerritMartin Mágr proposed openstack/puppet-neutron: LBaaSv2 support  https://review.openstack.org/23472810:54
*** paramite is now known as paramite|afk10:58
openstackgerritMerged openstack/puppet-openstacklib: Update os_database_connection with extra param  https://review.openstack.org/26243810:59
*** markvoelker has joined #puppet-openstack11:13
*** markvoelker has quit IRC11:18
*** karimb has joined #puppet-openstack11:30
*** bklei has joined #puppet-openstack11:30
*** bklei has quit IRC11:35
*** karimb has quit IRC11:48
*** zhangjn has quit IRC11:49
*** karimb has joined #puppet-openstack11:50
*** zhangjn has joined #puppet-openstack11:58
*** zhangjn has quit IRC11:58
*** zhangjn has joined #puppet-openstack11:59
*** zhangjn has quit IRC11:59
*** zhangjn has joined #puppet-openstack11:59
*** zhangjn has quit IRC12:00
*** zhangjn has joined #puppet-openstack12:01
*** paramite|afk is now known as paramite12:01
*** morazi has quit IRC12:08
*** karimb has quit IRC12:12
*** karimb has joined #puppet-openstack12:13
*** markvoelker has joined #puppet-openstack12:14
*** pietervanw has joined #puppet-openstack12:14
EmilienMgood morning12:14
pietervanwhi all, good morning! (or afternoon where I am located :) )12:15
pietervanwI'm having some difficulties with my (dev) openstack deployment with Puppet, about which I sent an email to the OpenStack Operators mailing list.. but did not receive any reply yet. I hope there's somebody here who can give me some guidance...12:17
pietervanw(My issue is described here: http://lists.openstack.org/pipermail/openstack-operators/2015-December/009239.html)12:17
*** markvoelker has quit IRC12:18
pietervanwI'm exporting the @@ring_account_device (and others) resources, but some need to go to cluster A, and some to cluster B. Can I make such a differentiation with the puppet-openstack modules?12:22
EmilienMpietervanw: Vinsh is our puppet-swift expert, he might help but maybe early now for him12:47
*** zhangjn has quit IRC12:48
*** delattec has joined #puppet-openstack12:50
*** cdelatte has quit IRC12:52
*** gfidente is now known as gfidente|pto12:58
iurygregorymorning =)13:03
*** zhangjn has joined #puppet-openstack13:06
*** dprince has joined #puppet-openstack13:07
*** morazi has joined #puppet-openstack13:07
*** ducttape_ has joined #puppet-openstack13:10
*** jayg|g0n3 is now known as jayg13:16
*** Madkiss has quit IRC13:17
*** Madkiss has joined #puppet-openstack13:17
*** Madkiss has quit IRC13:17
*** Madkiss has joined #puppet-openstack13:17
*** foxhunt72 is now known as foxhunt_away13:18
*** markvoelker has joined #puppet-openstack13:18
*** bklei has joined #puppet-openstack13:19
pietervanwEmilienM: ok, thanks! I'm patient :)13:20
*** ducttape_ has quit IRC13:22
*** bklei has quit IRC13:23
*** karimb has quit IRC13:25
*** jpena is now known as jpena|lunch13:34
*** trown|outttypeww is now known as trown13:35
*** bklei has joined #puppet-openstack13:43
*** richm has joined #puppet-openstack13:55
EmilienMmwhahaha: it would be nice to explain what you're doing by a commit message in https://review.openstack.org/264562 and other patches13:59
mwhahahaYea sorry I goofed on that13:59
mwhahahaI'll fix it up13:59
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-neutron: Correctly parse csv even with extra output.  https://review.openstack.org/26387414:00
mwhahahaWasn't paying attention :/14:00
iurygregoryit happens =)14:00
EmilienMreading the commit message is the first thing I do when I review :)14:02
*** foxhunt_away is now known as foxhunt7214:04
*** delatte has joined #puppet-openstack14:05
*** EmilienM sets mode: +o mwhahaha14:05
*** EmilienM sets mode: +o mfisch14:05
*** EmilienM sets mode: +o clayton14:05
*** EmilienM sets mode: +o spredzy14:05
*** EmilienM sets mode: +o chem14:05
*** EmilienM sets mode: +o richm14:05
*** foxhunt72 is now known as foxhunt_away14:06
*** delattec has quit IRC14:08
*** mkoderer_cloud has quit IRC14:11
*** _matthias__ is now known as _matthias_14:15
openstackgerritMerged openstack/puppet-tripleo: Upgrade all packages after puppet managed ones  https://review.openstack.org/26104114:15
openstackgerritMerged openstack/puppet-tripleo: Haproxy has non-working Horizon session persistence.  https://review.openstack.org/25848314:16
*** karimb has joined #puppet-openstack14:24
*** foxhunt_away is now known as foxhunt7214:24
*** bklei has quit IRC14:31
*** bklei has joined #puppet-openstack14:33
openstackgerritMerged openstack/puppet-openstack-cookiecutter: Update typos  https://review.openstack.org/26317814:34
*** jpena|lunch is now known as jpena14:41
Vinshpietervanw: Hello there, you around still?14:41
openstackgerritMerged openstack/puppet-neutron: Adds the ability to force metadata with dhcp agent  https://review.openstack.org/26108414:45
*** Guest55888 is now known as saneax_AFK14:51
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-trove: update Setup section  https://review.openstack.org/26480514:52
openstackgerritLukas Bezdicka proposed openstack/puppet-ceilometer: Support identity_uri and auth_uri properly and add auth_version option  https://review.openstack.org/18228114:53
*** ducttape_ has joined #puppet-openstack14:57
*** Vinsh has quit IRC14:57
bkleiemilienm: are you ok with this change https://review.openstack.org/#/c/264394?  we're not on the latest puppet-keystone, so this change should work with old and new keystone14:58
*** dprince has quit IRC14:58
*** dprince has joined #puppet-openstack14:59
*** dfisher has joined #puppet-openstack14:59
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-designate: update Setup section  https://review.openstack.org/26481115:00
openstackgerritMerged openstack/puppet-keystone: Update typos  https://review.openstack.org/26318215:01
*** zhangjn has quit IRC15:02
*** vinsh has joined #puppet-openstack15:04
*** xarses has quit IRC15:08
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-murano: update Setup section  https://review.openstack.org/26482615:13
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Use identity_uri and auth_uri by default  https://review.openstack.org/26279915:13
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Deploy & Test Ironic  https://review.openstack.org/26296515:14
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-gnocchi: update Setup section  https://review.openstack.org/26482915:19
*** jtomasek has quit IRC15:20
openstackgerritAlex Schultz proposed openstack/puppet-modulesync-configs: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26483015:21
mwhahahaEmilienM: -^ there's the original commit that I should have done before I accidentally ran the module sync. That has the details in the commit message. once that gets all reviewed i'll update the other ones with the appropriate info15:22
EmilienMmwhahaha: awesome15:23
EmilienMI like it really15:23
EmilienMmwhahaha: commented15:24
mwhahahathanks15:24
mwhahahai'll check it out later15:25
*** jtomasek has joined #puppet-openstack15:25
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: stop managing puppet-tuskar  https://review.openstack.org/26483315:26
*** dprince has quit IRC15:32
*** dprince_ has joined #puppet-openstack15:32
*** xarses has joined #puppet-openstack15:33
*** karimb has quit IRC15:33
*** karimb has joined #puppet-openstack15:35
*** foxhunt72 is now known as foxhunt_away15:36
*** slagle has quit IRC15:39
*** dhollinger has joined #puppet-openstack15:48
*** mdorman has joined #puppet-openstack15:49
*** dprince_ has quit IRC15:51
*** dprince has joined #puppet-openstack15:51
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/20087715:54
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-aodh: update Setup section  https://review.openstack.org/26484215:56
openstackgerritMerged openstack/puppet-monasca: Fix for older versions of keystone module  https://review.openstack.org/26439416:01
*** ntpttr has joined #puppet-openstack16:08
EmilienMbklei: I don't review puppet-monasca much16:09
EmilienMbklei: the module is not in good shape imho and do not follow our coding stye16:10
EmilienMstyle*16:10
EmilienMand I don't see real efforts on your side to make it happen16:10
EmilienMso I'm not sure why I should spend some time taking care of reviewing it16:10
EmilienMthat said, I'm willing to help once you guys are ready16:10
bkleiprobably all true :)  but the change that was made recently broke us :(16:11
bkleithat auth module is way out of sync with other modules, i saw that yesterday.  will do some work to bring in-line16:11
*** baker has joined #puppet-openstack16:26
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/25488916:27
*** karimb has quit IRC16:32
*** gfidente|pto has quit IRC16:36
iurygregoryif anyone have time to review - https://review.openstack.org/#/q/topic:update_readme+is:open+owner:%22Iury+Gregory+Melo+Ferreira+%253Ciurygregory%2540gmail.com%253E%2216:40
EmilienMiurygregory: +216:44
iurygregorytks o/16:44
*** pietervanw has left #puppet-openstack16:52
*** dgurtner has quit IRC16:55
*** rlaurika has quit IRC16:56
chemEmilienM: I marked https://bugs.launchpad.net/puppet-keystone/+bug/1529374 as invalid, see https://bugs.launchpad.net/tempest/+bug/153191716:57
openstackLaunchpad bug 1529374 in puppet-keystone "keystone::endpoint::version set to empty has no effect" [High,Invalid] - Assigned to Sofer Athlan-Guyot (sofer-athlan-guyot)16:57
openstackLaunchpad bug 1531917 in tempest "Strange error when using versionless identity endpoint." [Undecided,New]16:57
*** rlaurika has joined #puppet-openstack16:57
EmilienMchem: cool ! thx17:03
EmilienMmaybe we can ping mtreinish17:03
chemEmilienM: I had a discussion with sdague17:03
chemEmilienM: he the one who told me that versionless endpoint were not supported but that the error message could be better17:04
ntpttrHey everyone, does anyone know why the keystone::auth spec started failing in https://review.openstack.org/#/c/253216/ at the gate tests? It wasn't failing before and I didn't make any changes to it, and comparing it to some other modules it looks like it should be working in the same way.17:07
chemrichm: he, this one is ready for the taking https://review.openstack.org/#/c/202689/17:07
*** rahulait has joined #puppet-openstack17:10
*** samuelBartel has quit IRC17:11
*** trown is now known as trown|lunch17:13
iurygregoryntpttr, any reason why this 2 patches have different values for common_package_name ( https://review.openstack.org/#/c/254392/2/manifests/params.pp  and https://review.openstack.org/#/c/253216/10/manifests/params.pp )17:18
EmilienMchem: that means we need to set a version for our endpoints? or just keystone17:20
iurygregoryntpttr, since they modify the same file, it's good to have dependence between them17:20
chemEmilienM: hum all of them is the safe way :)17:21
ntpttriurygregory: I think that's because I was correcting the package name in the init but forgot when adding api17:21
EmilienMchem: odd17:21
chemEmilienM: why17:21
ntpttriurygregory: I can go ahead and create the dependency, should api be depenant on init?17:21
iurygregoryyour choice XD17:22
ntpttriurygregory: fair enough :)17:22
EmilienMchem: because we did all the work to support version less and now tempest is not supporting it17:22
EmilienMchem: I don't know if it's a blocker to set the default value to '' then17:22
iurygregoryntpttr, good luck o/ i'll be happy to review it17:22
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/20087717:22
ntpttriurygregory: regardless the failure in keystone::auth is perplexing me, since no changes have been made to it since initial commit and it worked fine until those patches got approved..17:23
iurygregoryntpttr, yeah i notice that.. when i have time i'll take a look =)17:23
ntpttriurygregory: thanks :). I'll keep investigating in the meantime17:23
iurygregorythe guy from keystone is chem XD17:24
EmilienMchem: he said, 'most clients do not support it' - I guess that's a blocker17:24
chemEmilienM: well I just think that we are ahead of the curve right now.  Don't get the second part --- ah oki.  get it now17:24
chemiurygregory: was busy elsewhere, I backtrack17:24
iurygregoryyeah i notice chem =)17:25
*** ken_barber has joined #puppet-openstack17:28
ntpttrI'm thinking perhaps I need to make this change to magnum, since it's the only change I can find being made to the other keystone::auth manifests https://github.com/openstack/puppet-barbican/commit/395e69546ffac4c280d3ae1b44b7d19bcb3926bc17:32
chemntpttr: this should only make a warning disappear, but that's nice to have and that would be a requirement from mitaka on17:33
chemntpttr: if that solve it then, it will be an added bonus.17:34
chemntpttr: I'm trying to reproduce the error locally17:34
*** baker has quit IRC17:35
ntpttrchem: it looks like the unit tests that were failing match up with the ones changed to include the ::{service_name} suffix, so I'm not sure but I'm hopeful it'll work17:37
chemntpttr: as, I think I see the light... let me check 2s17:39
*** rochaporto has quit IRC17:42
openstackgerritNate Potter proposed openstack/puppet-magnum: Make Keystone_endpoint match service by name/type  https://review.openstack.org/26489317:44
ntpttr^ there's the change, it fixed the problem locally, let's see about Jenkins17:45
*** jistr has quit IRC17:45
iurygregoryntpttr, to check if this solve your problem in the other patch you can put depends-on, i think is worth to try ^^17:46
chemntpttr: I made it work using that :     it { is_expected.to contain_keystone_service('magnum::container').with(17:46
ntpttriurygregory: will just including it in the commit message do the trick?17:47
iurygregoryit should =D17:47
chemntpttr: well that what you did in the patch, so all the rspec problem will disappear17:47
ntpttrchem: yep, looking like problem solved17:48
ntpttrthanks17:48
EmilienMchem: we need to find a way to stop WARNING in puppet catalog for the versionless17:50
chemntpttr: i did nothing, and too late, but you're welcome :)17:51
EmilienMbut setting the default value to empty does look the right choice if tempest and "most of clients" (quoting sdague) do not support it.17:51
EmilienMdoes not* sorry ^17:51
EmilienMdoes not look the right choice I meant17:51
chemEmilienM: yeap I understood17:51
chemEmilienM: well we just have to revert \o/ .  I love to revert :)17:51
chemEmilienM: kidding17:51
EmilienMchem: revert? no please :)17:52
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP swift dedicated replication network  https://review.openstack.org/26452417:52
*** gchamoul has quit IRC17:52
chemEmilienM: oki, why not just remove the warning only ?17:52
EmilienMchem: that's something we can consider, yeah17:52
chemEmilienM: i'll do17:53
EmilienMmaybe a "Notice" ?17:53
chemEmilienM: ah17:53
EmilienMchem: don't worry, I can work on it, if you already have too much things in progress17:53
chemEmilienM: notice will be seen by ... verbose ?17:53
EmilienMnot sure17:53
EmilienMnot sure Notice is really useful17:53
EmilienMdocumentation and good communication would be though17:53
chemEmilienM: notify you mean, so it will appear on every run17:54
chemEmilienM: ^ agree17:54
ntpttrit appears on every run, but it kind of blends in to the background and it's hard to notice anything unless it's a warning highlighted in red17:55
openstackgerritNate Potter proposed openstack/puppet-magnum: Initial commit of api.pp  https://review.openstack.org/25439217:57
*** rahulait has quit IRC17:58
*** jpena has quit IRC17:59
*** vinsh has quit IRC18:06
*** vinsh has joined #puppet-openstack18:07
*** iurygregory has left #puppet-openstack18:07
*** iurygregory has joined #puppet-openstack18:07
*** trown|lunch is now known as trown18:08
*** paramite is now known as paramite|afk18:11
*** derekh has quit IRC18:11
*** ilbot3 has quit IRC18:11
*** ilbot3 has joined #puppet-openstack18:12
*** jayg has quit IRC18:13
*** baker has joined #puppet-openstack18:14
*** markvoelker has quit IRC18:14
*** jayg has joined #puppet-openstack18:15
*** mwoodson has quit IRC18:15
*** trown has quit IRC18:15
*** trown|brb has joined #puppet-openstack18:16
iurygregorymwhahaha can you take a look at the others? https://review.openstack.org/#/q/topic:update_readme+is:open+owner:%22Iury+Gregory+Melo+Ferreira+%253Ciurygregory%2540gmail.com%253E%2218:17
mwhahahayea i'm getting there :D18:17
iurygregorymore easy with the link XD18:18
iurygregorybtw congrats :D18:18
mwhahahathanks18:18
*** mwoodson has joined #puppet-openstack18:21
*** trown|brb is now known as trown18:21
*** cwolferh has quit IRC18:21
*** cwolferh has joined #puppet-openstack18:21
*** cwolferh has quit IRC18:26
*** rahulait has joined #puppet-openstack18:32
openstackgerritMerged openstack/puppet-aodh: update Setup section  https://review.openstack.org/26484218:34
*** jaosorior has quit IRC18:34
*** jaosorior has joined #puppet-openstack18:35
openstackgerritNate Potter proposed openstack/puppet-glance: Implement multiple store configuration  https://review.openstack.org/24271818:37
openstackgerritMerged openstack/puppet-designate: update Setup section  https://review.openstack.org/26481118:41
openstackgerritMerged openstack/puppet-gnocchi: update Setup section  https://review.openstack.org/26482918:41
*** paramite|afk is now known as paramite18:42
*** v1k0d3n has quit IRC18:43
*** v1k0d3n has joined #puppet-openstack18:45
openstackgerritMerged openstack/puppet-murano: update Setup section  https://review.openstack.org/26482618:45
*** v1k0d3n has quit IRC18:47
*** gchamoul has joined #puppet-openstack18:47
*** v1k0d3n has joined #puppet-openstack18:47
openstackgerritMerged openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/20268918:56
*** severion has joined #puppet-openstack18:57
*** v1k0d3n has quit IRC18:59
*** markvoelker has joined #puppet-openstack19:03
*** markvoelker_ has joined #puppet-openstack19:05
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/26492719:06
*** baker_ has joined #puppet-openstack19:06
*** aimon has joined #puppet-openstack19:06
chemis there a way to create a dependence to patch in another module ?19:07
EmilienMchem: Depends-On ?19:07
*** baker has quit IRC19:07
iurygregorychem, https://review.openstack.org/#/c/256576/19:08
chemEmilienM: iurygregory thats uber nice ! thanks.19:09
*** markvoelker has quit IRC19:09
iurygregory=)19:09
EmilienMchem, iurygregory: it works for unit tests, beaker and integration jobs FYI19:12
EmilienMthe secret here is that openstack infra use zuul cloner19:12
EmilienMchem: see Rakefile of all modules (for unit test)19:12
mfischmgagne: did you tell me once that you have made it so that you're not using the admin token after bootstrap?19:12
iurygregorynice to know more about, tks EmilienM19:12
mfischEmilienM: you still have a -1 on this? https://review.openstack.org/#/c/258260/19:13
*** markvoelker has joined #puppet-openstack19:14
mfischwho is alex schultz on irc?19:14
mfischchem: are you Alex?19:15
iurygregorymfisch, is mwhahaha19:15
mfischah ok19:15
mfischmwhahaha: can we talk about https://review.openstack.org/#/c/262012/19:15
iurygregorymfisch, chem is Athlan-Guyot sofer19:15
mwhahahaI am19:15
mfischthanks iurygregory19:16
mwhahahaGive me a few mins19:16
mfischsure19:16
chemmfisch: I'n not Alex indeed19:16
iurygregorynp o/19:16
mfischi'm too old to remember all this stuff19:16
*** cwolferh has joined #puppet-openstack19:17
*** markvoelker_ has quit IRC19:17
EmilienMmfisch: I can approve it19:17
EmilienMyou're so young mfisch19:17
EmilienMin your mind ;-)19:17
iurygregorylol19:17
mfischI can always be immature mentally19:17
EmilienMmfisch: +A19:18
mfischty19:18
EmilienMmfisch: https://review.openstack.org/26483319:19
EmilienMmfisch: https://review.openstack.org/26439919:19
*** paramite is now known as paramite|afk19:19
EmilienMit's weird I still see some failures on our integration jobs for scenario001 on centos19:19
EmilienMthey look random though19:19
EmilienMhttp://logs.openstack.org/87/264387/1/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/1208661/console.html#_2016-01-07_13_27_12_45919:20
mfischEmilienM: is that what shows when you run keystone service list?19:20
EmilienMif you see this kind of error, we have a problem19:20
EmilienMmfisch: no19:21
EmilienMmfisch: oh wait19:21
EmilienMthe patch?19:21
EmilienMyes19:21
EmilienMwe merged it for Glance19:21
EmilienMnow doing Cinder19:21
mfischyeah our names are like 'cinder'19:22
mfisch'cinderv2'19:22
openstackgerritMerged openstack/puppet-modulesync-configs: stop managing puppet-tuskar  https://review.openstack.org/26483319:23
EmilienMmfisch: look the commti message19:23
EmilienMit explains why such a change19:24
mfischyeah I did19:24
mfischI saw the email too I was just trying to see what we used19:24
EmilienMmfisch: https://github.com/openstack/keystone/blob/master/etc/default_catalog.templates19:24
mwhahahamfisch: what's up?19:25
EmilienMmfisch: the final goal of all of it is having standard for https://github.com/openstack/puppet-cinder/blob/master/manifests/api.pp#L13819:25
mfischmwhahaha: looking at https://review.openstack.org/#/c/262012/3/manifests/params.pp19:25
mfischisnt ubuntu in the debian os_family?19:25
mwhahahayea but that change is for the debian/ubuntu name split19:26
mwhahahaso it uses os_package_type19:26
mwhahahaunfortunately the debian and UCA packages have a few differences in service and package names19:26
mfischso os_package_type is specific debian vs ubuntu19:26
mwhahahawe've done it before around libvirt and a few other spots19:26
mwhahahayea19:26
mfischokay thats what I was missing19:27
mfischI figured they both use debs19:27
mwhahahayea so we came up with os_package_type to support running debian naming conventions on ubuntu (we do it for fuel)19:27
mwhahahasince techincally we deploy on ubuntu but our packages are debian based19:27
mwhahahaand os_package_type is a fact that can be overriden allowing us to use debian naming conventions with the upstream modules19:28
mwhahahaso that change is to handle the split for the openvswitch agent19:28
mwhahahasimilar thing over in puppet-nova, https://github.com/openstack/puppet-nova/blob/master/manifests/params.pp#L104-L12019:29
mfischok thanks19:29
EmilienMq!19:29
mwhahahahttps://github.com/openstack/puppet-openstacklib/blob/master/lib/facter/os_package_type.rb19:29
mwhahahathere's the fact itself, but you can override it with a file in /etc/facter/facts.d :)19:30
mwhahahathat's how i'm trying to wedge UCA support into fuel19:30
mfischnice19:30
EmilienMthere is something wrong with puppet-cinder19:32
EmilienMbut db-sync fails in our integration jobs19:32
EmilienMI'm investigating19:32
*** mdorman_ has joined #puppet-openstack19:32
EmilienMmwhahaha, mfisch: can we land https://review.openstack.org/263968 and https://review.openstack.org/263969 for trove backports?19:33
*** tdb has joined #puppet-openstack19:33
mwhahahai'm ok with it19:33
mfischwfm19:34
EmilienMI also would like to land puppet-rally https://review.openstack.org/#/c/262970/19:34
EmilienMso Andy and I can follow up to add his bits19:34
mgagnemfisch we have privileged keystone nodes. we provision keystone resources on them and remove the admin_token middleware on the other non-privileged nodes.19:34
EmilienMand we could even start functional testing19:34
mwhahahasure let me take a look at puppet-rally19:35
mfischmgagne: do you have any use for the admin token after bootstrap?19:35
mfischpuppet or otherwise19:35
*** kgalanov_ has joined #puppet-openstack19:35
mgagnemfisch we don't use the admin_token (users or system) other than with puppet by provisioning keystone resources on the privileged nodes.19:36
mfischmgagne: once puppet bootstraps admin is the token still used?19:36
mgagnemfisch bootstrap really happens on the first puppet run19:36
mfischright19:36
mfischthats what I thought19:36
*** joshsamuelson has joined #puppet-openstack19:37
mgagnemfisch yes due to subsequent puppet runs19:37
*** beddari_ has joined #puppet-openstack19:37
EmilienMmgagne: how many do you need?19:37
mgagneone19:37
mfischmgagne: so puppet expects that token to be there every time even if it has nothing to do19:37
mgagnecatalog is templated so no exported resources19:37
EmilienMmgagne: nice19:38
mgagnemfisch only if you use keystone puppet ressources AFAIK19:38
EmilienMmgagne: one puppet run that is idempotent? that's really cool19:38
mfischwe have users and endpoints19:38
*** mdorman has quit IRC19:38
*** dfisher has quit IRC19:38
*** tdb_ has quit IRC19:38
*** kbyrne has quit IRC19:38
*** kgalanov has quit IRC19:38
*** jprs has quit IRC19:38
*** beddari has quit IRC19:38
*** beddari_ is now known as beddari19:38
*** mdorman_ is now known as mdorman19:38
mfischmgagne: how do you disable the middwalre19:38
mfischmiddleware19:38
mgagnemfisch in paste config19:38
mgagnepipeline:public_api/pipeline, pipeline:admin_api/pipeline and pipeline:api_v3/pipeline19:39
*** kgalanov_ is now known as kgalanov19:39
mgagneremove admin_token_auth19:39
mfischmgagne: thinking that if I do that after bootstrap it should be harmless19:39
mfischbut it will mean that puppet cant make new users/endpoints19:40
mgagnemfisch yea, that's why we use privileged keystone nodes which aren't public19:40
mfischhas anyone looked into having puppet not need the admin token after bootstrap?19:40
openstackgerritMerged openstack/puppet-cinder: Default service_name to 'Volume Service'  https://review.openstack.org/26439919:42
*** dfisher has joined #puppet-openstack19:42
*** kbyrne has joined #puppet-openstack19:42
mwhahahahttps://github.com/openstack/fuel-library/blob/master/deployment/puppet/osnailyfacter/modular/astute/service_token_off.pp19:43
mwhahahawe pull it after we deploy our controllers19:43
mwhahahabut we don't add more users after the fact i don't think19:44
mtreinishEmilienM: you pinged me earlier about something?19:44
mfischmwhahaha: we dont usually either19:44
mfischmwhahaha: why exec and not file-line?19:44
mwhahahawe're tasked based19:44
mwhahahaso we execute that last19:44
mfischhow do you control that running last?19:45
mwhahahagoes with how fuel determines tasks19:45
mwhahahait's complicated :D19:45
mfischI could boostrap with the token and then use this code to remove it19:45
mfischwe could add anything after the fact by hand or with ansible19:46
mwhahahaoh and it uses sed because I think they were trying to handle where the paste.ini might change over time. This just pulls the admin_auth_token item out of the already existing config19:47
EmilienMmtreinish: yeah, we have found https://bugs.launchpad.net/tempest/+bug/153191719:47
openstackLaunchpad bug 1531917 in tempest "Strange error when using versionless identity endpoint." [Undecided,New]19:47
EmilienMI wanted your insights19:47
EmilienMif we should default our modules to use versionless at the risk of not able to use tempest or some openstack clients19:47
openstackgerritMerged openstack/puppet-ironic: Allow to configure api_workers  https://review.openstack.org/26296619:48
openstackgerritMerged openstack/puppet-neutron: Fixes OVS agent pkg and service name  https://review.openstack.org/26201219:48
openstackgerritMerged openstack/puppet-nova: Deprecate nova::utilities  https://review.openstack.org/25826019:48
openstackgerritMerged openstack/puppet-trove: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/26396819:49
mtreinishEmilienM: tempest needs to have the version in the url, bknudson_ is trying to implement version discovery in tempest-lib with: https://review.openstack.org/#/q/status:open+project:openstack/tempest-lib+branch:master+topic:bug/153018119:49
EmilienMnice19:49
openstackgerritMerged openstack/puppet-rally: puppet-rally: Initial commit  https://review.openstack.org/26297019:49
EmilienMmtreinish: ok so that means keystone endpoint need to have the version (when doing openstack endpoint list)19:50
mfischmwhahaha: what does puppet do when the admin token is absent and the keystone resources run?19:50
EmilienMmtreinish: isn't enough to have the version in tempest.conf?19:50
mwhahahawell we've setup an openrc file so I'd assume it uses that19:50
mtreinishEmilienM: it needs a version in the service catalog iirc19:51
EmilienMmhh ok19:51
EmilienMchem: ^19:51
mtreinishit's mostly because there isnt a uniform way to do version discovery across projects19:51
EmilienMmtreinish: good to know19:51
EmilienMI guess we'll have to deal with that and set the version19:52
chemEmilienM ack (I have tried to put a versionless in tempest.conf but to no good)19:52
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Implement WSGI support for Ironic API  https://review.openstack.org/26345020:05
openstackgerritMerged openstack/puppet-neutron: Make neutron::server inherit from neutron::params  https://review.openstack.org/26387520:06
openstackgerritMerged openstack/puppet-trove: update Setup section  https://review.openstack.org/26480520:08
openstackgerritMerged openstack/puppet-ironic: [inspector] Specify SELinux type context for tftp dir  https://review.openstack.org/26418020:10
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: Use identity_uri and auth_uri by default  https://review.openstack.org/26279920:12
openstackgerritMerged openstack/puppet-trove: Follow-up on PyMySQL support for Red Hat platforms  https://review.openstack.org/26396920:14
*** foxhunt_away is now known as foxhunt7220:18
*** openstackgerrit has quit IRC20:23
*** openstackgerrit has joined #puppet-openstack20:25
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/26495120:25
openstackgerritMerged openstack/puppet-trove: Typo in README.md  https://review.openstack.org/25751520:27
*** gchamoul has quit IRC20:30
*** migi has quit IRC20:31
*** gchamoul has joined #puppet-openstack20:32
*** migi has joined #puppet-openstack20:32
*** dprince has quit IRC20:33
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/26492720:44
EmilienMmfisch, mwhahaha: looks good to you? https://review.openstack.org/#/c/262965/20:57
EmilienMwe have the integ jobs running for puppet-ironic now20:58
EmilienMwill be very useful imho20:58
EmilienMI can't reproduce the cinder issue in my env21:01
EmilienMthat's weird21:01
*** bklei has quit IRC21:01
*** jayg is now known as jayg|g0n321:02
EmilienMmwhahaha: also https://review.openstack.org/263760 we need it for mitaka21:07
EmilienMthanks21:07
mwhahahak i'll look21:07
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP Support for erasure code in swift  https://review.openstack.org/26496621:07
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/25563521:07
EmilienMmwhahaha: another blocker for mitaka: https://review.openstack.org/#/c/257522/ (you +1-ed)21:09
EmilienMoh there is a comment21:09
openstackgerritAdam Vinsh proposed openstack/puppet-swift: WIP Support for erasure code in swift  https://review.openstack.org/26496621:11
mwhahahaoh there was just a tab21:12
EmilienMso good news21:13
EmilienMubuntu packaging seems ready for mitaka21:13
mwhahahainteresting21:13
EmilienMdmsimard: my next focus it getting rdo working too21:13
EmilienMso we can bump puppet CI to mitaka!21:13
EmilienMthe blocker will be to have a tested rpo for rdo21:14
vinshGood work!21:16
openstackgerritMerged openstack/puppet-openstack-integration: scenario002 swift ring devices >= replica count  https://review.openstack.org/26376021:17
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/25563521:17
*** cwolferh has quit IRC21:18
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/25563521:18
EmilienMthis patch should tell us if mitaka works for our scenarios ^21:18
*** bklei has joined #puppet-openstack21:35
openstackgerritMerged openstack/puppet-openstacklib: Fix handling of ensure_absent_val  https://review.openstack.org/26471021:35
openstackgerritMerged openstack/puppet-nova: Drop use of deprecated neutron auth options  https://review.openstack.org/25752221:39
openstackgerritMerged openstack/puppet-designate: Configure rabbit_virtual_host  https://review.openstack.org/24446921:40
openstackgerritMerged openstack/puppet-ironic: Enable changing list of inspection collectors  https://review.openstack.org/26323021:43
*** paramite|afk is now known as paramite21:43
*** cwolferh has joined #puppet-openstack21:45
*** bklei has quit IRC21:45
*** paramite is now known as paramite|afk21:48
*** beddari has quit IRC21:50
*** kgalanov has quit IRC21:50
*** jaosorior has quit IRC21:50
*** rlaurika has quit IRC21:50
*** llua has quit IRC21:50
*** beddari has joined #puppet-openstack21:51
*** kgalanov has joined #puppet-openstack21:51
*** jaosorior has joined #puppet-openstack21:51
*** rlaurika has joined #puppet-openstack21:51
*** llua has joined #puppet-openstack21:51
EmilienMI've noticed that error in tempest: http://logs.openstack.org/99/262799/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/d4d162d/console.html#_2016-01-07_20_44_26_78721:54
EmilienMmaybe worth to investigate21:54
EmilienMlooks like related to this log: http://logs.openstack.org/99/262799/4/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-trusty/d4d162d/logs/cinder/cinder-api.txt.gz#_2016-01-07_20_44_24_18821:55
*** baker_ has quit IRC21:58
*** trown is now known as trown|outttypeww22:04
*** bklei has joined #puppet-openstack22:12
*** bklei has quit IRC22:13
*** bklei has joined #puppet-openstack22:13
*** mdorman has quit IRC22:16
mfischEmilienM: hey22:19
*** xarses has quit IRC22:30
*** jtomasek has quit IRC22:45
*** xarses has joined #puppet-openstack22:48
*** foxhunt72 is now known as foxhunt_away22:49
*** mdorman has joined #puppet-openstack22:55
openstackgerritCody Herriges proposed openstack/puppet-vswitch: Remove duplicate loading of constants  https://review.openstack.org/26500523:11
EmilienMmfisch: yo23:14
*** dhollinger has quit IRC23:16
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/26492723:19
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/26495123:21
*** bklei has quit IRC23:23
*** chem has quit IRC23:37
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/26495123:43
*** ducttape_ has quit IRC23:51
*** ducttape_ has joined #puppet-openstack23:51
*** dfisher has quit IRC23:53
*** ducttape_ has quit IRC23:55

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