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

[Wiki] Add further details to identity bug of RSV to documentation #2624

Merged
merged 6 commits into from
Feb 23, 2023

Conversation

fblix
Copy link
Contributor

@fblix fblix commented Jan 27, 2023

Description

This PR adds further details of the ongoing identity issues with the Recovery Services Vault.
Refer to #2391

Type of Change

  • Update to documentation

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (readme)
  • I did format my code

cc: @JPEasier

@fblix fblix added the documentation Improvements or additions to documentation label Jan 27, 2023
@fblix fblix requested a review from a team as a code owner January 27, 2023 16:23
@fblix
Copy link
Contributor Author

fblix commented Jan 27, 2023

@eriqua this seems to be a bigger issue related to Azure (either RSV PG or Bicep). How should we update your issue ticket (#2391) since the issue does not seem to be resolvable by us alone.

See issue in the bicep repo for more details.

@eriqua eriqua changed the title Add further details to identity bug of RSV to documentation [Wiki] Add further details to identity bug of RSV to documentation Jan 28, 2023
@eriqua
Copy link
Contributor

eriqua commented Jan 28, 2023

@eriqua this seems to be a bigger issue related to Azure (either RSV PG or Bicep). How should we update your issue ticket (#2391) since the issue does not seem to be resolvable by us alone.

See issue in the bicep repo for more details.

@fblix thanks for testing and opening the bug! To your question: replied directly in issue #2391

docs/wiki/Known issues.md Outdated Show resolved Hide resolved
Copy link
Contributor

@eriqua eriqua left a comment

Choose a reason for hiding this comment

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

Hi @fblix! Thanks again for your contribution and for raising the issue to the PG. Please let me know if suggested changes make sense to you or if we are waiting for anything in the main issue to be addressed before merging this in.

JPEasier and others added 2 commits February 22, 2023 14:20
Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>
JPEasier
JPEasier previously approved these changes Feb 23, 2023
Copy link
Contributor

@eriqua eriqua left a comment

Choose a reason for hiding this comment

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

🆗

@fblix fblix merged commit f9fdd78 into main Feb 23, 2023
@fblix fblix deleted the users/f_blix_jpeasier/rsvidentitybugdoc branch February 23, 2023 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation wait for author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants