Filter port-list based on security_groups associated not working

The request doesn't reflect the attribute that should be used to filter by security group, which is security_group_id.
So I doubt the request in this form should work as you expect.

You are right, filter passed need not be attribute name only.
For this issue, we should add a check in the server side, if filters contains security group,
then it should return Ports associated to the specified security group.

Fix proposed to branch: master
Review: https://review.openstack.org/180413

Change abandoned by Kyle Mestery (<email address hidden>) on branch: master
Review: https://review.openstack.org/180413
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Proposed a Blueprint.
https://review.openstack.org/#/c/221233/

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/180413
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

[Expired for neutron because there has been no activity for 60 days.]

Fix proposed to branch: master
Review: https://review.openstack.org/527105

For the record, once we switch ML2 to Port OVO, it should already work with the following filter name:

extra_filter_names = {'security_group_ids'}

We may need to extend client to support that, not sure.

Reviewed: https://review.openstack.org/527105
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=43d3e88a07b4275ad814c6875fa037efd94223bb
Submitter: Zuul
Branch: master

commit 43d3e88a07b4275ad814c6875fa037efd94223bb
Author: Ahmed Zaid <email address hidden>
Date: Wed Jan 17 09:32:59 2018 -0500

    Filter port-list based on security_groups.

    This patch allows users to filter ports depending on security groups.
    In addition to that I added a unit test to verify this change.
    TODO: move security_groups_port_filtering_lib.py into neutron-lib.

    Closes-Bug: 1405057
    Change-Id: I528719d895fc1d89f74e2ee85b9ddc470323c601

This issue was fixed in the openstack/neutron 13.0.0.0b1 development milestone.

Reopening. It appears this work was never completed; the api def never made it into neutron-lib and was consumed, so I'll do that.

Boden - thanks for picking this up, I had just noticed the same issue but you beat me to it:

WARNING neutron.api.extensions [req-1b8305d9-1be2-4ffd-b8b3-bfa3d84f81d2 - - - - -] Did not find expected name "Security_groups_port_filtering_lib" in /opt/stack/new/neutron/neutron/extensions/security_groups_port_filtering_lib.py

The name mis-match reminded me of other places that were fixed by moving to neutron-lib.

Reviewed: https://review.openstack.org/575243
Committed: https://git.openstack.org/cgit/openstack/neutron-lib/commit/?id=2544b1e906cb73607d38cf2198304d0097dff94c
Submitter: Zuul
Branch: master

commit 2544b1e906cb73607d38cf2198304d0097dff94c
Author: Boden R <email address hidden>
Date: Wed Jun 13 15:42:12 2018 -0600

    rehome port-security-groups-filtering api extension

    This patch rehomes the port-security-groups-filtering API extension as
    a neutron-lib API definition.

    Partial-Bug: 1405057

    Change-Id: I5e1f61a666c1001a71663466b1e70839ce6c8adb

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/652118

Change abandoned by Bernard Cafarelli (<email address hidden>) on branch: stable/queens
Review: https://review.opendev.org/652118
Reason: No updates in 6 months and overall opinion that it should not be backported, abandoning it.
Feel free to reopen if situation has changed

This issue was fixed in the openstack/python-openstackclient 5.8.0 release.