Skip to content
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

Q2 2024 updates #11049

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

Q2 2024 updates #11049

wants to merge 25 commits into from

Conversation

nlepagnez
Copy link
Contributor

Required items, please complete

Change(s):

  • Deprecate Microsoft Exchange Logs and Events and creating multiple Data Connectors dedicated for AMA
  • Correct bug on lastReceivedData for Exchange On-Premise and Online Collector Data connector
  • Add a comparison system in Workbooks

Reason for Change(s):

  • Solution evolution

@v-atulyadav v-atulyadav added the Solution Solution specialty review needed label Aug 30, 2024
@nlepagnez nlepagnez marked this pull request as ready for review August 30, 2024 10:36
@nlepagnez nlepagnez requested review from a team as code owners August 30, 2024 10:36
@itsjusthaif
Copy link

Hi
Currently in the process of deploying this solution and I am stuck at the same issue due to which this pull was raised. When can I expect the solution to be updated?

@nlepagnez
Copy link
Contributor Author

Hi @v-prasadboke, @v-atulyadav do we have any news on the analysis of this PR ?

@nlepagnez
Copy link
Contributor Author

Hi Currently in the process of deploying this solution and I am stuck at the same issue due to which this pull was raised. When can I expect the solution to be updated?

Hi itsjusthaif, we are waiting for review from MS Sentinel team. We will update you as soon as possible.

@v-prasadboke
Copy link
Contributor

Hello @nlepagnez & @itsjusthaif Sorry for the delay in response. Had some priority tasks on my name. Will get on this PR and update you soon

@v-prasadboke v-prasadboke added the Parser Parser specialty review needed label Sep 12, 2024
@nlepagnez
Copy link
Contributor Author

@v-prasadboke any update ?

@nlepagnez
Copy link
Contributor Author

nlepagnez commented Oct 1, 2024

@v-prasadboke, the "DataConnector" test detects that ESI-Opt6ExchangeMessageTrackingLogs.json and ESI-Opt7ExchangeHTTPProxyLogs.json are "Azure Function" Data connector, but it's completely false.

The detection has this conclusion because the table is not "Event" table and because he find "Deploy to Azure" in instruction but :

  • Yes it's not Event table because we collect a specific log file in the server
  • Yes he can find "Deploy to Azure" because the new Azure Monitor impose DCR that are deployed using ARM Template. It's not an Azure function, we use the Azure Monitor Agent.

Can you please correct the test because I cannot add a "Virtual" permission in my Data connector to request "Website permission" only to satisfy the test and create a confusion for end user.

@itsjusthaif
Copy link

Hi @nlepagnez, any update on this, please? Still waiting to go ahead or will have to manually map our what changes were made and reflect that to our environment.

@nlepagnez
Copy link
Contributor Author

Hi @itsjusthaif, no news on the Microsoft Sentinel team. As you can see, I don't received any answer on my last comment.
@v-prasadboke, @v-atulyadav we are waiting on you. For remember, this PR is now opened since more than 1 month.

@v-prasadboke
Copy link
Contributor

v-prasadboke commented Oct 8, 2024

Hello @nlepagnez & @itsjusthaif, Apologies for the inconvenience but I was on leave from 2nd of October till 6th of October IST

But I still see some validation failure for Provided permissions does not match with Azure Function Connector Template from last commit.
Guidelines are provided in the failure checks to resolve permissions failure, Template path is provided in the failure description.

@nlepagnez
Copy link
Contributor Author

Hello @nlepagnez & @itsjusthaif, Apologies for the inconvenience but I was on leave from 2nd of October till 6th of October IST

But I still see some validation failure for Provided permissions does not match with Azure Function Connector Template from last commit. Guidelines are provided in the failure checks to resolve permissions failure, Template path is provided in the failure description.

Hi @v-prasadboke, as you can see in my previous comment, this connector is not an Azure Function Connector, but an Azure Monitor Connector ingesting data in a custom table. I will not add a permission "Website permission" to my connector just to satisfy a test, a permission that will confuse users.

The test script detect my connector as Azure function connector because it's a custom table (not event table) and it found the word "Azure Deploy" inside instruction. Is it means that we cannot use Azure Deploy to deploy the DCRs needed by the "New" AMA ?!

So again, I ask you to correct the test to be compatible with AMA collecting custom data, not adapting a Data connector because the test do not works.

@itsjusthaif
Copy link

Hi @v-prasadboke please see the comment above and help progress this. We have customers waiting on this solution and this is now delayed by a good margin.

v-prasadboke
v-prasadboke previously approved these changes Oct 10, 2024
v-prasadboke
v-prasadboke previously approved these changes Oct 11, 2024
v-dvedak
v-dvedak previously approved these changes Oct 11, 2024
@v-prasadboke v-prasadboke dismissed stale reviews from v-dvedak and themself via bd39087 October 11, 2024 05:26
@itsjusthaif
Copy link

Any update on this one please?

@v-prasadboke
Copy link
Contributor

v-prasadboke commented Oct 14, 2024

Hello @nlepagnez & @itsjusthaif, We skipvalidated the template but still not able to get this PR merged

We are working on this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Connector Connector specialty review needed Parser Parser specialty review needed Solution Solution specialty review needed
Projects
None yet
5 participants