Bug 995836
Summary: | [command] remove the use of @LockIdNameAttribute | |||
---|---|---|---|---|
Product: | Red Hat Enterprise Virtualization Manager | Reporter: | Alon Bar-Lev <alonbl> | |
Component: | ovirt-engine | Assignee: | Ravi Nori <rnori> | |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Martin Mucha <mmucha> | |
Severity: | medium | Docs Contact: | ||
Priority: | unspecified | |||
Version: | unspecified | CC: | bazulay, gklein, iheim, lpeer, oourfali, rbalakri, rgolan, Rhev-m-bugs, yeylon, yzaslavs | |
Target Milestone: | --- | Keywords: | CodeChange | |
Target Release: | 3.5.0 | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | infra | |||
Fixed In Version: | vt1.3 | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 997376 (view as bug list) | Environment: | ||
Last Closed: | 2015-02-17 17:15:06 UTC | Type: | Bug | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | Infra | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 997376, 1142923, 1156165 |
Description
Alon Bar-Lev
2013-08-11 12:08:15 UTC
I would like to add that annotations in java are not inherited . Since we use inherience in the commands, this is a problem. The problem exists in other commands - for example in AddDiskCommand. Bare in mind that although the keywords, we have issues with current behavior of several commands. This is why I acked this to 3.3. grep "getExclusiveLocks" and extract what is missing @LockIdNameAttribute bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommandBase.java bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmTemplateCommand.java bll/src/main/java/org/ovirt/engine/core/bll/ExportRepoImageCommand.java just to make it clear, the storage commands are calling explicitly aquireLockInternal in the canDoAction. since there is no contract (no interface) we are open to mistakes and misuse. this should be rectified. In reply to comment #3 - Calling aquireLockInternal in CDA can still be risky, IMHO. If we decide to postpone this from 3.3 - we should fix the above commands. ExportRepoImageCommand has the annotation It's also in all subclasses of RunVmCommandBase *** Bug 997376 has been marked as a duplicate of this bug. *** This is a bit late to get into 3.4 (2 weeks from RC) and this may be a bit riskey Hence moving to 3.5 rhev 3.5.0 was released. closing. |