Rajvir Jhawar
08/01/2022, 9:00 AMMathijs Carlu
08/01/2022, 9:22 AMread-path
and write-path
, although I agree that tagging them with storage
would be better
⢠All blocks tagged run-infrastructure
can be passed as --infra
or -ib
⢠If you're passing -ib
, you should convert all characters in type to lowercase and change spaces to dashes, e.g. Kubernetes Job
becomes kubernetes-job
. This should indeed be documentedAnna Geller
prefect deployment build --help
Anna Geller
Mathijs Carlu
08/01/2022, 11:14 AMRajvir Jhawar
08/01/2022, 1:54 PMhelp="The slug of the storage block. Use the syntax: 'block_type/block_name', where block_type must be one of 'local-file-system', 'remote-file-system', 's3', 'gcs', 'azure'",
Would it not be preferable to have something like this:
help="The slug of the storage block. Use the syntax: 'block_type/block_name', where block_type must have a tag of "storage-block" in the block's UI, ex: ...
A followup to my point about the "Copy Name" feature:
If there was a "Copy Block Name" option it would have aided me learning how to format the string for the deployment build command. If I had clicked the "Copy Block Name" button I would have been able to see that the string should be formatted as: kubernetes-job/<name>
(This point May not be super relevant it is just something I feel is a little inconsistent in terms of the UI)
One last point here is that the formatting is different for different types of blocks:
For instance a google storage block displays:
gcs/<name>
--> this is a direct mapping
In the case of remote/local file system and k8 jobs this is not the case:
Kubernetes Job/<name>
--> this is not a direct mapping
Would it not be preferable to display this instead:
kubnernetes-job/<name>
Anna Geller
Anna Geller
Rajvir Jhawar
08/02/2022, 12:00 PMAnna Geller