-
Notifications
You must be signed in to change notification settings - Fork 43
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
ci: Use supported ansible-lint action; run ansible-lint against the c…
…ollection The old ansible-community ansible-lint is deprecated. There is a new ansible-lint github action. The latest Ansible repo gating tests run ansible-lint against the collection format instead of against individual roles. We have to convert the role to collection format before running ansible-test. This also requires tox-lsr 3.2.1 - bump other actions to use 3.2.1 Role developers can run this locally using `tox -e collection,ansible-lint-collection` See linux-system-roles/tox-lsr#125 Signed-off-by: Rich Megginson <[email protected]>
- Loading branch information
Showing
4 changed files
with
20 additions
and
30 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,9 @@ on: # yamllint disable-line rule:truthy | |
branches: | ||
- main | ||
workflow_dispatch: | ||
env: | ||
LSR_ROLE2COLL_NAMESPACE: fedora | ||
LSR_ROLE2COLL_NAME: linux_system_roles | ||
permissions: | ||
contents: read | ||
jobs: | ||
|
@@ -26,18 +29,21 @@ jobs: | |
- name: Checkout repo | ||
uses: actions/checkout@v4 | ||
|
||
- name: Fix up role meta/main.yml namespace and name | ||
- name: Install tox, tox-lsr | ||
run: | | ||
set -euxo pipefail | ||
mm=meta/main.yml | ||
if [ -f "$mm" ]; then | ||
if ! grep -q '^ *namespace:' "$mm"; then | ||
sed "/galaxy_info:/a\ namespace: linux_system_roles" -i "$mm" | ||
fi | ||
if ! grep -q '^ *role_name:' "$mm"; then | ||
sed "/galaxy_info:/a\ role_name: timesync" -i "$mm" | ||
fi | ||
fi | ||
pip3 install "git+https://github.com/linux-system-roles/[email protected]" | ||
- name: Convert role to collection format | ||
run: | | ||
set -euxo pipefail | ||
TOXENV=collection lsr_ci_runtox | ||
coll_dir=".tox/ansible_collections/$LSR_ROLE2COLL_NAMESPACE/$LSR_ROLE2COLL_NAME" | ||
# ansible-lint action requires a .git directory??? | ||
# https://github.com/ansible/ansible-lint/blob/main/action.yml#L45 | ||
mkdir -p "$coll_dir/.git" | ||
- name: Run ansible-lint | ||
uses: ansible-community/ansible-lint-action@v6 | ||
uses: ansible/ansible-lint@v6 | ||
with: | ||
working_directory: .tox/ansible_collections/${{ env.LSR_ROLE2COLL_NAMESPACE }}/${{ env.LSR_ROLE2COLL_NAME }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters