| opendevreview | Merged openstack/neutron stable/wallaby: bw-limit: Pass int parameters to Open vSwitch https://review.opendev.org/c/openstack/neutron/+/820612 | 00:19 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/neutron stable/victoria: bw-limit: Pass int parameters to Open vSwitch https://review.opendev.org/c/openstack/neutron/+/821895 | 00:28 |
| opendevreview | shanyunfan33 proposed openstack/neutron master: Add wait event for metadataagent sb_idl https://review.opendev.org/c/openstack/neutron/+/821927 | 02:50 |
| opendevreview | OpenStack Proposal Bot proposed openstack/neutron-vpnaas-dashboard stable/victoria: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/821930 | 03:46 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Update irrelevant-files for stadium project's tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/821414 | 06:41 |
| opendevreview | yatin proposed openstack/neutron master: List ports when attempt to delete network with ports https://review.opendev.org/c/openstack/neutron/+/821935 | 06:49 |
| opendevreview | Nitish Venkata Patcherla proposed openstack/neutron-lib master: Add api-ref for tap service and tap flow https://review.opendev.org/c/openstack/neutron-lib/+/821937 | 07:27 |
| ralonsoh | bcafarel, https://bugs.launchpad.net/neutron/+bug/1955010 | 08:20 |
| ralonsoh | we have a problem with py27 | 08:20 |
| ralonsoh | (something expected, to be hones) | 08:21 |
| ralonsoh | honest* | 08:21 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: Restructure move to nested virt nodes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/821939 | 08:21 |
| ralonsoh | the library failing is python-lazy-object-proxy. What should we do now? Cap this library version in py27 stable releases? | 08:21 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Adopt rehomed QoS FIP extension from neutron-lib 2.18.0 https://review.opendev.org/c/openstack/neutron/+/821838 | 08:24 |
| bcafarel | sigh :( | 08:29 |
| bcafarel | ralonsoh: that sounds like the way to go yes | 08:31 |
| ralonsoh | I'll push a patch, to check if that works | 08:33 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/train: [stable-only] Cap lazy-object-proxy to 1.7.0 in Python 2.7 https://review.opendev.org/c/openstack/neutron/+/821944 | 08:37 |
| bcafarel | ok so it is "only" functional and UT with python 2.7? (looking at recent train backports) | 08:37 |
| bcafarel | hopefully this can be capped easily with these jobs :) | 08:38 |
| ralonsoh | bcafarel, https://review.opendev.org/c/openstack/neutron/+/821566 | 08:38 |
| ralonsoh | tools like, yes | 08:38 |
| ralonsoh | only UT and FT in py27 | 08:38 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/train: [stable-only] Cap lazy-object-proxy to 1.6.0 in Python 2.7 https://review.opendev.org/c/openstack/neutron/+/821944 | 09:32 |
| opendevreview | Merged openstack/neutron master: Add upgrade check for floating IP QoS inheritance from network https://review.opendev.org/c/openstack/neutron/+/818399 | 10:08 |
| frickler | ralonsoh: this may be intersting to you, because you kind of triggered it: https://bugs.launchpad.net/neutron/+bug/1954384 | 10:16 |
| frickler | not sure yet whether to mark the bug as invalid or whether you would at least want to document the change better | 10:17 |
| ralonsoh | frickler, let me check | 10:17 |
| frickler | actually, some change is needed, because this code part will now never be triggered anymore https://opendev.org/openstack/neutron/src/branch/master/neutron/plugins/ml2/extensions/dns_integration.py#L375-L376 | 10:20 |
| ralonsoh | ok, I'll check the code in dns-integration and I'll document the change. But the VLAN range expansion was something needed by the VLAN driver | 10:23 |
| ralonsoh | and actually that was the assumption: when no vlan ranges are specified, all are available to this network | 10:23 |
| frickler | ralonsoh: the difference lies in what happens when you run "openstack network create test --provider-network-type vlan". in victoria this fails, because you need to explicitly specify vlan ID and physnet | 10:30 |
| frickler | with master, it will choose a random vlan ID and physnet public (in the default devstack setup) | 10:31 |
| ralonsoh | that's what was addressed with this change | 10:31 |
| ralonsoh | and now we reserver all VLAN IDs when Neutron server starts | 10:31 |
| ralonsoh | that was the intended behaviour | 10:32 |
| ralonsoh | I'll update the config option description | 10:32 |
| frickler | the thing is, that with the current code, dns_integration assumes that all these vlans can now be created by tenants and thus disallows dns entries to be created | 10:33 |
| ralonsoh | frickler, that's ok, the admin should restrict the VLAN range in the config | 10:33 |
| frickler | hmm, maybe it should check tenant_network_types in addition | 10:34 |
| ralonsoh | where in the code? | 10:34 |
| ralonsoh | I'm not following you | 10:34 |
| frickler | probably right at the start of _is_vlan_tenant_network() | 10:35 |
| frickler | let me propose a patch, that'll make it easier to discuss | 10:35 |
| frickler | might take a bit though, meetings upcoming | 10:35 |
| ralonsoh | for sure, that will be better to check a patch | 10:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Do not announce any DNS resolver if "0.0.0.0" or "::" provided https://review.opendev.org/c/openstack/neutron/+/820858 | 10:51 |
| opendevreview | yatin proposed openstack/neutron master: List ports when attempt to delete network with ports https://review.opendev.org/c/openstack/neutron/+/821935 | 11:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Amend "network_vlan_ranges" configuration option help text https://review.opendev.org/c/openstack/neutron/+/821959 | 11:09 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Update secure RBAC policies accrodingly to the new guidelines https://review.opendev.org/c/openstack/neutron/+/821208 | 11:17 |
| opendevreview | Merged openstack/neutron master: When creating a VXLAN interface, a device is mandatory https://review.opendev.org/c/openstack/neutron/+/821271 | 11:22 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [DNM] Test switch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/821965 | 11:31 |
| *** averdaguer is now known as averdaguer|brb | 11:31 | |
| *** averdaguer|brb is now known as averdaguer | 11:31 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/train: [stable-only] Cap lazy-object-proxy to 1.6.0 in Python 2.7 https://review.opendev.org/c/openstack/neutron/+/821944 | 11:34 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Remove the expired reservations in a separate DB transaction https://review.opendev.org/c/openstack/neutron/+/821967 | 11:39 |
| ralonsoh | bcafarel, https://a52a7f35e59a2014d9da-da3f945bb6e5ca02e53e7fc9f3f751d2.ssl.cf1.rackcdn.com/821944/3/check/requirements-check/e46574d/job-output.txt | 11:42 |
| ralonsoh | ERROR: Requirement '{Requirement(package='lazy-object-proxy', location='', specifiers='<=1.6.0', markers="python_version=='2.7'", comment='# BSD', extras=frozenset())}' not in openstack/requirements | 11:42 |
| ralonsoh | I don't know if we can modify requirements project for this | 11:42 |
| ralonsoh | to add "lazy-object-proxy" to the blacklist | 11:43 |
| ralonsoh | or is there a way to define a local backlist in Neutron? | 11:43 |
| ralonsoh | I'm going to ask in openstack-dev | 11:44 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Allocate IPs in bulk requests in separate transactions https://review.opendev.org/c/openstack/neutron/+/821727 | 11:45 |
| ralonsoh | slaweq, ^^ | 11:48 |
| ralonsoh | same question here, the problem we have in py27 and Train | 11:48 |
| ralonsoh | with a library I want to cap: lazy-object-proxy. But that will fail in requirements-check | 11:48 |
| ralonsoh | I'm asking this in openstack-dev, to know the correct procedure | 11:49 |
| bcafarel | ralonsoh: I think it is ok to go in requirements for it, like https://review.opendev.org/c/openstack/requirements/+/733007 | 11:50 |
| ralonsoh | bcafarel, thanks! I was looking for something like this | 11:50 |
| ralonsoh | thanks a lot, I'll push a patch for requirements | 11:50 |
| bcafarel | np, I guess it will also be useful at some point for other projects, we are probably not the only ones using that | 11:52 |
| ralonsoh | https://review.opendev.org/c/openstack/requirements/+/821972 | 12:02 |
| slaweq | ralonsoh: sorry, I missed Your ping earlier | 12:27 |
| slaweq | I agree with bcafarel here and I hope it can be accepted in requirements repo | 12:28 |
| ralonsoh | thanks! | 12:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Do not announce any DNS resolver if "0.0.0.0" or "::" provided https://review.opendev.org/c/openstack/neutron/+/820858 | 13:20 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Allocate IPs in bulk requests in separate transactions https://review.opendev.org/c/openstack/neutron/+/821727 | 13:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Remove the expired reservations in a separate DB transaction https://review.opendev.org/c/openstack/neutron/+/821967 | 13:35 |
| opendevreview | Bence Romsics proposed openstack/neutron master: Make the dead vlan actually dead https://review.opendev.org/c/openstack/neutron/+/820897 | 14:03 |
| bcafarel | ^ zombie vlan :) | 14:51 |
| holtgrewe | I have a question regarding policies. I'm running xena. I have a custom policy >>"create_port:fixed_ips:ip_address": "rule:context_is_advsvc or rule:network_owner or role:admin and system_scope:all or role:admin and project_id:%(project_id)s or rule:shared"<<. I have a project "hpc" and provide the RBAC policy action=access_as_shared target project=hpc network=private. | 15:05 |
| holtgrewe | Wait, wrong channel. | 15:05 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove _safe_plug_new method from the interface driver https://review.opendev.org/c/openstack/neutron/+/821998 | 16:05 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Make the Placement reporter compatible with OVN https://review.opendev.org/c/openstack/neutron/+/821855 | 16:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Read the initial config in the maintenance worker https://review.opendev.org/c/openstack/neutron/+/821865 | 16:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add floating IP pools extension to OVN L3 https://review.opendev.org/c/openstack/neutron/+/821818 | 16:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] [OVN][Placement] Drive binding by placement allocation https://review.opendev.org/c/openstack/neutron/+/786478 | 17:10 |
| *** tbachman_ is now known as tbachman | 17:24 | |
| *** tbachman_ is now known as tbachman | 17:32 | |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Add test_create_and_update_port_with_dns_name https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/821079 | 17:35 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Use Port_Binding up column to set Neutron port status https://review.opendev.org/c/openstack/neutron/+/821544 | 17:44 |
| *** tbachman_ is now known as tbachman | 17:55 | |
| opendevreview | Merged openstack/neutron master: Update placement allocation of bound ports when network QoS policy is changed https://review.opendev.org/c/openstack/neutron/+/818889 | 20:13 |
| rm_work | mlavalle: mind taking a glance at https://review.opendev.org/c/openstack/neutron/+/821779 and telling me if you think it's trivial enough to get merged without too much of a debate? 😀 | 20:34 |
| mlavalle | rm_work: I think it is. I'll look at it carefully latter today | 20:35 |
| mlavalle | :-) | 20:35 |
| rm_work | \o/ thanks | 20:37 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove _safe_plug_new method from the interface driver https://review.opendev.org/c/openstack/neutron/+/821998 | 20:57 |
| opendevreview | Arnau Verdaguer proposed openstack/neutron master: [ovn] Specify port type if it's a router port when updating https://review.opendev.org/c/openstack/neutron/+/821401 | 21:00 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Functional] Fix expected number of the enqueue_state_change calls https://review.opendev.org/c/openstack/neutron/+/822051 | 21:30 |
| opendevreview | Merged openstack/neutron master: Fix stackalytics' link https://review.opendev.org/c/openstack/neutron/+/821704 | 21:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!