Skip to content
This repository has been archived by the owner on Jan 23, 2025. It is now read-only.

v1.0 - Refactored legacy-submission-processor #4

Open
bountyC0d3r opened this issue Feb 4, 2020 · 10 comments
Open

v1.0 - Refactored legacy-submission-processor #4

bountyC0d3r opened this issue Feb 4, 2020 · 10 comments
Assignees
Milestone

Comments

@bountyC0d3r
Copy link
Collaborator

bountyC0d3r commented Feb 4, 2020

  • all refactored legacy submission processor, implemented using nodejs v10
  • single processor handeling MM and Non-MM submissions
@bountyC0d3r bountyC0d3r added this to the v1.0 milestone Feb 4, 2020
@bountyC0d3r bountyC0d3r self-assigned this Feb 4, 2020
@lakshmiathreya
Copy link

-- Verified MM and non-MM submissions
-- Verified setting of Initial and Final scores for MM submissions
-- Verified non-DMZ url for submissions

@lakshmiathreya
Copy link

cc @sachin-maheshwari can you pls update the current status here for future record? cc @drasticdpk

@sachin-maheshwari
Copy link

@lakshmiathreya on OR we are seeing multiple submissions for development challenge.

@drasticdpk
Copy link

@drasticdpk
Copy link

Prod Verification

Development
image

Data Science

image

Design

image

QA

image

@drasticdpk
Copy link

Note : We have revert back it from prod successfully.
Reference : topcoder-platform/legacy-sub-processor#36
cc @sachin-maheshwari @lakshmiathreya

@sachin-maheshwari
Copy link

sachin-maheshwari commented Mar 17, 2021

Thanks @drasticdpk.
Just for our reference -
This new version have following issues, we need to fix and verify again before deploying on prod -

  • Multiple submissions are displaying at OR for development/code challenges. As per system-design, development/code only last submission should be considered, previous submission should be marked as delete ("upload_status_id" =2, upload table, tcs_catalog) in the legacy (informix) DB.
  • In this version, there is no support of S3 URL update, which is required by support team to fix the existing issue (DMZ url missing)(not sure, but as informed by support team root cause is - due to large size of submission)

I am not sure how many other issues this version have. I think, QA team need to first analyze the existing functionalities of current (older) version.

@sachin-maheshwari
Copy link

@lakshmiathreya I didn't get, why you added its QA passed on prod. It was deployed on 1 March 2021 with "shapeup-pure-v5-task", Its failed on prod due to some funcalities were not working as mentioned above and we reverted yesterday.
Also i think it's not P0 ? its should be P1 or P2 ? as basic functionality (syncing to legacy system) is working.

@lakshmiathreya
Copy link

@sachin-maheshwari point noted - this is P0 but will remain open since resolution was not a fix but was a revert ...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants