| EmilienM | yeah, we'll figure that later, EOD /me off | 00:00 |
|---|---|---|
| *** dfisher has quit IRC | 00:13 | |
| *** cwolferh has quit IRC | 00:14 | |
| *** ducttape_ has quit IRC | 00:22 | |
| *** ansmith has quit IRC | 00:23 | |
| *** cwolferh has joined #puppet-openstack | 00:31 | |
| *** zhongshengping has joined #puppet-openstack | 00:33 | |
| * bkero waves | 00:34 | |
| openstackgerrit | Merged openstack/puppet-nova: Fix facts for policy test https://review.openstack.org/326651 | 00:44 |
| openstackgerrit | zhongshengping proposed openstack/puppet-ec2api: Test multiple operating systems for policy https://review.openstack.org/325387 | 00:50 |
| openstackgerrit | zhongshengping proposed openstack/puppet-ironic: Test multiple operating systems for policy https://review.openstack.org/325393 | 00:51 |
| openstackgerrit | zhongshengping proposed openstack/puppet-mistral: Test multiple operating systems for policy https://review.openstack.org/325399 | 00:53 |
| openstackgerrit | zhongshengping proposed openstack/puppet-octavia: Test multiple operating systems for policy https://review.openstack.org/325403 | 00:54 |
| *** markvoelker has joined #puppet-openstack | 01:13 | |
| *** markvoelker has quit IRC | 01:18 | |
| *** ilbot3 has quit IRC | 01:46 | |
| *** ilbot3 has joined #puppet-openstack | 01:48 | |
| openstackgerrit | Dan Prince proposed openstack/puppet-tripleo: Make neutron server/ml2 profiles match t-h-t https://review.openstack.org/326119 | 02:08 |
| openstackgerrit | zhongshengping proposed openstack/puppet-cinder: Fix facts for policy test https://review.openstack.org/326781 | 02:09 |
| openstackgerrit | zhongshengping proposed openstack/puppet-designate: Add purge_config option https://review.openstack.org/326783 | 02:22 |
| *** ducttape_ has joined #puppet-openstack | 02:23 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-designate: Add purge_config option https://review.openstack.org/326783 | 02:31 |
| *** ducttape_ has quit IRC | 02:33 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-gnocchi: Add purge_config option https://review.openstack.org/326786 | 02:38 |
| *** richm has quit IRC | 02:45 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-heat: Add purge_config option https://review.openstack.org/326789 | 02:47 |
| *** openstackgerrit has quit IRC | 02:47 | |
| *** openstackgerrit has joined #puppet-openstack | 02:48 | |
| *** ducttape_ has joined #puppet-openstack | 02:48 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-ironic: Add purge_config option https://review.openstack.org/326790 | 02:53 |
| openstackgerrit | zhongshengping proposed openstack/puppet-magnum: Add purge_config option https://review.openstack.org/326793 | 03:01 |
| openstackgerrit | zhongshengping proposed openstack/puppet-manila: Add purge_config option https://review.openstack.org/326796 | 03:07 |
| openstackgerrit | zhongshengping proposed openstack/puppet-mistral: Add purge_config option https://review.openstack.org/326797 | 03:12 |
| *** ducttape_ has quit IRC | 03:24 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-murano: dd purge_config option https://review.openstack.org/326804 | 03:34 |
| openstackgerrit | zhongshengping proposed openstack/puppet-octavia: Add purge_config option https://review.openstack.org/326805 | 03:40 |
| *** furlongm has quit IRC | 03:41 | |
| *** furlongm has joined #puppet-openstack | 03:42 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Import ::nova class from THT https://review.openstack.org/325982 | 03:46 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile https://review.openstack.org/325937 | 03:48 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile https://review.openstack.org/325937 | 03:52 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-nova: [WIP] Move nova to use OSDefaults https://review.openstack.org/326221 | 03:53 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova Scheduler profile https://review.openstack.org/326092 | 03:54 |
| 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:55 |
| *** iurygregory_ has quit IRC | 03:57 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-rally: Add purge_config option https://review.openstack.org/326812 | 03:59 |
| 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:00 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Deploy nova::db classes in api profile https://review.openstack.org/326027 | 04:01 |
| 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:05 |
| openstackgerrit | zhongshengping proposed openstack/puppet-rally: Add purge_config option https://review.openstack.org/326812 | 04:12 |
| *** cwolferh has quit IRC | 04:14 | |
| *** ducttape_ has joined #puppet-openstack | 04:25 | |
| *** ducttape_ has quit IRC | 04:30 | |
| *** jaosorior has joined #puppet-openstack | 04:50 | |
| *** v1k0d3n has quit IRC | 04:57 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-openstack-guide: Fix meetings 7th June https://review.openstack.org/326824 | 05:15 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-glance: Provide consisent default name for service_identity resource https://review.openstack.org/326826 | 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 |
| *** fawadkhaliq has joined #puppet-openstack | 05:33 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-nova: Provide default service_name for keystone endpoint https://review.openstack.org/326831 | 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 | zhongshengping proposed openstack/puppet-zaqar: Provide default service_name for keystone endpoint https://review.openstack.org/326833 | 05:53 |
| *** saneax_AFK is now known as saneax | 05:57 | |
| *** jtomasek has quit IRC | 06:01 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-neutron: Provide default service_name for keystone endpoint https://review.openstack.org/326836 | 06:02 |
| *** v1k0d3n has joined #puppet-openstack | 06:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL https://review.openstack.org/326837 | 06:06 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-glance: Provide consisent default name for service_identity resource https://review.openstack.org/326826 | 06:08 |
| openstackgerrit | zhongshengping proposed openstack/puppet-sahara: Provide default service_name for keystone endpoint https://review.openstack.org/326840 | 06:10 |
| *** rcernin has joined #puppet-openstack | 06:12 | |
| *** mflobo has left #puppet-openstack | 06:13 | |
| *** mflobo has joined #puppet-openstack | 06:13 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-nova: Provide default service_name for keystone endpoint https://review.openstack.org/326831 | 06:16 |
| *** dgurtner has joined #puppet-openstack | 06:21 | |
| *** dgurtner has quit IRC | 06:21 | |
| *** dgurtner has joined #puppet-openstack | 06:21 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-octavia: Provide default service_name for keystone endpoint https://review.openstack.org/326847 | 06:25 |
| *** ducttape_ has joined #puppet-openstack | 06:26 | |
| *** darkxploit has quit IRC | 06:26 | |
| openstackgerrit | zhongshengping proposed openstack/puppet-sahara: Add purge_config option https://review.openstack.org/326815 | 06:30 |
| *** ducttape_ has quit IRC | 06:31 | |
| *** faizy has joined #puppet-openstack | 06:33 | |
| *** cwolferh has joined #puppet-openstack | 06:39 | |
| *** darkxploit has joined #puppet-openstack | 06:43 | |
| *** openstackgerrit has quit IRC | 06:48 | |
| *** openstackgerrit has joined #puppet-openstack | 06:48 | |
| *** jpena|off is now known as jpena | 06:50 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-barbican: Change default service_name to 'barbican' https://review.openstack.org/326860 | 06:53 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-barbican: Change default service_name to 'barbican' https://review.openstack.org/326860 | 06:54 |
| *** priteau has joined #puppet-openstack | 06:59 | |
| *** tesseract has joined #puppet-openstack | 07:00 | |
| *** foxhunt_away is now known as foxhunt72 | 07:01 | |
| *** amoralej|off is now known as amoralej | 07:04 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-barbican: Change default service_name to 'barbican' https://review.openstack.org/326860 | 07:18 |
| *** ifarkas has joined #puppet-openstack | 07:20 | |
| *** fawadkhaliq has quit IRC | 07:23 | |
| *** karimb has joined #puppet-openstack | 07:24 | |
| *** danpawlik has joined #puppet-openstack | 07:28 | |
| *** shardy has joined #puppet-openstack | 07:39 | |
| *** dtantsur|afk is now known as dtantsur | 07:44 | |
| *** pepijn has joined #puppet-openstack | 07:47 | |
| *** openstackgerrit has quit IRC | 07:48 | |
| *** openstackgerrit has joined #puppet-openstack | 07:48 | |
| *** paramite|afk is now known as paramite | 07:55 | |
| *** chandankumar has joined #puppet-openstack | 08:00 | |
| *** karimb has quit IRC | 08:03 | |
| *** jtomasek has joined #puppet-openstack | 08:05 | |
| *** fawadkhaliq has joined #puppet-openstack | 08:09 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-nova: Add missing block_device_allocate_* params in nova.conf https://review.openstack.org/290496 | 08:17 |
| *** jaosorior has quit IRC | 08:25 | |
| *** jaosorior has joined #puppet-openstack | 08:25 | |
| *** aarefiev_ is now known as aarefiev | 08:26 | |
| *** ducttape_ has joined #puppet-openstack | 08:27 | |
| *** ducttape_ has quit IRC | 08:33 | |
| *** dmk0202 has joined #puppet-openstack | 08:37 | |
| *** karimb has joined #puppet-openstack | 08:42 | |
| *** derekh has joined #puppet-openstack | 08:42 | |
| *** fawadkhaliq has quit IRC | 08:45 | |
| *** openstackgerrit has quit IRC | 08:48 | |
| *** openstackgerrit has joined #puppet-openstack | 08:48 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Add acceptance tests for client and application provider https://review.openstack.org/246480 | 08:49 |
| *** andreww has 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:50 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Add acceptance tests for client and application provider https://review.openstack.org/246480 | 08:51 |
| *** xarses_ has quit IRC | 08:52 | |
| *** dtantsur is now known as dtantsur|bbl | 08:57 | |
| *** jtomasek has quit IRC | 09:04 | |
| *** chandankumar has quit IRC | 09:11 | |
| *** chandankumar has joined #puppet-openstack | 09:12 | |
| *** jtomasek has joined #puppet-openstack | 09:18 | |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP https://review.openstack.org/310725 | 09:20 |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP https://review.openstack.org/310725 | 09:22 |
| *** dgurtner has quit IRC | 09:31 | |
| *** fawadkhaliq has joined #puppet-openstack | 09:32 | |
| *** zhongshengping has quit IRC | 09:32 | |
| *** fawadkhaliq has quit IRC | 09:33 | |
| *** fawadkhaliq has joined #puppet-openstack | 09:33 | |
| *** fawadkhaliq has quit IRC | 09:33 | |
| *** dgurtner has joined #puppet-openstack | 09:38 | |
| *** dgurtner has quit IRC | 09:38 | |
| *** dgurtner has joined #puppet-openstack | 09:38 | |
| *** fawadkhaliq has joined #puppet-openstack | 09:42 | |
| *** markvoelker has joined #puppet-openstack | 09:53 | |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP https://review.openstack.org/310725 | 09:55 |
| openstackgerrit | Carlos Camacho proposed openstack/puppet-tripleo: Composable roles within services - NTP https://review.openstack.org/310725 | 09:57 |
| *** ansmith has joined #puppet-openstack | 10:02 | |
| *** dtantsur|bbl is now known as dtantsur | 10:11 | |
| openstackgerrit | Merged openstack/puppet-openstack-cookiecutter: Update keystone::auth service name parameter https://review.openstack.org/326636 | 10:18 |
| *** ducttape_ has joined #puppet-openstack | 10:29 | |
| *** chandankumar has quit IRC | 10:31 | |
| *** ansmith has quit IRC | 10:32 | |
| *** ducttape_ has quit IRC | 10:34 | |
| *** dgurtner has quit IRC | 10:45 | |
| openstackgerrit | Merged openstack/puppet-murano: Add purge_config option https://review.openstack.org/326804 | 10:49 |
| *** tesseract has quit IRC | 10:50 | |
| *** tesseract has joined #puppet-openstack | 10:51 | |
| openstackgerrit | Merged openstack/puppet-manila: Add purge_config option https://review.openstack.org/326796 | 10:52 |
| *** johnmilton has quit IRC | 10:59 | |
| *** andreww has quit IRC | 10:59 | |
| *** yolanda has quit IRC | 10:59 | |
| *** xb_ has joined #puppet-openstack | 11:00 | |
| openstackgerrit | Daniel Pawlik proposed openstack/puppet-watcher: This commit adds the initial project files. https://review.openstack.org/326593 | 11:07 |
| *** fawadkhaliq has quit IRC | 11:12 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-keystone: Fix facts for policy test https://review.openstack.org/326642 | 11:15 |
| *** dgurtner has joined #puppet-openstack | 11:17 | |
| *** fawadkhaliq has joined #puppet-openstack | 11:19 | |
| *** chandankumar has joined #puppet-openstack | 11:23 | |
| *** fawadkhaliq has quit IRC | 11:25 | |
| *** moravec has quit IRC | 11:27 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system https://review.openstack.org/327025 | 11:28 |
| 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:33 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system https://review.openstack.org/327025 | 11:34 |
| *** fawadkhaliq has joined #puppet-openstack | 11:38 | |
| *** ducttape_ has joined #puppet-openstack | 11:38 | |
| *** fawadkhaliq has quit IRC | 11:40 | |
| *** fawadkhaliq has joined #puppet-openstack | 11:40 | |
| *** jpena is now known as jpena|lunch | 11:40 | |
| *** moravec has joined #puppet-openstack | 11:42 | |
| *** morazi has joined #puppet-openstack | 11:42 | |
| *** fawadk has joined #puppet-openstack | 11:43 | |
| *** fawadkhaliq has quit IRC | 11:44 | |
| *** fawadkhaliq has joined #puppet-openstack | 11:44 | |
| *** fawadk has quit IRC | 11:48 | |
| iurygregory | Morning guys =) | 11:49 |
| *** yolanda has joined #puppet-openstack | 11:49 | |
| *** iberezovskiy_afk is now known as iberezovskiy | 11:49 | |
| iberezovskiy | hi | 11:49 |
| *** johnmilton has joined #puppet-openstack | 11:50 | |
| *** ducttape_ has quit IRC | 11:50 | |
| *** ducttape_ has joined #puppet-openstack | 11:51 | |
| degorenko | iurygregory, you wanted zeromq - https://review.openstack.org/#/c/327025/ - take it please :) | 11:52 |
| *** faizy has quit IRC | 11:53 | |
| iurygregory | degorenko, lol I was just thinking if we should have =) | 11:53 |
| 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:54 |
| *** xarses has joined #puppet-openstack | 11:55 | |
| * iurygregory breakfast time | 11:55 | |
| *** ducttape_ has quit IRC | 11:55 | |
| *** markvoelker has quit IRC | 11:56 | |
| *** ifarkas has quit IRC | 11:58 | |
| *** moravec has quit IRC | 12:00 | |
| iurygregory | EmilienM, not more one module now, but two =P [puppet] Request to create puppet-tacker | 12:02 |
| *** fawadkhaliq has quit IRC | 12:03 | |
| *** ifarkas has joined #puppet-openstack | 12:06 | |
| *** ayoung has quit IRC | 12:07 | |
| zigo | How is configured Keystone in puppet-openstack? Is it using Apache? | 12:07 |
| 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:08 |
| 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:09 |
| degorenko | we deprecated eventlet in mitaka | 12:10 |
| degorenko | and it will be removed soon at all | 12:10 |
| iurygregory | yeah | 12:10 |
| *** jayg|g0n3 is now known as jayg | 12:11 | |
| *** markvoelker has joined #puppet-openstack | 12:12 | |
| *** moravec has joined #puppet-openstack | 12:14 | |
| *** dprince has joined #puppet-openstack | 12:15 | |
| *** fawadkhaliq has joined #puppet-openstack | 12:22 | |
| openstackgerrit | Merged openstack/puppet-mistral: Add purge_config option https://review.openstack.org/326797 | 12:23 |
| *** fawadkhaliq has quit IRC | 12:26 | |
| openstackgerrit | Merged openstack/puppet-ironic: Add purge_config option https://review.openstack.org/326790 | 12:26 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service https://review.openstack.org/250436 | 12:27 |
| EmilienM | good morning | 12:29 |
| EmilienM | iurygregory: ack, will figure that today | 12:29 |
| 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:30 |
| iberezovskiy | I like to align the stars :D | 12:31 |
| iberezovskiy | done | 12:31 |
| *** trown|outtypewww is now known as trown | 12:31 | |
| openstackgerrit | Merged openstack/puppet-sahara: Add purge_config option https://review.openstack.org/326815 | 12:33 |
| *** fawadkhaliq has joined #puppet-openstack | 12:33 | |
| 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:43 |
| *** ducttape_ has joined #puppet-openstack | 12:44 | |
| *** pepijn has quit IRC | 12:45 | |
| *** jpena|lunch is now known as jpena | 12:49 | |
| *** xarses has quit IRC | 12:50 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Rename PUPPET_VERSION to PUPPET_MAJ_VERSION https://review.openstack.org/326740 | 12:54 |
| *** amoralej is now known as amoralej|lunch | 12:56 | |
| *** ansmith has joined #puppet-openstack | 12:56 | |
| 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:03 |
| *** ducttape_ has quit IRC | 13:04 | |
| *** fawadkhaliq has quit IRC | 13:04 | |
| openstackgerrit | Merged openstack/puppet-barbican: Change default service_name to 'barbican' https://review.openstack.org/326860 | 13:04 |
| iberezovskiy | mnaser, could you please share your manifest? | 13:06 |
| iberezovskiy | it's really strange because we have ensure_packages everywhere | 13:06 |
| 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:07 |
| mnaser | the only 3 modules i have installed which reference python-memcache are oslo, horizon and keystone | 13:08 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Add nodepool-xenial.yml file https://review.openstack.org/320554 | 13:11 |
| 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:12 |
| 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:13 |
| *** priteau has quit IRC | 13:14 | |
| iberezovskiy | joined to the channel | 13:14 |
| *** priteau has joined #puppet-openstack | 13:15 | |
| *** samuelBartel has joined #puppet-openstack | 13:18 | |
| *** paramite is now known as paramite|afk | 13:19 | |
| *** richm has joined #puppet-openstack | 13:20 | |
| *** chandankumar has quit IRC | 13:23 | |
| *** ayoung has joined #puppet-openstack | 13:26 | |
| *** samuelBartel has quit IRC | 13:32 | |
| *** gchamoul is now known as gchamoul|afk | 13:32 | |
| *** gchamoul|afk is now known as gchamoul | 13:32 | |
| *** amoralej|lunch is now known as amoralej | 13:43 | |
| *** paramite|afk is now known as paramite | 13:43 | |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-openstack-integration: Test horizon https://review.openstack.org/327099 | 13:44 |
| 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:45 |
| 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:46 |
| 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:47 |
| mnaser | too fast | 13:48 |
| EmilienM | degorenko, iberezovskiy: can we get this one before ttx process the release? ^ | 13:48 |
| iberezovskiy | degorenko, land it please :) | 13:49 |
| degorenko | already did it | 13:50 |
| degorenko | colonwq, ping | 13:52 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-neutron: Provide default service_name for keystone endpoint https://review.openstack.org/326836 | 13:56 |
| openstackgerrit | Dan Prince proposed openstack/puppet-tripleo: Drop the neutron dnsmasq config file options https://review.openstack.org/327109 | 13:57 |
| *** dfisher has joined #puppet-openstack | 14:00 | |
| EmilienM | iberezovskiy, degorenko: thx guys and thx mnaser for reporting it | 14:02 |
| iurygregory | yeah, stars aligned \o/ | 14:03 |
| EmilienM | cool | 14:04 |
| iurygregory | lol the openstack bot didn't put here XD | 14:04 |
| *** ducttape_ has 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:05 |
| openstackgerrit | Alan Pevec proposed openstack/puppet-tempest: python-murano-tests is currently not subpackged https://review.openstack.org/327114 | 14:08 |
| *** yolanda_ has joined #puppet-openstack | 14:08 | |
| *** ducttape_ has quit IRC | 14:09 | |
| *** yolanda has quit IRC | 14:09 | |
| *** yolanda_ has quit IRC | 14:09 | |
| *** yolanda has joined #puppet-openstack | 14:09 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable TLS in the internal network for heat https://review.openstack.org/327069 | 14:12 |
| 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:14 |
| 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:15 |
| 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:16 |
| 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:18 |
| iurygregory | humm | 14:19 |
| 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 has 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:21 |
| EmilienM | iurygregory: only if you want and have time :) | 14:22 |
| iurygregory | EmilienM, just that problem that i told you about =P | 14:22 |
| jmiu | EmilienM: waiting with bated breath | 14:24 |
| 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 |
| *** jaosorior has quit IRC | 14:26 | |
| mnaser | latest puppet-{keystone,horizon} | 14:26 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-openstack-integration: scenario003: enable application catalog service https://review.openstack.org/250436 | 14:27 |
| mnaser | latest stdlib too might be needed (for ensure_packages) | 14:29 |
| mnaser | oh. shit. | 14:31 |
| iurygregory | what happen? O.o | 14:31 |
| 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:32 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: metadata: fix module name https://review.openstack.org/327134 | 14:33 |
| 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:35 |
| *** _stowa has joined #puppet-openstack | 14:37 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-cookiecutter: Update metadata name https://review.openstack.org/327142 | 14:38 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-watcher: Update metadata name https://review.openstack.org/327143 | 14:39 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-ec2api: Update metadata name https://review.openstack.org/327147 | 14:41 |
| *** _stowa has quit IRC | 14:41 | |
| 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:42 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-keystone: Remove ensure_packages for Memcache libraries https://review.openstack.org/327153 | 14:43 |
| openstackgerrit | Merged openstack/puppet-mistral: Test multiple operating systems for policy https://review.openstack.org/325399 | 14:46 |
| *** severion has joined #puppet-openstack | 14:46 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Import ::nova class from THT https://review.openstack.org/325982 | 14:47 |
| *** v1k0d3n has quit IRC | 14:48 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-magnum: Update metadata name https://review.openstack.org/327163 | 14:53 |
| iurygregory | EmilienM, pacemaker too? | 14:54 |
| EmilienM | iurygregory: yes | 14:54 |
| iurygregory | EmilienM, sending | 14:54 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-rally: Update metadata name https://review.openstack.org/327164 | 14:55 |
| iurygregory | EmilienM, i've done i search for all modules i think it's ok now =D | 14:56 |
| EmilienM | cool thx | 14:57 |
| iurygregory | np =D | 14:57 |
| openstackgerrit | Merged openstack/puppet-cinder: Fix facts for policy test https://review.openstack.org/326781 | 14:59 |
| *** fawadkhaliq has joined #puppet-openstack | 15:00 | |
| openstackgerrit | Marcellin Fom Tchassem proposed openstack/puppet-ironic: Change default service_name to 'Ironic' https://review.openstack.org/327117 | 15:01 |
| *** zhongshengping has joined #puppet-openstack | 15:05 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Add new modules in Puppetfile https://review.openstack.org/327174 | 15:05 |
| *** zhongshengping has quit IRC | 15:06 | |
| *** zhongshengping has joined #puppet-openstack | 15:07 | |
| *** faizy has joined #puppet-openstack | 15:10 | |
| *** _nick is now known as yankcrime | 15:12 | |
| *** dprince has quit IRC | 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:13 |
| 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:14 |
| 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:15 |
| *** zhongshengping has quit IRC | 15:15 | |
| 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:16 |
| 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 has 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 has 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:17 |
| 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:18 |
| 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:19 |
| openstackgerrit | Merged openstack/puppet-magnum: Add purge_config option https://review.openstack.org/326793 | 15:20 |
| mnaser | the reason oslo::cache wasn't breaking because it (mistakingly) had a reference to keystone-package, heh | 15:21 |
| 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:22 |
| 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:23 |
| openstackgerrit | Merged openstack/puppet-rally: Add purge_config option https://review.openstack.org/326812 | 15:24 |
| iurygregory | mnaser, thanks =) | 15:24 |
| *** faizy has quit IRC | 15:24 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Rename PUPPET_VERSION to PUPPET_MAJ_VERSION https://review.openstack.org/326740 | 15:24 |
| *** xarses has joined #puppet-openstack | 15:25 | |
| *** samuelBartel has quit IRC | 15:25 | |
| 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:27 |
| openstackgerrit | Merged openstack/puppet-octavia: Add purge_config option https://review.openstack.org/326805 | 15:28 |
| openstackgerrit | Marcellin Fom Tchassem proposed openstack/puppet-designate: Change default service_name to 'designate' https://review.openstack.org/327190 | 15:29 |
| *** dmk0202 has quit IRC | 15:29 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: Provide default service_name for keystone endpoint https://review.openstack.org/326645 | 15:29 |
| 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:31 |
| mnaser | oh let me have a look | 15:32 |
| 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:33 |
| *** foxhunt72 is now known as foxhunt_away | 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:36 |
| openstackgerrit | Mohammed Naser proposed openstack/puppet-oslo: Drop keystone-package from python-memcache tags https://review.openstack.org/327185 | 15:37 |
| 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:38 |
| 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:39 |
| *** samuelBartel has joined #puppet-openstack | 15:40 | |
| iberezovskiy | mnaser, nice catch about tags, thanks | 15:41 |
| mnaser | np :> | 15:41 |
| mnaser | let's hope it works now | 15:41 |
| *** chandankumar has quit IRC | 15:42 | |
| 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:43 |
| mnaser | relief | 15:45 |
| mnaser | it works | 15:45 |
| 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:49 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: refacto: move run_puppet & catch_selinux_alerts into functions https://review.openstack.org/327197 | 15:51 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: WIP - keystone - using_domain_config to True https://review.openstack.org/327122 | 15:53 |
| *** saneax is now known as saneax_AFK | 15:53 | |
| *** zhongshengping has quit IRC | 15:59 | |
| *** samuelBartel has quit IRC | 15:59 | |
| *** chandankumar has joined #puppet-openstack | 16:00 | |
| *** ducttape_ has joined #puppet-openstack | 16:06 | |
| *** fawadkhaliq has quit IRC | 16:07 | |
| *** xarses has quit IRC | 16:07 | |
| 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:08 |
| _ody | Investigating quickly. | 16:09 |
| openstackgerrit | Bradley Klein proposed openstack/puppet-monasca: Allow vertica db hint for queries https://review.openstack.org/327204 | 16:09 |
| *** ducttape_ has quit IRC | 16:10 | |
| _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:11 |
| _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 has joined #puppet-openstack | 16:12 | |
| *** xarses has quit IRC | 16:13 | |
| *** xarses has joined #puppet-openstack | 16:13 | |
| EmilienM | I'll investigate after lunch /me afk | 16:14 |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Enable bulk delete in swift for gnocchi https://review.openstack.org/327214 | 16:17 |
| *** xarses has quit IRC | 16:20 | |
| *** jtomasek has quit IRC | 16:20 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-nova: Add ability to configure cache https://review.openstack.org/325588 | 16:20 |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-nova: Add ability to configure cache https://review.openstack.org/325588 | 16:22 |
| *** zhongshengping has joined #puppet-openstack | 16:30 | |
| *** trown is now known as trown|lunch | 16:31 | |
| *** zhongshengping has quit IRC | 16:31 | |
| *** tesseract has quit IRC | 16:31 | |
| *** dtantsur is now known as dtantsur|afk | 16:32 | |
| *** zhongshengping has joined #puppet-openstack | 16:35 | |
| *** zhongshengping has quit IRC | 16:36 | |
| *** yolanda has quit IRC | 16:37 | |
| *** jpena is now known as jpena|off | 16:37 | |
| openstackgerrit | Sven Anderson proposed openstack/puppet-tripleo: Implement Nova Compute profile https://review.openstack.org/327236 | 16:38 |
| *** velovec has joined #puppet-openstack | 16:38 | |
| *** yolanda has joined #puppet-openstack | 16:40 | |
| *** dprince has joined #puppet-openstack | 16:41 | |
| *** amoralej is now known as amoralej|off | 16:41 | |
| *** rcernin has quit IRC | 16:42 | |
| *** fawadkhaliq has joined #puppet-openstack | 16:42 | |
| *** fawadkhaliq has quit IRC | 16:42 | |
| *** fawadkhaliq has joined #puppet-openstack | 16:42 | |
| 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:47 |
| *** karimb has quit IRC | 16:51 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-ec2api: Update metadata name and remove verbose https://review.openstack.org/327147 | 16:59 |
| 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:00 |
| *** ayoung has quit IRC | 17:01 | |
| *** ducttape_ has joined #puppet-openstack | 17:01 | |
| *** ducttape_ has quit IRC | 17:03 | |
| *** ducttape_ has joined #puppet-openstack | 17:04 | |
| *** chandankumar has quit IRC | 17:05 | |
| iurygregory | degorenko, should i put my change rebased on Dimitry's? | 17:07 |
| *** xarses has joined #puppet-openstack | 17:07 | |
| degorenko | iurygregory, ?? | 17:07 |
| iurygregory | locally ec2api is failling to in verbose tests.. | 17:08 |
| iurygregory | that is why i put =) | 17:08 |
| *** derekh has quit IRC | 17:08 | |
| *** chandankumar has joined #puppet-openstack | 17:11 | |
| 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:13 |
| 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:14 |
| openstackgerrit | Ivan Berezovskiy proposed openstack/puppet-openstack-integration: Test horizon https://review.openstack.org/327099 | 17:16 |
| openstackgerrit | Marcellin Fom Tchassem proposed openstack/puppet-designate: Change default service_name to 'designate' https://review.openstack.org/327190 | 17:17 |
| 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:18 |
| 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:19 |
| openstackgerrit | Marcellin Fom Tchassem proposed openstack/puppet-cinder: Change default service_name to 'cinder' https://review.openstack.org/327193 | 17:20 |
| *** trown|lunch is now known as trown | 17:20 | |
| degorenko | mwhahaha, 7:22 PM - green fuel | 17:20 |
| mwhahaha | yea merged | 17:21 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system https://review.openstack.org/327025 | 17:21 |
| *** chandankumar has quit IRC | 17:21 | |
| openstackgerrit | Marcellin Fom Tchassem proposed openstack/puppet-ironic: Change default service_name to 'ironic' https://review.openstack.org/327117 | 17:22 |
| *** dprince has quit IRC | 17:22 | |
| *** dprince has 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:23 |
| openstackgerrit | Marcellin Fom Tchassem proposed openstack/puppet-trove: Change default service_name to 'trove' https://review.openstack.org/327152 | 17:24 |
| degorenko | EmilienM, when you will have time, please: https://review.openstack.org/#/q/topic:314596 :) | 17:26 |
| *** dgurtner has quit IRC | 17:27 | |
| *** iberezovskiy is now known as iberezovskiy|off | 17:28 | |
| openstackgerrit | Marcellin Fom Tchassem proposed openstack/puppet-swift: Change default service_name to 'swift' https://review.openstack.org/327252 | 17:29 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-murano: Use regexp by whole name for murano_application https://review.openstack.org/326021 | 17:31 |
| *** degorenko is now known as _degorenko|afk | 17:32 | |
| *** chandankumar has joined #puppet-openstack | 18:01 | |
| *** velovec has quit IRC | 18:01 | |
| EmilienM | _degorenko|afk: ack | 18:04 |
| *** priteau has quit IRC | 18:05 | |
| *** priteau has joined #puppet-openstack | 18:05 | |
| *** chandankumar has quit IRC | 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:11 | |
| *** chandankumar has 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:19 |
| 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:21 |
| 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:23 |
| *** shaikapsar has joined #puppet-openstack | 18:26 | |
| *** ayoung has 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:28 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system https://review.openstack.org/327025 | 18:31 |
| openstackgerrit | Shaik Apsar proposed openstack/puppet-trove: Allow guestagent to use unique rabbit_use_ssl param. https://review.openstack.org/321227 | 18:36 |
| 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:41 |
| openstackgerrit | Dmitry Burmistrov proposed openstack/puppet-nova: Configurable "vncproxy_package_name" https://review.openstack.org/327251 | 18:42 |
| 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:45 |
| 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:46 |
| 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:47 |
| 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:48 |
| *** yolanda has quit IRC | 18:48 | |
| 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 |
| *** ducttape_ has quit IRC | 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:49 |
| 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:50 |
| 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_ has joined #puppet-openstack | 18:51 | |
| 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:52 |
| 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:53 |
| 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:54 |
| EmilienM | mwhahaha: ah ok | 18:55 |
| EmilienM | BASH 101 today | 18:55 |
| EmilienM | I thought we could pass the env | 18:55 |
| *** priteau has quit IRC | 18:57 | |
| mwhahaha | or is it that you're exporting on the jenkins slave and running the install_modules on the nodepool node | 18:58 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper: beaker: export PUPPET_MAJ_VERSION https://review.openstack.org/327287 | 18:59 |
| EmilienM | mwhahaha: good? ^ | 19:00 |
| mwhahaha | probably | 19:01 |
| mwhahaha | brb | 19:01 |
| *** jtomasek has joined #puppet-openstack | 19:01 | |
| *** xb_ has quit IRC | 19:03 | |
| *** ducttape_ has quit IRC | 19:07 | |
| *** foxhunt_away is now known as foxhunt72 | 19:12 | |
| *** chandankumar has quit IRC | 19:13 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Add ceilometer profiles https://review.openstack.org/305143 | 19:20 |
| *** ducttape_ has joined #puppet-openstack | 19:22 | |
| *** rcernin has joined #puppet-openstack | 19:25 | |
| *** johnmilton has quit IRC | 19:28 | |
| *** ifarkas has quit IRC | 19:31 | |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: WIP Add support for swift storage policies https://review.openstack.org/327298 | 19:31 |
| *** johnmilton has joined #puppet-openstack | 19:32 | |
| *** dhollinger has joined #puppet-openstack | 19:37 | |
| *** johnmilton has quit IRC | 19:42 | |
| *** saneax_AFK is now known as saneax | 19:48 | |
| *** yolanda has joined #puppet-openstack | 19:50 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tripleo: Implement Nova API profile https://review.openstack.org/325937 | 19:59 |
| *** fawadkhaliq has quit IRC | 20:01 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Add ceilometer profiles https://review.openstack.org/305143 | 20:01 |
| *** fawadkhaliq has joined #puppet-openstack | 20:01 | |
| *** fawadkhaliq has quit IRC | 20:05 | |
| iurygregory | EmilienM, i was thinking today about the midcycle lol | 20:11 |
| EmilienM | our stars are aligned | 20:11 |
| iurygregory | yep | 20:12 |
| EmilienM | mwhahaha: can we get https://review.openstack.org/#/c/327287/ please? | 20:19 |
| mwhahaha | k | 20:19 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-oslo: Add support for ZeroMQ messaging system https://review.openstack.org/327025 | 20:21 |
| iurygregory | EmilienM, you think we should wait for the other patches for puppet-ec2api ? https://review.openstack.org/327147 i have no problem =) | 20:22 |
| EmilienM | I don't really know | 20:23 |
| 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:24 |
| *** johnmilton has joined #puppet-openstack | 20:26 | |
| *** johnmilton has quit IRC | 20:31 | |
| openstackgerrit | Merged openstack/puppet-vitrage: Test multiple operating systems for vitrage::db::postgresql https://review.openstack.org/324171 | 20:31 |
| *** saneax is now known as saneax_AFK | 20:34 | |
| *** dgurtner has joined #puppet-openstack | 20:37 | |
| openstackgerrit | Merged openstack/puppet-sahara: Provide default service_name for keystone endpoint https://review.openstack.org/326840 | 20:41 |
| *** jayg is now known as jayg|g0n3 | 20:41 | |
| *** johnmilton has joined #puppet-openstack | 20:43 | |
| EmilienM | 9.0.0 is out! | 20:44 |
| 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:45 |
| iurygregory | i'm thinking in add new thinks to http://docs.openstack.org/developer/puppet-openstack-guide/coding-style.html | 20:46 |
| *** shaikapsar has quit IRC | 20:47 | |
| openstackgerrit | Merged openstack/puppet-monasca: Allow vertica db hint for queries https://review.openstack.org/327204 | 20:47 |
| *** openstackgerrit has quit IRC | 20:48 | |
| *** openstackgerrit has joined #puppet-openstack | 20:48 | |
| *** ansmith has quit IRC | 20:59 | |
| openstackgerrit | Merged openstack/puppet-zaqar: Provide default service_name for keystone endpoint https://review.openstack.org/326833 | 21:00 |
| *** cwolferh has quit IRC | 21:02 | |
| *** cwolferh has joined #puppet-openstack | 21:09 | |
| *** foxhunt72 is now known as foxhunt_away | 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:09 |
| 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:10 |
| 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:11 |
| 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:12 |
| 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:13 |
| 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:14 |
| iurygregory | have a nice night people o/ leaving the office =) | 21:15 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Add new modules in Puppetfile https://review.openstack.org/327174 | 21:16 |
| EmilienM | iurygregory: o/ | 21:16 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: Add nodepool-xenial.yml file https://review.openstack.org/320554 | 21:17 |
| *** trown is now known as trown|outtypewww | 21:18 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-aodh: Update to include new aodhclient https://review.openstack.org/311762 | 21:35 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-octavia: Implement api.pp https://review.openstack.org/326142 | 21:40 |
| *** dfisher has quit IRC | 21:43 | |
| openstackgerrit | Pradeep Kilambi proposed openstack/puppet-tripleo: Add gnocchi profiles https://review.openstack.org/315527 | 21:47 |
| *** dfisher has joined #puppet-openstack | 21:48 | |
| *** shaikapsar has joined #puppet-openstack | 21:50 | |
| *** rcernin is now known as rcernin|off | 21:54 | |
| *** dhollinger has quit IRC | 21:54 | |
| *** ducttape_ has quit IRC | 21:57 | |
| *** HoloIRCUser3 has joined #puppet-openstack | 21:58 | |
| *** dprince has quit IRC | 22:01 | |
| *** cwolferh has quit IRC | 22:03 | |
| *** ducttape_ has joined #puppet-openstack | 22:04 | |
| *** cwolferh has 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:10 |
| 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:11 |
| 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:12 |
| 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:14 |
| 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:15 |
| 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 |
| *** ducttape_ has quit IRC | 22:16 | |
| clayton | I remember when I started doing them that way, I would have guessed it was nova | 22:16 |
| 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:17 |
| mwhahaha | that's not the same | 22:18 |
| mwhahaha | i'm not sure it's contained | 22:18 |
| 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:19 |
| clayton | how so? | 22:20 |
| *** ducttape_ has 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:20 |
| *** rcernin|off has quit IRC | 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:21 |
| 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:22 |
| 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:24 |
| 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:25 |
| 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:26 |
| clayton | EmilienM: will probably have the equivalent glance patch up soonish | 22:33 |
| *** chandankumar has joined #puppet-openstack | 22:34 | |
| *** chandankumar_ has joined #puppet-openstack | 22:34 | |
| EmilienM | clayton: w00t | 22:38 |
| clayton | glance was slightly easier to do than neutron was :) | 22:38 |
| *** chandankumar_ has quit IRC | 22:40 | |
| *** chandankumar has quit IRC | 22:40 | |
| mwhahaha | we are missing the nova_api_config<||> in deps | 22:40 |
| *** dgurtner has quit IRC | 22:41 | |
| mwhahaha | and nova::db has no relations for the database anchor that i can see | 22:42 |
| 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:43 |
| *** fawadkhaliq has 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:44 |
| mwhahaha | i bet the create_resources is messing with the collector | 22:45 |
| 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:46 |
| 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:47 |
| 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:48 |
| 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:49 |
| 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:50 |
| 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:51 |
| Hunner | Here's the loop I'm thinking of https://github.com/puppetlabs/puppet/blob/2911cc6ae56c6fbdc02a8f3cb3ad46f4747eaf07/lib/puppet/parser/compiler.rb#L612-L616 | 22:53 |
| Hunner | Okay, I see nova::deps. What declares the nova_config resources? | 22:55 |
| *** yolanda has quit IRC | 22:55 | |
| Hunner | eh, I see a bunch of them | 22:55 |
| Hunner | So never mind about that | 22:55 |
| EmilienM | Hunner: https://github.com/openstack/puppet-oslo/blob/master/manifests/db.pp#L187 | 22:56 |
| 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:58 |
| 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 | 22:59 |
| *** dfisher has quit IRC | 23:01 | |
| *** ducttape_ has quit IRC | 23:03 | |
| *** ducttape_ has joined #puppet-openstack | 23:04 | |
| *** saneax_AFK is now known as saneax | 23:07 | |
| *** bnemec has quit IRC | 23:11 | |
| *** markvoelker has quit IRC | 23:13 | |
| *** fawadkhaliq has quit IRC | 23:17 | |
| *** fawadkhaliq has joined #puppet-openstack | 23:18 | |
| *** bnemec has joined #puppet-openstack | 23:19 | |
| *** fawadkhaliq has quit IRC | 23:21 | |
| openstackgerrit | Merged openstack/puppet-tripleo: Remove loadbalancer profile https://review.openstack.org/325893 | 23:24 |
| *** markvoelker has joined #puppet-openstack | 23:44 | |
| *** markvoelker has quit IRC | 23:49 | |
| *** chandankumar has joined #puppet-openstack | 23:50 | |
| *** iurygregory_ has joined #puppet-openstack | 23:50 | |
| *** chandankumar has quit IRC | 23:54 | |
| *** ducttape_ has quit IRC | 23:57 | |
| *** ducttape_ has joined #puppet-openstack | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!