unixPB: improves Ant-Contrib role compatibility

[mahdipub]

It helps improve Ant-Contrib role by replacing wget with get_url module. Also when extracing, on macos the unarchive module might fails as it asks for gtar but default macos tar is bsdtar. The workaround is to run it as command on macos machines.

Signed-off-by: [email protected]

Checklist

• commit message has one of the standard prefixes
• faq.md updated if appropriate
• other documentation is changed or added (if applicable)
• playbook changes run through VPC or QPC (if you have access)
• VPC/QPC not applicable for this PR
• for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

[karianna]

@mahdipub linter failures

[sxa]

Given the comment above the line, This needs to be tested on Solaris before it can be approved.
VPC run at https://ci.adoptium.net/view/Tooling/job/VagrantPlaybookCheck/1969/

[mahdipub]

@mahdipub linter failures

Thanks, that's resolved.

[karianna]

Given the comment above the line, This needs to be tested on Solaris before it can be approved. VPC run at https://ci.adoptium.net/view/Tooling/job/VagrantPlaybookCheck/1969/

Don't see Solaris as part of that run?

[sxa]

Given the comment above the line, This needs to be tested on Solaris before it can be approved. VPC run at https://ci.adoptium.net/view/Tooling/job/VagrantPlaybookCheck/1969/

Don't see Solaris as part of that run?

😕 Solaris10 is in there and green

[karianna]

Some of the runs failed but solaris passed.

[sxa]

Tagging in @gdams for a review since he put in the use of wget for Solaris as part of #1585 but maybe the underlying problem was resolved in more recent versions of ansible

[sxa]

Seems a bit of a change to be moving from wget to an ansible module but also switching from unarchive to a custom command on macos. Can you clarify where you've seen an error with this, as I don't think it's been a problem for us in our playbook executions.
Would be good to remove the ? but on the URL before putting this in as it's unnecessary.

[sxa]

New VPC run: https://ci.adoptium.net/job/VagrantPlaybookCheck/2149/

[sxa]

New VPC run: https://ci.adoptium.net/job/VagrantPlaybookCheck/2149/

@mahdipub Assuming you still want to get this in, can you rebase your changes on top of master as there's some fixes we need in there to be able to run these checks

[mahdipub]

@sxa that is done.

[sxa]

Running another VPC at https://ci.adoptium.net/job/VagrantPlaybookCheck/2150/