ibm-messaging / mq-ansible

Ansible roles for IBM MQ
Apache License 2.0
29 stars 26 forks source link

Linux Fixpack installation #88

Closed bimsara-yasitha01 closed 7 months ago

bimsara-yasitha01 commented 9 months ago

Provides support for Ubuntu and RHEL fixpack upgrades

JamRamPage commented 7 months ago

Just a few questions. I see that this is similar to the installmq role.

bimsara-yasitha01 commented 7 months ago

Hi @JamRamPage, thanks for the comments:

Do we need a hyphen for "" in playbooks/mq-upgrade.yml. Is this a variable that the user adds with extra-vars in their command to run this playbook?

No, in the role the variables fixpack_zip and fixpack_dir are derived from the user-provided fixpack_local variable, so don't need to be supplied to the role as well.

Below, I see us refer to "fixpack_tar" - can we be consistent with tar and zip and call it fixpack-zip above?

Good spot, I've changed fixpack_tar to fixpack_zip to avoid any errors.

I see that Linux_installmq.yml includes the following lines in two tasks. Are they not needed here?

I've now added the inverse of that condition into the upgrade role (e.g. only run when mq is already installed) to avoid errors when a developer tries to run an upgrade without an existing MQ base installation (unlikely scenario but it's covered now nonetheless).

Is there a difference in how our code treats "Linux x86" to "Linux"?

Not at the moment. If I'm following your thread you're talking about the naming of the tasks? If so, I've renamed Linux_x86_64_applyfixpack.yml to Linux_applyfixpack.yml.

bimsara-yasitha01 commented 7 months ago

@JamRamPage

  1. fixpack_local in this playbook just shows an example path, there is no default location that we can provide to the developer as all fixpacks will be grabbed from Fix Central. See AIX_ppc64le_downloadmq.yml, Linux_s390x_downloadmq.yml. The hypen (and the rest of that variable) is just a placeholder.

  2. Done.