Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI matrix update #226

Merged
merged 7 commits into from
Oct 12, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 16 additions & 1 deletion .github/workflows/ansible-test-plugins.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ jobs:
- stable-2.9
- stable-2.10
- stable-2.11
- stable-2.12
- devel
steps:

Expand Down Expand Up @@ -61,16 +62,29 @@ jobs:
- stable-2.9
- stable-2.10
- stable-2.11
#- devel
- stable-2.12
- devel
python:
- 3.6
- 3.8
connector:
- pymysql==0.7.10
- pymysql==0.9.3
- mysqlclient==2.0.1
exclude:
- mysql: 8.0.22
connector: pymysql==0.7.10
- python: 3.8
ansible: stable-2.9
- python: 3.8
ansible: stable-2.10
- python: 3.8
ansible: stable-2.11
- python: 3.6
ansible: stable-2.12
- python: 3.6
ansible: devel

steps:

- name: Check out code
Expand Down Expand Up @@ -116,6 +130,7 @@ jobs:
- stable-2.9
- stable-2.10
- stable-2.11
- stable-2.12
- devel

steps:
Expand Down
21 changes: 21 additions & 0 deletions tests/integration/targets/setup_mysql/tasks/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,27 @@
- name: "{{ role_name }} | install | install python packages"
pip:
name: "{{ python_packages }}"
register: connector

- name: Debug connector var content
Andersson007 marked this conversation as resolved.
Show resolved Hide resolved
debug:
msg: '{{ connector }}'

- name: Extract connector.name.0 content
set_fact:
connector_name: "{{ connector.name.0 }}"

- name: Debug connector_name content
debug:
msg: '{{ connector_name }}'

- name: Extract connector version
set_fact:
connector_ver: "{{ connector_name.split('=')[2].strip() }}"

- name: Debug connector_ver var content
debug:
msg: '{{ connector_ver }}'

- name: "{{ role_name }} | install | install packages required by mysql"
apt:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,27 +11,26 @@
- name: Add fake port to config file
shell: 'echo "port = {{ fake_port }}" >> {{ config_file }}'

- name: Get pymysql version
shell: pip show pymysql | awk '/Version/ {print $2}'
register: pymysql_version

- name: Add blank line
shell: 'echo "" >> {{ config_file }}'
when: (pymysql_version.stdout | default('1000', true)) is version('0.9.3', '>=')
when:
- (connector.name.0 is search('pymysql') and connector_ver is version('0.9.3', '>=')) or connector.name.0 is not search('pymysql')

- name: Create include_dir
file:
path: '{{ include_dir }}'
state: directory
mode: '0777'
when: (pymysql_version.stdout | default('1000', true)) is version('0.9.3', '>=')
when:
- (connector.name.0 is search('pymysql') and connector_ver is version('0.9.3', '>=')) or connector.name.0 is not search('pymysql')

- name: Add include_dir
lineinfile:
path: '{{ config_file }}'
line: '!includedir {{ include_dir }}'
insertafter: EOF
when: (pymysql_version.stdout | default('1000', true)) is version('0.9.3', '>=')
when:
- (connector.name.0 is search('pymysql') and connector_ver is version('0.9.3', '>=')) or connector.name.0 is not search('pymysql')

- name: Create database using fake port to connect to, must fail
mysql_db:
Expand Down
7 changes: 2 additions & 5 deletions tests/integration/targets/test_mysql_db/tasks/issue-28.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@
block:

# ============================================================
- shell: pip show pymysql | awk '/Version/ {print $2}'
register: pymysql_version

- name: get server certificate
copy:
content: "{{ lookup('pipe', \"openssl s_client -starttls mysql -connect localhost:3307 -showcerts 2>/dev/null </dev/null | sed -ne '/-BEGIN CERTIFICATE-/,/-END CERTIFICATE-/p'\") }}"
Expand Down Expand Up @@ -49,12 +46,12 @@
- assert:
that:
- result is failed
when: pymysql_version.stdout != ""
when: connector.name.0 is search('pymysql')

- assert:
that:
- result is succeeded
when: pymysql_version.stdout == ""
when: connector.name.0 is not search('pymysql')

- name: attempt connection with newly created user ignoring hostname
mysql_db:
Expand Down
7 changes: 2 additions & 5 deletions tests/integration/targets/test_mysql_info/tasks/issue-28.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@
block:

# ============================================================
- shell: pip show pymysql | awk '/Version/ {print $2}'
register: pymysql_version

- name: get server certificate
copy:
content: "{{ lookup('pipe', \"openssl s_client -starttls mysql -connect localhost:3307 -showcerts 2>/dev/null </dev/null | sed -ne '/-BEGIN CERTIFICATE-/,/-END CERTIFICATE-/p'\") }}"
Expand Down Expand Up @@ -47,12 +44,12 @@
- assert:
that:
- result is failed
when: pymysql_version.stdout != ""
when: connector.name.0 is search('pymysql')

- assert:
that:
- result is succeeded
when: pymysql_version.stdout == ""
when: connector.name.0 is not search('pymysql')

- name: attempt connection with newly created user ignoring hostname
mysql_info:
Expand Down
7 changes: 2 additions & 5 deletions tests/integration/targets/test_mysql_query/tasks/issue-28.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@
block:

# ============================================================
- shell: pip show pymysql | awk '/Version/ {print $2}'
register: pymysql_version

- name: get server certificate
copy:
content: "{{ lookup('pipe', \"openssl s_client -starttls mysql -connect localhost:3307 -showcerts 2>/dev/null </dev/null | sed -ne '/-BEGIN CERTIFICATE-/,/-END CERTIFICATE-/p'\") }}"
Expand Down Expand Up @@ -47,12 +44,12 @@
- assert:
that:
- result is failed
when: pymysql_version.stdout != ""
when: connector.name.0 is search('pymysql')

- assert:
that:
- result is succeeded
when: pymysql_version.stdout == ""
when: connector.name.0 is not search('pymysql')

- name: attempt connection with newly created user ignoring hostname
mysql_query:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@
block:

# ============================================================
- shell: pip show pymysql | awk '/Version/ {print $2}'
register: pymysql_version

- name: get server certificate
copy:
content: "{{ lookup('pipe', \"openssl s_client -starttls mysql -connect localhost:3307 -showcerts 2>/dev/null </dev/null | sed -ne '/-BEGIN CERTIFICATE-/,/-END CERTIFICATE-/p'\") }}"
Expand Down Expand Up @@ -48,12 +45,12 @@
- assert:
that:
- result is failed
when: pymysql_version.stdout != ""
when: connector.name.0 is search('pymysql')

- assert:
that:
- result is succeeded
when: pymysql_version.stdout == ""
when: connector.name.0 is not search('pymysql')

- name: attempt connection with newly created user ignoring hostname
mysql_replication:
Expand Down
7 changes: 2 additions & 5 deletions tests/integration/targets/test_mysql_user/tasks/issue-28.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@
block:

# ============================================================
- shell: pip show pymysql | awk '/Version/ {print $2}'
register: pymysql_version

- name: get server certificate
copy:
content: "{{ lookup('pipe', \"openssl s_client -starttls mysql -connect localhost:3307 -showcerts 2>/dev/null </dev/null | sed -ne '/-BEGIN CERTIFICATE-/,/-END CERTIFICATE-/p'\") }}"
Expand Down Expand Up @@ -50,12 +47,12 @@
- assert:
that:
- result is failed
when: pymysql_version.stdout != ""
when: connector.name.0 is search('pymysql')

- assert:
that:
- result is succeeded
when: pymysql_version.stdout == ""
when: connector.name.0 is not search('pymysql')

- name: attempt connection with newly created user ignoring hostname
mysql_user:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,6 @@
block:

# ============================================================
- shell: pip show pymysql | awk '/Version/ {print $2}'
register: pymysql_version

- name: get server certificate
copy:
content: "{{ lookup('pipe', \"openssl s_client -starttls mysql -connect localhost:3307 -showcerts 2>/dev/null </dev/null | sed -ne '/-BEGIN CERTIFICATE-/,/-END CERTIFICATE-/p'\") }}"
Expand Down Expand Up @@ -48,12 +45,12 @@
- assert:
that:
- result is failed
when: pymysql_version.stdout != ""
when: connector.name.0 is search('pymysql')

- assert:
that:
- result is succeeded
when: pymysql_version.stdout == ""
when: connector.name.0 is not search('pymysql')

- name: attempt connection with newly created user ignoring hostname
mysql_variables:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -151,9 +151,6 @@
# ============================================================
# Verify mysql_variable fails when setting an incorrect value (out of range)
#
- shell: pip show pymysql | awk '/Version/ {print $2}'
register: pymysql_version

- name: set mysql variable value to a number out of range
mysql_variables:
<<: *mysql_params
Expand All @@ -163,10 +160,10 @@
ignore_errors: true

- include: assert_var.yml changed=true output={{ oor_result }} var_name=max_connect_errors var_value=1
when: pymysql_version.stdout == ""
when: connector.name.0 is not search('pymysql')

- include: assert_fail_msg.yml output={{ oor_result }} msg='Truncated incorrect'
when: pymysql_version.stdout != ""
when: connector.name.0 is search('pymysql')

# ============================================================
# Verify mysql_variable fails when setting an incorrect value (incorrect type)
Expand Down