Added specific CLI return error-code if inventory can not be created

[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?

Add a specific exit code for the CLI to signal failure on initializing the "inventory" storage.

What is the current behavior?

In case of failure to initialize inventory, the CLI returned a generic error code 7, i.e. ErrBadArgument.

What is the new behavior?

In case of failure to initialize inventory, the CLI return an error code 10, i.e. ErrInitializingInventory.

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

Other information

Fix #2352

[kittaakos]

Thanks for the PR!

Avoid startup fatal error if the data directory can not be created.

Why? Is the CLI (daemon) fully functional if the directories.data is missing?

Why no fail with a specific exit code?

[cmaglie]

Why? Is the CLI (daemon) fully functional if the directories.data is missing?

Uhm... actually you can recover from this situation by changing the data_dir using Settings API, but you're right, it's a very convoluted use case, in this case is better to exit with another exit code.

I've changed the PR accordingly.

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (aeac305) 64.60% compared to head (2d40049) 64.58%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2402      +/-   ##
==========================================
- Coverage   64.60%   64.58%   -0.02%     
==========================================
  Files         207      207              
  Lines       19634    19634              
==========================================
- Hits        12684    12681       -3     
- Misses       5861     5863       +2     
- Partials     1089     1090       +1     

Flag	Coverage Δ
unit	64.58% <0.00%> (-0.02%)	⬇️

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

Files	Coverage Δ
internal/cli/cli.go	87.26% <0.00%> (ø)

... and 2 files with indirect coverage changes

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