Eucalyptus

Merge lp:~graziano.obertelli/eucalyptus/lp738164 into lp:eucalyptus/2.0

  1. lp738164
  2. Merge into 2.0.0-lp
Proposed by graziano obertelli
Status: Merged
Merged at revision: 1260
Proposed branch: lp:~graziano.obertelli/eucalyptus/lp738164
Merge into: lp:eucalyptus/2.0
Diff against target: 12 lines (+1/-1)
1 file modified
tools/get_iscsitarget.pl (+1/-1)
To merge this branch: bzr merge lp:~graziano.obertelli/eucalyptus/lp738164
Reviewer Review Type Date Requested Status
graziano obertelli Approve
Review via email: mp+101996@code.launchpad.net

Description of the change

Fix for lp:738164 (thanks zap!).

To post a comment you must log in.
Revision history for this message
graziano obertelli (graziano.obertelli) wrote :

The patch was floated around #eucalyputs-devel and fast-tracked due to its simplicity (one liner). Since we are short of QA cycle, we'll QA it when 2.0.4 comes out. Approved.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tools/get_iscsitarget.pl'
2--- tools/get_iscsitarget.pl 2010-06-17 23:25:50 +0000
3+++ tools/get_iscsitarget.pl 2012-04-14 01:07:23 +0000
4@@ -35,7 +35,7 @@
5 $found_target = 0;
6 while (<GETSESSION>) {
7 if($_ =~ /Target: (.*)\n/) {
8- $found_target = 1 if $1 == $store;
9+ $found_target = 1 if $1 eq $store;
10 } elsif($_ =~ /.*Attached scsi disk ([a-zA-Z0-9]+).*\n/) {
11 if($found_target == 1) {
12 return "/dev/", $1;

Subscribers

People subscribed via source and target branches