Skip to content

Conversation

@pingsutw
Copy link
Member

Tracking issue

NA

Why are the changes needed?

Put the connector proto in one place

What changes were proposed in this pull request?

Move the connector proto to the connector folder

(kevin) (kevin) ➜  flyte git:(update-connector-idl) tree flyteidl2/connector 
flyteidl2/connector
├── connector.proto
└── service.proto

How was this patch tested?

NA

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: Kevin Su <pingsutw@apache.org>
Signed-off-by: Kevin Su <pingsutw@apache.org>
@flyte-bot
Copy link
Collaborator

Bito Automatic Review Skipped - Branch Excluded

Bito didn't auto-review because the source or target branch is excluded from automatic reviews.
No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the branch exclusion settings here, or contact your Bito workspace admin at eduardo@union.ai.

Copy link
Contributor

@EngHabu EngHabu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you 🙏🏽

@pingsutw pingsutw merged commit 913f425 into v2 Dec 11, 2025
4 checks passed
@pingsutw pingsutw deleted the update-connector-idl branch December 11, 2025 19:12
kumare3 pushed a commit to flyteorg/flyte-sdk that referenced this pull request Dec 12, 2025
Connector IDL has been updated in
flyteorg/flyte#6797, so we need to update the
import modules in the SDK

---------

Signed-off-by: Kevin Su <pingsutw@apache.org>
yuhuan130 pushed a commit to yuhuan130/flyte-sdk that referenced this pull request Dec 14, 2025
Connector IDL has been updated in
flyteorg/flyte#6797, so we need to update the
import modules in the SDK

---------

Signed-off-by: Kevin Su <pingsutw@apache.org>
Signed-off-by: “Alex <alexchien130@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants