-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update Finch to v0.7.7 #223
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Let's wait for the Jenkins from DACCS Pipeline result then you can perform the bump2version jazz and merge this PR. Don't forget to copy/paste the PR description into the merge commit.
Oh, update the CHANGES.md as well.
Same procedure as in #195 (review)
E2E Test ResultsDACCS-iac Pipeline ResultsBuild URL : http://daccs-jenkins.crim.ca:80/job/DACCS-iac-birdhouse/772/Result : failure BIRDHOUSE_DEPLOY_BRANCH : finch_update_v0.7.6 DACCS_CONFIGS_BRANCH : master PAVICS_E2E_WORKFLOW_TESTS_BRANCH : master PAVICS_SDI_BRANCH : master DESTROY_INFRA_ON_EXIT : true PAVICS_HOST : https://host-140-88.rdext.crim.ca PAVICS-e2e-workflow-tests Pipeline ResultsTests URL : http://daccs-jenkins.crim.ca:80/job/PAVICS-e2e-workflow-tests/job/master/658/NOTEBOOK TEST RESULTS |
Thanks for the quick fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@huard I've spotted an issue while deploying 0.7.6.
See bird-house/finch#213 for the fix.
E2E Test ResultsDACCS-iac Pipeline ResultsBuild URL : http://daccs-jenkins.crim.ca:80/job/DACCS-iac-birdhouse/773/Result : failure BIRDHOUSE_DEPLOY_BRANCH : finch_update_v0.7.6 DACCS_CONFIGS_BRANCH : master PAVICS_E2E_WORKFLOW_TESTS_BRANCH : master PAVICS_SDI_BRANCH : master DESTROY_INFRA_ON_EXIT : true PAVICS_HOST : https://host-140-88.rdext.crim.ca PAVICS-e2e-workflow-tests Pipeline ResultsTests URL : http://daccs-jenkins.crim.ca:80/job/PAVICS-e2e-workflow-tests/job/master/659/NOTEBOOK TEST RESULTS |
E2E Test ResultsDACCS-iac Pipeline ResultsBuild URL : http://daccs-jenkins.crim.ca:80/job/DACCS-iac-birdhouse/774/Result : failure BIRDHOUSE_DEPLOY_BRANCH : finch_update_v0.7.6 DACCS_CONFIGS_BRANCH : master PAVICS_E2E_WORKFLOW_TESTS_BRANCH : master PAVICS_SDI_BRANCH : master DESTROY_INFRA_ON_EXIT : true PAVICS_HOST : https:// PAVICS-e2e-workflow-tests Pipeline ResultsTests URL :NOTEBOOK TEST RESULTS
|
@matprov did the deployment failed in the pipeline? The results are empty. |
@tlvu It's actually caused by the fact that Finch 0.7.7 wasn't released when the infra has been launched.
|
run tests |
E2E Test ResultsDACCS-iac Pipeline ResultsBuild URL : http://daccs-jenkins.crim.ca:80/job/DACCS-iac-birdhouse/775/Result : failure BIRDHOUSE_DEPLOY_BRANCH : finch_update_v0.7.6 DACCS_CONFIGS_BRANCH : master PAVICS_E2E_WORKFLOW_TESTS_BRANCH : master PAVICS_SDI_BRANCH : master DESTROY_INFRA_ON_EXIT : true PAVICS_HOST : https://host-140-46.rdext.crim.ca PAVICS-e2e-workflow-tests Pipeline ResultsTests URL : http://daccs-jenkins.crim.ca:80/job/PAVICS-e2e-workflow-tests/job/master/660/NOTEBOOK TEST RESULTS |
E2E Test ResultsDACCS-iac Pipeline ResultsBuild URL : http://daccs-jenkins.crim.ca:80/job/DACCS-iac-birdhouse/776/Result : failure BIRDHOUSE_DEPLOY_BRANCH : finch_update_v0.7.6 DACCS_CONFIGS_BRANCH : master PAVICS_E2E_WORKFLOW_TESTS_BRANCH : master PAVICS_SDI_BRANCH : master DESTROY_INFRA_ON_EXIT : true PAVICS_HOST : https://host-140-46.rdext.crim.ca PAVICS-e2e-workflow-tests Pipeline ResultsTests URL : http://daccs-jenkins.crim.ca:80/job/PAVICS-e2e-workflow-tests/job/master/661/NOTEBOOK TEST RESULTS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Overview
Changes
Non-breaking changes
Related Issue / Discussion
Additional Information
Revert "added SENTRY_ENV environment variable" by @matprov in #213
Refresh notebooks by @Zeitsperre in bird-house/finch#201
Add SENTRY_ENV param by @matprov in bird-house/finch#203
Multiple rcps for ensemble processes by @aulemahal in bird-house/finch#202
Remove bugging drop after geoserver upgrade by @aulemahal in bird-house/finch#207
Update to xclim 0.31 by @aulemahal in bird-house/finch#209
Fix xarray, dask and gunicorn related hang by @cjauvin in bird-house/finch#204
run load() before switching to single-threaded scheduler by @huard in bird-house/finch#211
Add average flag for ensemble processes by @aulemahal in bird-house/finch#210
Revert "added SENTRY_ENV environment variable" by @matprov in bird-house/finch#213