Stephen Johnson

Please note I'm not a go coder, I've no idea how terraform works under the hood and this is a total guess on how to add the argument. This relates...

I've had to hack this ```name: Create terraform plan on: pull_request: branches: [main] concurrency: group: terraform-lock jobs: plan: runs-on: ubuntu-latest name: Plan env: TERRAFORM_ACTIONS_GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} steps: - name:...

I'm happy to merge this just waiting on the response to @joelesalas comment.

Is everyone ok with just add that one property then?

The current master , has a work around for this. Im not sure if this is the correct way to do this. As now we have to keep the GPG...

I'll get to this on the weekend, sorry about the delay.

I'm holding off merging this until you have made the changes, outlined above.

The last think we need is some tests as this is a massive change. Have a look in spec/acceptance/ and we don't have any spec tests to test both branches...

@gavinrogers If you look at the spec_helper_acceptance if osfamily == 'RedHat' on host, puppet('module', 'install', 'stahnma-epel'), { :acceptable_exit_codes => [0,1] } on host, puppet('apply', '-e', '"class {epel:}"'), { :acceptable_exit_codes =>...

The issues are caused by your merges not being clean this really needs rebasing on matser.