Skip to content

Ciac 14226 rebrand fire eye etp integrations #41218

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
Merged
merged 12 commits into from
Sep 14, 2025
Merged

Ciac 14226 rebrand fire eye etp integrations #41218

merged 12 commits into from
Sep 14, 2025

Conversation

@lironcohen272
Copy link
Contributor

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-14226

Description

reberand pack from "FireEye Email Threat Protection (ETP)", to Trellix Email Security

@github-actions
Copy link

github-actions bot commented Sep 8, 2025 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/CommonScripts/Scripts/ProvidesCommand
   ProvidesCommand.py54885%29–31, 43–45, 64, 70
Packs/FireEyeETP/Integrations/FireEyeETP
   FireEyeETP.py36820544%76–78, 81–83, 86–88, 91–92, 95, 98, 100–101, 104, 106–107, 111–115, 119–121, 136, 138, 144, 149, 154, 158–162, 164–166, 168, 186, 189–194, 197–202, 205–210, 212–215, 218–223, 226, 228–229, 233, 237, 248, 251, 254–255, 257–259, 261–266, 268, 272–276, 278–280, 284–285, 287–288, 290, 292, 309, 312, 315–317, 321, 329, 333–337, 342, 344, 346, 349–351, 355, 363, 366, 373, 377, 426, 429–440, 442–447, 451, 453–454, 456–457, 460, 468, 471–472, 484, 487, 495, 583, 626, 633, 643–644, 661, 663, 668–669, 683, 707–709, 711, 713–715, 717–718, 720, 722–723, 725–751
Packs/FireEyeETP/Integrations/FireEyeETPEventCollector
   FireEyeETPEventCollector.py2742889%207–209, 280, 319–320, 328–329, 383, 439–440, 498–499, 509–511, 542, 562, 585–589, 597–600, 602
TOTAL69624165% 

Tests Skipped Failures Errors Time
33 0 💤 0 ❌ 0 🔥 26.222s ⏱️

@lironcohen272 lironcohen272 added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label Sep 9, 2025
@eepstain eepstain added the bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url label Sep 9, 2025
@richardbluestone richardbluestone requested review from julieschwartz18 and removed request for richardbluestone September 9, 2025 12:58
julieschwartz18
julieschwartz18 reviewed Sep 9, 2025
View reviewed changes
Copy link
Contributor

@julieschwartz18 julieschwartz18 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

@lironcohen272 Doc review complete. Please check the suggestions and comments, I have some questions. In the meantime, adding docs-approved label.

@lironcohen272 lironcohen272 added the ForceMerge Forcing the merge of the PR despite the build status label Sep 11, 2025
@lironcohen272
Copy link
Contributor Author

A force merge is needed because the changes involve updating display names in the YAML file without modifying the 'id' parameter to match.

@content-bot
Copy link
Collaborator

Validate summary
The following errors were thrown as a part of this pr: BC105, ST111.
The following errors cannot be ignored: BC105, ST111.
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.
The following errors don't run as part of the nightly flow and therefore can be force merged: BC105, ST111.

Verdict: PR can be force merged from validate perspective? ✅

@DeanArbel DeanArbel merged commit 7997dee into master Sep 14, 2025
17 of 19 checks passed
@DeanArbel DeanArbel deleted the CIAC-14226-Rebrand-FireEye-ETP-Integrations branch September 14, 2025 08:07
scohenkadosh pushed a commit that referenced this pull request Sep 15, 2025
@lironcohen272 @scohenkadosh
Ciac 14226 rebrand fire eye etp integrations (#41218)
cd802bb 
* docs: change FireEye ETP to Trellix Email Security - Cloud across  pack files

* rename: update modeling rule name from FireEye ETP to Trellix Email Security - Cloud

* update Docker image

* update FireEye ETP modeling rules id to Trellix Email Security Cloud

* change FireEye ETP references to Trellix Email Security Cloud on description and logs

* update FireEye ETP integration images

* Fixing after doc-review

* doc-review fixing

* empty commit

* add BA101 ignore rule for FireEye ETP Indicators Hunting playbook
xsoar-bot pushed a commit to xsoar-contrib/content that referenced this pull request Sep 17, 2025
@lironcohen272 @xsoar-bot
Ciac 14226 rebrand fire eye etp integrations (demisto#41218)
6546243 
* docs: change FireEye ETP to Trellix Email Security - Cloud across  pack files

* rename: update modeling rule name from FireEye ETP to Trellix Email Security - Cloud

* update Docker image

* update FireEye ETP modeling rules id to Trellix Email Security Cloud

* change FireEye ETP references to Trellix Email Security Cloud on description and logs

* update FireEye ETP integration images

* Fixing after doc-review

* doc-review fixing

* empty commit

* add BA101 ignore rule for FireEye ETP Indicators Hunting playbook
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@julieschwartz18 julieschwartz18 julieschwartz18 left review comments

@thefrieddan1 thefrieddan1 thefrieddan1 approved these changes

@DeanArbel DeanArbel DeanArbel approved these changes

@eepstain eepstain eepstain approved these changes

@kamalq97 kamalq97 Awaiting requested review from kamalq97

Assignees

No one assigned

Labels

bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url docs-approved ForceMerge Forcing the merge of the PR despite the build status ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@lironcohen272 @content-bot @thefrieddan1 @DeanArbel @julieschwartz18 @eepstain