fix: rename pull_policies to pull_policy in config.toml

[kayman-mk]

Description

The runner_worker_docker_options mentioned a pull_policies attribute which was directly used in config.toml which is wrong. The correct name is pull_policy. pull_policy is ignored by the Runner and has no effect. Instead a pull policy of always is used automatically.

This PR renames the pull_policies to pull_policy before inserting it into the configuration file. If pull_policies is used at the moment it has no effect.

Closes #1065

Migrations required

In case var.runner_worker_docker_options.pull_policy is used, rename pull_policy to pull_policies and make sure that it is a list(string), e.g. pull_policies = ["always"]

Verification

Still to be done

• pull_policies not given --> `pull_policy = ["always"] is used.
• pull_policies given --> Specified value used

[github-actions]

Hey @kayman-mk! 👋

Thank you for your contribution to the project. Please refer to the contribution rules for a quick overview of the process.

Make sure that this PR clearly explains:

• the problem being solved
• the best way a reviewer and you can test your changes

With submitting this PR you confirm that you hold the rights of the code added and agree that it will published under this LICENSE.

The following ChatOps commands are supported:

• /help: notifies a maintainer to help you out

Simply add a comment with the command in the first line. If you need to pass more information, separate it with a blank line from the command.

This message was generated automatically. You are welcome to improve it.

[github-actions]

🦙 MegaLinter status: ✅ SUCCESS

Descriptor	Linter	Files	Fixed	Errors	Elapsed time
✅ COPYPASTE	jscpd	yes		no	1.85s
✅ REPOSITORY	checkov	yes		no	16.75s
✅ REPOSITORY	dustilock	yes		no	0.35s
✅ REPOSITORY	gitleaks	yes		no	1.16s
✅ REPOSITORY	git_diff	yes		no	0.01s
✅ REPOSITORY	grype	yes		no	14.02s
✅ REPOSITORY	secretlint	yes		no	1.32s
✅ REPOSITORY	syft	yes		no	0.28s
✅ REPOSITORY	trivy-sbom	yes		no	1.46s
✅ REPOSITORY	trufflehog	yes		no	11.11s
✅ SPELL	cspell	5		0	3.29s
✅ TERRAFORM	terraform-fmt	3	0	0	0.43s
✅ TERRAFORM	terragrunt	1	0	0	0.31s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by

[kayman-mk]

I think the above code makes no sense at all. Better to rename pull_policies before inserting it into the configuration file. This way we know that it accepts a list of strings as pull_policy implies one value only (at least for me).