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

Bootstrap test-proxy service #1956

Merged
merged 3 commits into from
Dec 12, 2024
Merged

Bootstrap test-proxy service #1956

merged 3 commits into from
Dec 12, 2024

Conversation

heaths
Copy link
Member

@heaths heaths commented Dec 12, 2024

Relates to #1874 but does not complete it.

Relates to Azure#1874 but does not complete it.
@heaths
Copy link
Member Author

heaths commented Dec 12, 2024

@RickWinter @analogrelay I'm still working on the transport but nearly done. I figured enough was here already I should push the bootstrapping portion (minus automatic acquisition, tracked in #1955 so I can get an MVP out by EOW) to alleviate some of the review burden. I'll rebase --onto whatever gets merged here.

Copy link
Member

@analogrelay analogrelay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small issue with tests on the sanitizers, but I trust you to take care of it, so nothing blocking on my end.

sdk/core/azure_core_test/src/proxy.rs Outdated Show resolved Hide resolved
sdk/core/azure_core_test/src/sanitizers.rs Outdated Show resolved Hide resolved
sdk/core/azure_core_test/src/sanitizers.rs Outdated Show resolved Hide resolved
@heaths heaths merged commit b6bbc74 into Azure:main Dec 12, 2024
13 checks passed
@heaths heaths deleted the test-proxy branch December 12, 2024 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants