| *** tosky has quit IRC | 00:16 | |
| *** sai_p has quit IRC | 00:55 | |
| openstackgerrit | Merged openstack/puppet-cloudkitty master: Deprecate pki related options https://review.openstack.org/619402 | 01:05 |
|---|---|---|
| openstackgerrit | Merged openstack/puppet-octavia master: Deprecate pki related options https://review.openstack.org/619403 | 01:24 |
| openstackgerrit | Merged openstack/puppet-trove master: Deprecate pki related options https://review.openstack.org/619405 | 01:58 |
| *** mschuppert has quit IRC | 02:27 | |
| openstackgerrit | Merged openstack/puppet-mistral master: Deprecate pki related options https://review.openstack.org/619404 | 02:34 |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Blacklist tempest_horizon on Debian based https://review.openstack.org/619684 | 03:10 |
| *** ykarel|away has joined #puppet-openstack | 03:57 | |
| *** ykarel|away is now known as ykarel | 04:06 | |
| *** ykarel has quit IRC | 04:46 | |
| *** ykarel has joined #puppet-openstack | 05:03 | |
| *** dpawlik has joined #puppet-openstack | 06:47 | |
| openstackgerrit | Merged openstack/puppet-rally master: Convert to rspec-puppet-facts https://review.openstack.org/619894 | 06:50 |
| openstackgerrit | Merged openstack/puppet-panko master: Convert to rspec-puppet-facts https://review.openstack.org/619893 | 06:52 |
| openstackgerrit | Merged openstack/puppet-oslo master: Convert to rspec-puppet-facts https://review.openstack.org/619891 | 06:53 |
| *** dpawlik has quit IRC | 06:54 | |
| *** rcernin has quit IRC | 06:58 | |
| openstackgerrit | Merged openstack/puppet-magnum master: Convert to rspec-puppet-facts https://review.openstack.org/619884 | 06:58 |
| *** mschuppert has joined #puppet-openstack | 07:07 | |
| *** dpawlik has joined #puppet-openstack | 07:23 | |
| *** hjensas has joined #puppet-openstack | 07:34 | |
| *** ykarel is now known as ykarel|lunch | 07:35 | |
| *** dpawlik has quit IRC | 07:38 | |
| *** dpawlik has joined #puppet-openstack | 07:44 | |
| zigo | I'm having truobles with the ceph module. It gets stuck injecting the admin key, doing: | 08:07 |
| zigo | ceph --name 'mon.' --keyring '/var/lib/ceph/mon/ceph-swift01-cephmon-1.infomaniak.ch/keyring' auth get client.admin -o $OLD_KEYRING || true | 08:07 |
| zigo | What am I doing wrong? | 08:07 |
| zigo | Why can't it get the key? | 08:07 |
| *** jtomasek has joined #puppet-openstack | 08:23 | |
| *** shardy has joined #puppet-openstack | 08:25 | |
| *** ykarel|lunch is now known as ykarel | 08:32 | |
| *** tosky has joined #puppet-openstack | 08:38 | |
| *** jpena|off is now known as jpena | 08:51 | |
| *** gfidente has joined #puppet-openstack | 09:05 | |
| *** skramaja has joined #puppet-openstack | 09:30 | |
| *** derekh has joined #puppet-openstack | 09:40 | |
| *** rpittau has joined #puppet-openstack | 10:33 | |
| *** salmankhan has joined #puppet-openstack | 11:35 | |
| *** salmankhan1 has joined #puppet-openstack | 11:38 | |
| *** salmankhan has quit IRC | 11:40 | |
| *** salmankhan1 is now known as salmankhan | 11:40 | |
| *** salmankhan has quit IRC | 11:42 | |
| *** gfidente has quit IRC | 11:47 | |
| *** gfidente has joined #puppet-openstack | 11:56 | |
| *** rpittau_ has joined #puppet-openstack | 12:16 | |
| *** rpittau has quit IRC | 12:19 | |
| *** amab_ has joined #puppet-openstack | 12:19 | |
| *** vnogin has joined #puppet-openstack | 12:21 | |
| *** amab_ has left #puppet-openstack | 12:22 | |
| *** vnogin has quit IRC | 12:22 | |
| *** rpittau has joined #puppet-openstack | 12:24 | |
| *** rpittau_ has quit IRC | 12:28 | |
| *** salmankhan has joined #puppet-openstack | 12:34 | |
| *** jpena is now known as jpena|lunch | 12:36 | |
| *** ykarel is now known as ykarel|away | 12:42 | |
| *** morazi has joined #puppet-openstack | 12:46 | |
| *** vnogin has joined #puppet-openstack | 12:51 | |
| *** trown has quit IRC | 12:51 | |
| *** trown|brb has joined #puppet-openstack | 12:53 | |
| *** ykarel|away has quit IRC | 12:56 | |
| *** rpittau_ has joined #puppet-openstack | 13:08 | |
| *** rpittau has quit IRC | 13:11 | |
| *** salmankhan has quit IRC | 13:12 | |
| *** salmankhan has joined #puppet-openstack | 13:18 | |
| *** rpittau_ is now known as rpittau | 13:20 | |
| *** salmankhan has quit IRC | 13:24 | |
| *** gfidente has quit IRC | 13:28 | |
| *** ykarel|away has joined #puppet-openstack | 13:28 | |
| *** gfidente has joined #puppet-openstack | 13:29 | |
| *** jpena|lunch is now known as jpena | 13:38 | |
| *** vnogin has quit IRC | 13:41 | |
| *** trown|brb is now known as trown | 13:45 | |
| openstackgerrit | Merged openstack/puppet-keystone master: Remove deprecated class keystone::python https://review.openstack.org/619378 | 13:50 |
| *** ykarel|away has quit IRC | 14:16 | |
| *** vnogin has joined #puppet-openstack | 14:32 | |
| openstackgerrit | Pierre Gaxatte proposed openstack/puppet-mistral master: Add missing database_* arguments in init.pp https://review.openstack.org/620333 | 14:37 |
| openstackgerrit | Carlos Goncalves proposed openstack/puppet-octavia master: Re-include octavia::db in octavia::api https://review.openstack.org/620147 | 14:43 |
| openstackgerrit | Carlos Goncalves proposed openstack/puppet-octavia master: Re-include octavia::db in octavia::api https://review.openstack.org/620147 | 14:44 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-glance master: Update python packaging version for RHEL8 https://review.openstack.org/620150 | 14:56 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-glance master: Update python packaging version for RHEL8 https://review.openstack.org/620150 | 14:57 |
| *** salmankhan has joined #puppet-openstack | 14:58 | |
| *** salmankhan has quit IRC | 14:58 | |
| *** dpawlik has quit IRC | 15:04 | |
| *** skramaja has quit IRC | 15:13 | |
| *** salmankhan has joined #puppet-openstack | 15:17 | |
| openstackgerrit | Rajini Karthik proposed openstack/puppet-cinder stable/rocky: Dell EMC SC: Add use_multipath_for_image_xfer https://review.openstack.org/614618 | 15:18 |
| openstackgerrit | Rajini Karthik proposed openstack/puppet-cinder stable/queens: Dell EMC SC: Add use_multipath_for_image_xfer https://review.openstack.org/614619 | 15:20 |
| openstackgerrit | Rajini Karthik proposed openstack/puppet-cinder stable/queens: Dell EMC SC: Add use_multipath_for_image_xfer https://review.openstack.org/614619 | 15:21 |
| *** rpittau has quit IRC | 15:21 | |
| *** shardy has quit IRC | 15:26 | |
| *** pgaxatte has joined #puppet-openstack | 15:27 | |
| pgaxatte | hello | 15:27 |
| pgaxatte | can someone take a look at https://review.openstack.org/#/c/620333 ? | 15:27 |
| *** salmankhan has quit IRC | 15:29 | |
| *** ykarel|away has joined #puppet-openstack | 15:36 | |
| *** vnogin has quit IRC | 15:40 | |
| *** vnogin has joined #puppet-openstack | 15:41 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova master: Update python packaging version for RHEL8 https://review.openstack.org/620149 | 15:44 |
| *** shardy has joined #puppet-openstack | 15:44 | |
| *** dpawlik has joined #puppet-openstack | 15:50 | |
| *** gfidente has quit IRC | 15:54 | |
| *** gfidente has joined #puppet-openstack | 15:54 | |
| *** salmankhan has joined #puppet-openstack | 16:11 | |
| pgaxatte | tobias-urdin: hello | 16:12 |
| *** salmankhan has quit IRC | 16:12 | |
| pgaxatte | following up your comment on https://review.openstack.org/#/c/620333 | 16:13 |
| pgaxatte | I don't see why should I pass anything to mistral::db::sync? | 16:13 |
| *** vnogin has quit IRC | 16:15 | |
| mwhahaha | pgaxatte: you don't have anything consuming those variables, so it'd need to be referenced somewhere | 16:18 |
| tobias-urdin | sorry i'm tired, my comment was wrong but what i meant was correct | 16:18 |
| tobias-urdin | they dont do anything | 16:18 |
| pgaxatte | mwhahaha: oh no it's much dirtier than that :D | 16:18 |
| pgaxatte | they are picked in mistral::db :D | 16:19 |
| tobias-urdin | oh no... the pick in mistral::db | 16:19 |
| mwhahaha | oh because we dropped them | 16:19 |
| mwhahaha | they were probably deprecated | 16:19 |
| tobias-urdin | those picks should be removed | 16:19 |
| mwhahaha | yea the picks should be removed | 16:20 |
| pgaxatte | mwhahaha, tobias-urdin I'd need to fix this and backport it down to pike, should we merge this or fix the picks instead? | 16:33 |
| mwhahaha | pgaxatte: the fix is to set the hiera to use ::mistral::db::<vars> | 16:34 |
| mwhahaha | it's not to add these vars back in | 16:34 |
| pgaxatte | sure | 16:34 |
| pgaxatte | but these vars will show up in the mistral class arguments anyways right? | 16:35 |
| mwhahaha | no | 16:35 |
| mwhahaha | not unless they are specified | 16:35 |
| mwhahaha | if they don't exist it falls back to the :mistral::db::<var> | 16:35 |
| mwhahaha | which is the point of pick | 16:35 |
| pgaxatte | i'm looking at puppet-cinder | 16:42 |
| pgaxatte | ah bad example it has picks too | 16:42 |
| pgaxatte | :D | 16:42 |
| pgaxatte | all the projects have the picks for the db parameters? | 16:43 |
| mwhahaha | for backwards compatibility | 16:44 |
| mwhahaha | it's long standing tech debt | 16:44 |
| pgaxatte | ok | 16:44 |
| pgaxatte | but still the main class has the database_* arguments | 16:45 |
| pgaxatte | do you also consider it part of the technical debt? | 16:45 |
| tobias-urdin | it's probably better we determine the way forward, since there is inconsistency | 16:45 |
| mwhahaha | the way forward is to use the <class>::db::<vars> | 16:45 |
| mwhahaha | the pick is there only for backwards compatibility | 16:45 |
| tobias-urdin | i agree | 16:46 |
| tobias-urdin | and that has probably been the consensus before as well | 16:46 |
| pgaxatte | maybe there ought to be a deprecation message when using the picks? | 16:46 |
| tobias-urdin | agree on that as well | 16:46 |
| mwhahaha | there usually is | 16:47 |
| pgaxatte | could we maybe just add this little patch of mine and fix all the rest right after? :D | 16:51 |
| tobias-urdin | why do you need it? why not just define mistal::db and be done? ^^ | 16:52 |
| pgaxatte | hmmm I'll have to change it on other projects too on my side, we use the database_* arguments everywhere :D | 16:53 |
| pgaxatte | alright i'll do that :D | 16:54 |
| tobias-urdin | i'll work up some policy about that and propose to puppet-openstack-guide and include you for review | 16:55 |
| tobias-urdin | if you want to push the work to unify the <module>::db let me know otherwise i'll add it to my todo | 16:55 |
| tobias-urdin | mwhahaha: if you have time, please go through some of my open reviews, have some small stuff that can prob be approved if ok | 16:58 |
| mwhahaha | tobias-urdin: k i'll spend some time reviewing today | 16:58 |
| tobias-urdin | ty | 16:59 |
| *** salmankhan has joined #puppet-openstack | 17:03 | |
| *** dpawlik has quit IRC | 17:13 | |
| *** dpawlik has joined #puppet-openstack | 17:14 | |
| *** dpawlik has quit IRC | 17:18 | |
| openstackgerrit | Merged openstack/puppet-octavia master: Convert to rspec-puppet-facts https://review.openstack.org/619896 | 17:19 |
| openstackgerrit | Merged openstack/puppet-mistral master: Convert to rspec-puppet-facts https://review.openstack.org/619887 | 17:22 |
| *** derekh has quit IRC | 17:23 | |
| *** ykarel|away has quit IRC | 17:28 | |
| *** dpawlik has joined #puppet-openstack | 17:29 | |
| *** sai_p has joined #puppet-openstack | 17:34 | |
| *** dpawlik has quit IRC | 17:34 | |
| *** trown is now known as trown|lunch | 17:42 | |
| openstackgerrit | Merged openstack/puppet-freezer master: Deprecate pki related options https://review.openstack.org/619407 | 17:43 |
| openstackgerrit | Merged openstack/puppet-congress master: Deprecate pki related options https://review.openstack.org/619420 | 17:45 |
| openstackgerrit | Merged openstack/puppet-glare master: Deprecate pki related options https://review.openstack.org/619409 | 17:48 |
| *** salmankhan has quit IRC | 18:00 | |
| openstackgerrit | Merged openstack/puppet-monasca master: Convert to rspec-puppet-facts and fix monasca::alarmdefs https://review.openstack.org/619888 | 18:06 |
| *** bnemec has quit IRC | 18:06 | |
| *** bnemec has joined #puppet-openstack | 18:06 | |
| openstackgerrit | Merged openstack/puppet-murano master: Convert to rspec-puppet-facts https://review.openstack.org/619890 | 18:09 |
| *** jpena is now known as jpena|off | 18:09 | |
| openstackgerrit | Merged openstack/puppet-murano master: Use puppet4 functions-api https://review.openstack.org/614238 | 18:09 |
| *** jtomasek has quit IRC | 18:12 | |
| openstackgerrit | Merged openstack/puppet-magnum master: Deprecate pki related options https://review.openstack.org/619423 | 18:17 |
| openstackgerrit | Merged openstack/puppet-aodh master: Deprecate pki related options https://review.openstack.org/619422 | 18:20 |
| openstackgerrit | Merged openstack/puppet-barbican master: Deprecate pki related options https://review.openstack.org/619416 | 18:25 |
| *** ykarel|away has joined #puppet-openstack | 18:27 | |
| *** dpawlik has joined #puppet-openstack | 18:37 | |
| *** dpawlik has quit IRC | 18:42 | |
| *** dpawlik has joined #puppet-openstack | 18:53 | |
| *** salmankhan has joined #puppet-openstack | 18:54 | |
| *** salmankhan has quit IRC | 18:54 | |
| *** salmankhan1 has joined #puppet-openstack | 18:54 | |
| openstackgerrit | Merged openstack/puppet-horizon master: Change default member role to member https://review.openstack.org/619077 | 18:55 |
| *** morazi has quit IRC | 18:56 | |
| *** salmankhan1 is now known as salmankhan | 18:56 | |
| *** ykarel|away has quit IRC | 18:56 | |
| *** dpawlik has quit IRC | 18:57 | |
| *** trown|lunch is now known as trown | 18:58 | |
| *** salmankhan has quit IRC | 19:11 | |
| openstackgerrit | Merged openstack/puppet-heat master: Deprecate pki related options https://review.openstack.org/619411 | 19:12 |
| *** salmankhan has joined #puppet-openstack | 19:15 | |
| *** salmankhan has quit IRC | 19:15 | |
| *** salmankhan has joined #puppet-openstack | 19:16 | |
| *** salmankhan has quit IRC | 19:17 | |
| *** salmankhan has joined #puppet-openstack | 19:17 | |
| openstackgerrit | Merged openstack/puppet-openstack-cookiecutter master: Deprecate pki related options https://review.openstack.org/619413 | 19:17 |
| openstackgerrit | Merged openstack/puppet-octavia master: Create certificates are package installs https://review.openstack.org/619797 | 19:22 |
| openstackgerrit | Merged openstack/puppet-tacker master: Deprecate pki related options https://review.openstack.org/619421 | 19:28 |
| *** salmankhan has quit IRC | 19:28 | |
| *** gfidente has quit IRC | 19:28 | |
| openstackgerrit | Rajini Karthik proposed openstack/puppet-cinder stable/rocky: Dell EMC SC: Add use_multipath_for_image_xfer https://review.openstack.org/614618 | 19:33 |
| openstackgerrit | Rajini Karthik proposed openstack/puppet-cinder stable/rocky: Dell EMC SC: Add use_multipath_for_image_xfer https://review.openstack.org/614618 | 19:34 |
| *** shardy has quit IRC | 19:35 | |
| openstackgerrit | Rajini Karthik proposed openstack/puppet-cinder stable/rocky: Dell EMC SC: Add use_multipath_for_image_xfer https://review.openstack.org/614618 | 19:36 |
| openstackgerrit | Rajini Karthik proposed openstack/puppet-cinder stable/queens: Dell EMC SC: Add use_multipath_for_image_xfer https://review.openstack.org/614619 | 19:38 |
| *** dpawlik has joined #puppet-openstack | 19:38 | |
| openstackgerrit | Merged openstack/puppet-octavia master: Set show_diff to false on certificates https://review.openstack.org/619798 | 19:40 |
| *** salmankhan has joined #puppet-openstack | 19:51 | |
| openstackgerrit | Merged openstack/puppet-watcher master: Deprecate pki related options https://review.openstack.org/619430 | 19:56 |
| openstackgerrit | Merged openstack/puppet-panko master: Deprecate pki related options https://review.openstack.org/619410 | 20:05 |
| *** jtomasek has joined #puppet-openstack | 20:17 | |
| openstackgerrit | Merged openstack/puppet-sahara master: Deprecate pki related options https://review.openstack.org/619424 | 20:17 |
| openstackgerrit | Alex Schultz proposed openstack/puppet-nova master: Update python packaging version for RHEL8 https://review.openstack.org/620149 | 20:29 |
| *** jtomasek has quit IRC | 20:45 | |
| *** salmankhan has quit IRC | 20:52 | |
| *** morazi has joined #puppet-openstack | 20:58 | |
| *** salmankhan has joined #puppet-openstack | 21:45 | |
| *** salmankhan has quit IRC | 21:46 | |
| *** salmankhan1 has joined #puppet-openstack | 21:46 | |
| *** salmankhan1 is now known as salmankhan | 21:49 | |
| *** trown is now known as trown|outtypewww | 21:49 | |
| *** dpawlik has quit IRC | 21:54 | |
| *** dpawlik has joined #puppet-openstack | 21:55 | |
| openstackgerrit | Merged openstack/puppet-zaqar master: Deprecate pki related options https://review.openstack.org/619425 | 21:57 |
| *** dpawlik has quit IRC | 22:00 | |
| mwhahaha | i feel like we should drop the integration tests on the older versions of puppet for master | 22:17 |
| mwhahaha | the job count on https://review.openstack.org/#/c/618745/ is insane | 22:18 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Move debug to <module>::logging class https://review.openstack.org/620415 | 22:24 |
| *** rcernin has joined #puppet-openstack | 22:57 | |
| *** salmankhan has quit IRC | 23:11 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-octavia stable/rocky: Set show_diff to false on certificates https://review.openstack.org/620421 | 23:26 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-octavia stable/rocky: Create certificates are package installs https://review.openstack.org/620422 | 23:27 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!