-
Notifications
You must be signed in to change notification settings - Fork 130
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
Modify the samples for Tables to avoid using connection string overloads. #6142
Conversation
/azp run cpp - tables |
Azure Pipelines successfully started running 1 pipeline(s). |
API change check API changes are not detected in this pull request. |
It looks like running the samples inherently rely on the connection string env var, containing the account name and sas token. Ideally, fetching the account name from the env var should work, but it seems to not find the resource:
If I had to guess, there's some resource setup mismatch. cc @danieljurek |
I'm not comfortable checking in samples that we've not verified actually work. When you run them on your dev client, do they run successfully (in other words, is the error an error in the pipeline configuration, or is it that the entra id stuff doesn't work because of a test-resources issue)? Is it possible this is as simple as not having a delay in the test-resources to let the entra ID provisioning to replicate (we've had to add a delay to eventhubs and attestation for this reason). |
Superseded by #6181 |
No description provided.