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.

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

[BUG]: Register artifact store via CLI #2441

Open
1 task done
niccle27 opened this issue Feb 14, 2024 · 1 comment
Open
1 task done

[BUG]: Register artifact store via CLI #2441

niccle27 opened this issue Feb 14, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@niccle27
Copy link

Contact Details [Optional]

No response

System Information

zenml server v0.55.1 (deployed with docker-compose)

What happened?

after connection using zenml client, we try to register an artifact store via :

zenml artifact-store update my_s3_store --client_kwargs='{"endpoint_url": "http://my-s3-endpoint"}'

as stated in the documentation (help of the cli command). Thought the client_kwargs is interpreted as a string and not a json: see
image

Reproduction steps

  1. zenml connect -u http://server-host:8100 --username admin --password zenml
  2. zenml artifact-store update my_s3_store --client_kwargs='{"endpoint_url": "http://my-s3-endpoint"}'

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@niccle27 niccle27 added the bug Something isn't working label Feb 14, 2024
@niccle27
Copy link
Author

Following up on my discoveries. The zenml CLI seem to escape the " which is then misinterpreted by the backend as "client_kwargs": "{\"endpoint_url\": \"http://my-s3-endpoint\"}" }, The issue comes from there

@strickvl strickvl changed the title [BUG]: register artifact store via cli [BUG]: Register artifact store via CLI Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants