ovirt-node-upgrade : remove /data/tmpiso_* during cleanup

Previously the function "_cleanup()" for ovirt-node-upgrade tools
didn't remove /data/tmpiso_* during cleanup.
Now it works.

Change-Id: I34672e4ff30d0ad37b7989780950d537155560ab
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=971299
Signed-off-by: hadong <hadong0720@gmail.com>
security_page: Disable "Bytes Used" when selecting
"Disable AES-NI"

Previously when selecting "Disable AES-NI", the item "Bytes Used"
still can be edited.
Now disable "Bytes Used" when selecting "Disable AES-NI"

Change-Id: I33133fff8c33e78cd13a7f409cc48a04f46ac501
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=972319
Signed-off-by: hadong <hadong0720@gmail.com>
network: Remove node_nics() doctest

Because it is relying on runtime informations.

Change-Id: I609f64ba3ca640d20d08aabe26005b365b8191fb
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
cim_model.py: Miss titles for class "ConfigureCIM"
and "SetCIMPassword"

Set titles for transactions in the CIM plugin
rhbz#971709

Change-Id: I9660f02ef76f6a44c8c8868a2e5b1abb6f42aaeb
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=971709
Signed-off-by: hadong <hadong0720@gmail.com>
Revert "network: Remove node_nics() doctest"

This reverts commit dc0030313804e845408069f118165a521635f8e9

Change-Id: If79faac65d99612b55d5ccf09344ff6a4c0fee0f
security: Migrate password_check

The password_check function was migrated to the security module and is
now also used by ovirtfunctions.

Change-Id: Id111da3a61b74213eb8c254de3d8e4d7805bdcc2
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
add in legacy dracut modules for el6 variants

Change-Id: I2f496d9cdc9ccce121e6fafca1c92e52b6740603
Signed-off-by: Joey Boggs <jboggs@redhat.com>
Signed-off-by: Mike Burns <mburns@redhat.com>
edit-node: Catch the OSError when Set "SELinux" disabled

Previously when the state of SELinux is disabled on the system,
edit-node will show an OSError exception.
Now edit-node catch this exception

Change-Id: Ibf65d5d8bb53152f5465baedf9db7584e4ecaefa
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=974934
Signed-off-by: hadong <hadong0720@gmail.com>
progress_page: shown wrong title for "SetPassword" transaction

Previously "SetPassword" transaction shown "Setting Root Password"
title, but actually it should display "Setting Admin Password".
Now it shown correctly.

Change-Id: I9b54505e9fc6d8bb4aba7b55167d4fbc0eb280c2
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=974981
Signed-off-by: hadong <hadong0720@gmail.com>
Work around corrupt GPT tables in the diagnostic page

Echo "ok" to parted to quash an error message from parted

Change-Id: Ic04103d395b46564479e46b834f723b0c8976fef
Signed-off-by: Ryan Barry <rbarry@redhat.com>
check for volume groups with storage_domain tag

before removing them forcefully, make sure we're not
overwriting important data

Bug-Url: https://bugzilla.redhat.com/889071
Signed-off-by: Joey Boggs <jboggs@redhat.com>
Change-Id: Id2af60c0a2e5dda7a0075e081c3b8d5a6c6c975b
fix findfs call in legacy grub setup

Change-Id: Ic3c331bc99b5c999dec79ecaece362510bd3ace3
Signed-off-by: Joey Boggs <jboggs@redhat.com>
edit-node: use hard link for kernel/initrd in EFI/BOOT dir

rhbz#957437

Signed-off-by: Joey Boggs <jboggs@redhat.com>
Change-Id: Ib341ebd9049c0d57633caf4e77ce7f7bb320d3ff
Update reboot task to run async from installer and install module

rhbz#955543

Signed-off-by: Joey Boggs <jboggs@redhat.com>
Change-Id: Ic1b4e048ff56b8c9ebd49a82b3d5546c7c326df8
process: Allow override of common args

Previously the common args to Popen couldn't be overriden, this is now
possible.

Change-Id: I3d0aeffe04bea7892f548662372d10a87a87cd11
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Mike Burns <mburns@redhat.com>
utils: Add lockfile to transactions

Previously multiple transaction could be run at once. Now a lock is
acuired before each transaction run to enforce only one transaction urn
at a time.

Change-Id: I0f3bfd60f2cda41ffcf13ff6dfc224d6d2d4a3ab
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=971673
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Mike Burns <mburns@redhat.com>
fs: Introduce File and FakeFs

Two classes are added to allow unittesting without access to the
filesystem.
Previously unit tests were nearly impossible because of the many
informations which are retrieved from the filesystem (at runtime).
by wrapping the file i/o in a class a mocking framework (e.g.
python-mock) can be used to patch this file i/o class during unit tests
and redirect file i/o requests to a fake filesystem (e.g. an
in-memory fs).
FakeFs is a simple class implementing such a fake in-memory filesystem
which can be used during unit tests.

Change-Id: I7889240a956b359f2375955766f3cafcfbeaf0ee
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Mike Burns <mburns@redhat.com>
network: Rework runtime network model

Previously there were strong assumptions about the runtime network
setup, thse assumptions are now more relaxed.
The code is now able to parse more complex setups.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=971410
Change-Id: I8053ea21af5029190bc16b89ee109d2d7937a05f
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Mike Burns <mburns@redhat.com>
network: Rewrite network config backend

The network backend doing the actual OS level configuration work was
rewritten to allow different topologies, the migration to NM also had
an impact on the design decisions.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=971410
Change-Id: I653023c9f856f1be51e45b86457ce24a3878015b
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Mike Burns <mburns@redhat.com>
Use fs.File in more places

Change-Id: I33e057c4caefaba5fa3cc97bd26773a2c8ba09f8
Signed-off-by: Fabian Deutsch <fabiand@fedoraproject.org>
Signed-off-by: Mike Burns <mburns@redhat.com>