Skip to content

Gigamon Connector Updated Version with CCF Connector#13597

Closed
Mrudula-Oruganti-Gigamon wants to merge 0 commit intoAzure:masterfrom
Mrudula-Oruganti-Gigamon:master
Closed

Gigamon Connector Updated Version with CCF Connector#13597
Mrudula-Oruganti-Gigamon wants to merge 0 commit intoAzure:masterfrom
Mrudula-Oruganti-Gigamon:master

Conversation

@Mrudula-Oruganti-Gigamon
Copy link
Contributor

Required items, please complete

Change(s):

  • Added few more attributes to Sentinel
  • Changed to CCF Connector method

Reason for Change(s):

  • Previous method of integration has been deprecated (using LA)

Version Updated:

  • Added few more files that are required for CCF ingestion like DCR.Json, table.json

Testing Completed:

  • No

@v-maheshbh v-maheshbh added New Solution For new Solutions which are new to Microsoft Sentinel and removed Solution Solution specialty review needed labels Feb 10, 2026
@v-maheshbh
Copy link
Contributor

Hi @Mrudula-Oruganti-Gigamon
For the CCF connector, please ensure that all file names follow the required naming convention:
solutionName_ConnectorDefinition,
solutionName_PollerConfig,
solutionName_DCR,
solutionName_Table.
Kindly update the filenames accordingly.

Kindly refer to the below-mentioned solution for reference, and ensure that the table name is correctly specified and consistently referenced across the PollerConfig, DCR, and Table files.

https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/Cloudflare%20CCF

Thanks!

@Mrudula-Oruganti-Gigamon
Copy link
Contributor Author

@v-maheshbh , Thanks for the comments. Made respective changes. Please review and let me know.

Copy link
Contributor Author

@Mrudula-Oruganti-Gigamon Mrudula-Oruganti-Gigamon left a comment

Choose a reason for hiding this comment

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

Submitting review as these files are missing and blocking my package to merge.

Copy link
Contributor Author

@Mrudula-Oruganti-Gigamon Mrudula-Oruganti-Gigamon left a comment

Choose a reason for hiding this comment

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

Made respective changes as mentioned by reviewer

@Mrudula-Oruganti-Gigamon
Copy link
Contributor Author

Hi @Mrudula-Oruganti-Gigamon For the CCF connector, please ensure that all file names follow the required naming convention: solutionName_ConnectorDefinition, solutionName_PollerConfig, solutionName_DCR, solutionName_Table. Kindly update the filenames accordingly.

Kindly refer to the below-mentioned solution for reference, and ensure that the table name is correctly specified and consistently referenced across the PollerConfig, DCR, and Table files.

https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/Cloudflare%20CCF

Thanks!

Hi @v-maheshbh , I have made respective changes in the files as you suggested. However, the review has stopped as there were some conflicts to merge and the bot has recommended to include those changes and i did the same.
Would like to know if there are next steps at my end. Could you please review and let me know. Thanks.

@v-maheshbh
Copy link
Contributor

Hi @Mrudula-Oruganti-Gigamon

Could you please confirm whether the below-mentioned tool changes are part of this PR?

image

Thanks!

@Mrudula-Oruganti-Gigamon
Copy link
Contributor Author

Hi @Mrudula-Oruganti-Gigamon

Could you please confirm whether the below-mentioned tool changes are part of this PR?

image Thanks!

No @v-maheshbh . It's not mine.

@Mrudula-Oruganti-Gigamon
Copy link
Contributor Author

@v-atulyadav , Could you please confirm whether i need to merge the below two files into my reporsitory ?? It is often blocking my merging. Please confirm :

Tools/Solutions Analyzer/solutions_connectors_tables_issues_and_exceptions_report.csv
Tools/Solutions Analyzer/solutions_connectors_tables_mapping.csv

@v-maheshbh
Copy link
Contributor

Hi @Mrudula-Oruganti-Gigamon
Could you please confirm whether the below-mentioned tool changes are part of this PR?
image
Thanks!

No @v-maheshbh . It's not mine.

Kindly revert the tool-related changes, as they are not part of this PR.

@Mrudula-Oruganti-Gigamon
Copy link
Contributor Author

Hi @Mrudula-Oruganti-Gigamon
Could you please confirm whether the below-mentioned tool changes are part of this PR?
image
Thanks!

No @v-maheshbh . It's not mine.

Kindly revert the tool-related changes, as they are not part of this PR.

image

I dnt see them in my environment @v-maheshbh . Could you confirm where i could make those changes ?

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

Labels

Codeless Connector Framework (CCF) Connector New Solution For new Solutions which are new to Microsoft Sentinel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants