Hello Sanjay Can you please let me know what happens during an upgrade? Specifically, wanted to confirm that the default setting of the global alert.purge.delay is set to do-not-delete, which is the current behavior?
Also, is there an equivalent global/setting for events? Hari -----Original Message----- From: Sanjay Tripathi [mailto:nore...@reviews.apache.org] On Behalf Of Sanjay Tripathi Sent: Tuesday, March 5, 2013 11:07 AM To: Sateesh Chodapuneedi; Min Chen; Devdeep Singh Cc: cloudstack; Sanjay Tripathi Subject: Review Request: CLOUDSTACK-874 Ability to delete Events and Alerts ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9753/ ----------------------------------------------------------- Review request for cloudstack, Devdeep Singh, Sateesh Chodapuneedi, and Min Chen. Description ------- This feature will provide the functionality to delete or archive the Alerts/Events. Delete or archive alerts APIs are available only for root-admin, delete or archive events are available for regular users. Functional Spec:https://cwiki.apache.org/confluence/display/CLOUDSTACK/Feature+-+Ability+to+delete+Events+and+Alerts This addresses bugs Ability, Alerts, CLOUDSTACK-874, Events, and, delete, and to. Diffs ----- api/src/com/cloud/alert/Alert.java 050f97f api/src/com/cloud/event/Event.java 1a61636 api/src/com/cloud/server/ManagementService.java 1736da3 api/src/org/apache/cloudstack/api/ApiConstants.java 1b544fd api/src/org/apache/cloudstack/api/command/admin/resource/ArchiveAlertsCmd.java PRE-CREATION api/src/org/apache/cloudstack/api/command/admin/resource/DeleteAlertsCmd.java PRE-CREATION api/src/org/apache/cloudstack/api/command/user/event/ArchiveEventsCmd.java PRE-CREATION api/src/org/apache/cloudstack/api/command/user/event/DeleteEventsCmd.java PRE-CREATION client/tomcatconf/commands.properties.in 5b387bd core/src/com/cloud/alert/AlertVO.java f6089d6 core/src/com/cloud/event/EventVO.java ac46f24 core/src/com/cloud/event/dao/EventDao.java bfcb818 core/src/com/cloud/event/dao/EventDaoImpl.java 44fbb03 server/src/com/cloud/alert/dao/AlertDao.java eb1faa5 server/src/com/cloud/alert/dao/AlertDaoImpl.java 2f3be88 server/src/com/cloud/api/ApiDispatcher.java cf86afc server/src/com/cloud/api/query/QueryManagerImpl.java 746e8cb server/src/com/cloud/api/query/vo/EventJoinVO.java f29a942 server/src/com/cloud/configuration/Config.java 04ad640 server/src/com/cloud/server/ManagementServerImpl.java 0e53079 setup/db/db/schema-410to420.sql 6f68e12 Diff: https://reviews.apache.org/r/9753/diff/ Testing ------- Tested the new API archiveAlerts, deleteAlerts, archiveEvents, deleteEvents with all the possible combinations of parameters as input on my local CloudStack setup. Thanks, Sanjay Tripathi
