New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix broken condition introduced by #1213 #1214
Conversation
consoles/sshVirtsh.pm
Outdated
if ($stderr =~ /lock/i) { | ||
die "Too many attempts to format HDD" unless (--$bucket); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry but Pre-increment operator in negated post-if operator is a bit too much obfusciation for me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I personally liked it (except for the useless brackets).
verification run? |
See the description |
250cdbc
to
5822119
Compare
consoles/sshVirtsh.pm
Outdated
if ($stderr =~ /lock/i) { | ||
$bucket--; | ||
die "Too many attempts to format HDD" unless ($bucket); #Only die when there are no attempts left |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that comment is a little too obvious and we don't use () if it's a simple statement
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@coolo updated
Backend should only die when all of the attempts have finished and still the command is not successful.
5822119
to
5ce52d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last verification run: http://phobos.suse.de/tests/1748062
Travis failure seems unrelated... <.< |
I agree |
FiBackend should only die when all of the attempts have finished and still
the command is not successful.
This should solve the broken xen and zkvm jobs :)
Tested defining manually the domain using the same resource image, and triggering the jobs.