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.

Sign up for GitHub

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

Jump to bottom

Fix MSI breaking change release note to apply to all Beats #38076

Merged
merged 2 commits into from
Feb 20, 2024
Merged

Fix MSI breaking change release note to apply to all Beats #38076

merged 2 commits into from
Feb 20, 2024

Conversation

cmacknz
Copy link
Member

@cmacknz cmacknz commented Feb 20, 2024

Fix per #37915 (comment), I expected the first section to be the "Affecting All Beats" section but missed that it was missing.

@cmacknz cmacknz self-assigned this Feb 20, 2024
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Feb 20, 2024
@cmacknz cmacknz added forwardport-main Automated forwardport with mergify and removed needs_team Indicates that the issue/PR needs a Team:* label labels Feb 20, 2024
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Feb 20, 2024
@cmacknz cmacknz mentioned this pull request Feb 20, 2024
Merged
@elasticmachine
Copy link
Collaborator

💔 Build Failed

Failed CI Steps

cc @cmacknz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @cmacknz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @cmacknz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @cmacknz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @cmacknz

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

cc @cmacknz

@elasticmachine
Copy link
Collaborator

💚 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: 2024-02-20T20:51:33.154+0000

  • Duration: 9 min 36 sec

❕ Flaky test report

No test was executed to be analysed.

🤖 GitHub comments

Expand to view the 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!)

@cmacknz cmacknz merged commit 7a75693 into elastic:8.12 Feb 20, 2024
12 checks passed
mergify bot pushed a commit that referenced this pull request Feb 20, 2024
@cmacknz @mergify
Fix MSI breaking change release note to apply to all Beats (#38076)
d5bf0e9 
* Add MSI installer change to 8.12 release notes.

* Fix location of MSI breaking change notice.

(cherry picked from commit 7a75693)
@mergify mergify bot mentioned this pull request Feb 20, 2024
Merged
cmacknz added a commit that referenced this pull request Feb 20, 2024
@mergify @cmacknz
Fix MSI breaking change release note to apply to all Beats (#38076) (#…
cdea924 
…38077)

* Add MSI installer change to 8.12 release notes.

* Fix location of MSI breaking change notice.

(cherry picked from commit 7a75693)

Co-authored-by: Craig MacKenzie <craig.mackenzie@elastic.co>
@cmacknz cmacknz deleted the fix-msi-release-note branch February 20, 2024 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@strawgate strawgate Awaiting requested review from strawgate

@kilfoyle kilfoyle Awaiting requested review from kilfoyle

Assignees

@cmacknz cmacknz

Labels
forwardport-main Automated forwardport with mergify needs_team Indicates that the issue/PR needs a Team:* label
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cmacknz @elasticmachine