Skip to content

Conversation

TheMichaelHu
Copy link
Contributor

@TheMichaelHu TheMichaelHu commented Sep 21, 2022

First-party pipelines are not yet available in AR, meaning other than using a local file, the only way to access a first-party pipeline is using its GitHub URI. Since support was added for AR URIs, it is not much more effort to support general HTTPS URIs.

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes b/247878583 🦕

@TheMichaelHu TheMichaelHu requested a review from a team as a code owner September 21, 2022 02:00
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: vertex-ai Issues related to the googleapis/python-aiplatform API. labels Sep 21, 2022
@TheMichaelHu TheMichaelHu force-pushed the mh-http-pipeline branch 4 times, most recently from 3b95610 to 45e0d9f Compare September 21, 2022 14:16
@rosiezou
Copy link
Contributor

@sararob please review

@sasha-gitg sasha-gitg requested review from a team and removed request for a team September 22, 2022 20:58
Copy link
Member

@chensun chensun left a comment

Choose a reason for hiding this comment

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

/lgtm

@TheMichaelHu TheMichaelHu added the automerge Merge the pull request once unit tests and other checks pass. label Sep 26, 2022
@gcf-merge-on-green gcf-merge-on-green bot merged commit 926d0b6 into googleapis:main Sep 26, 2022
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Sep 26, 2022
nayaknishant pushed a commit to nayaknishant/python-aiplatform that referenced this pull request Sep 27, 2022
First-party pipelines are not yet available in AR, meaning other than using a local file, the only way to access a first-party pipeline is using its GitHub URI. Since support was added for AR URIs, it is not much more effort to support general HTTPS URIs.

- [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/python-aiplatform/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [x] Ensure the tests and linter pass
- [x] Code coverage does not decrease (if any source code was changed)
- [x] Appropriate docs were updated (if necessary)

Fixes b/247878583 🦕
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: vertex-ai Issues related to the googleapis/python-aiplatform API. size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants