Un-hide --base-path install flag

[ycombinator]

What does this PR do?

This PR un-hides the --base-path flag defined on the elastic-agent install sub-command.

Why is it important?

To allow users to install Elastic Agent in custom locations.

Checklist

• My code follows the style guidelines of this project
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation See Re-add docs for install '--base-path' parameter ingest-docs#532
• I have made corresponding change to the default configuration files
• I have added tests that prove my fix is effective or that my feature works Already covered by Allow users to specify base path with elastic-agent install #2500
• I have added an entry in ./changelog/fragments using the changelog tool Already covered by Allow users to specify base path with elastic-agent install #2500
• I have added an integration test or an E2E test Already covered by Add integration test for Agent installation #2578

[elasticmachine]

Pinging @elastic/elastic-agent (Team:Elastic-Agent)

[cmacknz]

I would backport this to 8.11 since it is just making the help text visible, the feature is already present and tested.

[elasticmachine]

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

• Start Time: 2023-10-04T21:38:05.749+0000

• Duration: 26 min 44 sec

Test stats 🧪

Test	Results
Failed	0
Passed	6425
Skipped	59
Total	6484

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

• /test : Re-trigger the build.

• /package : Generate the packages.

• run integration tests : Run the Elastic Agent Integration tests.

• run end-to-end tests : Generate the packages and run the E2E Tests.

• run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

[elasticmachine]

🌐 Coverage report

Name	Metrics % (covered/total)	Diff
Packages	98.78% (81/82)	👍
Files	65.667% (197/300)	👍
Classes	64.991% (362/557)	👍
Methods	52.311% (1143/2185)	👍
Lines	37.973% (13010/34261)	👎 -0.031
Conditionals	100.0% (0/0)	💚

[elastic-sonarqube]

SonarQube Quality Gate

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[nimarezainia]

@kilfoyle would you happen to have a doc issue for this already? thx

[kilfoyle]

would you happen to have a doc issue for this already? thx

@nimarezainia The v8.11 and higher version docs were updated just recently via elastic/ingest-docs#532 to show the --base-path option. I'll add a note to the support issue.

[kilfoyle]

@ycombinator I notice that the install command docs haven't been updated to include the --base-path option. I can open up a PR to fix that. Am I correct to understand that the option is available for the install command and not the enroll command?

[ycombinator]

@ycombinator I notice that the install command docs haven't been updated to include the --base-path option. I can open up a PR to fix that.

Yes, good catch — thank you!

Am I correct to understand that the option is available for the install command and not the enroll command?

That's correct.