Skip to content

Cherry-pick #17960 to 7.x: Use data subfolder as default for process logs#18120

Merged
michalpristas merged 9 commits intoelastic:7.xfrom
michalpristas:backport_17960_7.x
May 4, 2020
Merged

Cherry-pick #17960 to 7.x: Use data subfolder as default for process logs#18120
michalpristas merged 9 commits intoelastic:7.xfrom
michalpristas:backport_17960_7.x

Conversation

@michalpristas
Copy link
Copy Markdown
Contributor

@michalpristas michalpristas commented Apr 30, 2020 •
edited
Loading

Requires: #18169
Cherry-pick of PR #17960 to 7.x branch. Original message:

What does this PR do?

This PR changes default path for process logs from /var/log to {path.data}/logs

this will conflict with paths CLI PR will need to rebase then

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
  • 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
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

@michalpristas
[Elastic-Agent] Use data subfolder as default for process logs (elast…
66f7fb9 
…ic#17960)

[Elastic-Agent] Use data subfolder as default for process logs (elastic#17960)
@michalpristas michalpristas added backport Project:fleet Ingest Management:alpha1 Group issues for ingest management alpha1 labels Apr 30, 2020
@michalpristas michalpristas self-assigned this Apr 30, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 30, 2020

Pinging @elastic/ingest-management (Team:Ingest Management)

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Apr 30, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Apr 30, 2020 •
edited
Loading

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview stats

Expand to view the summary

Build stats

@ph
Copy link
Copy Markdown
Contributor

ph commented Apr 30, 2020

We will need to update this correct?

@andresrc andresrc removed the needs_team Indicates that the issue/PR needs a Team:* label label May 2, 2020
michalpristas and others added 2 commits May 4, 2020 08:48
@michalpristas
Merge branch '7.x' into backport_17960_7.x
d56bcac
@michalpristas @ph
[Elastic-Agent] Use /tmp for default monitoring endpoint location for…
d4202fe 
… libbeat (elastic#18131)

* default tmp socket

* changelog

* Update x-pack/elastic-agent/CHANGELOG.asciidoc

Co-authored-by: Pier-Hugues Pellerin <phpellerin@gmail.com>

Co-authored-by: Pier-Hugues Pellerin <phpellerin@gmail.com>
@michalpristas
Copy link
Copy Markdown
Contributor Author

michalpristas commented May 4, 2020

@ph updated with /tmp additional cherry pick

@michalpristas michalpristas mentioned this pull request May 4, 2020
Merged
6 tasks
@michalpristas michalpristas merged commit 6478f45 into elastic:7.x May 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@ph ph ph approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@michalpristas michalpristas

Labels

backport Ingest Management:alpha1 Group issues for ingest management alpha1 review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@michalpristas @elasticmachine @ph @andresrc