-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Fleet] Duplicate error coming when adding system integration with new agent policy #125968
Comments
Pinging @elastic/fleet (Team:Fleet) |
@jen-huang what do you think about the severity of this bug? I'm not sure how frequently users are adding system integration specifically (with the default name). I'm thinking low or medium. I can fix this next week with a small effort I think. |
@crowens Thank a lot for raising that! @juliaElastic The initial report is a bit of an edge case, but the one reported by Chris is far more concerning to me, could you look into that? |
@crowens @jen-huang the update issue reported by Chris is not related to this issue. The former happens in 8.0 and caused by having two integration policies named system-1 I think. The latter happens only in 8.1. |
Hi @juliaElastic
Build details: Screen Recording: System.-.Integrations.-.Elastic.-.Google.Chrome.2022-03-03.12-21-36.mp4Hence marking this as QA:Validated. Thanks |
Kibana version: 8.1
Describe the bug:
Bug introduced by #108456
When trying to add system integration with default incremental name (e.g. system-3 if there are 2 of them already existing) and new agent policy, the result is an error message saying that system-3 already exists.
This is because the new agent policy comes with system integration added by default, which uses the same incremental name as the default name on the create integration page.
Steps to reproduce:
Expected behavior:
The error message shouldn't come and the Agent policy should be created with one system integration policy.
I think the right thing to do here is to have a check in the logic, not to create the agent policy with system monitoring, in case it is added for system integration.
Screenshots (if relevant):
The text was updated successfully, but these errors were encountered: