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

config, integrate dynamic galaxy servers #83129

Open
wants to merge 18 commits into
base: devel
Choose a base branch
from

Conversation

bcoca
Copy link
Member

@bcoca bcoca commented Apr 23, 2024

Based on code from #78396

ISSUE TYPE
  • Bugfix Pull Request

@ansibot ansibot added bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Apr 23, 2024
@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label Apr 24, 2024
@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Apr 24, 2024
@ansible ansible deleted a comment from ansibot Apr 24, 2024
@bcoca bcoca marked this pull request as ready for review April 24, 2024 19:22
@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label Apr 25, 2024
@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Apr 25, 2024
@mattclay mattclay removed the needs_triage Needs a first human triage before being processed. label Apr 25, 2024
@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label Apr 26, 2024
@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label Apr 26, 2024
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
@ansibot ansibot removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Apr 26, 2024
@ansibot ansibot added the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label May 1, 2024
lib/ansible/cli/config.py Outdated Show resolved Hide resolved
@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label May 2, 2024
@ansibot ansibot removed the ci_verified Changes made in this PR are causing tests to fail. label May 2, 2024
- debug: msg='{{ (galaxy_server_dump.results[0].stdout | from_json) }}'

- set_fact:
galaxy_server_dump_base: '{{ (galaxy_server_dump.results[0].stdout | from_json).get("GALAXY_SERVERS") }}'
Copy link
Member

@webknjaz webknjaz May 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TASK [ansible-config : set_fact] ***********************************************
task path: /root/ansible/test/results/.tmp/integration/ansible-config-lde2bi52-ÅÑŚÌβŁÈ/test/integration/targets/ansible-config/tasks/main.yml:91
fatal: [testhost]: FAILED! => {
    "msg": "The task includes an option with an undefined variable. The error was: 'list object' has no attribute 'get'\n\nThe error appears to be in '/root/ansible/test/results/.tmp/integration/ansible-config-lde2bi52-ÅÑŚÌβŁÈ/test/integration/targets/ansible-config/tasks/main.yml': line 91, column 5, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n  - set_fact:\n    ^ here\n"
}

(https://dev.azure.com/ansible/ansible/_build/results?buildId=110925&view=logs&j=d1429bdc-f019-529f-70df-edbf8109ffa3&t=399aa4e2-6013-52c9-655d-d64ce9856c4d&l=8245)

@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug. ci_verified Changes made in this PR are causing tests to fail. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants