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

Fullfilling 189 #190

Merged
merged 5 commits into from
Dec 19, 2024
Merged

Fullfilling 189 #190

merged 5 commits into from
Dec 19, 2024

Conversation

jbusecke
Copy link
Collaborator

@jbusecke
Copy link
Collaborator Author

Ahh brilliant! This confirms that the error was indeed only occuring with the GISS model. I am tracking that smaller issue here and will merge and run this one here once this action run is completed. @scoats please remind me if I should forget to deploy this.

@jbusecke
Copy link
Collaborator Author

Just changed the PR config and iid list to include one OOM failcase from #194. If that works, I am considering switching from Dataflow prime to fixed instances to unblock all these HighResMIP datasets.

@jbusecke
Copy link
Collaborator Author

jbusecke commented Dec 19, 2024

Ok so it seems that the test job did not OOM, but ran into a test error. Ill leave the worker config this high for now, but maybe worth checking back on that in a bit? This run cost about 12 cents, so not too terrible. Lets run this bad boy and see which ones we might get.

@jbusecke jbusecke merged commit b252c4d into main Dec 19, 2024
5 of 6 checks passed
@jbusecke jbusecke deleted the fulfil-189 branch December 19, 2024 17:06
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.

1 participant