Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:06 EmilienM dmsimard: ok
00:06 EmilienM dmsimard: please just update swift
00:06 EmilienM and +2
00:06 dmsimard EmilienM: will do
00:08 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: PoC with openvswitch - Composable service validation within profiles  https://review.openstack.org/342202
00:14 ducttape_ joined #puppet-openstack
00:17 EmilienM alee:  https://[::1]:5000/auth/tokens
00:17 EmilienM still failing
00:17 EmilienM http://logs.openstack.org/06/339806/10/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/2b4228c/logs/cinder/cinder-api.txt.gz#_2016-07-14_22_55_56_192
00:17 EmilienM barbican is hardcoding the url or?
00:20 EmilienM anyway, will continue tomorrow. Peace
00:20 xiaohua joined #puppet-openstack
00:47 ducttape_ joined #puppet-openstack
00:56 iurygregory_ joined #puppet-openstack
01:46 ducttape_ joined #puppet-openstack
01:47 ilbot3 joined #puppet-openstack
01:47 Topic for #puppet-openstack is now CI status: green - Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/
02:03 xiaohua joined #puppet-openstack
02:07 richm joined #puppet-openstack
02:28 ducttape_ joined #puppet-openstack
02:48 markvoelker joined #puppet-openstack
03:13 xarses joined #puppet-openstack
03:16 xarses joined #puppet-openstack
03:21 xarses joined #puppet-openstack
03:25 xarses joined #puppet-openstack
03:26 xarses joined #puppet-openstack
03:34 xarses joined #puppet-openstack
03:40 ducttape_ joined #puppet-openstack
04:40 ducttape_ joined #puppet-openstack
05:26 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: Make ::tripleo::profile::base classes work with multiple nodes  https://review.openstack.org/338387
05:42 ducttape_ joined #puppet-openstack
05:48 skramaja joined #puppet-openstack
06:10 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/342625
06:14 imcsk8 joined #puppet-openstack
06:18 rcernin joined #puppet-openstack
06:30 markvoelker joined #puppet-openstack
06:33 openstackgerrit joined #puppet-openstack
06:41 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: Make ::tripleo::profile::base classes work with multiple nodes  https://review.openstack.org/338387
06:44 ducttape_ joined #puppet-openstack
06:50 atom_ joined #puppet-openstack
06:59 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: Make ::tripleo::profile::base classes work with multiple nodes  https://review.openstack.org/338387
07:15 tesseract- joined #puppet-openstack
07:22 shardy joined #puppet-openstack
07:35 jaosorior joined #puppet-openstack
07:37 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: WIP Next generation HA architecture work  https://review.openstack.org/342650
07:40 atom_ joined #puppet-openstack
07:42 ifarkas joined #puppet-openstack
07:45 ducttape_ joined #puppet-openstack
08:04 derekh joined #puppet-openstack
08:04 danpawlik joined #puppet-openstack
08:04 openstackgerrit Saravanan KR proposed openstack/puppet-neutron: WIP: Added an parameter for neutron::agents::ml2::ovs to enable ovs-dpdk  https://review.openstack.org/341596
08:17 alee joined #puppet-openstack
08:21 atom_ joined #puppet-openstack
08:23 markvoelker joined #puppet-openstack
08:29 openstackgerrit Saravanan KR proposed openstack/puppet-vswitch: OVS DPDK package support with basic options  https://review.openstack.org/340367
08:31 dgurtner joined #puppet-openstack
08:46 ducttape_ joined #puppet-openstack
08:55 atom_ joined #puppet-openstack
08:56 dmburmistrov joined #puppet-openstack
09:17 markvoelker joined #puppet-openstack
09:35 atom_ joined #puppet-openstack
09:46 ducttape_ joined #puppet-openstack
09:56 openstackgerrit Daniel Pawlik proposed openstack/puppet-watcher: Added planer sections  https://review.openstack.org/341788
09:58 openstackgerrit Merged openstack/puppet-cinder: Add key manager parameters  https://review.openstack.org/342379
10:03 aernhart joined #puppet-openstack
10:11 markvoelker joined #puppet-openstack
10:31 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: openstack-core resource does not have interleave=true  https://review.openstack.org/342724
10:38 johnmilton joined #puppet-openstack
10:40 johnmilton joined #puppet-openstack
10:42 johnmilton joined #puppet-openstack
10:44 johnmilton joined #puppet-openstack
10:45 rcernin joined #puppet-openstack
10:47 johnmilton joined #puppet-openstack
10:47 johnmilton joined #puppet-openstack
10:51 johnmilton joined #puppet-openstack
11:04 aernhart joined #puppet-openstack
11:05 markvoelker joined #puppet-openstack
11:22 cdelatte joined #puppet-openstack
11:34 openstackgerrit Saravanan KR proposed openstack/puppet-vswitch: OVS DPDK package support with basic options  https://review.openstack.org/340367
11:42 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add ensure parameter to service identity resource  https://review.openstack.org/342754
11:44 alee_ joined #puppet-openstack
11:48 ducttape_ joined #puppet-openstack
11:58 EmilienM hi
11:59 markvoelker joined #puppet-openstack
12:04 dprince joined #puppet-openstack
12:07 colonwq joined #puppet-openstack
12:08 openstackgerrit Michele Baldessari proposed openstack/puppet-tripleo: openstack-core resource does not have interleave=true  https://review.openstack.org/342724
12:12 ducttape_ joined #puppet-openstack
12:15 EmilienM iurygregory: hey
12:16 EmilienM iurygregory: do you have plans to add authtoken thing in puppet-cinder?
12:20 mjblack EmilienM: not sure where you're seeing 'paramter'
12:20 EmilienM mjblack: the file name
12:20 mjblack ah ok
12:20 mjblack yeah sleep deprivation :D
12:21 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add ensure parameter to service identity resource  https://review.openstack.org/342754
12:22 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: PoC Validation - Make it fail (test)  https://review.openstack.org/342205
12:25 iurygregory EmilienM, i have plans for all modules =P you said that we should merge one before start the work in all other modules .-.
12:25 iurygregory morning
12:25 iurygregory XD
12:26 EmilienM iurygregory: do you need help to make quick progress or you got it covered?
12:27 iurygregory np ;) i'll pray to not break fuel i think XD
12:28 EmilienM iurygregory: I will probably push the cinder thing today or so, except if you have it in progress already
12:28 iurygregory EmilienM, i have in neutron
12:28 iurygregory go ahead
12:28 iurygregory o/
12:28 EmilienM this is a blocker for us, we would like to test barbican and it needs cinder.conf with v3 parameters
12:29 iurygregory EmilienM, np, put me in the review o/
12:32 alee joined #puppet-openstack
12:33 openstackgerrit Saravanan KR proposed openstack/puppet-vswitch: OVS DPDK package support with basic options  https://review.openstack.org/340367
12:34 openstackgerrit Matthew J Black proposed openstack/puppet-openstack-cookiecutter: Added ensure parameter to pass to service identity  https://review.openstack.org/342768
12:44 imcsk8 joined #puppet-openstack
12:45 degorenko EmilienM, for horizon the problem is that on ubuntu apache won't redirect base request to horizon page - it is on base default configuration. On centos we don't have such problem
12:45 degorenko as one possible solution for ubuntu - we have to specify bind_address
12:45 degorenko then we have proper redirect
12:45 EmilienM ok cool
12:45 degorenko but this we have one more problem
12:46 degorenko all wsgi manifests are including base apache configuration
12:46 degorenko whcih create base listen port to 80
12:46 degorenko then when we are trying to create listen port 80 on some address - we have conflict
12:47 degorenko we can set add_listen to false, but then will not be created 443 port for ssl from horizon :D
12:47 EmilienM can you just create a "if $redhat { class horizon blabla } elsif $ubuntu { class horizon do what you want }
12:47 degorenko yeees, but the problem is that we can set bind_address for horizon at all
12:47 degorenko with 80 port
12:47 degorenko because it is already created by apache itself
12:47 degorenko cant*
12:48 degorenko mwhahaha, ^
12:48 ducttape_ joined #puppet-openstack
12:48 degorenko in fuel we just passing add_listen false every time, and we have different implementation for apache - it already includes all needed ports
12:49 EmilienM do you use puppet-horizon?
12:49 degorenko yes, we use
12:49 degorenko again - we are passing to puppet-horizon::wsgi add_listen false
12:49 degorenko that means that puppet-horizon will not manage apache ports
12:50 degorenko and we have different call for apache for managing ports
12:50 degorenko so, it looks like bug in puppet-horizon module
12:50 degorenko i can upload possible solution here
12:50 degorenko but it should be discussed
12:51 EmilienM I'm going to propose a patch that enable testing on centos only now
12:51 EmilienM and I'll let you figure for ubuntu a solution
12:51 degorenko for me, question number 1) - why apache on ubuntu won't redirect by default to horizon
12:51 degorenko also we have different apache version on centos and ubuntu
12:52 degorenko 2.4.18 on ubuntu and 2.4.6 on centos
12:52 degorenko probably something changed in apache
12:52 mjblack degorenko: where are you setting add_listen to false?
12:52 degorenko in horizon
12:53 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: install tempest-horizon from source  https://review.openstack.org/340959
12:53 degorenko via extra_params
12:53 mjblack oh
12:53 degorenko so we don't manage in fuel apache ports from puppet-horizon
12:53 degorenko we do it in apache class
12:54 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: install tempest-horizon from source  https://review.openstack.org/340959
12:54 EmilienM degorenko: ^ now, we'll only test centos
12:54 EmilienM until you propose something for ubuntu
12:55 degorenko mjblack, any ideas why apache 2.4.18 won't redirect to horizon by default? :)
12:55 mjblack on ubuntu? removed the default vhost?
12:55 degorenko mjblack, it's present
12:55 mjblack that could be why
12:55 degorenko but it's present also on centos
12:55 degorenko 15-default.conf  15-horizon_ssl_vhost.conf  15-horizon_vhost.conf  openstack-dashboard.conf
12:56 degorenko 15-default.conf  15-horizon_vhost.conf
12:56 degorenko 1) centos, 2) ubuntu
12:56 degorenko on both OS its present
12:56 mjblack got a link to a review where this is happening?
12:56 degorenko mjblack, if i'll specify interface address for vhost horizon - it works on both
12:57 degorenko mjblack, patch from Emilien above
12:57 degorenko 5 patch set for example
12:58 degorenko mjblack, also we have broken binding address :) it uses same port 80 as apache and we have error https://github.com/puppetlabs/puppetlabs-apache/blob/master/manifests/vhost.pp#L426
12:59 degorenko i can upload change with collectors, for overriding base 80 port to interface specific port, if bind_address is enabled
13:00 degorenko but it will not help, when we have add_listen = false, from extra_params
13:00 mjblack so
13:00 mjblack on
13:00 mjblack the way apache works
13:00 degorenko yes, apache works
13:00 mjblack even though you have the same priority for for the conf file
13:01 degorenko have i can check, that priority same?
13:01 mjblack for namebasedvirtualhosts, it will default I believe to the servername that comes alphabetically first
13:02 mjblack so in the case of ubuntu
13:02 degorenko then they have same priority
13:02 mjblack servername starts with xenial
13:02 alee_ joined #puppet-openstack
13:02 mjblack but on centos it starts with centos
13:02 degorenko yes, but configs are same
13:02 mjblack right
13:02 degorenko how can fqdn affect it?
13:02 mjblack it doesnt know which one to use
13:03 mjblack so it resorts back to servername I believe, which ever hits first alphabetically
13:03 markvoelker joined #puppet-openstack
13:03 degorenko but both configs use same servername
13:04 mjblack right
13:04 mjblack but if you look at the servername for centos and ubuntu
13:05 mjblack for horizon
13:05 mjblack centos servername is starts with "centos"
13:05 mjblack and ubuntu starts with "xenial"
13:05 mjblack c > d
13:05 mjblack d > x
13:06 degorenko yes
13:06 degorenko and default servername is default
13:06 mjblack right
13:06 degorenko but
13:06 mjblack but the horizon vhost
13:06 mjblack servername is based off distro
13:06 mjblack centos -> http://logs.openstack.org/59/340959/5/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/ea8da24/logs/apache_config/15-horizon_vhost.conf.txt.gz
13:06 degorenko my server name for xenial and centos starts with sk_....
13:06 degorenko so i have sk_xenail
13:06 degorenko and sk_centos
13:06 mjblack ubuntu -> http://logs.openstack.org/59/340959/5/check/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-xenial/6ccad0b/logs/apache_config/15-horizon_vhost.conf.txt.gz
13:07 degorenko and in both cases default should be used first
13:07 degorenko but i have working centos and non-working ubuntu
13:07 degorenko how you can explain this? :)
13:08 mjblack I'd need to see it, you told me to look at that review from EmilienM :)
13:08 degorenko that's not on review
13:08 alee_ EmilienM, looks like the encryption_auth_url must be a v3 one.  https://github.com/openstack/cinder/blob/master/cinder/keymgr/barbican.py#L104
13:08 degorenko that's on my testing vms
13:08 degorenko i can share config files
13:09 degorenko mjblack, so here is ubuntu http://paste.openstack.org/show/533056/
13:09 degorenko mjblack, centos http://paste.openstack.org/show/533057/
13:10 mjblack centos config isnt using wildcard
13:10 mjblack line 37
13:10 degorenko oh wait
13:10 degorenko for centos not current configuration
13:11 degorenko let me sec
13:12 mjblack degorenko: what else is being configured under conf.d?
13:12 degorenko http://paste.openstack.org/show/533058/
13:12 mjblack for centos
13:13 EmilienM alee_: I figured /me working on puppet-cinder
13:13 alee_ EmilienM, ok
13:14 degorenko mjblack, they have a bit difference, centos is under ssl
13:14 degorenko and under ipv6 also
13:15 degorenko but for horizon there are no ipv6 settings
13:15 mjblack yeah but if you're being redirected from http to https then its skipping default vhost
13:18 mjblack degorenko: what I'm thinking is maybe on centos an apache config place by the package might not be getting purged
13:20 degorenko mjblack, looks like you're right about redirection
13:20 degorenko when i disable ssl on centos
13:20 degorenko i got same problem as on ubuntu
13:21 degorenko so we need to fix puppet-horizon, to work with bind_address
13:22 degorenko mjblack, did you read my comments above about conflicting base horizon 80 with default apache?
13:23 mjblack where it tried to create two of the same port settings?
13:23 degorenko mjblack, as one possible solution, we can have workaround for integration as: 1) ubuntu add add_listen = false + bind_address, 2) centos leave it as it now
13:23 ducttape_ joined #puppet-openstack
13:24 degorenko mjblack, it includes apache here https://github.com/openstack/puppet-horizon/blob/master/manifests/wsgi/apache.pp#L104 which creates base listen port here https://github.com/puppetlabs/puppetlabs-apache/blob/master/manifests/vhost.pp#L429
13:24 degorenko then when we are trying to do same for base horizon port - we have error from 426 line
13:24 alee_ EmilienM, the code we're using is basically whats in master, right?  or is it a branch?
13:24 mjblack degorenko: but thats only if you use default vhost
13:25 degorenko mjblack, which is enabled by default from base apache class.....
13:25 mjblack yeah but there is a great way to solve it :)
13:25 degorenko hiera i know
13:26 alee_ EmilienM, looks like things have changed a little on the nova side in master -- https://github.com/openstack/nova/blob/master/nova/keymgr/__init__.py#L49
13:28 degorenko EmilienM, try to rebase your change on https://review.openstack.org/#/c/341775/5 - with enabled ubuntu
13:28 degorenko it should work
13:30 mjblack degorenko: puppet-horizon is doing "include", cant you do also class { '::apache': default_vhost => false }?
13:30 degorenko mjblack, as another option yes
13:30 degorenko but hiera solution looks better
13:30 degorenko for me, imho :)
13:30 mjblack degorenko: heh, wont get an argument there from me
13:31 mjblack trying to do as much as possible through hiera instead of custom manifests
13:31 degorenko mjblack, so, in fuel we have exactly solution witohut hiera :D
13:32 mwhahaha lively discussions this morning
13:32 * mjblack isnt alive
13:32 mwhahaha yes i know you're dead inside like the rest of us
13:32 iurygregory yeah =(
13:33 mwhahaha anyway so i can update my patch set to also include osfamily specific hiera so we could just default_vhost => false for ubuntu only
13:33 mjblack mwhahaha: I'm sustained right now by enough energy drinks to kill an army
13:33 degorenko mwhahaha, no, your patch is ok for both OS
13:33 mwhahaha oh is it?
13:33 degorenko centos is working because of ssl :D
13:34 mjblack degorenko: maybe so but I'd like to see distro and distro-major_ver in the hiera.yaml too :D
13:34 mwhahaha oh it was failing yesterday, i hadn't gotten to the recheck results yet
13:38 EmilienM alee_: master
13:39 alee_ EmilienM, ok - I'll work a bit on the nova side in the meantime then
13:39 alee_ EmilienM, once we resolve cinder, we'll hit nova side issues next
13:39 EmilienM ok
13:39 EmilienM not sure
13:40 EmilienM it's deprecated
13:40 EmilienM not removed
13:40 EmilienM I'm finish the patch on cinder now
13:40 EmilienM err, I'm finishing*
13:40 karimb joined #puppet-openstack
13:40 richm joined #puppet-openstack
13:40 alee_ EmilienM, well -- even for the old way, there needs to be a [barbican] section
13:41 EmilienM degorenko: it just sad to see you need hiera to make horizon working on ubuntu
13:41 EmilienM alee_: ok
13:41 EmilienM alee_: please update puppet nova for it
13:41 degorenko EmilienM, no :) mjblack solution also will work
13:41 alee_ I think .. checking with the guys who wrote this ..
13:41 EmilienM alee_: can you also double check barbican options in cinder?
13:41 alee_ EmilienM, yes - doing that too.
13:41 EmilienM degorenko: I haven't followed all discussion.
13:42 degorenko EmilienM, we have at least 2 ways how we can enable horizon on ubuntu :D
13:45 alee_ EmilienM, https://github.com/openstack/nova/blob/master/nova/keymgr/__init__.py#L60-L62   - the setting is deprecated, but it gets translated to the new keymgr
13:48 dmsimard EmilienM: uh, hey, why aren't there any trove logs on scenario003 jobs ?
13:48 dmsimard http://logs.openstack.org/91/342391/2/check/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-xenial/72b23d1/logs/
13:48 dmsimard http://logs.openstack.org/91/342391/2/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/e97f5ba/logs/
13:48 EmilienM dmsimard: https://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario003.pp#L35
13:48 dmsimard it's not even installed ?
13:48 EmilienM just read the code
13:49 dmsimard ah, damnit
13:49 EmilienM yeah, #disableallthethings
13:49 dmsimard You should put "disabled" in the test matrix in the readme then :p
13:50 EmilienM right
13:50 dmsimard I was going to see if I could put memcached_tokens for the projects that are missing it in p-o-i
13:50 dmsimard I guess we can merge that as is and then address the deprecations later on
13:50 dmsimard I'll fix the swift thing
13:52 degorenko mwhahaha, is this patch ready? https://review.openstack.org/#/c/341775/
13:52 mwhahaha probably now, let me pull the  WIP from the commit message
13:52 degorenko mwhahaha, just suggestion
13:52 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/341775
13:52 degorenko may be better to set hiera config via parameter?
13:52 degorenko not just hardcoded puppet args?
13:52 degorenko in *
13:53 mwhahaha yea we can do that
13:53 degorenko so, you need to update patch again :D
13:53 mwhahaha i'll update it today and maybe throw in OS specific hiera items
13:53 degorenko mwhahaha, ok, ping me, when it will be ready :)
13:55 ducttape_ joined #puppet-openstack
13:55 openstackgerrit David Moreau Simard proposed openstack/puppet-swift: Fix comment typo  https://review.openstack.org/342871
13:56 dmsimard vinsh: hey o/
13:57 dmsimard vinsh: I haven't done swift with puppet-swift in a long time, where's memcached typically configured ? I see https://github.com/openstack/puppet-swift/blob/c659733d39c1c3e4739db272645d80e7985e8883/manifests/proxy.pp#L2
13:57 dmsimard The only place I see memcached servers being configured is in swift proxy cache but unless my memory is bad, it ought to be configured elsewhere too ..
13:59 openstackgerrit Saravanan KR proposed openstack/puppet-vswitch: OVS DPDK package support with basic options  https://review.openstack.org/340367
14:08 EmilienM iurygregory: by looking at your patch on puppet-nova, I found some things I would not have done
14:08 EmilienM I'll finish my cinder patch and let you a review
14:08 iurygregory EmilienM, ack ;)
14:08 EmilienM so we can compare the 2 approach
14:08 iurygregory sure ^^
14:08 atom_ joined #puppet-openstack
14:10 EmilienM iurygregory: so we don't need identity_uri and auth_uri anymore? I'm confused
14:10 EmilienM I don't see them in manifests/resource/authtoken.pp
14:12 iurygregory EmilienM, in authtoken.pp we have added all options according to configuration from keystonemiddleware, the identity_uri are in examples only
14:12 iurygregory we don't need to set both i think
14:12 EmilienM everything authtoken should go in this define
14:13 EmilienM and we should not let cinder_config { authtoken } thing in our manifests
14:13 EmilienM only use the define
14:13 iurygregory in nova we have the 'admin_password' 'admin_user'
14:13 iurygregory but keystone i haven't found reference for this
14:14 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: In-process token caching is deprecated, use memcached instead  https://review.openstack.org/342391
14:16 markvoelker joined #puppet-openstack
14:19 openstackstatus NOTICE: Gerrit is restarting to correct memory/performance issues.
14:28 jaosorior joined #puppet-openstack
14:29 vinsh dmsimard: Hi, memcache servers.. yeah.  in proxy.conf
14:30 vinsh But, you need to add the memcached module to a node also.. to provide the actual service
14:30 dmsimard vinsh: I guess it's only in swify proxy cache middleware manifest, I thought it was elsewhere.
14:30 dmsimard Yup.
14:31 vinsh Most all other services.. expirer.. reconcilator run on the proxy and can leverage that config too
14:36 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: authtoken: allow memcached_servers to be empty or os_service_default  https://review.openstack.org/342895
14:36 EmilienM iurygregory: ^
14:36 EmilienM we need to do the memcache_servers join thing here ^
14:36 EmilienM and not in modules
14:37 markvoelker joined #puppet-openstack
14:37 iurygregory EmilienM, oh cool :D
14:38 openstackgerrit Merged openstack/puppet-tripleo: Add resource for requesting certificates for HAProxy  https://review.openstack.org/341405
14:41 EmilienM this authtoken thing is a lot of work
14:42 iurygregory it's a little crazy
14:42 * iurygregory hides
14:42 llua joined #puppet-openstack
14:43 karthiks joined #puppet-openstack
14:43 iurygregory EmilienM, if the option is not in the bible we will let be configurable by the resource from the module? like admin_password in nova, we let nova_conf configure?
14:47 jaosorior_ joined #puppet-openstack
14:48 jaosorior joined #puppet-openstack
14:48 openstackgerrit Bob Fournier proposed openstack/puppet-neutron:    Remove unused enable_tunneling in openvswitch_agent.ini  https://review.openstack.org/342467
14:54 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: WIP - Configure keystone authtoken options  https://review.openstack.org/342905
14:54 EmilienM iurygregory: early patch (wip) ^
14:55 EmilienM it should be backward compatible
14:59 xarses joined #puppet-openstack
15:00 openstackgerrit Merged openstack/puppet-tripleo: Add MySQL profiles  https://review.openstack.org/340444
15:10 iurygregory EmilienM, ack
15:25 openstackgerrit Merged openstack/puppet-openstack-integration: install tempest-horizon from source  https://review.openstack.org/340959
15:36 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/342905
15:36 EmilienM iurygregory: can you review it? ^
15:37 iurygregory back from lunch o/
15:37 iurygregory i'll ;)
15:38 dmsimard wth is that error http://logs.openstack.org/91/342391/3/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/3f0e712/console.html#_2016-07-15_14_32_55_790226
15:38 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/342905
15:39 EmilienM just release note update ^
15:40 fvollero joined #puppet-openstack
15:40 Hunner beaker bugfix released
15:40 EmilienM dmsimard: I saw it already
15:40 EmilienM Hunner: yeah? wow
15:40 EmilienM let's try
15:41 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_spec_helper: Revert "Pin beaker to 2.46.0"  https://review.openstack.org/342931
15:41 EmilienM mwhahaha: fyi ^
15:41 Hunner changelog https://github.com/puppetlabs/beaker/blob/2.47.1/HISTORY.md#LATEST
15:41 mwhahaha cool
15:46 EmilienM colonwq: any progress on rgw?
15:47 dmsimard EmilienM: eh, swift has it's own ipv6 parsing thing
15:47 dmsimard the inet6 format expected by python-memcache doesn't work for swift
15:47 EmilienM damn
15:47 dmsimard http://logs.openstack.org/91/342391/3/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/3f0e712/logs/syslog.txt.gz#_Jul_15_14_35_11
15:47 dmsimard I'll fix it
15:48 EmilienM in swift?
15:48 EmilienM swift doesn't work with ipv6? lol
15:48 dmsimard it works
15:48 dmsimard just the parsing is different
15:48 dmsimard it detects that it's an ipv6
15:48 iurygregory EmilienM, looks good but i think you need to update puppet-keystone or CI will not pass :D and need add tests for new parameters
15:50 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
15:51 EmilienM iurygregory: ok
15:57 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/341775
15:57 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/342935
16:04 openstackgerrit Merged openstack/puppet-openstacklib: service validation: log output on failures  https://review.openstack.org/342279
16:15 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: authtoken: allow memcached_servers to be empty or os_service_default  https://review.openstack.org/342895
16:18 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/342905
16:18 karimb joined #puppet-openstack
16:27 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: In-process token caching is deprecated, use memcached instead  https://review.openstack.org/342391
16:36 ducttape_ joined #puppet-openstack
16:42 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Bump RDO repositories  https://review.openstack.org/342951
16:45 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/337860
16:45 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/337860
16:48 degorenko EmilienM, https://review.openstack.org/#/c/342935/ - ubuntu horizon tests passed
16:48 degorenko depends on https://review.openstack.org/#/c/341775/7
17:11 tdb_ do the 8.x mitaka openstack modules work on xenial?
17:11 iurygregory tdb, to me mitaka is in trusty (14.04)
17:12 iurygregory xenial is for newton and after i think =)
17:12 tdb from an Ubuntu point of view, 16.04 was release with mitaka
17:13 iurygregory O.o you can try i think XD
17:13 tdb maybe from a puppet module point of view it's as you say
17:13 iurygregory yeah =)
17:13 tdb which is I guess what my question was :)
17:14 EmilienM _degorenko|afk: ok so we need hiera
17:15 EmilienM tdb: right, in theory you can deploy Mitaka on Xenial, and our modules should work (or almost) but we officialy started to support Xenial from newton
17:16 tdb right, ok. a little annoying, but I understand
17:17 tdb I wanted to use the latest released openstack with the latest ubuntu
17:17 tdb I guess there's nothing much wrong with 14.04 until newton is out
17:21 openstackgerrit Brent Eagles proposed openstack/puppet-octavia: Implement worker.pp  https://review.openstack.org/328495
17:31 EmilienM tdb: I think you can deploy mitaka on xenial with stable/mitaka
17:31 EmilienM give it a try and let us know
17:32 tdb I did; it failed with upstart/systemd issue
17:32 EmilienM ah
17:32 tdb which is why I started checking if it was supported before I went to far with it
17:32 EmilienM so use master to deploy mitaka
17:32 EmilienM it should work too I think
17:32 tdb oh, I wasn't sure if it did
17:34 EmilienM I think it should
17:34 EmilienM let us know if not
17:34 tdb I'll give it a whirl
17:37 EmilienM yeah mitaka & xenial were released in the same time, it was really tricky for us to test it together
17:37 EmilienM so we decided we would start xenial support during newton cycle
17:37 EmilienM thing we did and finished!
17:38 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/342905
17:39 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
17:53 EmilienM mwhahaha: is https://review.openstack.org/#/c/341775/ ready ?
17:59 rcernin joined #puppet-openstack
18:00 johnmilton joined #puppet-openstack
18:03 mwhahaha yea should be
18:04 EmilienM dmsimard: why designate patch fails on tempest about trove?
18:04 iurygregory LOL
18:05 EmilienM mwhahaha: +2
18:06 mwhahaha EmilienM: oh i was adding in configurability in hiera i guess i forgot to push it this morning
18:06 EmilienM mwhahaha: cool I'll +2 again
18:06 dmsimard EmilienM: what ?
18:09 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/341775
18:11 EmilienM dmsimard: see logs
18:11 dmsimard EmilienM: I don't know, I'm focused on making designate work
18:12 dmsimard there's some mistral stuff too
18:12 EmilienM yeah
18:12 EmilienM I'm digging
18:12 dmsimard keystone endpoint not found errors again
18:12 EmilienM 2016-07-15 17:23:13.478646 | {1} designate_tempest_plugin.tests.api.v2.test_zones.ZonesTest.test_delete_zone [0.209007s] ... FAILED
18:12 dmsimard I've seen that before never found what it was
18:12 EmilienM anyway designate fails too
18:12 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/342935
18:12 dmsimard yeah I know but we're making progress, no more stack traces
18:13 dmsimard but designate keystone authentication is failing now
18:13 dmsimard http://logs.openstack.org/60/337860/12/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/f69be31/logs/designate/api.txt.gz#_2016-07-15_17_22_51_971
18:13 dmsimard shouldn't that be ipv6 ?
18:13 EmilienM http://logs.openstack.org/60/337860/12/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/f69be31/logs/designate/api.txt.gz#_2016-07-15_17_23_02_975
18:13 EmilienM it's doing http and ipv4
18:13 dmsimard ohhhh
18:13 dmsimard I forgot to configure it
18:13 dmsimard I took exceprts from beaker tests
18:16 rwsu joined #puppet-openstack
18:17 johnmilton joined #puppet-openstack
18:18 dmsimard EmilienM: nooooooooooo
18:18 dmsimard http://nooooooooooooooo.com/
18:18 iurygregory LOL
18:18 EmilienM dmsimard: https://cdn.meme.am/instances/39036008.jpg
18:18 dmsimard I never thought I'd see split keystone parameters like these ever https://github.com/openstack/puppet-designate/blob/master/manifests/api.pp#L85-L87
18:19 iurygregory EmilienM, LOL
18:19 mwhahaha some old code right there
18:19 dmsimard I still remember mgagne trying to kill all of them
18:19 mwhahaha needs to be updated
18:19 dmsimard from a past file
18:19 dmsimard life*
18:19 ducttape_ joined #puppet-openstack
18:19 mgagne :D
18:19 EmilienM dmsimard: you'll need something like https://review.openstack.org/342905
18:19 iurygregory We don't need for authtoken
18:19 dmsimard EmilienM: I'll need something like nothing because that'll be the way it's going to work for the time being :p
18:20 EmilienM :-O
18:20 dmsimard there are several fixes to do to puppet-designate already
18:20 dmsimard I should be filing bugs
18:20 dmsimard lots of bad defaults amongst other things
18:20 mwhahaha the land of unloved puppet modules
18:20 EmilienM yep
18:20 dmsimard mwhahaha: it doesn't beat swift
18:21 EmilienM we have 30 modules a very few active contributors
18:21 EmilienM it's quite challenging
18:21 mwhahaha no one likes swift, that ones been intentional :D
18:21 iurygregory i like swift .-.
18:21 mgagne puppet-swift =)
18:21 iurygregory from the user perspective =P
18:22 dmsimard 2014 damn https://bugs.launchpad.net/puppet-nova/+bug/1274979
18:22 openstack Launchpad bug 1274979 in puppet-swift "Introduce public_url, internal_url and admin_url parameters for endpoint configuration" [Undecided,Fix released] - Assigned to Mathieu Gagné (mgagne)
18:22 mwhahaha it said puppet-designate was fixed
18:22 mwhahaha such lies
18:23 dmsimard probably in keystone auth or some other place
18:23 dmsimard and missed ::api
18:23 iurygregory :O
18:23 iurygregory git blame =P
18:27 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/337860
18:27 dmsimard EmilienM: ^ has a better chance of working
18:30 dmsimard EmilienM: do you mind if I configure puppet-swift to log to /var/log/swift in p-o-i ? It sucks to search through journalctl/syslog
18:30 EmilienM go ahead
18:30 EmilienM I always wished someone would do it
18:30 mwhahaha no you may not do useful work
18:30 mwhahaha only meaningless, unhelpful reviews will be accepted going forward
18:33 dmsimard Would you put it in swift.pp ? Or, say, in swift_logs.pp or right in the scenario fixture ?
18:33 EmilienM iurygregory: http://git.openstack.org/cgit/openstack/keystonemiddleware/tree/keystonemiddleware/auth_token/_auth.py#n46
18:33 EmilienM identity_uri is still required
18:33 EmilienM I'm so confused
18:33 dmsimard The way I did it back at $oldjob was through rsyslog
18:34 EmilienM I'm going to ask steve
18:34 iurygregory EmilienM,  can i cry?
18:34 danpawlik EmilienM: :(
18:40 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/342905
18:40 EmilienM let's try again with auth_type = password
18:40 iurygregory good to know that identity_uri is deprecated XD
18:45 EmilienM yeah
18:46 EmilienM it's *at least*  one good news :P
18:46 iurygregory yep
18:46 EmilienM iurygregory: you see how I did https://review.openstack.org/#/c/342905/ ?
18:46 EmilienM it should be backward compatible and transparent
18:47 iurygregory yeah ;) i i'll update nova to be like your patch :D
18:48 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/343031
18:48 iurygregory sorry for the trouble =)
18:48 openstackgerrit Merged openstack/puppet-openstack-integration: Bump RDO repositories  https://review.openstack.org/342951
18:49 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/343031
18:49 openstackgerrit Brent Eagles proposed openstack/puppet-octavia: Implement worker.pp  https://review.openstack.org/328495
18:51 openstackgerrit Merged openstack/puppet-openstack_spec_helper: Revert "Pin beaker to 2.46.0"  https://review.openstack.org/342931
18:53 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/343031
18:55 bkero dmsimard: is that File['/var/log/swift'] resource going to conflict with puppet-swift?
18:55 dmsimard bkero: puppet-swift doesn't manage /var/log/swift
18:55 dmsimard or it'd be caught in the logs (but remain empty)
18:55 dmsimard http://logs.openstack.org/91/342391/4/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/3e7ce6b/logs/ no swift directory
18:56 bkero Huh, yeah.
18:56 bkero [bkero@Whitefall puppet-swift]$ grep -R "/var/log/swift" *
18:56 bkero [bkero@Whitefall puppet-swift]$
18:57 dmsimard the directories are mostly left up to packaging now
18:57 dmsimard possible it was managed before
18:57 dmsimard swift is "special"
18:57 bkero Hmm
18:57 bkero There is a bunch of logging stuff in manifests/proxy.pp
18:58 bkero log_headers, log udp port/host, address, etc
18:59 bkero including references to /dev/log O__o
18:59 * bkero whistles quietly while looking at https://blueprints.launchpad.net/swift/+spec/swift-oslo-logging
19:00 dmsimard the default log facility LOG_LOCAL1 will work
19:00 dmsimard https://review.openstack.org/#/c/343031/3/files/rsyslog-swift.conf local1
19:00 bkero yup
19:00 * bkero was thinking of ways that involved writing less rsyslog config files :P
19:01 dmsimard Well, RDO could embed defaults into packaging
19:01 dmsimard but that'd be cheating
19:01 dmsimard I don't remember what UCA does
19:01 dmsimard I guess it leaves the same defaults to syslog
19:02 dmsimard yeah, also syslog
19:02 openstackgerrit Alex Schultz proposed openstack/puppet-openstack_spec_helper: Revert "Pin beaker to 2.46.0"  https://review.openstack.org/343036
19:02 openstackgerrit Alex Schultz proposed openstack/puppet-openstack_spec_helper: Revert "Pin beaker to 2.46.0"  https://review.openstack.org/343037
19:03 dmsimard I kind of cheated and took part of https://github.com/openstack/swift/blob/master/etc/swift-rsyslog.conf-sample
19:03 dmsimard I guess I could leave it vanilla
19:05 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/341775
19:05 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/342935
19:06 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/343031
19:08 EmilienM dmsimard: why not doing it in puppet-swift ? ^
19:08 EmilienM it's a nice feature
19:08 dmsimard EmilienM: it's so weird that designate installation breaks mistral, aodh and trove
19:08 EmilienM yeah
19:09 EmilienM next time when you go back from PTO, just read emails and continue your regular work
19:09 dmsimard sorry for trying to fix things
19:09 dmsimard EmilienM: re: logs
19:10 dmsimard EmilienM: I guess I can put it in puppet-swift
19:14 openstackgerrit Brent Eagles proposed openstack/puppet-octavia: Adding health_worker.pp  https://review.openstack.org/343044
19:16 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/342905
19:18 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/337860
19:22 xarses_ joined #puppet-openstack
19:23 xarses_ joined #puppet-openstack
19:23 xarses_ joined #puppet-openstack
19:27 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: compute: update doc for keymgr api_class  https://review.openstack.org/343047
19:54 openstackgerrit Emilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/342905
20:04 iurygregory bye people have nice weekend o/
20:04 alee_afk joined #puppet-openstack
20:05 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: compute: update doc for keymgr api_class  https://review.openstack.org/343047
20:06 EmilienM iurygregory: thanks, bye !
20:09 mjblack iurygregory: you too
20:25 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/337860
20:26 dmsimard EmilienM, mwhahaha: oh, look, swift logs in /var/log/swift: http://logs.openstack.org/31/343031/4/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/d0f82f3/logs/swift/
20:26 dmsimard but now EmilienM wants it in puppet-swift T_T
20:28 EmilienM oh nice
20:28 EmilienM yeah it would be so cool
20:28 EmilienM thanks for doing it
20:29 dmsimard that'll cost you one more beer in barcelona
20:31 aernhart joined #puppet-openstack
20:32 EmilienM ok :)
20:33 dmsimard hm, wait, it doesn't look like it works for Ubuntu
20:33 dmsimard I think for Ubuntu the directory needs particular permissions
20:33 dmsimard but the job didn't fail ?
20:33 EmilienM heh I managed to make cinder working with keystone v3 on https://review.openstack.org/#/c/342905/
20:34 mwhahaha it won't fail you just get no longs
20:34 mwhahaha yea /var/log/swift needs specific perms under ubuntu
20:34 dmsimard mwhahaha: yeah but I mean the directory /var/log/swift apparently didn't ge5 created
20:34 dmsimard It's created here: http://logs.openstack.org/31/343031/4/check/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-xenial/cc8794e/console.html#_2016-07-15_19_20_45_810985
20:35 dmsimard But it's not captured in the artifacts ? http://logs.openstack.org/31/343031/4/check/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-xenial/cc8794e/logs/
20:35 mwhahaha under ubuntu it needs swift:adm for the dir
20:35 mwhahaha i believe
20:35 mwhahaha or syslog:adm
20:35 dmsimard yeah syslog:adm
20:36 dmsimard hm, what'd be the clean way to do that ? if distro ubuntu spaceship and add owner/group ?
20:38 mwhahaha there is no clean way for this
20:38 mwhahaha this is why we don't manage users/groups/files via puppet ;)
20:38 dmsimard http://paste.openstack.org/show/533098/ ?
20:38 dmsimard mwhahaha: ^
20:39 mwhahaha yea that could work
20:39 dmsimard I'll try
20:42 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
20:42 EmilienM ok https://review.openstack.org/#/c/342905/ is ready for review
20:42 EmilienM the CI proves it's backward compatible
20:42 EmilienM I'll let reviewers vote
20:43 mwhahaha it'll break fuel
20:43 mwhahaha :D
20:43 mwhahaha or not we'll see
20:43 EmilienM oh
20:44 EmilienM yeah, I can -2 it if you want
20:44 mwhahaha naw it's ok we'll wait
20:44 EmilienM or feel free to do it if you see risk
20:45 openstackgerrit Chris Jones proposed openstack/puppet-nova: WIP - DUMMY - Testing 338387  https://review.openstack.org/343067
20:46 EmilienM mwhahaha: what ? https://review.openstack.org/#/c/342935/
20:46 EmilienM where is code
20:46 mwhahaha *magic*
20:46 mwhahaha all i did was rebase :D
20:46 mwhahaha via the UI
20:46 mwhahaha gerrit ate it, i'll republish
20:47 EmilienM ahah
20:47 EmilienM it's the 1st time I see it
20:49 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/341775
20:49 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/342935
20:49 mwhahaha cause the parent was on an old version
20:49 openstackgerrit David Moreau Simard proposed openstack/puppet-swift: Add openstack::swift::rsyslog to configure rsyslog for Swift  https://review.openstack.org/343070
20:50 mwhahaha i had rebased mine via the UI, but then republished a local version which had a pre-horizon scenario003 parent
20:50 mwhahaha so the code got eaten via gerrit
20:51 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/343031
20:51 dmsimard EmilienM: ^ integration in puppet-swift and p-o-i depends-on
20:52 EmilienM dmsimard: cool!
20:52 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/342935
20:52 openstackgerrit David Moreau Simard proposed openstack/puppet-swift: Add openstack::swift::rsyslog to configure rsyslog for Swift  https://review.openstack.org/343070
20:52 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/343031
20:55 EmilienM dmsimard: dude you're going to hate me but we can't deploy rsyslog that way
20:55 EmilienM if we do that, we break ton of deployments
20:56 dmsimard EmilienM: it's an optional class
20:56 EmilienM I can write the code
20:56 vinsh I just add rsyslog to my profiles for nodes on its own .. and swift pumps into that
20:56 EmilienM yeah but I would rather do it in openstacklib
20:56 EmilienM it's an helper thing
20:56 dmsimard feel free to take the patch :p
20:56 dmsimard I was just tired of digging through syslog for p-o-i logs
20:56 EmilienM yeah, if you don't do it, I'll probably take over but right now I'm already multi tasking
20:57 dmsimard EmilienM: want me to leave it in p-o-i standalone for the time being then ?
20:58 EmilienM no, keep your class now
20:58 EmilienM I'll move it later
20:58 EmilienM your code is wrong anyway, class name is not good
21:00 EmilienM dmsimard: let me finish another task and I'll doo it
21:14 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
21:15 EmilienM dmsimard: mhh.. ok let's do it in p-o-i now
21:15 EmilienM I realize people probably handle rsyslog themselves
21:16 EmilienM we could think at a helper but later
21:17 dmsimard EmilienM: ok
21:18 dmsimard EmilienM: I guess I'll add it to beaker too ?
21:18 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
21:18 EmilienM dmsimard: good idea
21:19 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/343031
21:20 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/343031
21:20 EmilienM dmsimard: +2
21:20 dmsimard hang on, gotta see if it works for Ubuntu first
21:21 dmsimard it didn't the first time around but I added the pew pew spaceship operator
21:21 haplo37_ joined #puppet-openstack
21:21 EmilienM i'm out a bit, bbl
21:35 openstackgerrit Bob Fournier proposed openstack/puppet-neutron:    Remove unused enable_tunneling in openvswitch_agent.ini  https://review.openstack.org/342467
21:43 karimb joined #puppet-openstack
22:09 EmilienM dmsimard: mistral tests are now running by default oO
22:11 EmilienM nevermind, wrong log
22:11 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
22:12 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: compute: add missing options for Barbican  https://review.openstack.org/343102
22:13 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/339806
22:17 EmilienM mwhahaha: I'm trying to find what is breaking in Fuel
22:17 mwhahaha we specify auth_version
22:17 EmilienM ah!
22:17 mwhahaha we'll just have to coordinate next week
22:18 EmilienM mwhahaha: can't we use depends-on?
22:18 mwhahaha nope
22:18 mwhahaha fuel-ci doesn't do depends-on :/
22:18 EmilienM ok
22:18 EmilienM do you want me to submit a patch in fuel?
22:18 mwhahaha we could do the inverse, just no one else is around to help me merge
22:19 EmilienM let's do it on Monday
22:19 EmilienM there is no hurry
22:19 mwhahaha we already have a review to remove the duplicated signing_dir thing
22:19 mwhahaha we just need to move auth_version as well, https://github.com/openstack/fuel-library/blob/master/deployment/puppet/openstack_tasks/manifests/openstack_cinder/openstack_cinder.pp#L190-L194
22:19 EmilienM cool
22:19 mwhahaha i created a bug last week to get the ball rolling to support this effort
22:20 mwhahaha so nova, glance and cinder i know had issues
22:20 EmilienM my patch in puppet-cinder is painful and we'll need it everywhere
22:20 EmilienM we'll probably use some help
22:20 EmilienM if you know people won't don't know what to do :P
22:20 EmilienM who don't*
22:20 mwhahaha keystone_authtoken config options are painful :D
22:21 EmilienM yeah, but this work is really going to make it consistent across our modules
22:21 mwhahaha https://bugs.launchpad.net/fuel/+bug/1600276
22:21 mwhahaha indeed
22:21 openstack Launchpad bug 1600276 in Fuel for OpenStack "upstream puppet modules changing way to configure keystone_authtoken parameters" [High,Confirmed] - Assigned to MOS Puppet Team (mos-puppet)
22:21 EmilienM so yeah it's a pain in the ass, but it's worth it
22:21 EmilienM excellent
22:21 mwhahaha especially since we don't expose many of the options
22:22 EmilienM after oslo things and keyste_authtoken, we start to be really good
22:25 mjblack EmilienM: got maybe an easy question
22:25 mjblack where is the CI job for cookiecutter?
22:27 EmilienM mjblack: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L343-L361
22:27 mwhahaha in the CI config
22:27 mwhahaha i'll show myself out now
22:27 EmilienM cheers
22:27 EmilienM me too actually
22:27 * mjblack replaces mwhahaha with hiera
22:27 mwhahaha and nothing of value was lost :D
22:28 mjblack you have a low opinion of yourself :D
22:29 mjblack mwhahaha: reason I ask is it looks like cookiecutter is failing because of localization
22:30 mwhahaha yes i am aware
22:30 mwhahaha i poked openstack-infra but the missing locale stuff hasn't been fixed yet
22:31 mwhahaha i was going to look into setting the LOCALE in the job as a temporary workaround but i find it odd that it works in all our other jobs but not in that one
22:31 mwhahaha because it does the same stuff as every other lint job
22:32 mjblack its not lint though
22:32 mwhahaha but it is
22:32 mwhahaha it's the thing that does the license check is loading a file with a non-ascii char
22:33 mjblack umm but its not lint its the validate rake task
22:33 mjblack well I guess its still lint but a technicality
22:33 mwhahaha http://eavesdrop.openstack.org/irclogs/%23puppet-openstack/%23puppet-openstack.2016-07-11.log.html#t2016-07-11T18:19:25
22:33 mwhahaha yea but our lint task runs the validate stuff
22:33 mwhahaha or rather our lint job does
22:36 mwhahaha so on monday i went over to #openstack-infra which went into a very long conversation about where locale.conf comes from
22:36 mwhahaha turns out it's magic and the image building process doesn't do it
22:36 mwhahaha cause of reasons
22:37 mjblack lol
22:37 mjblack where is the builders defined?
22:37 mwhahaha systemd, chroot, pain
22:37 mwhahaha all were discussed
22:37 mjblack lint doesnt have tasks but builders
22:39 mwhahaha https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml
22:40 mwhahaha so we could try exporting LANG=en_US.utf8 before the cookiecutter job
22:40 mwhahaha not sure if it'll fix it
22:40 mwhahaha and it requires a job change :/
22:41 mjblack so the reason it works for actual modules
22:41 mjblack is its not using ruby json library to parse the json
22:41 mjblack https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml#L350
22:42 mjblack where cookiecutter is failing is at https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L360
22:42 mwhahaha https://review.openstack.org/343106
22:43 mjblack heh
22:43 mwhahaha actually do we not run spec validate against the regular modules?
22:43 mjblack nope
22:44 mwhahaha probably why the do the same thing https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml#L373-L382 for the infra modules
22:44 mjblack yeah
22:44 mjblack so cookiecutter and actual modules do lint and spec but in cookiecutter validate is executed too
22:45 mwhahaha it used to work, i think there was a new release of the https://github.com/domcleal/spdx-licenses/ since we last really did any work on the cookie cuter modules
22:45 mjblack in actual module python is used to validate json
22:45 mwhahaha something for next week
22:45 * mwhahaha wanders off
22:45 mjblack lol ok
23:00 openstackgerrit Merged openstack/puppet-keystone: authtoken: allow memcached_servers to be empty or os_service_default  https://review.openstack.org/342895
23:05 openstackgerrit Merged openstack/puppet-keystone: Add ensure parameter to service identity resource  https://review.openstack.org/342754
23:06 openstackgerrit Merged openstack/puppet-swift: Fix comment typo  https://review.openstack.org/342871
23:10 alee joined #puppet-openstack
23:25 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add ensure parameter to service identity resource  https://review.openstack.org/343113
23:30 openstackgerrit Ade Lee proposed openstack/puppet-nova: Add missing parameters for Barbican key manager  https://review.openstack.org/343114
23:30 haplo37_ joined #puppet-openstack
23:31 EmilienM alee: I already did it
23:31 EmilienM alee: https://review.openstack.org/#/c/343102/
23:31 alee EmilienM, yeah - I just noticed that ..
23:31 alee EmilienM, mine has a couple things different
23:32 alee EmilienM, but is essentially the same -- need to have python-castellan installed
23:32 EmilienM python-castellan should be a dependency in cinder afik
23:32 EmilienM it's already let me check
23:33 EmilienM tempest is currently running, let's see if it works now
23:33 alee EmilienM,  well castellan integration for volume encryption is only there for nova right now
23:34 alee EmilienM, cinder has not made it yet - so I'd be surprised if it were a cinder prereq
23:35 alee EmilienM, anyways - gotta go to bed now ...
23:36 EmilienM python2-castellan-0.4.1-0.20160606180822.44233de.el7.centos.noarch
23:36 EmilienM http://logs.openstack.org/06/339806/10/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/2b4228c/logs/rpm-qa.txt.gz
23:36 EmilienM already installed
23:36 EmilienM alee: good night !
23:36 alee cool
23:46 alee EmilienM, we're still getting 404 -- but I think thats because the auth_url needs a v3 on it
23:46 alee for cinder ..
23:47 alee EmilienM, hmm .. looking ..
23:51 alee EmilienM, ah  perhaps I'm not looking at the current tempest run ..
23:56 EmilienM http://logs.openstack.org/06/339806/17/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/cc965c0/logs/cinder/cinder-api.txt.gz#_2016-07-15_23_43_43_588
23:56 EmilienM alee: ^
23:56 EmilienM yeah, let's see why it fails
23:56 EmilienM alee: in barbican.pp ? auth_url?
23:57 alee EmilienM, no -- this is ok -- the 404 error was because it needed a v3
23:58 alee EmilienM, this is an error coming from the barbican server
23:58 alee it doesn;t like the token presented
23:58 EmilienM v3 where?
23:59 EmilienM alee: ^

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