Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

Update OpenShift quickstart#9120

Closed
cblecker wants to merge 1 commit intoMicrosoftDocs:livefrom
cblecker:patch-1
Closed

Update OpenShift quickstart#9120
cblecker wants to merge 1 commit intoMicrosoftDocs:livefrom
cblecker:patch-1

Conversation

@cblecker
Copy link
Copy Markdown
Contributor

@cblecker cblecker commented Jun 29, 2023

  • Fix links to OpenShift storage docs for Azure
  • Don't suggest provisioning a custom storage class, as one comes preconfigured in OpenShift
  • Remove static fsGroup
  • Ensure fsGroup will be rewritten if it doesn't match
  • Add required NET_BIND_SERVICE capability

@prmerger-automator
Copy link
Copy Markdown
Contributor

@cblecker : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit edf8e38:

✅ Validation status: passed

File Status Preview URL Details
docs/linux/quickstart-sql-server-containers-azure.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 4426bbb:

✅ Validation status: passed

File Status Preview URL Details
docs/linux/quickstart-sql-server-containers-azure.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Jak-MS
Copy link
Copy Markdown
Contributor

Jak-MS commented Jun 29, 2023

@rwestMSFT

  • Can you review this PR?
  • IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Jun 29, 2023
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 5540324:

✅ Validation status: passed

File Status Preview URL Details
docs/linux/quickstart-sql-server-containers-azure.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@rwestMSFT
Copy link
Copy Markdown
Contributor

@cblecker Thanks for your contribution. I will ask a PM to review it in the next few days.

@amvin87
Copy link
Copy Markdown

amvin87 commented Jul 31, 2023

Hi @rwestMSFT just approved these changes we can go ahead and update the article please.

@rwestMSFT
Copy link
Copy Markdown
Contributor

rwestMSFT commented Jul 31, 2023

The changes from this PR will be included in an internal refresh. When that happens, this PR will automatically close.

@cblecker Thanks for your contribution! You can expect to see the updated article in the next day or so.

@cblecker cblecker deleted the patch-1 branch July 31, 2023 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants