Skip to content

[V2] Dataproxy check project exists#7246

Open
machichima wants to merge 2 commits intoflyteorg:v2from
machichima:dataproxy-check-project-exists
Open

[V2] Dataproxy check project exists#7246
machichima wants to merge 2 commits intoflyteorg:v2from
machichima:dataproxy-check-project-exists

Conversation

@machichima
Copy link
Copy Markdown
Member

@machichima machichima commented Apr 21, 2026

Tracking issue

Why are the changes needed?

Currently, when doing flyte run or flyte deploy with project that does not exists, we can successfully execute the runs and deploy tasks, but we can never see it on the UI as the project does not exists. We should block submitting runs / tasks to the project that does not exists.

What changes were proposed in this pull request?

In CreateUploadLocation, validate if the project exists before constructing signed url. If cannot find the project, raise "Not Found" error so flyte-sdk can raise it in CLI.

How was this patch tested?

Run flyte run and flyte deploy with project that does not exists. An error should be raise.

image

Labels

Please add one or more of the following labels to categorize your PR:

  • added: For new features.
  • changed: For changes in existing functionality.
  • deprecated: For soon-to-be-removed features.
  • removed: For features being removed.
  • fixed: For any bug fixed.
  • security: In case of vulnerabilities

This is important to improve the readability of release notes.

Setup process

Screenshots

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Docs link

Signed-off-by: machichima <nary12321@gmail.com>
Signed-off-by: machichima <nary12321@gmail.com>
@github-actions github-actions Bot mentioned this pull request Apr 21, 2026
3 tasks
@pingsutw pingsutw added this to the V2 GA milestone Apr 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants