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

disable galaxy role testing until we have an offline testing mode #2334

Closed
ssbarnea opened this issue Sep 27, 2019 · 0 comments · Fixed by #2336
Closed

disable galaxy role testing until we have an offline testing mode #2334

ssbarnea opened this issue Sep 27, 2019 · 0 comments · Fixed by #2336
Assignees
Labels

Comments

@ssbarnea
Copy link
Member

Until we have a solution or working for ansible/galaxy#2030 we should disable test_command_dependency_ansible_galaxy tests as they can easily fail due to github banning us:

        https://galaxy.ansible.com/api/v1/roles/?owner__username=yatesr&name=timezone
        https://galaxy.ansible.com/api/v1/roles/1429/versions/?page_size=50
        - downloading role from https://github.com/yatesr/ansible-timezone/archive/1.1.0.tar.gz
 [ERROR]: failed to download the file: HTTP Error 401: Unauthorized

 [WARNING]: - timezone was NOT installed successfully.
@ssbarnea ssbarnea added the bug label Sep 27, 2019
ssbarnea added a commit to ssbarnea/molecule that referenced this issue Sep 27, 2019
Workaround-For: ansible/galaxy#2030
Fixes: ansible#2334
Signed-off-by: Sorin Sbarnea <ssbarnea@redhat.com>
@helpr helpr bot added the pr-available label Sep 27, 2019
ssbarnea added a commit to ssbarnea/molecule that referenced this issue Sep 27, 2019
Workaround-For: ansible/galaxy#2030
Fixes: ansible#2334
Signed-off-by: Sorin Sbarnea <ssbarnea@redhat.com>
ssbarnea added a commit that referenced this issue Sep 27, 2019
Workaround-For: ansible/galaxy#2030
Fixes: #2334
Signed-off-by: Sorin Sbarnea <ssbarnea@redhat.com>
@helpr helpr bot added pr-merged and removed pr-available labels Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants