Monday, 2021-06-28

opendevreviewShuai Qian proposed openstack/neutron-lib stable/victoria: add global_request_id in from_dict  https://review.opendev.org/c/openstack/neutron-lib/+/79823400:54
opendevreviewMerged openstack/neutron master: Move dns-integration extension to the ML2_SUPPORTED_API_EXTENSIONS list  https://review.opendev.org/c/openstack/neutron/+/79597900:54
opendevreviewMerged openstack/neutron master: [ML2] Change way how list of supported API extensions is made  https://review.opendev.org/c/openstack/neutron/+/79314100:54
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Use ovs constants from neutron-lib  https://review.opendev.org/c/openstack/neutron/+/79712005:50
opendevreviewSlawek Kaplonski proposed openstack/neutron-lib master: Rehome DHCP_IPV{4,6}_TABLE constants to neutron-lib  https://review.opendev.org/c/openstack/neutron-lib/+/79824806:01
isabekHi slaweq, lajoskatona! Can you please review this patch: https://review.opendev.org/c/openstack/neutron/+/674044 ? When will this patch merge, I will fix merge conflict on PORT events.08:15
opendevreviewShuai Qian proposed openstack/neutron-lib master: add global_request_id in from_dict  https://review.opendev.org/c/openstack/neutron-lib/+/79826108:15
lajoskatonaisabek: sure, it was on mile list of reviews, try to review ti today08:16
isabeklajoskatona: thank you!08:16
MrClayPoleMorning all, We currently have two OpenStack ansible rocky deployments. One using Linux bridges and the other using Open vSwitch both on Ubuntu 18.04. We would like to migrate the environment running Linux bridges to Open vSwitch. I'm hoping it's just a case of updating the networking to ovs and then running the playbooks to update neutron on the neutron and compute nodes but just wondering if there is anything else to08:20
MrClayPole consider?08:20
opendevreviewliuyulong proposed openstack/neutron master: Add fullstack test case for OVS DHCP extension  https://review.opendev.org/c/openstack/neutron/+/77656809:30
slaweqlajoskatona obondarev hi, can You check https://review.opendev.org/c/openstack/neutron/+/797595 ? thx in advance10:31
lajoskatonaslaweq: done10:48
slaweqThank You10:54
opendevreviewLajos Katona proposed openstack/neutron master: Privsep with timout for get_link_devices call  https://review.opendev.org/c/openstack/neutron/+/79499411:10
opendevreviewMerged openstack/neutron master: use callback payloads for SECURITY_GROUP  https://review.opendev.org/c/openstack/neutron/+/67404411:18
lajoskatonaslaweq: Last week I asked infra about taas: https://meetings.opendev.org/irclogs/%23openstack-infra/%23openstack-infra.2021-06-24.log.html#t2021-06-24T16:58:1811:23
lajoskatonaslaweq: we canpush new patches and those will be "moved" together with the repo11:24
lajoskatonaslaweq: so I misunderstood this process :-)11:24
opendevreviewliuyulong proposed openstack/neutron master: Refactor DHCP common config options  https://review.opendev.org/c/openstack/neutron/+/79828511:38
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: use payloads for PORT AFTER_UPDATE events  https://review.opendev.org/c/openstack/neutron/+/79511711:45
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: use payloads for PORT AFTER_DELETE events  https://review.opendev.org/c/openstack/neutron/+/79700411:45
slaweqlajoskatona thx for info11:55
slaweqso we can move on with my patches to tap-as-a-service, right?11:55
lajoskatonaslaweq: yes, and even "hanging" patches will move together with taas, so no need to repropose them11:59
slaweqgreat12:00
amotokilajoskatona: it is a rename of a project, so we don't need to wait pending patches, right?12:09
lajoskatonaamotoki: yes, this what infra said, and was not clear for me12:11
amotokilajoskatona: thanks for clarification. it is really nice.12:11
opendevreviewBalazs Gibizer proposed openstack/neutron master: Add limitation about QoS min bw for PF less SRIOV platforms  https://review.opendev.org/c/openstack/neutron/+/79829413:00
opendevreviewTerry Wilson proposed openstack/networking-ovn stable/train: Use TCP keepalives for ovsdb connections  https://review.opendev.org/c/openstack/networking-ovn/+/79563313:05
opendevreviewAkihiro Motoki proposed openstack/networking-odl stable/rocky: Dropping lower constraints testing (stable Stein)  https://review.opendev.org/c/openstack/networking-odl/+/79829813:12
opendevreviewsean mooney proposed openstack/neutron master: [docs] extend bw qos limitations to include pf netdev name  https://review.opendev.org/c/openstack/neutron/+/79830213:21
opendevreviewMerged openstack/neutron master: [OVN] Add subnet-service-types as supported by OVN mech driver  https://review.opendev.org/c/openstack/neutron/+/79759513:44
lajoskatonaslaweq, amotoki: regarding http://lists.openstack.org/pipermail/openstack-discuss/2021-June/023341.html13:51
lajoskatonaslaweq, amotoki: I see https://review.opendev.org/c/openstack/networking-odl/+/798298 can work for stein, but as these branches are em-d isn't there a way to make these jobs with bad config dropped?13:52
slaweqlajoskatona I'm in the meeting now13:55
slaweqwill check it later13:55
lajoskatonaslaweq: ok13:55
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: use payloads for PORT AFTER_UPDATE events  https://review.opendev.org/c/openstack/neutron/+/79511713:58
opendevreviewMamatisa Nurmatov proposed openstack/neutron master: use payloads for PORT AFTER_DELETE events  https://review.opendev.org/c/openstack/neutron/+/79700413:59
amotokilajoskatona: slaweq: yes, we can drop failing jobs in em'ed branches. I just first try to fix the zuul config error but all non-UT jobs are failing. it looks a good idea to drop these failing jobs.14:32
opendevreviewsean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges  https://review.opendev.org/c/openstack/os-vif/+/79805514:33
amotokilajoskatona: slaweq: one thing I am not sure is a failure in the requirements-check job. I will look into it in detail.14:33
opendevreviewLajos Katona proposed openstack/networking-odl stable/rocky: Dropping lower constraints testing (stable Stein)  https://review.opendev.org/c/openstack/networking-odl/+/79829814:54
gmannslaweq: I pinged on weekend about this https://review.opendev.org/c/openstack/tempest/+/79813015:34
gmannslaweq: I saw you changed the create network RBAC to system_admin_or_project_member15:35
gmannbut how system admin will be able to create network as it need project id ?15:35
mnasiadkaGood afternoon15:43
mnasiadkaAny plans for Neutron with ML2/OVN to support 6to4 NAT or something similar?15:44
slaweqgmann you can pass project_id in the request body as an admin15:51
slaweqAnd actually system admin has to do it that way always as there is no project_id in the context15:51
gmannslaweq: i see, neutron API accept the project-id https://docs.openstack.org/api-ref/network/v2/?expanded=create-network-detail#networks15:53
gmannslaweq: I am thinking for tempest tests. I think skipping is best option as we do not know the project id in case of system admin tests15:54
gmannslaweq: and as system admin cannot create nova VM, not sure having network created will actually be used or not15:55
gmannwhat you say?15:55
opendevreviewLajos Katona proposed openstack/networking-odl stable/rocky: Dropping lower constraints testing (stable Stein)  https://review.opendev.org/c/openstack/networking-odl/+/79829816:34
opendevreviewSzymon Wróblewski proposed openstack/neutron master: Fix ObjectChangeHandler thread usage  https://review.opendev.org/c/openstack/neutron/+/78851016:34
opendevreviewRodolfo Alonso proposed openstack/neutron master: Correctly label port as SubPort in SubPortNotFound  https://review.opendev.org/c/openstack/neutron/+/78870617:19
opendevreviewBrian Haley proposed openstack/ovn-octavia-provider master: Add Health Monitor support  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/71325317:43
opendevreviewBrian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support  https://review.opendev.org/c/openstack/networking-ovn/+/79606317:43
slaweqgmann I'm ok with skipping tests in such case but please keep in mind that by default only system admin can create network for any project18:05
slaweqproject admin can only create resources for own project18:05
slaweqso we may have lower test coverage probably18:06
slaweqmnasiadka: I don't think we have any RFE for that so currently probably answear is "no"18:07
gmannslaweq:  yeah, for existing tests we can skip and then later we can add coverage for system admin create network once we sort out how system admin can create VM case18:10
gmannslaweq: would appreciate your review in this https://review.opendev.org/c/openstack/tempest/+/79813018:11
slaweqgmann sure18:12
slaweqI will add it to my review list for tomorrow morning, ok?18:13
slaweqor is it urgent?18:13
gmannslaweq: thanks, sure tomorrow is fine. 18:17
slaweqOk. Thx18:24
opendevreviewMerged openstack/neutron master: Make explicit the network backend used in the CI jobs  https://review.opendev.org/c/openstack/neutron/+/79705118:55
opendevreviewMerged openstack/neutron master: Use OVS backend for testing os-ken library  https://review.opendev.org/c/openstack/neutron/+/79805818:55
opendevreviewsean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges  https://review.opendev.org/c/openstack/os-vif/+/79805519:23
opendevreviewBrian Haley proposed openstack/ovn-octavia-provider master: Add Health Monitor support  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/71325319:38
opendevreviewBrian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support  https://review.opendev.org/c/openstack/networking-ovn/+/79606319:41
opendevreviewBrian Haley proposed openstack/ovn-octavia-provider master: Update to flake8>3.9.0  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/79837519:44
opendevreviewBrian Haley proposed openstack/ovn-octavia-provider master: Update to flake8>3.9.0  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/79837520:04
opendevreviewBrian Haley proposed openstack/ovn-octavia-provider master: Move linters dependencies to tox.ini  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/77906020:38
opendevreviewMerged openstack/neutron master: Correctly label port as SubPort in SubPortNotFound  https://review.opendev.org/c/openstack/neutron/+/78870621:27
opendevreviewBrian Haley proposed openstack/ovn-octavia-provider master: Move linters dependencies to tox.ini  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/77906021:27
opendevreviewBrian Haley proposed openstack/networking-ovn stable/train: Add Health Monitor support  https://review.opendev.org/c/openstack/networking-ovn/+/79606321:37
opendevreviewBrian Haley proposed openstack/ovn-octavia-provider master: Move linters dependencies to tox.ini  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/77906022:39

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