Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-01-07

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:43 zhangjn joined #puppet-openstack
00:53 bklei joined #puppet-openstack
01:06 zhangjn joined #puppet-openstack
01:12 ducttape_ joined #puppet-openstack
01:20 bklei joined #puppet-openstack
01:33 angdraug joined #puppet-openstack
02:20 ducttape_ joined #puppet-openstack
02:23 bklei joined #puppet-openstack
02:38 zhangjn joined #puppet-openstack
02:45 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP swift dedicated replication network  https://review.openstack.org/264524
02:47 ilbot3 joined #puppet-openstack
02:47 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
02:59 bklei joined #puppet-openstack
03:02 bklei joined #puppet-openstack
03:03 bklei_ joined #puppet-openstack
03:22 bklei joined #puppet-openstack
03:49 openstackgerrit Xingchao Yu proposed openstack/puppet-trove: Add rabbit_ha_queues option  https://review.openstack.org/214458
03:54 v1k0d3n joined #puppet-openstack
04:03 baker joined #puppet-openstack
04:04 openstackgerrit Xingchao Yu proposed openstack/puppet-trove: Introduce amqp_durable_queues option  https://review.openstack.org/214477
04:08 baker joined #puppet-openstack
04:09 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Use rspec-puppet-facts for fact management  https://review.openstack.org/242330
04:10 llua^_^AznSmile- joined #puppet-openstack
04:10 baker_ joined #puppet-openstack
04:18 llua joined #puppet-openstack
04:18 bklei joined #puppet-openstack
04:27 openstackgerrit Alex Schultz proposed openstack/puppet-aodh: Update to pull in rspec-puppet-facts  https://review.openstack.org/264538
04:27 openstackgerrit Alex Schultz proposed openstack/puppet-barbican: Update to pull in rspec-puppet-facts  https://review.openstack.org/264539
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-ceilometer: Update to pull in rspec-puppet-facts  https://review.openstack.org/264540
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Update to pull in rspec-puppet-facts  https://review.openstack.org/264541
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-designate: Update to pull in rspec-puppet-facts  https://review.openstack.org/264542
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-glance: Update to pull in rspec-puppet-facts  https://review.openstack.org/264543
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-gnocchi: Update to pull in rspec-puppet-facts  https://review.openstack.org/264544
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-heat: Update to pull in rspec-puppet-facts  https://review.openstack.org/264545
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-horizon: Update to pull in rspec-puppet-facts  https://review.openstack.org/264546
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-ironic: Update to pull in rspec-puppet-facts  https://review.openstack.org/264547
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-keystone: Update to pull in rspec-puppet-facts  https://review.openstack.org/264548
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-magnum: Update to pull in rspec-puppet-facts  https://review.openstack.org/264549
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-manila: Update to pull in rspec-puppet-facts  https://review.openstack.org/264550
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-mistral: Update to pull in rspec-puppet-facts  https://review.openstack.org/264551
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-murano: Update to pull in rspec-puppet-facts  https://review.openstack.org/264552
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-neutron: Update to pull in rspec-puppet-facts  https://review.openstack.org/264553
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Update to pull in rspec-puppet-facts  https://review.openstack.org/264554
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-openstacklib: Update to pull in rspec-puppet-facts  https://review.openstack.org/264555
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-openstack_extras: Update to pull in rspec-puppet-facts  https://review.openstack.org/264556
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-rally: Update to pull in rspec-puppet-facts  https://review.openstack.org/264557
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-sahara: Update to pull in rspec-puppet-facts  https://review.openstack.org/264558
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-swift: Update to pull in rspec-puppet-facts  https://review.openstack.org/264559
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-tempest: Update to pull in rspec-puppet-facts  https://review.openstack.org/264560
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-tripleo: Update to pull in rspec-puppet-facts  https://review.openstack.org/264561
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-trove: Update to pull in rspec-puppet-facts  https://review.openstack.org/264562
04:28 openstackgerrit Alex Schultz proposed openstack/puppet-tuskar: Update to pull in rspec-puppet-facts  https://review.openstack.org/264563
04:29 openstackgerrit Alex Schultz proposed openstack/puppet-vitrage: Update to pull in rspec-puppet-facts  https://review.openstack.org/264564
04:29 openstackgerrit Alex Schultz proposed openstack/puppet-vswitch: Update to pull in rspec-puppet-facts  https://review.openstack.org/264565
04:29 openstackgerrit Alex Schultz proposed openstack/puppet-zaqar: Update to pull in rspec-puppet-facts  https://review.openstack.org/264566
04:35 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Update tests to leverage rspec-puppet-facts  https://review.openstack.org/242330
04:35 openstackgerrit Merged openstack/puppet-swift: Add support to change meta entry for ring devices  https://review.openstack.org/258677
04:37 mwhahaha oops probably shouldn't have run that msync yet :/ my bad
04:45 jaosorior joined #puppet-openstack
04:45 openstackgerrit Merged openstack/puppet-designate: Update designate::db class to match other module pattern  https://review.openstack.org/227217
04:54 baker joined #puppet-openstack
04:58 openstackgerrit Merged openstack/puppet-sahara: Do not show warning on sahara/notify  https://review.openstack.org/261481
05:03 openstackgerrit Alex Schultz proposed openstack/puppet-designate: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/263475
05:21 openstackgerrit Alex Schultz proposed openstack/puppet-designate: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/263475
05:23 zhangjn joined #puppet-openstack
05:34 zhangjn_ joined #puppet-openstack
05:45 zhangjn joined #puppet-openstack
05:54 zhangjn joined #puppet-openstack
06:00 zhangjn joined #puppet-openstack
06:02 zhangjn joined #puppet-openstack
06:06 bklei joined #puppet-openstack
06:07 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Update tests to leverage rspec-puppet-facts  https://review.openstack.org/242330
06:11 openstackgerrit Alex Schultz proposed openstack/puppet-designate: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/263475
06:35 zhangjn joined #puppet-openstack
06:36 dgurtner joined #puppet-openstack
06:36 dgurtner joined #puppet-openstack
06:36 zhangjn joined #puppet-openstack
06:55 itsuugo joined #puppet-openstack
06:58 dgurtner joined #puppet-openstack
07:04 zhangjn joined #puppet-openstack
07:11 markvoelker joined #puppet-openstack
07:21 dgurtner joined #puppet-openstack
07:21 dgurtner joined #puppet-openstack
07:53 jtomasek joined #puppet-openstack
07:54 bklei joined #puppet-openstack
07:55 dgurtner joined #puppet-openstack
07:55 dgurtner joined #puppet-openstack
08:05 zhangjn joined #puppet-openstack
08:21 jpena joined #puppet-openstack
08:30 jtomasek joined #puppet-openstack
08:35 gfidente joined #puppet-openstack
08:35 gfidente joined #puppet-openstack
08:39 samuelBartel joined #puppet-openstack
08:39 tdb joined #puppet-openstack
08:48 mkoderer_cloud joined #puppet-openstack
08:56 tdb_ joined #puppet-openstack
08:57 openstackgerrit Xingchao Yu proposed openstack/puppet-glance: Add rabbit_ha_queues option  https://review.openstack.org/215500
08:58 samuelBartel joined #puppet-openstack
09:12 markvoelker joined #puppet-openstack
09:32 jaosorior joined #puppet-openstack
09:35 derekh joined #puppet-openstack
09:40 rochaporto joined #puppet-openstack
09:41 jistr joined #puppet-openstack
09:42 bklei joined #puppet-openstack
09:59 zhangjn joined #puppet-openstack
10:06 paramite joined #puppet-openstack
10:22 openstackgerrit Gael Chamoulaud proposed openstack/puppet-trove: Use identity_uri and auth_uri in api module  https://review.openstack.org/233229
10:35 jaosorior joined #puppet-openstack
10:54 openstackgerrit Martin Mágr proposed openstack/puppet-openstacklib: Fix handling of ensure_absent_val  https://review.openstack.org/264710
10:54 openstackgerrit Martin Mágr proposed openstack/puppet-neutron: LBaaSv2 support  https://review.openstack.org/234728
10:59 openstackgerrit Merged openstack/puppet-openstacklib: Update os_database_connection with extra param  https://review.openstack.org/262438
11:13 markvoelker joined #puppet-openstack
11:30 karimb joined #puppet-openstack
11:30 bklei joined #puppet-openstack
11:50 karimb joined #puppet-openstack
11:58 zhangjn joined #puppet-openstack
11:59 zhangjn joined #puppet-openstack
11:59 zhangjn joined #puppet-openstack
12:01 zhangjn joined #puppet-openstack
12:13 karimb joined #puppet-openstack
12:14 markvoelker joined #puppet-openstack
12:14 pietervanw joined #puppet-openstack
12:14 EmilienM good morning
12:15 pietervanw hi all, good morning! (or afternoon where I am located :) )
12:17 pietervanw I'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:22 pietervanw I'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:47 EmilienM pietervanw: Vinsh is our puppet-swift expert, he might help but maybe early now for him
12:50 delattec joined #puppet-openstack
13:03 iurygregory morning =)
13:06 zhangjn joined #puppet-openstack
13:07 dprince joined #puppet-openstack
13:07 morazi joined #puppet-openstack
13:10 ducttape_ joined #puppet-openstack
13:17 Madkiss joined #puppet-openstack
13:17 Madkiss joined #puppet-openstack
13:18 markvoelker joined #puppet-openstack
13:19 bklei joined #puppet-openstack
13:20 pietervanw EmilienM: ok, thanks! I'm patient :)
13:43 bklei joined #puppet-openstack
13:55 richm joined #puppet-openstack
13:59 EmilienM mwhahaha: it would be nice to explain what you're doing by a commit message in https://review.openstack.org/264562 and other patches
13:59 mwhahaha Yea sorry I goofed on that
13:59 mwhahaha I'll fix it up
14:00 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-neutron: Correctly parse csv even with extra output.  https://review.openstack.org/263874
14:00 mwhahaha Wasn't paying attention :/
14:00 iurygregory it happens =)
14:02 EmilienM reading the commit message is the first thing I do when I review :)
14:05 delatte joined #puppet-openstack
14:15 openstackgerrit Merged openstack/puppet-tripleo: Upgrade all packages after puppet managed ones  https://review.openstack.org/261041
14:16 openstackgerrit Merged openstack/puppet-tripleo: Haproxy has non-working Horizon session persistence.  https://review.openstack.org/258483
14:24 karimb joined #puppet-openstack
14:33 bklei joined #puppet-openstack
14:34 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: Update typos  https://review.openstack.org/263178
14:41 Vinsh pietervanw: Hello there, you around still?
14:45 openstackgerrit Merged openstack/puppet-neutron: Adds the ability to force metadata with dhcp agent  https://review.openstack.org/261084
14:52 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-trove: update Setup section  https://review.openstack.org/264805
14:53 openstackgerrit Lukas Bezdicka proposed openstack/puppet-ceilometer: Support identity_uri and auth_uri properly and add auth_version option  https://review.openstack.org/182281
14:57 ducttape_ joined #puppet-openstack
14:58 bklei emilienm: 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 keystone
14:59 dprince joined #puppet-openstack
14:59 dfisher joined #puppet-openstack
15:00 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-designate: update Setup section  https://review.openstack.org/264811
15:01 openstackgerrit Merged openstack/puppet-keystone: Update typos  https://review.openstack.org/263182
15:04 vinsh joined #puppet-openstack
15:13 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-murano: update Setup section  https://review.openstack.org/264826
15:13 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Use identity_uri and auth_uri by default  https://review.openstack.org/262799
15:14 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Deploy & Test Ironic  https://review.openstack.org/262965
15:19 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-gnocchi: update Setup section  https://review.openstack.org/264829
15:21 openstackgerrit Alex Schultz proposed openstack/puppet-modulesync-configs: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/264830
15:22 mwhahaha EmilienM: -^ 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 info
15:23 EmilienM mwhahaha: awesome
15:23 EmilienM I like it really
15:24 EmilienM mwhahaha: commented
15:24 mwhahaha thanks
15:25 mwhahaha i'll check it out later
15:25 jtomasek joined #puppet-openstack
15:26 openstackgerrit Emilien Macchi proposed openstack/puppet-modulesync-configs: stop managing puppet-tuskar  https://review.openstack.org/264833
15:32 dprince_ joined #puppet-openstack
15:33 xarses joined #puppet-openstack
15:35 karimb joined #puppet-openstack
15:48 dhollinger joined #puppet-openstack
15:49 mdorman joined #puppet-openstack
15:51 dprince joined #puppet-openstack
15:54 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/200877
15:56 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-aodh: update Setup section  https://review.openstack.org/264842
16:01 openstackgerrit Merged openstack/puppet-monasca: Fix for older versions of keystone module  https://review.openstack.org/264394
16:08 ntpttr joined #puppet-openstack
16:09 EmilienM bklei: I don't review puppet-monasca much
16:10 EmilienM bklei: the module is not in good shape imho and do not follow our coding stye
16:10 EmilienM style*
16:10 EmilienM and I don't see real efforts on your side to make it happen
16:10 EmilienM so I'm not sure why I should spend some time taking care of reviewing it
16:10 EmilienM that said, I'm willing to help once you guys are ready
16:11 bklei probably all true :)  but the change that was made recently broke us :(
16:11 bklei that auth module is way out of sync with other modules, i saw that yesterday.  will do some work to bring in-line
16:26 baker joined #puppet-openstack
16:27 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/254889
16:40 iurygregory if anyone have time to review - https://review.openstack.org/#/q/topic:update_readme+is:open+owner:%22Iury+Gregory+Melo+Ferreira+%253Ciurygregory%2540gmail.com%253E%22
16:44 EmilienM iurygregory: +2
16:44 iurygregory tks o/
16:52 pietervanw left #puppet-openstack
16:57 chem EmilienM: I marked https://bugs.launchpad.net/puppet-keystone/+bug/1529374 as invalid, see https://bugs.launchpad.net/tempest/+bug/1531917
16:57 openstack Launchpad 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 openstack Launchpad bug 1531917 in tempest "Strange error when using versionless identity endpoint." [Undecided,New]
16:57 rlaurika joined #puppet-openstack
17:03 EmilienM chem: cool ! thx
17:03 EmilienM maybe we can ping mtreinish
17:03 chem EmilienM: I had a discussion with sdague
17:04 chem EmilienM: he the one who told me that versionless endpoint were not supported but that the error message could be better
17:07 ntpttr Hey 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 chem richm: he, this one is ready for the taking https://review.openstack.org/#/c/202689/
17:10 rahulait joined #puppet-openstack
17:18 iurygregory ntpttr, 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:20 EmilienM chem: that means we need to set a version for our endpoints? or just keystone
17:20 iurygregory ntpttr, since they modify the same file, it's good to have dependence between them
17:21 chem EmilienM: hum all of them is the safe way :)
17:21 ntpttr iurygregory: I think that's because I was correcting the package name in the init but forgot when adding api
17:21 EmilienM chem: odd
17:21 chem EmilienM: why
17:21 ntpttr iurygregory: I can go ahead and create the dependency, should api be depenant on init?
17:22 iurygregory your choice XD
17:22 ntpttr iurygregory: fair enough :)
17:22 EmilienM chem: because we did all the work to support version less and now tempest is not supporting it
17:22 EmilienM chem: I don't know if it's a blocker to set the default value to '' then
17:22 iurygregory ntpttr, good luck o/ i'll be happy to review it
17:22 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/200877
17:23 ntpttr iurygregory: 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 iurygregory ntpttr, yeah i notice that.. when i have time i'll take a look =)
17:23 ntpttr iurygregory: thanks :). I'll keep investigating in the meantime
17:24 iurygregory the guy from keystone is chem XD
17:24 EmilienM chem: he said, 'most clients do not support it' - I guess that's a blocker
17:24 chem EmilienM: well I just think that we are ahead of the curve right now.  Don't get the second part --- ah oki.  get it now
17:24 chem iurygregory: was busy elsewhere, I backtrack
17:25 iurygregory yeah i notice chem =)
17:28 ken_barber joined #puppet-openstack
17:32 ntpttr I'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/395e69546ffac4c280d3ae1b44b7d19bcb3926bc
17:33 chem ntpttr: this should only make a warning disappear, but that's nice to have and that would be a requirement from mitaka on
17:34 chem ntpttr: if that solve it then, it will be an added bonus.
17:34 chem ntpttr: I'm trying to reproduce the error locally
17:37 ntpttr chem: 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 work
17:39 chem ntpttr: as, I think I see the light... let me check 2s
17:44 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Make Keystone_endpoint match service by name/type  https://review.openstack.org/264893
17:45 ntpttr ^ there's the change, it fixed the problem locally, let's see about Jenkins
17:46 iurygregory ntpttr, to check if this solve your problem in the other patch you can put depends-on, i think is worth to try ^^
17:46 chem ntpttr: I made it work using that :     it { is_expected.to contain_keystone_service('magnum::container').with(
17:47 ntpttr iurygregory: will just including it in the commit message do the trick?
17:47 iurygregory it should =D
17:47 chem ntpttr: well that what you did in the patch, so all the rspec problem will disappear
17:48 ntpttr chem: yep, looking like problem solved
17:48 ntpttr thanks
17:50 EmilienM chem: we need to find a way to stop WARNING in puppet catalog for the versionless
17:51 chem ntpttr: i did nothing, and too late, but you're welcome :)
17:51 EmilienM but 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 EmilienM does not* sorry ^
17:51 EmilienM does not look the right choice I meant
17:51 chem EmilienM: yeap I understood
17:51 chem EmilienM: well we just have to revert \o/ .  I love to revert :)
17:51 chem EmilienM: kidding
17:52 EmilienM chem: revert? no please :)
17:52 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP swift dedicated replication network  https://review.openstack.org/264524
17:52 chem EmilienM: oki, why not just remove the warning only ?
17:52 EmilienM chem: that's something we can consider, yeah
17:53 chem EmilienM: i'll do
17:53 EmilienM maybe a "Notice" ?
17:53 chem EmilienM: ah
17:53 EmilienM chem: don't worry, I can work on it, if you already have too much things in progress
17:53 chem EmilienM: notice will be seen by ... verbose ?
17:53 EmilienM not sure
17:53 EmilienM not sure Notice is really useful
17:53 EmilienM documentation and good communication would be though
17:54 chem EmilienM: notify you mean, so it will appear on every run
17:54 chem EmilienM: ^ agree
17:55 ntpttr it 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 red
17:57 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Initial commit of api.pp  https://review.openstack.org/254392
18:07 vinsh joined #puppet-openstack
18:07 iurygregory left #puppet-openstack
18:07 iurygregory joined #puppet-openstack
18:12 ilbot3 joined #puppet-openstack
18:12 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
18:14 baker joined #puppet-openstack
18:15 jayg joined #puppet-openstack
18:16 trown|brb joined #puppet-openstack
18:17 iurygregory mwhahaha 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%22
18:17 mwhahaha yea i'm getting there :D
18:18 iurygregory more easy with the link XD
18:18 iurygregory btw congrats :D
18:18 mwhahaha thanks
18:21 mwoodson joined #puppet-openstack
18:21 cwolferh joined #puppet-openstack
18:32 rahulait joined #puppet-openstack
18:34 openstackgerrit Merged openstack/puppet-aodh: update Setup section  https://review.openstack.org/264842
18:35 jaosorior joined #puppet-openstack
18:37 openstackgerrit Nate Potter proposed openstack/puppet-glance: Implement multiple store configuration  https://review.openstack.org/242718
18:41 openstackgerrit Merged openstack/puppet-designate: update Setup section  https://review.openstack.org/264811
18:41 openstackgerrit Merged openstack/puppet-gnocchi: update Setup section  https://review.openstack.org/264829
18:45 v1k0d3n joined #puppet-openstack
18:45 openstackgerrit Merged openstack/puppet-murano: update Setup section  https://review.openstack.org/264826
18:47 gchamoul joined #puppet-openstack
18:47 v1k0d3n joined #puppet-openstack
18:56 openstackgerrit Merged openstack/puppet-keystone: Resource keystone_identity_provider for Keystone.  https://review.openstack.org/202689
18:57 severion joined #puppet-openstack
19:03 markvoelker joined #puppet-openstack
19:05 markvoelker_ joined #puppet-openstack
19:06 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/264927
19:06 baker_ joined #puppet-openstack
19:06 aimon joined #puppet-openstack
19:07 chem is there a way to create a dependence to patch in another module ?
19:07 EmilienM chem: Depends-On ?
19:08 iurygregory chem, https://review.openstack.org/#/c/256576/
19:09 chem EmilienM: iurygregory thats uber nice ! thanks.
19:09 iurygregory =)
19:12 EmilienM chem, iurygregory: it works for unit tests, beaker and integration jobs FYI
19:12 EmilienM the secret here is that openstack infra use zuul cloner
19:12 EmilienM chem: see Rakefile of all modules (for unit test)
19:12 mfisch mgagne: did you tell me once that you have made it so that you're not using the admin token after bootstrap?
19:12 iurygregory nice to know more about, tks EmilienM
19:13 mfisch EmilienM: you still have a -1 on this? https://review.openstack.org/#/c/258260/
19:14 markvoelker joined #puppet-openstack
19:14 mfisch who is alex schultz on irc?
19:15 mfisch chem: are you Alex?
19:15 iurygregory mfisch, is mwhahaha
19:15 mfisch ah ok
19:15 iurygregory mfisch, chem is Athlan-Guyot sofer
19:15 mfisch mwhahaha: can we talk about https://review.openstack.org/#/c/262012/
19:15 mwhahaha I am
19:16 mfisch thanks iurygregory
19:16 mwhahaha Give me a few mins
19:16 mfisch sure
19:16 chem mfisch: I'n not Alex indeed
19:16 iurygregory np o/
19:16 mfisch i'm too old to remember all this stuff
19:17 cwolferh joined #puppet-openstack
19:17 EmilienM mfisch: I can approve it
19:17 EmilienM you're so young mfisch
19:17 EmilienM in your mind ;-)
19:17 iurygregory lol
19:17 mfisch I can always be immature mentally
19:18 EmilienM mfisch: +A
19:18 mfisch ty
19:19 EmilienM mfisch: https://review.openstack.org/264833
19:19 EmilienM mfisch: https://review.openstack.org/264399
19:19 EmilienM it's weird I still see some failures on our integration jobs for scenario001 on centos
19:19 EmilienM they look random though
19:20 EmilienM http://logs.openstack.org/87/264387/1/check/gate-puppet-openstack-integration-scenario001-tempest-dsvm-centos7/1208661/console.html#_2016-01-07_13_27_12_459
19:20 EmilienM if you see this kind of error, we have a problem
19:20 mfisch EmilienM: is that what shows when you run keystone service list?
19:21 EmilienM mfisch: no
19:21 EmilienM mfisch: oh wait
19:21 EmilienM the patch?
19:21 EmilienM yes
19:21 EmilienM we merged it for Glance
19:21 EmilienM now doing Cinder
19:22 mfisch yeah our names are like 'cinder'
19:22 mfisch 'cinderv2'
19:23 openstackgerrit Merged openstack/puppet-modulesync-configs: stop managing puppet-tuskar  https://review.openstack.org/264833
19:23 EmilienM mfisch: look the commti message
19:24 EmilienM it explains why such a change
19:24 mfisch yeah I did
19:24 mfisch I saw the email too I was just trying to see what we used
19:24 EmilienM mfisch: https://github.com/openstack/keystone/blob/master/etc/default_catalog.templates
19:25 mwhahaha mfisch: what's up?
19:25 EmilienM mfisch: the final goal of all of it is having standard for https://github.com/openstack/puppet-cinder/blob/master/manifests/api.pp#L138
19:25 mfisch mwhahaha: looking at https://review.openstack.org/#/c/262012/3/manifests/params.pp
19:25 mfisch isnt ubuntu in the debian os_family?
19:26 mwhahaha yea but that change is for the debian/ubuntu name split
19:26 mwhahaha so it uses os_package_type
19:26 mwhahaha unfortunately the debian and UCA packages have a few differences in service and package names
19:26 mfisch so os_package_type is specific debian vs ubuntu
19:26 mwhahaha we've done it before around libvirt and a few other spots
19:26 mwhahaha yea
19:27 mfisch okay thats what I was missing
19:27 mfisch I figured they both use debs
19:27 mwhahaha yea so we came up with os_package_type to support running debian naming conventions on ubuntu (we do it for fuel)
19:27 mwhahaha since techincally we deploy on ubuntu but our packages are debian based
19:28 mwhahaha and os_package_type is a fact that can be overriden allowing us to use debian naming conventions with the upstream modules
19:28 mwhahaha so that change is to handle the split for the openvswitch agent
19:29 mwhahaha similar thing over in puppet-nova, https://github.com/openstack/puppet-nova/blob/master/manifests/params.pp#L104-L120
19:29 mfisch ok thanks
19:29 EmilienM q!
19:29 mwhahaha https://github.com/openstack/puppet-openstacklib/blob/master/lib/facter/os_package_type.rb
19:30 mwhahaha there's the fact itself, but you can override it with a file in /etc/facter/facts.d :)
19:30 mwhahaha that's how i'm trying to wedge UCA support into fuel
19:30 mfisch nice
19:32 EmilienM there is something wrong with puppet-cinder
19:32 EmilienM but db-sync fails in our integration jobs
19:32 EmilienM I'm investigating
19:32 mdorman_ joined #puppet-openstack
19:33 EmilienM mwhahaha, mfisch: can we land https://review.openstack.org/263968 and https://review.openstack.org/263969 for trove backports?
19:33 tdb joined #puppet-openstack
19:33 mwhahaha i'm ok with it
19:34 mfisch wfm
19:34 EmilienM I also would like to land puppet-rally https://review.openstack.org/#/c/262970/
19:34 EmilienM so Andy and I can follow up to add his bits
19:34 mgagne mfisch 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 EmilienM and we could even start functional testing
19:35 mwhahaha sure let me take a look at puppet-rally
19:35 mfisch mgagne: do you have any use for the admin token after bootstrap?
19:35 mfisch puppet or otherwise
19:35 kgalanov_ joined #puppet-openstack
19:36 mgagne mfisch we don't use the admin_token (users or system) other than with puppet by provisioning keystone resources on the privileged nodes.
19:36 mfisch mgagne: once puppet bootstraps admin is the token still used?
19:36 mgagne mfisch bootstrap really happens on the first puppet run
19:36 mfisch right
19:36 mfisch thats what I thought
19:37 joshsamuelson joined #puppet-openstack
19:37 mgagne mfisch yes due to subsequent puppet runs
19:37 beddari_ joined #puppet-openstack
19:37 EmilienM mgagne: how many do you need?
19:37 mgagne one
19:37 mfisch mgagne: so puppet expects that token to be there every time even if it has nothing to do
19:37 mgagne catalog is templated so no exported resources
19:38 EmilienM mgagne: nice
19:38 mgagne mfisch only if you use keystone puppet ressources AFAIK
19:38 EmilienM mgagne: one puppet run that is idempotent? that's really cool
19:38 mfisch we have users and endpoints
19:38 mfisch mgagne: how do you disable the middwalre
19:38 mfisch middleware
19:38 mgagne mfisch in paste config
19:39 mgagne pipeline:public_api/pipeline, pipeline:admin_api/pipeline and pipeline:api_v3/pipeline
19:39 mgagne remove admin_token_auth
19:39 mfisch mgagne: thinking that if I do that after bootstrap it should be harmless
19:40 mfisch but it will mean that puppet cant make new users/endpoints
19:40 mgagne mfisch yea, that's why we use privileged keystone nodes which aren't public
19:40 mfisch has anyone looked into having puppet not need the admin token after bootstrap?
19:42 openstackgerrit Merged openstack/puppet-cinder: Default service_name to 'Volume Service'  https://review.openstack.org/264399
19:42 dfisher joined #puppet-openstack
19:42 kbyrne joined #puppet-openstack
19:43 mwhahaha https://github.com/openstack/fuel-library/blob/master/deployment/puppet/osnailyfacter/modular/astute/service_token_off.pp
19:43 mwhahaha we pull it after we deploy our controllers
19:44 mwhahaha but we don't add more users after the fact i don't think
19:44 mtreinish EmilienM: you pinged me earlier about something?
19:44 mfisch mwhahaha: we dont usually either
19:44 mfisch mwhahaha: why exec and not file-line?
19:44 mwhahaha we're tasked based
19:44 mwhahaha so we execute that last
19:45 mfisch how do you control that running last?
19:45 mwhahaha goes with how fuel determines tasks
19:45 mwhahaha it's complicated :D
19:45 mfisch I could boostrap with the token and then use this code to remove it
19:46 mfisch we could add anything after the fact by hand or with ansible
19:47 mwhahaha oh 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 config
19:47 EmilienM mtreinish: yeah, we have found https://bugs.launchpad.net/tempest/+bug/1531917
19:47 openstack Launchpad bug 1531917 in tempest "Strange error when using versionless identity endpoint." [Undecided,New]
19:47 EmilienM I wanted your insights
19:47 EmilienM if we should default our modules to use versionless at the risk of not able to use tempest or some openstack clients
19:48 openstackgerrit Merged openstack/puppet-ironic: Allow to configure api_workers  https://review.openstack.org/262966
19:48 openstackgerrit Merged openstack/puppet-neutron: Fixes OVS agent pkg and service name  https://review.openstack.org/262012
19:48 openstackgerrit Merged openstack/puppet-nova: Deprecate nova::utilities  https://review.openstack.org/258260
19:49 openstackgerrit Merged openstack/puppet-trove: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/263968
19:49 mtreinish EmilienM: 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/1530181
19:49 EmilienM nice
19:49 openstackgerrit Merged openstack/puppet-rally: puppet-rally: Initial commit  https://review.openstack.org/262970
19:50 EmilienM mtreinish: ok so that means keystone endpoint need to have the version (when doing openstack endpoint list)
19:50 mfisch mwhahaha: what does puppet do when the admin token is absent and the keystone resources run?
19:50 EmilienM mtreinish: isn't enough to have the version in tempest.conf?
19:50 mwhahaha well we've setup an openrc file so I'd assume it uses that
19:51 mtreinish EmilienM: it needs a version in the service catalog iirc
19:51 EmilienM mhh ok
19:51 EmilienM chem: ^
19:51 mtreinish it's mostly because there isnt a uniform way to do version discovery across projects
19:51 EmilienM mtreinish: good to know
19:52 EmilienM I guess we'll have to deal with that and set the version
19:52 chem EmilienM ack (I have tried to put a versionless in tempest.conf but to no good)
20:05 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Implement WSGI support for Ironic API  https://review.openstack.org/263450
20:06 openstackgerrit Merged openstack/puppet-neutron: Make neutron::server inherit from neutron::params  https://review.openstack.org/263875
20:08 openstackgerrit Merged openstack/puppet-trove: update Setup section  https://review.openstack.org/264805
20:10 openstackgerrit Merged openstack/puppet-ironic: [inspector] Specify SELinux type context for tftp dir  https://review.openstack.org/264180
20:12 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Use identity_uri and auth_uri by default  https://review.openstack.org/262799
20:14 openstackgerrit Merged openstack/puppet-trove: Follow-up on PyMySQL support for Red Hat platforms  https://review.openstack.org/263969
20:25 openstackgerrit joined #puppet-openstack
20:25 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/264951
20:27 openstackgerrit Merged openstack/puppet-trove: Typo in README.md  https://review.openstack.org/257515
20:32 gchamoul joined #puppet-openstack
20:32 migi joined #puppet-openstack
20:44 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/264927
20:57 EmilienM mfisch, mwhahaha: looks good to you? https://review.openstack.org/#/c/262965/
20:58 EmilienM we have the integ jobs running for puppet-ironic now
20:58 EmilienM will be very useful imho
21:01 EmilienM I can't reproduce the cinder issue in my env
21:01 EmilienM that's weird
21:06 EmilienM mwhahaha: also https://review.openstack.org/263760 we need it for mitaka
21:07 EmilienM thanks
21:07 mwhahaha k i'll look
21:07 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP Support for erasure code in swift  https://review.openstack.org/264966
21:07 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/255635
21:09 EmilienM mwhahaha: another blocker for mitaka: https://review.openstack.org/#/c/257522/ (you +1-ed)
21:09 EmilienM oh there is a comment
21:11 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP Support for erasure code in swift  https://review.openstack.org/264966
21:12 mwhahaha oh there was just a tab
21:13 EmilienM so good news
21:13 EmilienM ubuntu packaging seems ready for mitaka
21:13 mwhahaha interesting
21:13 EmilienM dmsimard: my next focus it getting rdo working too
21:13 EmilienM so we can bump puppet CI to mitaka!
21:14 EmilienM the blocker will be to have a tested rpo for rdo
21:16 vinsh Good work!
21:17 openstackgerrit Merged openstack/puppet-openstack-integration: scenario002 swift ring devices >= replica count  https://review.openstack.org/263760
21:17 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/255635
21:18 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Test Mitaka on UCA & RDO  https://review.openstack.org/255635
21:18 EmilienM this patch should tell us if mitaka works for our scenarios ^
21:35 bklei joined #puppet-openstack
21:35 openstackgerrit Merged openstack/puppet-openstacklib: Fix handling of ensure_absent_val  https://review.openstack.org/264710
21:39 openstackgerrit Merged openstack/puppet-nova: Drop use of deprecated neutron auth options  https://review.openstack.org/257522
21:40 openstackgerrit Merged openstack/puppet-designate: Configure rabbit_virtual_host  https://review.openstack.org/244469
21:43 openstackgerrit Merged openstack/puppet-ironic: Enable changing list of inspection collectors  https://review.openstack.org/263230
21:45 cwolferh joined #puppet-openstack
21:51 beddari joined #puppet-openstack
21:51 kgalanov joined #puppet-openstack
21:51 jaosorior joined #puppet-openstack
21:51 rlaurika joined #puppet-openstack
21:51 llua joined #puppet-openstack
21:54 EmilienM I'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_787
21:54 EmilienM maybe worth to investigate
21:54 EmilienM looks 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_188
22:12 bklei joined #puppet-openstack
22:13 bklei joined #puppet-openstack
22:19 mfisch EmilienM: hey
22:48 xarses joined #puppet-openstack
22:55 mdorman joined #puppet-openstack
23:11 openstackgerrit Cody Herriges proposed openstack/puppet-vswitch: Remove duplicate loading of constants  https://review.openstack.org/265005
23:14 EmilienM mfisch: yo
23:19 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-openstacklib: Utility to handle IPv6 address brackets.  https://review.openstack.org/264927
23:21 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/264951
23:43 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/264951
23:51 ducttape_ joined #puppet-openstack

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary