Skip to content

Commit 2a5cc4a

Browse files
committed
ci: install flytectl directly instead of the broken setup action
unionai-oss/flytectl-setup-action resolves the latest flytectl from the first page of flyteorg/flyte releases. That page is now entirely Flyte 2.0 (v2.0.x) tags with no flytectl/ release, so the action crashes with "Cannot read properties of undefined (reading 'assets')" and reds the integration jobs on master and every PR. Download the last flytectl release (v0.9.8) from its asset URL instead. Signed-off-by: 1fanwang <1fannnw@gmail.com>
1 parent 6cb953a commit 2a5cc4a

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

‎.github/workflows/pythonbuild.yml‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -242,7 +242,12 @@ jobs:
242242
make setup-global-uv
243243
uv pip freeze
244244
- name: Install FlyteCTL
245-
uses: unionai-oss/flytectl-setup-action@master
245+
# unionai-oss/flytectl-setup-action resolves "latest" from the first page of flyteorg/flyte releases, which is
246+
# now all Flyte 2.0 (v2.0.x) tags with no flytectl/ release on page 1, so it crashes on `reading 'assets'`.
247+
# Install the last flytectl release directly instead.
248+
run: |
249+
curl -sL "https://github.com/flyteorg/flyte/releases/download/flytectl/v0.9.8/flytectl_Linux_x86_64.tar.gz" | sudo tar -xz -C /usr/local/bin flytectl
250+
flytectl version
246251
- name: Setup Flyte Sandbox
247252
run: |
248253
flytectl demo start --disable-agent

0 commit comments

Comments
 (0)