Federico Simoncelli has posted comments on this change.
Change subject: core: add the image block alignment scan
......................................................................
Patch Set 8: (7 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetDiskAlignmentCommand.java
Line 75: }
Line 76:
Line 77: private String getDiskIsUsedByGetAlignment() {
Line 78: return new
StringBuilder(VdcBllMessages.ACTION_TYPE_FAILED_DISK_IS_USED_BY_GET_ALIGNMENT.name())
Line 79: .append(String.format("DiskAlias %1$s",
getDiskAlias()))
Done
Line 80: .toString();
Line 81: }
Line 82:
Line 83: @Override
Line 235: }
Line 236: return vdsGroupId;
Line 237: }
Line 238:
Line 239: protected Guid getVdsIdInPool() {
Done
Line 240: if (vdsInPool == null && getVdsGroupId() != null) {
Line 241: List<VDS> vdsInPoolList =
getVdsDAO().getAllForVdsGroupWithStatus(getVdsGroupId(), VDSStatus.Up);
Line 242: if (!vdsInPoolList.isEmpty()) {
Line 243: vdsInPool = vdsInPoolList.get(0).getId();
Line 263: }
Line 264:
Line 265: public VM getVm() {
Line 266: if (diskVm == null && getDisk() != null) {
Line 267: for (VM vm :
getVmDAO().getVmsListForDisk(getDisk().getId())) {
If you mean that it should be checked for being plugged/unplugged. Unneeded.
Line 268: diskVm = vm;
Line 269: break;
Line 270: }
Line 271: }
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/BaseDisk.java
Line 203: }
Line 204:
Line 205: public void setLastAlignmentScan(Date value) {
Line 206: lastAlignmentScan = (value != null) ? (Date) value.clone() :
null;
Line 207: }
Not sure what you're proposing here, for other opinions just read the previous
comments.
Line 208:
Line 209: @Override
Line 210: public int hashCode() {
Line 211: final int prime = 31;
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/DiskAlignment.java
Line 2:
Line 3: import java.util.HashMap;
Line 4: import java.util.Map;
Line 5:
Line 6: public enum DiskAlignment {
Already discussed in patch set 4.
""" EnumMap is used to map enums to other values (integers, strings, etc.).
What I'm trying to do here is the opposite (see forValue), I want to quickly
get the enum from an integer (typically in the db). """
Line 7:
Line 8: Unknown(0),
Line 9: NotApplicable(1), // future use, e.g. disks with no partition table
Line 10: Misaligned(2),
....................................................
File backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
Line 806: ACTION_TYPE_FAILED_VM_SNAPSHOT_NOT_IN_PREVIEW=Cannot ${action}
${type} to a Snapshot that is not being previewed. Please select the correct
Snapshot to restore to: Either the one being previewed, or the one before the
preview.
Line 807: ACTION_TYPE_FAILED_SHAREABLE_DISK_NOT_SUPPORTED=Cannot ${action} a
shareable ${type} (${diskAliases}). This operation is not supported.
Line 808: ACTION_TYPE_FAILED_DISK_NOT_EXIST=Cannot ${action} ${type}. The
specified disk does not exist.
Line 809: ACTION_TYPE_FAILED_NO_DISKS_SPECIFIED=Cannot ${action} ${type}. No
disks have been specified.
Line 810: ACTION_TYPE_FAILED_DISK_IS_NOT_VM_DISK=Cannot ${action} ${type}. The
following disks are not attached to any VM: ${diskAliases}.
Done
Line 811: ACTION_TYPE_FAILED_DISK_IS_NOT_TEMPLATE_DISK=Cannot ${action}
${type}. The selected disk is not a template disk. Only template disks can be
copied.
Line 812: ACTION_TYPE_FAILED_SOURCE_AND_TARGET_SAME=Cannot ${action} ${type}.
The source and target storage domains are the same.
Line 813: ACTION_TYPE_FAILED_CANNOT_MOVE_TEMPLATE_DISK=Cannot ${action}
${type}. Template disks cannot be moved.
Line 814:
....................................................
File
backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties
Line 96: job.DisableGlusterHook=Disabling Gluster Hook ${GlusterHookName}
Line 97: job.UpdateGlusterHook=Updating Gluster Hook ${GlusterHookName} on
conflicting servers in Cluster ${Cluster}
Line 98: job.AddGlusterHook=Adding Gluster Hook ${GlusterHookName} on
conflicting servers in Cluster ${Cluster}
Line 99: job.RemoveGlusterHook=Removing Gluster Hook ${GlusterHookName} from
all servers in Cluster ${Cluster}
Line 100: job.GetDiskAlignment=Scanning Block Alignment on Disk ${DiskAlias}
The majority are upper case.
Line 101:
Line 102: # Step types
Line 103: step.VALIDATING=Validating
Line 104: step.EXECUTING=Executing
--
To view, visit http://gerrit.ovirt.org/11946
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4858b7bbfa453230fcafecfbc5358c715d5d825b
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Liron Ar <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
