Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:05 baker joined #puppet-openstack
00:25 johnmilton joined #puppet-openstack
00:27 dfisher joined #puppet-openstack
00:28 dfisher _ody: ping again?
00:42 openstackgerrit Mark Kirkwood proposed openstack/puppet-swift: Allow a replication network to be specified Closes-Bug: #1491660 Change-Id: I1d1d327b7d32f37550edcc036f5ab2bbb1b37ba1  https://review.openstack.org/302483
00:42 openstack bug 1491660 in puppet-swift "Cannot specify replicaton network in swift-object-device" [Undecided,In progress] https://launchpad.net/bugs/1491660 - Assigned to Adam Vinsh (adam-vinsh)
00:47 dmsimard Came across a new issue in my Mitaka testing, filed a bug: https://bugs.launchpad.net/puppet-nova/+bug/1567157
00:47 openstack Launchpad bug 1567157 in puppet-nova "nova::network::neutron should set use_neutron=true" [Undecided,New]
01:24 dprince joined #puppet-openstack
01:28 baker joined #puppet-openstack
01:34 ducttape_ joined #puppet-openstack
01:40 vinsh_ joined #puppet-openstack
01:46 baker joined #puppet-openstack
02:11 baker joined #puppet-openstack
02:16 xingchao joined #puppet-openstack
02:20 ducttape_ joined #puppet-openstack
02:39 baker_ joined #puppet-openstack
02:51 social joined #puppet-openstack
03:02 xarses joined #puppet-openstack
03:06 michchap joined #puppet-openstack
03:18 chandankumar joined #puppet-openstack
03:19 v1k0d3n joined #puppet-openstack
03:20 ducttape_ joined #puppet-openstack
04:02 fedexo joined #puppet-openstack
04:20 chandankumar joined #puppet-openstack
05:13 jaosorior joined #puppet-openstack
05:24 michchap joined #puppet-openstack
05:42 fedexo joined #puppet-openstack
05:45 rcernin joined #puppet-openstack
05:46 xingchao joined #puppet-openstack
05:58 jmikolajczak joined #puppet-openstack
06:03 gfidente joined #puppet-openstack
06:03 gfidente joined #puppet-openstack
06:45 tesseract joined #puppet-openstack
06:46 openstackgerrit Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/293436
07:02 dgurtner joined #puppet-openstack
07:17 openstackgerrit Babu Shanmugam proposed openstack/puppet-neutron: Add support for OVN plugin  https://review.openstack.org/301613
07:41 amoralej joined #puppet-openstack
07:44 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Remove individual service certificates  https://review.openstack.org/301493
07:44 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add missing services ports to service_ports map  https://review.openstack.org/302573
07:44 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer listeners  https://review.openstack.org/302574
07:47 jaosorior joined #puppet-openstack
07:49 guessi joined #puppet-openstack
08:07 paramite joined #puppet-openstack
08:23 openstackgerrit Merged openstack/puppet-barbican: Modify ensure_package parameter in dogtag plugin  https://review.openstack.org/301869
08:28 skolekonov joined #puppet-openstack
08:30 jistr joined #puppet-openstack
08:30 openstackstatus joined #puppet-openstack
08:35 openstackstatus NOTICE: jobs depending on npm are now working again
08:43 yolanda joined #puppet-openstack
08:45 rmart04 joined #puppet-openstack
08:50 yolanda joined #puppet-openstack
08:50 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer listeners  https://review.openstack.org/302574
08:51 hjensas joined #puppet-openstack
08:51 hjensas joined #puppet-openstack
08:52 openstackgerrit Mark Kirkwood proposed openstack/puppet-swift: Allow a replication network to be specified Closes-Bug: #1491660 Change-Id: I1d1d327b7d32f37550edcc036f5ab2bbb1b37ba1  https://review.openstack.org/302483
08:52 openstack bug 1491660 in puppet-swift "Cannot specify replicaton network in swift-object-device" [Undecided,In progress] https://launchpad.net/bugs/1491660 - Assigned to Mark Kirkwood (mark-kirkwood)
08:53 mgould joined #puppet-openstack
08:58 ToMiles joined #puppet-openstack
09:00 jaosorior joined #puppet-openstack
09:01 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer listeners  https://review.openstack.org/302574
09:06 openstackgerrit Babu Shanmugam proposed openstack/puppet-neutron: Add support for OVN plugin  https://review.openstack.org/301613
09:09 derekh joined #puppet-openstack
09:22 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Pass firewall_driver to ML2 plugin conf file  https://review.openstack.org/300442
10:13 karimb joined #puppet-openstack
10:38 karimb joined #puppet-openstack
10:52 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-oslo: Add oslo::messaging::default define  https://review.openstack.org/302664
10:54 karimb joined #puppet-openstack
11:08 johnmilton joined #puppet-openstack
11:10 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-oslo: Add oslo::messaging::default define  https://review.openstack.org/302664
11:16 openstackgerrit Babu Shanmugam proposed openstack/puppet-ovn: Added .gitreview for the puppet-ovn project  https://review.openstack.org/302044
11:17 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/302390
11:17 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/302376
11:35 EmilienM good morning
11:44 dmburmistrov hi!
11:44 dmburmistrov EmilienM, can you look at https://review.openstack.org/#/c/301079/ ?
11:44 cgfbee joined #puppet-openstack
11:46 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/302390
11:46 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/302376
11:48 mgould morning EmilienM
11:51 openstackgerrit Merged openstack/puppet-neutron: Pass firewall_driver to ML2 plugin conf file  https://review.openstack.org/300442
11:52 EmilienM dmburmistrov: sure, I'll look today
11:56 xingchao joined #puppet-openstack
12:00 jhesketh joined #puppet-openstack
12:00 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
12:03 openstackgerrit Merged openstack/puppet-ironic: Revert "Allow chainloading of Inspector ramdisk over UEFI"  https://review.openstack.org/302348
12:07 jhesketh joined #puppet-openstack
12:07 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Pass firewall_driver to ML2 plugin conf file  https://review.openstack.org/302703
12:13 moravec joined #puppet-openstack
12:18 ducttape_ joined #puppet-openstack
12:19 dprince joined #puppet-openstack
12:20 dmburmistrov EmilienM, ok, thank you!
12:23 openstackgerrit Merged openstack/puppet-neutron: drop useless releasenote  https://review.openstack.org/301407
12:25 skolekonov joined #puppet-openstack
12:26 degorenko EmilienM, hey :) welcome back!
12:26 EmilienM degorenko: thanks!
12:32 morazi joined #puppet-openstack
12:44 social_ joined #puppet-openstack
12:46 Matum joined #puppet-openstack
12:47 cdelatte joined #puppet-openstack
12:49 mjblack EmilienM: I might have a solution to the cinder wsgi problem
12:52 EmilienM mjblack: which one?
12:52 mjblack the one where its missing the application entry point
12:53 EmilienM mjblack: feel free to use my patch and push over
12:53 openstackgerrit Sergey Kolekonov proposed openstack/puppet-openstack-integration: Set correct firewall_driver for Neutron ML2  https://review.openstack.org/300538
12:53 mjblack yeah I'm about to test it out and see if it will work
12:53 v1k0d3n joined #puppet-openstack
12:53 delattec joined #puppet-openstack
12:54 mjblack its a shame that the change that is causing the problem wasnt put into the mitaka release notes
12:55 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-keystone: Set oslo options in keystone module through puppet-oslo  https://review.openstack.org/302738
12:56 richm joined #puppet-openstack
13:01 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-keystone: Set oslo options in keystone module through puppet-oslo  https://review.openstack.org/302738
13:12 xingchao joined #puppet-openstack
13:15 xarses joined #puppet-openstack
13:15 xarses joined #puppet-openstack
13:16 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic: Expose an option to enable/disable cleaning  https://review.openstack.org/302747
13:18 ducttape_ joined #puppet-openstack
13:18 ducttape_ joined #puppet-openstack
13:20 moravec joined #puppet-openstack
13:34 openstackgerrit Merged openstack/puppet-magnum: Update typos  https://review.openstack.org/263181
13:39 moravec1 joined #puppet-openstack
13:39 xingchao joined #puppet-openstack
13:40 ducttape_ joined #puppet-openstack
13:41 chandankumar joined #puppet-openstack
13:41 mgould hi everyone
13:41 mgould I'm trying to run the unit tests for puppet-ironic
13:41 mgould is https://wiki.openstack.org/wiki/Puppet/Unit_testing the right instructions?
13:43 EmilienM degorenko: sounds like something is broken in Murano, when trying to run tempest: http://logs.openstack.org/36/250436/30/check/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-trusty/cbde3db/console.html#_2016-04-05_14_32_42_799
13:43 EmilienM mgould: yes, you're on the right page
13:43 mgould OK, thanks
13:43 mjblack EmilienM: you got a mitaka on centos 7 handy?
13:44 mgould I get the error "rake is not part of the bundle. Add it to Gemfile."
13:44 EmilienM mjblack: sudo yum -y install ruby-devel rubygems
13:45 mjblack EmilienM: I think I found the wsgi script, I just wanted to verify on centos its in the correct spot
13:46 openstackgerrit Matthew J Black proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/287964
13:46 degorenko EmilienM, nothing changed from last week, i already mentioned, that murano has missed package. It was fixed in: https://review.openstack.org/#/c/297647/1
13:46 EmilienM degorenko: cool!
13:46 degorenko just waiting for new package
13:47 EmilienM degorenko: indeed
13:52 openstackgerrit Sergey Kolekonov proposed openstack/puppet-nova: Do not use network_api_class option  https://review.openstack.org/293010
13:55 mgould OK, tests running: I had to delete Gemfile.lock and run `bundle install` again
13:58 mwhahaha I see the polar bears were unsuccessful in consuming EmilienM
13:59 dmsimard lol, polar bears are much, much more up north :p
14:00 mwhahaha :D
14:00 EmilienM north coast is so beautiful
14:00 EmilienM I wanted to stay there
14:01 EmilienM but there is no internet :-P
14:01 mgould EmilienM, where were you?
14:03 EmilienM mgould: somewhere around https://goo.gl/maps/U1nspNdM43q
14:04 mgould awesome!
14:05 mgould looks beautiful
14:05 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/302390
14:05 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/302376
14:05 EmilienM mgould: that is the place, if you want to observe whales
14:06 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Implement a script to generate Puppetfile  https://review.openstack.org/300696
14:08 EmilienM omg _ody got puppet4 jobs working
14:08 openstackgerrit Merged openstack/puppet-cinder: Fix typo in README  https://review.openstack.org/302277
14:08 openstackgerrit Merged openstack/puppet-glance: Release notes for glance_image updates  https://review.openstack.org/302391
14:08 openstackgerrit Merged openstack/puppet-glance: Add ability to set properties with glance_image  https://review.openstack.org/301706
14:08 degorenko mwhahaha, can you +A this one? https://review.openstack.org/302197
14:10 xingchao_ joined #puppet-openstack
14:10 mwhahaha yup
14:11 EmilienM now, we have Puppet OpenStack modules working on Puppet4, I'm wondering if we should switch puppet-openstack-integration to gate on puppet4
14:11 EmilienM another solution is to keep puppet3 by default, and have periodic jobs for puppet4
14:11 openstackgerrit Merged openstack/puppet-glance: Switch glance to os_service_default facts  https://review.openstack.org/299822
14:11 EmilienM both together might be too much for openstack infra
14:11 openstackgerrit Merged openstack/puppet-glance: Implement ability to pass CA bundle certificate for vCenter server  https://review.openstack.org/302461
14:11 EmilienM pabelanger: what do you think?^
14:12 openstackgerrit Merged openstack/puppet-gnocchi: Switch gnocchi to rspec-puppet-facts  https://review.openstack.org/298528
14:12 degorenko EmilienM, as i know we have a few modules, that we have problem with Puppet4, don't we?
14:12 mwhahaha thought swift was not puppet4 compatible
14:12 EmilienM degorenko: it seems like puppet4 jobs are green
14:12 degorenko yeah, at least swift
14:13 degorenko EmilienM, we can check this only running test jobs for all modules
14:13 EmilienM mwhahaha, degorenko: see https://review.openstack.org/#/c/296557/
14:13 pabelanger EmilienM: should be fine. Just means longer wait time for your patches to land
14:13 EmilienM pabelanger, mwhahaha: all went fine on the JJB migration?
14:13 pabelanger it went
14:13 EmilienM pabelanger: I was afk 2 days
14:13 EmilienM excellent, thanks
14:13 mwhahaha fine being relative ;)
14:13 pabelanger we lost access to nodepool.o.o yesterday, which blocked puppet runs for about 3.5 hours
14:14 pabelanger once puppet started running again, puppet-jobs eventually started working again
14:14 pabelanger 2 issues found
14:15 EmilienM pabelanger: do we have more bits to migrate?
14:15 pabelanger python-requests is installed by default now (from pip) in centos-7, so we had to remove that. And bindep needed to be run first before calling puppet-prepare-node, because you remove RDO repos from the slave
14:15 pabelanger and bindep depends on RDO for centos-7
14:16 pabelanger EmilienM: no, puppet jobs are on ubuntu-trusty and centos-7 dibs now
14:16 twm2016 joined #puppet-openstack
14:16 pabelanger good until we turn up ubuntu-xenial imagtes
14:16 pabelanger images*
14:16 pabelanger well, they do work, but we're asking projects to only use them in experimental for now
14:18 EmilienM pabelanger, mwhahaha: we might want to start testing our modules on next LTS to be ready
14:18 EmilienM we can work on it during newton
14:32 baker joined #puppet-openstack
14:36 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend.  https://review.openstack.org/296370
14:40 baker joined #puppet-openstack
14:43 mdorman joined #puppet-openstack
14:45 openstackgerrit Merged openstack/puppet-keystone: Make the SSL apache module happens at correct time  https://review.openstack.org/301924
14:46 openstackgerrit Merged openstack/puppet-swift: Keystone endpoints should go before the service  https://review.openstack.org/301971
14:51 dhollinger joined #puppet-openstack
14:53 mgould degorenko, mwhahaha: do I need the unless clauses in https://review.openstack.org/#/c/302376/ or not?
14:53 mwhahaha you do
14:53 degorenko you not
14:53 mgould hahaha
14:53 degorenko :D
14:53 * mgould doesn't understand the idempotence argument
14:53 mwhahaha without it the exec runs every time the manifest is processed
14:53 mgould "run this command, unless you run this command successfully"
14:53 mwhahaha which makes it non-idempotent
14:54 mgould yeah, but to test if you need to run it you run it anyway...
14:54 mwhahaha so by adding the unless, it skips the check if the file exists
14:54 mwhahaha we only want those execs as fail tests
14:54 mwhahaha other than that we don't want them to run
14:54 mwhahaha techincally you could just switch command to exit 1 and just test with the unless
14:55 mwhahaha because the condition will trigger then
14:55 degorenko oh, yes, right, i forgot about idempotency
14:55 * mwhahaha slaps degorenko
14:55 degorenko mgould, yes, you need :)
14:55 mwhahaha :D
14:55 degorenko D:
14:55 twm2016 @mwhahaha: I almost suggested that
14:55 mgould OK, sounds like we should make it "fail unless"
14:55 mgould which I think makes the intention clearer
14:56 mwhahaha yea we're using an exec to throw a fatal only if a file doesn't exist
14:56 degorenko we should run exec unless :)
14:56 twm2016 yeah that makes it clear
14:56 mgould what happens if we don't have the unless?
14:56 mwhahaha it'll run all every run
14:56 mgould first run: test -f $file
14:56 degorenko it will not pass idempotency test
14:56 mwhahaha you're getting into the weirdness of puppet
14:57 mgould OK, please help me understand
14:57 mgould with the unless clause, on the second run
14:57 mwhahaha exec { 'file missing': path=>"file", command => "exit 1", unless => "test -f file" }
14:58 mwhahaha basically the unless prevents the thing from running
14:58 degorenko exec will not be run when file exists at all
14:58 mwhahaha so puppet just skips it
14:58 mgould OK
14:58 mgould but if the body of the exec is "test -f" and there's no unless clause, it runs test -f every time, and fails if the file isn't present
14:58 mwhahaha if you just have exec { 'file test': path => "file", command => "test -f file" }, it'll run every time the manifest is processed
14:58 mwhahaha which in puppet terms is non-idempotent
14:59 mwhahaha you should only want to have something run one time
14:59 mgould nb the path argument is actually the executable PATH
14:59 mgould aaaaah
14:59 mwhahaha yea my brain isn't working enough yet, but i'm just throwing out some basic examples
14:59 mgould so Puppet doesn't care that the exec's body is idempotent, it only cares that you ran it twice?
15:00 mwhahaha yea
15:00 mgould gotcha
15:00 mwhahaha since the exec gets run twice it considers it as non-idempotent and shows up as a change every time
15:00 mgould cool
15:00 mwhahaha even tho it's a noop
15:00 mgould I suppose checking the body of the exec for idempotence would be Hard
15:00 mgould uncomputable? probably
15:00 mwhahaha yea
15:00 mwhahaha thats where the unless comes in
15:01 mwhahaha it's you telling puppet when the change has been done so don't do the exec again
15:01 mgould right, OK
15:01 mgould I'll do fail { unless }, I think, for clarity
15:04 dgurtner joined #puppet-openstack
15:05 openstackgerrit Merged openstack/puppet-tempest: Update spec tests for Tempest  https://review.openstack.org/302197
15:06 7JTAALB9C joined #puppet-openstack
15:09 guimaluf joined #puppet-openstack
15:09 mgould nope, fail is a function
15:10 mwhahaha yea you can't use fail
15:10 mwhahaha you still have to use exec
15:10 mwhahaha :D
15:10 baker joined #puppet-openstack
15:10 mwhahaha notice is the only one that you could do that with
15:10 mwhahaha but no one reads those
15:11 * mgould nods
15:11 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/302390
15:11 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/302376
15:11 mgould yes, gerrit, I do want to upload multiple commits
15:11 yottatsa joined #puppet-openstack
15:11 mgould Git is really really good at sequences of commits, stop trying to discourage me from using one of its most powerful features
15:12 mwhahaha it's because they are dependencies so it has to update the 2nd one since you're working on the first one
15:12 moravec joined #puppet-openstack
15:13 EmilienM dmburmistrov, degorenko: hey about https://review.openstack.org/#/c/301079/4/manifests/policy.pp
15:13 EmilienM I don't have strong opinion
15:13 EmilienM it just adds conditionnals in the code
15:15 dmburmistrov EmilienM, <mwhahaha> suggested to be permissive to not brake other modules (and other existing code)
15:15 EmilienM ok
15:15 EmilienM approved
15:16 mwhahaha we generally have been allowing the string values so adding the array restrictions into oslo may break other things
15:16 mwhahaha if we want to switch to an array, i'd recommend a deprecation-like messaging in the modules that switch to oslo
15:16 mwhahaha then we can switch to arrays in a later release
15:17 dmburmistrov sounds reasonable - which  way should I rework my patches today?
15:17 mwhahaha any2array in oslo
15:17 dmburmistrov good, ok
15:17 mwhahaha pull the logic out of the modules that switch to oslo and just pass the values
15:18 mwhahaha it'll make life easier all around :)
15:18 EmilienM dmburmistrov: should I drop +A ?
15:18 mwhahaha once they all get switched we could just add the warning logic into oslo if we want to add those restrictions (but i doubt it)
15:19 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-nova: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/292309
15:19 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-nova: Configure oslo related sections using puppet-oslo module  https://review.openstack.org/302839
15:20 xingchao joined #puppet-openstack
15:20 EmilienM dmburmistrov: I -1 it to avoid merge
15:20 dmburmistrov EmilienM, yeah, thanks!
15:23 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-nova: Configure oslo related sections using puppet-oslo module  https://review.openstack.org/302839
15:23 mgould mwhahaha, yeah, I know why it's doing it
15:23 mgould I just think it's stupid :-)
15:24 mwhahaha i find it more annoying when it's a chain of like 6 of them :D
15:25 mgould mwhahaha, degorenko: remind me, what was wrong with ensure { file, noop }?
15:25 dtantsur a couple of times it actually saved me from submitting a couple of wrong commits
15:26 dtantsur so it's not so stupid (or maybe I am stupid? :D)
15:26 mwhahaha it won't fail
15:26 degorenko ^ yes
15:26 mwhahaha file would try and create it
15:26 mwhahaha our goal is for failure :D
15:26 mgould OK, cool
15:26 EmilienM crinkle: have you seen https://bugs.launchpad.net/puppet-keystone/+bug/1566190 ?
15:26 openstack Launchpad bug 1566190 in puppet-keystone "'disable_admin_token_auth' should also handle admin_token in the config" [Undecided,New]
15:27 mgould OK, I think I've addressed all your comments; I'm gonna disappear for a bit to do some DHCP testing
15:28 mwhahaha ninja vanish
15:32 EmilienM skolekonov, mjblack: for https://review.openstack.org/#/c/301797/ - we need to engage efforts on puppet-octavia. I initiated the module, we have a basic structure ready. Please let me know if you're interested to continue the work a bit
15:34 mjblack EmilienM: that is the plugin for octavia
15:34 mjblack octavia is its own self contained service
15:34 EmilienM mjblack: ok so we need to configure plugin in puppet-neutron and service in puppet-octavia?
15:35 mjblack yup
15:35 EmilienM kk
15:35 EmilienM mjblack: still add tests :P
15:35 mjblack I'm working on it :D
15:35 mjblack I just wanted to make sure the layout was ok
15:35 EmilienM it looks good.
15:35 mjblack EmilienM: one thing to keep in mind with lbaas, the neutron::agents::lbaas is *only* for haproxy
15:36 openstackgerrit Pradeep Kilambi proposed openstack/puppet-gnocchi: Add sync_db param to gnocchi api class  https://review.openstack.org/300604
15:36 mjblack or rather
15:37 mjblack the only plugin that work with the lbaas agent that I know of is haproxy
15:37 skolekonov mjblack, is it possible to fully configure lbaasv2? AFAIK it requires a specific image to work
15:38 ntaylor1781 joined #puppet-openstack
15:38 mjblack skolekonov: right now, the changes for lbaasv2 service in neutron is still in progress
15:39 mjblack the lbaas agent that puppet-neutron handles is only v1 but it can be updated to do v2 as well
15:40 crinkle EmilienM: i did but forgot to respond, i don't believe it's a bug
15:40 EmilienM crinkle: yeah, I commented
15:41 EmilienM does it make sense?
15:41 crinkle EmilienM: my understanding of auth_token is disabling it in the pipeline will render having it set in keystone.conf useless so it doesn't need to be removed
15:42 EmilienM crinkle: exactly
15:42 crinkle EmilienM: and puppet can't really handle setting it, using it, then unsetting it in one puppet run, so doing it like this is easiest
15:43 EmilienM exactly
15:43 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-oslo: Update oslo::policy define  https://review.openstack.org/301079
15:43 EmilienM crinkle: invalid bug.
15:43 aimon joined #puppet-openstack
15:44 dfisher joined #puppet-openstack
15:44 afranc joined #puppet-openstack
15:45 mjblack EmilienM: looks like scenarios are failing because of keystone
15:45 EmilienM degorenko: do you have news on murano packaging? all deps are solved?
15:46 mjblack seeing "Error: Could not prefetch cinder_type provider 'openstack': Could not authenticate"
15:46 degorenko EmilienM, no, i don't. For tempest we just waiting for rc3
15:46 EmilienM degorenko: for RDO I mean
15:46 openstackgerrit Merged openstack/puppet-neutron: Set ini/openstack libs to pass file_path properly  https://review.openstack.org/302408
15:46 degorenko EmilienM, ah, also don't know. Going to ask
15:47 openstackgerrit Matthew J Black proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/287964
15:47 jaosorior joined #puppet-openstack
15:52 xingchao joined #puppet-openstack
15:56 ducttape_ joined #puppet-openstack
16:00 moravec1 joined #puppet-openstack
16:02 xarses joined #puppet-openstack
16:03 openstackgerrit Merged openstack/puppet-oslo: Add oslo::messaging::default define  https://review.openstack.org/302664
16:03 openstackgerrit Merged openstack/puppet-nova: Autorequire nova security group  https://review.openstack.org/301534
16:03 openstackgerrit Merged openstack/puppet-vswitch: add missing dev keyword to ip link command  https://review.openstack.org/301176
16:04 angdraug joined #puppet-openstack
16:06 openstackgerrit Pradeep Kilambi proposed openstack/puppet-aodh: Add sync_db param to aodh api class  https://review.openstack.org/302921
16:10 openstackgerrit Denis Egorenko proposed openstack/puppet-tempest: Correct comments for config class  https://review.openstack.org/301826
16:17 baker joined #puppet-openstack
16:17 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-oslo: Process list values inside oslo module  https://review.openstack.org/302271
16:23 twm2016 I know this review has gotten a lot of attention lately https://review.openstack.org/#/c/302376/ but I think instead of unless, we should use creates.
16:23 twm2016 Because of this page i found https://review.openstack.org/#/c/302376/
16:23 twm2016 sorry this one: http://www.puppetcookbook.com/posts/run-exec-if-file-absent.html
16:23 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-oslo: Process list values inside oslo module  https://review.openstack.org/302271
16:23 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-oslo: Update oslo::policy define  https://review.openstack.org/301079
16:30 dmburmistrov EmilienM, mwhahaha - jfyi: I reworked my patches (local test succeeded), waiting for zuul; urls: https://review.openstack.org/#/c/301079/ & https://review.openstack.org/#/c/302271/
16:30 EmilienM dmburmistrov: excellent
16:34 moravec joined #puppet-openstack
16:36 moravec2 joined #puppet-openstack
16:37 haplo37 joined #puppet-openstack
16:43 mjblack EmilienM,mfisch,degorenko: I'm seeing the integration tests failing and it looks like keystone
16:43 EmilienM mjblack: no it's httpd failing to start
16:43 mjblack well close enough :D
16:43 EmilienM I haven't read logs but I'm sure that's it :)
16:43 mjblack that might be the case though
16:44 mjblack where can I get the logs?
16:44 EmilienM mjblack: look httpd logs when you have it, I'm sure that's it
16:44 EmilienM well, zuul has not finished to execute all jobs on https://review.openstack.org/#/c/287964/
16:44 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-openstacklib: Add the possibility to execute without retry.  https://review.openstack.org/302442
16:44 EmilienM mjblack: but otherwise they'll be on http://logs.openstack.org/64/287964/11/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/46fb35d/logs/apache/
16:47 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: WIP: Remove user/role prefetch to support multi-domain.  https://review.openstack.org/299301
16:48 openstackgerrit Merged openstack/puppet-ironic: Expose an option to enable/disable cleaning  https://review.openstack.org/302747
16:52 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend.  https://review.openstack.org/296370
16:53 moravec joined #puppet-openstack
16:56 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/293436
16:59 dprince joined #puppet-openstack
17:09 mwhahaha i think the centos 7 beaker test is broken for barbican
17:11 EmilienM mwhahaha: how that?
17:11 mwhahaha cause it keeps failing
17:11 mwhahaha i need to look into it further but i don't think i've seen it pass in the last 2 days
17:12 dfisher has anybody seen _ody?  I've been unable to maintain a constant IRC connection due to being at a conference
17:18 _ody o.O
17:19 mgould joined #puppet-openstack
17:20 dfisher why hello there :)
17:20 dfisher _ody: are there plans to cut the 8.0.0 tarballs on the forge?
17:21 _ody dfisher: Yes.  I am holding off for packages to be released.
17:21 dfisher weren't they released just today?
17:22 _ody I hadn't heard but if yes then yes I'll start cutting releases.
17:22 _ody Well looks like it has according to openstack.org.
17:22 dfisher dhellman sent out the email officially announcing Mitaka
17:22 jmikolajczak joined #puppet-openstack
17:23 dfisher many thanks for cutting the 8.0.0 releases (preemptive but I'll probably vanish from IRC when this conference session ends)
17:24 _ody ok.  Yeah.  I'll get it down by EOW.
17:24 dfisher \o/
17:25 moravec joined #puppet-openstack
17:33 rcernin joined #puppet-openstack
17:52 rmart04 joined #puppet-openstack
17:53 dfisher joined #puppet-openstack
17:53 ducttape_ joined #puppet-openstack
17:56 xarses joined #puppet-openstack
17:59 gchamoul joined #puppet-openstack
18:00 mgould joined #puppet-openstack
18:00 dprince joined #puppet-openstack
18:05 richm joined #puppet-openstack
18:06 twm2016_ joined #puppet-openstack
18:08 mjblack EmilienM,mfisch: whats your opinion of having lbaas v1 and v2 agents installed on the same host? This is really only affects haproxy, no other load balancer that I can see
18:08 moravec joined #puppet-openstack
18:09 mjblack I want to say that it should be one or the other
18:15 dprince joined #puppet-openstack
18:24 richm joined #puppet-openstack
18:43 ducttape_ joined #puppet-openstack
18:43 dprince joined #puppet-openstack
18:44 angdraug joined #puppet-openstack
18:53 rstarmer joined #puppet-openstack
18:57 jmikolajczak joined #puppet-openstack
19:03 cdelatte joined #puppet-openstack
19:09 mwhahaha mjblack: i thought there was a mailing list post about that you can't have both
19:09 mwhahaha or was that for the api and not the agents
19:12 mjblack mwhahaha: sure but that doesnt stop someone from installing both ;)
19:12 mwhahaha yup
19:13 mjblack just wondering if I need to add in something to block it...
19:13 mwhahaha you could probably do a defined check to throw a warning, but we shouldn't block it
19:14 EmilienM it sounds complicated
19:15 mjblack mwhahaha: I was going to reuse the lbaas agent and just have a flag for v2
19:15 mjblack since they both use the same config file
19:16 mwhahaha hmm then maybe we should throw something to preven it
19:16 mwhahaha prevent rather
19:16 mjblack I was going to have a $enable_lbaasv2
19:16 mjblack which would turn off v1 service and turn on v2 service
19:16 mjblack redhat the v1 and v2 agent is bundled together but on ubuntu it is two packages
19:17 openstackgerrit Merged openstack/puppet-tempest: Correct comments for config class  https://review.openstack.org/301826
19:17 mwhahaha so just reusing the lbaas class and adding a v2 flag?
19:18 mjblack yeah
19:18 mwhahaha perhaps add both
19:18 mjblack it would but only one service would be enabled
19:18 mwhahaha i'd add an enable_v1 and enable_v2 flag and perhaps throw a warning if both are enabled
19:18 mwhahaha with v1 currently enabled by default
19:19 mjblack that might be better but I think instead of a warning a fail would be better
19:20 mjblack once the agent stands up, it'll register with neutron so if they dont want v1, for example, and later turn it off, it'll show up as a dead agent
19:20 mwhahaha sure fail would be ok as well
19:20 mwhahaha the point being to add it with those flags and add some kind of logic around it
19:20 EmilienM I'm working on puppet-octavia right no
19:20 EmilienM now*
19:21 mjblack EmilienM: does it create the init/upstart/systemd scripts? :D
19:21 EmilienM mjblack: no but packaging should
19:21 EmilienM mjblack: why would it be doing that?
19:22 mjblack cause octavia so far has no packages
19:22 mjblack well...no packages for ubuntu, the devs say to install it with pip
19:22 EmilienM are you sure?
19:22 mjblack yeah
19:22 mjblack I saw rpm packages for it though
19:22 dgautam joined #puppet-openstack
19:22 EmilienM ah :)
19:22 EmilienM RDO provides packaging
19:22 mjblack https://github.com/openstack-packages/octavia
19:23 EmilienM yeah, I confirm we have packaging already
19:23 EmilienM that's why we can start deploying it
19:23 dgautam I am facing puppet/type/keystone_tenant.rb error. Any poiters to resolve this ?
19:23 dgautam Apr  7 12:21:48 cmbu-cl62 puppet-agent[4567]: Could not autoload puppet/type/keystone_tenant: interning empty string Apr  7 12:21:48 cmbu-cl62 puppet-agent[4567]: Could not retrieve catalog from remote server: Could not intern from text/pson: Could not autoload puppet/type/keystone_tenant: interning empty string
19:23 dgurtner joined #puppet-openstack
19:23 dgurtner joined #puppet-openstack
19:24 mjblack dgautam: what version of puppet? and what release of puppet-keystone?
19:24 dgautam puppet-keystone is 6.1.0 (kilo)
19:25 dgautam master is 3.7.3
19:26 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add generic manifest for loadbalancer listeners  https://review.openstack.org/302574
19:29 jmikolajczak joined #puppet-openstack
19:30 dgautam #puppet told me, there was some known bug in puppet/type/keystone_tenant.rb.
19:31 mjblack are you able to create a paste with your manifest?
19:31 EmilienM mjblack: did you figure why https://review.openstack.org/#/c/287964/ fails?
19:32 mjblack EmilienM: nothing that I could figure out but it is with keystone
19:32 mjblack it's before the cinder wsgi gets installed
19:33 mjblack only thing I see that probably is the culprit for it failing is this...
19:33 mjblack 2016-04-07 14:09:05.644 11656 WARNING keystone.common.wsgi [req-b30dec48-081d-4bf2-ad8c-f6947afb2d9a - - - - -] Authorization failed. The request you have made requires authentication. from ::1
19:33 EmilienM http://logs.openstack.org/64/287964/12/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/af61737/console.html#_2016-04-07_17_23_25_394
19:33 openstackgerrit Sergey Kolekonov proposed openstack/puppet-nova: Do not use network_api_class option  https://review.openstack.org/293010
19:33 EmilienM mjblack: https://review.openstack.org/#/c/287964/12/manifests/keystone/auth.pp
19:34 EmilienM that's because of that
19:34 mjblack I stand corrected...I'm seeing this
19:34 EmilienM I'm working on it.
19:34 mjblack keystone.token.controllers [req-b30dec48-081d-4bf2-ad8c-f6947afb2d9a - - - - -] User d65e96f34f1b4b90a99ffd614b86abe9 is unauthorized for tenant 7a782b913a644e8594ea76e9da8be79e
19:34 mjblack you want me to just quickly update it?
19:34 EmilienM I'm on it
19:34 mjblack ok
19:36 dgautam should I paste full manifest file ?
19:37 mjblack dgautam: sure
19:38 dgautam http://paste.openstack.org/show/sYzy0PnraASjKuGQdcSG/
19:40 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/287964
19:40 EmilienM mjblack: https://review.openstack.org/#/c/287964/12..13/manifests/keystone/auth.pp
19:41 EmilienM mjblack: and https://review.openstack.org/#/c/287964/12..13/manifests/api.pp
19:41 EmilienM mwhahaha: you ok with https://review.openstack.org/#/c/300696/ now?
19:41 EmilienM I addressed your review
19:42 mwhahaha yea i think so i'll take a look in a bit
19:42 karimb joined #puppet-openstack
19:43 mjblack EmilienM: looks good to me, lets see if it passes the tempest scenario tests now
19:44 mjblack dgautam: have you verified the tenants hiera data?
19:44 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: deploy cinder api in WSGI with Apache  https://review.openstack.org/288091
19:44 EmilienM mjblack: let's see if when enabled, it pass CI ^
19:45 EmilienM oops I failed in rebase
19:46 mjblack dgautam: also, I dont know about 3.7 puppet but 3.8 puppet uses hiera 1.3
19:46 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: deploy cinder api in WSGI with Apache  https://review.openstack.org/288091
19:46 mjblack you can reduce a lot of your manifests with hiera 1.3
19:47 dgautam mkblack: yes, i checked hieradata. http://paste.openstack.org/show/493410/
19:52 dgautam mjblack: surely I'll look for hiera 1.3 . do you have any reference/example about reducing manifest with hiera 1.3 ?
19:53 mjblack yeah, you can do hiera data binding
19:53 mjblack in the hiera you do this
19:53 mjblack cinder::keystone::auth::password: "mypassword"
19:54 mjblack and then you dont need to call the class with that
19:54 mjblack so like in the first paste, lines 32-70, you probably can remove completely
19:55 mjblack do hiera data bindings and then do hiera_include on an array that contains the classes
19:56 EmilienM _ody: weird, it's failing now on https://review.openstack.org/#/c/296557/ - it was working before
19:56 mjblack dgautam: and then you can do a few lookups to create a string
19:58 dfisher joined #puppet-openstack
19:58 dgautam got it. surely work on that. I guess https://docs.puppet.com/hiera/3.1/complete_example.html#writing-the-data-sources should be good example.
19:59 EmilienM _ody: sounds like a packaging issue: http://logs.openstack.org/57/296557/32/experimental/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/672e134/console.html#_2016-04-07_17_13_34_911
19:59 EmilienM with new centos7 images I guess
19:59 EmilienM nodepool images for centos7 have been updated yesterday or so, I presume we need to add a package
20:00 EmilienM pabelanger: ^
20:01 dgautam mjblack: any pointer for keystone_tenant.rb ?
20:02 pabelanger EmilienM: have a few ways for handing it, either we update bindep-fallback.txt or start adding other-requirements.txt to projects
20:02 dgautam what way i should debug it ?
20:02 pabelanger EmilienM: but let me check why it is missing
20:02 mjblack dgautam: still looking into it
20:03 dgautam Thanks
20:03 pabelanger EmilienM: odd, we are not running bindep for that job
20:03 pabelanger EmilienM: let me check JJB
20:04 EmilienM pabelanger: it's an experimental job
20:05 EmilienM pabelanger: gate-puppet-openstack-integration-4-scenario00(1,2,3)-tempest-centos-7
20:05 jtomasek joined #puppet-openstack
20:05 mjblack dgautam: one thing that might be an issue
20:05 mjblack scope
20:06 mjblack what you can try to do is
20:06 mjblack in the class contrail::profile::openstack::provision, have it accept two parameters
20:06 mjblack $users = {} and $tenants = {}
20:07 mjblack and then what you would do next is change the hiera key from openstack::keystone::tenants to contrail::profile::openstack::provision::tenants
20:07 mjblack the same would need to be done for users
20:09 dgautam let me try
20:12 dgautam mjblack: nope, same error.
20:12 dgautam 40     class { '::keystone::roles::admin':
20:12 dgautam internally calls keystone_tenant ()
20:12 dgautam even if I remove create_resouces(), same error comes.
20:14 EmilienM rdo trunk is down now
20:14 Topic for #puppet-openstack is now CI status: orange (RDO trunk repos are down) - Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
20:15 bkero :(
20:15 bkero I always wonder why it goes down and who to talk to about it.
20:15 _ody EmilienM: I figured such a think might happen with all the churn on images lately and the other nodepool issues about.  Why I ran the job one more time before it got merged.
20:16 EmilienM _ody: yeah, pabelanger is on it
20:16 EmilienM bkero: #rdo
20:17 bkero ah
20:26 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Add Glance profiles  https://review.openstack.org/296076
20:27 ntaylor1781 So I was hoping for some insight. I have been looking at adding a purge_config option for the various modules. Glance had it in the liberty release, but it is no longer fucntional in master. It looks like this is the commit that removed it: https://review.openstack.org/#/c/211593/, specifically patch_set 9. It removed the resources decleration that made the purge_config work. That being said the option is still in the a
20:27 ntaylor1781 I'm curious if this was on purpose, or if it got missed.
20:29 mwhahaha probably accidental
20:30 ntaylor1781 I can get in a review to add it again, or at least get it looked at. Looking at the bug report and the commit, it doesn't look like that is the intention.
20:30 EmilienM what is the use case of purging config?
20:30 mwhahaha not having extra stuff
20:30 EmilienM oki
20:31 EmilienM works for me
20:31 mwhahaha so only have what you declare
20:32 ntaylor1781 Honestly that is a big one for me, with the system I manage.
20:33 openstackgerrit Jerzy Mikolajczak proposed openstack/puppet-neutron: Remove deprecated options from agents  https://review.openstack.org/301106
20:33 ntaylor1781 So should I make a bug report for this one, or should I just do the review.
20:33 EmilienM ntaylor1781: send the code
20:40 ericksonsantos joined #puppet-openstack
20:41 cdelatte joined #puppet-openstack
20:42 ducttape_ joined #puppet-openstack
20:43 openstackgerrit Nathan Taylor proposed openstack/puppet-glance: Fix the purge_config option for api and registry  https://review.openstack.org/303064
20:48 dgurtner joined #puppet-openstack
20:48 dgurtner joined #puppet-openstack
20:51 ntaylor1781 EmilienM here is the review https://review.openstack.org/#/c/303064/, The options were still in there and the specs still had settings for the options, so the only thing I needed to add was the resources decleration back.
20:51 EmilienM ntaylor1781: excellent. Will review it once CI is green
20:52 ntaylor1781 Thanks!
20:53 EmilienM mwhahaha: I found something very weird
20:53 EmilienM http://logs.openstack.org/26/301826/2/gate/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/3b88dcc/console.html#_2016-04-07_18_48_46_643
20:53 EmilienM it is non-fatal, job is passing
20:54 * mwhahaha patiently waits for chrome to get on with it
20:54 EmilienM actually, image create works fine
20:54 EmilienM only list fails
20:55 mwhahaha so i think i noticed something similar when i was reworking the provider
20:55 mwhahaha (still waiting for the page)
20:55 mwhahaha 2016-04-07 18:48:46.644 | openstack image list: error: unrecognized arguments:'. Retrying for 58 more seconds
20:55 EmilienM yes
20:55 mwhahaha so something is bad in the cmd line
20:56 mwhahaha it's that trailing space
20:56 mwhahaha 2016-04-07 18:49:13.251 | Debug: Executing '/usr/bin/openstack image list --quiet --format csv '
20:56 mwhahaha openstack client hates it
20:56 mwhahaha don't ask me why
20:57 mwhahaha or it's a null or something
20:58 mwhahaha but i saw the same thing when i was messing with
20:58 mwhahaha with it
20:59 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: deploy & test neutron lbaasv1 service  https://review.openstack.org/272297
20:59 EmilienM mjblack: ^
20:59 EmilienM mjblack: just to test what is working now
21:01 mjblack EmilienM: yeah I'll need to update that too later to add in the v2 tests
21:02 EmilienM mjblack: if you got v2 working, you can take over my patch
21:02 mjblack EmilienM: I think your v1 should be its own patch since we should be testing v1 to begin with
21:02 EmilienM cool²
21:03 openstackgerrit Merged openstack/puppet-gnocchi: Add sync_db param to gnocchi api class  https://review.openstack.org/300604
21:04 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Implement Cinder API WSGI support  https://review.openstack.org/287964
21:11 dfisher joined #puppet-openstack
21:14 dhollinger joined #puppet-openstack
21:15 openstackgerrit Merged openstack/puppet-neutron: Require the neutron package be installed first  https://review.openstack.org/301845
21:22 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: deploy cinder api in WSGI with Apache  https://review.openstack.org/288091
21:28 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001/centos7: deploy Ceph on IPv6  https://review.openstack.org/303086
21:31 dgurtner joined #puppet-openstack
21:31 dgurtner joined #puppet-openstack
21:32 ayoung joined #puppet-openstack
21:32 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001/centos7: deploy Ceph on IPv6  https://review.openstack.org/303086
21:38 rstarmer joined #puppet-openstack
22:00 ducttape_ joined #puppet-openstack
22:06 baker joined #puppet-openstack
22:14 ntpttr_ joined #puppet-openstack
22:16 ducttape_ joined #puppet-openstack
22:23 openstackgerrit Mark Kirkwood proposed openstack/puppet-swift: Allow a replication network to be specified  https://review.openstack.org/302483
22:26 ayoung joined #puppet-openstack
22:40 ntpttr_ joined #puppet-openstack
23:00 moravec joined #puppet-openstack
23:09 openstackgerrit Merged openstack/puppet-oslo: Update oslo::policy define  https://review.openstack.org/301079
23:14 openstackgerrit Merged openstack/puppet-aodh: Add sync_db param to aodh api class  https://review.openstack.org/302921
23:32 ducttape_ joined #puppet-openstack
23:51 moravec joined #puppet-openstack
23:59 markvoelker joined #puppet-openstack

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