Skip to content

sap_ha_install_anydb_ibmdb2: linting and sles bug fixes#803

Merged
berndfinger merged 11 commits into
sap-linuxlab:devfrom
sean-freeman:ha_ibmdb2_lint
Jul 17, 2024
Merged

sap_ha_install_anydb_ibmdb2: linting and sles bug fixes#803
berndfinger merged 11 commits into
sap-linuxlab:devfrom
sean-freeman:ha_ibmdb2_lint

Conversation

@sean-freeman
Copy link
Copy Markdown
Member

@sean-freeman sean-freeman commented Jul 15, 2024

Added linting fixes and appended additional logic to provide further idempotency

At request and in parallel, tested with SLES 15 SP5 + IBM Db2 for LUW 11.5 MP8 FP0 SAP2 LINUX x86_64 and provided bug fixes accordingly. Db2 version was chosen to assess compatibility of older release with latest patched OS

@sean-freeman sean-freeman requested a review from berndfinger July 15, 2024 13:30
Comment thread roles/sap_ha_install_anydb_ibmdb2/tasks/db2_hadr_pcmk_cluster_create.yml Outdated
Comment thread roles/sap_ha_install_anydb_ibmdb2/tasks/passwordless_ssh.yml Outdated
Comment thread roles/sap_ha_install_anydb_ibmdb2/tasks/passwordless_ssh.yml Outdated
Comment thread roles/sap_ha_install_anydb_ibmdb2/tasks/platform/db2cm_cloud_msazure_vm.yml Outdated
Comment thread roles/sap_ha_install_anydb_ibmdb2/tasks/platform/db2cm_cloud_msazure_vm.yml Outdated
@berndfinger
Copy link
Copy Markdown
Member

@sean-freeman I found several tasks (despite the ones I mentioned before) where the Ansible module comes after registering its output. We already have two different places for a when: condition (below name: for blocks and at the end for non-blocks) but let's please place the Ansible modules at the top of a task and put the register: line below.

Copy link
Copy Markdown
Member

@berndfinger berndfinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@berndfinger berndfinger merged commit 3c9d6da into sap-linuxlab:dev Jul 17, 2024
@sean-freeman sean-freeman deleted the ha_ibmdb2_lint branch July 17, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants