| *** zhangjn has joined #puppet-openstack | 00:00 | |
| *** zhangjn has quit IRC | 00:13 | |
| openstackgerrit | Gilles Dubreuil proposed openstack/puppet-keystone: keystone_endpoint provider for Keystone v3 api https://review.openstack.org/178456 | 00:27 |
|---|---|---|
| *** guessi has joined #puppet-openstack | 00:27 | |
| *** dtalton has quit IRC | 00:42 | |
| *** zhangjn has joined #puppet-openstack | 01:01 | |
| *** bogdando has quit IRC | 01:01 | |
| *** markvoelker has joined #puppet-openstack | 01:02 | |
| *** bogdando has joined #puppet-openstack | 01:08 | |
| *** richm has quit IRC | 01:13 | |
| *** kbyrne has quit IRC | 01:14 | |
| *** kbyrne has joined #puppet-openstack | 01:15 | |
| *** nlahouti has quit IRC | 01:19 | |
| *** zhangjn has quit IRC | 01:22 | |
| *** zhangjn has joined #puppet-openstack | 01:22 | |
| *** zhangjn has quit IRC | 01:24 | |
| *** guessi has quit IRC | 01:29 | |
| *** zhangjn has joined #puppet-openstack | 01:32 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service https://review.openstack.org/237662 | 01:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service https://review.openstack.org/237662 | 01:41 |
| *** guessi has joined #puppet-openstack | 01:47 | |
| *** crinkle has quit IRC | 01:49 | |
| *** crinkle has joined #puppet-openstack | 01:51 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-swift: Prepare 6.1.0 release https://review.openstack.org/237163 | 01:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: Prepare 6.1.0 release https://review.openstack.org/237140 | 01:58 |
| *** markvoelker has quit IRC | 02:42 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-keystone: Support for Keystone as Service Provider https://review.openstack.org/216821 | 02:43 |
| *** nlahouti has joined #puppet-openstack | 02:46 | |
| *** nlahouti1 has joined #puppet-openstack | 02:50 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: Prepare 6.1.0 release https://review.openstack.org/237140 | 02:52 |
| *** nlahouti has quit IRC | 02:53 | |
| *** ferest has joined #puppet-openstack | 02:53 | |
| *** ferest has quit IRC | 02:58 | |
| *** khushbu_ has joined #puppet-openstack | 03:01 | |
| *** khushbu_ has quit IRC | 03:01 | |
| *** nlahouti1 has quit IRC | 03:04 | |
| *** khushbu_ has joined #puppet-openstack | 03:04 | |
| *** khushbu_ has quit IRC | 03:06 | |
| *** khushbu_ has joined #puppet-openstack | 03:21 | |
| *** nlahouti has joined #puppet-openstack | 03:25 | |
| openstackgerrit | Merged openstack/puppet-swift: Prepare 6.1.0 release https://review.openstack.org/237163 | 03:26 |
| *** zhangjn has quit IRC | 03:26 | |
| *** nlahouti has quit IRC | 03:43 | |
| *** gildub has quit IRC | 03:47 | |
| openstackgerrit | Sam Morrison proposed openstack/puppet-heat: Support engine_workers variable https://review.openstack.org/237866 | 04:00 |
| *** zhangjn has joined #puppet-openstack | 04:12 | |
| *** zhangjn has quit IRC | 04:19 | |
| *** zhangjn has joined #puppet-openstack | 04:23 | |
| *** khushbu_ has quit IRC | 04:27 | |
| *** chem has quit IRC | 04:33 | |
| *** khushbu_ has joined #puppet-openstack | 04:44 | |
| *** guessi has quit IRC | 04:48 | |
| *** guessi has joined #puppet-openstack | 04:48 | |
| *** nlahouti has joined #puppet-openstack | 04:56 | |
| *** sanjayu has joined #puppet-openstack | 04:57 | |
| *** mihgen has quit IRC | 05:00 | |
| *** khushbu_ has quit IRC | 05:05 | |
| *** zerda has joined #puppet-openstack | 05:12 | |
| *** khushbu_ has joined #puppet-openstack | 05:12 | |
| *** openstackgerrit has quit IRC | 05:16 | |
| *** openstackgerrit has joined #puppet-openstack | 05:17 | |
| *** khushbu_ has quit IRC | 05:17 | |
| *** khushbu_ has joined #puppet-openstack | 05:23 | |
| *** guessi has left #puppet-openstack | 05:36 | |
| *** khushbu_ has quit IRC | 05:44 | |
| *** khushbu_ has joined #puppet-openstack | 05:47 | |
| *** khushbu_ has quit IRC | 05:47 | |
| *** zhangjn has quit IRC | 05:47 | |
| *** khushbu_ has joined #puppet-openstack | 06:01 | |
| *** khushbu_ has quit IRC | 06:05 | |
| *** khushbu_ has joined #puppet-openstack | 06:08 | |
| *** khushbu_ has quit IRC | 06:09 | |
| *** khushbu_ has joined #puppet-openstack | 06:12 | |
| *** khushbu_ has quit IRC | 06:21 | |
| *** khushbu_ has joined #puppet-openstack | 06:25 | |
| *** khushbu_ has quit IRC | 06:26 | |
| *** nlahouti has quit IRC | 06:27 | |
| *** nlahouti has joined #puppet-openstack | 06:27 | |
| *** khushbu_ has joined #puppet-openstack | 06:29 | |
| *** khushbu_ has quit IRC | 06:39 | |
| *** khushbu_ has joined #puppet-openstack | 06:54 | |
| *** sergmelikyan has joined #puppet-openstack | 06:56 | |
| *** khushbu_ has quit IRC | 07:05 | |
| *** sergmelikyan has quit IRC | 07:06 | |
| *** linkedinyou has quit IRC | 07:06 | |
| *** dgurtner has joined #puppet-openstack | 07:07 | |
| *** dgurtner has quit IRC | 07:07 | |
| *** dgurtner has joined #puppet-openstack | 07:07 | |
| *** simon-AS5591 has quit IRC | 07:08 | |
| *** zhangjn has joined #puppet-openstack | 07:19 | |
| *** zhangjn has quit IRC | 07:19 | |
| *** bnemec has quit IRC | 07:19 | |
| *** zhangjn has joined #puppet-openstack | 07:20 | |
| *** bnemec has joined #puppet-openstack | 07:25 | |
| *** jpena has joined #puppet-openstack | 07:26 | |
| *** jongchoi has joined #puppet-openstack | 07:31 | |
| openstackgerrit | Bartłomiej Piotrowski proposed openstack/puppet-swift: Retry dispersion up to 5 times if it failed https://review.openstack.org/236911 | 07:33 |
| *** magicboiz has joined #puppet-openstack | 07:36 | |
| *** nlahouti has quit IRC | 07:41 | |
| *** zhangjn has quit IRC | 07:42 | |
| *** zhangjn has joined #puppet-openstack | 07:49 | |
| *** paramite has joined #puppet-openstack | 08:05 | |
| *** rochaporto has joined #puppet-openstack | 08:10 | |
| *** simon-AS5591 has joined #puppet-openstack | 08:11 | |
| *** simon-AS5591 has quit IRC | 08:12 | |
| *** derekh has joined #puppet-openstack | 08:14 | |
| *** simon-AS559 has joined #puppet-openstack | 08:15 | |
| *** simon-AS559 has quit IRC | 08:17 | |
| *** gfidente|afk is now known as gfidente | 08:23 | |
| *** jistr has joined #puppet-openstack | 08:23 | |
| *** jongchoi has quit IRC | 08:48 | |
| *** _degorenko is now known as degorenko | 08:56 | |
| *** openstackgerrit has quit IRC | 09:01 | |
| *** openstackgerrit has joined #puppet-openstack | 09:02 | |
| *** paramite is now known as paramite|afk | 09:09 | |
| *** khushbu_ has joined #puppet-openstack | 09:30 | |
| *** openstackgerrit has quit IRC | 09:31 | |
| *** openstackgerrit has joined #puppet-openstack | 09:32 | |
| *** sergmelikyan has joined #puppet-openstack | 09:34 | |
| *** magicboiz has quit IRC | 09:42 | |
| *** _nick has quit IRC | 09:45 | |
| *** _nick has joined #puppet-openstack | 09:47 | |
| openstackgerrit | Lukas Bezdicka proposed openstack/puppet-neutron: Cleanup list output https://review.openstack.org/237610 | 09:47 |
| *** chem has joined #puppet-openstack | 09:50 | |
| openstackgerrit | Merged openstack/puppet-cinder: Simplify rpc_backend parameter https://review.openstack.org/233184 | 09:52 |
| *** sergmelikyan has quit IRC | 09:53 | |
| openstackgerrit | Merged openstack/puppet-aodh: metadata: Fix openstacklib module namespace https://review.openstack.org/237548 | 10:15 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-glance: Introduce glance::db class https://review.openstack.org/237970 | 10:17 |
| *** skolekonov has joined #puppet-openstack | 10:23 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-glance: add more settings to glance api, registry, rabbitmq https://review.openstack.org/237035 | 10:24 |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-glance: Add support of os_service_default fact in glance spec https://review.openstack.org/237973 | 10:24 |
| *** sergmelikyan has joined #puppet-openstack | 10:26 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/puppet-tripleo: Resolve repeated ports for ssl frontends https://review.openstack.org/237974 | 10:30 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-heat: Change section name for AMQP qpid parameters https://review.openstack.org/237975 | 10:31 |
| *** bogdando has quit IRC | 10:37 | |
| *** khushbu_ has quit IRC | 10:39 | |
| *** bogdando has joined #puppet-openstack | 10:39 | |
| *** paramite|afk is now known as paramite | 10:40 | |
| *** khushbu_ has joined #puppet-openstack | 10:49 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-glance: Add support of os_service_default fact in glance spec https://review.openstack.org/237973 | 10:50 |
| openstackgerrit | Merged openstack/puppet-murano: db: Use postgresql lib class for psycopg package https://review.openstack.org/237679 | 10:51 |
| *** sergmelikyan has quit IRC | 10:55 | |
| *** jayg|g0n3 has quit IRC | 10:57 | |
| *** jayg|g0n3 has joined #puppet-openstack | 10:57 | |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-heat: add more heat.conf settings to heat, engine classes https://review.openstack.org/233621 | 11:01 |
| openstackgerrit | Merged openstack/puppet-openstack-cookiecutter: db: Use postgresql lib class for psycopg package https://review.openstack.org/237669 | 11:03 |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-glance: add more settings to glance api, registry, rabbitmq https://review.openstack.org/237035 | 11:08 |
| social | skolekonov: re neutron stdout/stderr only neutron failed | 11:13 |
| social | skolekonov: because most of other modules use openstacklib and it was hit by this before so they have the checks there | 11:13 |
| skolekonov | social, by the way, do you use SSL for endpoints in Keystone? | 11:13 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-glance: Change section name for AMQP qpid parameters https://review.openstack.org/237989 | 11:14 |
| social | skolekonov: for packstack not yet, orhers I hope do use it | 11:14 |
| social | skolekonov: but I think noone should use it directly in the python service/wsgi | 11:15 |
| social | skolekonov: use haproxy or pound instead infront of the service | 11:15 |
| openstackgerrit | Merged openstack/puppet-barbican: db: Use postgresql lib class for psycopg package https://review.openstack.org/237576 | 11:15 |
| skolekonov | social, Neutron client, for example, seems to use public endpoint even if internal is forced in some way. And SSL can be on a public endpoint - then puppet will fail also at resources creation (not only listing) | 11:17 |
| openstackgerrit | Merged openstack/puppet-tuskar: db: Use postgresql lib class for psycopg package https://review.openstack.org/237666 | 11:17 |
| openstackgerrit | Merged openstack/puppet-gnocchi: db: Use postgresql lib class for psycopg package https://review.openstack.org/237591 | 11:17 |
| openstackgerrit | Merged openstack/puppet-heat: db: Use postgresql lib class for psycopg package https://review.openstack.org/237606 | 11:17 |
| openstackgerrit | Merged openstack/puppet-sahara: db: Use postgresql lib class for psycopg package https://review.openstack.org/237667 | 11:17 |
| openstackgerrit | Merged openstack/puppet-ceilometer: db: Use postgresql lib class for psycopg package https://review.openstack.org/237583 | 11:17 |
| skolekonov | SSL is just an example, I guess openstackclient can also produce warnings and this is a potential problem | 11:17 |
| social | skolekonov: well we should switch puppet-neutron to openstacklib | 11:18 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-neutron: Change section name for AMQP qpid parameters https://review.openstack.org/237990 | 11:18 |
| openstackgerrit | Merged openstack/puppet-ironic: db: Use postgresql lib class for psycopg package https://review.openstack.org/237605 | 11:19 |
| skolekonov | probably, but this is a bit different issue | 11:19 |
| openstackgerrit | Merged openstack/puppet-nova: db: Use postgresql lib class for psycopg package https://review.openstack.org/237668 | 11:19 |
| openstackgerrit | Merged openstack/puppet-manila: db: Use postgresql lib class for psycopg package https://review.openstack.org/237671 | 11:20 |
| *** khushbu_ has quit IRC | 11:20 | |
| openstackgerrit | Merged openstack/puppet-neutron: db: Use postgresql lib class for psycopg package https://review.openstack.org/237670 | 11:20 |
| *** zhangjn has quit IRC | 11:22 | |
| openstackgerrit | Merged openstack/puppet-keystone: db: Use postgresql lib class for psycopg package https://review.openstack.org/237672 | 11:22 |
| degorenko | social, hey! i agree with skolekonov - when we try to use ssl and create some resource we also will get warnings. | 11:23 |
| degorenko | and also yes, we probably need to switch neutron providers to openstackclient | 11:23 |
| degorenko | but we must be sure, that OSC supports all actions in neutron providers | 11:24 |
| social | degorenko: what warning do you get with SSL? | 11:25 |
| skolekonov | social, something like this - http://paste.openstack.org/show/476984/ (Ubuntu 14.04) | 11:27 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-cinder: Change section name for AMQP qpid parameters https://review.openstack.org/237997 | 11:31 |
| openstackgerrit | Gael Chamoulaud proposed openstack/puppet-trove: Use identity_uri and auth_uri in api module https://review.openstack.org/233229 | 11:35 |
| degorenko | social, yep, something like skolekonov gave you | 11:35 |
| openstackgerrit | Sebastien Badia proposed openstack/puppet-aodh: db: Use postgresql lib class for psycopg package https://review.openstack.org/237549 | 11:35 |
| social | degorenko: do I understand it correctly it does unverified request? so the ssl par is useless there? :) | 11:37 |
| *** ken_barber has joined #puppet-openstack | 11:39 | |
| degorenko | social, probably yes | 11:41 |
| degorenko | but in such situations puppet will failed | 11:42 |
| openstackgerrit | Gael Chamoulaud proposed openstack/puppet-trove: Use identity_uri and auth_uri in api module https://review.openstack.org/233229 | 11:43 |
| social | will it work for you with the patch for puppet-neutron I sent? (yeah it needs test fixes) | 11:43 |
| *** morazi has joined #puppet-openstack | 11:44 | |
| *** khushbu has joined #puppet-openstack | 11:46 | |
| EmilienM | good morning | 11:47 |
| social | dmsimard: I'm dissapoint about your tempest patch | 11:47 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-ceilometer: Change section name for AMQP qpid parameters https://review.openstack.org/238002 | 11:47 |
| social | ow wrong channel | 11:48 |
| *** khushbu has quit IRC | 11:49 | |
| openstackgerrit | Sergey Kolekonov proposed openstack/puppet-neutron: Add driver option for metering agent https://review.openstack.org/238003 | 11:49 |
| *** iberezovskiy has joined #puppet-openstack | 11:50 | |
| EmilienM | jamespage: the aodh-dbsync works on centos but not on ubuntu: http://logs.openstack.org/62/237662/18/check/gate-puppet-aodh-puppet-beaker-rspec-dsvm-trusty/3c33f9d/console.html#_2015-10-21_01_48_38_148 | 11:50 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-heat: Change section name for AMQP qpid parameters https://review.openstack.org/237975 | 11:52 |
| *** khushbu has joined #puppet-openstack | 11:53 | |
| *** khushbu has quit IRC | 11:55 | |
| *** cdelatte has joined #puppet-openstack | 11:56 | |
| *** khushbu_ has joined #puppet-openstack | 11:58 | |
| *** khushbu_ has quit IRC | 12:02 | |
| *** khushbu_ has joined #puppet-openstack | 12:06 | |
| *** delattec has joined #puppet-openstack | 12:08 | |
| *** ducttape_ has joined #puppet-openstack | 12:09 | |
| *** cdelatte has quit IRC | 12:09 | |
| EmilienM | zigo_: aodh package is broken, jamespage found that alembic.ini is missing | 12:09 |
| jamespage | I believe its because of the diff between the result of python setup.py sdist (as used to cut upstream release tarballs) vs generating a tarball direct from the release tag in git | 12:12 |
| *** delattec has quit IRC | 12:12 | |
| openstackgerrit | Merged openstack/puppet-murano: Make Murano config providers inherit Openstack config provider https://review.openstack.org/237761 | 12:16 |
| *** openstackgerrit has quit IRC | 12:16 | |
| *** openstackgerrit has joined #puppet-openstack | 12:17 | |
| *** trown|outttypeww is now known as trown | 12:19 | |
| *** dprince has joined #puppet-openstack | 12:20 | |
| *** jayg|g0n3 is now known as jayg | 12:21 | |
| openstackgerrit | Merged openstack/puppet-cinder: db: Use postgresql lib class for psycopg package https://review.openstack.org/237760 | 12:28 |
| *** ducttape_ has quit IRC | 12:34 | |
| *** markvoelker has joined #puppet-openstack | 12:35 | |
| *** zerda has quit IRC | 12:35 | |
| *** jpena is now known as jpena|lunch | 12:35 | |
| *** khushbu_ has quit IRC | 12:37 | |
| *** vinsh has quit IRC | 12:41 | |
| openstackgerrit | Merged openstack/puppet-keystone: Deletes tenant parameter from keystone_user https://review.openstack.org/228744 | 12:43 |
| *** richm has joined #puppet-openstack | 12:53 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-ironic: Change section name for AMQP qpid parameters https://review.openstack.org/238044 | 12:56 |
| *** khushbu has joined #puppet-openstack | 12:58 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-nova: Change section name for AMQP qpid parameters https://review.openstack.org/238048 | 13:00 |
| *** khushbu has quit IRC | 13:04 | |
| *** khushbu_ has joined #puppet-openstack | 13:07 | |
| degorenko | EmilienM, hey :) take a look: https://review.openstack.org/#/q/topic:oslo_qpid_section,n,z | 13:12 |
| EmilienM | degorenko: cool ok | 13:12 |
| *** sanjayu has quit IRC | 13:13 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: First implementation of init class https://review.openstack.org/237352 | 13:15 |
| *** ducttape_ has joined #puppet-openstack | 13:16 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Update aodh::logging https://review.openstack.org/237356 | 13:17 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-designate: Change section name for AMQP rabbit parameters https://review.openstack.org/238064 | 13:18 |
| *** sanjayu has joined #puppet-openstack | 13:18 | |
| *** ducttape_ has quit IRC | 13:18 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement API service https://review.openstack.org/237587 | 13:19 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement API service https://review.openstack.org/237587 | 13:19 |
| *** leanderthal has quit IRC | 13:22 | |
| EmilienM | jamespage: where can I report the aodh issue? | 13:23 |
| jamespage | EmilienM, bugs.launchpad.net/ubuntu/+source/aodh | 13:23 |
| EmilienM | jamespage: ty | 13:24 |
| EmilienM | jamespage: https://bugs.launchpad.net/ubuntu/+source/aodh/+bug/1508463 | 13:26 |
| openstack | Launchpad bug 1508463 in aodh (Ubuntu) "alembic.ini is missing from the aodh install source" [Undecided,New] | 13:26 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service https://review.openstack.org/237662 | 13:27 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service https://review.openstack.org/237662 | 13:27 |
| openstackgerrit | Merged openstack/puppet-glance: Change section name for AMQP qpid parameters https://review.openstack.org/237989 | 13:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Client class https://review.openstack.org/237706 | 13:30 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Client class https://review.openstack.org/237706 | 13:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service https://review.openstack.org/237708 | 13:31 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service https://review.openstack.org/237708 | 13:32 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service https://review.openstack.org/237709 | 13:33 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service https://review.openstack.org/237709 | 13:33 |
| *** paramite is now known as paramite|afk | 13:33 | |
| *** leanderthal has joined #puppet-openstack | 13:35 | |
| degorenko | EmilienM, this patch https://review.openstack.org/#/c/228744/ has broken all unit tests :) | 13:36 |
| degorenko | http://logs.openstack.org/90/237990/1/gate/gate-puppet-neutron-puppet-unit-3.3-dsvm-centos7/5438d5b/console.html#_2015-10-21_13_27_14_694 | 13:36 |
| EmilienM | I see that | 13:36 |
| degorenko | we need fix all keystone_auth specs | 13:36 |
| degorenko | EmilienM, have you start working on it or i can? | 13:37 |
| EmilienM | degorenko: no you can | 13:37 |
| degorenko | ok | 13:37 |
| EmilienM | degorenko: let's share the work | 13:37 |
| degorenko | yep | 13:37 |
| degorenko | which modules are your? | 13:37 |
| EmilienM | degorenko: https://etherpad.openstack.org/p/JvbbvOdFWX | 13:37 |
| degorenko | then i will take all modules which i fixed for oslo_qpid | 13:38 |
| degorenko | EmilienM, ^^ | 13:38 |
| EmilienM | degorenko: topic: puppet/fix-tests | 13:39 |
| degorenko | EmilienM, ack | 13:40 |
| *** paramite|afk is now known as paramite | 13:40 | |
| *** jpena|lunch is now known as jpena | 13:40 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-barbican: Update rspec tests for keyston https://review.openstack.org/238076 | 13:42 |
| EmilienM | degorenko: ^ | 13:42 |
| degorenko | EmilienM, typo in commit message btw :) | 13:43 |
| EmilienM | degorenko: fixing it | 13:43 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-barbican: Update rspec tests for keystone https://review.openstack.org/238076 | 13:43 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-heat: Update rspec tests for keystone https://review.openstack.org/238077 | 13:44 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Update rspec tests for keystone https://review.openstack.org/238078 | 13:44 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-ceilometer: Update rspec tests for keystone https://review.openstack.org/238079 | 13:45 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-ironic: Update rspec tests for keystone https://review.openstack.org/238080 | 13:45 |
| EmilienM | degorenko: I updated your list, I already did ceilometer | 13:45 |
| degorenko | EmilienM, i saw ;) | 13:45 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-gnocchi: Update rspec tests for keystone https://review.openstack.org/238081 | 13:46 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-neutron: Update rspec tests for keystone https://review.openstack.org/238082 | 13:46 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-manila: Update rspec tests for keystone https://review.openstack.org/238085 | 13:47 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-mistral: Update rspec tests for keystone https://review.openstack.org/238087 | 13:47 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-murano: Update rspec tests for keystone https://review.openstack.org/238088 | 13:48 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-cinder: Update rspec tests for keystone https://review.openstack.org/238089 | 13:48 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-cookiecutter: Update rspec tests for keystone https://review.openstack.org/238090 | 13:49 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-designate: Update rspec tests for keystone https://review.openstack.org/238091 | 13:50 |
| *** ducttape_ has joined #puppet-openstack | 13:50 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-trove: Update rspec tests for keystone https://review.openstack.org/238092 | 13:50 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-sahara: Update rspec tests for keystone https://review.openstack.org/238093 | 13:50 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tuskar: Update rspec tests for keystone https://review.openstack.org/238094 | 13:50 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-zaqar: Update rspec tests for keystone https://review.openstack.org/238095 | 13:51 |
| EmilienM | degorenko: we're done | 13:51 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-glance: Create glance::cache::logging class https://review.openstack.org/238096 | 13:51 |
| *** dsutyagin has joined #puppet-openstack | 13:51 | |
| EmilienM | waiting for CI now | 13:52 |
| EmilienM | review link: https://review.openstack.org/#/q/status:open+branch:master+topic:puppet/fix-tests,n,z | 13:52 |
| EmilienM | thx degorenko for the help | 13:52 |
| degorenko | EmilienM, yep :) no problem | 13:52 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: First implementation of init class https://review.openstack.org/237352 | 13:53 |
| openstackgerrit | Yanis Guenane proposed openstack/puppet-glance: Create glance::cache::logging class https://review.openstack.org/238096 | 13:54 |
| *** nihilifer has quit IRC | 13:55 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: First implementation of init class https://review.openstack.org/237352 | 13:57 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Update aodh::logging https://review.openstack.org/237356 | 13:57 |
| *** nihilifer has joined #puppet-openstack | 13:58 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement API service https://review.openstack.org/237587 | 14:00 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service https://review.openstack.org/237662 | 14:01 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Client class https://review.openstack.org/237706 | 14:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service https://review.openstack.org/237708 | 14:02 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service https://review.openstack.org/237709 | 14:03 |
| openstackgerrit | Merged openstack/puppet-openstack-cookiecutter: Update rspec tests for keystone https://review.openstack.org/238090 | 14:05 |
| *** tristanC has quit IRC | 14:07 | |
| *** jongchoi has joined #puppet-openstack | 14:09 | |
| openstackgerrit | Merged openstack/puppet-barbican: Update rspec tests for keystone https://review.openstack.org/238076 | 14:12 |
| degorenko | spredzy, sbadia take a look please on: https://review.openstack.org/#/c/237970/ | 14:13 |
| *** khushbu_ has quit IRC | 14:14 | |
| openstackgerrit | Merged openstack/puppet-aodh: Update rspec tests for keystone https://review.openstack.org/238078 | 14:15 |
| openstackgerrit | Merged openstack/puppet-ironic: Update rspec tests for keystone https://review.openstack.org/238080 | 14:18 |
| *** stamak has joined #puppet-openstack | 14:21 | |
| *** stamak has quit IRC | 14:23 | |
| *** stamak has joined #puppet-openstack | 14:23 | |
| openstackgerrit | Merged openstack/puppet-murano: Update rspec tests for keystone https://review.openstack.org/238088 | 14:27 |
| *** jongchoi has quit IRC | 14:28 | |
| *** sanjayu has quit IRC | 14:28 | |
| openstackgerrit | Merged openstack/puppet-gnocchi: Update rspec tests for keystone https://review.openstack.org/238081 | 14:29 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-heat: Update rspec tests for keystone https://review.openstack.org/238077 | 14:29 |
| *** bklei has joined #puppet-openstack | 14:35 | |
| *** dprince has quit IRC | 14:36 | |
| openstackgerrit | Merged openstack/puppet-designate: Update rspec tests for keystone https://review.openstack.org/238091 | 14:37 |
| openstackgerrit | Merged openstack/puppet-manila: Update rspec tests for keystone https://review.openstack.org/238085 | 14:37 |
| *** dfisher has joined #puppet-openstack | 14:37 | |
| *** xarses has joined #puppet-openstack | 14:43 | |
| openstackgerrit | Merged openstack/puppet-neutron: Update rspec tests for keystone https://review.openstack.org/238082 | 14:49 |
| openstackgerrit | Merged openstack/puppet-cinder: Update rspec tests for keystone https://review.openstack.org/238089 | 14:50 |
| *** kindjal has joined #puppet-openstack | 14:50 | |
| *** paramite has quit IRC | 14:53 | |
| openstackgerrit | Alexey Deryugin proposed openstack/puppet-murano: Add basic acceptance test https://review.openstack.org/233591 | 14:54 |
| *** mdorman has joined #puppet-openstack | 14:56 | |
| EmilienM | aderyugin: http://logs.openstack.org/91/233591/6/check/gate-puppet-murano-puppet-beaker-rspec-dsvm-centos7/78b44f4/console.html#_2015-10-21_11_27_25_582 | 14:59 |
| openstackgerrit | Merged openstack/puppet-ceilometer: Update rspec tests for keystone https://review.openstack.org/238079 | 14:59 |
| degorenko | EmilienM, we have also broken kilo branch :( | 15:00 |
| EmilienM | degorenko: what? | 15:00 |
| EmilienM | why? | 15:01 |
| degorenko | EmilienM, same error | 15:01 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Client class https://review.openstack.org/237706 | 15:01 |
| degorenko | https://review.openstack.org/237530 | 15:01 |
| degorenko | EmilienM, ^ | 15:01 |
| degorenko | that's awfully | 15:01 |
| EmilienM | https://review.openstack.org/#/q/project:openstack/puppet-keystone+branch:stable/kilo,n,z | 15:02 |
| EmilienM | I don't see why | 15:02 |
| *** stamak has quit IRC | 15:02 | |
| degorenko | укь | 15:02 |
| degorenko | erm | 15:02 |
| degorenko | that's strange | 15:02 |
| aderyugin | EmilienM: looks like there's no RPM packages for Murano, or I used a wrong repository... | 15:04 |
| degorenko | EmilienM, aderyugin - RPM packages for Murano ARE NOT READY yet. They are on review | 15:06 |
| EmilienM | degorenko: I know why | 15:09 |
| EmilienM | because of .fixtures.yaml, targeting keystone/master | 15:09 |
| EmilienM | we need to backport my patches about fixtures to stable/kilo | 15:10 |
| degorenko | EmilienM, that's was my thought :) | 15:11 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service https://review.openstack.org/237708 | 15:13 |
| EmilienM | degorenko: I can't do it atm | 15:14 |
| EmilienM | degorenko: go ahead if you can | 15:14 |
| openstackgerrit | Merged openstack/puppet-tuskar: Update rspec tests for keystone https://review.openstack.org/238094 | 15:15 |
| degorenko | EmilienM, ack. Is there common topic for your patches? | 15:15 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service https://review.openstack.org/237709 | 15:15 |
| EmilienM | degorenko: yes | 15:15 |
| *** xarses has quit IRC | 15:15 | |
| degorenko | good. I will take it then | 15:15 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Client class https://review.openstack.org/237706 | 15:15 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service https://review.openstack.org/237708 | 15:15 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-aodh: Implement Listener service https://review.openstack.org/237709 | 15:15 |
| degorenko | EmilienM, puppet/dropfixtures - right? | 15:15 |
| EmilienM | degorenko: yeah that one | 15:16 |
| degorenko | EmilienM, btw, we also have broken another branches? like juno? D: | 15:17 |
| EmilienM | yes | 15:17 |
| degorenko | ........ | 15:17 |
| EmilienM | this is why I wanted this feature (fixtures) | 15:17 |
| EmilienM | so we can test the real things | 15:17 |
| degorenko | ok, i will upload for all branches then | 15:18 |
| *** xarses has joined #puppet-openstack | 15:18 | |
| *** xarses has quit IRC | 15:18 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-nova: Don't add non-existent hosts to host aggregates https://review.openstack.org/236181 | 15:19 |
| *** xarses has joined #puppet-openstack | 15:19 | |
| *** dgurtner has quit IRC | 15:19 | |
| *** dgurtner has joined #puppet-openstack | 15:20 | |
| mfisch | degorenko: I've updated that code with some caching | 15:20 |
| mfisch | its better | 15:20 |
| mfisch | https://review.openstack.org/#/c/236181/ | 15:20 |
| mfisch | mdorman: ^ | 15:20 |
| degorenko | mfisch, \o/ will look | 15:21 |
| mfisch | I am not a ruby coder at all but that wasnt too difficult | 15:21 |
| mfisch | like using a static local in C | 15:21 |
| degorenko | mfisch, yeah, it looks like what we need | 15:23 |
| degorenko | will see what's say ci | 15:23 |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-glance: add more settings to glance api, registry, rabbitmq https://review.openstack.org/237035 | 15:26 |
| openstackgerrit | Mykyta Karpin proposed openstack/puppet-glance: Add support of os_service_default fact in glance spec https://review.openstack.org/237973 | 15:26 |
| *** xarses has quit IRC | 15:28 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-tuskar: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238140 | 15:30 |
| mdorman | thanks mfisch | 15:31 |
| EmilienM | degorenko: kilo / juno would be enough. We don't support icehouse afik | 15:31 |
| mfisch | degorenko: it will be way faster to cache that aggregate details call too | 15:32 |
| *** xarses has joined #puppet-openstack | 15:33 | |
| degorenko | EmilienM, ack. I've uploaded patch for tuskar for kilo. Will see ci answer | 15:33 |
| EmilienM | degorenko: oki | 15:34 |
| openstackgerrit | Merged openstack/puppet-zaqar: Update rspec tests for keystone https://review.openstack.org/238095 | 15:37 |
| EmilienM | degorenko: commented the tuskar patch. | 15:40 |
| openstackgerrit | Merged openstack/puppet-mistral: Update rspec tests for keystone https://review.openstack.org/238087 | 15:40 |
| degorenko | EmilienM, yeah, i saw :) | 15:41 |
| EmilienM | degorenko: http://logs.openstack.org/77/238077/2/check/gate-puppet-heat-puppet-beaker-rspec-dsvm-trusty/26c2d8a/console.html#_2015-10-21_15_10_52_896 | 15:44 |
| EmilienM | for https://review.openstack.org/#/c/238077/2 | 15:44 |
| *** iberezovskiy has quit IRC | 15:45 | |
| degorenko | EmilienM, looking | 15:46 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-tuskar: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238140 | 15:47 |
| EmilienM | I found it | 15:48 |
| degorenko | what was it? | 15:48 |
| EmilienM | a sec | 15:48 |
| EmilienM | let me cry first | 15:48 |
| degorenko | :D | 15:48 |
| EmilienM | degorenko: https://github.com/openstack/puppet-keystone/commit/8461e9acfbf7783e69489a4bdd6a24532bd4e33a?diff=unified#diff-18cdfe1a748b93f953fdd6b4c491ca46L179 | 15:49 |
| EmilienM | it used to create _member_ before | 15:49 |
| degorenko | oh, holly sh.... | 15:50 |
| EmilienM | richm: we need your insight here | 15:50 |
| chem | EmilienM: you have to create the _member_ and use keystone_user_role | 15:51 |
| *** gfidente is now known as gfidente|afk | 15:51 | |
| chem | EmilienM: I think ... | 15:51 |
| EmilienM | so gilles drop this role management by default | 15:51 |
| EmilienM | should we create this role again by default? | 15:51 |
| *** nlahouti has joined #puppet-openstack | 15:51 | |
| chem | EmilienM: I think it was created by default only to support the backward comptatibility. | 15:52 |
| EmilienM | mhhk | 15:52 |
| EmilienM | degorenko: I take over the heat patch and try something | 15:52 |
| chem | EmilienM: as user-create with role support dropped in openstack, so is the resource's support for it. | 15:52 |
| degorenko | ok, go ahead | 15:53 |
| chem | EmilienM: the default role _member_ was use to make sur the user creation would succeed when a tenant was specified | 15:53 |
| degorenko | but this is very funny - fix one problem and has one more instead | 15:54 |
| EmilienM | chem: maybe I can drop it from acceptance | 15:54 |
| EmilienM | chem: should we create _member_ ? | 15:54 |
| chem | EmilienM: I don't see why if nobody beside puppet use it. | 15:54 |
| EmilienM | ok | 15:54 |
| EmilienM | :) | 15:54 |
| degorenko | chem, as i know it uses in devstack | 15:55 |
| chem | degorenko: you mean the _member_ role exist in devstack ? | 15:55 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-heat: Fix rspec unit & beaker tests https://review.openstack.org/238077 | 15:55 |
| EmilienM | chem: degorenko ^ please look | 15:55 |
| richm | right | 15:55 |
| richm | when the patch was merged to remove the tenant parameter of keystone_user | 15:55 |
| EmilienM | richm: please review also ^ | 15:56 |
| richm | that also removed keystone_user creating _member_ role | 15:56 |
| *** rochaporto has quit IRC | 15:56 | |
| degorenko | chem, yep, exists :) so, this role unused everywhere? | 15:56 |
| richm | because that role is _only_ needed to add the user to the given tenant | 15:56 |
| *** bklei has quit IRC | 15:56 | |
| chem | degorenko: my guess would be they use it for the same reason. To make sure they can add user to project | 15:57 |
| degorenko | EmilienM, looks good | 15:57 |
| degorenko | chem, yes, it seems like that | 15:57 |
| EmilienM | degorenko: fyi, I'm off tomorrow & friday | 15:58 |
| *** EmilienM sets mode: +o degorenko | 15:59 | |
| *** EmilienM sets mode: +o crinkle | 15:59 | |
| *** EmilienM sets mode: +o mfisch | 15:59 | |
| *** EmilienM sets mode: +o mgagne | 15:59 | |
| degorenko | EmilienM, ack | 15:59 |
| degorenko | trip to tokyo? :) | 15:59 |
| EmilienM | degorenko: yep | 16:00 |
| EmilienM | looong trip from Quebec ! | 16:00 |
| openstackgerrit | Merged openstack/puppet-aodh: First implementation of init class https://review.openstack.org/237352 | 16:00 |
| openstackgerrit | Merged openstack/puppet-aodh: Update aodh::logging https://review.openstack.org/237356 | 16:00 |
| degorenko | i will have also looong trip from Moscow :) but first - to Moscow from Saratov :) | 16:00 |
| degorenko | btw, EmilienM can i change my topic on HND-puppet-code and add one more? :) | 16:01 |
| EmilienM | degorenko: do what you like Sir | 16:02 |
| openstackgerrit | Maksym Yatsenko proposed openstack/puppet-nova: [WIP] https://review.openstack.org/233011 | 16:02 |
| degorenko | EmilienM, :D ok, thanks | 16:02 |
| openstackgerrit | Merged openstack/puppet-aodh: Implement API service https://review.openstack.org/237587 | 16:03 |
| openstackgerrit | Merged openstack/puppet-neutron: Change section name for AMQP qpid parameters https://review.openstack.org/237990 | 16:08 |
| *** jistr has quit IRC | 16:08 | |
| *** dmsimard is now known as dmsimard|tokyo | 16:09 | |
| *** bnemec has quit IRC | 16:10 | |
| *** magicboiz has joined #puppet-openstack | 16:11 | |
| openstackgerrit | Merged openstack/puppet-sahara: Update rspec tests for keystone https://review.openstack.org/238093 | 16:13 |
| *** skolekonov is now known as _skolekonov | 16:14 | |
| *** bnemec has joined #puppet-openstack | 16:17 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-tempest: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238154 | 16:25 |
| EmilienM | degorenko: wait before patching all modules | 16:26 |
| EmilienM | we need to see if that works | 16:26 |
| mfisch | aderyugin: you here? | 16:26 |
| openstackgerrit | Merged openstack/puppet-cinder: Change section name for AMQP qpid parameters https://review.openstack.org/237997 | 16:26 |
| EmilienM | degorenko: it's not working | 16:26 |
| degorenko | EmilienM, yes, just want be sure that it works/doesn't work for at least two modules | 16:26 |
| degorenko | :( | 16:26 |
| EmilienM | we need another backport | 16:26 |
| EmilienM | I take care of it | 16:26 |
| degorenko | which one? | 16:26 |
| degorenko | ok | 16:26 |
| aderyugin | Hi, mfisch! | 16:27 |
| *** logan2 has quit IRC | 16:27 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Create install_modules_unit.sh https://review.openstack.org/238155 | 16:27 |
| *** dprince has joined #puppet-openstack | 16:27 | |
| mfisch | aderyugin: hey over the next few weeks as time permits I'm going to work on getting murano running in docker, that means there's some work to do in the puppet module like was done for puppet-designate | 16:28 |
| openstackgerrit | Merged openstack/puppet-trove: Update rspec tests for keystone https://review.openstack.org/238092 | 16:28 |
| openstackgerrit | Sergey Vasilenko proposed openstack/puppet-neutron: Use json output instead plain-text https://review.openstack.org/238156 | 16:28 |
| mfisch | aderyugin: something like this | 16:28 |
| mfisch | https://github.com/openstack/puppet-designate/commit/5a37cc81276cb8f8ee6dca9b9b532930e6ac86de | 16:28 |
| aderyugin | mfisch: ok, what should be done from side? | 16:29 |
| EmilienM | degorenko: wait before reviewing | 16:29 |
| EmilienM | we need some backports, a lot | 16:29 |
| *** xarses has quit IRC | 16:29 | |
| degorenko | EmilienM, waiting for ci of course :) | 16:29 |
| mfisch | aderyugin: well I know you are working on some parameter names and things, I wanted you to know that this was also coming at some point | 16:29 |
| mfisch | Murano is a lower priority for me, so I work on it when my Data center move work is stuck which happens more often than it should | 16:29 |
| *** trown is now known as trown|lunch | 16:30 | |
| degorenko | mfisch, you can also ping me :) if you need some help. We are working with aderyugin in one location, fyi | 16:30 |
| mfisch | sure thing | 16:30 |
| mfisch | where are you guys located? | 16:30 |
| mfisch | Moscow? | 16:30 |
| degorenko | Saratov, Russia | 16:30 |
| * mfisch googles | 16:31 | |
| degorenko | but you probably don't know where is it :D | 16:31 |
| *** zhangjn has joined #puppet-openstack | 16:31 | |
| mfisch | ah the volga was wondering thats a big river | 16:31 |
| degorenko | yep :) | 16:31 |
| mfisch | and I see "Engels" a good socialist city name ;) | 16:31 |
| degorenko | exactly :D | 16:31 |
| degorenko | also we have in our region 'Marks' | 16:32 |
| *** derekh has quit IRC | 16:32 | |
| mfisch | surely | 16:32 |
| *** derekh has joined #puppet-openstack | 16:34 | |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Give execute permissions to install_modules.sh https://review.openstack.org/238157 | 16:34 |
| *** vinsh has joined #puppet-openstack | 16:36 | |
| chem | richm: I'm implementing the keystone_ldap_backend. There is a point in the trello that says "The keystone::ldap will be deprecated and implemented using the new keystone::ldap_backend resource" | 16:38 |
| chem | richm: I think it's a mistake... | 16:38 |
| mfisch | degorenko: the caching broke some tests :( | 16:38 |
| degorenko | mfisch, that's not good :( | 16:38 |
| chem | richm: if I do that this trigger a unforseen restart of keystone | 16:38 |
| EmilienM | crinkle: we need to backport the integration scripts into stable/kilo. There are a lot of backports. Does it make sense to cherry-picka all of them? | 16:39 |
| mfisch | it was expecting a call but its cached now | 16:39 |
| mfisch | fixing tests | 16:39 |
| EmilienM | crinkle: or copy/paste the scripts in stable/kilo | 16:39 |
| EmilienM | crinkle: it's about ~10 backports | 16:39 |
| chem | richm: like I'm a operator, I've upgraded puppet, but nothing has changed on the manifest side but I get a restart of my keystone server | 16:39 |
| chem | richm: I think keystone::ldap should remain the same only with a depreciation warning | 16:40 |
| chem | richm: wdyt ? | 16:40 |
| mfisch | anyone know what the opposite of klass.expects()? | 16:40 |
| *** zhangjn has quit IRC | 16:41 | |
| degorenko | mfisch, it expects that will be called auth nova - but it didn't | 16:41 |
| mfisch | it called it once | 16:42 |
| mfisch | but not the 2nd time | 16:42 |
| mfisch | caching in action | 16:42 |
| richm | chem: not sure I understand - if keystone::ldap remains the same, then it will be a duplicate of keystone::ldap_backend | 16:43 |
| richm | how would that trigger a restart? | 16:43 |
| degorenko | mfisch, you need fix tests | 16:43 |
| degorenko | line 197 | 16:43 |
| degorenko | or | 16:44 |
| degorenko | no | 16:44 |
| degorenko | 234 | 16:44 |
| mfisch | yes I'm there | 16:44 |
| mfisch | it has 2 klass.expects(:auth_nova) lines | 16:44 |
| chem | richm: arghh... oki. first: if keystone::ldap remains the same it's a class completely different from the keystone::ldap_config define so no duplicate here. | 16:44 |
| mfisch | the 2nd one wont get called because of caching | 16:44 |
| mfisch | so I'm looking for how to say | 16:44 |
| mfisch | klass.not_expects() | 16:44 |
| mfisch | got it | 16:45 |
| degorenko | works? :) | 16:45 |
| *** xarses has joined #puppet-openstack | 16:45 | |
| mfisch | klass.expects(:auth_nova).never | 16:45 |
| mfisch | probanly | 16:45 |
| mfisch | probably | 16:45 |
| mfisch | I dont know ruby or mocha so a bit of learning | 16:45 |
| chem | richm: second: the restart would be triggered because keystone::ldap_backend is using keystone_domain_config. For this which should setup the domain_config_dir and the like and destroy the equivalent resource in keystone_config | 16:45 |
| degorenko | yea, it is kind of tricky | 16:46 |
| richm | chem: hmm | 16:46 |
| chem | richm: I'm going to post the code as it is, with a comment, and I'll refactor tomorrow. | 16:47 |
| *** logan2 has joined #puppet-openstack | 16:47 | |
| richm | chem: I see - ok then - I was just hoping to prevent a lot of code duplication - I suppose you could use ensure_resource, and ::ldap would use 'keystone_config' as the resource name, and ::ldap_config would use 'keystone_domain_config' | 16:48 |
| openstackgerrit | Matt Fischer proposed openstack/puppet-nova: Don't add non-existent hosts to host aggregates https://review.openstack.org/236181 | 16:48 |
| mfisch | degorenko: updated | 16:48 |
| mfisch | nice to have a test for the cachine too | 16:48 |
| mfisch | caching | 16:48 |
| degorenko | awesome :) | 16:48 |
| degorenko | waiting for ci :) | 16:48 |
| mfisch | worked locally | 16:49 |
| chem | richm: yeah, something like this. I try very hard to avoid code duplication. But in the operators don't care about "beautiful" code :) So I'll post my code and I'll refactor tomorrow. There is a intersting hack to pass the 97 parameters to the keystone::ldap_config that's worth looking at ... | 16:50 |
| richm | chem: ok | 16:50 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration: Update module install scripts from master https://review.openstack.org/238162 | 16:50 |
| EmilienM | degorenko: too much backports, I think we could do that ^ | 16:50 |
| richm | chem: I guess it doesn't matter anyway, since we won't be changing keystone::ldap - any new fixes, features will go into ldap_config | 16:51 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-cinder: Change section name for AMQP rabbit parameters https://review.openstack.org/238163 | 16:51 |
| openstackgerrit | Athlan-Guyot sofer proposed openstack/puppet-keystone: Support for multiple ldap backend. https://review.openstack.org/238164 | 16:51 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-tuskar: Try to use zuul-cloner to prepare fixtures https://review.openstack.org/238140 | 16:51 |
| EmilienM | degorenko: I added the Depends-On ^ to see if it works | 16:52 |
| degorenko | EmilienM, ok | 16:52 |
| openstackgerrit | Emilien Macchi proposed openstack/puppet-keystone: CI test - DNM https://review.openstack.org/238165 | 16:52 |
| EmilienM | degorenko: testing it here ^ so we can see beaker jobs results | 16:52 |
| richm | chem: nice hack! | 16:53 |
| chem | richm: too bad, It's going to disappear :) | 16:53 |
| degorenko | EmilienM, will ci what will say ci :) | 16:54 |
| degorenko | see* | 16:54 |
| mfisch | you guys are redoing my ldap code? my very first contrib and some of my very first puppet code ever <tear> | 16:57 |
| *** zigo_ has quit IRC | 16:59 | |
| richm | mfisch: yeah - not really redoing it, just making it work for multiple domains | 16:59 |
| *** derekh has quit IRC | 17:00 | |
| *** zigo has joined #puppet-openstack | 17:00 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-glance: Change section name for AMQP rabbit parameters https://review.openstack.org/238169 | 17:01 |
| EmilienM | degorenko: that means for stable/juno we'll have to make the change manually in each repo | 17:04 |
| EmilienM | degorenko: because puppet-openstack-integration does not have a stable/juno branch | 17:05 |
| degorenko | EmilienM, nice :) | 17:05 |
| EmilienM | degorenko: let's fix kilo first | 17:05 |
| degorenko | yep | 17:05 |
| EmilienM | we will figure out how to fix juno later | 17:05 |
| degorenko | i guess, we can found somebody who can help us | 17:05 |
| degorenko | agreed | 17:05 |
| *** dsutyagin has quit IRC | 17:10 | |
| *** vinsh has quit IRC | 17:12 | |
| *** vinsh has joined #puppet-openstack | 17:12 | |
| *** jpena has quit IRC | 17:13 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-heat: Change section name for AMQP rabbit parameters https://review.openstack.org/238174 | 17:15 |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-ironic: Change section name for AMQP rabbit parameters https://review.openstack.org/238175 | 17:15 |
| *** vinsh has quit IRC | 17:17 | |
| openstackgerrit | Denis Egorenko proposed openstack/puppet-nova: Change section name for AMQP rabbit parameters https://review.openstack.org/238177 | 17:21 |
| degorenko | today we make ci very busy :) | 17:21 |
| EmilienM | yeah | 17:22 |
| *** pasquier-s has quit IRC | 17:25 | |
| *** pasquier-s has joined #puppet-openstack | 17:27 | |
| degorenko | i'm off for today :) good night! | 17:27 |
| *** degorenko is now known as _degorenko | 17:27 | |
| *** rahulait has joined #puppet-openstack | 17:28 | |
| *** trown|lunch is now known as trown | 17:28 | |
| *** xarses has quit IRC | 17:29 | |
| EmilienM | degorenko: thx for your help man | 17:29 |
| EmilienM | degorenko: good night | 17:29 |
| *** tchaypo has quit IRC | 17:29 | |
| *** tchaypo has joined #puppet-openstack | 17:31 | |
| *** dgurtner has quit IRC | 17:31 | |
| *** ducttape_ has quit IRC | 17:35 | |
| *** kindjal_ has joined #puppet-openstack | 17:43 | |
| *** kindjal has quit IRC | 17:45 | |
| *** kindjal_ is now known as kindjal | 17:45 | |
| *** kindjal has quit IRC | 17:53 | |
| mfisch | _degorenko: tests passed when you have a minute | 17:56 |
| *** nlahouti has quit IRC | 18:00 | |
| *** nlahouti has joined #puppet-openstack | 18:02 | |
| openstackgerrit | Rohit Pagedar proposed openstack/puppet-nova: Making instance_name_template configurable. https://review.openstack.org/234972 | 18:05 |
| openstackgerrit | Rohit Pagedar proposed openstack/puppet-nova: Making instance_name_template configurable. https://review.openstack.org/234972 | 18:09 |
| *** xarses has joined #puppet-openstack | 18:10 | |
| *** xarses has quit IRC | 18:10 | |
| *** ducttape_ has joined #puppet-openstack | 18:11 | |
| *** xarses has joined #puppet-openstack | 18:11 | |
| *** xarses has quit IRC | 18:11 | |
| *** xarses has joined #puppet-openstack | 18:12 | |
| *** xarses_ has joined #puppet-openstack | 18:13 | |
| *** xarses has quit IRC | 18:16 | |
| *** dgurtner has joined #puppet-openstack | 18:26 | |
| *** dgurtner has quit IRC | 18:26 | |
| *** dgurtner has joined #puppet-openstack | 18:26 | |
| *** angdraug has joined #puppet-openstack | 18:38 | |
| *** vinsh has joined #puppet-openstack | 19:01 | |
| *** paramite has joined #puppet-openstack | 19:01 | |
| *** rahulait has quit IRC | 19:02 | |
| *** vinsh has quit IRC | 19:03 | |
| *** dgurtner has quit IRC | 19:08 | |
| *** kindjal has joined #puppet-openstack | 19:10 | |
| *** xarses_ has quit IRC | 19:13 | |
| *** xarses has joined #puppet-openstack | 19:14 | |
| *** linkedinyou has joined #puppet-openstack | 19:15 | |
| *** gfidente|afk has quit IRC | 19:20 | |
| EmilienM | can anyone look at https://review.openstack.org/#/c/238162/ ? | 19:20 |
| EmilienM | sorry for asking, trying to unblock our stable branch CI | 19:20 |
| EmilienM | crinkle, mfisch, clayton, sbadia ^ | 19:20 |
| mfisch | so one big one instead of backports | 19:23 |
| *** nlahouti has quit IRC | 19:23 | |
| openstackgerrit | Matt Fischer proposed openstack/puppet-nova: Don't add non-existent hosts to host aggregates https://review.openstack.org/236181 | 19:24 |
| *** zacdev has joined #puppet-openstack | 19:24 | |
| *** nlahouti has joined #puppet-openstack | 19:26 | |
| *** xarses_ has joined #puppet-openstack | 19:35 | |
| *** nlahouti has quit IRC | 19:36 | |
| *** nlahouti has joined #puppet-openstack | 19:37 | |
| *** xarses has quit IRC | 19:38 | |
| openstackgerrit | Sergey Vasilenko proposed openstack/puppet-neutron: Use json output instead plain-text https://review.openstack.org/238156 | 19:39 |
| *** nlahouti has quit IRC | 19:40 | |
| *** rahulait has joined #puppet-openstack | 19:40 | |
| *** nlahouti has joined #puppet-openstack | 19:45 | |
| *** nlahouti has quit IRC | 19:48 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration: Update module install scripts from master https://review.openstack.org/238162 | 19:53 |
| EmilienM | _degorenko: backport puppet/dropfixtures to stable/kilo is now possible | 19:56 |
| *** kindjal has quit IRC | 20:08 | |
| *** xarses has joined #puppet-openstack | 20:09 | |
| *** xarses_ has quit IRC | 20:10 | |
| openstackgerrit | Merged openstack/puppet-heat: Fix rspec unit & beaker tests https://review.openstack.org/238077 | 20:11 |
| *** xarses has quit IRC | 20:40 | |
| *** xarses has joined #puppet-openstack | 20:40 | |
| *** angdraug has quit IRC | 20:40 | |
| openstackgerrit | Merged openstack/puppet-ceilometer: Change section name for AMQP qpid parameters https://review.openstack.org/238002 | 20:41 |
| *** paramite has quit IRC | 20:44 | |
| *** nlahouti has joined #puppet-openstack | 20:48 | |
| *** dgurtner has joined #puppet-openstack | 20:49 | |
| *** kindjal has joined #puppet-openstack | 21:00 | |
| *** dprince has quit IRC | 21:07 | |
| *** jongchoi_ has joined #puppet-openstack | 21:11 | |
| *** kindjal_ has joined #puppet-openstack | 21:14 | |
| *** kindjal_ has quit IRC | 21:15 | |
| *** kindjal has quit IRC | 21:15 | |
| *** trown is now known as trown|outttypeww | 21:28 | |
| *** xarses_ has joined #puppet-openstack | 21:30 | |
| *** xarses has quit IRC | 21:31 | |
| *** rahulait has quit IRC | 21:39 | |
| EmilienM | mgagne: https://github.com/openstack/puppet-openstack-release-tools is up - I'll let you push your tools, if you don't mind | 21:40 |
| mgagne | EmilienM: copy/paste in there? | 21:40 |
| EmilienM | mgagne: whatever you like | 21:41 |
| EmilienM | mgagne: maybe in tools/ I don't know | 21:41 |
| EmilienM | I have no idea :) | 21:41 |
| mgagne | EmilienM: we can rename later | 21:41 |
| mgagne | lets import and see what happens | 21:41 |
| EmilienM | yeah | 21:41 |
| EmilienM | oki | 21:41 |
| EmilienM | mgagne: there is no CI, it's noop jobs | 21:41 |
| openstackgerrit | Mathieu Gagné proposed openstack/puppet-openstack-release-tools: Initial import from upstream https://review.openstack.org/238259 | 21:45 |
| mgagne | EmilienM: ^ | 21:45 |
| EmilienM | mgagne: thanks! I'll push my tools on top of it, and add some doc | 21:46 |
| mgagne | sure | 21:47 |
| *** ducttape_ has quit IRC | 21:49 | |
| *** morazi has quit IRC | 21:56 | |
| *** jongchoi_ has quit IRC | 22:02 | |
| *** jongchoi_ has joined #puppet-openstack | 22:06 | |
| *** zacdev has quit IRC | 22:07 | |
| *** jongchoi_ has quit IRC | 22:10 | |
| *** rcarrillocruz has quit IRC | 22:18 | |
| *** rcarrillocruz has joined #puppet-openstack | 22:22 | |
| *** dgurtner has quit IRC | 22:22 | |
| *** dgurtner has joined #puppet-openstack | 22:27 | |
| *** dgurtner has quit IRC | 22:27 | |
| *** dgurtner has joined #puppet-openstack | 22:27 | |
| *** jayg is now known as jayg|g0n3 | 22:30 | |
| *** dgurtner has quit IRC | 22:48 | |
| *** xarses_ has quit IRC | 22:48 | |
| openstackgerrit | Sam Morrison proposed openstack/puppet-heat: Support num_engine_workers variable https://review.openstack.org/237866 | 22:51 |
| *** dfisher has quit IRC | 22:52 | |
| openstackgerrit | Sam Morrison proposed openstack/puppet-heat: Fix up doc string for workers variable https://review.openstack.org/238268 | 22:53 |
| *** xarses_ has joined #puppet-openstack | 23:00 | |
| openstackgerrit | Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider https://review.openstack.org/203220 | 23:01 |
| *** xarses has joined #puppet-openstack | 23:05 | |
| *** xarses_ has quit IRC | 23:05 | |
| *** markvoelker has quit IRC | 23:08 | |
| *** ducttape_ has joined #puppet-openstack | 23:19 | |
| *** ducttape_ has quit IRC | 23:20 | |
| *** mdorman has quit IRC | 23:21 | |
| *** gildub has joined #puppet-openstack | 23:41 | |
| *** ducttape_ has joined #puppet-openstack | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!