What is the proper way to specify required Cargo.toml fields using environment variable? #808
Replies: 3 comments 1 reply
|
Hi. Can you try without the |
|
Yes, I'm already did. In this case docker fails with error: |
|
Hi! I had some time to look deeper into the issue. You found a bug! It is not possible to de-serialize an array from an environment variable like that. The field works, when used from a TOML file but that is not feasible for Docker/Kubernetes. I found a fix and currently test it locally. If it works, I'll uptream it and the next version of Kellnr should be able to parse an array from the environment correctly. |
|
When #818 is merged an released the proper way to use an array of values for the field is a comma separated list. For example: KELLNR_REGISTRY__REQUIRED_CRATE_FIELDS=authors,descriptionI'll close the discussion and proceed in the created bug ticket. |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
I found that documentation suggests using environment variable for setup using docker.
The corresponding row of the table of configuration values reads as:
table (source)
[registry.required_crate_fields]KELLNR_REGISTRY__REQUIRED_CRATE_FIELDS[]["authors", "repository"], all uploaded crates have to have the authors and repository defined in their Cargo.toml.Setting
KELLNR_REGISTRY__REQUIRED_CRATE_FIELDSin compose.yml produces the error:compose.yml
Do I perform something wrong? What is the proper way?
All reactions