Remove azure.profile.yml link from tls.benchmarks.yml #2119
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
…tly duplicating the config reference causing the application to get send twice in a single run. Other files leave azure.profile.yml to be set at the build/*-scenarios.yml level, so this brings that to being copied.
LoopedBard3 requested review from DeagleGross and Copilot and removed request for Copilot There was a problem hiding this comment.
Pull Request Overview
This PR removes a duplicate reference to azure.profile.yml from the tls.benchmarks.yml configuration file to prevent applications from being sent twice in a single benchmark run.
- Removes the azure.profile.yml link from the imports section
- Aligns with the pattern used by other benchmark files where azure.profile.yml is configured at the build/*-scenarios.yml level
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
DeagleGross approved these changes Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Remove azure.profile.yml link from tls.benchmarks.yml as it is currently duplicating the config reference causing the application to get send twice in a single run. Other files leave azure.profile.yml to be set at the build/*-scenarios.yml level, so this brings that to being copied.