8000 [AI-5281] Barracuda Secure Edge Integration PR by vinodkumar-sacumen · Pull Request #20423 · DataDog/integrations-core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

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

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

vinodkumar-sacumen
Copy link
@vinodkumar-sacumen vinodkumar-sacumen commented Jun 2, 2025

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
cswatt previously approved these changes Jun 2, 2025
Copy link
Contributor
@cswatt cswatt left a comment

Choose a reason for hiding this comment

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

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

@temporal-github-worker-1 temporal-github-worker-1 bot dismissed cswatt’s stale review June 3, 2025 06:35

Review from cswatt is dismissed. Related teams and files:

  • documentation
    • barracuda_secure_edge/manifest.json
iliakur
iliakur previously approved these changes Jun 4, 2025
Copy link
Contributor
@iliakur iliakur left a comment

Choose a reason for hiding this comment

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

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

@torosmassa
Copy link

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.

@temporal-github-worker-1 temporal-github-worker-1 bot dismissed iliakur’s stale review June 17, 2025 13:27

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
Copy link

Choose a reason for hiding this comment

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

Looks like this one is missing some steps

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

Choose a reason for hiding this comment

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

I believe this is placeholder text

{
"id": 5966799396123124,
"definition": {
"title": "Network Event Timeline",
Copy link

Choose a reason for hiding this comment

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

For this title, checking if the double space was intentional

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

Choose a reason for hiding this comment

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

Same as before, noticed double spaces

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

Choose a reason for hiding this comment

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

"L" to be capitalized as part of title

"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",
Copy link

Choose a reason for hiding this comment

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

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).

Copy link
Author

Choose a reason for hiding this comment

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

Have restricted it due to line length CI check issue.

Copy link
Contributor
@brett0000FF brett0000FF left a comment

Choose a reason for hiding this comment

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

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

Comment on lines 54 to 65
### Metrics

Barracuda_Secure_Edge does not include any metrics.


### Log Collection
## Data Collected
The Barracuda Secure Edge logs contain key information such as the event timestamp, source and destination IPs and ports, protocol used, firewall action (allow/deny), the matched rule name, user identity (if available), log type (e.g., firewall, VPN, authentication), network interface, device name, and status of the operation, all of which help monitor traffic behavior, access control, and system activity and many more which are collected by DataDog.

### Events

The Secure_edge integration includes log events such as failed logins and rule hits.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### Metrics
Barracuda_Secure_Edge does not include any metrics.
### Log Collection
## Data Collected
The Barracuda Secure Edge logs contain key information such as the event timestamp, source and destination IPs and ports, protocol used, firewall action (allow/deny), the matched rule name, user identity (if available), log type (e.g., firewall, VPN, authentication), network interface, device name, and status of the operation, all of which help monitor traffic behavior, access control, and system activity and many more which are collected by DataDog.
### Events
The Secure_edge integration includes log events such as failed logins and rule hits.
## Data Collected
### Metrics
Barracuda_Secure_Edge does not include any metrics.
### Events
The Barracuda Secure Edge integration does not include any events.
### Logs
The Barracuda Secure Edge integration collects logs containing the following types of information:
- **Security Events**: Firewall actions (allow/deny), rule matches, and security policy violations
- **Network Traffic**: Source and destination IPs/ports, protocols, and network interfaces
- **Authentication**: User login attempts, successes, and failures
- **VPN Activity**: VPN connection events and status
- **System Events**: Device status, configuration changes, and system health


## Overview

This integration monitors [barracuda_secure_edge][4].
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This integration monitors [barracuda_secure_edge][4].
Barracuda Secure Edge is a unified Secure Access Service Edge (SASE) platform that includes Next-Generation Firewall (NGFW), zero trust, and secure Software-Defined Wide Area Network (SD-WAN) capabilities. This integration allows you to collect and analyze logs from your [barracuda_secure_edge][4] deployment to monitor security events, network traffic, and system activity.

Comment on lines 31 to 36
### Prerequisites

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).
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be the first subsection of Setup, so users see this info before starting.


```yaml
logs:
- type: file
Copy link
Contributor

Choose a reason for hiding this comment

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

It seems like we are mixing file-based and syslog methods. The setup steps use a file, but the validation steps use syslog? My understanding is we should be consistent here.

Copy link
Author

Choose a reason for hiding this comment

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

Basically, syslog logs are being exported to a file and datadog reads the file for ingesiton into the agent. I have updated few phrases. Thanks

Need help? Contact [Datadog support][1].

[1]: https://docs.datadoghq.com/help/
[2]: https://app.datadoghq.com/account/settings/agent/latest
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[2]: https://app.datadoghq.com/account/settings/agent/latest
[2]: /account/settings/agent/latest

[2]: https://app.datadoghq.com/account/settings/agent/latest
[3]: https://docs.datadoghq.com/agent/guide/agent-commands/#start-stop-and-restart-the-agent
[4]: https://www.barracuda.com/products/network-protection/secureedge
[5]: https://app.datadoghq.com/logs/livetail
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[5]: https://app.datadoghq.com/logs/livetail
[5]: /logs/livetail

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

Successfully merging this pull request may close these issues.

6 participants
0