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

[8.12](backport #3948) [Upgrade Details] Fix corruption when writing upgrade marker file #3951

Merged
merged 1 commit into from
Dec 22, 2023

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Dec 22, 2023

This is an automatic backport of pull request #3948 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

)

* Truncate marker file before writing

* Add unit test for truncation

(cherry picked from commit 8f543bb)
@mergify mergify bot requested a review from a team as a code owner December 22, 2023 09:17
@mergify mergify bot added the backport label Dec 22, 2023
@mergify mergify bot requested review from AndersonQ and pchila and removed request for a team December 22, 2023 09:17
@pierrehilbert pierrehilbert added the Team:Elastic-Agent Label for the Agent team label Dec 22, 2023
@elasticmachine
Copy link
Contributor

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

Copy link

@pierrehilbert
Copy link
Contributor

As mentioned #3947 (comment), failures on integration test are expected so I'm force merging this one as it will solve the original integration test failure.

@pierrehilbert pierrehilbert merged commit 29fe96e into 8.12 Dec 22, 2023
9 checks passed
@pierrehilbert pierrehilbert deleted the mergify/bp/8.12/pr-3948 branch December 22, 2023 11:30
cmacknz pushed a commit that referenced this pull request Jan 17, 2024
) (#3951)

* Truncate marker file before writing

* Add unit test for truncation

(cherry picked from commit 8f543bb)

Co-authored-by: Shaunak Kashyap <ycombinator@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:Elastic-Agent Label for the Agent team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants