P3-Core-Dev-Team / P3-Q-A

This repo is for tracking features
Other
1 stars 0 forks source link

ADS-Audit - no option to disable the retention - Effectv/Dell #100

Open raniangamuthu opened 8 months ago

raniangamuthu commented 8 months ago

Describe the bug Max is 720 days for retention but no option to disable the retention if the user prefers not to apply retention for the audit log


To reproduce Steps to reproduce the behavior: System Management >> Setting >>Audit Retention Period

Screenshots Screenshot 2024-03-25 084458


Additional context Build no : ADS v Client : Effectv

kilam005 commented 8 months ago

Audit logs are no longer maintained after 2 years. Storing data for long days will encounter storage cost and the purpose of audit is to track for some fail over or for security reasons. This will have to be checked almost immediately once the incident happens.

upto 2 years is a long durations.

I have been asking the team to provision some thing like an audit archive application to store audit longs older than the define audit retention duration. There is a idea for this but its not converted into a requirement yet.

kilam005 commented 8 months ago

@maheswarareddyi please get this planned with saumya and team

kilam005 commented 8 months ago

@raniangamuthu please provide all expectations as a requirment hint list to get the team focus on the requirment drafting first.

maheswarareddyi commented 7 months ago

@raniangamuthu As discussed on April 12, 2024, Please consolidate the expectations and provide clarifications on the questions asked during the discussion to frame the requirements more clearly.

kilam005 commented 6 months ago

Requriment drafting is in progress and expected to move to next phase by eod of next week.

kilam005 commented 6 months ago

Requirment were reviewed by Harsha, Rajesh, Malik, Ashok, Susella, etc on 21 May. Feebacks and items to consider during planning were informed. Feedback provided are being in coropoated. @maheswarareddyi is planning to send the reqruiment doc and schedule a walkthrough by this week.

maheswarareddyi commented 6 months ago

The requirement document has been submitted for review and is expected to be signed off by June 04, 2024.

maheswarareddyi commented 5 months ago

A walkthrough of the requirement changes was conducted with the stakeholders. All concerns raised and comments provided were addressed and resolved. The requirement sign-off is expected by June 13, 2024.

maheswarareddyi commented 5 months ago

Requirements are signed off by the stakeholders and delivered to the development team.

@selvakumarchandran @prabhakaranUI Please find the signed-off requirements here

kilam005 commented 5 months ago

Planning review was completed and development has started.

kilam005 commented 4 months ago

Work is in progress. ETA expcted by 5th July. Actual status need to be confirmed.

@selvakumarchandran or @ShaliniSettu please update

ShaliniSettu commented 4 months ago

audit tracker application feature development is in-progress

kilam005 commented 4 months ago

@ShaliniSettu what is the ETA

ShaliniSettu commented 4 months ago

@kilam005 development completed, dev testing is going on. It is expected to deliver by tomorrow EOD

ShaliniSettu commented 4 months ago

this fix will be deliver along with audit application feature

Harik726 commented 4 months ago

Development has been completed. It will be released in the next release

kilam005 commented 2 months ago

Fix is under reveiw. @ashoksri0 need to approve to package it. @ashoksri0 has confirmed to start review today.

maheswarareddyi commented 2 months ago

PR review is in progress and we expected to get the review done by @ashoksri0 this week.

kilam005 commented 2 months ago

@ashoksri0 PR review pending for about 2 weeks now. Is it completed? please inform ETA to pass this for testing

kilam005 commented 2 months ago

PR has been merged and will be send for testing by EOW

kilam005 commented 2 months ago

@selvakumarchandran and @Udhayanila8 please check and updatethe current status asap

Udhayanila8 commented 1 month ago

@kilam005 As confirmed with @Harik726, the build has still not been delivered for testing.

Harik726 commented 1 month ago

@kilam005 and @Udhayanila8 Feature have been merged. Will be delivered in next release

kilam005 commented 1 month ago

Delivery has been posponted from the planned date. It is scheudled to be deployed today and handed over for Testing.

kilam005 commented 1 month ago

Testing started last week. Halted coz of application installation flow issue. Dev team has fixed and given correction and handed over for testing again today.

Udhayanila8 commented 1 month ago

http://192.168.56.146:4081/ version: 2024.10.45564_RC3

Basic scenarios are working fine, but still testing for Audit Application feature is InProgress. Once testing is done will give closure to this issue.

Screenshots: image image image image image image

kilam005 commented 1 month ago

@Udhayanila8 please start verfying on the new build shared today.

maheswarareddyi commented 1 month ago

Feature testing is in progress by @Udhayanila8 who identified some blockers, once the blockers are fixed and delivered this feature can be tested and certified by the testing team.

kilam005 commented 2 weeks ago

Previously identfiied blockers were relsoved and new build was shared. @Udhayanila8 to continue testing.

maheswarareddyi commented 1 week ago

Feature testing is in progress by @Udhayanila8.

Srividhya2506 commented 1 day ago

For the audit application, the test case execution was completed. 3 issues need to be retest. For 2 issues, comments have been added, and we are waiting for the response; another issue is in IR status. Once the 3 issues got retested, we will add verification tag and added issues list for reference. https://github.com/P3-Core-Dev-Team/ADS_BE/issues/3392 https://github.com/P3-Core-Dev-Team/ADS_BE/issues/3375 https://github.com/P3-Core-Dev-Team/ADS_BE/issues/3382