-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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.
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
[backend] Query Strings in the Pipeline Root are ignored #10318
Comments
Using custom s3 endpoints now works for me (given #10319):
|
…10318) (kubeflow#10319) * feat: preserve querystring in pipeline root * refactor: create AppendToPipelineRoot Also apply to client.go * feat: remove query string from URIs (kubeflow#1) * feat: remove query string from URIs * refactor(GenerateOutputURI): move and preserve comments
I used minio, I configured it this way, but it didn't work @TobiasGoerke |
Hard to say what's going wrong given your screenshot. |
This is the example I use |
You'll either need to specify these vars manually or use tools like OPA / Kyverno. |
defaultPipelineRoot: s3://kubeflow-pipelines/v2?endpoint=minio.examplecom:443&s3ForcePathStyle=true |
Seems correct. Although you will also need to mount the env variables to you pipeline pods, like this (better you OPA / Kyverno though):
|
To my understanding, using custom artifact stores in pipelines v2 requires configuring the pipelineRoot.
According to the blob documentation, an endpoint needs to be configured by using a queryString that gets appended to the pipelineRoot.
However, the kfp driver ignores query strings and assembles wrong bucket names.
This issue effectively blocks using custom S3 artifact stores for pipelines v2.
Environment
KFP 2.0.5.
Steps to reproduce
Set a custom Pipeline Root, e.g. by setting the kfp-launcher's ConfigMap key
defaultPipelineRoot
tos3://my-bucket/my-dir?endpoint=my.endpoint&s3ForcePathStyle=true
.Expected result
We'll see errors like
Also, after the driver has been executed, the wrongly concatenated pipelineRoot is visible in the workflow yaml.
Impacted by this bug? Give it a 👍.
The text was updated successfully, but these errors were encountered: