Closed tumido closed 3 days ago
Hi @tumido. Thanks for your PR.
I'm waiting for a kubeflow member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
@hbelmiro can you please approve the CI to be run on this? It seems ok-to-test
is not enough and some additional button clicking or keyboard smashing is still required. 🤷♂️ 🙂
/lgtm /approve
@tumido thank you! if you are up for it there are some other low hanging fruits here as well that can help with connecting against endpoints, I'm mainly thinking of ssl_ca_cert
and verify_ssl
if you're up for it 🥲
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: HumairAK
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Description of your changes:
This PR exposes
existing_token
option ofkfp.Client
to the CLI.Resolves: https://github.com/kubeflow/pipelines/issues/11399
Checklist: