community.mysql icon indicating copy to clipboard operation
community.mysql copied to clipboard

GHA: add testing against MariaDB 10.11 LTS

Open laurent-indermuehle opened this issue 2 months ago • 3 comments

SUMMARY

The collection should test every LTS version of MariaDB. Currently 10.11 is not tested

ISSUE TYPE
  • Feature Idea
COMPONENT NAME

tests

ADDITIONAL INFORMATION

In order to limit the number of tests, I think when we add a new version of a component, the collection should use only latests version of the other components. Namely Python, ansible-core and the connectors. So the new matrix could be something like this:

strategy:
  matrix:
    ansible:
      - stable-2.15
      - stable-2.16
      - stable-2.17
      - devel
    db_engine_name:
      - mysql
      - mariadb
    db_engine_version:
      - '5.7.40'
      - '8.0.31'
      - '10.4.27'
      - '10.5.18'
      - '10.6.11'
    python:
      - '3.8'
      - '3.9'
      - '3.10'
    connector_name:
      - pymysql
      - mysqlclient
    connector_version:
      - '0.7.11'  # Released Apr 05, 2017
      - '0.9.3'    # Released Dec 18, 2018
      - '1.0.2'
      - '2.0.1'
      - '2.0.3'
      - '2.1.1'
    include:
      - db_engine_version: 10.11.7
        db_engine_name: mariadb
        ansible: stable-2.17
        python: '3.10' 
        connector_version: '0.9.3'
        connector_name: pymysql

      - db_engine_version: 10.11.7
        db_engine_name: mariadb
        ansible: stable-2.17
        python: '3.10' 
        connector_version: '2.1.1'
        connector_name: mysqlclient

laurent-indermuehle avatar Apr 10 '24 08:04 laurent-indermuehle

A few points to mention:

  • We might not test against devel. It's good but it's not a requirement
  • We could also kick out all MySQL/MariaDB versions in between the oldest and the newest we support (same for connectors). So in this particular case we could kick out 10.5.18 and some connector versions.

Does it make sense?

Andersson007 avatar Apr 10 '24 09:04 Andersson007

Does it make sense?

10.5.x is also an LTS release.
Dropping every version that is EOL makes most sense imho.

  • https://endoflife.date/mysql
  • https://endoflife.date/mariadb

markuman avatar Apr 10 '24 17:04 markuman

Thank you @Andersson007 and @markuman for your replies.

Good idea to limit to latest Ansible. I've updated the matrix accordingly. I'll work on a PR soon if nobody offer a better idea.

MariaDB 10.4 support ends in 2 months. After that we can remove it.

laurent-indermuehle avatar Apr 11 '24 07:04 laurent-indermuehle