Fixing 100 kilobyte max error message size

[hamersaw]

Tracking issue

NA

Why are the changes needed?

This MaxErrorMessageLength value is now used to truncate ArrayNode errors, since it was actually 100MB this resulted in potentially large FlyteWorkflow CRs.

What changes were proposed in this pull request?

Correctly computing the 100KB to B conversion.

How was this patch tested?

NA

Setup process

NA

Screenshots

NA

Check all the applicable boxes

• I updated the documentation accordingly.
• All new and existing tests passed.
• All commits are signed-off.

Related PRs

NA

Docs link

NA

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2aab954) 59.02% compared to head (0f7d9fe) 59.02%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #4631   +/-   ##
=======================================
  Coverage   59.02%   59.02%           
=======================================
  Files         622      622           
  Lines       52816    52816           
=======================================
+ Hits        31174    31175    +1     
+ Misses      19151    19150    -1     
  Partials     2491     2491           

Flag	Coverage Δ
unittests	59.02% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.