Compare commits

...

6 commits

Author SHA1 Message Date
simonLeary42
c351c4249a
Merge eead50b287 into ed3d322fd5 2025-03-26 19:36:11 +00:00
Abhijeet Kasurde
eead50b287
Apply suggestions from code review 2025-03-26 12:36:08 -07:00
Simon Leary
4ff0e3aa13 remove assertion of changed 2025-03-26 15:20:42 -04:00
Simon Leary
979f00ce75 fix lint errors 2025-03-26 15:18:35 -04:00
softwarefactory-project-zuul[bot]
ed3d322fd5
Merge pull request #617 from Akasurde/ci_fix
[CI] update test containers

SUMMARY
Signed-off-by: Abhijeet Kasurde Akasurde@redhat.com
ISSUE TYPE

Bugfix Pull Request

Reviewed-by: Matt Clay
2025-03-26 19:00:30 +00:00
Abhijeet Kasurde
d9f54eb9d4 [CI] update test containers
Signed-off-by: Abhijeet Kasurde <Akasurde@redhat.com>
2025-03-26 09:02:59 -07:00
3 changed files with 21 additions and 25 deletions

View file

@ -122,14 +122,14 @@ stages:
parameters: parameters:
testFormat: devel/linux/{0}/1 testFormat: devel/linux/{0}/1
targets: targets:
- name: Fedora 40 - name: Fedora 41
test: fedora40 test: fedora41
- name: Ubuntu 22.04 - name: Ubuntu 22.04
test: ubuntu2204 test: ubuntu2204
- name: Ubuntu 24.04 - name: Ubuntu 24.04
test: ubuntu2404 test: ubuntu2404
- stage: Docker_2_18 - stage: Docker_2_18
displayName: Docker devel displayName: Docker 2.18
dependsOn: [] dependsOn: []
jobs: jobs:
- template: templates/matrix.yml - template: templates/matrix.yml
@ -201,14 +201,14 @@ stages:
parameters: parameters:
testFormat: devel/{0}/1 testFormat: devel/{0}/1
targets: targets:
- name: RHEL 9.4 - name: RHEL 9.5
test: rhel/9.4 test: rhel/9.5
- name: FreeBSD 14.1 - name: FreeBSD 14.2
test: freebsd/14.1 test: freebsd/14.2
- name: FreeBSD 13.4 - name: FreeBSD 13.5
test: freebsd/13.4 test: freebsd/13.5
- stage: Remote_2_18 - stage: Remote_2_18
displayName: Remote devel displayName: Remote 2.18
dependsOn: [] dependsOn: []
jobs: jobs:
- template: templates/matrix.yml - template: templates/matrix.yml

View file

@ -134,7 +134,7 @@
register: bind_result_freebsd_dry_run register: bind_result_freebsd_dry_run
check_mode: true check_mode: true
- name: attempt to stat bind mounted file - name: Attempt to stat bind mounted file
ansible.builtin.stat: ansible.builtin.stat:
path: '{{ output_dir }}/mount_dest/test_file' path: '{{ output_dir }}/mount_dest/test_file'
when: ansible_system in ('FreeBSD', 'Linux') when: ansible_system in ('FreeBSD', 'Linux')
@ -143,11 +143,7 @@
- name: Assert the bind mount did not take place - name: Assert the bind mount did not take place
ansible.builtin.assert: ansible.builtin.assert:
that: that:
- linux_and_changed or freebsd_and_changed
- not dest_stat['stat']['exists'] - not dest_stat['stat']['exists']
vars:
linux_and_changed: "{{ ansible_system == 'Linux' and bind_result_linux_dry_run['changed'] }}"
freebsd_and_changed: "{{ ansible_system == 'FreeBSD' and bind_result_freebsd['changed'] }}"
when: ansible_system in ('FreeBSD', 'Linux') when: ansible_system in ('FreeBSD', 'Linux')
# bind mount # bind mount
@ -330,12 +326,12 @@
when: ansible_system in ('FreeBSD', 'Linux') when: ansible_system in ('FreeBSD', 'Linux')
# SWAP ############################################################# # SWAP #############################################################
- name: swap - name: Swap
when: ansible_system in ('Linux') when: ansible_system in ('Linux')
block: block:
# mount swap check mode # mount swap check mode
- name: Stat /etc/fstab - name: Stat /etc/fstab
stat: ansible.builtin.stat:
path: /etc/fstab path: /etc/fstab
register: stat_fstab_before register: stat_fstab_before
@ -349,7 +345,7 @@
check_mode: true check_mode: true
- name: Stat /etc/fstab - name: Stat /etc/fstab
stat: ansible.builtin.stat:
path: /etc/fstab path: /etc/fstab
register: stat_fstab_after register: stat_fstab_after
@ -410,7 +406,7 @@
# remove swap check mode # remove swap check mode
- name: Stat /etc/fstab - name: Stat /etc/fstab
stat: ansible.builtin.stat:
path: /etc/fstab path: /etc/fstab
register: stat_fstab_before register: stat_fstab_before
@ -422,7 +418,7 @@
check_mode: true check_mode: true
- name: Stat /etc/fstab - name: Stat /etc/fstab
stat: ansible.builtin.stat:
path: /etc/fstab path: /etc/fstab
register: stat_fstab_after register: stat_fstab_after
@ -474,7 +470,7 @@
- not swap2_removed_again['changed'] - not swap2_removed_again['changed']
# FIXUP ############################################################# # FIXUP #############################################################
- name: fix incomplete entry already present in fstab - name: Fix incomplete entry already present in fstab
when: ansible_system == 'Linux' when: ansible_system == 'Linux'
block: block:
- name: Create fstab record with missing last two fields - name: Create fstab record with missing last two fields
@ -834,7 +830,7 @@
failed_when: false failed_when: false
changed_when: false changed_when: false
- name: Assert the mount occured and the fstab is unchanged - name: Assert the mount occurred and the fstab is unchanged
ansible.builtin.assert: ansible.builtin.assert:
that: that:
- check_mountinfo.stdout|int == 0 - check_mountinfo.stdout|int == 0

View file

@ -62,15 +62,15 @@ else
retry pip install "https://github.com/ansible/ansible/archive/stable-${ansible_version}.tar.gz" --disable-pip-version-check retry pip install "https://github.com/ansible/ansible/archive/stable-${ansible_version}.tar.gz" --disable-pip-version-check
fi fi
export ANSIBLE_COLLECTIONS_PATHS="${PWD}/../../../" export ANSIBLE_COLLECTIONS_PATH="${PWD}/../../../"
# START: HACK install dependencies # START: HACK install dependencies
if [ "${ansible_version}" == "2.9" ] || [ "${ansible_version}" == "2.10" ]; then if [ "${ansible_version}" == "2.9" ] || [ "${ansible_version}" == "2.10" ]; then
# Note: Since community.general 5.x, Ansible Core versions prior to 2.11 are not supported. # Note: Since community.general 5.x, Ansible Core versions prior to 2.11 are not supported.
# So we need to use 4.8.1 for Ansible 2.9 and Ansible Engine 2.10. # So we need to use 4.8.1 for Ansible 2.9 and Ansible Engine 2.10.
retry git clone --depth=1 --single-branch -b 4.8.1 https://github.com/ansible-collections/community.general.git "${ANSIBLE_COLLECTIONS_PATHS}/ansible_collections/community/general" retry git clone --depth=1 --single-branch -b 4.8.1 https://github.com/ansible-collections/community.general.git "${ANSIBLE_COLLECTIONS_PATH}/ansible_collections/community/general"
else else
retry git clone --depth=1 --single-branch https://github.com/ansible-collections/community.general.git "${ANSIBLE_COLLECTIONS_PATHS}/ansible_collections/community/general" retry git clone --depth=1 --single-branch https://github.com/ansible-collections/community.general.git "${ANSIBLE_COLLECTIONS_PATH}/ansible_collections/community/general"
fi fi
# Note: we're installing with git to work around Galaxy being a huge PITA (https://github.com/ansible/galaxy/issues/2429) # Note: we're installing with git to work around Galaxy being a huge PITA (https://github.com/ansible/galaxy/issues/2429)
# END: HACK # END: HACK