Fix the messaging for resource configuration

[pmahindrakar-oss]

Signed-off-by: Prafulla Mahindrakar [email protected]

TL;DR

Fix the messaging for task resource configurations.

Eg :
Messages on parsing error for the resource

Parsing of CPU request failed for value 200Q - reason  quantities must match the regular expression '^([+-]?[0-9.]+)([eEinumkKMGTP]*[-+]?[0-9]*)$'. Please follow K8s conventions for resources https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/

In case of values going over platform limits

Requested CPU limit [1536Mi] is greater than current limit set in the platform configuration [1Gi]. Please contact Flyte Admins to change these limits or consult the configuration

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

Complete description

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

flyteorg/flyte#512

Follow-up issue

NA
OR
https://github.com/flyteorg/flyte/issues/

[codecov]

Codecov Report

Merging #251 (5096ba4) into master (5c86bbe) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #251      +/-   ##
==========================================
+ Coverage   57.47%   57.53%   +0.05%     
==========================================
  Files         141      141              
  Lines       10481    10490       +9     
==========================================
+ Hits         6024     6035      +11     
+ Misses       3787     3786       -1     
+ Partials      670      669       -1     

Flag	Coverage Δ
unittests	56.33% <100.00%> (+0.06%)	⬆️

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

Impacted Files	Coverage Δ
pkg/manager/impl/validation/task_validator.go	83.33% <100.00%> (+2.48%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c86bbe...5096ba4. Read the comment docs.