agent container: Add support for sriov

Bug #1723491 reported by Hari Prasad Killi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R5.0
Fix Committed
Critical
Prabhu Seshachellam
Trunk
Fix Committed
Critical
Prabhu Seshachellam

Bug Description

In 4.0 and above, we don't support dpdk and sriov in the agent container.

This will need

1. Changes in contrail-docker repo to modify json schema to validate against the /etc/contrailctl/agent.conf configuration.
2. Change in contrail-ansible to populate the the /etc/contrailctl/agent.conf
3. Change in src/vnsw/provisoning python vrouter provisioning script to read the inputs from /etc/contrailctl/agent.conf

Revision history for this message
Kiran (kiran-kn80) wrote :

Hi Mike, pl reassign.

Jeba Paulaiyan (jebap)
tags: added: sanityblocker
Revision history for this message
Abhay Joshi (abhayj) wrote :

Prabhu will look into SRIOV part. We will keep this bug for DPDK (which is being worked on by Kiran and Michael) and assign to Michael. The SRIOV portion Prabhu will take a stab at it (he will open a new bug for that). In case, he is not able to take care may reassign as needed.

Revision history for this message
Prabhu Seshachellam (sprabhu72) wrote :

Hi!

It looks like dpdk support is available in agent container as per latest contrail-ansible-deployer repo.

I see that only sriov is missing. I'm looking into that.

summary: - agent container: support dpdk and sriov
+ agent container: Add support for sriov
Revision history for this message
Prabhu Seshachellam (sprabhu72) wrote :

dpdk support for 5.0 is already available with contrail-ansisble-deployer.

There is no need for a separate docker router agent container for sriov. It looks like,
sriov needs to be enabled during provisioning of vrouter. ( similar to specifying sriov in server.json file }

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/41779
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] contrail/ocata

Review in progress for https://review.opencontrail.org/41843
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/41779
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] contrail/ocata

Review in progress for https://review.opencontrail.org/41843
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/41779
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/41779
Committed: http://github.com/Juniper/contrail-ansible-deployer/commit/7d02a176822187afd7e8bcd000f9dca99424c058
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 7d02a176822187afd7e8bcd000f9dca99424c058
Author: Prabhu Seshachellam <email address hidden>
Date: Wed Apr 11 21:55:05 2018 -0700

Agent container Support for SR-IOV

Moved setting VF to vrouter container init.

Change-Id: Ia1b82bca92042db38d1079ca87e0504023fa7de1
Closes-Bug: #1723491

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] contrail/ocata

Review in progress for https://review.opencontrail.org/41843
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
Prabhu Seshachellam (sprabhu72) wrote :

Two more commits are pending for this bug. Both are in review.

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/42109
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/42109
Submitter: Andrey Pavlov (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/42109
Committed: http://github.com/Juniper/contrail-container-builder/commit/82071cc5af1aa4585bab84580a0df5ae56e5d45e
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 82071cc5af1aa4585bab84580a0df5ae56e5d45e
Author: Prabhu Seshachellam <email address hidden>
Date: Wed Apr 18 12:20:11 2018 -0700

vrouter agent init for SRIOV

set the VFs for given sriov phy interface

Change-Id: I8ed2a1f3286913b64b84f5d9f74a50e409620e25
Closes-Bug: #1723491

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0

Review in progress for https://review.opencontrail.org/42202
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/42203
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/42202
Committed: http://github.com/Juniper/contrail-container-builder/commit/26b0914a4c314c49ef0bf1dbc227c92fdc8656d8
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0

commit 26b0914a4c314c49ef0bf1dbc227c92fdc8656d8
Author: Prabhu Seshachellam <email address hidden>
Date: Wed Apr 18 12:20:11 2018 -0700

vrouter agent init for SRIOV

set the VFs for given sriov phy interface

Change-Id: I8ed2a1f3286913b64b84f5d9f74a50e409620e25
Closes-Bug: #1723491
(cherry picked from commit 82071cc5af1aa4585bab84580a0df5ae56e5d45e)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/41843
Committed: http://github.com/Juniper/contrail-kolla-ansible/commit/798fbae259077939db670fe87796fa36110dcafe
Submitter: Zuul v3 CI (<email address hidden>)
Branch: contrail/ocata

commit 798fbae259077939db670fe87796fa36110dcafe
Author: Prabhu Seshachellam <email address hidden>
Date: Thu Apr 12 14:21:36 2018 -0700

sriov support

Nova related changes for sriov - Incorporated review comments.

Change-Id: I950fa78824754135c5ce4b6ed073bcdaa963ed5e
Closes-Bug: #1723491

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] contrail/ocata

Review in progress for https://review.opencontrail.org/42353
Submitter: Prabhu Seshachellam (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/42353
Committed: http://github.com/Juniper/contrail-kolla-ansible/commit/2145c8753fe55bc380498ac606a9f944cf9db36d
Submitter: Zuul v3 CI (<email address hidden>)
Branch: contrail/ocata

commit 2145c8753fe55bc380498ac606a9f944cf9db36d
Author: Prabhu Seshachellam <email address hidden>
Date: Fri Apr 20 16:43:03 2018 -0700

sriov support

Fixed a minor with sriov variable in nova template

Change-Id: I37104aab3c873449747b17b14fa6c0720999d690
Closes-Bug: #1723491

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.