Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deploy & run load-transform in production #1956

Closed
jamesbursa opened this issue May 8, 2024 · 2 comments
Closed

Deploy & run load-transform in production #1956

jamesbursa opened this issue May 8, 2024 · 2 comments
Assignees
Labels
refinement topic: backend Backend development tickets topic: infra Infrastructure related tickets

Comments

@jamesbursa
Copy link
Collaborator

Summary

Deploy the latest code to production and run the load-transform job in production.

Troubleshoot and fix any errors or issues. Verify that the database looks right.

Acceptance criteria

  • Code has been run in production.
  • Fixes have been made and/or follow up tickets created.
@jamesbursa jamesbursa added the project: grants.gov Grants.gov Modernization tickets label May 8, 2024
@jamesbursa jamesbursa added this to the Search API - ELT Implementation milestone May 8, 2024
@jamesbursa jamesbursa self-assigned this May 8, 2024
@jamesbursa
Copy link
Collaborator Author

Deployment was successful (https://github.com/HHS/simpler-grants-gov/releases/tag/2024.5.9-1) after setting up schemas #1848.

The foreign tables were created manually.

Then executed the load using:

bin/run-command.sh api prod '["flask", "--app=src.app:create_app", "data-migration", "load-transform", "--load", "--no-transform", "--no-set-current"]'

It was successful on the following tables:

  • "tforecast"
  • "tforecast_hist"
  • "tapplicanttypes_forecast"
  • "tapplicanttypes_forecast_hist"
  • "tfundactcat_forecast"
  • "tfundactcat_forecast_hist"
  • "tfundinstr_forecast"
  • "tfundinstr_forecast_hist"
  • "topportunity"
  • "topportunity_cfda"

Failed on tables:

  • "tsynopsis" - psycopg.errors.FdwUnableToCreateExecution: error fetching result: OCIStmtFetch2 failed to fetch next result row / DETAIL: ORA-01555: snapshot too old: rollback segment number 21 with name "_SYSSMU21_2881811878$" too small

@jamesbursa jamesbursa moved this from Sprint Ready to In Progress in Simpler.Grants.gov Product Backlog May 9, 2024
@mxk0 mxk0 changed the title [Task]: deploy & run load-transform in production Deploy & run load-transform in production Oct 2, 2024
@mxk0 mxk0 moved this from In Progress to Todo in Simpler.Grants.gov Product Backlog Oct 4, 2024
@mxk0 mxk0 added topic: backend Backend development tickets refinement infra topic: infra Infrastructure related tickets and removed infra project: grants.gov Grants.gov Modernization tickets labels Oct 6, 2024
@chouinar chouinar self-assigned this Oct 15, 2024
@chouinar chouinar moved this from Todo to In Progress in Simpler.Grants.gov Product Backlog Oct 15, 2024
@chouinar
Copy link
Collaborator

Comparing the raw counts of the opportunity statuses between the legacy and our process after running fully in prod.

We have 3 fewer forecasted opportunities.
The same amount of posted.
The same amount of closed.
The same amount of archived.

Looking a bit closer:

344227: forecasted != archived
344228: forecasted != archived
337347: forecasted != archived
322537: forecasted != archived
336962: None != forecasted

The left value is grants.gov - the right value is our data

I remember looking at 336962 before the break - I wasn’t sure why it wasn’t forecasted in the legacy system. I’m guessing just some issue with the record getting loaded into their search?

https://grants.gov/search-results-detail/344227 and https://grants.gov/search-results-detail/344228 are both clearly the “has both a synopsis and forecast - but uses the forecast” case that the legacy system supports and we do not.

https://grants.gov/search-results-detail/337347 and https://grants.gov/search-results-detail/322537 are also the same case - but it’s not as obvious from the UI. The synopsis record they have that we have take precedence doesn’t display on the UI because they only have a single version of the synopsis (the legacy UI never displays the most recent version of the forecast/synopsis when the synopsis/forecast shows)

@margaretspring margaretspring removed this from the Search API - ELT Implementation milestone Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refinement topic: backend Backend development tickets topic: infra Infrastructure related tickets
Projects
Development

No branches or pull requests

4 participants