Moti Asayag has posted comments on this change.

Change subject: engine: Add audit log messages for vnic profile commands
......................................................................


Patch Set 25:

(2 comments)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/VnicProfileCommandBase.java
Line 51:     }
Line 52: 
Line 53:     private Network getNetwork() {
Line 54:         if (network == null) {
Line 55:             network = 
getNetworkDAO().get(getVnicProfile().getNetworkId());
but audit log mechanism will log the message only if we entered the execution 
part, meaning only if we passed the can-do-action.

therefore no NPE is expected here.
Line 56:         }
Line 57: 
Line 58:         return network;
Line 59:     }


....................................................
File 
backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
Line 482: INVALID_INTERFACE_FOR_MANAGEMENT_NETWORK_CONFIGURATION=Failed to 
configure management network on host ${VdsName}. Host ${VdsName} has an invalid 
interface ${InterfaceName} for the management network configuration.
Line 483: VLAN_ID_MISMATCH_FOR_MANAGEMENT_NETWORK_CONFIGURATION=Failed to 
configure management network on host ${VdsName}. Host ${VdsName} has an 
interface ${InterfaceName} for the management network configuration with 
VLAN-ID (${VlanId}), which is different from data-center definition 
(${MgmtVlanId}).
Line 484: SETUP_NETWORK_FAILED_FOR_MANAGEMENT_NETWORK_CONFIGURATION=Failed to 
configure management network on host ${VdsName} due to setup networks failure.
Line 485: PERSIST_NETWORK_FAILED_FOR_MANAGEMENT_NETWORK=Failed to activate host 
${VdsName} due to failure in persisting the management network configuration.
Line 486: ADD_VNIC_PROFILE=VM network interface profile ${VnicProfileName} was 
added to network ${NetworkName} in Data Center: ${DataCenterName}.
Done
Line 487: ADD_VNIC_PROFILE_FAILED=Failed to add VM network interface profile 
${VnicProfileName} to network ${NetworkName} in Data Center: ${DataCenterName}
Line 488: UPDATE_VNIC_PROFILE=VM network interface profile ${VnicProfileName} 
was updated for network ${NetworkName} in Data Center: ${DataCenterName}.
Line 489: UPDATE_VNIC_PROFILE_FAILED=Failed to update VM network interface 
profile ${VnicProfileName} for network ${NetworkName} in Data Center: 
${DataCenterName}.
Line 490: REMOVE_VNIC_PROFILE=VM network interface profile ${VnicProfileName} 
was removed from network ${NetworkName} in Data Center: ${DataCenterName}.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I3d41b5487faf7eafaeae0ed929fd53ad26e7416a
Gerrit-PatchSet: 25
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtri...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to