lendly-telus-com / cloud-build-demo

Create a simple CI-CD
0 stars 0 forks source link

<retry> Solution for CRQ API limitation #58

Closed lendly-telus-com closed 1 year ago

lendly-telus-com commented 1 year ago

Description Risk Engine service account on ITSM CRQ API only allows one process at a time. Errors related to request timeouts could occur if multiple teams use the ITSM CRQ API simultaneously. Sam from ITSM also suggest a solution for the mean time which is retry

Acceptance Criteria Risk Engine service account should be promoted to admin account so that it would allow multiple process at a time. If the Risk Engine cannot be promoted to admin account. Retry function for API calls should be applied on TDRM and Risk Engine ITSM Plugin

Jeff: Retry function is already work in progress. Will apply to TDRM repo first after completing the function.

lendly-telus-com commented 1 year ago

Todo: handle diff https status

lendly-telus-com commented 1 year ago

retry mechanism is complete Critical: integration ITSM/TDRM repo

lendly-telus-com commented 1 year ago

itsm - effort tested 56 and got 10 failed

lendly-telus-com commented 1 year ago

testing 100 -- ok no fail todo : implementation ITSM plugin

lendly-telus-com commented 1 year ago

tested by improvise todo: final implementation , . 90%

lendly-telus-com commented 1 year ago

code is complete merge

lendly-telus-com commented 1 year ago

todo: integration test

lendly-telus-com commented 1 year ago

code review with Joseph

lendly-telus-com commented 1 year ago

affected by GKE migration

lendly-telus-com commented 1 year ago

local is fine - tested CRQ is fine Joseph - Vinh confirmation prior closing