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

[UbsUx][APM] Delete "Agent Configuration" API failing on localhost #198019

Closed
miloszmarcinkowski opened this issue Oct 28, 2024 · 8 comments
Closed
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team

Comments

@miloszmarcinkowski
Copy link
Contributor

The API fails when trying to delete Agent Configuration in APM settings.

Failing API: DELETE /api/apm/settings/agent-configuration 2023-10-31

The currently running edge-oblt instance (commit: 10ec2041) doesn't have that issue. However, it doesn't help when reverting to 10ec2041 on localhost.

It looks like body parameter schema is causing bad request, but we don't know why does it work on cloud instance.

AC:

  • investigate why API is failing on localhost and provide a fix

400 bad request examples:
Image
Image

Edge-oblt working example:
Image

Working example when removed schema validation for body parameter:
Image

@miloszmarcinkowski miloszmarcinkowski added the technical debt Improvement of the software architecture and operational architecture label Oct 28, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Oct 28, 2024
@miloszmarcinkowski miloszmarcinkowski added Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team and removed needs-team Issues missing a team label labels Oct 28, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@miloszmarcinkowski miloszmarcinkowski added bug Fixes for quality problems that affect the customer experience and removed technical debt Improvement of the software architecture and operational architecture labels Oct 28, 2024
@bryce-b bryce-b self-assigned this Nov 11, 2024
@bryce-b
Copy link
Contributor

bryce-b commented Nov 11, 2024

Oddly enough, this feature still works on edge-lite (f51481d) but not locally. I'll continue trying to figure out why this is.

It seems edge and edge-lite are currently running older revisions than this ticket initially tested on? 10ec204 is Oct 21st, and edge/edge-lite are running a Sept 7th revision for kibana.

@miloszmarcinkowski
Copy link
Contributor Author

miloszmarcinkowski commented Nov 12, 2024

Oddly enough, this feature still works on edge-lite (f51481d) but not locally. I'll continue trying to figure out why this is.

It seems edge and edge-lite are currently running older revisions than this ticket initially tested on? 10ec204 is Oct 21st, and edge/edge-lite are running a Sept 7th revision for kibana.

@bryce-b Thank you for picking it up. I was checking https://edge-oblt.kb.us-west2.gcp.elastic-cloud.com edge-oblt instance, which runs d70e01aa (Oct 28th) revision. I check commit hash here https://edge-oblt.kb.us-west2.gcp.elastic-cloud.com/status. Which edge-olbt instance did you try?

@bryce-b
Copy link
Contributor

bryce-b commented Nov 12, 2024

I was using the same instance, but I clicked on the "version-info" link at the top to get the hash. Image which goes to this page : https://studious-disco-k66oojq.pages.github.io/edge-oblt/

I tried running a couple of different versions of kibana & ES from these earlier version to current, and the issue reproduces on all of them. Very odd to see this reproduce locally, but not on the cloud instance. I'm going to try spinning up some cloud instances to see if it reproduces there.

@bryce-b
Copy link
Contributor

bryce-b commented Nov 12, 2024

8.16 on cloud works fine.

@bryce-b
Copy link
Contributor

bryce-b commented Nov 15, 2024

I opted to move the parameters into the query, hopefully that is acceptable.
#200281

@bryce-b
Copy link
Contributor

bryce-b commented Nov 19, 2024

root cause: #200722

@cauemarcondes
Copy link
Contributor

We can close this one now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants