Thursday, 2023-02-16

opendevreviewMerged openstack/neutron-fwaas master: Add CONTEXT_WRITER to _get_default_fwg_id  https://review.opendev.org/c/openstack/neutron-fwaas/+/87389700:41
opendevreviewDmitrii Shcherbakov proposed openstack/neutron master: [WIP] Allow Multiple External Gateways  https://review.opendev.org/c/openstack/neutron/+/87359301:05
opendevreviewMerged openstack/neutron stable/ussuri: Fix deprecation warnings in pkg_resources  https://review.opendev.org/c/openstack/neutron/+/87378301:26
opendevreviewZhouHeng proposed openstack/neutron master: [ovn]disable security group notifier  https://review.opendev.org/c/openstack/neutron/+/87385001:34
*** dmitriis is now known as Guest499503:31
opendevreviewZhouHeng proposed openstack/neutron master: [ovn]disable security group notifier  https://review.opendev.org/c/openstack/neutron/+/87385006:04
opendevreviewyatin proposed openstack/neutron-fwaas stable/zed: Add get_firewall_groups_count method  https://review.opendev.org/c/openstack/neutron-fwaas/+/87396006:12
opendevreviewyatin proposed openstack/neutron-fwaas stable/zed: Add CONTEXT_WRITER to _get_default_fwg_id  https://review.opendev.org/c/openstack/neutron-fwaas/+/87396106:13
lajoskatonasahid: Hi, We have a bug in fullstack for multisegments: https://bugs.launchpad.net/neutron/+bug/2007152 , it seems to appear only with fullstack07:14
lajoskatonasahid: Could you please check it if you have some free time?07:14
opendevreviewyatin proposed openstack/neutron master: [dhcp agent] Fix clean_devices  https://review.opendev.org/c/openstack/neutron/+/87405707:35
sahidlajoskatona: o/08:06
lajoskatonasahid: Hi08:06
sahidsure looking at it08:06
lajoskatonasahid: thanks08:07
ralonsohykarel, hi! about the upper patch you pushed (that I think is correct), where did you see this error? what CI job?08:09
ykarelralonsoh, can be seen in all ovs jobs08:09
ykareli noticed while checking udhcpc issue in nova-next08:10
ralonsohykarel, thanks08:10
ykarelralonsoh, ex https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_6f9/868236/5/gate/nova-next/6f9f3d0/controller/logs/screen-q-dhcp.txt08:11
ralonsohuffff hundred of times...08:11
opendevreviewyatin proposed openstack/neutron-fwaas master: Use upper-constraints in docs job  https://review.opendev.org/c/openstack/neutron-fwaas/+/87406208:20
opendevreviewyatin proposed openstack/neutron-fwaas stable/zed: Use upper-constraints in docs job  https://review.opendev.org/c/openstack/neutron-fwaas/+/87406308:21
elodillesralonsoh: thx o/08:32
ralonsohyw08:32
ralonsohlajoskatona, slaweq please check https://review.opendev.org/c/openstack/os-ken/+/87402808:36
slaweqralonsoh +208:37
ralonsohthanks08:37
opendevreviewLuis Tomas Bolivar proposed openstack/ovsdbapp master: Add new function ls_get_localnet_ports  https://review.opendev.org/c/openstack/ovsdbapp/+/87385308:45
opendevreviewyatin proposed openstack/neutron-fwaas stable/zed: Use upper-constraints in docs job  https://review.opendev.org/c/openstack/neutron-fwaas/+/87406308:59
opendevreviewLajos Katona proposed openstack/neutron stable/xena: Fullstack: Wait placement process fixtrue to really stop  https://review.opendev.org/c/openstack/neutron/+/87396209:17
opendevreviewLajos Katona proposed openstack/neutron stable/yoga: Do not ignore attributes in bulk port create  https://review.opendev.org/c/openstack/neutron/+/87396309:18
sahidlajoskatona: you shared link to "openseaarch" it seems that a user/passzord is needed?09:38
lajoskatonasahid: openstack/openstack :-)09:39
sahidthanks you09:39
lajoskatonasahid: it is for the community, so no personal credentials are needed09:39
sahid:-)09:39
opendevreviewRodolfo Alonso proposed openstack/neutron master: Allow other kwargs in ``DhcpBase.disable``  https://review.opendev.org/c/openstack/neutron/+/87407309:49
ralonsohsahid, please check this error after ^^ and https://review.opendev.org/c/openstack/neutron/+/87405709:50
ralonsohmaybe these patches can help 09:50
sahidquick question guys, we hqve some dup entries in ml2_port_bindings, port_id/host{1,2,3} I guess it's when we do a migration09:50
sahidI'm zwondering if those dup port_id should have been cleaned09:51
sahidralonsoh: ack i'm going to look at it09:51
ralonsohplease print the ml2_port_bindings table (the dup registers)09:51
ralonsohbut most probably is due to a failed migration09:51
sahidralonsoh: yes it's what i have in mind, the question is should we remove it? (and fix neutron for that)09:52
ralonsohthere is no fix in Neutron, Nova commands the port binding09:52
ralonsohwe have a tool to fix that09:52
sahidoh...09:52
ralonsohcmd/remove_duplicated_port_bindings09:53
sahidi think i remember of that now09:53
ralonsohyou can execute it in dry-out mode09:53
ralonsohonly printing the errors09:53
ralonsohdry_run09:53
sahidperfect, thanks!09:53
sahidreporting your point and i will look at the patch you shares to me, thanks you09:54
opendevreviewMerged openstack/os-ken master: Fix tox.ini for tox4  https://review.opendev.org/c/openstack/os-ken/+/87402810:20
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Avoid use of ovn metadata port IP for HM checks  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/87342610:22
opendevreviewBence Romsics proposed openstack/neutron master: DNM debug logs and dev helper scripts  https://review.opendev.org/c/openstack/neutron/+/87290610:42
opendevreviewyatin proposed openstack/neutron-fwaas stable/zed: Add get_firewall_groups_count method  https://review.opendev.org/c/openstack/neutron-fwaas/+/87396010:51
opendevreviewyatin proposed openstack/neutron-fwaas stable/zed: Add CONTEXT_WRITER to _get_default_fwg_id  https://review.opendev.org/c/openstack/neutron-fwaas/+/87396110:52
opendevreviewMerged openstack/neutron-fwaas stable/zed: Use upper-constraints in docs job  https://review.opendev.org/c/openstack/neutron-fwaas/+/87406311:55
opendevreviewMerged openstack/neutron master: [dhcp agent] Fix clean_devices  https://review.opendev.org/c/openstack/neutron/+/87405712:03
ralonsohykarel, hi (I don't know if it is too late for you)12:05
ralonsohhttps://zuul.opendev.org/t/openstack/build/1431f0bd980c4af5b353c507f958c44512:05
ralonsoh"test_network_basic_ops" is failing too often12:05
ralonsohcould it be related to the metadata flows?12:05
ykarelralonsoh, nope not late12:05
ykarellet me check12:05
opendevreviewFrode Nordahl proposed openstack/neutron master: [WIP] Allow Multiple External Gateways  https://review.opendev.org/c/openstack/neutron/+/87359312:06
ykarelralonsoh, yes it's that issue, no output:230 in https://e77ca85d21cbdb712ce7-c684a40c384a27e613f3f0e997584032.ssl.cf1.rackcdn.com/874073/1/check/neutron-ovn-tempest-ipv6-only-ovs-release/1431f0b/controller/logs/screen-br-int-flows.txt12:08
ralonsohok, I'll push a patch to skip this test12:08
ralonsohsame as https://review.opendev.org/c/openstack/neutron/+/873684/2/zuul.d/tempest-singlenode.yaml#74312:08
ykarelralonsoh, ack but it could happen to any test, let me open an issue in ubuntu if ovn can be updated there12:10
ralonsohperfect12:10
ykarelbut till then we can skip tests12:11
ykarelor if needed make job non-voting12:11
ralonsohI prefer to avoid this test only (for now)12:12
opendevreviewRodolfo Alonso proposed openstack/neutron master: [ovn-ipv6] Temporary exclude ``test_network_basic_ops``  https://review.opendev.org/c/openstack/neutron/+/87409312:15
*** dasm|off is now known as dasm14:00
opendevreviewRodolfo Alonso proposed openstack/neutron master: [ovn-ipv6] Temporary exclude ``TestNetworkBasicOps`` class  https://review.opendev.org/c/openstack/neutron/+/87409314:17
*** arnau__ is now known as averdaguer14:34
opendevreviewFrode Nordahl proposed openstack/neutron master: [WIP] Allow Multiple External Gateways  https://review.opendev.org/c/openstack/neutron/+/87359314:36
ykarelralonsoh, so wile creating bug found https://bugs.launchpad.net/ubuntu/+source/ovn/+bug/2003056 so new version should be available in some time14:50
ralonsohykarel, thanks. BTW, should we trying manually compiling a newer version in "neutron-ovn-tempest-ipv6-only-base"?14:51
ralonsohthat could work to (until the ubuntu bug is fixed)14:51
ykarelralonsoh, hmm could be done and that should even verify the fixes14:53
ralonsohperfect, I'll do it14:53
ykareljust that job name is -release but we compiling but should be fine as we doing it temporary and others are knowing it14:53
ralonsohhmmm right, this is a "release" job indeed14:57
*** dkehn_ is now known as dkehn14:59
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Remove "update_port_qos_with_external_ids_reference"  https://review.opendev.org/c/openstack/neutron/+/87410515:00
opendevreviewnorman shen proposed openstack/neutron master: Filter out unsatisfied routers in SQL  https://review.opendev.org/c/openstack/neutron/+/84214115:17
opendevreviewMaor Blaustein proposed openstack/neutron-tempest-plugin master: Drop stateless_sg property check from base method  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/87410915:35
opendevreviewyatin proposed openstack/neutron master: [DNM] Check issue and fix with ovn source installs  https://review.opendev.org/c/openstack/neutron/+/87411215:50
opendevreviewBernard Cafarelli proposed openstack/neutron stable/xena: Do not ignore attributes in bulk port create  https://review.opendev.org/c/openstack/neutron/+/87413015:59
opendevreviewBernard Cafarelli proposed openstack/neutron stable/wallaby: Do not ignore attributes in bulk port create  https://review.opendev.org/c/openstack/neutron/+/87413115:59
opendevreviewMerged openstack/neutron-fwaas stable/zed: Add get_firewall_groups_count method  https://review.opendev.org/c/openstack/neutron-fwaas/+/87396016:05
opendevreviewMerged openstack/neutron-fwaas stable/zed: Add CONTEXT_WRITER to _get_default_fwg_id  https://review.opendev.org/c/openstack/neutron-fwaas/+/87396116:13
*** dasm is now known as Guest504616:46
vishalmanchandahaleyb: this query is regarding floating-ip-port-forwarding, you were right some issue with my env. that why I am getting error "ResourceNotFound: 404" which is fixed now.17:04
haleybvishalmanchanda: ack, it sounded like an extension not loaded error since it's a complaint about the path not found. glad its working now17:05
vishalmanchandahaleyb: but now I hitting "BadRequestException: 400:  Invalid input for protocol. Reason: TCP is not in valid_values", any idea what I am doing wrong?17:05
ralonsohvishalmanchanda, use lowercase17:07
vishalmanchandaralonsoh: ok thanks17:09
vishalmanchandait worked17:10
vishalmanchandahaleyb: ralonsoh : thanks for the help17:11
*** dasm is now known as Guest505218:10
guilhermesp_____hey there! im looking into making reader role to work on neutron. It apparently works on zed when we enable enforce_scope and enforce_new_defaults to true, however, as a member user when i try to attach a port to a vm, neutron returns that the port could not be found -- is there a know issue opened in regards to the usage of reader role with neutron? 18:57
haleybguilhermesp_____: it would be best to ask slaweq about rbac questions, don't know if he's still around today19:09
guilhermesp_____np haleyb ! we are talking to nova team as well to see if its related 19:22
haleybralonsoh: you still around? had a question on a bug19:31
slaweq@guilhermesp_____ hi. I'm not at my computer now but there were some fixes related to new default rules there. Please compare what you have with current master. Maybe you will find differences there. If not, please ping me tomorrow, I can try to look at this19:42
guilhermesp_____hi slaweq i think maybe this https://github.com/openstack/neutron/commit/6d8ada0ac93beed05b45adb9582c3ef23bef49d2 im trying to see if the same happens to a member user since i could reproduce the bug with admin role19:46
gmannslaweq: ralonsoh: we need to backport these to applicable stable branches https://review.opendev.org/c/openstack/neutron/+/865040  https://review.opendev.org/c/openstack/neutron/+/865032  https://review.opendev.org/c/openstack/neutron-lib/+/864213  https://review.opendev.org/c/openstack/neutron-lib/+/864809 21:57
opendevreviewGhanshyam proposed openstack/neutron-lib stable/zed: Do not set system scope on context  https://review.opendev.org/c/openstack/neutron-lib/+/87413321:58
ralonsohgmann, I'LL DO IT TOMORROW, THANKS22:04
ralonsohsorry!!22:04
ralonsoh(uffff too late for me, sorry for the uppercase)22:05
gmannralonsoh: np. thanks 22:06
opendevreviewMerged openstack/neutron master: Improve scheduling L3/DHCP agents, missing lower binding indexes  https://review.opendev.org/c/openstack/neutron/+/87310722:14
opendevreviewBrian Haley proposed openstack/neutron master: WIP: Change DHCP agent setup code to deal with small MTUs  https://review.opendev.org/c/openstack/neutron/+/87416722:26
*** dasm_ is now known as dasm|off23:55

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