Skip to content

Commit 47e74f6

Browse files
authored
Merge pull request #1156 from MartinWa/sql-auth-enhancements_documentation_fix
Updated SQL documentation example for the new enhancements
2 parents ae6569d + f59cdf1 commit 47e74f6

1 file changed

Lines changed: 4 additions & 12 deletions

File tree

  • docs/content/api-overview/resources

docs/content/api-overview/resources/sql.md

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -120,20 +120,12 @@ template
120120
open Farmer
121121
open Farmer.Builders
122122
open Sql
123-
open Farmer.Arm.Sql
124123
125-
let activeDirectoryAdmin: ActiveDirectoryAdminSettings =
126-
{
127-
Login = "adadmin"
128-
Sid = "F9D49C34-01BA-4897-B7E2-3694BF3DE2CF"
129-
PrincipalType = ActiveDirectoryPrincipalType.User
130-
AdOnlyAuth = false // when false, admin_username is required
131-
// when true admin_username is ignored
132-
}
124+
let entraAdminGroup = AccessPolicy.findGroups [ "adadmin" ] |> Array.head
133125
134126
let myDatabases = sqlServer {
135127
name "my_server"
136-
active_directory_admin (Some(activeDirectoryAdmin))
128+
entra_id_admin_group entraAdminGroup.DisplayName entraAdminGroup.Id
137129
admin_username "admin_username"
138130
enable_azure_firewall
139131
@@ -166,9 +158,9 @@ let template = arm {
166158
add_resource myDatabases
167159
}
168160
169-
template
170-
|> Writer.quickWrite "sql-example"
161+
template |> Writer.quickWrite "sql-example"
171162
172163
template
173164
|> Deploy.execute "my-resource-group" [ "password-for-my_server", "*****" ]
165+
|> ignore
174166
```

0 commit comments

Comments
 (0)