Skip to content

Add script to request model versions#2531

Merged
nikhilwoodruff merged 1 commit into
masterfrom
feat/v1-versioning-script
Jun 3, 2025
Merged

Add script to request model versions#2531
nikhilwoodruff merged 1 commit into
masterfrom
feat/v1-versioning-script

Conversation

@anth-volk
Copy link
Copy Markdown
Collaborator

Fixes #2530

Adds a Bash script to invoke the simulation API's `wait_for_country_version" Google Cloud workflow. This workflow uses a polling structure to asynchronously determine the simulation API's model versions and return SUCCESS if they match the ones provided to the script, or to fail if not.

In a future PR, I will run this script as part of a Google Cloud workflow, passing in the API deployment's model versions.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.62%. Comparing base (8b08421) to head (38c5f8f).
Report is 12 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2531   +/-   ##
=======================================
  Coverage   63.62%   63.62%           
=======================================
  Files          56       56           
  Lines        2161     2161           
  Branches      287      287           
=======================================
  Hits         1375     1375           
  Misses        729      729           
  Partials       57       57           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Collaborator

@mikesmit mikesmit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Collaborator

@mikesmit mikesmit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (for reals)

@nikhilwoodruff nikhilwoodruff merged commit 83ed92e into master Jun 3, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create Bash script to check for simulation API version using wait_for_country_versions.yaml

3 participants