Yedidyah Bar David has posted comments on this change.
Change subject: packging: fixed remote install setup
......................................................................
Patch Set 2:
(4 comments)
....................................................
File packaging/legacy-setup/ovirt-engine-dwh-setup.py
Line 391:
Line 392: # Check that oVirt-Engine is installed, otherwise exit
gracefully with an informative message
Line 393: if not isOvirtEngineInstalled():
Line 394: logging.debug("ovirt-engine is not set up, cannot
continue")
Line 395: print "Please install & setup oVirt engine by executing
\"engine-setup\" prior to setting up the data warehouse."
just the text - "install" -> "setup"
choose "Reference Version" "Patch Set 1" and you can see that exactly.
Line 396: return 0
Line 397:
Line 398: db_dict = getDbDictFromOptions()
Line 399: PGPASS_TEMP = utils.createTempPgpass(db_dict)
Line 498: dbExists, owned, hasData, working_db_dict =
getDBStatus(db_dict, PGPASS_TEMP)
Line 499: if not dbExists:
Line 500: print 'Could not connect to remote
database - please try again.\n'
Line 501: else:
Line 502: # Why do we allow host/port options? If they are
different from engine db, we won't be able to connect to it
It is - if engine and dwh are on different db hosts, and there is just one
'host' value, either one will fail or the other.
Line 503: db_dict['host'] = options['REMOTE_DB_HOST']
Line 504: db_dict['port'] = options['REMOTE_DB_PORT']
Line 505: db_dict['username'] = options['REMOTE_DB_USER']
Line 506: db_dict['password'] =
options['REMOTE_DB_PASSWORD']
Line 569: if doBackup:
Line 570: print DB_FILE.format(
Line 571: dbfile=backupFile
Line 572: )
Line 573: print DB_RESTORE
Yes. I just copy/pasted. Probably here is enough - I'll remove the other.
Line 574:
Line 575: if utils.localHost(db_dict['host']) and not owned:
Line 576: if not userExists(db_dict['username']):
Line 577: utils.createUser(
Line 654: logging.debug("user chose not to stop engine")
Line 655: print "Installation stopped, Goodbye."
Line 656:
Line 657: logging.debug("main() ended")
Line 658: except:
That was your previous comment... isn't it enough? Just to output it right
after backup, so that if it fails later user already has the message?
Line 659: logging.error("Exception caught!")
Line 660: logging.error(traceback.format_exc())
Line 661: print "Error encountered while installing %s, please consult
the log file: %s" % (DWH_PACKAGE_NAME,log_file)
Line 662: rc = 1
--
To view, visit http://gerrit.ovirt.org/22701
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic315856f91c60f240c6830fb90e5b259774e9745
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Yaniv Dary <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yaniv Dary <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
