-
Notifications
You must be signed in to change notification settings - Fork 20
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
(Chore): Refine Single Table Migration #384
Conversation
The style in this PR agrees with This formatting comment was generated automatically by a script in uc-cdis/wool. |
Please find the detailed integration test report here Please find the ci env pod logs here |
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.
👍
can you note the k8 job change in the details for this pr
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.
These changes are pretty straightforward. This all looks good!
Additionally it looks like the failing test is in tests/test_data_upload.py which you didn't edit so there may be issues with that unit test or it might just be worth re-running. |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/DCF-1882
Related PR: uc-cdis/cloud-automation#2657
New Features
Breaking Changes
Bug Fixes
Improvements
Dependency updates
Deployment changes