-
Notifications
You must be signed in to change notification settings - Fork 55
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
[Carbon Black Cloud] Observability Window Restraint #2877
Conversation
1fd4ff8
to
74998eb
Compare
f5bbbce
to
8ec8abf
Compare
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Show resolved
Hide resolved
499a823
to
5802205
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be worth a quick call, but not sure I'm quite following everything here. Tried my best to add comments on the bits for clarification - might need inline comments if the logic is correct for the next person coming behind to understand.
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Outdated
Show resolved
Hide resolved
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Outdated
Show resolved
Hide resolved
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Outdated
Show resolved
Hide resolved
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Outdated
Show resolved
Hide resolved
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Show resolved
Hide resolved
f57ca6c
to
f192f17
Compare
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Outdated
Show resolved
Hide resolved
0e9465f
to
f192f17
Compare
f192f17
to
f5e1dbe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding a couple of small comments we should check and probably would catch during testing, but worth checking now
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Show resolved
Hide resolved
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Show resolved
Hide resolved
plugins/carbon_black_cloud/icon_carbon_black_cloud/tasks/monitor_alerts/task.py
Show resolved
Hide resolved
e48dac5
to
8e1ef7a
Compare
* Restrain observability Window * feedback
* Restrain observability Window * feedback
* Restrain observability Window * feedback
* Restrain observability Window * feedback
* Restrain observability Window * feedback
* Restrain observability Window * feedback
* Restrain observability Window * feedback
Proposed Changes
Description
Restrains the Observability window to a customizable window
PR Requirements
Developers, verify you have completed the following items by checking them off:
Testing
Unit Tests
Review our documentation on generating and writing plugin unit tests
In-Product Tests
If you are an InsightConnect customer or have access to an InsightConnect instance, the following in-product tests should be done:
Style
Review the style guide
USER nobody
in theDockerfile
when possiblerapid7/insightconnect-python-3-38-slim-plugin:{sdk-version-num}
andrapid7/insightconnect-python-3-38-plugin:{sdk-version-num}
insight-plugin validate
which callsicon_validate
to linthelp.md
Functional Checklist
tests/
directory created withinsight-plugin samples
tests/$action_bad.json
insight-plugin run -T tests/example.json --debug --jq
insight-plugin run -T all --debug --jq
(use PR format at end)insight-plugin run -R tests/example.json --debug --jq
insight-plugin run --debug --jq
(use PR format at end)Assessment
You must validate your work to reviewers:
insight-plugin validate
and make sure everything passesinsight-plugin run -A
. For single action validation:insight-plugin run tests/{file}.json -A
insight-plugin ... | pbcopy
) and paste the output in a new post on this PR