cristianoag / microsoft-teams-apps-company-communicator

Company Communicator app template
MIT License
16 stars 38 forks source link

Recipients not syncing after secret expired #76

Open ostromecko opened 9 months ago

ostromecko commented 9 months ago

Hi, I'm running the latest master version v5.31. Recently our secrets have expired, and what we did is that we generated new ones, and update them in key vault. Currently I'm receiving below error when trying to send a test message using a CSV file: image

Failed to prepare the message for sending:The orchestrator function 'SyncRecipientsOrchestrator' failed: "The activity function 'SyncCSVActivity' failed: "Specified argument was out of the range of valid values. (Parameter 'partitionCount') Actual value was .". See the function execution logs for additional details.". See the function execution logs for additional details.

So I would need help with resolving this, and checking if I did not miss any secret update anywhere in the configuration.

luishdemetrio commented 8 months ago

Hi @ostromecko, could you please share the function execution logs?

The Log Stream is a feature in Azure that allows you to view the live logging information directly in the Azure portal. Here’s how you can use it:

  1. Navigate to your function app in the Azure portal.
  2. Select the Monitoring section.
  3. Click on Log Stream.

Once you’re in the Log Stream, you should start seeing logs as your functions execute. If you don’t see any logs, make sure your function app is running and executing functions.

ostromecko commented 8 months ago

Hi @luishdemetrio, which function from the three functions I should open and check if there are logs? Data-Function, Function or Prep-function? And what you mean by "make sure your function app is running and executing functions."? Company communicator is an app which is used ad-hoc.

ostromecko commented 7 months ago

Hi @luishdemetrio, can you please support me with the check if the functions are executed?

ostromecko commented 7 months ago

Hi @ostromecko, could you please share the function execution logs?

The Log Stream is a feature in Azure that allows you to view the live logging information directly in the Azure portal. Here’s how you can use it:

  1. Navigate to your function app in the Azure portal.
  2. Select the Monitoring section.
  3. Click on Log Stream.

Once you’re in the Log Stream, you should start seeing logs as your functions execute. If you don’t see any logs, make sure your function app is running and executing functions.

@luishdemetrio I have opened Log stream, but it's constantly loading and not showing anything. Can you please support on this?

ostromecko commented 7 months ago

@cristianoag could you help me with this issue?

cristianoag commented 7 months ago

When you renew the secrets in KeyVault you need to make sure the configuration variables are referencing the new key. Can you check all functions and the app service and make sure they are pointing to the right place?

Get Outlook for iOShttps://aka.ms/o0ukef


From: ostromecko @.> Sent: Friday, March 22, 2024 10:04:20 AM To: cristianoag/microsoft-teams-apps-company-communicator @.> Cc: Cristiano Goncalves @.>; Mention @.> Subject: Re: [cristianoag/microsoft-teams-apps-company-communicator] Recipients not syncing after secret expired (Issue #76)

@cristianoaghttps://github.com/cristianoag could you help me with this issue?

— Reply to this email directly, view it on GitHubhttps://github.com/cristianoag/microsoft-teams-apps-company-communicator/issues/76#issuecomment-2015061632, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALSAPKUCYOTFEO6MU764U63YZQT5JAVCNFSM6AAAAABDIIG4FGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJVGA3DCNRTGI. You are receiving this because you were mentioned.Message ID: @.***>

ostromecko commented 7 months ago

Currently I'm facing this issue: Failed to prepare the message for sending:The orchestrator function 'SyncRecipientsOrchestrator' failed: "The activity function 'SyncCSVActivity' failed: "Specified argument was out of the range of valid values. (Parameter 'partitionCount') Actual value was .". See the function execution logs for additional details.". See the function execution logs for additional details.

cristianoag commented 7 months ago

Are you using a standard CSV file as input? Can you post it so I can check the format?

Get Outlook for iOShttps://aka.ms/o0ukef


From: ostromecko @.> Sent: Friday, March 22, 2024 10:21:24 AM To: cristianoag/microsoft-teams-apps-company-communicator @.> Cc: Cristiano Goncalves @.>; Mention @.> Subject: Re: [cristianoag/microsoft-teams-apps-company-communicator] Recipients not syncing after secret expired (Issue #76)

Currently I'm facing this issue: Failed to prepare the message for sending:The orchestrator function 'SyncRecipientsOrchestrator' failed: "The activity function 'SyncCSVActivity' failed: "Specified argument was out of the range of valid values. (Parameter 'partitionCount') Actual value was .". See the function execution logs for additional details.". See the function execution logs for additional details.

— Reply to this email directly, view it on GitHubhttps://github.com/cristianoag/microsoft-teams-apps-company-communicator/issues/76#issuecomment-2015088972, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALSAPKVY4IRC2UKCLUZSWN3YZQV5JAVCNFSM6AAAAABDIIG4FGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJVGA4DQOJXGI. You are receiving this because you were mentioned.Message ID: @.***>

ostromecko commented 7 months ago

of course. I'm attaching file which I'm using. companycomm.csv

cristianoag commented 7 months ago

I've just tested, and the format you provided appears to be in order and is being recognized. I suggest considering a few potential areas for investigation.

Verify if all functions are correctly configured to access the appropriate keys in KeyVault. Assuming you've already done this, another possibility could be discrepancies between the versions of functions being run compared to the version of the app service. It's essential to ensure that all resources are synchronized to the same version. I recommend checking the synchronization log file for any errors or inconsistencies.

ostromecko commented 7 months ago

I have checked all three functions and app service. All four instances are pointing to the same commit ID 9d9ce88.

cristianoag commented 7 months ago

Do you have email addresses different than upns?

Get Outlook for iOShttps://aka.ms/o0ukef


From: ostromecko @.> Sent: Friday, March 22, 2024 12:10:11 PM To: cristianoag/microsoft-teams-apps-company-communicator @.> Cc: Cristiano Goncalves @.>; Mention @.> Subject: Re: [cristianoag/microsoft-teams-apps-company-communicator] Recipients not syncing after secret expired (Issue #76)

I have checked all three functions and app service. All four instances are pointing to the same commit ID 9d9ce88https://github.com/cristianoag/microsoft-teams-apps-company-communicator/commit/9d9ce881db2fcd5a01b407e90d438b6b25010b80.

— Reply to this email directly, view it on GitHubhttps://github.com/cristianoag/microsoft-teams-apps-company-communicator/issues/76#issuecomment-2015308572, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALSAPKVHVCN3I4UVZURZ5ZDYZRCVHAVCNFSM6AAAAABDIIG4FGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJVGMYDQNJXGI. You are receiving this because you were mentioned.Message ID: @.***>

ostromecko commented 7 months ago

No. I was even testing this with a file which was containing only two email address (mine, and my colleague) and still it was not working

cristianoag commented 7 months ago

Email address or UPN? Very important, the file needs to have upns, not emails.

Get Outlook for iOShttps://aka.ms/o0ukef


From: ostromecko @.> Sent: Friday, March 22, 2024 12:22:53 PM To: cristianoag/microsoft-teams-apps-company-communicator @.> Cc: Cristiano Goncalves @.>; Mention @.> Subject: Re: [cristianoag/microsoft-teams-apps-company-communicator] Recipients not syncing after secret expired (Issue #76)

No. I was even testing this with a file which was containing only two email address (mine, and my colleague) and still it was not working

— Reply to this email directly, view it on GitHubhttps://github.com/cristianoag/microsoft-teams-apps-company-communicator/issues/76#issuecomment-2015334039, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALSAPKUBSP2HJKLGUFVARW3YZREE3AVCNFSM6AAAAABDIIG4FGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJVGMZTIMBTHE. You are receiving this because you were mentioned.Message ID: @.***>

ostromecko commented 7 months ago

UPN matches e-mail address in our case.

cristianoag commented 7 months ago

Collect and upload the function log files. So we can check what is going on. Need more detailed error mess

Get Outlook for iOShttps://aka.ms/o0ukef


From: ostromecko @.> Sent: Friday, March 22, 2024 12:32:54 PM To: cristianoag/microsoft-teams-apps-company-communicator @.> Cc: Cristiano Goncalves @.>; Mention @.> Subject: Re: [cristianoag/microsoft-teams-apps-company-communicator] Recipients not syncing after secret expired (Issue #76)

UPN matches e-mail address in our case.

— Reply to this email directly, view it on GitHubhttps://github.com/cristianoag/microsoft-teams-apps-company-communicator/issues/76#issuecomment-2015352231, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALSAPKUAOYGS2QNWZBIRZ43YZRFKNAVCNFSM6AAAAABDIIG4FGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJVGM2TEMRTGE. You are receiving this because you were mentioned.Message ID: @.***>

ostromecko commented 7 months ago

Unfortunatly the log stream for me is constantly loading, and it doesn't provide any logs image

Or I can collect them in different way?

onealll commented 2 months ago

@cristianoag Im facing the same problem, all my secrets has expired, you have any script to do the secrets update to the keyvault? or can you tell me what to do exacly to update?, im using versing 5.31 expired The problem is this, when I try to select the users to send the message, Im reciving this messagem: imagem

"Permissions required. Ask your IT administrator to enable this option for you"

Im assuming this message is becouse of the expired secredts that don allow access to groups/users of m365