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

WIP: add tests #52

Closed
wants to merge 18 commits into from
Closed

WIP: add tests #52

wants to merge 18 commits into from

Conversation

edenreich
Copy link
Contributor

This pull request is still ongoing and related to #13

There is probably a better way to structure molecule tests (I'm a bit new to molecule)
One thing I've noticed is that idempotence test is failing because we're deleting the downloaded binary everytime.

Feel free to add on this or restructure it :)

@St0rmingBr4in
Copy link
Collaborator

Nice, definitely a step in the right direction

@edenreich
Copy link
Contributor Author

edenreich commented Jun 20, 2020

I've found that we're missing selinux-policy and libselinux-python yum packages on centos, but maybe it's just the docker image so I'm not entirely sure, because I've read somewhere it should come preinstalled

@edenreich edenreich changed the title WIP: add download test WIP: add tests Jun 21, 2020
@edenreich
Copy link
Contributor Author

edenreich commented Jun 21, 2020

Struggling to include roles from a different directory on a molecule test. any ideas ?
I'm trying to include k3s/master and k3s/node in k3s role.

I've tried import_tasks, import_playbooks, but then it yells at me that it cannot find jinja template :S

Wish it was simple as just giving it a path attribute (unfortunately it's not the case):

- name: Include master
  include_role:
    name: master
    path: master

@St0rmingBr4in
Copy link
Collaborator

I will look into this, I think we need to get this working. It will catch a lot of little mistakes that inadvertently get merged.

@FinalDes
Copy link

FinalDes commented Mar 2, 2021

I also building my own molecule test have with follow coverage playbook

---
- name: Converge
  vars:
    ansible_user: ansible
  ansible.builtin.import_playbook: ./../../site.yml

have follow error when run the molecule test

 UNREACHABLE! => {"changed": false, "msg": "Failed to create temporary directory.In some cases, you may have been able to authenticate and did not have permissions on the target directory. Consider changing the remote tmp path in ansible.cfg to a path rooted in \"/tmp\", for more error information use -vvv. Failed command was: ( umask 77 && mkdir -p \"` echo ~/.ansible/tmp `\"&& mkdir \"` echo ~/.ansible/tmp/ansible-tmp-1614678369.911774-14191-28401085203069 `\" && echo ansible-tmp-1614678369.911774-14191-28401085203069=\"` echo ~/.ansible/tmp/ansible-tmp-1614678369.911774-14191-28401085203069 `\" ), exited with result 1", "unreachable": true}

I want to know how to fix it?
I using image jeff geerling will little modification

@dereknola
Copy link
Member

Closing as out of date. Leaving the Issue open for a new push to add CI.

@dereknola dereknola closed this Nov 13, 2023
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

Successfully merging this pull request may close these issues.

4 participants