[Fleet] Output API endpoints #118589
[Fleet] Output API endpoints #118589
Conversation
nchaulet
added
v8.0.0
release_note:skip
|
Pinging @elastic/fleet (Team:Fleet) |
nchaulet
force-pushed
the
feature-outputs-api-endpoints
branch
from
85ffbc2 to
7d4f0f1
Compare
nchaulet
force-pushed
the
feature-outputs-api-endpoints
branch
from
7d4f0f1 to
ad02252
Compare
| @@ -226,6 +226,11 @@ class OutputService { | |||
| `Preconfigured output ${id} cannot be deleted outside of kibana config file.` | |||
| ); | |||
| } | |||
|
|
|||
| if (originalOutput.is_default && !fromPreconfiguration) { | |||
There was a problem hiding this comment.
is_default_monitoring check is not needed here?
There was a problem hiding this comment.
Yes good catch we should probably not allow to delete the default monitoring too 👍
| .expect(200); | ||
|
|
||
| expect(deleteResponse.id).to.eql(outputId); | ||
| }); |
There was a problem hiding this comment.
I don't see tests for attempting to delete default outputs
|
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Metrics [docs]Public APIs missing comments
Page load bundle
History
To update your PR or re-run it, just comment with: cc @nchaulet |
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
Co-authored-by: Nicolas Chaulet <[email protected]>
Summary
Related to #117317
This PR create the missing Output endpoints needed for the multiple output UI:
POST /outputsto create a new output.DELETE /outputs/{id}to delete an existing output. (You cannot delete the default output).I also added some integration test to cover these endpoints, and updated the openAPI spec.