Skip to content

[Internal] PPAF: Fixes Integration Tests to Intercept Gateway Response#5261

Merged
microsoft-github-policy-service[bot] merged 3 commits intomasterfrom
users/kundadebdatta/fix_ppaf_integration_tests
Jun 27, 2025
Merged

[Internal] PPAF: Fixes Integration Tests to Intercept Gateway Response#5261
microsoft-github-policy-service[bot] merged 3 commits intomasterfrom
users/kundadebdatta/fix_ppaf_integration_tests

Conversation

@kundadebdatta
Copy link
Copy Markdown
Member

@kundadebdatta kundadebdatta commented Jun 26, 2025

Pull Request Template

Description

Recently, we started observing in our CI pipeline gates that, the gateway started returning the ppaf enablement flag as False (instead of null) for the account used for end to end test. Below are the significance of the values returned from the gateway:

Flag GW Response Description
enablePerPartitionFailoverBehavior True PPAF was enabled for the account.
enablePerPartitionFailoverBehavior False PPAF was manually disabled for the account.
enablePerPartitionFailoverBehavior null PPAF was never set for the account.
enablePerPartitionFailoverBehavior flag not present PPAF was never set for the account.

This PR fixes the integration test by intercepting the enablePerPartitionFailoverBehavior flag for the gateway response to set it as null, so that the PPAF enablement environment variable can be honored.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Closing issues

To automatically close an issue: closes #IssueNumber

aavasthy
aavasthy previously approved these changes Jun 26, 2025
NaluTripician
NaluTripician previously approved these changes Jun 26, 2025
@kundadebdatta kundadebdatta dismissed stale reviews from NaluTripician and aavasthy via aded17a June 26, 2025 21:22
aavasthy
aavasthy previously approved these changes Jun 26, 2025
NaluTripician
NaluTripician previously approved these changes Jun 26, 2025
@kundadebdatta kundadebdatta dismissed stale reviews from NaluTripician and aavasthy via e457147 June 27, 2025 01:43
@microsoft-github-policy-service microsoft-github-policy-service Bot merged commit 60c2d97 into master Jun 27, 2025
27 checks passed
@microsoft-github-policy-service microsoft-github-policy-service Bot deleted the users/kundadebdatta/fix_ppaf_integration_tests branch June 27, 2025 03:56
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 PerPartitionAutomaticFailover

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants