| *** rcernin has quit IRC | 02:40 | |
| *** rcernin has joined #puppet-openstack | 03:14 | |
| *** rcernin has quit IRC | 03:19 | |
| *** rcernin has joined #puppet-openstack | 03:19 | |
| *** hjensas has joined #puppet-openstack | 05:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.opendev.org/749183 | 06:17 |
|---|---|---|
| *** jtomasek has joined #puppet-openstack | 07:12 | |
| *** tosky has joined #puppet-openstack | 07:21 | |
| *** rcernin has quit IRC | 07:36 | |
| *** jpena|off is now known as jpena | 07:52 | |
| *** rcernin has joined #puppet-openstack | 08:00 | |
| *** rcernin has quit IRC | 08:02 | |
| *** gfidente has joined #puppet-openstack | 08:32 | |
| *** derekh has joined #puppet-openstack | 08:46 | |
| *** dtantsur|afk is now known as dtantsur | 09:05 | |
| *** tkajinam has quit IRC | 09:54 | |
| *** tkajinam has joined #puppet-openstack | 09:55 | |
| *** mgagne has quit IRC | 10:39 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-placement master: Do not include placement::config by default https://review.opendev.org/749005 | 10:50 |
| *** tonyb has quit IRC | 11:05 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Use keymgr_* parameters in cinder class https://review.opendev.org/749244 | 11:25 |
| *** tonyb has joined #puppet-openstack | 11:31 | |
| *** jpena is now known as jpena|lunch | 11:34 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: [DNM] Test Ubuntu Focal https://review.opendev.org/741266 | 11:48 |
| EmilienM | tobias-urdin: hey, i commented on https://review.opendev.org/#/c/740602/ | 12:09 |
| tobias-urdin | EmilienM: ack, will investigate | 12:12 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Add Puppet Litmus and remove Puppet 5 testing https://review.opendev.org/740602 | 12:22 |
| tobias-urdin | EmilienM: ^ should get normal output, to get complete debug we need to update apply_manifest() call in modules and set :debug param | 12:23 |
| tobias-urdin | but we didn't get debug for beaker so i think it's fine, otherwise i can update all module related patches | 12:23 |
| tobias-urdin | also updates the commit msg | 12:24 |
| EmilienM | ok | 12:24 |
| tobias-urdin | recheck done on puppet-nova change https://review.opendev.org/#/c/740603/ to verify output | 12:24 |
| EmilienM | thanks | 12:24 |
| *** jpena|lunch is now known as jpena | 12:37 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Use keymgr_* parameters in cinder class https://review.opendev.org/749244 | 12:37 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Use keymgr_* parameters in cinder class https://review.opendev.org/749244 | 12:38 |
| *** derekh has quit IRC | 12:39 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.opendev.org/749183 | 12:50 |
| *** derekh has joined #puppet-openstack | 12:54 | |
| EmilienM | tobias-urdin: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_42a/740603/6/check/puppet-openstack-litmus-centos-8/42a93de/job-output.txt | 12:57 |
| EmilienM | it seems better | 12:57 |
| EmilienM | there is an error though (but it didn't make the job failing) | 12:57 |
| EmilienM | Error: Evaluation Error: Error while evaluating a Resource Statement, Unknown resource type: 'selboolean' (file: /home/zuul/workspace/spec/fixtures/modules/openstack_integration/manifests/neutron.pp, line: 47, column: 5) on node centos-8-inap-mtl01-0019573867[0m | 12:57 |
| EmilienM | 2020-09-01 12:36:45.998597 | centos-8 | [mNotice: Accepting previously invalid value for target type 'Enum['sql', 'template']' | 12:57 |
| tobias-urdin | EmilienM: ack, checking | 13:01 |
| tkajinam | https://github.com/puppetlabs/puppetlabs-stdlib/commit/f1edd271 | 13:04 |
| tkajinam | maybe we should update this ? https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L490 | 13:04 |
| tkajinam | it seems that validation is ignored now | 13:05 |
| tkajinam | tobias-urdin, EmilienM ^^ | 13:05 |
| tkajinam | though this is not directly related litmus but just version of stdlib we use | 13:05 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Add Puppet Litmus and remove Puppet 5 testing https://review.opendev.org/740602 | 13:06 |
| tobias-urdin | ^ let's see if this helps | 13:07 |
| tobias-urdin | tkajinam: hm what do we need to change? is it complaining about a undef value? can't see anywhere we it's not hardcoded to 'sql' | 13:10 |
| tkajinam | tobias-urdin, oh, maybe I misunderstood the change. so it lowered down log level from warning to notice so maybe we can just ignore the message ? | 13:16 |
| tobias-urdin | tkajinam: imo that warning should never be there unless we've set the value wrong somewhere, but seeing the if ! $catalog_driver makes me think we are changing the default 'sql' somewhere | 13:17 |
| tobias-urdin | can't find where tho | 13:17 |
| tkajinam | yeah | 13:18 |
| tkajinam | it seems the message was added to inform the change in validation logic in validate_legacy so we should be fine as far as the current logic works well for us | 13:18 |
| tkajinam | I mean, the message was added with warn level first, and then its level was lowered down to notice | 13:19 |
| tkajinam | we can just ignore it. it's not failure but a kind of trivial message . | 13:19 |
| openstackgerrit | Merged openstack/puppet-keystone stable/train: Convert more to rspec-puppet-facts https://review.opendev.org/748952 | 13:26 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-keystone master: Add support to override config group for authtoken https://review.opendev.org/749284 | 13:32 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-magnum master: Fix testing for magnum::keystone::authtoken https://review.opendev.org/749288 | 13:47 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-magnum master: Set keystone_auth section in magnum::keystone::authtoken https://review.opendev.org/749289 | 13:53 |
| openstackgerrit | Merged openstack/puppet-gnocchi master: Do not check distro names in each manifests https://review.opendev.org/749047 | 14:00 |
| openstackgerrit | Merged openstack/puppet-keystone stable/train: Fix python package names (ldappool and pysaml2) https://review.opendev.org/748953 | 14:04 |
| openstackgerrit | Merged openstack/puppet-gnocchi master: Remove deprecated database_connection in init https://review.opendev.org/748928 | 14:18 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Revert "Revert "Make CentOS8 beaker job voting"" https://review.opendev.org/749298 | 14:44 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-magnum master: Fix testing for magnum::keystone::authtoken https://review.opendev.org/749288 | 14:52 |
| *** dtantsur is now known as dtantsur|afk | 15:18 | |
| openstackgerrit | Merged openstack/puppet-nova master: Do not validate database_connection format https://review.opendev.org/749019 | 15:29 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Replace gem --no-ri with --no-document https://review.opendev.org/749350 | 15:57 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack_spec_helper master: Replace gem --no-ri with --no-document https://review.opendev.org/749351 | 15:58 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-guide master: Replace gem --no-ri with --no-document https://review.opendev.org/749352 | 15:58 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-cookiecutter master: Replace gem --no-ri with --no-document https://review.opendev.org/749353 | 15:59 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: [DNM] Test Ubuntu Focal https://review.opendev.org/741266 | 15:59 |
| tobias-urdin | tkajinam: weird, didn't we fix the libvirt-tcp.socket stuff https://review.opendev.org/#/c/740603/ | 16:00 |
| tobias-urdin | see log for litmus c8 test: https://zuul.opendev.org/t/openstack/build/5148f361334446228235430630381912/log/job-output.txt | 16:00 |
| *** bnemec has quit IRC | 16:06 | |
| *** bnemec has joined #puppet-openstack | 16:06 | |
| openstackgerrit | Merged openstack/puppet-placement master: Do not include placement::config by default https://review.opendev.org/749005 | 16:48 |
| *** derekh has quit IRC | 16:51 | |
| *** gfidente has quit IRC | 17:04 | |
| *** jpena is now known as jpena|off | 17:17 | |
| *** jtomasek has quit IRC | 17:26 | |
| *** tonyb has quit IRC | 17:44 | |
| *** tonyb has joined #puppet-openstack | 17:46 | |
| openstackgerrit | Merged openstack/puppet-oslo master: Support overriding resource type used in oslo::db class https://review.opendev.org/749034 | 18:01 |
| openstackgerrit | Merged openstack/puppet-placement master: Deprecate unused parameters in placement::api https://review.opendev.org/749065 | 19:15 |
| *** openstackgerrit has quit IRC | 19:21 | |
| *** Tuor has quit IRC | 19:57 | |
| *** Tuor has joined #puppet-openstack | 19:57 | |
| *** mkarpin has quit IRC | 21:16 | |
| *** mkarpin has joined #puppet-openstack | 21:21 | |
| *** openstackgerrit has joined #puppet-openstack | 22:42 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-magnum master: Fix testing for magnum::keystone::authtoken https://review.opendev.org/749288 | 22:42 |
| *** tosky has quit IRC | 22:43 | |
| *** rcernin has joined #puppet-openstack | 23:01 | |
| *** hjensas has quit IRC | 23:02 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!