[node-patches] Change in ovirt-node[master]: tui: Add engine page logic

mburns at redhat.com mburns at redhat.com
Wed Feb 6 13:14:48 UTC 2013


Michael Burns has posted comments on this change.

Change subject: tui: Add engine page logic
......................................................................


Patch Set 6: (2 inline comments)

i haven't fully tested yet, but the 2 in line comments should be fixed.

....................................................
File scripts/tui/src/ovirt/node/setup/engine_page.py
Line 37: 
Line 38:     def model(self):
Line 39:         model = {
Line 40:             "vdsm_cfg.address": "",
Line 41:             "vdsm_cfg.port": "7634",
default to 443
Line 42:             "vdsm_cfg.connect_and_validate": False,
Line 43:             "vdsm_cfg.password": "",
Line 44:             "vdsm_cfg.password_confirmation": "",
Line 45:         }


Line 38:     def model(self):
Line 39:         model = {
Line 40:             "vdsm_cfg.address": "",
Line 41:             "vdsm_cfg.port": "7634",
Line 42:             "vdsm_cfg.connect_and_validate": False,
default to True
Line 43:             "vdsm_cfg.password": "",
Line 44:             "vdsm_cfg.password_confirmation": "",
Line 45:         }
Line 46:         return model


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I62f4b5f398a11137676af3b4a729ca2728224bb9
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>



More information about the node-patches mailing list