Thursday, 2021-10-28

opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Use system scope credentials to request keystone  https://review.opendev.org/c/openstack/puppet-keystone/+/80647400:18
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Use system scope credentials to request keystone  https://review.opendev.org/c/openstack/puppet-keystone/+/80647401:00
opendevreviewTakashi Kajinami proposed openstack/puppet-openstack-integration stable/xena: Switch to RDO Xena  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/81576701:09
opendevreviewTakashi Kajinami proposed openstack/puppet-openstack-integration stable/xena: Switch to RDO Xena  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/81576701:10
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: DNM: Debugging unit test failures  https://review.opendev.org/c/openstack/puppet-keystone/+/81577002:48
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Use system scope credentials to request keystone  https://review.opendev.org/c/openstack/puppet-keystone/+/80647404:10
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: DNM: Debugging unit test failures  https://review.opendev.org/c/openstack/puppet-keystone/+/81577004:10
opendevreviewTakashi Kajinami proposed openstack/puppet-neutron master: Use openstack cli to manage neutron ports  https://review.opendev.org/c/openstack/puppet-neutron/+/81549204:22
opendevreviewTakashi Kajinami proposed openstack/puppet-neutron master: Use openstack cli to manage neutron ports  https://review.opendev.org/c/openstack/puppet-neutron/+/81549204:47
*** ysandeep|out is now known as ysandeep05:06
opendevreviewyatin proposed openstack/puppet-neutron stable/victoria: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81579005:46
*** ykarel|away is now known as ykarel05:46
ykareltkajinam, is it fine to backport ^ until train?05:46
ykarelvictoria one went clean, in ussuri i see conflicts, so will check05:48
opendevreviewwaleed mousa proposed openstack/puppet-neutron master: Change authentication method for Mellanox sdn controller  https://review.opendev.org/c/openstack/puppet-neutron/+/81455405:57
opendevreviewyatin proposed openstack/puppet-neutron stable/ussuri: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81577406:06
tkajinamykarel, I think we can06:06
ykareltkajinam, ok /me proposing06:07
tkajinamit just provides an additional interface so I'm not aware of any bad effect caused by that backport06:07
ykareltkajinam, is there any other way of modifying openvswitch_agent.ini apart from ^ without code changes?06:08
opendevreviewyatin proposed openstack/puppet-neutron stable/train: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81579106:08
tkajinamykarel, what is the change you need ?06:12
ykareltkajinam, 06:15
ykarel[ovs]06:15
ykarelovsdb_interface=vsctl06:15
ykarelof_interface=ovs-ofctl06:15
ykarelcontext https://bugzilla.redhat.com/show_bug.cgi?id=201617706:15
ykareli checked in puppet-neutron those configs are not exposed06:15
ykareland above cherrypicks it can be done atleast, not sure if there is some other way too?06:16
opendevreviewwaleed mousa proposed openstack/puppet-neutron master: Change authentication method for Mellanox sdn controller  https://review.opendev.org/c/openstack/puppet-neutron/+/81455406:27
tkajinamykarel, let me check it later. I think ideally we should implement a native puppet parameter but I know that parameter has been already removed from neutron06:31
tkajinamykarel, supporting arbitally configuration should help and we can move it forward. I'll give it a though to consider any better approach06:32
ykareltkajinam, okk Thanks06:32
opendevreviewwaleed mousa proposed openstack/puppet-neutron master: Change authentication method for Mellanox sdn controller  https://review.opendev.org/c/openstack/puppet-neutron/+/81455406:43
*** jpena|off is now known as jpena06:54
*** amoralej|off is now known as amoralej07:06
*** ysandeep is now known as ysandeep|lunch07:48
opendevreviewyatin proposed openstack/puppet-neutron stable/stein: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81579608:13
opendevreviewyatin proposed openstack/puppet-neutron stable/rocky: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81578608:20
opendevreviewyatin proposed openstack/puppet-neutron stable/queens: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81579708:23
*** ykarel is now known as ykarel|lunch08:34
opendevreviewTakashi Kajinami proposed openstack/puppet-neutron stable/train: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81579108:34
opendevreviewTakashi Kajinami proposed openstack/puppet-neutron stable/stein: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81579608:35
opendevreviewTakashi Kajinami proposed openstack/puppet-neutron stable/rocky: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81578608:38
opendevreviewTakashi Kajinami proposed openstack/puppet-neutron stable/queens: Support arbitrary configurations for ovs agent and sriov agent  https://review.opendev.org/c/openstack/puppet-neutron/+/81579708:38
*** ysandeep|lunch is now known as ysandeep08:45
tkajinamykarel|lunch, just fyi. I had another look about these parameters and I think backporting that arbitrary configurations would be the reasonable approach here. The parameter was completely removed from neutron during Stein and I don't think it really makes sense to implement native puppet parameters in very old versions like Queens now08:45
tkajinamafaik tripleo includes the neutron::config class so we can set these parameters using neutron::config::ovs_agent_config once that backport lands08:47
tkajinamparameter_defaults:08:47
tkajinam  ExtraConfig:08:47
tkajinam    neutron::config::ovs_agent_config:08:47
tkajinam      'OVS/ovsdb_interface':08:47
tkajinam        value: vsctl08:48
*** ysandeep is now known as ysandeep|afk09:46
*** ykarel|lunch is now known as ykarel10:04
ykareltkajinam, ack Thanks for checking 10:06
*** ysandeep|afk is now known as ysandeep10:37
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Use system scope credentials to request keystone  https://review.opendev.org/c/openstack/puppet-keystone/+/80647410:43
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: DNM: Debugging unit test failures  https://review.opendev.org/c/openstack/puppet-keystone/+/81577010:43
opendevreviewTakashi Kajinami proposed openstack/puppet-neutron master: Use openstack cli to manage neutron ports  https://review.opendev.org/c/openstack/puppet-neutron/+/81549211:12
*** ykarel is now known as ykarel|afk11:15
*** jpena is now known as jpena|lunch11:23
opendevreviewTakashi Kajinami proposed openstack/puppet-neutron master: Change authentication method for Mellanox sdn controller  https://review.opendev.org/c/openstack/puppet-neutron/+/81455411:24
*** ysandeep is now known as ysandeep|mtg11:35
opendevreviewOskari Lemmelä proposed openstack/puppet-keystone master: OIDC: Add support for setting OIDCResponseMode  https://review.opendev.org/c/openstack/puppet-keystone/+/81582611:35
*** ysandeep|mtg is now known as ysandeep11:51
opendevreviewTakashi Kajinami proposed openstack/puppet-neutron master: Use openstack cli to manage neutron ports  https://review.opendev.org/c/openstack/puppet-neutron/+/81549212:01
*** jpena|lunch is now known as jpena12:22
*** amoralej is now known as amoralej|lunch12:28
opendevreviewMerged openstack/puppet-tempest master: Do not override default of [auth] admin_domain_name  https://review.opendev.org/c/openstack/puppet-tempest/+/81460713:07
opendevreviewMerged openstack/puppet-tempest master: Support more *_domain_name options  https://review.opendev.org/c/openstack/puppet-tempest/+/81460813:15
*** ykarel|afk is now known as ykarel13:19
*** amoralej|lunch is now known as amoralej13:20
opendevreviewMerged openstack/puppet-tempest master: Add support for [auth] admin_system  https://review.opendev.org/c/openstack/puppet-tempest/+/81460913:36
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Use system scope credentials to request keystone  https://review.opendev.org/c/openstack/puppet-keystone/+/80647413:49
opendevreviewTakashi Kajinami proposed openstack/puppet-nova master: Add initial support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-nova/+/80631714:10
opendevreviewTakashi Kajinami proposed openstack/puppet-nova master: WIP: Use system scope credentials in providers  https://review.opendev.org/c/openstack/puppet-nova/+/81585014:10
opendevreviewMerged openstack/puppet-nova stable/wallaby: Centralize arbitrary configurations for libvirt daemons  https://review.opendev.org/c/openstack/puppet-nova/+/81511314:14
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Use system scope credentials to request keystone  https://review.opendev.org/c/openstack/puppet-keystone/+/80647414:29
opendevreviewTakashi Kajinami proposed openstack/puppet-nova master: WIP: Use system scope credentials in providers  https://review.opendev.org/c/openstack/puppet-nova/+/81585014:31
tkajinamykarel, Oops. I somehow missed we did have parameters in puppet-neutron for ovsdb_interface and of_interface14:38
tkajinamwhich Bernard pointed out in that bz14:39
tkajinamX-(14:39
ykareltkajinam, ahh i tried to find too, likely miss looked14:40
tkajinamykarel, regarding your backports, I think it'd be helpful to backport the patch back to Train. it seems gate job is broken for stein and older . My honest feeling is that older branches are not actively maintained recently so I tend to go for retiring them instead of try fixing gate and merge these backports14:41
ykareltkajinam, ok till train should be fine, older ones can be abandoned14:44
ykareltkajinam, and yes correct those params are available in stable/queens https://github.com/openstack/puppet-neutron/blob/stable/queens/manifests/agents/ml2/ovs.pp#L355-L35614:44
ykarelmy bad i checked only master :(14:44
tkajinamwe did a good job and dropped the parameter timely :-)14:45
ykarel:D14:51
*** ykarel is now known as ykarel|away14:51
opendevreviewTakashi Kajinami proposed openstack/puppet-nova master: WIP: Use system scope credentials in providers  https://review.opendev.org/c/openstack/puppet-nova/+/81585015:00
*** ysandeep is now known as ysandeep|mtg15:03
opendevreviewCarlos Eduardo proposed openstack/puppet-manila stable/wallaby: Add ability to configure backend availability zones  https://review.opendev.org/c/openstack/puppet-manila/+/81585815:32
opendevreviewCarlos Eduardo proposed openstack/puppet-manila stable/wallaby: Add ability to configure backend availability zones  https://review.opendev.org/c/openstack/puppet-manila/+/81574315:34
opendevreviewTakashi Kajinami proposed openstack/puppet-nova master: WIP: Use system scope credentials in providers  https://review.opendev.org/c/openstack/puppet-nova/+/81585015:35
*** ysandeep|mtg is now known as ysandeep15:49
*** amoralej is now known as amoralej|off15:52
*** ysandeep is now known as ysandeep|out15:53
opendevreviewTakashi Kajinami proposed openstack/puppet-manila stable/wallaby: Add ability to configure backend availability zones  https://review.opendev.org/c/openstack/puppet-manila/+/81586215:59
*** jpena is now known as jpena|off16:05
opendevreviewTakashi Kajinami proposed openstack/puppet-nova master: WIP: Use system scope credentials in providers  https://review.opendev.org/c/openstack/puppet-nova/+/81585016:22
*** derekh is now known as derekh_afk17:04
-opendevstatus- NOTICE: mirror.bhs1.ovh.opendev.org filled its disk around 17:25 UTC. We have corrected this issue around 18:25 UTC and jobs that failed due to this mirror can be rechecked.18:45
opendevreviewMerged openstack/puppet-openstack-integration master: Ubuntu: Use debian-<release> ceph repo  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/81566221:04
opendevreviewTakashi Kajinami proposed openstack/puppet-glance stable/victoria: Swift backend: the key parameter should be secret  https://review.opendev.org/c/openstack/puppet-glance/+/81588223:02
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Use system scope credentials to request keystone  https://review.opendev.org/c/openstack/puppet-keystone/+/80647423:40
opendevreviewTakashi Kajinami proposed openstack/puppet-nova master: WIP: Use system scope credentials in providers  https://review.opendev.org/c/openstack/puppet-nova/+/81585023:42

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!