Resolving issue with regex for composer environments
Description
Fixes #12580
Checklist
- [x] I have followed Sample Guidelines from AUTHORING_GUIDE.MD
- [ ] README is updated to include all relevant information
- [x] Tests pass:
nox -s py-3.9(see Test Environment Setup) - [x] Lint pass:
nox -s lint(see Test Environment Setup) - [ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
- [ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
- [ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
- [ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
- [x] Please merge this PR for me once it is approved
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.
Looks like this will require additional changes as it currently produces the following errors since minor and patch are not set.
2024-09-12 14:15:47,029 - Image version: composer-3-airflow-2.9.1-build.6
Traceback (most recent call last):
File "/home/rkillkelley/composer_dags.py", line 229, in <module>
main(
File "/home/rkillkelley/composer_dags.py", line 173, in main
airflow_version = (int(versions[3]), int(versions[4]), int(versions[5]))
IndexError: tuple index out of range
I will update to ensure minor and patch default to 0
@killkelleyr thanks for contributing this fix. @leahecole can you take a look?
This PR requires a rebase due to testing changes in this repo. If this PR is no longer needed, please close ✨