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

Automox XSOAR Content Pack #18404

Conversation

ax-jmiller
Copy link
Contributor

@ax-jmiller ax-jmiller commented Apr 1, 2022

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: link to the issue

Description

This pull request introduces the Automox Content Pack, including an integration and playbook for interfacing with the Automox platform through XSOAR.

Screenshots

Paste here any images that will help the reviewer

Minimum version of Cortex XSOAR

  • 6.0.0
  • 6.1.0
  • 6.2.0
  • 6.5.0

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

Must have

  • Tests
  • Documentation

* IN-104 Implemented commands related to Administration and Vulnerability Sync use cases.
Added playbook to automate report uploads
Added documentation for Playbook, Integration, and Pack
* Adding unit tests

* Reviewed checklist for XSOAR submission
@CLAassistant
Copy link

CLAassistant commented Apr 1, 2022

CLA assistant check
All committers have signed the CLA.

@content-bot content-bot added the Contribution Thank you! Contributions are always welcome! label Apr 1, 2022
@content-bot content-bot changed the base branch from master to contrib/ax-jmiller_automox-xsoar-pack April 1, 2022 15:20
@content-bot content-bot requested a review from yaakovi April 1, 2022 15:20
@content-bot
Copy link
Collaborator

Thank you for your contribution. Your generosity and caring are unrivaled! Make sure to register your contribution by filling the Contribution Registration form, so our content wizard @yaakovi will know he can start review the proposed changes.

@content-bot
Copy link
Collaborator

The CircleCI check from your latest pushed commit was unsuccessful. @ax-jmiller take a look at the build by clicking this link.


Failed Build Steps

  • Run Unit Testing and Lint

Try and address the listed CircleCI build step failures at your earliest convenience. This will greatly expedite the process of getting your proposed changes merged into master. Happy coding and may the force be with you.

@ax-jmiller ax-jmiller marked this pull request as ready for review April 4, 2022 15:05
@yaakovi yaakovi requested review from MLainer1 and removed request for yaakovi April 5, 2022 10:40
@yaakovi yaakovi assigned MLainer1 and unassigned yaakovi Apr 5, 2022
@content-bot content-bot added Contribution Form Filled Whether contribution form filled or not. Partner labels Apr 5, 2022
Copy link
Contributor

@MLainer1 MLainer1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! looks great :)
Make sure to go over the comments that I made and request another review when you need it

Packs/Automox/Integrations/Automox/Automox.py Show resolved Hide resolved
Packs/Automox/Integrations/Automox/Automox.py Show resolved Hide resolved
Packs/Automox/Integrations/Automox/Automox.py Outdated Show resolved Hide resolved
Packs/Automox/Integrations/Automox/Automox.py Show resolved Hide resolved
Packs/Automox/Integrations/Automox/Automox.py Show resolved Hide resolved
Packs/Automox/Integrations/Automox/Automox.yml Outdated Show resolved Hide resolved
Packs/Automox/Integrations/Automox/Automox.yml Outdated Show resolved Hide resolved
…hon. Fixed issues with fromversion and tests in Integration yaml.
@content-bot
Copy link
Collaborator

The CircleCI build failed again. @ax-jmiller take a look at the build details here - and try and fix the issues so that we can merge your proposed changes as soon as possible.


Failed Build Steps

  • Check if CircleCI's config file and requirements.txt files are up to date

@ax-jmiller
Copy link
Contributor Author

When I fetched upstream changes for my fork from github, it merged all the commits into the history for this branch, which is kind of messy. What would you recommend here?

@ax-jmiller ax-jmiller requested a review from MLainer1 April 6, 2022 19:56
@MLainer1 MLainer1 requested a review from tomer-pan April 11, 2022 08:57
@MLainer1 MLainer1 added the pending-demo Demo pending label Apr 13, 2022
@MLainer1 MLainer1 added post-demo and removed pending-demo Demo pending labels Apr 14, 2022
Copy link
Contributor

@MLainer1 MLainer1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your time in the demo. Make sure to address the notes that we raised regarding the integration. Feel free to send me a message if you have any questions, and request a review when you are done :)

Copy link
Contributor

@MLainer1 MLainer1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work :)
Make sure to look at the failed validation in the pr, it looks like a quick fix, and make sure to merge from master

Packs/Automox/Integrations/Automox/Automox.yml Outdated Show resolved Hide resolved
@content-bot
Copy link
Collaborator

The CircleCI build failed again. @ax-jmiller take a look at the build details here - and try and fix the issues so that we can merge your proposed changes as soon as possible.


Failed Build Steps

  • Check if CircleCI's config file and requirements.txt files are up to date
  • Validate Files and Yaml

Co-authored-by: MLainer1 <93524335+MLainer1@users.noreply.github.com>
@content-bot
Copy link
Collaborator

The CircleCI build failed again. @ax-jmiller take a look at the build details here - and try and fix the issues so that we can merge your proposed changes as soon as possible.


Failed Build Steps

  • Validate Files and Yaml
  • Check if CircleCI's config file and requirements.txt files are up to date

@content-bot
Copy link
Collaborator

The CircleCI build failed again. @ax-jmiller take a look at the build details here - and try and fix the issues so that we can merge your proposed changes as soon as possible.


Failed Build Steps

  • Check if CircleCI's config file and requirements.txt files are up to date

@ax-jmiller ax-jmiller requested a review from MLainer1 April 21, 2022 07:45
@MLainer1 MLainer1 added the ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. label Apr 24, 2022
@content-bot
Copy link
Collaborator

content-bot commented Apr 24, 2022

Successfully created a pipeline in Gitlab with url: https://code.pan.run/xsoar/content/-/pipelines/2810014

@MLainer1 MLainer1 merged commit e3ba1e9 into demisto:contrib/ax-jmiller_automox-xsoar-pack Apr 24, 2022
@content-bot content-bot mentioned this pull request Apr 24, 2022
11 tasks
MLainer1 added a commit that referenced this pull request Apr 25, 2022
* Automox Content Pack for XSOAR (#1)

* IN-104 Implemented commands related to Administration and Vulnerability Sync use cases.

* [IN-149] Adding Vulnerability Upload Playbook (#2)

Added playbook to automate report uploads

* [IN-152] Automox XSOAR Documentation (#3)

Added documentation for Playbook, Integration, and Pack

* IN-166 Automox XSOAR Unit Tests (#4)

* Adding unit tests

* Reviewed checklist for XSOAR submission

* Cleaning up integration test files

* Fixed validation issues with playbook

* Updated README and Integration yaml to be consistent with args in Python. Fixed issues with fromversion and tests in Integration yaml.

* Playbook documentation updates regarding supported csv formats

* Present errors in a pleasant and readable way. Handled basic http fail codes

* Updated delete commands to update objects in the context with deleted key

* Removed test playbook from integration yaml

Co-authored-by: MLainer1 <93524335+MLainer1@users.noreply.github.com>

* Added Automox integration instrumentation

* Fixed docker image version and schema validation in integration yaml

Co-authored-by: MLainer1 <93524335+MLainer1@users.noreply.github.com>

Co-authored-by: Jack Miller <94870866+ax-jmiller@users.noreply.github.com>
Co-authored-by: MLainer1 <93524335+MLainer1@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribution Form Filled Whether contribution form filled or not. Contribution Thank you! Contributions are always welcome! docs-approved Partner post-demo ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants