Friday, 2016-07-15

« Thursday, 2016-07-14 Index Saturday, 2016-07-16 »
EmilienMdmsimard: ok00:06
EmilienMdmsimard: please just update swift00:06
EmilienMand +200:06
dmsimardEmilienM: will do00:06
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: PoC with openvswitch - Composable service validation within profiles  https://review.openstack.org/34220200:09
*** ducttape_ has quit IRC00:14
*** ducttape_ has joined #puppet-openstack00:14
EmilienMalee:  https://[::1]:5000/auth/tokens00:17
EmilienMstill failing00:17
EmilienMhttp://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_19200:17
EmilienMbarbican is hardcoding the url or?00:17
EmilienManyway, will continue tomorrow. Peace00:20
*** xiaohua has joined #puppet-openstack00:20
*** ducttape_ has quit IRC00:22
*** alee has quit IRC00:28
*** saneax is now known as saneax_AFK00:31
*** ducttape_ has joined #puppet-openstack00:47
*** markvoelker has quit IRC00:48
*** xiaohua has quit IRC00:49
*** iurygregory_ has joined #puppet-openstack00:56
*** ducttape_ has quit IRC01:38
*** ducttape_ has joined #puppet-openstack01:44
*** ilbot3 has quit IRC01:46
*** ilbot3 has joined #puppet-openstack01:48
*** ducttape_ has quit IRC01:49
*** richm has quit IRC01:49
*** xiaohua has joined #puppet-openstack02:03
*** richm has joined #puppet-openstack02:07
*** ducttape_ has joined #puppet-openstack02:28
*** iurygregory_ has quit IRC02:38
*** ducttape_ has quit IRC02:39
*** markvoelker has joined #puppet-openstack02:48
*** xarses has quit IRC03:13
*** xarses has joined #puppet-openstack03:13
*** xarses has quit IRC03:16
*** xarses has joined #puppet-openstack03:16
*** xarses has quit IRC03:21
*** xarses has joined #puppet-openstack03:21
*** xarses has quit IRC03:25
*** xarses has joined #puppet-openstack03:25
*** xarses has quit IRC03:26
*** xarses has joined #puppet-openstack03:27
*** xarses has quit IRC03:34
*** xarses has joined #puppet-openstack03:34
*** ducttape_ has joined #puppet-openstack03:40
*** ducttape_ has quit IRC03:44
*** richm has quit IRC03:57
*** xiaohua has quit IRC04:23
*** ducttape_ has joined #puppet-openstack04:40
*** ducttape_ has quit IRC04:46
*** saneax_AFK is now known as saneax05:14
*** sanjayu|AFK is now known as sanjayu05:14
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Make ::tripleo::profile::base classes work with multiple nodes  https://review.openstack.org/33838705:26
*** ducttape_ has joined #puppet-openstack05:42
*** markvoelker has quit IRC05:43
*** ducttape_ has quit IRC05:47
*** skramaja has joined #puppet-openstack05:48
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/34262506:11
*** imcsk8 has quit IRC06:14
*** imcsk8 has joined #puppet-openstack06:14
*** rcernin has joined #puppet-openstack06:18
*** markvoelker has joined #puppet-openstack06:30
*** openstackgerrit has quit IRC06:33
*** openstackgerrit has joined #puppet-openstack06:33
*** markvoelker has quit IRC06:39
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Make ::tripleo::profile::base classes work with multiple nodes  https://review.openstack.org/33838706:41
*** ducttape_ has joined #puppet-openstack06:44
*** ducttape_ has quit IRC06:48
*** atom_ has joined #puppet-openstack06:50
*** xarses has quit IRC06:54
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: Make ::tripleo::profile::base classes work with multiple nodes  https://review.openstack.org/33838706:59
*** tesseract- has joined #puppet-openstack07:15
*** atom_ has quit IRC07:22
*** shardy has joined #puppet-openstack07:22
*** jaosorior has joined #puppet-openstack07:35
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: WIP Next generation HA architecture work  https://review.openstack.org/34265007:37
*** atom_ has joined #puppet-openstack07:41
*** ifarkas has joined #puppet-openstack07:42
*** ducttape_ has joined #puppet-openstack07:45
*** ducttape_ has quit IRC07:49
*** rwsu has quit IRC07:51
*** llua has quit IRC07:54
*** derekh has joined #puppet-openstack08:04
*** danpawlik has joined #puppet-openstack08:04
openstackgerritSaravanan KR proposed openstack/puppet-neutron: WIP: Added an parameter for neutron::agents::ml2::ovs to enable ovs-dpdk  https://review.openstack.org/34159608:04
*** atom_ has quit IRC08:07
*** alee has joined #puppet-openstack08:17
*** atom_ has joined #puppet-openstack08:21
*** alee has quit IRC08:23
*** markvoelker has joined #puppet-openstack08:23
*** markvoelker has quit IRC08:27
openstackgerritSaravanan KR proposed openstack/puppet-vswitch: OVS DPDK package support with basic options  https://review.openstack.org/34036708:29
*** dgurtner has joined #puppet-openstack08:31
*** atom_ has quit IRC08:41
*** dtantsur|afk is now known as dtantsur08:44
*** ducttape_ has joined #puppet-openstack08:46
*** ducttape_ has quit IRC08:50
*** atom_ has joined #puppet-openstack08:55
*** dmburmistrov has joined #puppet-openstack08:56
*** jaosorior is now known as jaosorior_brb09:11
*** markvoelker has joined #puppet-openstack09:17
*** jaosorior_brb has quit IRC09:19
*** markvoelker has quit IRC09:21
*** atom_ has quit IRC09:22
*** atom_ has joined #puppet-openstack09:35
*** ducttape_ has joined #puppet-openstack09:46
*** ducttape_ has quit IRC09:51
openstackgerritDaniel Pawlik proposed openstack/puppet-watcher: Added planer sections  https://review.openstack.org/34178809:56
openstackgerritMerged openstack/puppet-cinder: Add key manager parameters  https://review.openstack.org/34237909:58
*** aernhart has joined #puppet-openstack10:03
*** markvoelker has joined #puppet-openstack10:11
*** markvoelker has quit IRC10:16
*** _degorenko|afk is now known as degorenko10:21
*** rcernin has quit IRC10:26
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: openstack-core resource does not have interleave=true  https://review.openstack.org/34272410:31
*** aernhart has quit IRC10:36
*** johnmilton has joined #puppet-openstack10:38
*** johnmilton has quit IRC10:40
*** johnmilton has joined #puppet-openstack10:40
*** johnmilton has quit IRC10:42
*** johnmilton has joined #puppet-openstack10:42
*** johnmilton has quit IRC10:43
*** johnmilton has joined #puppet-openstack10:44
*** rcernin has joined #puppet-openstack10:45
*** johnmilton has quit IRC10:46
*** rcernin is now known as rcernin|lunch10:46
*** johnmilton has joined #puppet-openstack10:47
*** johnmilton has quit IRC10:47
*** johnmilton has joined #puppet-openstack10:47
*** johnmilton has quit IRC10:51
*** johnmilton has joined #puppet-openstack10:51
*** aernhart has joined #puppet-openstack11:04
*** markvoelker has joined #puppet-openstack11:05
*** markvoelker has quit IRC11:10
*** leanderthal|afk is now known as leanderthal|appt11:13
*** dtantsur is now known as dtantsur|brb11:20
*** cdelatte has joined #puppet-openstack11:22
*** shardy is now known as shardy_lunch11:28
openstackgerritSaravanan KR proposed openstack/puppet-vswitch: OVS DPDK package support with basic options  https://review.openstack.org/34036711:34
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add ensure parameter to service identity resource  https://review.openstack.org/34275411:42
*** alee_ has joined #puppet-openstack11:44
*** ducttape_ has joined #puppet-openstack11:48
*** derekh has quit IRC11:49
*** ducttape_ has quit IRC11:52
*** dtantsur|brb is now known as dtantsur11:55
*** colonwq has quit IRC11:56
EmilienMhi11:58
*** markvoelker has joined #puppet-openstack11:59
*** markvoelker has quit IRC12:03
*** shardy_lunch is now known as shardy12:04
*** dprince has joined #puppet-openstack12:04
*** jayg|g0n3 is now known as jayg12:04
*** alee_ has quit IRC12:06
*** colonwq has joined #puppet-openstack12:07
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo: openstack-core resource does not have interleave=true  https://review.openstack.org/34272412:08
*** ducttape_ has joined #puppet-openstack12:12
EmilienMiurygregory: hey12:15
EmilienMiurygregory: do you have plans to add authtoken thing in puppet-cinder?12:16
mjblackEmilienM: not sure where you're seeing 'paramter'12:20
EmilienMmjblack: the file name12:20
mjblackah ok12:20
mjblackyeah sleep deprivation :D12:21
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add ensure parameter to service identity resource  https://review.openstack.org/34275412:21
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: PoC Validation - Make it fail (test)  https://review.openstack.org/34220512:22
iurygregoryEmilienM, i have plans for all modules =P you said that we should merge one before start the work in all other modules .-.12:25
iurygregorymorning12:25
iurygregoryXD12:25
EmilienMiurygregory: do you need help to make quick progress or you got it covered?12:27
iurygregorynp ;) i'll pray to not break fuel i think XD12:27
EmilienMiurygregory: I will probably push the cinder thing today or so, except if you have it in progress already12:28
iurygregoryEmilienM, i have in neutron12:28
iurygregorygo ahead12:28
iurygregoryo/12:28
EmilienMthis is a blocker for us, we would like to test barbican and it needs cinder.conf with v3 parameters12:28
iurygregoryEmilienM, np, put me in the review o/12:29
*** trown|outtypewww is now known as trown12:29
*** alee has joined #puppet-openstack12:32
openstackgerritSaravanan KR proposed openstack/puppet-vswitch: OVS DPDK package support with basic options  https://review.openstack.org/34036712:33
openstackgerritMatthew J Black proposed openstack/puppet-openstack-cookiecutter: Added ensure parameter to pass to service identity  https://review.openstack.org/34276812:34
*** ducttape_ has quit IRC12:37
*** imcsk8 has quit IRC12:44
*** imcsk8 has joined #puppet-openstack12:44
degorenkoEmilienM, 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 problem12:45
degorenkoas one possible solution for ubuntu - we have to specify bind_address12:45
degorenkothen we have proper redirect12:45
EmilienMok cool12:45
degorenkobut this we have one more problem12:45
degorenkoall wsgi manifests are including base apache configuration12:46
degorenkowhcih create base listen port to 8012:46
degorenkothen when we are trying to create listen port 80 on some address - we have conflict12:46
*** alee has quit IRC12:47
degorenkowe can set add_listen to false, but then will not be created 443 port for ssl from horizon :D12:47
EmilienMcan you just create a "if $redhat { class horizon blabla } elsif $ubuntu { class horizon do what you want }12:47
degorenkoyeees, but the problem is that we can set bind_address for horizon at all12:47
degorenkowith 80 port12:47
degorenkobecause it is already created by apache itself12:47
degorenkocant*12:47
degorenkomwhahaha, ^12:48
*** ducttape_ has joined #puppet-openstack12:48
degorenkoin fuel we just passing add_listen false every time, and we have different implementation for apache - it already includes all needed ports12:48
EmilienMdo you use puppet-horizon?12:49
degorenkoyes, we use12:49
degorenkoagain - we are passing to puppet-horizon::wsgi add_listen false12:49
degorenkothat means that puppet-horizon will not manage apache ports12:49
degorenkoand we have different call for apache for managing ports12:50
degorenkoso, it looks like bug in puppet-horizon module12:50
degorenkoi can upload possible solution here12:50
degorenkobut it should be discussed12:50
EmilienMI'm going to propose a patch that enable testing on centos only now12:51
EmilienMand I'll let you figure for ubuntu a solution12:51
degorenkofor me, question number 1) - why apache on ubuntu won't redirect by default to horizon12:51
degorenkoalso we have different apache version on centos and ubuntu12:51
degorenko2.4.18 on ubuntu and 2.4.6 on centos12:52
degorenkoprobably something changed in apache12:52
mjblackdegorenko: where are you setting add_listen to false?12:52
degorenkoin horizon12:52
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: install tempest-horizon from source  https://review.openstack.org/34095912:53
degorenkovia extra_params12:53
mjblackoh12:53
degorenkoso we don't manage in fuel apache ports from puppet-horizon12:53
degorenkowe do it in apache class12:53
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: install tempest-horizon from source  https://review.openstack.org/34095912:54
EmilienMdegorenko: ^ now, we'll only test centos12:54
EmilienMuntil you propose something for ubuntu12:54
degorenkomjblack, any ideas why apache 2.4.18 won't redirect to horizon by default? :)12:55
mjblackon ubuntu? removed the default vhost?12:55
degorenkomjblack, it's present12:55
mjblackthat could be why12:55
degorenkobut it's present also on centos12:55
degorenko15-default.conf  15-horizon_ssl_vhost.conf  15-horizon_vhost.conf  openstack-dashboard.conf12:56
degorenko15-default.conf  15-horizon_vhost.conf12:56
degorenko1) centos, 2) ubuntu12:56
degorenkoon both OS its present12:56
mjblackgot a link to a review where this is happening?12:56
degorenkomjblack, if i'll specify interface address for vhost horizon - it works on both12:56
degorenkomjblack, patch from Emilien above12:57
degorenko5 patch set for example12:57
degorenkomjblack, 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#L42612:58
degorenkoi can upload change with collectors, for overriding base 80 port to interface specific port, if bind_address is enabled12:59
degorenkobut it will not help, when we have add_listen = false, from extra_params13:00
mjblackso13:00
mjblackon13:00
mjblackthe way apache works13:00
*** ducttape_ has quit IRC13:00
degorenkoyes, apache works13:00
mjblackeven though you have the same priority for for the conf file13:00
degorenkohave i can check, that priority same?13:01
mjblackfor namebasedvirtualhosts, it will default I believe to the servername that comes alphabetically first13:01
mjblackso in the case of ubuntu13:02
degorenkothen they have same priority13:02
mjblackservername starts with xenial13:02
*** alee_ has joined #puppet-openstack13:02
mjblackbut on centos it starts with centos13:02
degorenkoyes, but configs are same13:02
mjblackright13:02
degorenkohow can fqdn affect it?13:02
mjblackit doesnt know which one to use13:02
mjblackso it resorts back to servername I believe, which ever hits first alphabetically13:03
*** markvoelker has joined #puppet-openstack13:03
degorenkobut both configs use same servername13:04
mjblackright13:04
mjblackbut if you look at the servername for centos and ubuntu13:04
mjblackfor horizon13:05
mjblackcentos servername is starts with "centos"13:05
mjblackand ubuntu starts with "xenial"13:05
mjblackc > d13:05
mjblackd > x13:05
degorenkoyes13:06
degorenkoand default servername is default13:06
mjblackright13:06
degorenkobut13:06
mjblackbut the horizon vhost13:06
mjblackservername is based off distro13:06
mjblackcentos -> 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.gz13:06
degorenkomy server name for xenial and centos starts with sk_....13:06
degorenkoso i have sk_xenail13:06
degorenkoand sk_centos13:06
mjblackubuntu -> 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.gz13:06
degorenkoand in both cases default should be used first13:07
degorenkobut i have working centos and non-working ubuntu13:07
degorenkohow you can explain this? :)13:07
mjblackI'd need to see it, you told me to look at that review from EmilienM :)13:08
degorenkothat's not on review13: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#L10413:08
degorenkothat's on my testing vms13:08
degorenkoi can share config files13:08
degorenkomjblack, so here is ubuntu http://paste.openstack.org/show/533056/13:09
degorenkomjblack, centos http://paste.openstack.org/show/533057/13:09
mjblackcentos config isnt using wildcard13:10
mjblackline 3713:10
degorenkooh wait13:10
degorenkofor centos not current configuration13:11
degorenkolet me sec13:11
mjblackdegorenko: what else is being configured under conf.d?13:12
degorenkohttp://paste.openstack.org/show/533058/13:12
mjblackfor centos13:12
EmilienMalee_: I figured /me working on puppet-cinder13:13
alee_EmilienM, ok13:13
degorenkomjblack, they have a bit difference, centos is under ssl13:14
degorenkoand under ipv6 also13:15
degorenkobut for horizon there are no ipv6 settings13:15
mjblackyeah but if you're being redirected from http to https then its skipping default vhost13:15
mjblackdegorenko: what I'm thinking is maybe on centos an apache config place by the package might not be getting purged13:18
degorenkomjblack, looks like you're right about redirection13:20
degorenkowhen i disable ssl on centos13:20
degorenkoi got same problem as on ubuntu13:20
degorenkoso we need to fix puppet-horizon, to work with bind_address13:21
degorenkomjblack, did you read my comments above about conflicting base horizon 80 with default apache?13:22
mjblackwhere it tried to create two of the same port settings?13:23
degorenkomjblack, 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 now13:23
*** ducttape_ has joined #puppet-openstack13:23
degorenkomjblack, 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#L42913:24
degorenkothen when we are trying to do same for base horizon port - we have error from 426 line13:24
alee_EmilienM, the code we're using is basically whats in master, right?  or is it a branch?13:24
mjblackdegorenko: but thats only if you use default vhost13:25
degorenkomjblack, which is enabled by default from base apache class.....13:25
mjblackyeah but there is a great way to solve it :)13:25
degorenkohiera i know13:25
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#L4913:26
degorenkoEmilienM, try to rebase your change on https://review.openstack.org/#/c/341775/5 - with enabled ubuntu13:28
degorenkoit should work13:28
mjblackdegorenko: puppet-horizon is doing "include", cant you do also class { '::apache': default_vhost => false }?13:30
degorenkomjblack, as another option yes13:30
degorenkobut hiera solution looks better13:30
degorenkofor me, imho :)13:30
mjblackdegorenko: heh, wont get an argument there from me13:31
mjblacktrying to do as much as possible through hiera instead of custom manifests13:31
degorenkomjblack, so, in fuel we have exactly solution witohut hiera :D13:31
mwhahahalively discussions this morning13:32
* mjblack isnt alive 13:32
mwhahahayes i know you're dead inside like the rest of us13:32
iurygregoryyeah =(13:33
mwhahahaanyway so i can update my patch set to also include osfamily specific hiera so we could just default_vhost => false for ubuntu only13:33
mjblackmwhahaha: I'm sustained right now by enough energy drinks to kill an army13:33
degorenkomwhahaha, no, your patch is ok for both OS13:33
mwhahahaoh is it?13:33
degorenkocentos is working because of ssl :D13:33
mjblackdegorenko: maybe so but I'd like to see distro and distro-major_ver in the hiera.yaml too :D13:34
mwhahahaoh it was failing yesterday, i hadn't gotten to the recheck results yet13:34
*** ducttape_ has quit IRC13:37
EmilienMalee_: master13:38
alee_EmilienM, ok - I'll work a bit on the nova side in the meantime then13:39
alee_EmilienM, once we resolve cinder, we'll hit nova side issues next13:39
EmilienMok13:39
EmilienMnot sure13:40
EmilienMit's deprecated13:40
EmilienMnot removed13:40
EmilienMI'm finish the patch on cinder now13:40
EmilienMerr, I'm finishing*13:40
*** karimb has joined #puppet-openstack13:40
*** richm has joined #puppet-openstack13:40
alee_EmilienM, well -- even for the old way, there needs to be a [barbican] section13:40
EmilienMdegorenko: it just sad to see you need hiera to make horizon working on ubuntu13:41
EmilienMalee_: ok13:41
EmilienMalee_: please update puppet nova for it13:41
degorenkoEmilienM, no :) mjblack solution also will work13:41
alee_I think .. checking with the guys who wrote this ..13:41
EmilienMalee_: can you also double check barbican options in cinder?13:41
alee_EmilienM, yes - doing that too.13:41
EmilienMdegorenko: I haven't followed all discussion.13:41
degorenkoEmilienM, we have at least 2 ways how we can enable horizon on ubuntu :D13:42
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 keymgr13:45
dmsimardEmilienM: uh, hey, why aren't there any trove logs on scenario003 jobs ?13:48
dmsimardhttp://logs.openstack.org/91/342391/2/check/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-xenial/72b23d1/logs/13:48
dmsimardhttp://logs.openstack.org/91/342391/2/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/e97f5ba/logs/13:48
EmilienMdmsimard: https://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario003.pp#L3513:48
dmsimardit's not even installed ?13:49
EmilienMjust read the code13:49
dmsimardah, damnit13:49
EmilienMyeah, #disableallthethings13:49
dmsimardYou should put "disabled" in the test matrix in the readme then :p13:49
EmilienMright13:50
dmsimardI was going to see if I could put memcached_tokens for the projects that are missing it in p-o-i13:50
dmsimardI guess we can merge that as is and then address the deprecations later on13:50
dmsimardI'll fix the swift thing13:50
degorenkomwhahaha, is this patch ready? https://review.openstack.org/#/c/341775/13:52
mwhahahaprobably now, let me pull the  WIP from the commit message13:52
degorenkomwhahaha, just suggestion13:52
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/34177513:52
degorenkomay be better to set hiera config via parameter?13:52
degorenkonot just hardcoded puppet args?13:52
degorenkoin *13:53
mwhahahayea we can do that13:53
degorenkoso, you need to update patch again :D13:53
mwhahahai'll update it today and maybe throw in OS specific hiera items13:53
*** atom_ has quit IRC13:53
degorenkomwhahaha, ok, ping me, when it will be ready :)13:53
*** ducttape_ has joined #puppet-openstack13:55
openstackgerritDavid Moreau Simard proposed openstack/puppet-swift: Fix comment typo  https://review.openstack.org/34287113:55
dmsimardvinsh: hey o/13:56
dmsimardvinsh: 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#L213:57
dmsimardThe 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:57
*** shardy is now known as shardy_mtg13:58
openstackgerritSaravanan KR proposed openstack/puppet-vswitch: OVS DPDK package support with basic options  https://review.openstack.org/34036713:59
EmilienMiurygregory: by looking at your patch on puppet-nova, I found some things I would not have done14:08
EmilienMI'll finish my cinder patch and let you a review14:08
iurygregoryEmilienM, ack ;)14:08
EmilienMso we can compare the 2 approach14:08
iurygregorysure ^^14:08
*** atom_ has joined #puppet-openstack14:09
EmilienMiurygregory: so we don't need identity_uri and auth_uri anymore? I'm confused14:10
EmilienMI don't see them in manifests/resource/authtoken.pp14:10
iurygregoryEmilienM, in authtoken.pp we have added all options according to configuration from keystonemiddleware, the identity_uri are in examples only14:12
iurygregorywe don't need to set both i think14:12
EmilienMeverything authtoken should go in this define14:12
EmilienMand we should not let cinder_config { authtoken } thing in our manifests14:13
EmilienMonly use the define14:13
*** markvoelker has quit IRC14:13
iurygregoryin nova we have the 'admin_password' 'admin_user'14:13
iurygregorybut keystone i haven't found reference for this14:13
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: In-process token caching is deprecated, use memcached instead  https://review.openstack.org/34239114:14
*** markvoelker has joined #puppet-openstack14:16
*** markvoelker has quit IRC14:16
*** xb_ has quit IRC14:19
-openstackstatus- NOTICE: Gerrit is restarting to correct memory/performance issues.14:19
*** alee_ is now known as alee_afk14:21
*** pma has quit IRC14:22
*** jaosorior has joined #puppet-openstack14:28
*** alee_afk has quit IRC14:29
vinshdmsimard: Hi, memcache servers.. yeah.  in proxy.conf14:29
vinshBut, you need to add the memcached module to a node also.. to provide the actual service14:30
dmsimardvinsh: I guess it's only in swify proxy cache middleware manifest, I thought it was elsewhere.14:30
dmsimardYup.14:30
vinshMost all other services.. expirer.. reconcilator run on the proxy and can leverage that config too14:31
*** karthiks has quit IRC14:33
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: authtoken: allow memcached_servers to be empty or os_service_default  https://review.openstack.org/34289514:36
EmilienMiurygregory: ^14:36
EmilienMwe need to do the memcache_servers join thing here ^14:36
EmilienMand not in modules14:36
*** markvoelker has joined #puppet-openstack14:37
iurygregoryEmilienM, oh cool :D14:38
openstackgerritMerged openstack/puppet-tripleo: Add resource for requesting certificates for HAProxy  https://review.openstack.org/34140514:38
EmilienMthis authtoken thing is a lot of work14:41
*** sanjayu is now known as sanjayu|AFK14:42
*** shardy_mtg is now known as shardy14:42
iurygregoryit's a little crazy14:42
* iurygregory hides14:42
*** saneax is now known as saneax_AFK14:42
*** llua has joined #puppet-openstack14:42
*** karthiks has joined #puppet-openstack14:43
iurygregoryEmilienM, 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:43
*** jaosorior_ has joined #puppet-openstack14:47
*** jaosorior has quit IRC14:47
*** jaosorior has joined #puppet-openstack14:48
openstackgerritBob Fournier proposed openstack/puppet-neutron:    Remove unused enable_tunneling in openvswitch_agent.ini  https://review.openstack.org/34246714:48
*** ducttape_ has quit IRC14:51
*** jaosorior_ has quit IRC14:52
*** atom_ has quit IRC14:53
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: WIP - Configure keystone authtoken options  https://review.openstack.org/34290514:54
EmilienMiurygregory: early patch (wip) ^14:54
EmilienMit should be backward compatible14:55
*** xarses has joined #puppet-openstack14:59
openstackgerritMerged openstack/puppet-tripleo: Add MySQL profiles  https://review.openstack.org/34044415:00
*** karimb has quit IRC15:06
iurygregoryEmilienM, ack15:10
*** johnmilton has quit IRC15:21
openstackgerritMerged openstack/puppet-openstack-integration: install tempest-horizon from source  https://review.openstack.org/34095915:26
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/34290515:36
EmilienMiurygregory: can you review it? ^15:36
iurygregoryback from lunch o/15:37
iurygregoryi'll ;)15:37
dmsimardwth 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_79022615:38
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/34290515:39
EmilienMjust release note update ^15:39
*** fvollero has joined #puppet-openstack15:40
Hunnerbeaker bugfix released15:40
EmilienMdmsimard: I saw it already15:40
EmilienMHunner: yeah? wow15:40
EmilienMlet's try15:40
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: Revert "Pin beaker to 2.46.0"  https://review.openstack.org/34293115:41
EmilienMmwhahaha: fyi ^15:41
Hunnerchangelog https://github.com/puppetlabs/beaker/blob/2.47.1/HISTORY.md#LATEST15:41
mwhahahacool15:41
EmilienMcolonwq: any progress on rgw?15:46
dmsimardEmilienM: eh, swift has it's own ipv6 parsing thing15:47
dmsimardthe inet6 format expected by python-memcache doesn't work for swift15:47
EmilienMdamn15:47
dmsimardhttp://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_1115:47
dmsimardI'll fix it15:48
EmilienMin swift?15:48
EmilienMswift doesn't work with ipv6? lol15:48
dmsimardit works15:48
dmsimardjust the parsing is different15:48
dmsimardit detects that it's an ipv615:48
iurygregoryEmilienM, looks good but i think you need to update puppet-keystone or CI will not pass :D and need add tests for new parameters15:48
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980615:50
EmilienMiurygregory: ok15:52
openstackgerritDenis Egorenko proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/34177515:57
openstackgerritDenis Egorenko proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/34293515:57
*** foxhunt_away is now known as foxhunt7215:57
*** foxhunt72 is now known as foxhunt_away16:00
*** ifarkas has quit IRC16:03
openstackgerritMerged openstack/puppet-openstacklib: service validation: log output on failures  https://review.openstack.org/34227916:04
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: authtoken: allow memcached_servers to be empty or os_service_default  https://review.openstack.org/34289516:15
*** shardy has quit IRC16:15
*** tesseract- has quit IRC16:16
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/34290516:18
*** rcernin|lunch has quit IRC16:18
*** karimb has joined #puppet-openstack16:18
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: In-process token caching is deprecated, use memcached instead  https://review.openstack.org/34239116:27
*** cdelatte has quit IRC16:29
*** ducttape_ has joined #puppet-openstack16:36
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Bump RDO repositories  https://review.openstack.org/34295116:42
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/33786016:45
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/33786016:45
degorenkoEmilienM, https://review.openstack.org/#/c/342935/ - ubuntu horizon tests passed16:48
degorenkodepends on https://review.openstack.org/#/c/341775/716:48
*** dtantsur is now known as dtantsur|afk16:49
*** trown is now known as trown|lunch16:59
tdb_do the 8.x mitaka openstack modules work on xenial?17:11
*** tdb_ is now known as tdb17:11
iurygregorytdb, to me mitaka is in trusty (14.04)17:12
iurygregoryxenial is for newton and after i think =)17:12
tdbfrom an Ubuntu point of view, 16.04 was release with mitaka17:13
iurygregoryO.o you can try i think XD17:13
tdbmaybe from a puppet module point of view it's as you say17:13
*** degorenko is now known as _degorenko|afk17:13
iurygregoryyeah =)17:13
tdbwhich is I guess what my question was :)17:13
EmilienM_degorenko|afk: ok so we need hiera17:14
EmilienMtdb: right, in theory you can deploy Mitaka on Xenial, and our modules should work (or almost) but we officialy started to support Xenial from newton17:15
tdbright, ok. a little annoying, but I understand17:16
tdbI wanted to use the latest released openstack with the latest ubuntu17:17
tdbI guess there's nothing much wrong with 14.04 until newton is out17:17
openstackgerritBrent Eagles proposed openstack/puppet-octavia: Implement worker.pp  https://review.openstack.org/32849517:21
EmilienMtdb: I think you can deploy mitaka on xenial with stable/mitaka17:31
EmilienMgive it a try and let us know17:31
tdbI did; it failed with upstart/systemd issue17:32
EmilienMah17:32
tdbwhich is why I started checking if it was supported before I went to far with it17:32
EmilienMso use master to deploy mitaka17:32
EmilienMit should work too I think17:32
tdboh, I wasn't sure if it did17:32
EmilienMI think it should17:34
EmilienMlet us know if not17:34
tdbI'll give it a whirl17:34
EmilienMyeah mitaka & xenial were released in the same time, it was really tricky for us to test it together17:37
EmilienMso we decided we would start xenial support during newton cycle17:37
EmilienMthing we did and finished!17:37
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/34290517:39
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980617:39
EmilienMmwhahaha: is https://review.openstack.org/#/c/341775/ ready ?17:53
*** ducttape_ has quit IRC17:53
*** rcernin has joined #puppet-openstack17:59
*** johnmilton has joined #puppet-openstack18:00
mwhahahayea should be18:03
*** leanderthal|appt is now known as leanderthal|afk18:03
EmilienMdmsimard: why designate patch fails on tempest about trove?18:04
iurygregoryLOL18:04
EmilienMmwhahaha: +218:05
*** jaosorior has quit IRC18:05
*** johnmilton has quit IRC18:05
mwhahahaEmilienM: oh i was adding in configurability in hiera i guess i forgot to push it this morning18:06
EmilienMmwhahaha: cool I'll +2 again18:06
dmsimardEmilienM: what ?18:07
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/34177518:09
EmilienMdmsimard: see logs18:11
dmsimardEmilienM: I don't know, I'm focused on making designate work18:11
dmsimardthere's some mistral stuff too18:12
EmilienMyeah18:12
EmilienMI'm digging18:12
dmsimardkeystone endpoint not found errors again18:12
EmilienM2016-07-15 17:23:13.478646 | {1} designate_tempest_plugin.tests.api.v2.test_zones.ZonesTest.test_delete_zone [0.209007s] ... FAILED18:12
dmsimardI've seen that before never found what it was18:12
EmilienManyway designate fails too18:12
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/34293518:12
dmsimardyeah I know but we're making progress, no more stack traces18:12
dmsimardbut designate keystone authentication is failing now18:13
dmsimardhttp://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_97118:13
dmsimardshouldn't that be ipv6 ?18:13
EmilienMhttp://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_97518:13
EmilienMit's doing http and ipv418:13
dmsimardohhhh18:13
dmsimardI forgot to configure it18:13
dmsimardI took exceprts from beaker tests18:14
*** rwsu has joined #puppet-openstack18:16
*** johnmilton has joined #puppet-openstack18:17
dmsimardEmilienM: nooooooooooo18:18
dmsimardhttp://nooooooooooooooo.com/18:18
iurygregoryLOL18:18
EmilienMdmsimard: https://cdn.meme.am/instances/39036008.jpg18:18
dmsimardI never thought I'd see split keystone parameters like these ever https://github.com/openstack/puppet-designate/blob/master/manifests/api.pp#L85-L8718:19
iurygregoryEmilienM, LOL18:19
mwhahahasome old code right there18:19
dmsimardI still remember mgagne trying to kill all of them18:19
mwhahahaneeds to be updated18:19
dmsimardfrom a past file18:19
dmsimardlife*18:19
*** ducttape_ has joined #puppet-openstack18:19
mgagne:D18:19
EmilienMdmsimard: you'll need something like https://review.openstack.org/34290518:19
iurygregoryWe don't need for authtoken18:20
dmsimardEmilienM: I'll need something like nothing because that'll be the way it's going to work for the time being :p18:20
*** trown|lunch is now known as trown18:20
EmilienM:-O18:20
dmsimardthere are several fixes to do to puppet-designate already18:20
dmsimardI should be filing bugs18:20
dmsimardlots of bad defaults amongst other things18:20
mwhahahathe land of unloved puppet modules18:20
EmilienMyep18:20
dmsimardmwhahaha: it doesn't beat swift18:21
EmilienMwe have 30 modules a very few active contributors18:21
EmilienMit's quite challenging18:21
mwhahahano one likes swift, that ones been intentional :D18:21
iurygregoryi like swift .-.18:21
mgagnepuppet-swift =)18:21
iurygregoryfrom the user perspective =P18:21
dmsimard2014 damn https://bugs.launchpad.net/puppet-nova/+bug/127497918:22
openstackLaunchpad 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
mwhahahait said puppet-designate was fixed18:22
mwhahahasuch lies18:22
dmsimardprobably in keystone auth or some other place18:23
dmsimardand missed ::api18:23
iurygregory:O18:23
iurygregorygit blame =P18:23
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/33786018:27
dmsimardEmilienM: ^ has a better chance of working18:27
dmsimardEmilienM: do you mind if I configure puppet-swift to log to /var/log/swift in p-o-i ? It sucks to search through journalctl/syslog18:30
EmilienMgo ahead18:30
EmilienMI always wished someone would do it18:30
mwhahahano you may not do useful work18:30
mwhahahaonly meaningless, unhelpful reviews will be accepted going forward18:30
*** karimb has quit IRC18:31
dmsimardWould you put it in swift.pp ? Or, say, in swift_logs.pp or right in the scenario fixture ?18:33
EmilienMiurygregory: http://git.openstack.org/cgit/openstack/keystonemiddleware/tree/keystonemiddleware/auth_token/_auth.py#n4618:33
EmilienMidentity_uri is still required18:33
EmilienMI'm so confused18:34
dmsimardThe way I did it back at $oldjob was through rsyslog18:34
EmilienMI'm going to ask steve18:34
iurygregoryEmilienM,  can i cry?18:34
danpawlikEmilienM: :(18:34
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/34290518:40
EmilienMlet's try again with auth_type = password18:40
iurygregorygood to know that identity_uri is deprecated XD18:40
EmilienMyeah18:46
EmilienMit's *at least*  one good news :P18:46
iurygregoryyep18:46
EmilienMiurygregory: you see how I did https://review.openstack.org/#/c/342905/ ?18:46
EmilienMit should be backward compatible and transparent18:47
iurygregoryyeah ;) i i'll update nova to be like your patch :D18:47
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/34303118:48
iurygregorysorry for the trouble =)18:48
openstackgerritMerged openstack/puppet-openstack-integration: Bump RDO repositories  https://review.openstack.org/34295118:48
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/34303118:49
openstackgerritBrent Eagles proposed openstack/puppet-octavia: Implement worker.pp  https://review.openstack.org/32849518:49
openstackgerritMerged openstack/puppet-openstack_spec_helper: Revert "Pin beaker to 2.46.0"  https://review.openstack.org/34293118:51
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/34303118:53
bkerodmsimard: is that File['/var/log/swift'] resource going to conflict with puppet-swift?18:55
dmsimardbkero: puppet-swift doesn't manage /var/log/swift18:55
dmsimardor it'd be caught in the logs (but remain empty)18:55
dmsimardhttp://logs.openstack.org/91/342391/4/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/3e7ce6b/logs/ no swift directory18:55
bkeroHuh, yeah.18:56
bkero[bkero@Whitefall puppet-swift]$ grep -R "/var/log/swift" *18:56
bkero[bkero@Whitefall puppet-swift]$18:56
dmsimardthe directories are mostly left up to packaging now18:57
dmsimardpossible it was managed before18:57
dmsimardswift is "special"18:57
bkeroHmm18:57
bkeroThere is a bunch of logging stuff in manifests/proxy.pp18:58
bkerolog_headers, log udp port/host, address, etc18:58
bkeroincluding references to /dev/log O__o18:59
* bkero whistles quietly while looking at https://blueprints.launchpad.net/swift/+spec/swift-oslo-logging19:00
dmsimardthe default log facility LOG_LOCAL1 will work19:00
dmsimardhttps://review.openstack.org/#/c/343031/3/files/rsyslog-swift.conf local119:00
bkeroyup19:00
* bkero was thinking of ways that involved writing less rsyslog config files :P19:00
dmsimardWell, RDO could embed defaults into packaging19:01
dmsimardbut that'd be cheating19:01
dmsimardI don't remember what UCA does19:01
dmsimardI guess it leaves the same defaults to syslog19:01
dmsimardyeah, also syslog19:02
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Revert "Pin beaker to 2.46.0"  https://review.openstack.org/34303619:02
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Revert "Pin beaker to 2.46.0"  https://review.openstack.org/34303719:02
dmsimardI kind of cheated and took part of https://github.com/openstack/swift/blob/master/etc/swift-rsyslog.conf-sample19:03
dmsimardI guess I could leave it vanilla19:03
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/34177519:05
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/34293519:05
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/34303119:06
EmilienMdmsimard: why not doing it in puppet-swift ? ^19:08
EmilienMit's a nice feature19:08
dmsimardEmilienM: it's so weird that designate installation breaks mistral, aodh and trove19:08
EmilienMyeah19:08
EmilienMnext time when you go back from PTO, just read emails and continue your regular work19:09
dmsimardsorry for trying to fix things19:09
dmsimardEmilienM: re: logs19:09
dmsimardEmilienM: I guess I can put it in puppet-swift19:10
openstackgerritBrent Eagles proposed openstack/puppet-octavia: Adding health_worker.pp  https://review.openstack.org/34304419:14
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/34290519:16
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/33786019:18
*** xarses_ has joined #puppet-openstack19:22
*** xarses has quit IRC19:22
*** xarses_ has quit IRC19:23
*** xarses_ has joined #puppet-openstack19:23
*** xarses_ has quit IRC19:23
*** xarses_ has joined #puppet-openstack19:24
openstackgerritEmilien Macchi proposed openstack/puppet-nova: compute: update doc for keymgr api_class  https://review.openstack.org/34304719:27
*** xarses_ is now known as xarses19:37
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: Configure keystone authtoken options  https://review.openstack.org/34290519:54
*** aernhart has quit IRC19:58
iurygregorybye people have nice weekend o/20:04
*** alee_afk has joined #puppet-openstack20:04
*** alee_afk has quit IRC20:05
openstackgerritEmilien Macchi proposed openstack/puppet-nova: compute: update doc for keymgr api_class  https://review.openstack.org/34304720:05
EmilienMiurygregory: thanks, bye !20:06
mjblackiurygregory: you too20:09
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Add designate coverage to scenario003  https://review.openstack.org/33786020:25
dmsimardEmilienM, 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
dmsimardbut now EmilienM wants it in puppet-swift T_T20:26
EmilienMoh nice20:28
EmilienMyeah it would be so cool20:28
EmilienMthanks for doing it20:29
dmsimardthat'll cost you one more beer in barcelona20:29
*** aernhart has joined #puppet-openstack20:31
EmilienMok :)20:32
dmsimardhm, wait, it doesn't look like it works for Ubuntu20:33
dmsimardI think for Ubuntu the directory needs particular permissions20:33
dmsimardbut the job didn't fail ?20:33
EmilienMheh I managed to make cinder working with keystone v3 on https://review.openstack.org/#/c/342905/20:33
mwhahahait won't fail you just get no longs20:34
mwhahahayea /var/log/swift needs specific perms under ubuntu20:34
dmsimardmwhahaha: yeah but I mean the directory /var/log/swift apparently didn't ge5 created20:34
dmsimardIt'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_81098520:35
dmsimardBut 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
mwhahahaunder ubuntu it needs swift:adm for the dir20:35
mwhahahai believe20:35
mwhahahaor syslog:adm20:35
dmsimardyeah syslog:adm20:35
dmsimardhm, what'd be the clean way to do that ? if distro ubuntu spaceship and add owner/group ?20:36
mwhahahathere is no clean way for this20:38
mwhahahathis is why we don't manage users/groups/files via puppet ;)20:38
dmsimardhttp://paste.openstack.org/show/533098/ ?20:38
dmsimardmwhahaha: ^20:38
mwhahahayea that could work20:39
dmsimardI'll try20:39
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980620:42
EmilienMok https://review.openstack.org/#/c/342905/ is ready for review20:42
EmilienMthe CI proves it's backward compatible20:42
EmilienMI'll let reviewers vote20:42
mwhahahait'll break fuel20:43
mwhahaha:D20:43
mwhahahaor not we'll see20:43
EmilienMoh20:44
EmilienMyeah, I can -2 it if you want20:44
mwhahahanaw it's ok we'll wait20:44
EmilienMor feel free to do it if you see risk20:44
openstackgerritChris Jones proposed openstack/puppet-nova: WIP - DUMMY - Testing 338387  https://review.openstack.org/34306720:46
EmilienMmwhahaha: what ? https://review.openstack.org/#/c/342935/20:46
EmilienMwhere is code20:46
mwhahaha*magic*20:46
mwhahahaall i did was rebase :D20:46
mwhahahavia the UI20:46
mwhahahagerrit ate it, i'll republish20:47
EmilienMahah20:47
EmilienMit's the 1st time I see it20:47
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Add hiera support  https://review.openstack.org/34177520:49
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/34293520:49
mwhahahacause the parent was on an old version20:49
openstackgerritDavid Moreau Simard proposed openstack/puppet-swift: Add openstack::swift::rsyslog to configure rsyslog for Swift  https://review.openstack.org/34307020:49
mwhahahai had rebased mine via the UI, but then republished a local version which had a pre-horizon scenario003 parent20:50
*** dprince has quit IRC20:50
mwhahahaso the code got eaten via gerrit20:50
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/34303120:51
dmsimardEmilienM: ^ integration in puppet-swift and p-o-i depends-on20:51
EmilienMdmsimard: cool!20:52
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Enable Horizon tests on Ubuntu  https://review.openstack.org/34293520:52
openstackgerritDavid Moreau Simard proposed openstack/puppet-swift: Add openstack::swift::rsyslog to configure rsyslog for Swift  https://review.openstack.org/34307020:52
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/34303120:52
EmilienMdmsimard: dude you're going to hate me but we can't deploy rsyslog that way20:55
EmilienMif we do that, we break ton of deployments20:56
dmsimardEmilienM: it's an optional class20:56
EmilienMI can write the code20:56
vinshI just add rsyslog to my profiles for nodes on its own .. and swift pumps into that20:56
EmilienMyeah but I would rather do it in openstacklib20:56
EmilienMit's an helper thing20:56
dmsimardfeel free to take the patch :p20:56
dmsimardI was just tired of digging through syslog for p-o-i logs20:56
EmilienMyeah, if you don't do it, I'll probably take over but right now I'm already multi tasking20:57
dmsimardEmilienM: want me to leave it in p-o-i standalone for the time being then ?20:57
EmilienMno, keep your class now20:58
EmilienMI'll move it later20:58
EmilienMyour code is wrong anyway, class name is not good20:58
EmilienMdmsimard: let me finish another task and I'll doo it21:01
*** dgurtner has quit IRC21:06
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980621:14
EmilienMdmsimard: mhh.. ok let's do it in p-o-i now21:15
EmilienMI realize people probably handle rsyslog themselves21:16
EmilienMwe could think at a helper but later21:16
dmsimardEmilienM: ok21:17
dmsimardEmilienM: I guess I'll add it to beaker too ?21:18
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980621:18
EmilienMdmsimard: good idea21:18
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/34303121:19
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Setup logging to rsyslog for swift  https://review.openstack.org/34303121:20
EmilienMdmsimard: +221:20
dmsimardhang on, gotta see if it works for Ubuntu first21:20
dmsimardit didn't the first time around but I added the pew pew spaceship operator21:21
*** snarwade has quit IRC21:21
*** trown is now known as trown|outtypewww21:21
*** haplo37_ has joined #puppet-openstack21:21
EmilienMi'm out a bit, bbl21:21
openstackgerritBob Fournier proposed openstack/puppet-neutron:    Remove unused enable_tunneling in openvswitch_agent.ini  https://review.openstack.org/34246721:35
*** karimb has joined #puppet-openstack21:43
*** karimb has quit IRC21:49
*** haplo37_ has quit IRC21:52
EmilienMdmsimard: mistral tests are now running by default oO22:09
EmilienMnevermind, wrong log22:11
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980622:12
openstackgerritEmilien Macchi proposed openstack/puppet-nova: compute: add missing options for Barbican  https://review.openstack.org/34310222:12
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: enable Barbican and Volume Encryption testing  https://review.openstack.org/33980622:13
EmilienMmwhahaha: I'm trying to find what is breaking in Fuel22:17
mwhahahawe specify auth_version22:17
EmilienMah!22:17
mwhahahawe'll just have to coordinate next week22:17
EmilienMmwhahaha: can't we use depends-on?22:18
mwhahahanope22:18
mwhahahafuel-ci doesn't do depends-on :/22:18
EmilienMok22:18
EmilienMdo you want me to submit a patch in fuel?22:18
mwhahahawe could do the inverse, just no one else is around to help me merge22:18
EmilienMlet's do it on Monday22:19
EmilienMthere is no hurry22:19
mwhahahawe already have a review to remove the duplicated signing_dir thing22:19
mwhahahawe 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-L19422:19
EmilienMcool22:19
mwhahahai created a bug last week to get the ball rolling to support this effort22:20
mwhahahaso nova, glance and cinder i know had issues22:20
EmilienMmy patch in puppet-cinder is painful and we'll need it everywhere22:20
EmilienMwe'll probably use some help22:20
EmilienMif you know people won't don't know what to do :P22:20
EmilienMwho don't*22:20
mwhahahakeystone_authtoken config options are painful :D22:21
EmilienMyeah, but this work is really going to make it consistent across our modules22:21
mwhahahahttps://bugs.launchpad.net/fuel/+bug/160027622:21
openstackLaunchpad 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
mwhahahaindeed22:21
EmilienMso yeah it's a pain in the ass, but it's worth it22:21
EmilienMexcellent22:21
mwhahahaespecially since we don't expose many of the options22:22
EmilienMafter oslo things and keyste_authtoken, we start to be really good22:22
*** ducttape_ has quit IRC22:22
mjblackEmilienM: got maybe an easy question22:25
mjblackwhere is the CI job for cookiecutter?22:25
EmilienMmjblack: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L343-L36122:27
mwhahahain the CI config22:27
mwhahahai'll show myself out now22:27
EmilienMcheers22:27
EmilienMme too actually22:27
* mjblack replaces mwhahaha with hiera 22:27
mwhahahaand nothing of value was lost :D22:27
mjblackyou have a low opinion of yourself :D22:28
mjblackmwhahaha: reason I ask is it looks like cookiecutter is failing because of localization22:29
mwhahahayes i am aware22:30
mwhahahai poked openstack-infra but the missing locale stuff hasn't been fixed yet22:30
mwhahahai 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 one22:31
mwhahahabecause it does the same stuff as every other lint job22:31
mjblackits not lint though22:32
mwhahahabut it is22:32
mwhahahait's the thing that does the license check is loading a file with a non-ascii char22:32
mjblackumm but its not lint its the validate rake task22:33
mjblackwell I guess its still lint but a technicality22:33
mwhahahahttp://eavesdrop.openstack.org/irclogs/%23puppet-openstack/%23puppet-openstack.2016-07-11.log.html#t2016-07-11T18:19:2522:33
*** jayg is now known as jayg|g0n322:33
mwhahahayea but our lint task runs the validate stuff22:33
mwhahahaor rather our lint job does22:34
mwhahahaso on monday i went over to #openstack-infra which went into a very long conversation about where locale.conf comes from22:36
mwhahahaturns out it's magic and the image building process doesn't do it22:36
mwhahahacause of reasons22:37
mjblacklol22:37
mjblackwhere is the builders defined?22:37
mwhahahasystemd, chroot, pain22:37
mwhahahaall were discussed22:37
mjblacklint doesnt have tasks but builders22:37
mwhahahahttps://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml22:39
mwhahahaso we could try exporting LANG=en_US.utf8 before the cookiecutter job22:40
mwhahahanot sure if it'll fix it22:40
mwhahahaand it requires a job change :/22:40
mjblackso the reason it works for actual modules22:41
mjblackis its not using ruby json library to parse the json22:41
mjblackhttps://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml#L35022:41
*** danpawlik has quit IRC22:41
mjblackwhere cookiecutter is failing is at https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L36022:42
mwhahahahttps://review.openstack.org/34310622:42
mjblackheh22:43
mwhahahaactually do we not run spec validate against the regular modules?22:43
mjblacknope22:43
mwhahahaprobably why the do the same thing https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml#L373-L382 for the infra modules22:44
mjblackyeah22:44
mjblackso cookiecutter and actual modules do lint and spec but in cookiecutter validate is executed too22:45
mwhahahait 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 modules22:45
mjblackin actual module python is used to validate json22:45
mwhahahasomething for next week22:45
* mwhahaha wanders off22:45
mjblacklol ok22:45
openstackgerritMerged openstack/puppet-keystone: authtoken: allow memcached_servers to be empty or os_service_default  https://review.openstack.org/34289523:00
openstackgerritMerged openstack/puppet-keystone: Add ensure parameter to service identity resource  https://review.openstack.org/34275423:05
openstackgerritMerged openstack/puppet-swift: Fix comment typo  https://review.openstack.org/34287123:07
*** alee has joined #puppet-openstack23:10
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add ensure parameter to service identity resource  https://review.openstack.org/34311323:25
openstackgerritAde Lee proposed openstack/puppet-nova: Add missing parameters for Barbican key manager  https://review.openstack.org/34311423:30
*** rcernin has quit IRC23:30
*** haplo37_ has joined #puppet-openstack23:30
EmilienMalee: I already did it23:31
EmilienMalee: https://review.openstack.org/#/c/343102/23:31
aleeEmilienM, yeah - I just noticed that ..23:31
aleeEmilienM, mine has a couple things different23:31
aleeEmilienM, but is essentially the same -- need to have python-castellan installed23:32
EmilienMpython-castellan should be a dependency in cinder afik23:32
EmilienMit's already let me check23:32
EmilienMtempest is currently running, let's see if it works now23:33
aleeEmilienM,  well castellan integration for volume encryption is only there for nova right now23:33
aleeEmilienM, cinder has not made it yet - so I'd be surprised if it were a cinder prereq23:34
aleeEmilienM, anyways - gotta go to bed now ...23:35
EmilienMpython2-castellan-0.4.1-0.20160606180822.44233de.el7.centos.noarch23:36
EmilienMhttp://logs.openstack.org/06/339806/10/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/2b4228c/logs/rpm-qa.txt.gz23:36
EmilienMalready installed23:36
EmilienMalee: good night !23:36
aleecool23:37
aleeEmilienM, we're still getting 404 -- but I think thats because the auth_url needs a v3 on it23:46
*** xarses has quit IRC23:46
aleefor cinder ..23:46
*** hjensas has quit IRC23:47
aleeEmilienM, hmm .. looking ..23:47
aleeEmilienM, ah  perhaps I'm not looking at the current tempest run ..23:51
EmilienMhttp://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_58823:56
EmilienMalee: ^23:56
EmilienMyeah, let's see why it fails23:56
EmilienMalee: in barbican.pp ? auth_url?23:56
aleeEmilienM, no -- this is ok -- the 404 error was because it needed a v323:57
aleeEmilienM, this is an error coming from the barbican server23:58
aleeit doesn;t like the token presented23:58
EmilienMv3 where?23:58
EmilienMalee: ^23:59
« Thursday, 2016-07-14 Index Saturday, 2016-07-16 »

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