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.

Sign up for GitHub

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

Jump to bottom

adding log output to reindex tests #3278

Merged
merged 1 commit into from
May 3, 2023
Merged

adding log output to reindex tests #3278

merged 1 commit into from
May 3, 2023

Conversation

v-mserdar
Copy link
Contributor

Description

After seeing our reindex tests fail without enough information, this adds the same output as doing a GET on the reindex id. Hopefully this will give us more idea of why we have transient reindex tests.

Related issues

Addresses reindex tests.

Testing

Ran reindex 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.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

@v-mserdar v-mserdar added Enhancement-Test Enhancement on tests. 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 labels May 2, 2023
@v-mserdar v-mserdar added this to the S114 milestone May 2, 2023
@v-mserdar v-mserdar requested a review from a team as a code owner May 2, 2023 22:18
@v-mserdar v-mserdar merged commit d0b205a into main May 3, 2023
@v-mserdar v-mserdar deleted the personal/v-mserdar/reindex_test_log_improvement branch May 3, 2023 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PTaladay PTaladay PTaladay approved these changes

@fhibf fhibf fhibf approved these changes

Assignees
No one assigned
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 Enhancement-Test Enhancement on tests.
Projects
None yet
Milestone
S114
Development

Successfully merging this pull request may close these issues.
3 participants
@v-mserdar @fhibf @PTaladay