Yedidyah Bar David has uploaded a new change for review.

Change subject: packaging: setup: validate read-only user after input
......................................................................

packaging: setup: validate read-only user after input

Do not output "Error: user name cannot be empty" before asking for a
username.

Change-Id: Ia740c9ca659adc4b8ac7d80baad26312db61423c
Bug-Url: https://bugzilla.redhat.com/1057280
Signed-off-by: Yedidyah Bar David <[email protected]>
---
M packaging/ovirt-engine-dwh-setup.py
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-dwh refs/changes/77/23777/1

diff --git a/packaging/ovirt-engine-dwh-setup.py 
b/packaging/ovirt-engine-dwh-setup.py
index ead50ea..ca0bfd7 100755
--- a/packaging/ovirt-engine-dwh-setup.py
+++ b/packaging/ovirt-engine-dwh-setup.py
@@ -470,7 +470,9 @@
                         logging.debug('Skipping creation of read only DB 
user.')
                         print 'Skipping creation of read only DB user.'
                     elif options['CREATE_READONLY_USER'] is None:
-                        readonly_user = ''
+                        readonly_user = utils.askQuestion(
+                            question='Provide a username for read-only user'
+                        )
                         while not utils.userValid(readonly_user):
                             readonly_user = utils.askQuestion(
                                 question='Provide a username for read-only 
user'


-- 
To view, visit http://gerrit.ovirt.org/23777
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia740c9ca659adc4b8ac7d80baad26312db61423c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Yedidyah Bar David <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to