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

Changing openstack ansible module to os_server_info #5171

Merged
merged 1 commit into from
Jul 28, 2022

Conversation

thomas-crowe
Copy link
Contributor

Changing Openstack configs to use the "os_server_info" module instead of the openstack.cloud collection.

SUMMARY
ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request
  • New config Pull Request
  • New role Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

@thomas-crowe thomas-crowe requested a review from a team as a code owner July 28, 2022 14:50
@bbethell-1
Copy link
Contributor

Notice: A new release of pip available: 22.2 -> 22.2.1
Notice: To update, run: pip install --upgrade pip
py39-ansible_syntax create: /home/runner/work/agnosticd/agnosticd/tests/static/.tox/py39-ansible_syntax
py39-ansible_syntax installdeps: -r/home/runner/work/agnosticd/agnosticd/tests/static/test-requirements.txt
py39-ansible_syntax installed: ansible==2.9.15,appdirs==1.4.4,astroid==2.4.2,attrs==20.3.0,cffi==1.14.4,cryptography==3.3.2,distlib==0.3.1,filelock==3.0.12,flake8==3.8.4,iniconfig==1.1.1,isort==5.6.4,Jinja2==2.11.2,lazy-object-proxy==1.4.3,MarkupSafe==1.1.1,mccabe==0.6.1,packaging==20.7,pathspec==0.8.1,pluggy==0.13.1,py==1.9.0,pycodestyle==2.6.0,pycparser==2.20,pyflakes==2.2.0,pylint==2.6.0,pyparsing==2.4.7,pytest==6.1.2,PyYAML==5.3.1,setuptools-lint==0.6.0,six==1.15.0,toml==0.10.2,tox==3.20.1,virtualenv==20.2.2,wrapt==1.12.1,yamllint==1.25.0
py39-ansible_syntax run-test-pre: PYTHONHASHSEED='713240710'
py39-ansible_syntax run-test: commands[0] | /home/runner/work/agnosticd/agnosticd/tests/static/syntax-check.sh development c621112
OK .......... yamllint ansible/cloud_providers/osp_infrastructure_deployment.yml
WARNING ........ yamllint: No conf .yamllint found for ansible/configs/multi-cloud-capsule/infra_configs/osp_infrastructure_deployment.yml
OK .......... yamllint ansible/configs/osp-migration/destroy_env.yml
OK .......... yamllint ansible/configs/osp-migration/infra.yml
OK .......... yamllint ansible/configs/osp-satellite-vm/destroy_env.yml
OK .......... yamllint ansible/configs/osp-satellite-vm/infra.yml
WARNING ........ yamllint: No conf .yamllint found for ansible/lifecycle_osp.yml
OK .......... yamllint ansible/roles-infra/infra-osp-dry-run/tasks/main.yml
FAIL ........ Galaxy roles multi-cloud-capsule/sample_vars_ec2.yml

ERROR! couldn't resolve module/action 'amazon.aws.ec2_instance_facts'. This often indicates a misspelling, missing collection, or incorrect module path.

The error appears to be in '/home/runner/work/agnosticd/agnosticd/ansible/roles-infra/infra-ec2-create-inventory/tasks/main.yml': line 7, column 3, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

  • name: Gather EC2 facts
    ^ here
    ERROR: InvocationError for command /home/runner/work/agnosticd/agnosticd/tests/static/syntax-check.sh development c621112 (exited with code 2)
    ___________________________________ summary ____________________________________
    ERROR: py39-ansible_syntax: commands failed
    Error: Process completed with exit code 1.

@thomas-crowe
Copy link
Contributor Author

@bbethell-1 that error is being triggered on every commit since March 11th.

@thomas-crowe thomas-crowe merged commit e7e53d0 into development Jul 28, 2022
@thomas-crowe thomas-crowe deleted the TAC-os_server_info branch July 28, 2022 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants