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

Revert "Revert "[filebeat] Fix long filepaths in diagnostics exceeding max path limits on Windows"" #40996

Conversation

ycombinator
Copy link
Contributor

This PR reverts #40980, which was itself a PR reverting #40909, effectively restoring the changes made in #40909.

The reason #40909 was reverted was that we were suspecting the changes in it were causing Elastic Agent CI builds to fail. However, reverting #40909 (via #40980) did not have any effect on the Agent CI failures.

@ycombinator ycombinator requested review from a team as code owners September 25, 2024 20:55
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Sep 25, 2024
@botelastic
Copy link

botelastic bot commented Sep 25, 2024

This pull request doesn't have a Team:<team> label.

Copy link
Contributor

mergify bot commented Sep 25, 2024

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @ycombinator? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit

Copy link
Contributor

mergify bot commented Sep 25, 2024

backport-8.x has been added to help with the transition to the new branch 8.x.
If you don't need it please use backport-skip label and remove the backport-8.x label.

@mergify mergify bot added the backport-8.x Automated backport to the 8.x branch with mergify label Sep 25, 2024
Copy link
Contributor

@efd6 efd6 left a comment

Choose a reason for hiding this comment

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

Please include a reason in the commit message.

@ycombinator ycombinator marked this pull request as draft September 25, 2024 21:13
@ycombinator
Copy link
Contributor Author

We're still testing if the revert worked so this PR might be unnecessary. Moving to draft.

@ycombinator ycombinator marked this pull request as ready for review September 26, 2024 14:25
@ycombinator ycombinator merged commit 138e43c into main Sep 26, 2024
28 of 31 checks passed
@ycombinator ycombinator deleted the revert-40980-revert-40909-fix/filebeat_diagnostics_long_filename branch September 26, 2024 14:26
mergify bot pushed a commit that referenced this pull request Sep 26, 2024
…g max pa…" (#40996)

This reverts commit dbbfb5b.

(cherry picked from commit 138e43c)
ycombinator added a commit that referenced this pull request Sep 27, 2024
…g max pa…" (#40996) (#41010)

This reverts commit dbbfb5b.

(cherry picked from commit 138e43c)

Co-authored-by: Shaunak Kashyap <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8.x Automated backport to the 8.x branch with mergify needs_team Indicates that the issue/PR needs a Team:* label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants