OCF script ns_IPaddr2 does not interpolate variables

Bug #1455961 reported by Igor Zinovik
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Matthew Mosesohn
6.0.x
Won't Fix
Medium
Fuel Library (Deprecated)
6.1.x
Won't Fix
Medium
Bartłomiej Piotrowski
7.0.x
Won't Fix
Medium
Fuel Library (Deprecated)
8.0.x
Fix Released
Medium
Matthew Mosesohn
Changed in fuel:
importance: Medium → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Bartlomiej Piotrowski (bpiotrowski)
status: New → In Progress
Changed in fuel:
assignee: Bartlomiej Piotrowski (bpiotrowski) → Bogdan Dobrelya (bogdando)
Changed in fuel:
assignee: Bogdan Dobrelya (bogdando) → Bartlomiej Piotrowski (bpiotrowski)
Changed in fuel:
importance: High → Medium
status: In Progress → Won't Fix
no longer affects: fuel/7.0.x
Changed in fuel:
milestone: 6.1 → 7.0
status: Won't Fix → Triaged
Changed in fuel:
assignee: Bartlomiej Piotrowski (bpiotrowski) → Sergii Golovatiuk (sgolovatiuk)
status: Triaged → In Progress
Changed in fuel:
assignee: Sergii Golovatiuk (sgolovatiuk) → Bartlomiej Piotrowski (bpiotrowski)
Changed in fuel:
assignee: Bartlomiej Piotrowski (bpiotrowski) → Sergii Golovatiuk (sgolovatiuk)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/183948
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=297a5575573832ce37e643a4ddceeebb0c3dfd34
Submitter: Jenkins
Branch: master

commit 297a5575573832ce37e643a4ddceeebb0c3dfd34
Author: Bartłomiej Piotrowski <email address hidden>
Date: Sun May 17 22:48:26 2015 +0200

    Run test command against actual variables in ns_IPaddr2

    Change-Id: I29bffcbb441cc402ede64982054f6dd94b1c643c
    Closes-Bug: 1455961

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Artem Panchenko (apanchenko-8) wrote :
Changed in fuel:
status: Fix Committed → In Progress
Changed in fuel:
status: In Progress → Won't Fix
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

I see why it was bad and verted.. you can't run a test like this:
var=""
var2="something"
[ -z $var ]
[ -z $var2 ]
The first will just fail because $var is not encapsulated in quotes.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
status: Won't Fix → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/226814
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=3508d8bc3499f580205f32e5cdaf1dbbcedf7728
Submitter: Jenkins
Branch: master

commit 3508d8bc3499f580205f32e5cdaf1dbbcedf7728
Author: Matthew Mosesohn <email address hidden>
Date: Wed Sep 23 17:21:41 2015 +0300

    Interpolate vars in ns_IPaddr2 ocf

    Corrected all bash tests for zero and non-zero
    length vars. Corrected missing quotes when checking
    for netmask and ns string length.

    Renamed [[ to [ where appropriate for -n and -z tests.

    Change-Id: Ic83a406ab7681222679edee6688f4ba1aa0a19aa
    Closes-Bug: #1455961

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
Changed in fuel:
status: Fix Committed → Fix Released
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.