[SAASINT-3634] DDS: Symantec VIP: Crawler Integration Without Assets

[surabhipatel-crest]

What does this PR do?

PR includes skeleton of Symantec VIP integration without any assets like dashboards, image, README content, etc.

Motivation

This is beta release of crawler integration and is intended for internal testing before going live. We will raise separate PR with assets and all necessary information once this PR has been merged.

Additional Notes

Only crawler-based integration related changes are included in this PR.

Review checklist (to be filled by reviewers)

• Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
• Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
• If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.19%. Comparing base (724567b) to head (76fad6f).
Report is 140 commits behind head on master.

Additional details and impacted files

Flag	Coverage Δ
activemq	?
cassandra	?
hive	?
hivemq	?
hudi	?
ignite	?
jboss_wildfly	?
kafka	?
presto	?
solr	?

Flags with carried forward coverage won't be shown. Click here to find out more.

[gunterd]

Can you please add the following to CODEOWNERS?

/symantec_vip/                                                        @DataDog/saas-integrations
/symantec_vip/*.md                                                    @DataDog/saas-integrations @DataDog/documentation
/symantec_vip/manifest.json                                           @DataDog/saas-integrations @DataDog/documentation
/symantec_vip/assets/logs/                                            @DataDog/saas-integrations @DataDog/documentation @DataDog/logs-backend

[shubhamvekariya-crest]

Can you please add the following to CODEOWNERS?

/symantec_vip/                                                        @DataDog/saas-integrations
/symantec_vip/*.md                                                    @DataDog/saas-integrations @DataDog/documentation
/symantec_vip/manifest.json                                           @DataDog/saas-integrations @DataDog/documentation
/symantec_vip/assets/logs/                                            @DataDog/saas-integrations @DataDog/documentation @DataDog/logs-backend

Hi @gunterd, We have seen duplication in CODEOWNERS file once both the PRs(assets and no-assets) are merged. To avoid this duplication, we decided to keep CODEOWNERS file changes only in assets PR. We can also consider an alternative approach if needed. Please provide your thoughts on this.

[gunterd]

I don't see the codeowners changes in the assets PR: #19414

Review from HadhemiDD is dismissed.
Related teams and files:

• agent-integrations
  • .github/CODEOWNERS

[shubhamvekariya-crest]

We can also consider an alternative approach if needed. Please provide your thoughts on this

Hi @gunterd, Apologies. We have added it to the CODEOWNERS file in this PR. We will define an approach for whether to include it in a no-asset PR or an asset PR. Going forward, we will maintain consistency with the chosen approach.