Yair Zaslavsky has posted comments on this change.

Change subject: core: Add event for MAC pool initialization error
......................................................................


Patch Set 11: (3 inline comments)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MacPoolManager.java
Line 179:         }
Line 180:     }
Line 181: 
Line 182:     private void logInitializationError(String message) {
Line 183:         log.error("The Mac pool is not initialized");
Done
Line 184:         AuditLogableBase logable = new AuditLogableBase();
Line 185:         logable.AddCustomValue("Message",message);
Line 186:         AuditLogDirector.log(logable, 
AuditLogType.MAC_POOL_NOT_INITIALIZED);
Line 187:     }


Line 235:         log.info("MacPoolManager::freeMacs - entered");
Line 236:         mLocObj.writeLock().lock();
Line 237:         try {
Line 238:             if (!mInitialized) {
Line 239:                 logInitializationError("Failed to free macs.");
Done
Line 240:             }
Line 241:             for (String mac : macs) {
Line 242:                 internalFreeMac(mac);
Line 243:             }


....................................................
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
Line 450:     MAC_POOL_EMPTY(816),
Line 451:     MAC_ADDRESS_IS_IN_USE(833),
Line 452:     CERTIFICATE_FILE_NOT_FOUND(817),
Line 453:     RUN_VM_FAILED(818),
Line 454:     
MAC_POOL_NOT_INITIALIZED(837,AuditLogTimeInterval.HOUR.getValue()),
Done
Line 455:     // CBC
Line 456:     VDS_REGISTER_ERROR_UPDATING_HOST(819),
Line 457:     VDS_REGISTER_ERROR_UPDATING_HOST_ALL_TAKEN(820),
Line 458:     VDS_REGISTER_HOST_IS_ACTIVE(821),


--
To view, visit http://gerrit.ovirt.org/10197
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibaa3d9eb23016b4866e5cfa39f919378e09fdf2d
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Moti Asayag <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to