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

ansible posix module missing from requirements.yml in stable-6.0 branch #7261

Closed
ghost opened this issue Jul 28, 2022 · 1 comment
Closed

Comments

@ghost
Copy link

ghost commented Jul 28, 2022

Bug Report

What happened:
my CI Pipeline runs

ansible-playbook -i ../inventories/common -i ../inventories/dev --syntax-check site.yml

on ceph-ansible as a syntax check

What you expected to happen:

the check should be green

How to reproduce it (minimal and precise):

Dockerfile to deploy the stable-6.0 branch, which has a hard requirements check on ansible 2.10

FROM ubuntu:latest
ENV DEBIAN_FRONTEND=noninteractive
RUN apt update && \
    apt -y install python3-pip policycoreutils-python-utils && \
    apt clean && \
    rm -rf /var/lib/apt/lists/* && \
    pip3 install ansible-base==2.10.17 

then run above ansible syntax check and get:

ERROR! couldn't resolve module/action 'selinux_permissive'. This often indicates a misspelling, missing collection, or incorrect module path.
The error appears to be in '/builds/m-cloud/config/mw-ceph/ceph-ansible/roles/ceph-nfs/tasks/main.yml': line 35, column 3, but may
be elsewhere in the file depending on the exact syntax problem.
The offending line appears to be:
# NOTE (leseb): workaround for issues with ganesha and librgw
- name: add ganesha_t to permissive domain
  ^ here

I believe this can be fixed by altering the docker file like this:

ansible-galaxy collection install ansible.posix

as an alternative, this seems to be fixed in master branch in the requirements.yml file:

ansible-galaxy install -r requirements.yml

but the required posix module was only added in this commit, it seems:
aa68b06

Share your group_vars files, inventory and full ceph-ansibe log

Environment:

  • OS (e.g. from /etc/os-release): doesn't matter
  • Kernel (e.g. uname -a): doesn't matter
  • Docker version if applicable (e.g. docker version): doesn't matter
  • Ansible version (e.g. ansible-playbook --version): 2.10.17
  • ceph-ansible version (e.g. git head or tag or stable branch): stable-6.0
  • Ceph version (e.g. ceph -v): doesn't matter

I hope I didn't miss anything and this is a valid bug report, will make a Pull Request soon.

kind regards

Sven

ghost pushed a commit to artificial-intelligence/ceph-ansible that referenced this issue Jul 28, 2022
ghost pushed a commit to artificial-intelligence/ceph-ansible that referenced this issue Jul 28, 2022
Signed-off-by: Sven Kieske <s.kieske@mittwald.de>
ghost pushed a commit to artificial-intelligence/ceph-ansible that referenced this issue Jul 29, 2022
Signed-off-by: Sven Kieske <s.kieske@mittwald.de>
guits pushed a commit that referenced this issue Aug 3, 2022
Signed-off-by: Sven Kieske <s.kieske@mittwald.de>
@guits
Copy link
Collaborator

guits commented Aug 3, 2022

fixed by #7262

@guits guits closed this as completed Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant