core: AddVmCommand - add all SDs to permList

Adding all target storage domains specified in the
parameters to permList (instead of getStorageDomainId()
which returns an empty guid since MSD).

Change-Id: I7b942af8b5093848922f060c6be44e8a69374634
Bug-Url: https://bugzilla.redhat.com/928399
Signed-off-by: Daniel Erez <derez@redhat.com>
core: Java7 Engine build

Since Java 6 is reaching its EOL, it is clear that we should start
phasing it out, and move to Java 7 where possible.

Note that the current GWT version we're using does NOT support Java 7,
so the frontend code and the code it depends on remained in Java 6.

This patch introduces the following:
1. Upgrading the maven-compiler-plugin setting to 1.7.
2. Upgrading animal-sniffer to 1.9, and using the official artifact from
   codehaus, in order to support Java 7 JDK validations.
3. Added a whitelist of com.sun classes used by existing code that
   would be picked up by the newly updated version of animal-sniffer,
   which is slightly stricter than the previous one.
   Note that these are unsupported internal classes of the JDK which
   should probably not be used externally anyway, and it would be a
   pretty good idea to remove these usages in future patches.

Change-Id: I9ea655ab589b9907184b033ea08cc98d7ee57e63
Signed-off-by: Allon Mureinik <amureini@redhat.com>
Signed-off-by: Alissa Bonas <abonas@redhat.com>
engine : ovirt-node enter installing status when the install fails

When ovirt-node installation fails because of missing iso file
the status of the host is left as Installing.

This patch adds VdsDynamic data to compensation context if can do action
of InstalllVdsCommand fails. The status is reverted back to the previous
status in compensation logic. The error messages are added to the
return values CanDoActionMessages so the message can be displayed
in cli

Change-Id: I2d74e946cb68d2b9dbc11fc5c6bed9f63ba35fe2
Bug-Url: https://bugzilla.redhat.com/947982
Signed-off-by: Ravi Nori <rnori@redhat.com>
frontend: cleanup: remove unused Base64 class

Change-Id: Ie3dfa0901b3fd74e8bf35238f5734b1ce5bb61b1
Signed-off-by: Alon Bar-Lev <alonbl@redhat.com>
webadmin,userportal: error in custom properties sheet

Current custom properties sheet doesn't handle cases
where there are no properties stored in DB.

Extracting the new duplicate code (splitting logic) into AsyncDataProvider

Change-Id: Ia939952c367b6416ed90bb295f1485ce4ac63368
Bug-Url: https://bugzilla.redhat.com/924201
Signed-off-by: Gilad Chaplik <gchaplik@redhat.com>
webadmin: Stay in the same tab if possible

Suggested behaviour for the system tree:
 - Dont change the selected main tab if the new tree item can show that tab

Example case:
Hosts tab is selected -> select spesific storage in system tree
old behaviour: switch to clusters tab
new behaviour: stay in hosts tab (because it is possible)

Change-Id: I2c27536ecb5ef9a8f48205c217fbdf295860e363
Signed-off-by: noam slomianko <nslomian@redhat.com>
core: fix overriding placeholder val in audit log

Fix overriding placeholder value set in AugitLogableBase.customValues
with empty string in TypeCompat.
In cases when the value of a placeholder was already populated with
custom value,
no need to invoke a method to try populate it again since in some cases
an empty string returns causing the override of already set non empty value.

Example where bug is shown - storagePoolName is displayed as unknown.
"Host cannot access one of the Storage Domains attached to the Data
Center <UNKNOWN>. Setting Host state to Non-Operational."

Change-Id: I4239085d0bd0246369e93e38a394ab8306c05887
Signed-off-by: Alissa Bonas <abonas@redhat.com>
core: remove value from LockIdNameAttribute declaration

Remove specifying isReleaseAtEndOfExecute=true in @LockIdNameAttribute
annotation declaration since it is the default value anyway.

Change-Id: I2092f0c6c9a616b2133ce99cb557ba31a9cc1755
Signed-off-by: Alissa Bonas <abonas@redhat.com>
core:delete_async_tasks_and_compensation_data...

delete_async_tasks_and_compensation_data.sql exits on KeyError

Adding a CASCADE to the TRUNCATE commands to force tables cleanup.

Change-Id: I097bf1f388f082f768cd50c33d11ea1e9f081004
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=949229
Signed-off-by: Eli Mesika <emesika@redhat.com>
userportal,webadmin: Builders infrastructure

An implementation of a flexible, easy to
use, reuse an extend functionality of the builders between
backend and frontend models.

Change-Id: If81d4c71f9feebd6ececb9a2be46b353383046de
Signed-off-by: Tomas Jelinek <tjelinek@redhat.com>
webadmin: Display correct error when adding new storage

When adding a new Storage Domain (of NFS/Local type) the error displayed
when the connection was failed was always "Error creating a storage domain"
instead of the actual error from VDSM (no permission error etc.), the connect
command was changed to propegated the VDSM error to the client there it will
be tranlsated and be displayed as the translated user friendly value

Change-Id: Ib8f721d3f10596253ca194a36139ba9ebcdff246
Bug-Url: https://bugzilla.redhat.com/843440
Signed-off-by: Tal Nisan <tnisan@redhat.com>
packaging: proxy: add missing /console.vv

Change-Id: Ibfe591cf1087c05c9d6036456b3bb32193cca4e0
Signed-off-by: Alon Bar-Lev <alonbl@redhat.com>