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

throttle text wrapping #38881

Open
wants to merge 5 commits into
base: devel
from
Open

throttle text wrapping #38881

wants to merge 5 commits into from

Conversation

@bcoca
Copy link
Member

@bcoca bcoca commented Apr 17, 2018

SUMMARY
ISSUE TYPE
  • Feature Pull Request
  • New Module Pull Request
  • Bugfix Pull Request
  • Docs Pull Request
COMPONENT NAME
ANSIBLE VERSION

ADDITIONAL INFORMATION

@bcoca bcoca force-pushed the bcoca:throttle_msgs branch to 207c05c Apr 24, 2018
@ansible ansible deleted a comment from ansibot Apr 24, 2018
@ansible ansible deleted a comment from ansibot Apr 24, 2018
@mattclay
Copy link
Member

@mattclay mattclay commented Apr 26, 2018

CI failure due to traceback in unit tests:

test/units/modules/cloud/amazon/test_ec2_vpc_nat_gateway.py:45: in <module>
    inventory = InventoryManager(loader=loader)
lib/ansible/inventory/manager.py:145: in __init__
    self.parse_sources(cache=True)
lib/ansible/inventory/manager.py:217: in parse_sources
    display.warning("No inventory was parsed, only implicit localhost is available")
lib/ansible/utils/display.py:254: in warning
    wrapped = safe_wrap(new_msg, self.columns)
lib/ansible/utils/display.py:70: in safe_wrap
    wrapped += textwrap.wrap(''.join(msg[current:]), columns, drop_whitespace=drop_whitespace)
E   TypeError: cannot concatenate 'str' and 'list' objects

Related errors in integration tests as well.

@goneri
Copy link
Contributor

@goneri goneri commented Sep 20, 2019

Hi @bcoca, is this PR still active or can we close it?

@ansibot ansibot added pre_azp and removed ci_verified stale_ci labels Dec 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants