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

Does DFM_ALTERNATIVE_CONNECTION_STRING_* Support Identity connection ? #59

Open
MasterKuat opened this issue Sep 30, 2022 · 5 comments
Open
Labels
question Further information is requested

Comments

@MasterKuat
Copy link

Hi,

I try to use a standalone DFM installation to monitor 4 differents functions.
Set DFM_ALTERNATIVE_CONNECTION_STRING_* for all 4 works well if I set the full connection string with key and so on but is there a way to use DFM identity to connect other storage account ?

Thanks

@scale-tone scale-tone added enhancement New feature or request question Further information is requested and removed enhancement New feature or request labels Sep 30, 2022
@scale-tone
Copy link
Contributor

Hi @MasterKuat ,
No, identity-based connections are not currently supported for alternative storages (DFM_ALTERNATIVE_CONNECTION_STRING_*).

I suggest you either store those connection strings in Key Vault (and use the @Microsoft.KeyVault() references), or deploy separate DfMon instances for each storage, or just use DfMon in form of VsCode (or GitHub Codespaces) extension.

@MasterKuat
Copy link
Author

Hi @scale-tone ,

Thanks for your answer.
My initial guess was good.

For operationnal purposes, I wish to use only 1 instance of (your great) DfMon, could it be added ?
As Managed Identity seems to be The Future, sounds to me like a good idea.

@bhugot
Copy link
Contributor

bhugot commented Nov 23, 2022

Hi @MasterKuat from my point of view but I could be wrong support for managed identity is related to the migration to Azure.Storage for durable task. There is a PR for this Azure/durabletask#763 and it should solve part of the problem but @scale-tone, I think the same move should be done here for all usage of Microsoft.WindowsAzure.Storage

@MasterKuat
Copy link
Author

Hi @bhugot,
I'm not sure for the PR as DurableFunction (which use DurableTask) already support managed identity for storage account.

@bhugot
Copy link
Contributor

bhugot commented Nov 24, 2022

Ok my bad

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants