Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-06-08

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

All times shown according to UTC.

Time Nick Message
00:00 EmilienM yeah, we'll figure that later, EOD /me off
00:31 cwolferh joined #puppet-openstack
00:33 zhongshengping joined #puppet-openstack
00:34 * bkero waves
00:44 openstackgerrit Merged openstack/puppet-nova: Fix facts for policy test  https://review.openstack.org/326651
00:50 openstackgerrit zhongshengping proposed openstack/puppet-ec2api: Test multiple operating systems for policy  https://review.openstack.org/325387
00:51 openstackgerrit zhongshengping proposed openstack/puppet-ironic: Test multiple operating systems for policy  https://review.openstack.org/325393
00:53 openstackgerrit zhongshengping proposed openstack/puppet-mistral: Test multiple operating systems for policy  https://review.openstack.org/325399
00:54 openstackgerrit zhongshengping proposed openstack/puppet-octavia: Test multiple operating systems for policy  https://review.openstack.org/325403
01:13 markvoelker joined #puppet-openstack
01:48 ilbot3 joined #puppet-openstack
01:48 Topic for #puppet-openstack is now CI status: green - Puppet OpenStack modules - http://docs.openstack.org/developer/puppet-openstack-guide/
02:08 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Make neutron server/ml2 profiles match t-h-t  https://review.openstack.org/326119
02:09 openstackgerrit zhongshengping proposed openstack/puppet-cinder: Fix facts for policy test  https://review.openstack.org/326781
02:22 openstackgerrit zhongshengping proposed openstack/puppet-designate: Add purge_config option  https://review.openstack.org/326783
02:22 ducttape_ joined #puppet-openstack
02:31 openstackgerrit zhongshengping proposed openstack/puppet-designate: Add purge_config option  https://review.openstack.org/326783
02:38 openstackgerrit zhongshengping proposed openstack/puppet-gnocchi: Add purge_config option  https://review.openstack.org/326786
02:47 openstackgerrit zhongshengping proposed openstack/puppet-heat: Add purge_config option  https://review.openstack.org/326789
02:48 openstackgerrit joined #puppet-openstack
02:48 ducttape_ joined #puppet-openstack
02:53 openstackgerrit zhongshengping proposed openstack/puppet-ironic: Add purge_config option  https://review.openstack.org/326790
03:01 openstackgerrit zhongshengping proposed openstack/puppet-magnum: Add purge_config option  https://review.openstack.org/326793
03:07 openstackgerrit zhongshengping proposed openstack/puppet-manila: Add purge_config option  https://review.openstack.org/326796
03:12 openstackgerrit zhongshengping proposed openstack/puppet-mistral: Add purge_config option  https://review.openstack.org/326797
03:34 openstackgerrit zhongshengping proposed openstack/puppet-murano: dd purge_config option  https://review.openstack.org/326804
03:40 openstackgerrit zhongshengping proposed openstack/puppet-octavia: Add purge_config option  https://review.openstack.org/326805
03:42 furlongm joined #puppet-openstack
03:46 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Import ::nova class from THT  https://review.openstack.org/325982
03:48 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/325937
03:52 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/325937
03:53 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-nova: [WIP] Move nova to use OSDefaults  https://review.openstack.org/326221
03:54 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova Scheduler profile  https://review.openstack.org/326092
03:55 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova Consoleauth profile  https://review.openstack.org/326112
03:55 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova vncproxy profile  https://review.openstack.org/326130
03:59 openstackgerrit zhongshengping proposed openstack/puppet-rally: Add purge_config option  https://review.openstack.org/326812
04:00 openstackgerrit zhongshengping proposed openstack/puppet-murano: Add purge_config option  https://review.openstack.org/326804
04:00 openstackgerrit zhongshengping proposed openstack/puppet-mistral: Add purge_config option  https://review.openstack.org/326797
04:01 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Deploy nova::db classes in api profile  https://review.openstack.org/326027
04:05 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Deploy nova::db classes in api profile  https://review.openstack.org/326027
04:05 openstackgerrit zhongshengping proposed openstack/puppet-sahara: Add purge_config option  https://review.openstack.org/326815
04:12 openstackgerrit zhongshengping proposed openstack/puppet-rally: Add purge_config option  https://review.openstack.org/326812
04:25 ducttape_ joined #puppet-openstack
04:50 jaosorior joined #puppet-openstack
05:15 openstackgerrit zhongshengping proposed openstack/puppet-openstack-guide: Fix meetings 7th June  https://review.openstack.org/326824
05:18 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-glance: Provide consisent default name for service_identity resource  https://review.openstack.org/326826
05:28 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-glance: Provide consisent default name for service_identity resource  https://review.openstack.org/326826
05:33 fawadkhaliq joined #puppet-openstack
05:39 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-nova: Provide default service_name for keystone endpoint  https://review.openstack.org/326831
05:43 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-nova: Provide default service_name for keystone endpoint  https://review.openstack.org/326831
05:53 openstackgerrit zhongshengping proposed openstack/puppet-zaqar: Provide default service_name for keystone endpoint  https://review.openstack.org/326833
06:02 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-neutron: Provide default service_name for keystone endpoint  https://review.openstack.org/326836
06:03 v1k0d3n joined #puppet-openstack
06:06 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/326837
06:08 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-glance: Provide consisent default name for service_identity resource  https://review.openstack.org/326826
06:10 openstackgerrit zhongshengping proposed openstack/puppet-sahara: Provide default service_name for keystone endpoint  https://review.openstack.org/326840
06:12 rcernin joined #puppet-openstack
06:13 mflobo left #puppet-openstack
06:13 mflobo joined #puppet-openstack
06:16 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-nova: Provide default service_name for keystone endpoint  https://review.openstack.org/326831
06:21 dgurtner joined #puppet-openstack
06:21 dgurtner joined #puppet-openstack
06:25 openstackgerrit zhongshengping proposed openstack/puppet-octavia: Provide default service_name for keystone endpoint  https://review.openstack.org/326847
06:26 ducttape_ joined #puppet-openstack
06:30 openstackgerrit zhongshengping proposed openstack/puppet-sahara: Add purge_config option  https://review.openstack.org/326815
06:33 faizy joined #puppet-openstack
06:39 cwolferh joined #puppet-openstack
06:43 darkxploit joined #puppet-openstack
06:48 openstackgerrit joined #puppet-openstack
06:53 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-barbican: Change default service_name to 'barbican'  https://review.openstack.org/326860
06:54 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-barbican: Change default service_name to 'barbican'  https://review.openstack.org/326860
06:59 priteau joined #puppet-openstack
07:00 tesseract joined #puppet-openstack
07:18 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-barbican: Change default service_name to 'barbican'  https://review.openstack.org/326860
07:20 ifarkas joined #puppet-openstack
07:24 karimb joined #puppet-openstack
07:28 danpawlik joined #puppet-openstack
07:39 shardy joined #puppet-openstack
07:47 pepijn joined #puppet-openstack
07:48 openstackgerrit joined #puppet-openstack
08:00 chandankumar joined #puppet-openstack
08:05 jtomasek joined #puppet-openstack
08:09 fawadkhaliq joined #puppet-openstack
08:17 openstackgerrit Alexey Deryugin proposed openstack/puppet-nova: Add missing block_device_allocate_* params in nova.conf  https://review.openstack.org/290496
08:25 jaosorior joined #puppet-openstack
08:27 ducttape_ joined #puppet-openstack
08:37 dmk0202 joined #puppet-openstack
08:42 karimb joined #puppet-openstack
08:42 derekh joined #puppet-openstack
08:48 openstackgerrit joined #puppet-openstack
08:49 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add acceptance tests for client and application provider  https://review.openstack.org/246480
08:49 andreww joined #puppet-openstack
08:50 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add acceptance tests for client and application provider  https://review.openstack.org/246480
08:51 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add acceptance tests for client and application provider  https://review.openstack.org/246480
09:12 chandankumar joined #puppet-openstack
09:18 jtomasek joined #puppet-openstack
09:20 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/310725
09:22 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/310725
09:32 fawadkhaliq joined #puppet-openstack
09:33 fawadkhaliq joined #puppet-openstack
09:38 dgurtner joined #puppet-openstack
09:38 dgurtner joined #puppet-openstack
09:42 fawadkhaliq joined #puppet-openstack
09:53 markvoelker joined #puppet-openstack
09:55 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/310725
09:57 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP  https://review.openstack.org/310725
10:02 ansmith joined #puppet-openstack
10:18 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: Update keystone::auth service name parameter  https://review.openstack.org/326636
10:29 ducttape_ joined #puppet-openstack
10:49 openstackgerrit Merged openstack/puppet-murano: Add purge_config option  https://review.openstack.org/326804
10:51 tesseract joined #puppet-openstack
10:52 openstackgerrit Merged openstack/puppet-manila: Add purge_config option  https://review.openstack.org/326796
11:00 xb_ joined #puppet-openstack
11:07 openstackgerrit Daniel Pawlik proposed openstack/puppet-watcher: This commit adds the initial project files.  https://review.openstack.org/326593
11:15 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-keystone: Fix facts for policy test  https://review.openstack.org/326642
11:17 dgurtner joined #puppet-openstack
11:19 fawadkhaliq joined #puppet-openstack
11:23 chandankumar joined #puppet-openstack
11:28 openstackgerrit Denis Egorenko proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system  https://review.openstack.org/327025
11:33 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: WIP: Enable TLS in the internal network for keystone  https://review.openstack.org/327029
11:33 openstackgerrit Denis Egorenko proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system  https://review.openstack.org/327025
11:34 openstackgerrit Denis Egorenko proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system  https://review.openstack.org/327025
11:38 fawadkhaliq joined #puppet-openstack
11:38 ducttape_ joined #puppet-openstack
11:40 fawadkhaliq joined #puppet-openstack
11:42 moravec joined #puppet-openstack
11:42 morazi joined #puppet-openstack
11:43 fawadk joined #puppet-openstack
11:44 fawadkhaliq joined #puppet-openstack
11:49 iurygregory Morning guys =)
11:49 yolanda joined #puppet-openstack
11:49 iberezovskiy hi
11:50 johnmilton joined #puppet-openstack
11:51 ducttape_ joined #puppet-openstack
11:52 degorenko iurygregory, you wanted zeromq - https://review.openstack.org/#/c/327025/ - take it please :)
11:53 iurygregory degorenko, lol  I was just thinking if we should have =)
11:54 degorenko :)
11:54 iurygregory i'll take a look at my lunch time
11:54 iurygregory tks for start this man you rock o/
11:54 degorenko :D all of us
11:55 xarses joined #puppet-openstack
11:55 * iurygregory breakfast time
12:02 iurygregory EmilienM, not more one module now, but two =P [puppet] Request to create puppet-tacker
12:06 ifarkas joined #puppet-openstack
12:07 zigo How is configured Keystone in puppet-openstack? Is it using Apache?
12:08 zigo Since eventlet is removed, I'm about to modify the Keystone package, I'd like to do it in a puppet-openstack backward compat way.
12:08 zigo I'm doing a keystone-apache package which will automatically configure Apache.
12:08 zigo There wont be any daemon for Keystone anymore.
12:09 degorenko zigo, puppet-kestone can be configured with apache yes
12:09 iurygregory we have puppet-keystone working with apache =)
12:09 zigo degorenko: *can*, so it's an option?
12:09 degorenko zigo, for now yes, see https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L991
12:10 degorenko we deprecated eventlet in mitaka
12:10 degorenko and it will be removed soon at all
12:10 iurygregory yeah
12:12 markvoelker joined #puppet-openstack
12:14 moravec joined #puppet-openstack
12:15 dprince joined #puppet-openstack
12:22 fawadkhaliq joined #puppet-openstack
12:23 openstackgerrit Merged openstack/puppet-mistral: Add purge_config option  https://review.openstack.org/326797
12:26 openstackgerrit Merged openstack/puppet-ironic: Add purge_config option  https://review.openstack.org/326790
12:27 openstackgerrit Alexey Deryugin proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/250436
12:29 EmilienM good morning
12:29 EmilienM iurygregory: ack, will figure that today
12:30 openstackgerrit Merged openstack/puppet-heat: Add purge_config option  https://review.openstack.org/326789
12:30 EmilienM iberezovskiy: can you get https://review.openstack.org/#/c/326698/ please ?
12:31 iberezovskiy I like to align the stars :D
12:31 iberezovskiy done
12:33 openstackgerrit Merged openstack/puppet-sahara: Add purge_config option  https://review.openstack.org/326815
12:33 fawadkhaliq joined #puppet-openstack
12:43 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for keystone  https://review.openstack.org/327029
12:43 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/327069
12:44 ducttape_ joined #puppet-openstack
12:54 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Rename PUPPET_VERSION to PUPPET_MAJ_VERSION  https://review.openstack.org/326740
12:56 ansmith joined #puppet-openstack
13:03 mnaser its too early for this.. "Error while evaluating a Function Call, Duplicate declaration: Package[python-memcache] is already declared" .. anyone ran into this issue with puppet modules?  the module that's complaining about the duplicate declaration is puppet-horizon
13:03 mnaser however, i just grep'd all the modules and all puppet-memcached references seem to use ensure_packages
13:04 openstackgerrit Merged openstack/puppet-barbican: Change default service_name to 'barbican'  https://review.openstack.org/326860
13:06 iberezovskiy mnaser, could you please share your manifest?
13:06 iberezovskiy it's really strange because we have ensure_packages everywhere
13:07 mnaser iberezovskiy: for horizon specifically?  http://paste.openstack.org/show/508924/ .. it would be hard to share everything because its a huge repo
13:07 mnaser you do.. grep -ir python-memcache shows ensure_packages calls
13:08 mnaser the only 3 modules i have installed which reference python-memcache are oslo, horizon and keystone
13:11 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Add nodepool-xenial.yml file  https://review.openstack.org/320554
13:12 iberezovskiy mnaser, so you run only this manifest in the catalog (which you've shared) or smth else along with it?
13:12 mnaser iberezovskiy: there is much more (that's just one of the profile's attached to the controller node)
13:13 mnaser iberezovskiy: asked in #puppet and it seems like there was some bug that was resolved in 4.5.1 that might be the cause, trying to check
13:14 iberezovskiy joined to the channel
13:15 priteau joined #puppet-openstack
13:18 samuelBartel joined #puppet-openstack
13:20 richm joined #puppet-openstack
13:26 ayoung joined #puppet-openstack
13:44 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: Test horizon  https://review.openstack.org/327099
13:45 mnaser It seems the puppet oslo module is called "puppet-oslo" and the dependency references all make reference to "openstack-oslo"
13:45 mnaser is this a bug or something that is put in place on purpose until it's published on the forge?
13:46 EmilienM mnaser: all our modules are named openstack-*
13:46 mnaser EmilienM : https://github.com/openstack/puppet-oslo/blob/master/metadata.json -- "name": "puppet-oslo",
13:47 mnaser (trying to troubleshoot my issue, i noticed it becaue puppet module list --tree was complaining)
13:47 EmilienM that's a bug
13:47 mnaser would you like me to submit a quick review for it, EmilienM ?
13:47 openstackgerrit Emilien Macchi proposed openstack/puppet-oslo: metadata.json: fix module name  https://review.openstack.org/327103
13:48 mnaser too fast
13:48 EmilienM degorenko, iberezovskiy: can we get this one before ttx process the release? ^
13:49 iberezovskiy degorenko, land it please :)
13:50 degorenko already did it
13:52 degorenko colonwq, ping
13:56 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-neutron: Provide default service_name for keystone endpoint  https://review.openstack.org/326836
13:57 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Drop the neutron dnsmasq config file options  https://review.openstack.org/327109
14:00 dfisher joined #puppet-openstack
14:02 EmilienM iberezovskiy, degorenko: thx guys and thx mnaser for reporting it
14:03 iurygregory yeah, stars aligned \o/
14:04 EmilienM cool
14:04 iurygregory lol the openstack bot didn't put here XD
14:05 ducttape_ joined #puppet-openstack
14:05 degorenko _
14:05 degorenko O/
14:05 degorenko _/
14:05 iurygregory degorenko, you're dancing?
14:05 degorenko :D
14:05 degorenko yeah
14:08 openstackgerrit Alan Pevec proposed openstack/puppet-tempest: python-murano-tests is currently not subpackged  https://review.openstack.org/327114
14:08 yolanda_ joined #puppet-openstack
14:09 yolanda joined #puppet-openstack
14:12 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat  https://review.openstack.org/327069
14:14 mnaser alright i've narrowed it down to the keystone and horizon ensure_packages of python-memcached which are conflicting
14:14 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-ironic: Change default service_name to 'Ironic'  https://review.openstack.org/327117
14:15 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Make neutron server/ml2 profiles match t-h-t  https://review.openstack.org/326119
14:15 mnaser is there currently a scenario testing keystone with memcache for tokens and horizon with memcache at the same time? :\
14:16 mnaser https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L746-L753 and https://github.com/openstack/puppet-horizon/blob/master/manifests/init.pp#L410-L414 -- could be the lack of ensure => present in one of them (#justweirdpuppetthings) ?
14:18 iurygregory mnaser, i think the defaults for ensure is present https://github.com/puppetlabs/puppetlabs-stdlib/blob/c5486aba6284664ae87a65beaa011211c70ea03e/lib/puppet/parser/functions/ensure_packages.rb
14:18 mnaser yeah.. seems like it, i just tried getting those two ensure_packages to line up and it failed
14:19 iurygregory humm
14:20 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: WIP - keystone - using_domain_config to True  https://review.openstack.org/327122
14:20 EmilienM jmiu: ^ some testing
14:21 EmilienM jmiu: to see if we have the same error
14:21 chandankumar joined #puppet-openstack
14:21 EmilienM iurygregory: if you want to help, we need reviews on the new modules
14:21 EmilienM I think you already started
14:21 EmilienM but just fyi
14:21 iurygregory EmilienM, i'm doing i'll doble check this problems in my lunch time
14:22 EmilienM iurygregory: only if you want and have time :)
14:22 iurygregory EmilienM, just that problem that i told you about =P
14:24 jmiu EmilienM: waiting with bated breath
14:26 mnaser I am now able to reproduce this bug.. Puppet 4.5.1 and the following .. http://paste.openstack.org/show/508945/ .. it can't even build that catalog, would someone be kind enough to confirm it fails for them too with "Error 400 on SERVER: Evaluation Error: Error while evaluating a Function Call, Duplicate declaration: Package[python-memcache] is already declared; cannot redeclare"
14:26 mnaser latest puppet-{keystone,horizon}
14:27 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/250436
14:29 mnaser latest stdlib too might be needed (for ensure_packages)
14:31 mnaser oh. shit.
14:31 iurygregory what happen? O.o
14:32 mnaser keystone tries to ensure_packages python-memcache (https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L746-L753), then calls oslo::cache (https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L813-L827) whichs does the same thing again here (https://github.com/openstack/puppet-oslo/blob/master/manifests/cache.pp#L152-L164)
14:32 mnaser i suspect its the fact ensure_packages is called twice in the same manifest, perhaps no?
14:32 mnaser i will try and check
14:33 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: metadata: fix module name  https://review.openstack.org/327134
14:35 mnaser dealing with a different error now .. Evaluation Error: Error while evaluating a Function Call, Duplicate declaration: Package[python-memcache] is already declared; cannot redeclare at /etc/puppetlabs/code/environments/opta/modules/oslo/manifests/cache.pp:159:5 at /etc/puppetlabs/code/environments/opta/modules/keystone/manifests/init.pp:804 -- i wonder if tags are affecting it
14:37 _stowa joined #puppet-openstack
14:38 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: Update metadata name  https://review.openstack.org/327142
14:39 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-watcher: Update metadata name  https://review.openstack.org/327143
14:41 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-ec2api: Update metadata name  https://review.openstack.org/327147
14:42 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-vitrage: Update metadata name  https://review.openstack.org/327150
14:42 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-trove: Change default service_name to 'trove'  https://review.openstack.org/327152
14:43 openstackgerrit Mohammed Naser proposed openstack/puppet-keystone: Remove ensure_packages for Memcache libraries  https://review.openstack.org/327153
14:46 openstackgerrit Merged openstack/puppet-mistral: Test multiple operating systems for policy  https://review.openstack.org/325399
14:46 severion joined #puppet-openstack
14:47 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Import ::nova class from THT  https://review.openstack.org/325982
14:53 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-magnum: Update metadata name  https://review.openstack.org/327163
14:54 iurygregory EmilienM, pacemaker too?
14:54 EmilienM iurygregory: yes
14:54 iurygregory EmilienM, sending
14:55 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-rally: Update metadata name  https://review.openstack.org/327164
14:56 iurygregory EmilienM, i've done i search for all modules i think it's ok now =D
14:57 EmilienM cool thx
14:57 iurygregory np =D
14:59 openstackgerrit Merged openstack/puppet-cinder: Fix facts for policy test  https://review.openstack.org/326781
15:00 fawadkhaliq joined #puppet-openstack
15:01 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-ironic: Change default service_name to 'Ironic'  https://review.openstack.org/327117
15:05 zhongshengping joined #puppet-openstack
15:05 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Add new modules in Puppetfile  https://review.openstack.org/327174
15:07 zhongshengping joined #puppet-openstack
15:10 faizy joined #puppet-openstack
15:13 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: run Gnocchi tempest tests  https://review.openstack.org/318961
15:13 EmilienM iberezovskiy: lol
15:14 EmilienM how did you know horizon would work on xenial?
15:14 EmilienM did we do something?
15:14 iurygregory it's magic
15:14 iurygregory lol
15:15 openstackgerrit Andy Smith proposed openstack/puppet-sahara: Add transport_url parameters for oslo.messaging  https://review.openstack.org/322934
15:15 EmilienM well I don't see the tempest test running
15:16 EmilienM ok tempest test was not enabled :P
15:16 EmilienM iberezovskiy: -1 on your patch
15:16 degorenko he is afk now :)
15:16 EmilienM ok
15:17 mnaser ugh
15:17 mnaser it was the tags
15:17 mnaser i have a reproducable text case
15:17 mnaser test*
15:17 samuelBartel joined #puppet-openstack
15:17 iurygregory mnaser, maybe just let openstack?
15:17 mnaser so fyi, ensure_packages must have matching tags otherwise it fails
15:17 zhongshengping joined #puppet-openstack
15:17 mnaser http://paste.openstack.org/show/508957/
15:17 iurygregory mnaser, nice catch =D
15:17 mnaser thats a reproducable test case
15:17 mnaser ~2 hours later~
15:18 iberezovskiy EmilienM, yeah nice catch. I've tested locally with 39 line updated only
15:18 mnaser is it acceptable to drop it to just openstack for both oslo::cache and anything else referencing the memcache modules?
15:19 openstackgerrit Merged openstack/puppet-openstack-guide: Fix meetings 7th June  https://review.openstack.org/326824
15:19 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: Test horizon  https://review.openstack.org/327099
15:19 iurygregory mnaser, maybe openstack and memcached-package  or just openstack , EmilienM what you think?
15:20 openstackgerrit Merged openstack/puppet-magnum: Add purge_config option  https://review.openstack.org/326793
15:21 mnaser the reason oslo::cache wasn't breaking because it (mistakingly) had a reference to keystone-package, heh
15:22 mnaser iurygregory: im thinking openstack is better, adding memcached-package could probably open a can of worms of other references that we'd forget to update (and break other modules)
15:23 iurygregory mnaser, yeah I agree ;)
15:23 mnaser ill submit a patch with "openstack" to the 3 affected packages i see here (oslo, horizon and keystone) and the discussion can go on there
15:24 openstackgerrit Merged openstack/puppet-rally: Add purge_config option  https://review.openstack.org/326812
15:24 iurygregory mnaser, thanks =)
15:24 openstackgerrit Merged openstack/puppet-openstack-integration: Rename PUPPET_VERSION to PUPPET_MAJ_VERSION  https://review.openstack.org/326740
15:25 xarses joined #puppet-openstack
15:27 openstackgerrit Mohammed Naser proposed openstack/puppet-keystone: Drop keystone-package from python-memcache tags  https://review.openstack.org/327184
15:27 openstackgerrit Mohammed Naser proposed openstack/puppet-oslo: Drop keystone-package from python-memcache tags  https://review.openstack.org/327185
15:27 openstackgerrit Mohammed Naser proposed openstack/puppet-horizon: Drop horizon-package from python-memcache tags  https://review.openstack.org/327186
15:27 mnaser voila
15:27 mnaser would anyone be so kind to let me know how i can point to the sources of those packages with r10k so i can use them temp till they are merged?
15:28 openstackgerrit Merged openstack/puppet-octavia: Add purge_config option  https://review.openstack.org/326805
15:29 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-designate: Change default service_name to 'designate'  https://review.openstack.org/327190
15:29 openstackgerrit Merged openstack/puppet-ceilometer: Provide default service_name for keystone endpoint  https://review.openstack.org/326645
15:31 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-designate: Change default service_name to 'designate'  https://review.openstack.org/327190
15:31 iurygregory mnaser,i think you need to update the tests because we ensure tags
15:32 mnaser oh let me have a look
15:33 openstackgerrit Merged openstack/puppet-glance: Fix Keystone integration documentation  https://review.openstack.org/326752
15:33 openstackgerrit Merged openstack/puppet-octavia: Test multiple operating systems for policy  https://review.openstack.org/325403
15:35 mnaser keystone tests didnt even have it
15:36 openstackgerrit Mohammed Naser proposed openstack/puppet-keystone: Drop keystone-package from python-memcache tags  https://review.openstack.org/327184
15:37 openstackgerrit Mohammed Naser proposed openstack/puppet-oslo: Drop keystone-package from python-memcache tags  https://review.openstack.org/327185
15:38 iurygregory ok let's wait for CI
15:38 mnaser iurygregory: added it to keystone and updated the rest :)
15:38 openstackgerrit Mohammed Naser proposed openstack/puppet-horizon: Drop horizon-package from python-memcache tags  https://review.openstack.org/327186
15:38 iurygregory mnaser, good catch tks :D
15:38 mnaser i'll point my r10k repos to this commit
15:38 mnaser and hopefully celebrate
15:38 iurygregory good luck :D
15:39 openstackgerrit Merged openstack/puppet-oslo: metadata.json: fix module name  https://review.openstack.org/327103
15:39 openstackgerrit Merged openstack/puppet-heat: Fix facts for policy test  https://review.openstack.org/326648
15:40 samuelBartel joined #puppet-openstack
15:41 iberezovskiy mnaser, nice catch about tags, thanks
15:41 mnaser np :>
15:41 mnaser let's hope it works now
15:43 openstackgerrit Merged openstack/puppet-nova: Provide default service_name for keystone endpoint  https://review.openstack.org/326831
15:43 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-cinder: Change default service_name to 'cinder'  https://review.openstack.org/327193
15:45 mnaser relief
15:45 mnaser it works
15:49 openstackgerrit Merged openstack/puppet-ironic: Test multiple operating systems for policy  https://review.openstack.org/325393
15:49 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: refacto: create install_puppet function  https://review.openstack.org/327196
15:49 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: refacto: move run_puppet & catch_selinux_alerts into functions  https://review.openstack.org/327197
15:49 iurygregory mnaser, good to know =D
15:51 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: refacto: move run_puppet & catch_selinux_alerts into functions  https://review.openstack.org/327197
15:53 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: WIP - keystone - using_domain_config to True  https://review.openstack.org/327122
16:00 chandankumar joined #puppet-openstack
16:05 ducttape_ joined #puppet-openstack
16:08 EmilienM _ody: any idea why we have that on beaker jobs? http://logs.openstack.org/54/320554/7/experimental/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial-nv/17dffcc/console.html#_2016-06-08_13_24_16_965
16:09 _ody Investigating quickly.
16:09 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Allow vertica db hint for queries  https://review.openstack.org/327204
16:11 _ody EmilienM: Probably this, http://logs.openstack.org/54/320554/7/experimental/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial-nv/17dffcc/console.html#_2016-06-08_13_23_35_588
16:11 EmilienM _ody: it's weird, our integration jobs work fine,
16:11 EmilienM mhh
16:11 EmilienM interesting
16:11 EmilienM it should be /etc/puppetlabs/code
16:12 _ody Yeah.  We're missing a variable export someplace.
16:12 _ody https://github.com/openstack/puppet-openstack-integration/blob/master/install_modules.sh#L9
16:12 EmilienM _ody: this is good, look http://logs.openstack.org/54/320554/7/experimental/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial-nv/17dffcc/console.html#_2016-06-08_13_23_23_073
16:12 xarses joined #puppet-openstack
16:13 xarses joined #puppet-openstack
16:14 EmilienM I'll investigate after lunch /me afk
16:17 openstackgerrit Pradeep Kilambi proposed openstack/puppet-tripleo: Enable bulk delete in swift for gnocchi  https://review.openstack.org/327214
16:20 openstackgerrit Alexey Deryugin proposed openstack/puppet-nova: Add ability to configure cache  https://review.openstack.org/325588
16:22 openstackgerrit Alexey Deryugin proposed openstack/puppet-nova: Add ability to configure cache  https://review.openstack.org/325588
16:30 zhongshengping joined #puppet-openstack
16:35 zhongshengping joined #puppet-openstack
16:38 openstackgerrit Sven Anderson proposed openstack/puppet-tripleo: Implement Nova Compute profile  https://review.openstack.org/327236
16:38 velovec joined #puppet-openstack
16:40 yolanda joined #puppet-openstack
16:41 dprince joined #puppet-openstack
16:42 fawadkhaliq joined #puppet-openstack
16:42 fawadkhaliq joined #puppet-openstack
16:47 velovec Hi, folks! I'd like to discuss puppet-nova patch https://review.openstack.org/#/c/325588/ that add cache parameters. I've reworked it according to EmilienM comment and moved it to separate manifest, so I'd like to know your opinion to find optimal solution.
16:59 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-ec2api: Update metadata name and remove verbose  https://review.openstack.org/327147
17:00 iberezovskiy velovec, so some of the modules configure cache parameter from init class, do we have any similar parameter which probably could be moved out init.pp?
17:00 iberezovskiy s/parameter/parameters
17:01 ducttape_ joined #puppet-openstack
17:04 ducttape_ joined #puppet-openstack
17:07 iurygregory degorenko, should i put my change rebased on Dimitry's?
17:07 xarses joined #puppet-openstack
17:07 degorenko iurygregory, ??
17:08 iurygregory locally ec2api is failling to in verbose tests..
17:08 iurygregory that is why i put =)
17:11 chandankumar joined #puppet-openstack
17:13 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-nova: Configurable "vncproxy_package_name"  https://review.openstack.org/327251
17:13 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-swift: Change default service_name to 'swift'  https://review.openstack.org/327252
17:14 iurygregory mwhahaha, can you remove your -2 https://review.openstack.org/#/c/326031/ ?
17:14 iurygregory or a update in the patch works?
17:14 mwhahaha upgraded to -1
17:14 iurygregory yeah ;)
17:14 iurygregory tks
17:16 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-openstack-integration: Test horizon  https://review.openstack.org/327099
17:17 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-designate: Change default service_name to 'designate'  https://review.openstack.org/327190
17:18 degorenko mwhahaha, why -1? https://review.openstack.org/#/c/322113/
17:18 velovec iberezovskiy, this separate manifests covers all cache related parameters supported by oslo::cache, so there's no more cache related parameters that can be moved out of init.pp.
17:19 iberezovskiy velovec, I don't mean cache params only
17:19 iberezovskiy mya be some other common stuff
17:19 mwhahaha degorenko: my client still had -1 on the fuel ci
17:19 mwhahaha seems clear now
17:19 velovec As I can see, there's ssl related parameters, but I don't know if they can be moved out of init.pp
17:20 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-cinder: Change default service_name to 'cinder'  https://review.openstack.org/327193
17:20 degorenko mwhahaha, 7:22 PM - green fuel
17:21 mwhahaha yea merged
17:21 openstackgerrit Denis Egorenko proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system  https://review.openstack.org/327025
17:22 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-ironic: Change default service_name to 'ironic'  https://review.openstack.org/327117
17:22 dprince joined #puppet-openstack
17:23 velovec iberezovskiy, there's also amqp/rabbitmq related parameters that can be moved to separate manifests, but it requires addition discussion
17:24 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-trove: Change default service_name to 'trove'  https://review.openstack.org/327152
17:26 degorenko EmilienM, when you will have time, please: https://review.openstack.org/#/q/topic:314596 :)
17:29 openstackgerrit Marcellin Fom Tchassem proposed openstack/puppet-swift: Change default service_name to 'swift'  https://review.openstack.org/327252
17:31 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Use regexp by whole name for murano_application  https://review.openstack.org/326021
18:01 chandankumar joined #puppet-openstack
18:04 EmilienM _degorenko|afk: ack
18:05 priteau joined #puppet-openstack
18:11 openstackstatus NOTICE: Zuul has been restarted to correct an error condition. Events since 17:30 may have been missed; please 'recheck' your changes if they were uploaded since then, or have "NOT_REGISTERED" errors.
18:19 chandankumar joined #puppet-openstack
18:19 openstackgerrit Shaik Apsar proposed openstack/puppet-trove: Option to specify the trove guestagent log file.  https://review.openstack.org/321154
18:21 openstackgerrit Merged openstack/puppet-openstack-cookiecutter: Update metadata name  https://review.openstack.org/327142
18:21 openstackgerrit Merged openstack/puppet-vitrage: Update metadata name  https://review.openstack.org/327150
18:23 openstackgerrit Merged openstack/puppet-magnum: Update metadata name  https://review.openstack.org/327163
18:23 openstackgerrit Merged openstack/puppet-octavia: Provide default service_name for keystone endpoint  https://review.openstack.org/326847
18:26 shaikapsar joined #puppet-openstack
18:28 ayoung joined #puppet-openstack
18:28 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: WIP - keystone - using_domain_config to True  https://review.openstack.org/327122
18:31 openstackgerrit Emilien Macchi proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system  https://review.openstack.org/327025
18:36 openstackgerrit Shaik Apsar proposed openstack/puppet-trove: Allow guestagent to use unique rabbit_use_ssl param.  https://review.openstack.org/321227
18:41 shaikapsar Please review openstack/puppet-trove: introduce amqp_durable_queues option.  https://review.openstack.org/#/c/326237/
18:41 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-neutron: Configurable "ml2_server_package"  https://review.openstack.org/327281
18:42 openstackgerrit Dmitry Burmistrov proposed openstack/puppet-nova: Configurable "vncproxy_package_name"  https://review.openstack.org/327251
18:45 EmilienM mwhahaha: thought on https://review.openstack.org/314596 is that people should really work by iterations. Big patches like this are really hard to review.
18:46 EmilienM and the probabily to miss something important in the review is high as the LOC is high
18:46 EmilienM probability*
18:46 iurygregory +1 ^
18:47 mwhahaha i agree however for initial modules there has to be some sort of starting point
18:47 mwhahaha it's one thing if it's an established thing, but i think this is the nature of the openstack configurations. for a given service there are 50+ config items
18:48 EmilienM mwhahaha: have you reviewed this patch? is it good for you? I'll look at it this afternoon
18:48 mwhahaha that one is cause it's just the api and metadata services
18:48 EmilienM ok
18:48 mwhahaha i have a -1 one a patch futher down the chain
18:48 EmilienM well, ec2 api is nothing more AFIK
18:48 EmilienM ok cool
18:48 EmilienM I'll review it and press button
18:49 mwhahaha they split ec2api out into it's own thing now i guess? not sure on that history
18:49 EmilienM yea
18:49 mwhahaha i just know we were supporting it in previous releases and then when nova dropped we essentially lost that functionality
18:49 EmilienM mwhahaha: hey I need a second eye on something
18:49 EmilienM can you look http://logs.openstack.org/54/320554/7/experimental/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial-nv/17dffcc/console.html#_2016-06-08_13_23_35_588
18:50 mwhahaha sure
18:50 EmilienM it should be export PUPPET_BASE_PATH=/etc/puppetlabs/code
18:50 EmilienM did I miss something in install_modules.sh ?
18:51 EmilienM don't we collect PUPPET_MAJ_VERSION in the script?
18:51 EmilienM we export it in JJB
18:51 EmilienM my bash foo is low today
18:51 ducttape_ joined #puppet-openstack
18:52 mwhahaha where's the zuul config for this
18:52 EmilienM mwhahaha: see http://logs.openstack.org/54/320554/7/experimental/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-xenial-nv/17dffcc/console.html#_2016-06-08_13_23_23_073
18:52 EmilienM mwhahaha: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml#L74
18:53 mwhahaha i bet it doesn't make it to the thing that actually calls install modules.sh
18:53 mwhahaha i bet bundler starts a new bash session
18:53 EmilienM https://github.com/openstack/puppet-openstack_spec_helper/blob/master/lib/puppet-openstack_spec_helper/beaker_spec_helper.rb#L55
18:53 EmilienM ahah
18:53 EmilienM bash -c or something?
18:54 mwhahaha it's on that line right there
18:54 mwhahaha you're invoking a new bash
18:54 mwhahaha and only passing zuul info
18:54 EmilienM ok
18:54 mwhahaha so you'd need to pass PUPPET_MAJ_VERSION in the spec helper
18:55 EmilienM mwhahaha: ah ok
18:55 EmilienM BASH 101 today
18:55 EmilienM I thought we could pass the env
18:58 mwhahaha or is it that you're exporting on the jenkins slave and running the install_modules on the nodepool node
18:59 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_spec_helper: beaker: export PUPPET_MAJ_VERSION  https://review.openstack.org/327287
19:00 EmilienM mwhahaha: good? ^
19:01 mwhahaha probably
19:01 mwhahaha brb
19:01 jtomasek joined #puppet-openstack
19:20 openstackgerrit Pradeep Kilambi proposed openstack/puppet-tripleo: Add ceilometer profiles  https://review.openstack.org/305143
19:22 ducttape_ joined #puppet-openstack
19:25 rcernin joined #puppet-openstack
19:31 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP Add support for swift storage policies  https://review.openstack.org/327298
19:32 johnmilton joined #puppet-openstack
19:37 dhollinger joined #puppet-openstack
19:50 yolanda joined #puppet-openstack
19:59 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile  https://review.openstack.org/325937
20:01 openstackgerrit Pradeep Kilambi proposed openstack/puppet-tripleo: Add ceilometer profiles  https://review.openstack.org/305143
20:01 fawadkhaliq joined #puppet-openstack
20:11 iurygregory EmilienM, i was thinking today about the midcycle lol
20:11 EmilienM our stars are aligned
20:12 iurygregory yep
20:19 EmilienM mwhahaha: can we get https://review.openstack.org/#/c/327287/ please?
20:19 mwhahaha k
20:21 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system  https://review.openstack.org/327025
20:22 iurygregory EmilienM, you think we should wait for the other patches for puppet-ec2api ? https://review.openstack.org/327147 i have no problem =)
20:23 EmilienM I don't really know
20:24 openstackgerrit Alan Pevec proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service  https://review.openstack.org/250436
20:24 EmilienM my 2 wishes for new modules are: make it step by step and make it tested
20:24 iurygregory is just the fix in metadata but i need to remove the verbose..
20:26 johnmilton joined #puppet-openstack
20:31 openstackgerrit Merged openstack/puppet-vitrage: Test multiple operating systems for vitrage::db::postgresql  https://review.openstack.org/324171
20:37 dgurtner joined #puppet-openstack
20:41 openstackgerrit Merged openstack/puppet-sahara: Provide default service_name for keystone endpoint  https://review.openstack.org/326840
20:43 johnmilton joined #puppet-openstack
20:44 EmilienM 9.0.0 is out!
20:45 iurygregory yeah \o/
20:45 iurygregory nice email btw
20:45 openstackgerrit Merged openstack/puppet-trove: Introduce amqp_durable_queues option  https://review.openstack.org/326237
20:46 iurygregory i'm thinking in add new thinks to http://docs.openstack.org/developer/puppet-openstack-guide/coding-style.html
20:47 openstackgerrit Merged openstack/puppet-monasca: Allow vertica db hint for queries  https://review.openstack.org/327204
20:48 openstackgerrit joined #puppet-openstack
21:00 openstackgerrit Merged openstack/puppet-zaqar: Provide default service_name for keystone endpoint  https://review.openstack.org/326833
21:09 cwolferh joined #puppet-openstack
21:09 ntpttr_ EmilienM: Hi, I just saw your email about all the new modules getting added - I'm interested in making sure the Magnum module I helped start up is supported and gets on the official list of supported modules here http://docs.openstack.org/developer/puppet-openstack-guide/module-list.html, I just have other priorities at work now so I'll be doing it mostly in free time.
21:10 ntpttr_ EmilienM: What all does a module need to do in order to be officially supported in a release?
21:10 ntpttr_ is there anything I can do, I mean :)
21:11 EmilienM ntpttr_: puppet magnum is actually in good shape I think
21:11 iurygregory yep maybe we just forgot to add
21:11 EmilienM ntpttr_: it would be awesome to have a scenario in our puppet-openstack-integration
21:12 EmilienM ntpttr_: do you have an example of manifest that you use in your cloud to deploy magnum and make it work to deploy containers, etc?
21:12 iurygregory i think module list is every puppet module not only the ones with releases, Am I right?
21:13 EmilienM iurygregory: yes
21:13 iurygregory EmilienM, i'll update o/
21:13 ntpttr_ EmilienM: I don't, but I could work on putting one together. Or if I remember Michal Adamczyk did have one put together that he was using when he sent us that email looking to add his changes upstream
21:14 iurygregory ntpttr_, now you will see :D
21:14 ntpttr_ iurygregory: ah cool, thank you :)
21:14 iurygregory i'll add magnum and others modules sorry =)
21:14 EmilienM ntpttr_: cool let me know
21:15 iurygregory have a nice night people o/ leaving the office =)
21:16 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Add new modules in Puppetfile  https://review.openstack.org/327174
21:16 EmilienM iurygregory: o/
21:17 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: Add nodepool-xenial.yml file  https://review.openstack.org/320554
21:35 openstackgerrit Pradeep Kilambi proposed openstack/puppet-aodh: Update to include new aodhclient  https://review.openstack.org/311762
21:40 openstackgerrit Emilien Macchi proposed openstack/puppet-octavia: Implement api.pp  https://review.openstack.org/326142
21:47 openstackgerrit Pradeep Kilambi proposed openstack/puppet-tripleo: Add gnocchi profiles  https://review.openstack.org/315527
21:48 dfisher joined #puppet-openstack
21:50 shaikapsar joined #puppet-openstack
21:58 HoloIRCUser3 joined #puppet-openstack
22:04 ducttape_ joined #puppet-openstack
22:09 cwolferh joined #puppet-openstack
22:10 EmilienM mwhahaha: https://review.openstack.org/#/c/322113/
22:10 EmilienM we're luck zuul was broken
22:10 EmilienM it didn't merge
22:10 EmilienM we have the bug everywhere
22:10 EmilienM clayton: FYI ^
22:10 EmilienM we need oslo in all deps.pp
22:10 EmilienM I found a race cond in tripleo because of this
22:11 mwhahaha k
22:11 mwhahaha fortunately i tried to merge it during the zuul failure earlier today :D
22:11 EmilienM lol
22:11 EmilienM yeah
22:11 clayton I thought about this when I was doing glance earlier
22:12 clayton the config resources are already handled by the existing deps
22:12 mwhahaha there is no oslo specific provider
22:12 mwhahaha they reuse the existing ones
22:12 mwhahaha so i'm not sure how oslo plays into it
22:12 EmilienM mwhahaha: I spent my afternoon investigating for a bug
22:12 clayton I don't see anything else in oslo::db or oslo::cache that would be effected by this
22:12 EmilienM and I found out all our deps are broken
22:13 EmilienM we're missing Oslo resources
22:13 clayton the oslo resources are just wrappers around the normal config resources, right?
22:13 EmilienM and I found the case where nova db sync runs *before* configuring database in nova.conf
22:13 EmilienM clayton: that's what I thought
22:13 clayton oslo::db should just be creating nova_config resources
22:13 EmilienM maybe i'm wrong and I missed something
22:13 EmilienM right
22:14 mwhahaha sounds like a possible bug with the interaction with create_resources
22:14 clayton well, there are some package resources in those classes that might be broken, but I don't think my patch would change those
22:14 openstackgerrit Merged openstack/puppet-vitrage: Switch vitrage to oslo  https://review.openstack.org/324172
22:14 clayton we probably need to consider tagging the oslo and openstacklib package resources with something like openstack-package and oslo-package
22:15 EmilienM I'm about to submit a patch in puppet-nova and test it in tripleo CI so I can confirm we have a bug
22:15 clayton ok
22:15 EmilienM clayton: ++ for tagging
22:15 clayton EmilienM: sounds good, let me know if you find something
22:16 mwhahaha EmilienM: nova is missing the nova_config<||> ~> db sync stuf
22:16 clayton I was going to be pretty amazed if a patch that size merged with one patchset :)
22:16 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: deps: add missing Oslo resources  https://review.openstack.org/327397
22:16 mwhahaha see neutron, https://github.com/openstack/puppet-neutron/blob/master/manifests/db/sync.pp#L23
22:16 EmilienM mwhahaha: ah
22:16 mwhahaha unless that got put into deps
22:16 mwhahaha https://github.com/openstack/puppet-nova/blob/master/manifests/db/sync.pp#L23
22:16 clayton I remember when I started doing them that way, I would have guessed it was nova
22:17 clayton rather, I don't rememeber
22:17 clayton looking
22:17 mwhahaha nope it's not in nova::deps
22:17 mwhahaha so we need that
22:17 EmilienM well, we have Anchor['nova::dbsync::begin']
22:17 EmilienM mwhahaha: what is missing? can you send a patch so I see?
22:17 clayton EmilienM: it's in nova::deps - https://github.com/openstack/puppet-nova/blob/master/manifests/deps.pp#L21-L24
22:18 mwhahaha that's not the same
22:18 mwhahaha i'm not sure it's contained
22:19 clayton it is: https://github.com/openstack/puppet-nova/blob/master/manifests/db/sync.pp
22:19 clayton there is only one resource in the sync class
22:19 mwhahaha I think the issue is https://github.com/openstack/puppet-nova/blob/master/manifests/db/sync.pp#L23-L27
22:20 clayton how so?
22:20 ducttape_ joined #puppet-openstack
22:20 mwhahaha like you need Anchor['nova::dbsync::begin'] -> Exec['nova-db-sync'] -> Anchor['nova::dbsync::end']
22:20 mwhahaha where is that?
22:21 clayton that's what the subscribe/notify there does
22:21 mwhahaha no necessarily
22:21 mwhahaha i don't think they are the same
22:21 clayton subscribe/notify imply before/require
22:21 mwhahaha i guess the true way is to print the catalog ordering
22:22 mwhahaha EmilienM: do you have the log from where it ran before?
22:22 EmilienM mwhahaha: the logs of failures I see in tripleo?
22:23 mwhahaha yea
22:23 EmilienM http://logs.openstack.org/83/325983/3/check-tripleo/gate-tripleo-ci-f22-ha/4eea2e8/logs/postci.txt.gz#_2016-06-08_19_34_03_000
22:23 EmilienM grep "Error:"
22:23 clayton we've been running that code in our internal integration testing 30-40 times a day for months, that particular part works correctly.
22:23 EmilienM clayton: do you run dbsync with puppet or something else?
22:23 clayton with puppet
22:23 EmilienM ok
22:24 clayton we do a 5 multi-node test every hour + on every dev deploy that only uses puppet to do the db stuff
22:24 EmilienM mwhahaha: you'll notice Oslo::Db[nova_config] is missing in the catalog
22:25 EmilienM clayton: yeah, you have strong testing that's awesome
22:25 EmilienM maybe we found a corner case or something
22:25 clayton well, I'll be glad to do a fix if need be
22:25 EmilienM clayton: I think we can merge your puppet-neutron patch
22:25 EmilienM we hold it on until we sort this out for nova though
22:26 EmilienM I'm afk ~15 min. mwhahaha: feel free to send something in puppet-nova, i'll review when I'm back
22:26 mwhahaha wish these logs were readable :D
22:26 EmilienM mwhahaha: yeah tripleo logs suck
22:33 clayton EmilienM: will probably have the equivalent glance patch up soonish
22:34 chandankumar joined #puppet-openstack
22:34 chandankumar_ joined #puppet-openstack
22:38 EmilienM clayton: w00t
22:38 clayton glance was slightly easier to do than neutron was :)
22:40 mwhahaha we are missing the nova_api_config<||> in deps
22:42 mwhahaha and nova::db has no relations for the database anchor that i can see
22:43 clayton the db anchor is for creating the database, not configuring it
22:43 clayton it's used in nova::db::mysql and nova::db::postgresql
22:43 mwhahaha hmm ok
22:43 mwhahaha ah
22:44 fawadkhaliq joined #puppet-openstack
22:44 clayton I didn't leave out nova_api_config, it didn't exist when I did the hooks patch :)
22:45 mwhahaha i bet the create_resources is messing with the collector
22:46 mwhahaha we might need to Oslo::Db<||>
22:46 mwhahaha and all the oslo::* classes :/
22:46 clayton I'd be really surprised if that were the case
22:46 mwhahaha would probably want _ody or Hunner to validate that assumption
22:46 clayton collectors are applied after all the resources have been created
22:46 clayton I doubt the collector code even knows create_resources was used
22:46 mwhahaha i've learned not to assume anything :D
22:47 EmilienM see https://review.openstack.org/327397
22:47 EmilienM I'm going to try that out until you guys find something else
22:47 mwhahaha well all the oslo stuff should be before nova::config::end
22:47 clayton EmilienM: I think that's a bad idea, but if you do that, I think it need to go config::begin and config::end
22:47 mwhahaha but yea that
22:47 Hunner collector code is evaluated at the end of an internal loop of evaluating all known classes and defines. create_resources() is a function so runs as part of the class or define evaluation
22:48 EmilienM clayton: what do you suggest?
22:48 mwhahaha Hunner: so create_resources should be evaulated before the collector code right?
22:48 Hunner If you use create_resources() to declare a defined resources, and then a collector to collect resources that WILL BE declared inside the "created" defined resources, you may not actually get them (if your catalog is complex, you probably will though)
22:49 clayton well, we don't actually understand what is causing the problem.  putting in code to work around "maybe a bug in puppet" seems like the wrong approach
22:49 clayton Hunner: these are all native types/providers
22:49 Hunner mwhahaha: strictly, yes, but if you use create_resources() to declare resources that generate other resources, you may have oddness
22:49 Hunner clayton: Then its sounds fine
22:50 Hunner I haven't read scrollback to see what the problem is to start with though :)
22:50 clayton nod, np, appreciate the feedback
22:50 mwhahaha yea our oslo defined resources are using create_resources to create the *_configs
22:50 EmilienM Hunner: tl;dr: I found out a case where dbsync could run *before* configuring db parameters in nova.conf
22:51 EmilienM Hunner: and we use Oslo::Db define to configure it
22:51 Hunner So guessing that the collector didn't collect all the db parameters?
22:51 EmilienM Hunner: so we're wondering if puppet-nova/manifests/deps.pp is good
22:51 EmilienM Hunner: yes
22:51 EmilienM Hunner: and Oslo::Db use create_resources('nova_config')
22:53 Hunner Here's the loop I'm thinking of https://github.com/puppetlabs/puppet/blob/2911cc6ae56c6fbdc02a8f3cb3ad46f4747eaf07/lib/puppet/parser/compiler.rb#L612-L616
22:55 Hunner Okay, I see nova::deps. What declares the nova_config resources?
22:55 Hunner eh, I see a bunch of them
22:55 Hunner So never mind about that
22:56 EmilienM Hunner: https://github.com/openstack/puppet-oslo/blob/master/manifests/db.pp#L187
22:58 EmilienM I still don't understand why db_sync runs before database config
22:58 Hunner It seems like it would evaluate the collector, then evaluate oslo::db, that would cause it to evaluate the collector again, then it could end
22:58 EmilienM we have the suscribe thing
22:58 Hunner Yeah, I don't see why yet
22:59 EmilienM clayton: I'm going to test my patch anyway at least to know if it helps even if we won't fix it that way
22:59 Hunner You don't have a --graph of this do you?
22:59 EmilienM no
23:04 ducttape_ joined #puppet-openstack
23:18 fawadkhaliq joined #puppet-openstack
23:19 bnemec joined #puppet-openstack
23:24 openstackgerrit Merged openstack/puppet-tripleo: Remove loadbalancer profile  https://review.openstack.org/325893
23:44 markvoelker joined #puppet-openstack
23:50 chandankumar joined #puppet-openstack
23:50 iurygregory_ joined #puppet-openstack
23:58 ducttape_ joined #puppet-openstack

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