I've submitted a patch to bypass the failure created by that commit.

http://gerrit.ovirt.org/#/c/7865/1

Yaniv, please submit a proper solution for it if meant differently.


On 09/09/2012 12:53 PM, Eyal Edri wrote:
> this commit break ovirt-engine on jenkins.ovirt.org:
> 
> http://jenkins.ovirt.org/job/ovirt_engine_animal_sniffer_check/425/org.ovirt.engine.core$vdsbroker/changes#detail0
> 
> core: add vdsName to vdsBroker log info (#853883) (details)
> Commit 35c2e7ceb5109c9349e3699c3f53414cbe0b4c4e by ybronhei
> core: add vdsName to vdsBroker log info (#853883)
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=853883
> 
> Logger gets VdsCommandBase.toString() method's output,
> this patch adds call to getAdditionalInformation from
> toString to add to log string VdsName.
> 
> Change-Id: Ieb0589bb5f5e157c04f85e46f6769d66ef98b2d1
> Signed-off-by: Yaniv Bronhaim <ybron...@redhat.com>
> 
> 
> please check,
> 
> Eyal.
> 
> 
> 
> 
> 
> 
> 
> 
> 
> 
> [INFO] 
> [INFO] --- maven-compiler-plugin:2.3.2:compile (default-compile) @ vdsbroker 
> ---
> [INFO] -------------------------------------------------------------
> [ERROR] COMPILATION ERROR : 
> [INFO] -------------------------------------------------------------
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/RemoveVdsVDSCommand.java:[5,7]
>  error: RemoveVdsVDSCommand is not abstract and does not override abstract 
> method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/UpdateVmDynamicDataVDSCommand.java:[5,7]
>  error: UpdateVmDynamicDataVDSCommand is not abstract and does not override 
> abstract method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/FailedToRunVmVDSCommand.java:[5,7]
>  error: FailedToRunVmVDSCommand is not abstract and does not override 
> abstract method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ActivateVdsVDSCommand.java:[5,7]
>  error: ActivateVdsVDSCommand is not abstract and does not override abstract 
> method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java:[28,7]
>  error: CreateVmVDSCommand is not abstract and does not override abstract 
> method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/UpdateVdsVMsClearedVDSCommand.java:[5,7]
>  error: UpdateVdsVMsClearedVDSCommand is not abstract and does not override 
> abstract method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/AddVdsVDSCommand.java:[12,7]
>  error: AddVdsVDSCommand is not abstract and does not override abstract 
> method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/SetVdsStatusVDSCommand.java:[13,7]
>  error: SetVdsStatusVDSCommand is not abstract and does not override abstract 
> method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResumeVDSCommand.java:[10,7]
>  error: ResumeVDSCommand is not abstract and does not override abstract 
> method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/DestroyVmVDSCommand.java:[18,7]
>  error: DestroyVmVDSCommand is not abstract and does not override abstract 
> method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/MigrateVDSCommand.java:[21,7]
>  error: MigrateVDSCommand is not abstract and does not override abstract 
> method getAdditionalInformation() in VDSCommandBase
> [ERROR] 
> /home/jenkins/workspace/ovirt_engine_animal_sniffer_check/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/UpdateVdsDynamicDataVDSCommand.java:[5,7]
>  error: UpdateVdsDynamicDataVDSCommand is not abstract and does not override 
> abstract method getAdditionalInformation() in VDSCommandBase
> [INFO] 12 errors 
> [INFO] -------------------------------------------------------------
> mojoFailed 
> org.apache.maven.plugins:maven-compiler-plugin:2.3.2(default-compile)
> projectFailed org.ovirt.engine.core:vdsbroker:3.1.0
> _______________________________________________
> Engine-devel mailing list
> Engine-devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
> 

_______________________________________________
Engine-devel mailing list
Engine-devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel

Reply via email to