[AI-5281] Barracuda Secure Edge Integration PR #20423

vinodkumar-sacumen · 2025-06-02T12:29:44Z

What does this PR do?

This is a initial release PR of Barracuda SecureEdge Firewall integration including all the required assets.

Motivation

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

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

cswatt

because this PR is just a skeleton and "display_on_public_website": false is set in the manifest.json, I'm approving this on behalf of documentation

Review from cswatt is dismissed. Related teams and files:

documentation
- barracuda_secure_edge/manifest.json

iliakur

other than CI failures this looks ok to me, so approving as a first step.

torosmassa · 2025-06-11T13:41:48Z

because this PR is just a skeleton and "display_on_public_website": false is set in the manifest.json, I'm approving this on behalf of documentation

hey @cswatt looks like the description needs updating (@vinodkumar-sacumen will update), please have a look at the PR from a docs perspective. It does have Assets that we can use docs review/approval on.

Review from iliakur is dismissed. Related teams and files:

agent-integrations
- .github/CODEOWNERS
- .github/workflows/config/labeler.yml
- barracuda_secure_edge/CHANGELOG.md
- barracuda_secure_edge/README.md
- barracuda_secure_edge/assets/configuration/spec.yaml
- barracuda_secure_edge/assets/dashboards/barracuda_secure_edge_overview.json
- barracuda_secure_edge/changelog.d/20423.added
- barracuda_secure_edge/datadog_checks/barracuda_secure_edge/about.py
- barracuda_secure_edge/datadog_checks/barracuda_secure_edge/init.py
- barracuda_secure_edge/datadog_checks/barracuda_secure_edge/data/conf.yaml.example
- barracuda_secure_edge/images/secureEdge1.png
- barracuda_secure_edge/images/secureEdge2.png
- barracuda_secure_edge/images/secure_edge_overview.png
- barracuda_secure_edge/manifest.json
- barracuda_secure_edge/pyproject.toml

shrey4b · 2025-06-17T14:01:54Z

barracuda_secure_edge/README.md

Looks like this one is missing some steps

shrey4b · 2025-06-17T14:04:27Z

barracuda_secure_edge/assets/dashboards/barracuda_secure_edge_overview.json

@@ -0,0 +1,2498 @@
+{
+    "title": "Barracuda Secure Edge",
+    "description": "[[suggested_dashboards]]",


I believe this is placeholder text

shrey4b · 2025-06-17T15:05:31Z

barracuda_secure_edge/assets/dashboards/barracuda_secure_edge_overview.json

+            {
+              "id": 5966799396123124,
+              "definition": {
+                "title": "Network  Event Timeline",


For this title, checking if the double space was intentional

shrey4b · 2025-06-17T15:06:44Z

barracuda_secure_edge/assets/dashboards/barracuda_secure_edge_overview.json

+            {
+              "id": 6000789218138106,
+              "definition": {
+                "title": "Auth  Access Event  Timeline",


Same as before, noticed double spaces

shrey4b · 2025-06-17T15:07:06Z

barracuda_secure_edge/assets/dashboards/barracuda_secure_edge_overview.json

+            {
+              "id": 3172417779844822,
+              "definition": {
+                "title": "Auth Access Event level",


"L" to be capitalized as part of title

shrey4b · 2025-06-17T15:15:01Z

barracuda_secure_edge/manifest.json

+    "configuration": "README.md#Setup",
+    "support": "README.md#Support",
+    "changelog": "CHANGELOG.md",
+    "description": "SecureEdge is a unified SASE platform that includes NGFW, zero trust and secure SD-WAN",


I believe this description section is searchable. As such it should include "Barracuda". I also recommend writing out the acronyms and having the acronym itself follow in parentheses i.e. Secure Access Service Edge (SASE).

Have restricted it due to line length CI check issue.

brett0000FF

Left a few initial suggestions and feedback from Docs. Thanks!

barracuda_secure_edge/README.md

brett0000FF · 2025-06-27T21:47:20Z

barracuda_secure_edge/README.md

+
+1. Administrative access to Barracuda Secure Edge installed on your server.
+2. The Datadog Agent installed and running (on a server or container that can receive syslog messages).
+3. Network Access between the firewall and the Datadog Agent (usually port 514, but may be a custom value).


Suggested change

3. Network Access between the firewall and the Datadog Agent (usually port 514, but may be a custom value).

3. Network access between the firewall and the Datadog Agent (usually port 514, but may be a custom value).

brett0000FF · 2025-06-27T21:47:38Z

barracuda_secure_edge/README.md

+1. Administrative access to Barracuda Secure Edge installed on your server.
+2. The Datadog Agent installed and running (on a server or container that can receive syslog messages).
+3. Network Access between the firewall and the Datadog Agent (usually port 514, but may be a custom value).
+4. Syslog support enabled in the Datadog Agent (with a TCP or UDP  listener configured).


Suggested change

1. Administrative access to Barracuda Secure Edge installed on your server.

2. The Datadog Agent installed and running (on a server or container that can receive syslog messages).

3. Network Access between the firewall and the Datadog Agent (usually port 514, but may be a custom value).

4. Syslog support enabled in the Datadog Agent (with a TCP or UDP listener configured).

- Administrative access to Barracuda Secure Edge installed on your server.

- The Datadog Agent installed and running (on a server or container that can receive syslog messages).

- Network Access between the firewall and the Datadog Agent (usually port 514, but may be a custom value).

- Syslog support enabled in the Datadog Agent (with a TCP or UDP listener configured).

brett0000FF · 2025-06-27T21:47:53Z

barracuda_secure_edge/README.md

+3. Network Access between the firewall and the Datadog Agent (usually port 514, but may be a custom value).
+4. Syslog support enabled in the Datadog Agent (with a TCP or UDP  listener configured).
+
+### Setup Configurations


Suggested change

### Setup Configurations

### Setup configuration

brett0000FF · 2025-06-27T21:49:30Z

barracuda_secure_edge/README.md

+### Metrics
+
+Barracuda_Secure_Edge does not include any metrics.


Suggested change

### Metrics

Barracuda_Secure_Edge does not include any metrics.

Duplicated below.

vinodkumar-sacumen requested review from a team as code owners June 2, 2025 12:29

temporal-github-worker-1 bot added agent/review-requested docs/review-requested ecosystems/review-requested product/review-requested labels Jun 2, 2025

datadog-agent-integrations-bot bot added documentation dev/testing dev/tooling release qa/skip-qa Automatically skip this PR for the next QA team/agent-integrations team/documentation team/logs-backend team/logs-integrations-reviewers team/saas-integrations labels Jun 2, 2025

cswatt previously approved these changes Jun 2, 2025

View reviewed changes

temporal-github-worker-1 bot added docs/approved and removed docs/review-requested labels Jun 2, 2025

temporal-github-worker-1 bot added docs/review-requested and removed docs/approved labels Jun 3, 2025

iliakur previously approved these changes Jun 4, 2025

View reviewed changes

temporal-github-worker-1 bot added the agent/approved label Jun 4, 2025

temporal-github-worker-1 bot added ecosystems/review-requested and removed reveng/approved labels Jun 11, 2025

vinodkumar-sacumen force-pushed the secure_edge branch from ab6d522 to b00a197 Compare June 17, 2025 13:27

temporal-github-worker-1 bot added agent/review-requested and removed agent/approved labels Jun 17, 2025

vinodkumar-sacumen force-pushed the secure_edge branch from 92e4d60 to 48c4f43 Compare June 17, 2025 15:10

shrey4b requested changes Jun 17, 2025

View reviewed changes

vinodkumar-sacumen force-pushed the secure_edge branch from 48c4f43 to 00284d1 Compare June 20, 2025 08:18

brett0000FF reviewed Jun 23, 2025

View reviewed changes

vinodkumar-sacumen added 14 commits June 25, 2025 18:57

Barracuda Secure Edge Integration PR

1341822

description line lenght

7fc3592

updated metric id in logs files

24d9806

added conf.yml using -s and removed metadata

c9dd21c

resolved CI checks

fe10639

updated images and tests

6a656f4

udpated changelog.d file name

a8cd07c

updated images

8cb08d5

updated images

ac808d4

Updated tests

bbfdfbd

updated path of facets

4279b77

updated and added all test cases

b590fe3

updated readme

856bc73

reverted line lenght

348069e

vinodkumar-sacumen force-pushed the secure_edge branch from b565132 to 348069e Compare June 25, 2025 13:27

updated README

ee7027f

brett0000FF approved these changes Jun 27, 2025

View reviewed changes

temporal-github-worker-1 bot added docs/approved and removed docs/review-requested labels Jun 27, 2025

	3. Network Access between the firewall and the Datadog Agent (usually port 514, but may be a custom value).
	3. Network access between the firewall and the Datadog Agent (usually port 514, but may be a custom value).

		### Metrics

		Barracuda_Secure_Edge does not include any metrics.

[AI-5281] Barracuda Secure Edge Integration PR #20423

Are you sure you want to change the base?

[AI-5281] Barracuda Secure Edge Integration PR #20423

Uh oh!

Conversation

vinodkumar-sacumen commented Jun 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What does this PR do?

Motivation

Review checklist (to be filled by reviewers)

Uh oh!

cswatt left a comment

Choose a reason for hiding this comment

Uh oh!

iliakur left a comment

Choose a reason for hiding this comment

Uh oh!

torosmassa commented Jun 11, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

brett0000FF left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

vinodkumar-sacumen commented Jun 2, 2025 •

edited

Loading