Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1434688 - RHOS 11 fails to deploy sriov
RHOS 11 fails to deploy sriov
Status: CLOSED ERRATA
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-tripleo-heat-templates (Show other bugs)
11.0 (Ocata)
Unspecified Unspecified
urgent Severity urgent
: beta
: 11.0 (Ocata)
Assigned To: Saravanan KR
Maxim Babushkin
: AutomationBlocker, Regression, Triaged
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2017-03-22 03:50 EDT by Maxim Babushkin
Modified: 2017-05-17 16:10 EDT (History)
14 users (show)

See Also:
Fixed In Version: openstack-tripleo-heat-templates-6.0.0-0.7.el7ost
Doc Type: No Doc Update
Doc Text:
undefined
Story Points: ---
Clone Of:
Environment:
Last Closed: 2017-05-17 16:10:48 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Launchpad 1675036 None None None 2017-03-22 10:15 EDT
OpenStack gerrit 448600 None None None 2017-03-23 15:20 EDT
OpenStack gerrit 451168 None None None 2017-03-29 08:58 EDT
Red Hat Product Errata RHEA-2017:1245 normal SHIPPED_LIVE Red Hat OpenStack Platform 11.0 Bug Fix and Enhancement Advisory 2017-05-17 19:01:50 EDT

  None (edit)
Description Maxim Babushkin 2017-03-22 03:50:37 EDT
Description of problem:
RHOS 11 unable to deploy SR-IOV.
Get syntax error by using templates from RHOS 10.

Version-Release number of selected component (if applicable):
RHOS 11
2017-03-08.3 puddle

How reproducible:
Deploy RHOS 11 Undercloud
Deploy Overcloud with SR-IOV function

Actual results:
The Overcloud deployment fails with the following error:

Error: Evaluation Error: Error while evaluating a Function Call, 
Syntax error: [{"devname"=>"p5p1", "physical_network"=>"sriov"}] is not a String at 
/etc/puppet/modules/nova/manifests/compute.pp:233:29 on node compute-0.localdomain

Expected results:
The deployment should pass
Comment 1 Saravanan KR 2017-03-22 05:56:58 EDT
As per the initial analysis, with the change [1], the format of the nova::compute::pci_passthrough value is changed from string to JSON. But as per the puppet-nova implementation, the parameter is expected as JSON encoded string. Will provided more details after validating the fix. 


[1] https://github.com/openstack/tripleo-heat-templates/commit/787685101179f13f6074901070cf307bd9bb0731#diff-99a22d37b829681d157f41d35c38e4c5L100
Comment 2 Saravanan KR 2017-03-23 05:12:02 EDT
Proposed patch upstream - https://review.openstack.org/#/c/448600/
Comment 4 Nir Yechiel 2017-03-23 15:20:41 EDT
Fix is under review on master: https://review.openstack.org/448600

We will need it backported to stable/ocata
Comment 6 Yariv 2017-03-29 08:48:50 EDT
Saravanan

Please move to Post i see launchpad is merged.
https://review.openstack.org/#/c/448600/

Thanks
Comment 7 Nir Yechiel 2017-03-29 08:57:41 EDT
(In reply to Yariv from comment #6)
> Saravanan
> 
> Please move to Post i see launchpad is merged.
> https://review.openstack.org/#/c/448600/
> 
> Thanks

Need to backport to stable/ocata for RHOSP 11.
Comment 8 Saravanan KR 2017-03-30 01:24:16 EDT
Downport URL - https://code.engineering.redhat.com/gerrit/#/c/101910/
I will wait for CI and push it.
Comment 10 Eran Kuris 2017-04-04 10:08:18 EDT
Verified on OSP-11 

[stack@puma07 ~]$ rpm -qa |grep openstack-tripleo-heat-templates
openstack-tripleo-heat-templates-6.0.0-0.9.el7ost.noarch


Setup SRIOV 1 Compute  1 controller
Comment 11 errata-xmlrpc 2017-05-17 16:10:48 EDT
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2017:1245

Note You need to log in before you can comment on or make changes to this bug.