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

Added graceful error handling for child resource export #705

Merged
merged 1 commit into from
Nov 15, 2022
Merged

Added graceful error handling for child resource export #705

merged 1 commit into from
Nov 15, 2022

Conversation

daltondhcp
Copy link
Contributor

Overview/Summary

Partially adressing #699, specifically the RP-errors from the export API.
image

Testing Evidence

Tested in my own demo environment successfuly.
Test coverage for the scenarios in place. In case the specific child resources in the test cases are not exported due to RP issues, tests will obviously fail ❌

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • [X} Performed testing and provided evidence.
  • Updated relevant and associated documentation.

@daltondhcp daltondhcp requested review from a team as code owners November 15, 2022 17:50
@daltondhcp daltondhcp temporarily deployed to test November 15, 2022 17:50 Inactive
Copy link
Member

@Jefajers Jefajers left a comment

Choose a reason for hiding this comment

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

RTR♨️

@Jefajers Jefajers added this to the Release - v1.9.1 milestone Nov 15, 2022
@Jefajers Jefajers merged commit c25977f into Azure:main Nov 15, 2022
@Jefajers Jefajers added the enhancement New feature or request label Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants