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

FaultInjection: Refactors applyPercentage name to injectionRate to make feature clearer. #4511

Conversation

NaluTripician
Copy link
Contributor

Pull Request Template

Description

Changed the name of the ApplyPercentage feature to InjectionRate. This should make the function of the feature clearer to users. This will also be the name of the feature in the Java SDK.

Type of change

Please delete options that are not relevant.

  • [] Bug fix (non-breaking change which fixes an issue)
  • [] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [] This change requires a documentation update

Closing issues

To automatically close an issue: closes #IssueNumber

@NaluTripician NaluTripician self-assigned this May 21, 2024
@NaluTripician NaluTripician added the auto-merge Enables automation to merge PRs label May 21, 2024
Copy link
Member

@kundadebdatta kundadebdatta left a comment

Choose a reason for hiding this comment

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

LGTM. Let's verify why the tests are failing.

@NaluTripician
Copy link
Contributor Author

LGTM. Let's verify why the tests are failing.

Looks like they are from some flaky tests. They seem to be failing on other PRs too and I have seen these randomly fail before. I will rerun the test a few more time to double check but currently there are no tests that rely on this.

@microsoft-github-policy-service microsoft-github-policy-service bot merged commit b9b35bb into master May 29, 2024
21 checks passed
@microsoft-github-policy-service microsoft-github-policy-service bot deleted the users/nalutripician/faultInjectionInjectionRate branch May 29, 2024 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Enables automation to merge PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants