#587 Remove content md5 requirement
Pull request opened by
wild-endeavor
TL;DR
Remove the requirement that content md5 is provided.
Depends on
flyteorg/flytestdlib#160
Type
☐ Bug Fix
☑︎ Feature
☐ Plugin
Are all requirements met?
☐ Code completed
☑︎ Smoke tested
☑︎ Unit tests added
☐ Code documentation added
☐ Any pending items have an associated Issue
Complete description
If the full native path is fully specified, not random, then we check to see if it exists.
If it exists then the user needs to have given a hash, which admin will check against the etags.
etags are computed differently for different stores, so we try a few different ways.
if the tag doesn't match, then we error. if no hash was given then also error.
Tracking Issue
This is related to
flyteorg/flytekit#1674
flyteorg/flyteadmin
GitHub Actions: Goreleaser
GitHub Actions: Build & Push Flytescheduler Image
GitHub Actions: Build & Push Flyteadmin Image
GitHub Actions: Bump Version
✅ 10 other checks have passed
10/14 successful checks