restapi: provide full_version in product_info

Frontend uses ProductRPMVersion from vdc_options to show Engine version.
This patch will include a new element called full_version into ProductInfo xsd schema,
which will provide to users the same version info from Frontend.

Change-Id: I9102b3121154e1ba2096ec8b54765c6bfc6a97f1
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=905398
Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
engine: ApplicationException - removed

We are not C# project, removing C# inheritance

Change-Id: I681f44a5772e0ed2c220546fd5578a6993e862c1
Signed-off-by: Michael Kublin <mkublin@redhat.com>
engine: Exception handling - VDSCommandBase

The following patch will simplify exception handling in case of failure in xml rpc call.
All XmlRpcRunTimeException will be translated to network exception with error code VdcBllErrors.VDS_NETWORK_ERROR.
The XmlRpcRunTimeException - means that we failed in call , failed to parse xml and we did not get appropriate response
that we can parse, this is by definition VdcBllErrors.VDS_NETWORK_ERROR.
Now, VdcBllErrors.VDS_PROTOCOL error was removed as unused.

Change-Id: I0bd38cd75706ff7669c1ff055b951aeb864b2736
Signed-off-by: Michael Kublin <mkublin@redhat.com>
engine: TimeoutBase does not have to be serialized

TimeoutBase never needs to be serialized, so the Serializable interface
is removed.

Change-Id: Idb07e9cfaad74820013ca749676cc978aa8714eb
Signed-off-by: Laszlo Hornyak <lhornyak@redhat.com>
engine: remve command serialVersionUid fields

- removed serialVersionUID fields from command implementations
- removed @SuppressWarning("serial") annotations where no longer needed

Change-Id: I1b0b015935132ac0a4f12e696814fc953d62b9ff
Signed-off-by: Laszlo Hornyak <lhornyak@redhat.com>
engine: Change default 64-bit memory limit on upgrade

Changes of memory limit for 64-bit VMs by
14dcb41a97be002833d361adfdf7ac199743d671 are now reflected on upgrade, not only
on clean install.

Change-Id: I8dbe4747ce6e8b17c4bfff96cb8a81599632b7da
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=954021
Signed-off-by: Martin Betak <mbetak@redhat.com>
core: rhevm-upgrade creates log/helping files...

rhevm-upgrade creates log/helping files in /usr/share/ovirt-engine/dbscripts

1) Adding -g parameter to create_db.sh create_schema.sh and upgrade.sh
to disable MD5 files generation (default NOMD5=false)
2) Adding -m MD5DIR parameter to  create_db.sh create_schema.sh and
upgrade.sh to set the MD% directory (default =MD5DIR=$(pwd))
3) Removing auto-generation of .schema files and creating a
exportDbSchema.sh that generates this file since it is used only for
development

Change-Id: I89b970fdd93102d6354e6cb634af2bd74c0a5dc9
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=947344
Signed-off-by: Eli Mesika <emesika@redhat.com>
findbugs: Non-Boolean argument formatted as boolean

Change-Id: I82496f3ec4994a60df0706d0cf94b93b039ed372
Signed-off-by: Shahar Havivi <shavivi@redhat.com>
core: Remvoing useless AtomicInteger from EntityAsyncTask

Removing a useless Atomic integer that is not in use
anymore, besides the decrease and incrase of the value

Change-Id: Ic7d52dc4df7e39e7059e64eb0241861f88d45fef
Signed-off-by: Yair Zaslavsky <yzaslavs@redhat.com>
gluster: Add host: Improved gluster validation [2]

Currently we don't allow to add a host if it has peers. However, if one
(or more) of those peers are already part of the cluster in engine DB,
then it means that these have been added from gluster CLI and hence the
validation should not fail in such cases.

Change-Id: I36b3b6d00a71af50ef1b006e95daebfbfe3f67f5
Signed-off-by: Shireesh Anjal <sanjal@redhat.com>
tools: Added colon and space after new password prompt in engine-config

Added colon and space after new password prompt using:

  engine-config -s AdminPassword=interactive

Bug-Url: https://bugzilla.redhat.com/924603
Change-Id: I7abc9cba42e2407874c5022f81f40330fa7eba53
Signed-off-by: Martin Perina <mperina@redhat.com>
restapi: Throw exception if setting lower VM pool size

Report, if the user tries to lower the number
of VMs in the VM pool.

Change-Id: Ibe4ad50e6962159df5891be4653bc05eb3dfd237
Bug-Url: https://bugzilla.redhat.com/950049
Signed-off-by: Libor Spevak <lspevak@redhat.com>
webadmin: Making VM related substabs resizable

VM, template and host related subtabs

Change-Id: Ibb8a7edcf437650a4882d767f29ce5041f3b3be6
Signed-off-by: Tomas Jelinek <tjelinek@redhat.com>
restapi:Correct rsdl for add bricks

Add bricks command causes an exception in cli
when passed a collection of bricks.
Changed the rsdl to brick--COLLECTION
as per Michael's suggestion

Change-Id: I2a229faa90c8fcded33b53ab8801c4cb03094798
Bug-Url: https://bugzilla.redhat.com/923169
Signed-off-by: Sahina Bose <sabose@redhat.com>
findbugs: Redundant nullcheck

Change-Id: Id991b677df2d447b8124de5ae8a1f1a52aca98e5
Signed-off-by: Shahar Havivi <shavivi@redhat.com>
webadmin: Clean up Common.gwt.xml

Cleaned up Common.gwt.xml from numerous entries in order to make the
file more maintainable and speed up the build a tad.

The following entries were removed:
1. Classes that no longer exist in the code base (e.g.,
   time_lease_vm_pool_map).
2. Classes that are not used by the frontend (e.g.,
   LUN_storage_server_connection_map).
3. Double entries (e.g., UsbPolicy).
4. Specific entries that can be inferred from more general entries
   (e.g., common/action/VdcActionType.java does not need to be specified
   since common/action/* already is).

Change-Id: Idebce672b96192f7b114fc1596ba462fe097d9b8
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core, webadmin: Remove TabType

The TabType enum is not used anywhere in the codebase, and thus should
be removed.

This patch removes it and any localization references to it.

Change-Id: I2e17e4e533439d9038654213d2f5375c16b24f0a
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: Removed ImportCandidateTypeEnum

The aforementioned enum is not used anywhere in the codebase, and should
thus be removed.

Change-Id: Ic547d2356e310324fa53b49122786bfe3f769ab4
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: Remove StorageDomainOwnerType

The StorageDomainOwnerType enum contains only one value ("Unknown"), and
as such, is pretty useless and can be removed.

This patch removes it and all its references.

Change-Id: Iae13123bc33c2fdff2d7dd5e128c1a5cee7d262f
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: Remove StoragePoolIsoMapDAO.update impl

Since the owner column was removed from StoragePoolIsoMap in the
previous patch, it's only (database) mutable column is the status
column.
In other words, there is no difference between the update and
updateStatus methods.

This patch removes the Updatestorage_pool_iso_map stored procedure
and replaces the implementation of the update method with a
delegation to updateStatus.

Change-Id: Ia661353ed7b100795d90c9390866c955a1a6290d
Signed-off-by: Allon Mureinik <amureini@redhat.com>