| openstackgerrit | Dan Radez proposed openstack/puppet-neutron stable/queens: Add networking-ansible ml2 plugin support https://review.openstack.org/620672 | 00:13 |
|---|---|---|
| *** tosky has quit IRC | 00:27 | |
| *** mschuppert has quit IRC | 02:22 | |
| *** ykarel|away has joined #puppet-openstack | 03:22 | |
| *** janki has joined #puppet-openstack | 04:03 | |
| openstackgerrit | Alex Schultz proposed openstack/puppet-openstack-integration master: Reorganize the zuul configs https://review.openstack.org/621066 | 04:27 |
| *** markvoelker has quit IRC | 04:32 | |
| *** markvoelker has joined #puppet-openstack | 05:02 | |
| *** ykarel|away has quit IRC | 05:24 | |
| *** ykarel|away has joined #puppet-openstack | 05:39 | |
| *** ykarel|away is now known as ykarel | 05:39 | |
| openstackgerrit | Merged openstack/puppet-ironic master: Deprecate pki related options https://review.openstack.org/619414 | 06:07 |
| openstackgerrit | Merged openstack/puppet-openstack_extras master: Use puppet4 functions-api https://review.openstack.org/614226 | 06:10 |
| *** mschuppert has joined #puppet-openstack | 07:06 | |
| openstackgerrit | Merged openstack/puppet-tacker master: Remove auth_uri https://review.openstack.org/620727 | 07:08 |
| openstackgerrit | Merged openstack/puppet-murano master: Remove auth_uri https://review.openstack.org/620722 | 07:10 |
| openstackgerrit | Merged openstack/puppet-glare master: Remove auth_uri https://review.openstack.org/620736 | 07:10 |
| openstackgerrit | Merged openstack/puppet-magnum master: Remove auth_uri https://review.openstack.org/620741 | 07:13 |
| openstackgerrit | Merged openstack/puppet-cloudkitty master: Remove auth_uri https://review.openstack.org/620733 | 07:16 |
| openstackgerrit | Merged openstack/puppet-octavia stable/rocky: Create certificates are package installs https://review.openstack.org/620422 | 07:18 |
| openstackgerrit | Merged openstack/puppet-octavia stable/rocky: Set show_diff to false on certificates https://review.openstack.org/620421 | 07:18 |
| openstackgerrit | Merged openstack/puppet-octavia master: Remove auth_uri https://review.openstack.org/620729 | 07:18 |
| *** skramaja has joined #puppet-openstack | 07:19 | |
| openstackgerrit | Merged openstack/puppet-congress master: Remove auth_uri https://review.openstack.org/620721 | 07:28 |
| openstackgerrit | Merged openstack/puppet-freezer master: Remove auth_uri https://review.openstack.org/620723 | 07:28 |
| janki | Hi, Can I get some reviews for this cherry-pick https://review.openstack.org/#/c/620012/ | 07:36 |
| *** ykarel is now known as ykarel|lunch | 07:43 | |
| *** rpittau has joined #puppet-openstack | 07:57 | |
| *** jpena|off is now known as jpena | 08:01 | |
| *** rcernin has quit IRC | 08:06 | |
| *** jtomasek has joined #puppet-openstack | 08:06 | |
| *** dpawlik has joined #puppet-openstack | 08:07 | |
| *** shardy has joined #puppet-openstack | 08:23 | |
| *** shardy has quit IRC | 08:24 | |
| *** shardy has joined #puppet-openstack | 08:24 | |
| openstackgerrit | Merged openstack/puppet-zaqar master: Remove auth_uri https://review.openstack.org/620749 | 08:32 |
| openstackgerrit | Merged openstack/puppet-mistral master: Remove auth_uri https://review.openstack.org/620743 | 08:38 |
| *** ykarel|lunch is now known as ykarel | 08:38 | |
| *** tosky has joined #puppet-openstack | 08:42 | |
| *** dpawlik has quit IRC | 08:46 | |
| *** salmankhan has joined #puppet-openstack | 08:50 | |
| *** salmankhan has quit IRC | 08:56 | |
| *** dpawlik has joined #puppet-openstack | 09:20 | |
| *** dpawlik has quit IRC | 09:24 | |
| *** derekh has joined #puppet-openstack | 09:33 | |
| *** gfidente has joined #puppet-openstack | 09:40 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-ec2api master: Remove auth_uri https://review.openstack.org/620725 | 10:09 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-ironic master: Remove auth_uri https://review.openstack.org/620728 | 10:12 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-ceilometer master: Remove auth_uri https://review.openstack.org/620731 | 10:13 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-manila master: Remove auth_uri https://review.openstack.org/620742 | 10:14 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Change to www_authenticate_uri for those missed https://review.openstack.org/621136 | 10:27 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-swift master: Deprecate auth_uri in swift::proxy::authtoken https://review.openstack.org/621138 | 10:32 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-trove master: Remove auth_uri https://review.openstack.org/620748 | 10:33 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-designate master: Remove auth_uri https://review.openstack.org/620734 | 10:38 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Convert to rspec-puppet-facts https://review.openstack.org/620653 | 10:58 |
| *** dtantsur|mtg is now known as dtantsur|afk | 11:00 | |
| *** iurygregory is now known as iury|lunch | 11:03 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron master: Convert to rspec-puppet-facts https://review.openstack.org/620684 | 11:04 |
| *** vnogin has joined #puppet-openstack | 11:08 | |
| *** morazi has quit IRC | 11:10 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Convert to rspec-puppet-facts https://review.openstack.org/620653 | 11:14 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-neutron master: Convert to rspec-puppet-facts https://review.openstack.org/620684 | 11:24 |
| tobias-urdin | ^ sucks doing, but only puppet-keystone left :) | 11:24 |
| *** iury|lunch is now known as iurygregory | 11:36 | |
| *** vnogin has quit IRC | 11:52 | |
| openstackgerrit | Merged openstack/puppet-cinder stable/queens: Add cinder::nova class to configure nova section https://review.openstack.org/620093 | 12:02 |
| openstackgerrit | Lee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement https://review.openstack.org/604182 | 12:09 |
| openstackgerrit | Merged openstack/puppet-cinder stable/queens: Improve cinder::nova support for auth_type password https://review.openstack.org/620094 | 12:12 |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: [DNM][py3 scenario] Test glance without SSL https://review.openstack.org/620561 | 12:13 |
| *** jpena is now known as jpena|lunch | 12:41 | |
| *** morazi has joined #puppet-openstack | 12:49 | |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-designate master: Remove auth_uri https://review.openstack.org/620734 | 12:55 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-ironic master: Remove auth_uri https://review.openstack.org/620728 | 12:56 |
| *** iurygregory is now known as iurygregory|mtg | 12:59 | |
| *** salmankhan has joined #puppet-openstack | 13:09 | |
| openstackgerrit | Martin Schuppert proposed openstack/puppet-nova master: Add support for native TLS encryption on NBD for disk migration https://review.openstack.org/618763 | 13:16 |
| *** trown|outtypewww has quit IRC | 13:17 | |
| *** trown|brb has joined #puppet-openstack | 13:18 | |
| openstackgerrit | yatin proposed openstack/puppet-openstack-integration master: [DNM][py3 scenario] Test glance without SSL https://review.openstack.org/620561 | 13:24 |
| *** iurygregory|mtg is now known as iurygregory | 13:29 | |
| *** skramaja has quit IRC | 13:33 | |
| *** jpena|lunch is now known as jpena | 13:40 | |
| *** EmilienM is now known as EvilienM | 13:58 | |
| *** ykarel is now known as ykarel|away | 14:07 | |
| *** hjensas has quit IRC | 14:12 | |
| bandini | mwhahaha: mind merging/reviewing https://review.openstack.org/#/c/619892/ when you get online? | 14:18 |
| openstackgerrit | Lee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement https://review.openstack.org/604182 | 14:21 |
| openstackgerrit | Merged openstack/puppet-watcher master: Remove auth_uri https://review.openstack.org/620726 | 14:21 |
| lyarwood | ^ owalsh / mschuppert / tobias-urdin ; I still can't get the above puppet-placement change to actually call `placement-manage db sync` correctly, I'm going around in circles at this point so any help here would be really appreciated! http://paste.openstack.org/show/736483/ | 14:23 |
| tobias-urdin | checking | 14:26 |
| *** janki has quit IRC | 14:30 | |
| *** ykarel|away has quit IRC | 14:33 | |
| tobias-urdin | lyarwood: must be either 1) the dbsync class is never included anywhere or 2) db::mysql (an db::postgresql if that is used) doesn't notify the dbsync class properly | 14:38 |
| tobias-urdin | so in p-o-i placement manifest you should "include ::placement::db::sync" since there is no such include from any other resource what i can see | 14:39 |
| tobias-urdin | in nova there is a lot of ugly fixes like "sync_db" parameters that is set in several places to ensure it's included automatically | 14:40 |
| tobias-urdin | see this as example https://review.openstack.org/#/c/619875/2/manifests/nova.pp | 14:41 |
| tobias-urdin | otherwise it's probably this lines (is also in postgresql) that doesn't notify properly: ::Openstacklib::Db::Mysql['placement'] ~> Exec<| title == 'placement-manage db sync' |> | 14:41 |
| tobias-urdin | in placement::db::mysql (and postgresql class) | 14:41 |
| lyarwood | tobias-urdin: ack thanks | 14:43 |
| tobias-urdin | lyarwood: also saw that this file didn't exist in puppet-placement module, maybe you could throw that in in next PS as well | 14:43 |
| tobias-urdin | https://github.com/openstack/puppet-nova/blob/master/spec/acceptance/nodesets/nodepool-bionic.yml | 14:43 |
| lyarwood | tobias-urdin: ack will do | 14:46 |
| tobias-urdin | ty! | 14:47 |
| *** ykarel|away has joined #puppet-openstack | 14:52 | |
| *** ykarel|away is now known as ykarel | 14:52 | |
| *** pma78 has quit IRC | 14:53 | |
| *** salmankhan has quit IRC | 15:29 | |
| *** bnemec is now known as beekneemech | 15:34 | |
| mwhahaha | tobias-urdin: not sure if you saw https://review.openstack.org/#/c/621066/, i figured we need to chop up the .zuul.yaml cause it's nearly 1000 LOC | 15:41 |
| tobias-urdin | sweet | 15:41 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Add mount_core module needed by puppet-swift https://review.openstack.org/621191 | 15:47 |
| tobias-urdin | mwhahaha: ^ .... stupid shit | 15:47 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-swift master: Deprecate auth_uri in swift::proxy::authtoken https://review.openstack.org/621138 | 15:48 |
| tobias-urdin | mwhahaha: can we merge this https://review.openstack.org/#/c/620415/ | 15:49 |
| mwhahaha | yea | 15:49 |
| tobias-urdin | mwhahaha: also i've been stuck at this unit test failure for a while now, if you have a minute https://review.openstack.org/#/c/620653/ | 15:51 |
| mwhahaha | k | 15:52 |
| mwhahaha | it's failing on https://review.openstack.org/#/c/620653/4/spec/classes/nova_keystone_auth_spec.rb@144 will need to see why that with_notify is not there | 15:59 |
| mwhahaha | perhaps a bug in the code | 15:59 |
| mwhahaha | tobias-urdin: oh we're using a spaceship | 15:59 |
| mwhahaha | so i don't think it works if the service isn't there | 16:00 |
| mwhahaha | though that is a copy/pasta | 16:00 |
| mwhahaha | weird | 16:00 |
| tobias-urdin | weird it's only in centos, something must be keeping that service from being added | 16:00 |
| tobias-urdin | *declared | 16:00 |
| mwhahaha | oh it's not a service anymore | 16:00 |
| mwhahaha | on centos | 16:00 |
| mwhahaha | thats why it was on debian | 16:01 |
| mwhahaha | on centos it's httpd now | 16:01 |
| tobias-urdin | oh is wsgi enforced on centos by default already | 16:01 |
| mwhahaha | yea | 16:01 |
| tobias-urdin | sweet | 16:01 |
| tobias-urdin | didn't know | 16:01 |
| mwhahaha | https://github.com/openstack/puppet-nova/blob/master/manifests/api.pp#L345 | 16:02 |
| mwhahaha | so we're in that block of code | 16:02 |
| tobias-urdin | yeah but service_name defaults to $::nova::params::api_service_name which is 'openstack-nova-api' for centos | 16:03 |
| *** hjensas has joined #puppet-openstack | 16:04 | |
| tobias-urdin | hm not sure what magic happens with the nova::generic_service declaration because of that | 16:05 |
| tobias-urdin | but yeah somehow nova-api doesn't get declared so i'll just change that testing then | 16:06 |
| openstackgerrit | Tobias Urdin proposed openstack/puppet-nova master: Convert to rspec-puppet-facts https://review.openstack.org/620653 | 16:08 |
| tobias-urdin | the day all eventlet stuff can be dropped will be a great day | 16:09 |
| *** trown|brb is now known as regain | 16:55 | |
| *** regain is now known as trown | 16:55 | |
| *** trown is now known as trown|lunch | 16:56 | |
| *** rnoriega- has quit IRC | 16:56 | |
| *** rnoriega has joined #puppet-openstack | 16:58 | |
| *** jpena is now known as jpena|off | 16:59 | |
| *** rpittau has quit IRC | 17:00 | |
| *** iurygregory has quit IRC | 17:00 | |
| *** hjensas has quit IRC | 17:21 | |
| *** ykarel is now known as ykarel|away | 17:22 | |
| *** shardy has quit IRC | 17:22 | |
| *** vnogin has joined #puppet-openstack | 17:49 | |
| *** openstackgerrit has quit IRC | 17:51 | |
| *** vnogin has quit IRC | 17:53 | |
| *** derekh has quit IRC | 18:00 | |
| *** gfidente has quit IRC | 18:04 | |
| *** ykarel|away has quit IRC | 18:17 | |
| *** morazi has quit IRC | 19:11 | |
| *** openstackgerrit has joined #puppet-openstack | 19:16 | |
| openstackgerrit | Jeremy Stanley proposed openstack/puppet-openstack-cookiecutter master: Update contact address to openstack-discuss ML https://review.openstack.org/621271 | 19:16 |
| *** hjensas has joined #puppet-openstack | 19:58 | |
| *** morazi has joined #puppet-openstack | 20:05 | |
| openstackgerrit | Merged openstack/puppet-octavia master: Re-include octavia::db in octavia::api https://review.openstack.org/620147 | 20:08 |
| *** jistr has quit IRC | 20:11 | |
| *** jistr has joined #puppet-openstack | 20:13 | |
| *** openstackgerrit has quit IRC | 20:50 | |
| *** openstackgerrit has joined #puppet-openstack | 21:00 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Move debug to <module>::logging class https://review.openstack.org/620415 | 21:00 |
| *** salmankhan has joined #puppet-openstack | 21:31 | |
| *** chem has quit IRC | 21:34 | |
| *** salmankhan has quit IRC | 21:36 | |
| *** salmankhan has joined #puppet-openstack | 22:11 | |
| *** tosky has quit IRC | 23:42 | |
| openstackgerrit | Nathan Kinder proposed openstack/puppet-keystone master: update support for openidc in puppet-keystone https://review.openstack.org/575147 | 23:49 |
| *** salmankhan has quit IRC | 23:51 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!