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

add the role version number to the ansible-galaxy error/warning message #70136

Closed

Conversation

jainnikhil30
Copy link
Contributor

@jainnikhil30 jainnikhil30 commented Jun 18, 2020

SUMMARY

cc @sudhirmohanraj, This is taken from your original PR #52853 which was closed.

Fixes: #52111

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

ansible-galaxy

ADDITIONAL INFORMATION

Before:

[niks@localhost]$ ansible-galaxy install -r requirements.yml 
- downloading role 'dummy', owned by 
[WARNING]: - dummy was NOT installed successfully: None (HTTP Code: 400, Message: Bad Request)
ERROR! - you can use --ignore-errors to skip failed roles and finish processing the list.

After this PR:

(venv) [niks@localhost ]$ ansible-galaxy install -r /home/niks/ansible_test/requirements.yml 
[WARNING]: You are running the development version of Ansible. You should only run Ansible from "devel" if you are modifying the Ansible engine, or trying out features under development. This is a rapidly changing source of code and can
become unstable at any point.
Starting galaxy role install process
- downloading role 'dummy', owned by 
[WARNING]: - dummy 1.1 was NOT installed successfully: None (HTTP Code: 400, Message: Bad Request)
ERROR! - you can use --ignore-errors to skip failed roles and finish processing the list.

@ansibot ansibot added affects_2.11 bug This issue/PR relates to a bug. core_review In order to be merged, this PR must follow the core review workflow. needs_triage Needs a first human triage before being processed. small_patch support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Jun 18, 2020
@ansibot ansibot added has_issue needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed core_review In order to be merged, this PR must follow the core review workflow. labels Jun 18, 2020
@ansibot ansibot added support:community This issue/PR relates to code supported by the Ansible community. and removed small_patch labels Jun 19, 2020
@ansibot ansibot added core_review In order to be merged, this PR must follow the core review workflow. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. core_review In order to be merged, this PR must follow the core review workflow. labels Jun 19, 2020
lib/ansible/cli/galaxy.py Outdated Show resolved Hide resolved
@samdoran samdoran removed the needs_triage Needs a first human triage before being processed. label Jun 23, 2020
@samdoran
Copy link
Contributor

Looking at the original issue more closely, I don't think this fixes it. Simply displaying the version or * in the error message will just continue to mask the problem.

The real issue is the git checkout failure because a non-existing version was requested. That is what the error message should state.

@Akasurde
Copy link
Member

Akasurde commented Jul 1, 2020

@samdoran Could you please take a look? Thanks.

@samdoran
Copy link
Contributor

samdoran commented Jul 6, 2020

It still seems like this change is not helping to better identify the real underlying cause.

@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Jul 6, 2020
@jainnikhil30
Copy link
Contributor Author

@samdoran

Thanks for your review. I will try and make that error more explicit in terms of version not being available remotely.

@ansibot ansibot removed the core_review In order to be merged, this PR must follow the core review workflow. label Dec 6, 2020
@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. pre_azp This PR was last tested before migration to Azure Pipelines. and removed stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. labels Dec 6, 2020
@Akasurde
Copy link
Member

@jainnikhil30 Could you please address the @samdoran comment?

needs_info

@ansibot ansibot added the needs_info This issue requires further information. Please answer any outstanding questions. label May 24, 2021
@ansibot
Copy link
Contributor

ansibot commented Jun 25, 2021

@jainnikhil30 This pullrequest is waiting for your response. Please respond or the pullrequest will be closed.

click here for bot help

@ansibot
Copy link
Contributor

ansibot commented Jul 27, 2021

@jainnikhil30 You have not responded to information requests in this pullrequest so we will assume it no longer affects you. If you are still interested in this, please create a new pullrequest with the requested information.

click here for bot help

@ansibot ansibot closed this Jul 27, 2021
@ansible ansible locked and limited conversation to collaborators Aug 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.11 bug This issue/PR relates to a bug. has_issue needs_info This issue requires further information. Please answer any outstanding questions. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. pre_azp This PR was last tested before migration to Azure Pipelines. support:community This issue/PR relates to code supported by the Ansible community. support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ansible-galaxy has cryptic error message when role version is wrong
4 participants