x/ci-cd-pipeline-app-murano
Code Issues Proposed changes

Fixing F5 package issue

Browse Source 
* method installLBaaS is supposed to return dict,
   now F5 class returns exactly what it gets from
   LBaaS class because port and path is not changed
 * Minor changes related to removing unneeded argument
   from installLBaaS method.

Change-Id: I7192272be467cb34a7daa418f8833bdfee344b8d
This commit is contained in:
Nikolay Mahotkin 2016-04-06 17:07:08 +03:00
parent 83bdd28931
commit 4b391a7903
2 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
murano-apps/F5-based-LBaaS/package/Classes/F5BigIPLTM.yaml
View File

@ -36,11 +36,8 @@ Methods:
- $.implementation: f5ltm - $.implementation: f5ltm
installLBaaS: installLBaaS:
Arguments:
- implementation:
Contract: $.string().notNull()
Body: Body:
- $lbaas: $.super($.installLBaaS($implementation)) - $lbaas: $.cast(LBaaS).installLBaaS()
- $resources: new(sys:Resources) - $resources: new(sys:Resources)
- $template: $resources.yaml('DeployF5Driver.template') - $template: $resources.yaml('DeployF5Driver.template')

5
murano-apps/LBaaS-interface/Classes/LoadBalancer.yaml
View File

@ -64,7 +64,7 @@ Methods:
- $.environment.reporter.report($this, 'Instance is created.') - $.environment.reporter.report($this, 'Instance is created.')
- $resources: new(sys:Resources) - $resources: new(sys:Resources)
- $.installLoadBalancer() - $.installLoadBalancer()
- $lbaas: $.installLBaaS($.implementation) - $lbaas: $.installLBaaS()
- $.configureLBaaS() - $.configureLBaaS()
- $.startLBaaS() - $.startLBaaS()
- If: $.instance.assignFloatingIp - If: $.instance.assignFloatingIp
@ -113,9 +113,6 @@ Methods:
installLBaaS: installLBaaS:
# Installs LBaaS itself and its drivers on an instance. # Installs LBaaS itself and its drivers on an instance.
Arguments:
- implementation:
Contract: $.string().notNull()
Body: Body:
- $resources: new(sys:Resources) - $resources: new(sys:Resources)
- $template: $resources.yaml('DeployLBaaS.template') - $template: $resources.yaml('DeployLBaaS.template')