Fix issue where deleting all policies would delete all alerts too#61
Conversation
|
@jaykeshur: Thanks very much for bringing this to our attention. Now that I've reproduced the issue and confirmed that your PR works around the problem, I can say the side-effect you saw of the Alerts getting deleted from the Monitor side is not intended behavior. Therefore I've opened up a bug against the Sysdig Secure backend (SSPROD-1025) to fix the behavior of the underlying API, at which point the original Python client example that hits the In the meantime, I'm going to hold your PR open in case anyone else stumbles onto this. I'm anticipating I'll be able to update this and close without Merge once we've fixed the API behavior. |
|
@jaykeshur I'm about to merge the PR. The build failed, I'm trying to understand why, but I'll probably merge first and take it from there. In the meanwhile, we will also apply the proper fix to the API soon. Your patch will remain useful for some time after (e.g. with an older onprem deployment). Thanks for the help, and sorry for the delay! |
No description provided.