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.

Sign up for GitHub

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

Jump to bottom

[zscaler_zia] Updated user-guide of the package #2773

Merged
merged 4 commits into from
Mar 12, 2022
Merged

[zscaler_zia] Updated user-guide of the package #2773

merged 4 commits into from
Mar 12, 2022

Conversation

vinit-chauhan
Copy link
Contributor

@vinit-chauhan vinit-chauhan commented Mar 7, 2022 •
edited
Loading

What does this PR do?

  • Added detailed configuration steps for Zscale ZIA along with screenshots.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

How to test this PR locally

  • Clone integrations repo.
  • Install elastic-package locally.
  • Start elastic stack using elastic-package.
  • Move to integrations/packages/zscaler_zia directory.

Run the following command to run tests.
elastic-package test

Related issues

  • N/A

Screenshots

image
image
image
image

@vinit-chauhan vinit-chauhan requested a review from a team as a code owner March 7, 2022 14:13
@elasticmachine
Copy link

elasticmachine commented Mar 7, 2022 •
edited
Loading

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview [preview](https://ci-stats.elastic.co/app/apm/services/beats-ci/transactions/view?rangeFrom=2022-03-10T12:57:23.665Z&rangeTo=2022-03-10T13:17:23.665Z&transactionName=BUILD Ingest-manager/integrations/PR-{number}&transactionType=job&latencyAggregationType=avg&traceId=bc2a216d95ad7bbebf44121e2e34c49b&transactionId=e6507781e6b8f09b)

Expand to view the summary

Build stats

  • Start Time: 2022-03-10T13:07:23.665+0000

  • Duration: 16 min 0 sec

Test stats 🧪

Test Results
Failed 0
Passed 60
Skipped 0
Total 60

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@elasticmachine
Copy link

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)

@jamiehynds jamiehynds added documentation Improvements or additions to documentation Integration:zscaler_zia Zscaler Internet Access labels Mar 7, 2022
andrewkroh
andrewkroh reviewed Mar 7, 2022
View reviewed changes
Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

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

Please attach a screenshot to the PR description showing the rendering of the setup steps in Kibana.

@andrewkroh
Copy link
Member

/test

@andrewkroh
Copy link
Member

/test

@vinit-chauhan
- Updated the steps for ZIA configuration.
c7cb0fb 
- Removed unnecessary spec file.
@andrewkroh
Copy link
Member

/test

@andrewkroh andrewkroh merged commit f42aeea into elastic:main Mar 12, 2022
@andrewkroh andrewkroh mentioned this pull request Mar 14, 2022
Merged
eyalkraft pushed a commit to build-security/integrations that referenced this pull request Mar 30, 2022
@vinit-chauhan
[zscaler_zia] Updated user-guide of the package (elastic#2773)
5387662 
* Add detailed steps to setup zscaler zia.

* Updated the changelog entry.

* updated Indentation of the spec file.

* Updated the steps for ZIA configuration.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrewkroh andrewkroh andrewkroh approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation Integration:zscaler_zia Zscaler Internet Access
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@vinit-chauhan @elasticmachine @andrewkroh @jamiehynds