Skip to content

Speed up local tests, log id, and document includes usage#5349

Merged
LTA-Thinking merged 1 commit intomainfrom
personal/rojo/speedup-tests
Jan 26, 2026
Merged

Speed up local tests, log id, and document includes usage#5349
LTA-Thinking merged 1 commit intomainfrom
personal/rojo/speedup-tests

Conversation

@LTA-Thinking
Copy link
Contributor

Description

This PR fixes three small issues:

  • Logs the correlation id returned to the customer so it is easier to corelate responses with internal logs.
  • Adds documentation on the use of the includes continuation token in the deletion service.
  • Fixes a configuration issue with local E2E tests to speed up their execution.

Related issues

Addresses
User Story 180677
Bug 181178
Bug 180915

Testing

Manual testing and existing tests.

FHIR Team Checklist

  • Update the title of the PR to be succinct and less than 65 characters
  • Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
  • Tag the PR with the type of update: Bug, Build, Dependencies, Enhancement, New-Feature or Documentation
  • Tag the PR with Open source, Azure API for FHIR (CosmosDB or common code) or Azure Healthcare APIs (SQL or common code) to specify where this change is intended to be released.
  • Tag the PR with Schema Version backward compatible or Schema Version backward incompatible or Schema Version unchanged if this adds or updates Sql script which is/is not backward compatible with the code.
  • When changing or adding behavior, if your code modifies the system design or changes design assumptions, please create and include an ADR.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

@LTA-Thinking LTA-Thinking added this to the FY26\Q3\2Wk\2Wk15 milestone Jan 23, 2026
@LTA-Thinking LTA-Thinking requested a review from a team as a code owner January 23, 2026 21:23
@LTA-Thinking LTA-Thinking added Bug Bug bug bug. Enhancement Enhancement on existing functionality. Azure API for FHIR Label denotes that the issue or PR is relevant to the Azure API for FHIR Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs No-PaaS-breaking-change No-ADR ADR not needed labels Jan 23, 2026
@LTA-Thinking LTA-Thinking changed the title Speed up local tests, log correlation id, and document includes usage Speed up local tests, log id, and document includes usage Jan 23, 2026
// Enable background jobs.
configuration["TaskHosting:Enabled"] = "true";
configuration["TaskHosting:MaxRunningTaskCount"] = "2";
configuration["TaskHosting:PollingFrequencyInSeconds"] = "1";
Copy link
Contributor

Choose a reason for hiding this comment

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

Are there other operation specific settings which this line makes not needed?

@LTA-Thinking
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@LTA-Thinking LTA-Thinking merged commit e418457 into main Jan 26, 2026
61 of 68 checks passed
@LTA-Thinking LTA-Thinking deleted the personal/rojo/speedup-tests branch January 26, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Azure API for FHIR Label denotes that the issue or PR is relevant to the Azure API for FHIR Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs Bug Bug bug bug. Enhancement Enhancement on existing functionality. No-ADR ADR not needed No-PaaS-breaking-change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants