Skip to content
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.

Sign up for GitHub

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

Jump to bottom

docs(kfp): 1.7 upgrade notices and breaking changes #2895

Merged
merged 4 commits into from
Aug 27, 2021
Merged

docs(kfp): 1.7 upgrade notices and breaking changes #2895

merged 4 commits into from
Aug 27, 2021

Conversation

Bobgy
Copy link
Contributor

@Bobgy Bobgy commented Aug 26, 2021 •
edited
Loading

Preview: https://deploy-preview-2895--competent-brattain-de2d6d.netlify.app/docs/components/pipelines/upgrade/
Changes:

  • add sections in KFP upgrading page about notices and breaking changes
  • adds references to the upgrading page from related docs
  • update some information for KFP v2 compatibility mode

@google-cla google-cla bot added the cla: yes label Aug 26, 2021
@Bobgy
Copy link
Contributor Author

Bobgy commented Aug 26, 2021 •
edited
Loading

/hold
The PR is based on #2893. It should be merged first.

shuesc1
shuesc1 reviewed Aug 26, 2021
View reviewed changes
content/en/docs/components/pipelines/installation/compatibility-matrix.md Outdated Show resolved Hide resolved
content/en/docs/components/pipelines/installation/compatibility-matrix.md Outdated Show resolved Hide resolved
content/en/docs/components/pipelines/installation/compatibility-matrix.md Outdated Show resolved Hide resolved
content/en/docs/components/pipelines/installation/compatibility-matrix.md Outdated Show resolved Hide resolved
content/en/docs/components/pipelines/upgrade.md Outdated

For detailed configuration and migration instructions for both options, refer to [Argo Workflow Executors](https://www.kubeflow.org/docs/components/pipelines/installation/choose-executor/).

* **Notice**: [Kubeflow Pipelines SDK v2 compatibility mode](/docs/components/pipelines/sdk/v2/v2-compatibility/) (Beta) was recently released. The new mode adds support for tracking pipeline runs and artifacts using ML Metadata. In v1.7 backend release, complete UI support and caching capabilities for v2 compatibility mode is newly added, [your feedback is welcomed](https://github.com/kubeflow/pipelines/issues/6451).
Copy link

Choose a reason for hiding this comment

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

"In v1.7 backend release complete UI support and caching capabilities for v2 compatibility mode are newly added. File feedback for any issues you encounter."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @shuesc, in addition to issues people may encounter, we also want to welcome feedback if some new feature is working well for them. Do you have any suggestions how to convey this message?

Copy link

@shuesc1 shuesc1 Aug 27, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

Ah I see. In that case:

"We welcome any feedback on positive experiences or issues you encounter."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I'll send another PR

@Bobgy Bobgy mentioned this pull request Aug 27, 2021
Merged
@Bobgy
Copy link
Contributor Author

Bobgy commented Aug 27, 2021

Note, TFX compatibility matrix related content is merged in #2893.

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Bobgy

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Bobgy
Copy link
Contributor Author

Bobgy commented Aug 27, 2021

/unhold

@Bobgy Bobgy added the lgtm label Aug 27, 2021
@google-oss-robot google-oss-robot merged commit 4972fa3 into master Aug 27, 2021
@Bobgy
Copy link
Contributor Author

Bobgy commented Aug 27, 2021

Welcome continued comments, I'm merging to unblock release and other related documentation that needs to point to this.

@Bobgy Bobgy deleted the kfp-upgrade branch August 27, 2021 04:42
@Bobgy Bobgy mentioned this pull request Aug 30, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kramachandran kramachandran kramachandran requested changes

@shuesc1 shuesc1 shuesc1 left review comments

@alfsuse alfsuse Awaiting requested review from alfsuse

@joeliedtke joeliedtke Awaiting requested review from joeliedtke

Assignees
No one assigned
Labels
approved cla: yes lgtm size/M
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Bobgy @google-oss-robot @kramachandran @shuesc1