cityofaustin / atd-data-tech

Austin Transportation Data & Technology Services
17 stars 2 forks source link

BUG: Update Batch Permit Expiration Notification to Include PSM and subject line update (LP) (On PROD ) #11523

Closed Nadin-Nader closed 1 month ago

Nadin-Nader commented 1 year ago

The batch notification job of upcoming permit expiration needs to include PSM folders. Also the subject line should be updated from "RW" to "ROW".

Stakeholders: Paloma A.

Sreelatha-BlazeKannan commented 1 year ago

@Nadin-Nader , @TracyLinder - Analysis is in Progress

Nadin-Nader commented 1 year ago

@Sreelatha-BlazeKannan Hi ok so I don’t have access to view these on PROD. I asked George and this is what he found:

[2:37 PM] Shirai, George Batch job 50 is "Send Permit Expiry Email", and it's NOT active in PROD right now Batch job 72 is "Send Permit Expiry Email" also... this one IS active in PROD This one last ran yesterday around 7pm. No errors recently Batch job 143 is "RW Expired Permit", this is active in PROD Last ran this morning around 12:30 with no recent errors

Sreelatha-BlazeKannan commented 1 year ago

@Nadin-Nader @TracyLinder - Analysis Findings

  1. Batch Job ’34 – Permit Lapsed and about to expire/PLATE‘ – Report weekly runs on Sunday (report query pulls only RW foldertype) and Job is Active in PROD
  2. Batch Job ‘50 – Send Permit Expiry email’ - Before Procedure - “PermitExpiry_Notification” and in Procedure its calling COA_BATCH_PROCESS.f_permit_expiry_email package function and Job is not Active in PROD
  3. Batch Job ‘72 – Send Permit Expiry email’ - Before Procedure - “PermitExpiry_Notification” and in Procedure its calling COA_BATCH_PROCESS.f_permit_expiry_email package function and Job is Active in PROD (also pulls both RW & EX)
  4. Batch Job ‘143 – RW Expired Permit’ – After Procedure – “RW Expired Permit” and procedure has both RW & EX foldertype and Job is Active in PROD
Sreelatha-BlazeKannan commented 1 year ago

@Nadin-Nader , @TracyLinder - As per Paloma, this email getting generated for both RW and EX as well. So this card can be closed.

Nadin-Nader commented 1 year ago

This is working and providing EX and RW. CLOSING

TracyLinder commented 1 year ago

@Sreelatha-BlazeKannan can you check to see if this job is running in Dev or Test? if yes, please disable it. Batch is either 72 or 143.

Paloma: Expiring RW permit notification report

The rightofway inbox received an automated AMANDA email from Dev/Test. We don’t need to continue to receive these.

TracyLinder commented 1 year ago

Q: Is ROW still getting this report emailed from dev or test? A: Last email was 5/29 & forwarded to Sree. Paloma will have to ask Candice if this has happened since 5/29.

Sreelatha-BlazeKannan commented 1 year ago

@Nadin-Nader , @TracyLinder - Job #266 from DEV was active, which is turned off now. Job#143 from TEST was not active; however these two jobs are generating the email attached using procedure DEFAULTREPORT_RW_EXPIREDPERMIT. Let's observe for next time if we receive the same email, then need to do further research on this.

TracyLinder commented 1 year ago

Sree asked during today's Friday meeting if Paloma or her team is still getting this email. P-lo isn't in the meeting today, we'll need to check again next Friday.

TracyLinder commented 1 year ago

Update

TracyLinder commented 1 year ago

@Sreelatha-BlazeKannan please troubleshoot when you return. Thank you.

Nadin-Nader commented 1 year ago

@Sreelatha-BlazeKannan when you get a chance can you give me an update on this one

Sreelatha-BlazeKannan commented 1 year ago

@Nadin-Nader , @TracyLinder - Did some more research and again set Active flag No for DEV job id's 39 & 25. Let us observe for our next DEV/TEST Email.

TracyLinder commented 1 year ago

The ROW email account received this email today. This is not fixed. FYI @Sreelatha-BlazeKannan

image

KyleErich commented 11 months ago

Retitled this card to account for the current AC. We need the EX and PSM folders to send off the notification email, and to do this at the same interval as for the RW folder (10 business days out or any day sooner than that).

I tested this again yesterday on a RW folder an a EX folder, and received a notification email for the expiring RW, but not for the expiring EX.

TracyLinder commented 8 months ago

Batch Job Analysis Notes

Analysis Findings

  1. Batch Job ’34 – Permit Lapsed and about to expire/PLATE‘ – Report weekly runs on Sunday (report query pulls only RW foldertype) and Job is Active in PROD
  2. Batch Job ‘50 – Send Permit Expiry email’ - Before Procedure - “PermitExpiry_Notification” and in Procedure its calling COA_BATCH_PROCESS.f_permit_expiry_email package function and Job is not Active in PROD
  3. Batch Job ‘72 – Send Permit Expiry email’ - Before Procedure - “PermitExpiry_Notification” and in Procedure its calling COA_BATCH_PROCESS.f_permit_expiry_email package function and Job is Active in PROD (also pulls both RW & EX)
  4. Batch Job ‘143 – RW Expired Permit’ – After Procedure – “RW Expired Permit” and procedure has both RW & EX foldertype and Job is Active in PROD
TracyLinder commented 8 months ago

Conversation With ROW

image

TracyLinder commented 8 months ago

@Nadin-Nader can you please find out if EX & PSM notifications were ever deployed please? If yes, this card is a bug. If no, we need to build it.

Lpriya-LIT commented 8 months ago

@Nadin-Nader @KyleErich I verified the batch procedure , As per the coding email is sent to RW and Ex folders . I would like to confirm whether it is working for RW and EX or just RW. It would be helpful if i could get a screenshot of the email

TracyLinder commented 7 months ago

Conversation with ROW

TracyLinder commented 7 months ago

@Lpriya-LIT can you please provide an update?

Lpriya-LIT commented 7 months ago

@Nadin-Nader @TracyLinder Fixed the devOps issue and committed the changes. Created Dev deployment request , SCTASK0403552

Nadin-Nader commented 7 months ago

@Lpriya-LIT this has been deployed to DEV please UAT

Lpriya-LIT commented 7 months ago

@Nadin-Nader Verified on Dev, Looks good. Created Test deployment request , SCTASK0404992

Lpriya-LIT commented 6 months ago

@Nadin-Nader Changes deployed to Test , Ready for UAT

KyleErich commented 6 months ago

@Nadin-Nader @Lpriya-LIT @TracyLinder Could someone remind me what steps I need to take to properly UAT this card? I'm blanking on how to generate the auto-email that is referred to in this card. Thanks.

KyleErich commented 6 months ago

@Nadin-Nader @Lpriya-LIT UAT failed on TEST. I had 2021-066745 PSM (FolderRSN 12691229) set to expire on Apr. 12, but did not receive an automated email to the email address associated with my Applicant people record (PeopleRSN 5811726).

Lpriya-LIT commented 6 months ago

@KyleErich @Nadin-Nader The job was not active on Test , I have updated to Active and ran it today. Please check whether you received the email.

When I ran the query in database , there were 2 folders (RW,PSM) for the job to pick for today.Please refer screenshot

Lpriya-LIT commented 6 months ago
image.png
KyleErich commented 6 months ago

@Nadin-Nader @Lpriya-LIT Unfortunately it still seems like I haven't received an email, so I'll have to reject UAT on Test. I set 2021-066745 PSM (FolderRSN 12691229) to expire on Apr. 16, so that I would cross the 9 business day timeframe for the email to generate, but this never happened. Applicant people record 5811726.

Lpriya-LIT commented 5 months ago

@Nadin-Nader I created a test record on Test for PSM folder (13206126) to test the batch email . Please check with ROW team to check whether they received a email today at 9 am (email id - rightofway@austintexas.gov)

KyleErich commented 5 months ago

@Nadin-Nader @Lpriya-LIT There were no attachments present on the email received at 9am on May 1.

TracyLinder commented 5 months ago

UAT Bug ❌

Nadin-Nader commented 3 months ago

@Lpriya-LIT can you update this card or let me know what the status of this is please

Lpriya-LIT commented 3 months ago

Update - Paloma con firmed that there wont be any attachments but the email will have list of folders but the email they received on the test record doesnt includee the list of folders . Checking to find the issue

Lpriya-LIT commented 2 months ago

@Nadin-Nader Done a commit and requested for Dev and Test promotion, SCTASK0441614

Lpriya-LIT commented 2 months ago

@Nadin-Nader On last Friday's call ,Kim and kyle confirmed that the Subject line has ben updated and the list of folders were included in the email that was sent to the ROW email id. Done a commit to add PSM to the list . Changes deployed to Test . Ready for UAT

Lpriya-LIT commented 2 months ago

@Nadin-Nader Created prod deployment request , SCTASK0446737

Lpriya-LIT commented 1 month ago

@Nadin-Nader Ready for UAT on Prod

KyleErich commented 1 month ago

@Nadin-Nader @Lpriya-LIT Approving UAT on Prod, thanks.

Nadin-Nader commented 1 month ago

Approved, Closing