azure mysql postgres auto discovery docs#15986
Merged
GavinFrazar merged 17 commits intoSep 20, 2022
Merged
Conversation
GavinFrazar
added
database-access
github-actions
Bot
requested review from
ptgott,
r0mant,
sagesyr,
xinding33 and
zmb3
Base automatically changed from
gavinfrazar/azure_mysql_postgres_auto_discovery_watchers
to
master
August 30, 2022 23:42
GavinFrazar
changed the title
GavinFrazar
force-pushed
the
gavinfrazar/azure_mysql_postgres_auto_discovery_docs
branch
from
fe61996 to
a2eb122
Compare
GavinFrazar
force-pushed
the
gavinfrazar/azure_mysql_postgres_auto_discovery_docs
branch
from
13161aa to
81f5d60
Compare
GavinFrazar
force-pushed
the
gavinfrazar/azure_mysql_postgres_auto_discovery_docs
branch
5 times, most recently
from
e0cebb9 to
4aec549
Compare
greedy52
reviewed
Sep 7, 2022
Contributor
greedy52
left a comment
greedy52
left a comment
There was a problem hiding this comment.
Probably should blur or cross out real IDs (subscription ID, object ID) in the screenshot.
Co-authored-by: STeve (Xin) Huang <xin.huang@goteleport.com>
greedy52
approved these changes
Sep 7, 2022
smallinsky
approved these changes
Sep 13, 2022
alexfornuto
reviewed
Sep 19, 2022
Co-authored-by: Alex Fornuto <alex.fornuto@goteleport.com>
Co-authored-by: Alex Fornuto <alex.fornuto@goteleport.com>
Co-authored-by: Alex Fornuto <alex.fornuto@goteleport.com>
alexfornuto
approved these changes
Sep 19, 2022
Contributor
alexfornuto
left a comment
alexfornuto
left a comment
There was a problem hiding this comment.
I haven't tested the changes here, but the writing is 👍.
github-actions
Bot
removed request for
ptgott,
r0mant,
sagesyr,
xinding33 and
zmb3
Contributor
|
@GavinFrazar See the table below for backport results.
|
GavinFrazar
deleted the
gavinfrazar/azure_mysql_postgres_auto_discovery_docs
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates our Azure MySQL/Postgres docs to use database auto-discovery.
Branched off #15745 so github will retarget to master when that PR merges.
Changes:
teleport db configure createcommand to make the db config using auto-discovery instead of static configThis is part of the larger issue: #14688
Next steps will be to add bootstrap configurators to remove the manual IAM setup steps, although this will require significantly more work so for now the docs will just explain how to setup Azure IAM permissions manually.