----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5739/ -----------------------------------------------------------
(Updated July 10, 2012, 12:25 p.m.) Review request for cloudstack, Abhinandan Prateek, Sheng Yang, and Alena Prokharchyk. Changes ------- During account cleanup, associated networks get cleaned up. As part of network cleanup, source nat ip was not getting released (this logic got introduced as part of a fix to persist source nat ip during network restart). Added a fix to release the source nat ip when the network gets destroyed. Description ------- During account cleanup, associated network is accessed without checking whether it is present or not. Added a check to use it only when present. This addresses bug CS-15345. Diffs (updated) ----- server/src/com/cloud/network/NetworkManagerImpl.java 1e677a0 Diff: https://reviews.apache.org/r/5739/diff/ Testing ------- Yes Thanks, Koushik Das