Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cherry-pick #20929 to 7.9: Clarify use for shared_credential_file #21082

Merged
merged 1 commit into from
Sep 15, 2020
Merged

Cherry-pick #20929 to 7.9: Clarify use for shared_credential_file #21082

merged 1 commit into from
Sep 15, 2020

Conversation

kaiyan-sheng
Copy link
Contributor

@kaiyan-sheng kaiyan-sheng commented Sep 14, 2020

Cherry-pick of PR #20929 to 7.9 branch. Original message:

When starting Beats as a service, the Beat's PID will be owned by the user that manages the service. This would be root in most cases.

Users tend to run tests as non-root, running beats directly (./metricbeat) on the command line. Without the shared_credential_file path the beat checks for credentials under the user's home directory.

As a service, the home directory of the user managing the service (typically root) tends to be different than that of the user in testing and development, which can be difficult to figure out.

What does this PR do?

Clarifies AWS module's credential lookup behavior and when to use shared_credential_file to avoid ambiguity.

Why is it important?

Helps users avoid credential lookup issues, especially deviations between environments where Beats run as standalone process vs service.

When starting beats as a service, the PID will be owned by the user that manages the service. This would be root in most cases. Users tend to run tests as non-root, running beats directly (./metricbeat) on the command line. Without the shared_credential_file path the beat checks for credentials under the user's home directory. When starting beats as a service, the home directory of the user managing the service (typically root) is different than what was being used in testing and development, which can be hard to figure out.

(cherry picked from commit cbed0d6)
@kaiyan-sheng kaiyan-sheng added [zube]: In Review backport Team:Docs Label for the Observability docs team labels Sep 14, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/obs-docs (Team:Docs)

@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Sep 14, 2020
@kaiyan-sheng kaiyan-sheng self-assigned this Sep 14, 2020
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #21082 opened]

  • Start Time: 2020-09-14T18:54:38.914+0000

  • Duration: 23 min 52 sec

@kaiyan-sheng kaiyan-sheng merged commit eaa52b0 into elastic:7.9 Sep 15, 2020
@kaiyan-sheng kaiyan-sheng deleted the backport_20929_7.9 branch September 15, 2020 13:27
@zube zube bot removed the [zube]: Done label Dec 15, 2020
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
(cherry picked from commit f2c9616)

Co-authored-by: Kostas Botsas <kostas.botsas@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:Docs Label for the Observability docs team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants