ansible icon indicating copy to clipboard operation
ansible copied to clipboard

minor: Update supported Ansible versions

Open SuperQ opened this issue 1 year ago • 1 comments

Ansible versions 2.11 and older are not supportable on new platforms. Set the minimum supported Ansible version to 2.12 in order to make updating our integration testing possible.

  • Fixup minor linting issues.

SuperQ avatar Aug 19 '24 16:08 SuperQ

Docs Build 📝

This PR is closed and any previously published docsite has been unpublished.

github-actions[bot] avatar Aug 19 '24 16:08 github-actions[bot]

@gardar Fixed up, removing centos-7 here instead of #395 to make some tests pass.

SuperQ avatar Aug 20 '24 12:08 SuperQ

Ok, if you have a hint on how/where you were going to add the version matrix list, I could work on it.

SuperQ avatar Aug 21 '24 12:08 SuperQ

I'm not absolutely certain where it would be best to place the exclusion list, I don't think that it would fit in any of the ansible meta files. So I guess it would either have to be under env: in .github/workflows/ansible-ci.yml on in some new file under .github/

Something like this for example:

env:
  EXCLUSIONS: |
            - target: ubuntu-24.04
              ansible-core-version: 
                - stable-2.10
                - stable-2.11

Then we need to add it to the matrix json that's dynamically generated and used in .github/workflows/ansible-test-molecule.yml

The matrix that's currently generated ends something like this:

jobs:
  ansible-test-integration:
    runs-on: ubuntu-latest
    strategy:
      fail-fast: false
      max-parallel: 4
      matrix:
        target: [ubuntu-20.04, ubuntu-22.04, ubuntu-24.04]
        ansible-core-version: [stable-2.10, stable-2.11, stable-2.12, stable-2.13]

but after adding the excludes it would end something like this:

jobs:
  ansible-test-integration:
    runs-on: ubuntu-latest
    strategy:
      fail-fast: false
      max-parallel: 4
      matrix:
        target: [ubuntu-20.04, ubuntu-22.04, ubuntu-24.04]
        ansible-core-version: [stable-2.10, stable-2.11, stable-2.12, stable-2.13]
        exclude:
          - target: ubuntu-24.04
            ansible-core-version: 
              - stable-2.10
              - stable-2.11

gardar avatar Aug 21 '24 13:08 gardar

Closing in favor of https://github.com/prometheus-community/ansible/pull/416.

SuperQ avatar Sep 14 '24 12:09 SuperQ