Changes to SqlServiceAccount

- Made the description of parameter RestartService more descriptive (issue dsccommunity#960).
johlju · Dec 23, 2017 · 7f4926b · 7f4926b
1 parent 65c1dc2
commit 7f4926b
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,8 @@
 - Changes to SqlServiceAccount
   - Default services are now properly detected
     ([issue #930](https://github.com/PowerShell/SqlServerDsc/issues/930)).
+  - Made the description of parameter RestartService more descriptive
+    ([issue #960](https://github.com/PowerShell/SqlServerDsc/issues/960)).
 
 ## 10.0.0.0
 

diff --git a/DSCResources/MSFT_SqlServiceAccount/MSFT_SqlServiceAccount.psm1 b/DSCResources/MSFT_SqlServiceAccount/MSFT_SqlServiceAccount.psm1
@@ -92,7 +92,8 @@ function Get-TargetResource
         Credential of the service account that should be used.
 
     .PARAMETER RestartService
-        Determines whether the service is automatically restarted.
+        Determines whether the service is automatically restarted when a change
+        to the configuration was needed.
 
     .PARAMETER Force
         Forces the service account to be updated.
@@ -164,7 +165,8 @@ function Test-TargetResource
         Credential of the service account that should be used.
 
     .PARAMETER RestartService
-        Determines whether the service is automatically restarted.
+        Determines whether the service is automatically restarted when a change
+        to the configuration was needed.
 
     .PARAMETER Force
         Forces the service account to be updated.

diff --git a/DSCResources/MSFT_SqlServiceAccount/MSFT_SqlServiceAccount.schema.mof b/DSCResources/MSFT_SqlServiceAccount/MSFT_SqlServiceAccount.schema.mof
@@ -5,6 +5,6 @@ class MSFT_SqlServiceAccount : OMI_BaseResource
     [Key, Description("Name of the SQL instance.")] String InstanceName;
     [Key, Description("Type of service being managed."), ValueMap {"DatabaseEngine","SQLServerAgent","Search","IntegrationServices","AnalysisServices","ReportingServices","SQLServerBrowser","NotificationServices"}, Values {"DatabaseEngine","SQLServerAgent","Search","IntegrationServices","AnalysisServices","ReportingServices","SQLServerBrowser","NotificationServices"}] String ServiceType;
     [Required, EmbeddedInstance("MSFT_Credential"), Description("The service account that should be used when running the service.")] String ServiceAccount;
-    [Write, Description("Determines whether the service is automatically restarted.")] Boolean RestartService;
+    [Write, Description("Determines whether the service is automatically restarted when a change to the configuration was needed.")] Boolean RestartService;
     [Write, Description("Forces the service account to be updated. Useful for password changes.")] Boolean Force;
 };
diff --git a/README.md b/README.md
@@ -1254,7 +1254,7 @@ Manage the service account for SQL Server services.
 * **`[PSCredential]` ServiceAccount** (Required): The service account that should
   be used when running the service.
 * **`[Boolean]` RestartService** (Write): Determines whether the service is
-  automatically restarted.
+  automatically restarted when a change to the configuration was needed.
 * **`[Boolean]` Force** (Write): Forces the service account to be updated.
   Useful for password changes. This will cause `Set-TargetResource` to be run on
   each consecutive run.