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

Cannot add same query to pack more than once (results in 5xx) #88

Closed
noahtalerman opened this issue Dec 3, 2020 · 2 comments · Fixed by #96
Closed

Cannot add same query to pack more than once (results in 5xx) #88

noahtalerman opened this issue Dec 3, 2020 · 2 comments · Fixed by #96
Assignees
Labels
bug Something isn't working as documented
Milestone

Comments

@noahtalerman
Copy link
Member

What version of fleet are you using (fleet version --full)?

2.4.0

What operating system are you using?

Docker image: kolide/fleet:2.4.0

What did you do?

  1. create query 'my_awesome_query'
  2. create pack 'my_pack'
  3. (through the UI) add 'my_awesome_query' to pack, with snapshot false and interval 3600
  4. (through the UI) add 'my_awesome_query' to pack again, with snapshot true, and interval 43200

What did you expect to see?

Query added to pack twice with different schedules / intervals.

Alternately: 400 error page explaining that adding a query to a pack more than once is not supported. I don't think a 5xx response is correct.

What did you see instead?

5xx error page

@zwass zwass added the bug Something isn't working as documented label Dec 3, 2020
@zwass zwass added this to the 3.5.0 milestone Dec 3, 2020
@zwass
Copy link
Member

zwass commented Dec 3, 2020

The related issue kolide/fleet#2227 was resolved by kolide/fleet#2327.

cc @guillaumoplatte

@zwass
Copy link
Member

zwass commented Dec 4, 2020

@nyanshak you reported this in kolide/fleet#2202 and it will be resolved with #96.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as documented
Development

Successfully merging a pull request may close this issue.

2 participants