Skip to content

Only copy cert/key files if tls is `true`

Compare
Choose a tag to compare
@richm richm released this 24 Aug 16:11

Fix test scripts for BaseOS CI - removing unnecessary "become: true" from test scripts.

Fix test scripts for BaseOS CI - fixing the invalid expected value of tls.permittedpeer in tests_relp.yml.

Update copy tasks conditions with tls true

Playbook fails when certificates are not used.
This patch adds condition for the copy tasks to perform only when tls is set to true.

remove trailing spaces

remove trailing spaces to fix linter issues

Put multiple tasks under the condition,

item.tls is defined | ternary(item.tls, item.use_cert | d(true))
in one block.

Fixing a bug in "Check certs - key/certs data are provided, but tls is

false" found by @avlitman.

This case must have failed the Check certs task.

tls is false and certs+key defined: task was skipped.

Allowing the case, tls is false and key/certs vars are configured.

  • The case used to fail in the checking task with emitting an error.
    The error is transformed to a warning.
  • For the test case, TEST CASE 2 in tests_files_elasticsearch.yml
    is added.

In addition, some minor bugs in tests_files_elasticsearch.yml are
fixed.

ref: bz1994580