cim: disable cim transaction run without a password

Previously when users selected enable cim without a password
it will run cim transaction and enable cim service.
Now if the user selected enable cim without a password, it
will prompt ConfirmationDialog to show "Unable to configure CIM
without a password!" info.

Change-Id: I292ab6e9d00368e65b7e194e99cca8fc3b63ab72
Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1027110
Signed-off-by: hadong <hadong0720@gmail.com>
installer: make sure data size could be setup "0"

Previously when users want setup data_size as "0" to disable
data partition, it will create data partition with remaining
free space.It wasn't what we expect results.
Now we could setup data_size as "0" to disable
data partition.

Change-Id: I3445acb336116755e5c22302586171b2b44f1acc
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1026138
Signed-off-by: hadong <hadong0720@gmail.com>
Don't show output from ipmitool check status

Previously, output from the IPMI plugin was spuriously shown,
which gave the appearance of hanging if enter was pressed. Rather
than dumping it to /dev/null, just catch it instead.

Change-Id: I776fc9b56ed797e80f48cafb770e976258786884
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1026646
Signed-off-by: Ryan Barry <rbarry@redhat.com>
edit-node: Do not change iso file name when using with "-n" or "--name"

    Using with "-n" or "--name", the generated iso file name should just match the name,
    skipping to append plugins or packages name to it.

Change-Id: I41b9a3264961c929cce820a94adf258180401e49
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1030810
Signed-off-by: Ouyang guohua <ouyanggh0815@gmail.com>
installer: Improve storage loading time

Previously a time consuming function was called several times. This is
now still the case, but we cache the not-expected-to-change value.

Change-Id: Iee46ffac3586d1e5f998aabddf03f4afde19f4c9
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1041213
Signed-off-by: Fabian Deutsch <fabiand@redhat.com>
Clear the storage volume page when free space is positive

It was possible to get into a state where the storage volume page
was still flagged as invalid, even when all sizes were fine. Now,
we should retrigger validation of every invalid field when free
space is not negative, so it behaves as expected.

Change-Id: Iad090893533a2152f8419e234c287f2024594f1a
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1022475
Signed-off-by: Ryan Barry <rbarry@redhat.com>
Fix plugin info shown incorrectly issue when install multiple packages

Change-Id: I8d74b305411a2aa9128e4b589e41e69965bc1e7b
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1040829
Signed-off-by: hadong <hadong0720@gmail.com>
edit-node:rename iso_name more simpler when install multiple packages

Change-Id: I2ef7a53034a7aa424f207c94a8be96fac56c7a64
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1040827
Signed-off-by: hadong <hadong0720@gmail.com>
snmp: disable snmp transaction run without a password

Previously when users selected enable snmp without a password
it will run snmp transaction and enable snmp service.
Now if the user selected enable snmp without a password, it
will prompt ConfirmationDialog to show "Unable to configure SNMP
without a password!" info.

Change-Id: I8e471eaccfae1dd438bdafc380e341f4ebf6b6ff
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1026222
Signed-off-by: hadong <hadong0720@gmail.com>
Fix only calculate one disk driver size issue

Previously when selecting two disks as HOSTVG, storage_vol_page only
calculate one disk driver size(installation.device.current),
Now fix it as calculating all disk drivers size that was selected by users as
HOSTVG(installation.devices).

Change-Id: I387c6832be086ac6895c94282d1ed711d02500ec
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1022380
Signed-off-by: hadong <hadong0720@gmail.com>
installer: Fix data size again

Met miss create "Data" logical volume when using the default value
of "Data" in storage_vol_page issue again.
Now fixed it.

Change-Id: Ie746feffc5eb98e229ec954d2df7644e5be9107e
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1022375
Signed-off-by: hadong <hadong@redhat.com>
Let applications specify their own quit function

Keep the old behavior, and default to it if nothing is passed, but
allow Application() objects (setup, installer, et al) to decide
what happens when a user quits.

Change-Id: I2bffc0e5ea625457bae1eb1106976de7a7053dbb
Signed-off-by: Ryan Barry <rbarry@redhat.com>
Puppet autoinstall should actually write puppet.conf

Previously, we forgot to .enable_puppet() after writing the keys.
Do so.

Change-Id: Ie9508f4131c5223339c439fafdcc0fbdc9938fa3
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=987785
Signed-off-by: Ryan Barry <rbarry@redhat.com>
write splash.xpm into grub.conf on the right location for iSCSI installs

Change-Id: Id0f4d317bdf78bdb2a43806a91de4f8f3c9fc774
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=924527
Signed-off-by: hadong <hadong@redhat.com>
Show a confirmation dialog when exiting the TUI

No confirmation was shown when pressing escape. We should show a
dialog to make sure this is really what the user wants to do.

Change-Id: I2d56b0e816cc9c7abfb25dc346436963eae6a5aa
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1039266
Signed-off-by: Ryan Barry <rbarry@redhat.com>
Show a confirmation in the installer if "Quit" is selected

We should confirm that the user wants to quit, and reboot the
system rather than simply exiting if so.

Change-Id: I678a3eaf3d443b88ceb4d5a6a3632842f614dff6
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=980864
Signed-off-by: Ryan Barry <rbarry@redhat.com>
Flag the appropriate kdump field if empty

It was possible to attempt to configure kdump with an empty nfs or
ssh entry field after the appropriate checkboxes were ticked. Now,
we should dynamically determine which validator to use, remove the
ability to leave it empty, and present a warning.

Change-Id: Id3331d37771304d642515c9cb24b7a4ac9fd2f4c
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1017083
Signed-off-by: Ryan Barry <rbarry@redhat.com>
tuned: Capture service output

Change-Id: I00921313d3faa017ef25117d1d703cea8c17075d
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=988696
Signed-off-by: Fabian Deutsch <fabiand@redhat.com>