mirror of
https://github.com/linux-system-roles/network.git
synced 2026-01-23 10:25:28 +00:00
ci: use ANSIBLE_INJECT_FACT_VARS=false by default for testing
Ansible 2.20 has deprecated the use of Ansible facts as variables. For example, `ansible_distribution` is now deprecated in favor of `ansible_facts["distribution"]`. This is due to making the default setting `INJECT_FACTS_AS_VARS=false`. For now, this will create WARNING messages, but in Ansible 2.24 it will be an error. In order to ensure that commits and PRs conform to this, use ANSIBLE_INJECT_FACT_VARS=false by default in our CI testing. Update README-ostree.md if needed. See https://docs.ansible.com/projects/ansible/latest/porting_guides/porting_guide_core_2.20.html#inject-facts-as-vars Signed-off-by: Rich Megginson <rmeggins@redhat.com>
This commit is contained in:
parent
1c56680451
commit
c22e1fcd46
3 changed files with 4 additions and 2 deletions
|
|
@ -44,6 +44,7 @@ jobs:
|
|||
|
||||
env:
|
||||
TOX_ARGS: "--skip-tags tests::infiniband,tests::nvme,tests::scsi"
|
||||
ANSIBLE_INJECT_FACT_VARS: "false"
|
||||
|
||||
steps:
|
||||
- name: Checkout repo
|
||||
|
|
|
|||
1
.github/workflows/tft.yml
vendored
1
.github/workflows/tft.yml
vendored
|
|
@ -168,6 +168,7 @@ jobs:
|
|||
SR_ARTIFACTS_DIR=${{ steps.set_vars.outputs.ARTIFACTS_DIR }};\
|
||||
SR_TEST_LOCAL_CHANGES=false;\
|
||||
SR_LSR_USER=${{ vars.SR_LSR_USER }};\
|
||||
SR_ANSIBLE_INJECT_FACT_VARS=false;\
|
||||
SR_ARTIFACTS_URL=${{ steps.set_vars.outputs.ARTIFACTS_URL }}"
|
||||
# Note that LINUXSYSTEMROLES_SSH_KEY must be single-line, TF doesn't read multi-line variables fine.
|
||||
secrets: "SR_LSR_DOMAIN=${{ secrets.SR_LSR_DOMAIN }};\
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue