When binding container to network , port doesnt show in Horizon UI

Bug #1504001 reported by Gal Sagie
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kuryr
Fix Committed
High
Pengfei Ni

Bug Description

The connection between the port and network in Horizon UI "Network Topology" is not seen.
It seems the network has no ports.

(Need to add device_owner to port structure)

Gal Sagie (gal-sagie)
Changed in kuryr:
importance: Undecided → High
description: updated
Pengfei Ni (feiskyer)
Changed in kuryr:
assignee: nobody → feisky (feiskyer)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kuryr (master)

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

Pengfei Ni (feiskyer)
Changed in kuryr:
status: In Progress → Fix Committed
Revision history for this message
Irena Berezovsky (irenab) wrote :

I am not an expert on Horizon code, but it seems that to be present in the neutron topology, the entity should be one of the following entities: Network, Router, LB, Nova instance (VM/BM). I think special treatment will be required to present Container. I would like to suggest to update device_owner for kuryr plugged container with something like 'kuryr:container' or 'networking:container' and update device_id to be container ID.

Revision history for this message
Pengfei Ni (feiskyer) wrote :

@irena You are right. If set it to something like 'kuryr:container', then it will not be shown on Horizon.

@gal-sagie What's your opinion?

Revision history for this message
vikas choudhary (choudharyvikas16) wrote :

Its not merged yet, so setting status to "in progess".

Changed in kuryr:
status: Fix Committed → In Progress
Revision history for this message
Irena Berezovsky (irenab) wrote :

@feisky: I checked with Kilo OS release, and given this patch, Container is not shown in the topology view. But user can see list of ports connected to the networks and can see the port that Container plugged into with all relevant details (device_owner and device_id). Even if the 'nova:compute' helps to see container in Liberty, I would suggest to keep this patch as WIP and use for Demo but not merge (after all is a workaround for demo). I think the best would be to handle this properly by Horizon.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kuryr (master)

Reviewed: https://review.openstack.org/232472
Committed: https://git.openstack.org/cgit/openstack/kuryr/commit/?id=16250e932c27c089693ab48ec3a0b782104e7a57
Submitter: Jenkins
Branch: master

commit 16250e932c27c089693ab48ec3a0b782104e7a57
Author: Pengfei Ni <email address hidden>
Date: Thu Oct 8 18:56:44 2015 +0800

    Add device_owner while creating neutron port.

    1. Set device_id to endpoint id
    2. Set device_owner to kuryr:container

    Change-Id: Ib080ee59f6428732f341167088a99f726ef74891
    Closes-Bug: 1504001
    Signed-off-by: Pengfei Ni <email address hidden>

Changed in kuryr:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.