-
Notifications
You must be signed in to change notification settings - Fork 81
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
[Release] 8.9.0 changelog #2806
Conversation
checksum: 159350f6e8e2e60912b5ca57c2d0ee3fe10b1026 | ||
- kind: breaking-change | ||
summary: API default error is now 500 | ||
description: Up until now when any not expected error was encountered it resulted int Bad Request response.\n This is changing and any not expected error will return Internal Server Error while keeping most of the current behavior\n unchanged and returning Bad Request on expected failure paths (e.g. Agent Inactive, Missing Agent ID, Missing Auth Header). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replaced newlines with \n as the changelog tool complained, is there a way to check the rendered version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@juliaElastic The output here looks to be yaml format rather than asciidoc. It may not matter though, since I just copy the contents into a separate docs PR. Here's what I have for this breaking change:
Asciidoc:
[discrete]
[[breaking-2531]]
.API default error code is now 500
[%collapsible]
====
*Details* +
Previously, when {fleet-server} encountered an unexpected error it resulted in a `Bad Request` response.
*Impact* +
Now, any unexpected error returns an `Internal Server Error` response while keeping most of the current behavior
unchanged. On expected failure paths (for example, Agent Inactive, Missing Agent ID, Missing Auth Header) a `Bad Request` response is returned. For more information, refer to {fleet-server-pull}2531[#2531].
====
The build failed with this message. Hey @kroustou could you help fixing this?
|
This step should have been removed, I am on PTO, I can point to the PR that removes it next week |
Hello again #2771 should have removed this step |
Thanks! We should backport it to 8.9 branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a small comment but overall LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/test |
@kroustou The build step |
No idea, maybe you need some help from the @elastic/ci-systems team? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for the changelog text! 👍
Created a jira ticket for ci-systems: https://elasticco.atlassian.net/browse/ENGPRD-217 |
@juliaElastic You can close the ticket, this is due to the fleet server migration to buildkite. Jenkins resources has been removed but the check is still mandatory on the 8.9 branch as it was only removed on main. |
* 8.9.0 changelog * asciidoc * Backport pr 2771 * added quotes around apm.Tracer --------- Co-authored-by: Stavros Kroustouris <[email protected]> (cherry picked from commit 71625b6) # Conflicts: # .buildkite/pipeline.yml
* 8.9.0 changelog * asciidoc * Backport pr 2771 * added quotes around apm.Tracer --------- Co-authored-by: Stavros Kroustouris <[email protected]> (cherry picked from commit 71625b6) # Conflicts: # .buildkite/pipeline.yml Co-authored-by: Julia Bardi <[email protected]>
What is the problem this PR solves?
8.9.0 changelog
How does this PR solve the problem?
// Explain HOW you solved the problem in your code. It is possible that during PR reviews this changes and then this section should be updated.
How to test this PR locally
Design Checklist
Checklist
./changelog/fragments
using the changelog toolRelated issues