ucx icon indicating copy to clipboard operation
ucx copied to clipboard

[BUG]: Truncate logs from `acceptance` action

Open nfx opened this issue 2 years ago • 4 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Current Behavior

image

Expected Behavior

No response

Steps To Reproduce

No response

Cloud

AWS

Operating System

macOS

Version

latest via Databricks CLI

Relevant log output

No response

nfx avatar Apr 22 '24 19:04 nfx

when jobs fail, on the side of the workflow within DBR, we may produce more than 1MB of logs and if we try to post it into github comment, we get bad request error, because the total payload size of github comments or descriptions must not exceed 1MB. The fix for this must truncate the exceessive log output. See the relevant code:

https://github.com/databrickslabs/ucx/blob/e3bdd3d0395324238d51748322acdc11c43e7f9d/src/databricks/labs/ucx/installer/workflows.py#L125-L130

nfx avatar May 07 '24 10:05 nfx

Re-opening since #1655 doesn't seem to limit the uploaded size, see https://github.com/databrickslabs/ucx/actions/runs/9115359784?pr=1703

ericvergnaud avatar May 17 '24 07:05 ericvergnaud

@nfx exploring this again

I'm not sure if the suggested fix (implemented in #1655) is relevant:

  • it only applies to a small subset of the acceptance tests (the ones that run a workflow)
  • the failing upload seems to be attempting to upload the entire logs for the GH action, not just for the failing test

Wouldn't it be more relevant to limit the comment size in https://github.com/databrickslabs/sandbox/blob/88d1d5e50a001b9d8e4d6a819f0e9e47d37eb631/acceptance/boilerplate/actions.go#L96 ?

ericvergnaud avatar May 24 '24 10:05 ericvergnaud

Issued poped up again: https://github.com/databrickslabs/ucx/actions/runs/11007362115/job/30563142660

JCZuurmond avatar Sep 24 '24 06:09 JCZuurmond

fixed in https://github.com/databrickslabs/sandbox/pull/261

nfx avatar Oct 09 '24 10:10 nfx