unixPB: Add changed_when fields to shell modules, in gcc roles

[Willsparker]

Ref: #2422

As per my comment in the referenced issue, these queries are not changing the machine state, so they shouldn't report changed.

I prioritized these GCC roles, so they accurately reflect the new GCC-10 role from #2420.

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)
• for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

[sxa]

Lets make sure this has gone through a VPC run first without problems

[Willsparker]

Agreed- I haven't set it going just yet because we have 3 runs currently going on :-)

[sxa]

Hmmm this appears to be causing failures in ansible_lint
[EDIT: Probably not specific to the PR - raised https://github.com//issues/2426]

[Willsparker]

Linter is broken, as per your issue (#2426). The docker CentOS6 run seems happy :-)

[sxa]

LGTM based on https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/1382 :-)

(I'm assuming you've verified that it does the right thing when it's already installed i.e. on the second run of the playbook)

[Willsparker]

Double checking that now :-) Thanks for running VPC

[Willsparker]

Yep, it worked when I ran it twice. These roles now only report "ok" or "skipped", which is what we want :-)