Improved logging on GitHub Actions runners (using log folding commands)

[cmaglie]

Please check if the PR fulfills these requirements

See how to contribute

• The PR has no duplicates (please search among the Pull Requests
  before creating one)
• The PR follows
  our contributing guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)
• UPGRADING.md has been updated with a migration guide (for breaking changes)
• configuration.schema.json updated if new parameters are added.

What kind of change does this PR introduce?

• Adds logs folding in GH-actions runners, to improve the readability of logs.
• Remove the -v flags from go build command to avoid printing hundreds of package name being build, clogging up the log with useless information.

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change, and is titled accordingly?

Other information

https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#grouping-log-lines

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (17f0189) 67.15% compared to head (e887512) 67.16%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2414      +/-   ##
==========================================
+ Coverage   67.15%   67.16%   +0.01%     
==========================================
  Files         210      210              
  Lines       20523    20523              
==========================================
+ Hits        13782    13785       +3     
+ Misses       5602     5600       -2     
+ Partials     1139     1138       -1     

Flag	Coverage Δ
unit	67.16% <ø> (+0.01%)	⬆️

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

see 1 file with indirect coverage changes

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

[alessio-perugini]

🤩 Wow I didn't know this feature of GA!
Should we also remove the -v flag from the tests? Without that flag, only failing tests should print in the log.

  go:test:
    desc: Run unit tests and the Go-based integration tests
    deps:
      - task: go:build
    dir: '{{default "./" .GO_MODULE_PATH}}'
    cmds:
      - |
        go test \
          -v \ <------- remove this?
          -short \
          -run '{{default ".*" .GO_TEST_REGEX}}' \
          {{default "-timeout 20m -coverpkg=./... -covermode=atomic" .GO_TEST_FLAGS}} \
          -coverprofile=coverage_unit.txt \
          {{default .DEFAULT_GO_PACKAGES .GO_PACKAGES}} \
          {{.TEST_LDFLAGS}}

[umbynos]

Amazing!! 📁

[cmaglie]

Should we also remove the -v flag from the tests? Without that flag, only failing tests should print in the log.

That would be the optimal solution, but I like to have the full logs available.
BTW I'm merging this one as is for now, I'd like to see if I can report errors in the "summary" page with this: https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-error-message