feat: add compatibility for dbops v0.9

lowlydba · Feb 24, 2024 · 98fbe57 · 98fbe57
1 parent 740f4b2
commit 98fbe57
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -1,4 +1,4 @@
-# lowlydba.sqlserver Collection for Ansible
+# lowlydba.sqlserver Collection for Ansible<!-- omit in toc -->
 
 ![GPL v3](https://img.shields.io/github/license/lowlydba/lowlydba.sqlserver)
 [![CI](https://github.com/lowlydba/lowlydba.sqlserver/actions/workflows/ansible-test.yml/badge.svg)](https://github.com/lowlydba/lowlydba.sqlserver/actions/workflows/ansible-test.yml)
@@ -11,11 +11,17 @@
 - [Contributing to this collection](#contributing-to-this-collection)
 - [Collection maintenance](#collection-maintenance)
 - [Tested with](#tested-with)
+  - [Ansible](#ansible)
+  - [SQL Server](#sql-server)
 - [External requirements](#external-requirements)
 - [Using this collection](#using-this-collection)
   - [Installing the Collection from Ansible Galaxy](#installing-the-collection-from-ansible-galaxy)
-- [Release notes](#release-notes)
 - [Releasing](#releasing)
+  - [Release notes](#release-notes)
+  - [Next Release](#next-release)
+  - [Schedule](#schedule)
+  - [Versioning](#versioning)
+  - [Deprecation](#deprecation)
 
 ## Modules
 
@@ -73,7 +79,7 @@ To learn how to maintain / become a maintainer of this collection, refer to the
 
 - PowerShell modules
   - [dbatools][dbatools] >= 2.0.0
-  - [dbops][dbops] >= 0.8.0
+  - [dbops][dbops] >= 0.9.0
 
 ## Using this collection
 

diff --git a/plugins/modules/install_script.ps1 b/plugins/modules/install_script.ps1
@@ -7,15 +7,15 @@
 #AnsibleRequires -CSharpUtil Ansible.Basic
 #AnsibleRequires -PowerShell ansible_collections.lowlydba.sqlserver.plugins.module_utils._SqlServerUtils
 #Requires -Modules @{ ModuleName="dbatools"; ModuleVersion="2.0.0" }
-#Requires -Modules @{ ModuleName="dbops"; ModuleVersion="0.8.0" }
+#Requires -Modules @{ ModuleName="dbops"; ModuleVersion="0.9.0" }
 
 $spec = @{
     supports_check_mode = $true
     options = @{
         database = @{type = 'str'; required = $true }
         path = @{type = 'str'; required = $true }
         deployment_method = @{type = 'str'; required = $false; default = 'NoTransaction'
-            choices = @('NoTransaction', 'SingleTransaction', 'TransactionPerScript')
+            choices = @('NoTransaction', 'SingleTransaction', 'TransactionPerScript', 'AlwaysRollback')
         }
         schema_version_table = @{type = 'str'; required = $false }
         no_log_version = @{type = 'bool'; required = $false; default = $false }

diff --git a/plugins/modules/install_script.py b/plugins/modules/install_script.py
@@ -33,6 +33,7 @@
       - C(SingleTransaction) - wrap all the deployment scripts into a single transaction and rollback whole deployment on error.
       - C(TransactionPerScript) - wrap each script into a separate transaction; rollback single script deployment in case of error.
       - C(NoTransaction) - deploy as is.
+      - C(AlwaysRollback) - roll back the transaction.
     type: str
     required: false
     default: 'NoTransaction'

diff --git a/tests/integration/targets/setup_sqlserver/vars/main.yml b/tests/integration/targets/setup_sqlserver/vars/main.yml
@@ -6,7 +6,7 @@ dbatools_install_cmd: >
       Set-DbatoolsInsecureConnection -Scope FileUserLocal
   }
 
-dbops_min_version: 0.8.0
+dbops_min_version: 0.9.0
 dbops_install_cmd: >
   if (-not(Get-Module -FullyQualifiedName @{ModuleName='dbops';ModuleVersion='{{ dbops_min_version }}'} -ListAvailable)) {
       Install-Module dbops -MinimumVersion {{ dbops_min_version }} -Force -SkipPublisherCheck -AllowClobber