zalando-zmon / zmon-aws-agent

AWS API crawler to auto discover running services in your account
Apache License 2.0
11 stars 11 forks source link

Fatal error on elb_client.describe_tags #13

Closed marek-obuchowicz closed 8 years ago

marek-obuchowicz commented 8 years ago

The commit https://github.com/zalando/zmon-aws-agent/commit/ba70e59f33f20dd0e42670e33b31a64841317ae6 introduced fatal error, since this commit ELB discovery is not working and agent is broken for us completely:

Traceback (most recent call last):
  File "/zmon-agent.py", line 522, in <module>
    main()
  File "/zmon-agent.py", line 405, in main
    elbs = get_running_elbs(region, infrastructure_account)
  File "/zmon-agent.py", line 178, in get_running_elbs
    tag_desc = elb_client.describe_tags(LoadBalancerNames=elb_names)
  File "/usr/local/lib/python2.7/dist-packages/botocore/client.py", line 310, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/usr/local/lib/python2.7/dist-packages/botocore/client.py", line 382, in _make_api_call
    api_params, operation_model)
  File "/usr/local/lib/python2.7/dist-packages/botocore/client.py", line 423, in _convert_to_request_dict
    api_params, operation_model)
  File "/usr/local/lib/python2.7/dist-packages/botocore/validate.py", line 273, in serialize_to_request
    raise ParamValidationError(report=report.generate_report())
botocore.exceptions.ParamValidationError: Parameter validation failed:
Invalid length for parameter LoadBalancerNames, value: 62, valid range: 1-20
Jan-M commented 8 years ago

I did not see that we introduced something new, which indeed we did with the tags.

@a1exsh can you look into this? seems to need paging.

a1exsh commented 8 years ago

Oops, will work on a fix.

marek-obuchowicz commented 8 years ago

Thanks, it looks fine