search

helm / chart-testing-action

A GitHub Action to lint and test Helm charts
https://github.com/helm/chart-testing
Apache License 2.0
251 stars 71 forks source link

ct lint output crops useful information #67

Open tpolekhin opened 3 years ago

tpolekhin commented 3 years ago

Action output in workflow logs crops lint output about specific lines with issues

Output in the action logs:

Linting chart 'chart-1 => (version: "0.0.0", path: "charts/chart-1")'
Validating /path/to/git/repo/charts/chart-1/Chart.yaml...
Validation success! 👍
Validating maintainers...
Linting chart 'chart-2 => (version: "0.0.0", path: "charts/chart-2")'
Validating /path/to/git/repo/charts/chart-2/Chart.yaml...
Validation success! 👍
##[error][indentation] wrong indentation: expected 2 but found 4
##[error][indentation] wrong indentation: expected 2 but found 4
Error: Error linting charts: Error processing charts
------------------------------------------------------------------------------------------------------------------------
 ✖︎ chart-1 => (version: "0.0.0", path: "charts/chart-1") > Chart doesn't have maintainers
 ✖︎ chart-2 => (version: "0.0.0", path: "charts/chart-2") > Error waiting for process: exit status 1
------------------------------------------------------------------------------------------------------------------------
Error linting charts: Error processing charts
##[error]Process completed with exit code 1.

Output on local run:

Linting chart 'chart-1 => (version: "0.0.0", path: "charts/chart-1")'
Validating /path/to/git/repo/charts/chart-1/Chart.yaml...
Validation success! 👍
Validating maintainers...
Linting chart 'chart-2 => (version: "0.0.0", path: "charts/chart-2")'
Validating /path/to/git/repo/charts/chart-2/Chart.yaml...
Validation success! 👍
charts/chart-2/values.yaml
  19:5      error    wrong indentation: expected 2 but found 4  (indentation)
  20:5      error    wrong indentation: expected 2 but found 4  (indentation)

------------------------------------------------------------------------------------------------------------------------
 ✖︎ chart-1 => (version: "0.0.0", path: "charts/chart-1") > Chart doesn't have maintainers
 ✖︎ chart-2 => (version: "0.0.0", path: "charts/chart-2") > Error waiting for process: exit status 1
------------------------------------------------------------------------------------------------------------------------
Error: Error linting charts: Error processing charts
Error linting charts: Error processing charts
gcaracuel commented 3 years ago

I do have same problem, I have ensured I use exactly same command line options in local but action logs still are empty. I does mention the error as the example above but not the file causing the error nor the lines of that file that requires a fix.