Skip to content

Migrate azure security confidential ledger tests to use token auth.#40819

Merged
anuchandy merged 3 commits into
mainfrom
test/anuchan/azure-security-confidentialledger-token-auth
Jul 9, 2024
Merged

Migrate azure security confidential ledger tests to use token auth.#40819
anuchandy merged 3 commits into
mainfrom
test/anuchan/azure-security-confidentialledger-token-auth

Conversation

@anuchandy

Copy link
Copy Markdown
Member

No description provided.

@azure-sdk

Copy link
Copy Markdown
Collaborator

API change check

API changes are not detected in this pull request.

@g2vinay

g2vinay commented Jun 28, 2024

Copy link
Copy Markdown
Member

@anuchandy, we should validate the live tests too.

@anuchandy

Copy link
Copy Markdown
Member Author

@anuchandy

Copy link
Copy Markdown
Member Author

/azp run java - azure-security-confidentialledger - tests

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@anuchandy

Copy link
Copy Markdown
Member Author

@g2vinay, the reason for the second live test run failure is, there is no quote left for a resource-group in test subscription.

@anuchandy

Copy link
Copy Markdown
Member Author

/azp run java - azure-security-confidentialledger - tests

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@anuchandy

Copy link
Copy Markdown
Member Author

/azp run java - azure-security-confidentialledger - tests

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@anuchandy

Copy link
Copy Markdown
Member Author

@g2vinay - as discussed offline, I've removed the cert outputs from the ARM template as they are not needed.

@anuchandy anuchandy merged commit 85a16ab into main Jul 9, 2024
@anuchandy anuchandy deleted the test/anuchan/azure-security-confidentialledger-token-auth branch July 9, 2024 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants