)]}'
{"id":"openstack%2Fnetworking-generic-switch~975339","triplet_id":"openstack%2Fnetworking-generic-switch~master~I8aaf432b8c00f198c54694afb347105187382162","project":"openstack/networking-generic-switch","branch":"master","topic":"poc-l2vni-nxos-mcast-range","attention_set":{},"removed_from_attention_set":{"24245":{"account":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"last_update":"2026-02-03 22:41:05.000000000","reason":"Change was abandoned"}},"hashtags":[],"change_id":"I8aaf432b8c00f198c54694afb347105187382162","subject":"DNM - Claude generated mcasta range support","status":"ABANDONED","created":"2026-01-30 21:44:22.000000000","updated":"2026-02-03 22:41:05.000000000","total_comment_count":1,"unresolved_comment_count":1,"has_review_started":true,"meta_rev_id":"11dbc69a5e70b92257c28d75c14fc7ee64401488","_number":975339,"virtual_id_number":975339,"owner":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2026-01-30 22:47:17.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Do Not Backport"," 0":"Backport Review Needed","+1":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"CC":[{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"}],"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2026-01-30 21:51:13.000000000","updated_by":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"reviewer":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"state":"CC"},{"updated":"2026-01-30 22:47:17.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"c023c1d8a734970a736db0e196cdc161f354a87e","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"date":"2026-01-30 21:44:22.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"155a077c2e5d5d6880d3b68541105a9abd4d27f1","author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"date":"2026-01-30 21:51:13.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"46937617c669db27bcdd3d5de69af2cd167316a0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2026-01-30 22:47:17.000000000","message":"Patch Set 1: Verified-1\n\n(21 comments)\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/6baee2968a114715a2c4fd3790d8126b\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2942534b40714806929dadf3eaaca030 : FAILURE in 2m 34s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/3d1aea54824b4368ae8a2cade12802b1 : SUCCESS in 2m 42s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/d243ea99d3ef454aa93fe806a3991dca : SUCCESS in 2m 23s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/31b407416a694682be8ee2bda8e61a14 : SUCCESS in 7m 32s\n- networking-generic-switch-tempest-dlm https://zuul.opendev.org/t/openstack/build/72b58c7986924042b5fccec934c2a9b8 : SUCCESS in 17m 44s\n- ironic-tempest-ipa-wholedisk-direct-multinode https://zuul.opendev.org/t/openstack/build/f61daed593204921bc1abe6b1ab881b7 : SUCCESS in 56m 42s\n- networking-generic-switch-tox-codespell https://zuul.opendev.org/t/openstack/build/e0040b2569fa4d0e8e0e4c429607d213 : SUCCESS in 3m 13s","accounts_in_message":[],"_revision_number":1},{"id":"11dbc69a5e70b92257c28d75c14fc7ee64401488","tag":"autogenerated:gerrit:abandon","author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"date":"2026-02-03 22:41:05.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"2bb999dad601c31a8005632f304504c29d94a10b","revisions":{"2bb999dad601c31a8005632f304504c29d94a10b":{"kind":"REWORK","_number":1,"created":"2026-01-30 21:44:22.000000000","uploader":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"ref":"refs/changes/39/975339/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/networking-generic-switch","ref":"refs/changes/39/975339/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/39/975339/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/39/975339/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/networking-generic-switch refs/changes/39/975339/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/networking-generic-switch refs/changes/39/975339/1"}}},"commit":{"parents":[{"commit":"27a74db95b50b3eec3d3ec59492b82bda257c9f5","subject":"WIP: l2vni plug case with Cisco NXOS","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/27a74db95b50b3eec3d3ec59492b82bda257c9f5"}]}],"author":{"name":"Harald Jensås","email":"hjensas@redhat.com","date":"2026-01-30 19:26:43.000000000","tz":60},"committer":{"name":"Harald Jensås","email":"hjensas@redhat.com","date":"2026-01-30 21:44:07.000000000","tz":60},"subject":"DNM - Claude generated mcasta range support","message":"DNM - Claude generated mcasta range support\n\nThis is just a draft, generated by Claude Code.\n\nAdd CIDR multicast address range support for L2VNI\n\nExtends Cisco NX-OS driver to support multicast address ranges\nusing CIDR notation for L2VNI configuration. VNIs are mapped to\nmulticast addresses using SHA256-based consistent hashing, providing\ndeterministic mapping without coordination overhead.\n\nSupported formats for both per-physnet and global fallback:\n- Single IP: 239.1.1.1 (backward compatible)\n- CIDR: 239.1.1.0/24 (hash-based VNI mapping)\n\nKey features:\n- All addresses in CIDR range are used (no network/broadcast\n  exclusion as multicast doesn\u0027t have these concepts)\n- SHA256-based consistent hashing ensures same VNI always maps\n  to same address\n- Even distribution across available multicast addresses\n- No distributed coordination required (deterministic algorithm)\n- Backward compatible with single IP configuration\n- Vendor-agnostic utilities in devices.utils for code reuse:\n  * parse_physnet_mcast_config() - parse physnet:range config\n  * parse_mcast_range() - parse single IP or CIDR\n  * get_mcast_for_physnet() - resolve mcast address for physnet/VNI\n  * hash_vni_to_mcast() - deterministic VNI-to-address mapping\n- Documentation includes warnings about range immutability\n\nBenefits:\n- Improved scalability (256 addresses with /24)\n- Better BUM traffic isolation (dedicated multicast tree per VNI)\n- Fabric optimization for modern VXLAN deployments\n- More efficient address usage (all addresses valid for multicast)\n- Reusable by other vendors (Arista, SONiC, etc.)\n\nIMPORTANT - Range Immutability Warning:\nThe multicast address range MUST NOT be changed after initial deployment.\nChanging the range will cause VNIs to map to different multicast addresses,\nbreaking existing VXLAN tunnels. Plan the range size carefully based on\nmaximum expected VNI count. If more addresses are needed later, add a new\nphysical network with a different range rather than modifying existing ones.\n\nConfiguration example:\n  ngs_physnet_mcast_group \u003d datacenter1:239.1.1.0/24,\n                            datacenter2:239.1.2.0/26,\n                            tenant-net:239.1.3.1\n\nAssisted-By: Claude Code/claude-4.5-sonnet\nChange-Id: I8aaf432b8c00f198c54694afb347105187382162\nSigned-off-by: Harald Jensås \u003chjensas@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/2bb999dad601c31a8005632f304504c29d94a10b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/networking-generic-switch/commit/2bb999dad601c31a8005632f304504c29d94a10b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Backport-Candidate","description":"Backport candidate status","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}