Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Logic to keep ACI deployment name within Azure character limit error #146

Closed
1 task
kbialka-eqr opened this issue Mar 19, 2024 · 1 comment · Fixed by #147
Closed
1 task

Logic to keep ACI deployment name within Azure character limit error #146

kbialka-eqr opened this issue Mar 19, 2024 · 1 comment · Fixed by #147

Comments

@kbialka-eqr
Copy link

Expectation / Proposal

The logic to assure the ACI deployment name is within Azure's character limit specification applies len() to a UUID without first casting to a string. Issue is workers/container_instance.py here.

Traceback / Example

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/prefect/workers/base.py", line 896, in _submit_run_and_capture_errors
    result = await self.run(
  File "/usr/local/lib/python3.10/site-packages/prefect_azure/workers/container_instance.py", line 571, in run
    max_length=55 - len(flow_run.id),
TypeError: object of type 'UUID' has no len()
@zzstoatzz
Copy link
Contributor

zzstoatzz commented Mar 19, 2024

hi @kbialka-eqr - a fix (#147) for this has been released in v0.3.7

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants