Skip to content

[Imperva Cloud WAF] Add agentless support - #18227

Merged
moxarth-rathod merged 6 commits into
elastic:mainfrom
moxarth-rathod:imperva_cloud_waf-agentless
Jun 3, 2026
Merged

moxarth-rathod merged 6 commits into
elastic:mainfrom
moxarth-rathod:imperva_cloud_waf-agentless

Conversation

@moxarth-rathod

Copy link
Copy Markdown
Contributor

Proposed commit message

imperva_cloud_waf: add agentless deployment

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.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

  • Clone integrations repo.
  • Install the elastic package locally.
  • Start the elastic stack using the elastic package.
  • Move to integrations/packages/imperva_cloud_waf directory.
  • Run the following command to run tests.

elastic-package test -v

Related issues

@moxarth-rathod moxarth-rathod self-assigned this Apr 6, 2026
@moxarth-rathod
moxarth-rathod requested a review from a team as a code owner April 6, 2026 05:48
@moxarth-rathod moxarth-rathod added enhancement New feature or request Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Integration:imperva_cloud_waf Imperva Cloud WAF Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Apr 6, 2026
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@github-actions

github-actions Bot commented Apr 6, 2026 •

Copy link
Copy Markdown
Contributor

Vale Linting Results

Summary: 4 warnings found

⚠️ Warnings (4)
File Line Rule Message
packages/imperva_cloud_waf/_dev/build/docs/README.md 12 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'.
packages/imperva_cloud_waf/_dev/build/docs/README.md 12 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'.
packages/imperva_cloud_waf/docs/README.md 12 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'.
packages/imperva_cloud_waf/docs/README.md 12 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'.

The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Apr 6, 2026

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also implement comments from Andrew - #18141 (comment)

Co-authored-by: Krishna Chaitanya Reddy Burri <krish.reddy91@gmail.com>
@moxarth-rathod
moxarth-rathod requested a review from kcreddy April 14, 2026 05:21
@moxarth-rathod
moxarth-rathod requested a review from kcreddy April 14, 2026 08:46

@kcreddy kcreddy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Waiting for clarification on #18141 (comment). Will approve accordingly, once it is resolved or merged.

@moxarth-rathod
moxarth-rathod requested a review from kcreddy May 6, 2026 08:54
conditions:
kibana:
version: "^8.16.5 || ^9.0.0"
version: "^8.19.16 || ~9.3.5 || ^9.4.1"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we wait until these exist?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, moving this PR to draft

@moxarth-rathod
moxarth-rathod requested a review from efd6 June 1, 2026 10:50
@moxarth-rathod
moxarth-rathod marked this pull request as ready for review June 1, 2026 10:50
@moxarth-rathod
moxarth-rathod requested a review from a team as a code owner June 1, 2026 10:50
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @moxarth-rathod

@moxarth-rathod
moxarth-rathod merged commit b0facda into elastic:main Jun 3, 2026
11 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package imperva_cloud_waf - 1.14.0 containing this change is available at https://epr.elastic.co/package/imperva_cloud_waf/1.14.0/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:imperva_cloud_waf Imperva Cloud WAF Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[imperva_cloud_waf] Add Agentless Deployment Support

5 participants