Fix missing link target

[mpass99]

for listing the file system with a link, the executing user does not have enough privileges to read the target.

Closes #596

[codecov]

Codecov Report

Attention: Patch coverage is 53.33333% with 7 lines in your changes missing coverage. Please review.

Project coverage is 76.38%. Comparing base (f848fee) to head (14a7205).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/runner/nomad_runner.go	28.57%	5 Missing ⚠️
pkg/nullio/ls2json.go	75.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #662      +/-   ##
==========================================
+ Coverage   76.20%   76.38%   +0.18%     
==========================================
  Files          43       43              
  Lines        3551     3557       +6     
==========================================
+ Hits         2706     2717      +11     
+ Misses        620      617       -3     
+ Partials      225      223       -2     

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

[MrSerth]

Thanks, looks good! I tested the branch locally and can confirm that your patch resolves the issue for me.

There is one question left, though: When listing files where this (former) bug triggers, I still have an exit code 1 and thus the following warning: level=warning msg="Ignoring error of listing the file system" error="<nil>" exitCode=1 package=runner. Do we just continue to ignore it (and log as warning) or do we want to do something special (since we know why the error was generated in the first place....)

[MrSerth]

Perfect, works like a charm. Thanks!

[MrSerth]

I've created an issue for the flaky unit test #669.

[MrSerth]

I've also amended ticket #623 for the failed e2e test. It seems like we need to investigate in the test suite, once again.