Bug 1054616

Summary:	Power configurations with no entry for their password use 'None' as the password
Product:	[Retired] Beaker	Reporter:	Raymond Mancy <rmancy>
Component:	lab controller	Assignee:	Raymond Mancy <rmancy>
Status:	CLOSED CURRENTRELEASE	QA Contact:	tools-bugs <tools-bugs>
Severity:	high	Docs Contact:
Priority:	high
Version:	0.14	CC:	aigao, asaha, dcallagh, ebaak, llim, qwan, rmancy, xjia
Target Milestone:	0.15.3	Keywords:	Regression
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-02-03 04:51:54 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Raymond Mancy 2014-01-17 06:21:13 UTC

Description of problem:

If there is no password set for power configurations then the power script is passed the string 'None'

Version-Release number of selected component (if applicable):

0.14.1

How reproducible:

Always

Steps to Reproduce:
1. Set a blank power password
2. Power cycle the system
3.

Actual results:

Fails because incorrect password is given

Expected results:

Works.

Additional info:

Comment 3 Raymond Mancy 2014-01-17 08:05:20 UTC

The problem seems to be that we are passing None to SensitiveUnicode() in provision.py and it just converts it to a Unicode object. We should avoid passing the password value to SensitiveUnicode() unless it is not None.

Comment 4 Raymond Mancy 2014-01-20 22:39:49 UTC

http://gerrit.beaker-project.org/#/c/2702/

Comment 7 Nick Coghlan 2014-02-03 04:51:54 UTC

This change is included in the Beaker 0.15.3 maintenance release:

http://beaker-project.org/docs/whats-new/release-0.15.html#beaker-0-15-3