Skip to content

[7.17](backport elastic/elastic-agent#570) Add filemod times to contents of diagnostics collect command#31986

Merged
ph merged 2 commits intoelastic:7.17from
michel-laterman:backport-agent-570
Jun 22, 2022
Merged

[7.17](backport elastic/elastic-agent#570) Add filemod times to contents of diagnostics collect command#31986
ph merged 2 commits intoelastic:7.17from
michel-laterman:backport-agent-570

Conversation

@michel-laterman
Copy link
Copy Markdown
Contributor

Backport elastic/elastic-agent#570

  • Add filemod times to contents of diagnostics collect command

Add filemod times to the files and directories in the zip archive.
Log files (and sub dirs) will use the modtime returned by the fileinfo
for the source. Others will use the timestamp from when the zip is
created.

  • Fix linter

…diagnostics collect command

* Add filemod times to contents of diagnostics collect command

Add filemod times to the files and directories in the zip archive.
Log files (and sub dirs) will use the modtime returned by the fileinfo
for the source. Others will use the timestamp from when the zip is
created.

* Fix linter
@michel-laterman michel-laterman added backport Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team labels Jun 17, 2022
@michel-laterman michel-laterman requested a review from ph June 17, 2022 18:11
@michel-laterman michel-laterman requested a review from a team as a code owner June 17, 2022 18:11
@michel-laterman michel-laterman requested review from AndersonQ and removed request for a team June 17, 2022 18:11
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

@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 Jun 17, 2022
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jun 17, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-06-21T17:56:30.526+0000

  • Duration: 83 min 55 sec

Test stats 🧪

Test Results
Failed 0
Passed 1769
Skipped 6
Total 1775

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

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

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

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

Copy link
Copy Markdown
Contributor

@ph ph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change lgtm.

I will rekick the test, looking at the error at packaging this seems to be an ansible playbook issues.

@ph
Copy link
Copy Markdown
Contributor

ph commented Jun 20, 2022

test this please

@michel-laterman
Copy link
Copy Markdown
Contributor Author

/package

@michel-laterman
Copy link
Copy Markdown
Contributor Author

/test

1 similar comment
@michel-laterman
Copy link
Copy Markdown
Contributor Author

/test

@ph ph merged commit 3a04607 into elastic:7.17 Jun 22, 2022
@michel-laterman michel-laterman deleted the backport-agent-570 branch April 12, 2023 19:18
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…nts of diagnostics collect command (elastic#31986)

* [7.17](backport #elastic-agent/570) Add filemod times to contents of diagnostics collect command

* Add filemod times to contents of diagnostics collect command

Add filemod times to the files and directories in the zip archive.
Log files (and sub dirs) will use the modtime returned by the fileinfo
for the source. Others will use the timestamp from when the zip is
created.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants