| -openstackstatus- NOTICE: The Gerrit service at review.opendev.org is back up and running; for outage details see analysis here: http://lists.opendev.org/pipermail/service-announce/2020-October/000011.html | 00:40 | |
| *** ChanServ changes topic to "CI Status: GREENish | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/" | 00:40 | |
| *** owalsh has quit IRC | 01:08 | |
| *** owalsh has joined #puppet-openstack | 01:09 | |
| *** jtomasek has joined #puppet-openstack | 06:41 | |
| *** rpittau|afk is now known as rpittau | 06:42 | |
| *** openstackgerrit has quit IRC | 07:38 | |
| *** rcernin has quit IRC | 07:39 | |
| *** tosky has joined #puppet-openstack | 07:42 | |
| *** jtomasek has quit IRC | 07:47 | |
| *** jpena|off is now known as jpena | 07:56 | |
| *** rcernin has joined #puppet-openstack | 08:09 | |
| *** dtantsur|afk is now known as dtantsur | 08:10 | |
| *** rcernin has quit IRC | 08:14 | |
| *** openstackgerrit has joined #puppet-openstack | 08:17 | |
| openstackgerrit | Merged openstack/puppet-octavia master: Deprecate database options from the octavia class https://review.opendev.org/758909 | 08:17 |
|---|---|---|
| *** LarsErikP has joined #puppet-openstack | 08:28 | |
| *** rcernin has joined #puppet-openstack | 08:28 | |
| openstackgerrit | Benedikt Trefzer proposed openstack/puppet-ceilometer master: add parameter to sepcifiy if service on keystone should be configured https://review.opendev.org/758964 | 08:29 |
| LarsErikP | hi. does anyone recall why all the admin_* parameters was removed from magnum_config here? https://github.com/openstack/puppet-magnum/commit/3e357838fb8000875d2542809292f7fb80d4d520 | 08:30 |
| LarsErikP | I.. think that config is still needed? In fact, the install guide says you should set them: https://docs.openstack.org/magnum/ussuri/install/install-rdo.html | 08:30 |
| tkajinam | LarsErikP, https://github.com/openstack/puppet-magnum/commit/85b20b0db469a6a0777192a7f94ea454b9d1cce8 | 08:31 |
| tkajinam | LarsErikP, these parameters were remove because they were deprecated parameters in keystonemiddleware. but we found several issues caused by missing admin_* parameters and introdiced the magnum::keystone::keystone_auth class | 08:32 |
| tkajinam | we might need double-check from magnum team but AFAIK we should use the parameters in keystone_auth section instead of adding some deprecated parameters in keystone_authtoken section | 08:33 |
| LarsErikP | Oh.. | 08:33 |
| LarsErikP | But this was not backported to ussuri? | 08:34 |
| *** derekh has joined #puppet-openstack | 08:34 | |
| LarsErikP | actually; then install guide for victoria also tells us to set those deprecated parameters in keystone_authtoken.. https://docs.openstack.org/magnum/victoria/install/install-rdo.html | 08:35 |
| tkajinam | ah, ok | 08:36 |
| tkajinam | sorry, I was looking at victoria | 08:37 |
| tkajinam | maybe we should revert the change in ussuri to follow what is described in the doc | 08:37 |
| tkajinam | LarsErikP, do you mind to file a bug for puppet-magnum ? | 08:37 |
| LarsErikP | sounds like a good idea; because now it does not work :P | 08:37 |
| LarsErikP | tkajinam: I can do that. launchpad or storyboard? | 08:38 |
| tkajinam | LarsErikP, launchpad | 08:38 |
| LarsErikP | ack | 08:38 |
| tobias-urdin | fwiw, we run train and even then we hardcode keystone_auth section manually using magnum_config | 08:38 |
| tobias-urdin | some stuff is broken, last time i checked, if those are not set | 08:39 |
| LarsErikP | yep. i.e barbican communication fails | 08:39 |
| tkajinam | hm | 08:40 |
| LarsErikP | we installed magnum@stein. And everything was fine. Then we updated to train. still fine, because the config is not removed even though it's no longer managed by puppet (of course) | 08:40 |
| tkajinam | then it might be better to backport keystone_auth section for train ? | 08:40 |
| LarsErikP | but when we upgraded to ussuri, we had to reinstall our magnum-servers to CentOS 8, and then all the "old" config was lost | 08:40 |
| tkajinam | yeah I get your point | 08:40 |
| LarsErikP | tkajinam: maybe? and ussuri | 08:40 |
| LarsErikP | ;) | 08:41 |
| tkajinam | sorry, I should have said ussuri | 08:41 |
| LarsErikP | let me just check here if it works | 08:42 |
| LarsErikP | just to be sure :-) | 08:42 |
| tkajinam | yeah it's helpful if you can check which parameters are actually required | 08:42 |
| tkajinam | because I don't have deployments with magnum now... | 08:43 |
| LarsErikP | seems to work | 08:45 |
| openstackgerrit | Merged openstack/puppet-nova master: Add support for more instance timeout options https://review.opendev.org/758686 | 08:46 |
| LarsErikP | I basically just set all the parameters given in keystone_auth.pp manually | 08:46 |
| LarsErikP | and removed the admin_* from keystone_authtoken section | 08:46 |
| LarsErikP | tkajinam: something like that? https://bugs.launchpad.net/puppet-magnum/+bug/1900813 :-) | 08:55 |
| openstack | Launchpad bug 1900813 in puppet-magnum "keystone auth fails in train and ussuri" [Undecided,New] | 08:55 |
| *** rcernin has quit IRC | 08:56 | |
| openstackgerrit | Merged openstack/puppet-nova master: Add support for more image related options https://review.opendev.org/758687 | 08:59 |
| tkajinam | LarsErikP, thanks | 09:06 |
| LarsErikP | tkajinam: you're welcome ;-) | 09:07 |
| openstackgerrit | Merged openstack/puppet-heat master: Deprecate database options from the heat class https://review.opendev.org/758898 | 09:13 |
| openstackgerrit | Merged openstack/puppet-manila master: Deprecate database options from the manila class https://review.opendev.org/758907 | 09:16 |
| *** gfidente has joined #puppet-openstack | 09:19 | |
| tkajinam | tobias-urdin, do you have any preference about the fix ? I'm thinking to backport keystone_auth config | 09:30 |
| tkajinam | except for added inclusion of magnum::keystone::keystone_auth, maybe ? | 09:30 |
| tkajinam | https://github.com/openstack/puppet-magnum/commit/85b20b0db469a6a0777192a7f94ea454b9d1cce8#diff-73787a5d46dccd603a5241093662223a3092e1cd68f0310696fa90dec67d45dcR76 | 09:30 |
| tobias-urdin | well im fine with backporting it | 09:30 |
| tobias-urdin | but we might break existing deployments (read: me :D) that is setting it manually with magnum_config | 09:31 |
| tobias-urdin | but im fine with it | 09:31 |
| tobias-urdin | :p | 09:31 |
| tkajinam | :-P | 09:31 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-magnum stable/ussuri: New class to support keystone_auth config https://review.opendev.org/758981 | 09:41 |
| openstackgerrit | Merged openstack/puppet-mistral master: Deprecate database options from the mistral class https://review.opendev.org/758908 | 09:51 |
| *** social has quit IRC | 09:57 | |
| openstackgerrit | Merged openstack/puppet-cinder master: Deprecate database options from the cinder class https://review.opendev.org/758896 | 10:13 |
| LarsErikP | tkajinam: \o/ | 10:23 |
| tkajinam | I'll backport that patch to train, too, once it is merged into ussuri | 10:39 |
| *** jtomasek has joined #puppet-openstack | 11:08 | |
| *** jpena is now known as jpena|lunch | 11:32 | |
| *** jtomasek has quit IRC | 12:00 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-oslo master: Add support for the oslo_policy/enforce_scope parameter https://review.opendev.org/759008 | 12:03 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-keystone master: Add support for the oslo_policy/enforce_scope parameter https://review.opendev.org/759010 | 12:06 |
| redrobot | Hi friends! I'd love to get some reviews for https://review.opendev.org/#/c/758485/ | 12:16 |
| redrobot | It's a pretty small patch for a couple of new options added to barbican | 12:17 |
| tkajinam | redrobot, added one comment (and one personal note) on the patch. please check it | 12:25 |
| *** jpena|lunch is now known as jpena | 12:30 | |
| redrobot | thank you tkajinam, I'll review and update the patch. | 12:32 |
| tkajinam | redrobot, np. I'll check once you update it. let me know if you have any questions about my comment | 12:32 |
| tkajinam | redrobot, oops. I missed to mention one more thing. Could you please add a short release note about the new parameters ? | 12:33 |
| redrobot | tkajinam, sure, no problem. I'll add a note. | 12:34 |
| tkajinam | (posted the same on the patch | 12:34 |
| tkajinam | redrobot, thx | 12:34 |
| *** ysandeep is now known as ysandeep|ruck | 13:18 | |
| obre | Could we have this merge soon: https://review.opendev.org/#/c/758019/? | 13:26 |
| tkajinam | obre, we've found some additional issues related to that fix but now confirmed that https://review.opendev.org/#/c/758877/ will solve the remaining issue | 13:35 |
| tkajinam | obre, once we backport that to V then backport the same sets to U. that is why we didn't merge that backport for U yet | 13:35 |
| obre | tkajinam: Alright. Sorry for being impatient :P | 13:42 |
| openstackgerrit | Merged openstack/puppet-ironic master: Deprecate database options from the base class https://review.opendev.org/758900 | 13:49 |
| *** ysandeep|ruck is now known as ysandeep|ruck|af | 14:04 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-keystone master: Add support for the oslo_policy/enforce_scope parameter https://review.opendev.org/759010 | 14:07 |
| *** ysandeep|ruck|af is now known as ysandeep|ruck | 14:21 | |
| *** jtomasek has joined #puppet-openstack | 14:41 | |
| *** ysandeep|ruck is now known as ysandeep|away | 14:57 | |
| openstackgerrit | Merged openstack/puppet-aodh master: Deprecate database options from the aodh class https://review.opendev.org/758891 | 15:20 |
| openstackgerrit | Merged openstack/puppet-nova master: Fix incorrect release note about db options deprecation https://review.opendev.org/758893 | 15:20 |
| openstackgerrit | Merged openstack/puppet-nova master: Fix misplaced releasenote file https://review.opendev.org/758894 | 15:21 |
| openstackgerrit | Oliver Walsh proposed openstack/puppet-nova master: Expose nova config option [workaround]/never_download_image_if_on_rbd https://review.opendev.org/759080 | 15:42 |
| *** dtantsur is now known as dtantsur|afk | 15:55 | |
| *** rpittau is now known as rpittau|afk | 16:04 | |
| *** tosky has quit IRC | 16:16 | |
| openstackgerrit | Oliver Walsh proposed openstack/puppet-nova master: Expose nova config option [workaround]/never_download_image_if_on_rbd https://review.opendev.org/759080 | 16:21 |
| openstackgerrit | Merged openstack/puppet-keystone master: Deprecate database options from the keystone class https://review.opendev.org/758901 | 16:41 |
| *** jtomasek has quit IRC | 16:42 | |
| *** jtomasek has joined #puppet-openstack | 16:48 | |
| *** gfidente is now known as gfidente|afk | 16:51 | |
| *** jtomasek has quit IRC | 16:58 | |
| *** jpena is now known as jpena|off | 17:07 | |
| openstackgerrit | Merged openstack/puppet-keystone master: Add missing dependencies about keystone_puppet_config https://review.opendev.org/758905 | 17:10 |
| *** derekh has quit IRC | 17:18 | |
| openstackgerrit | Merged openstack/puppet-glance master: Deprecate database options from the glance::api class https://review.opendev.org/758910 | 17:32 |
| *** iurygregory has quit IRC | 18:18 | |
| *** iurygregory has joined #puppet-openstack | 18:20 | |
| *** iurygregory has quit IRC | 18:24 | |
| *** iurygregory has joined #puppet-openstack | 18:25 | |
| *** tosky has joined #puppet-openstack | 19:12 | |
| *** lbragstad_ is now known as lbragstad | 20:49 | |
| *** tosky has quit IRC | 22:10 | |
| *** tosky has joined #puppet-openstack | 22:11 | |
| *** gfidente|afk has quit IRC | 22:20 | |
| *** rcernin has joined #puppet-openstack | 22:51 | |
| *** tosky has quit IRC | 22:51 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-keystone master: Add support for the oslo_policy/enforce_scope parameter https://review.opendev.org/759010 | 23:35 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-keystone stable/victoria: Add missing dependencies about keystone_puppet_config https://review.opendev.org/759155 | 23:36 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-nova master: Move release note file to the correct place https://review.opendev.org/759156 | 23:46 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-cinder master: Fix the misplaced release note file https://review.opendev.org/759157 | 23:54 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-ironic master: Fix the misplaced release note file https://review.opendev.org/759158 | 23:56 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-nova master: Fix the misplaced release note files https://review.opendev.org/759156 | 23:58 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-sahara master: Fix the misplaced release note file https://review.opendev.org/759159 | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!