Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Fixing additional truncation of Error #478

Merged
merged 7 commits into from
Sep 23, 2022
Merged

Conversation

kumare3
Copy link
Contributor

@kumare3 kumare3 commented Aug 26, 2022

Signed-off-by: Ketan Umare [email protected]

TL;DR

Removes extra trimming of error message and keeps the 100KB default inplace. Also adds a better message to indicate trimming

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Tracking Issue

fixes flyteorg/flyte#2818

Signed-off-by: Ketan Umare <[email protected]>
go.mod Outdated Show resolved Hide resolved
@@ -40,7 +39,7 @@ type Config struct {
MaxPluginPhaseVersions int32 `json:"max-plugin-phase-versions" pflag:",Maximum number of plugin phase versions allowed for one phase."`
BarrierConfig BarrierConfig `json:"barrier" pflag:",Config for Barrier implementation"`
BackOffConfig BackOffConfig `json:"backoff" pflag:",Config for Exponential BackOff implementation"`
MaxErrorMessageLength int `json:"maxLogMessageLength" pflag:",Max length of error message."`
MaxErrorMessageLength int `json:"maxLogMessageLength" pflag:",Deprecated!!! Max length of error message."`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have this config in two places? is that why? can you add that to the deprecation message? "Use Blah.Blah.MaxErrorMessageLength" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no this is not used.

Signed-off-by: Ketan Umare <[email protected]>
@codecov
Copy link

codecov bot commented Aug 26, 2022

Codecov Report

Merging #478 (20e5bb1) into master (69158ea) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Signed-off-by: Ketan Umare <[email protected]>
Signed-off-by: Ketan Umare <[email protected]>
@kumare3 kumare3 merged commit 082b802 into master Sep 23, 2022
@kumare3 kumare3 deleted the remove-extra-error-trim branch September 23, 2022 18:12
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
* Fixing additional truncation of Error

Signed-off-by: Ketan Umare <[email protected]>

* added trunc message

Signed-off-by: Ketan Umare <[email protected]>

* updated

Signed-off-by: Ketan Umare <[email protected]>

* lint fix

Signed-off-by: Ketan Umare <[email protected]>

* fixed generate

Signed-off-by: Ketan Umare <[email protected]>

Signed-off-by: Ketan Umare <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] FlytePropeller trims error message, proactively
3 participants