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

Port isn't integer? #45

Closed
withinboredom opened this issue May 30, 2015 · 3 comments
Closed

Port isn't integer? #45

withinboredom opened this issue May 30, 2015 · 3 comments

Comments

@withinboredom
Copy link

Error from cli:

consulate register -p 3306 mysql-master no-check
Traceback (most recent call last):
  File "/usr/local/bin/consulate", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/dist-packages/consulate/cli.py", line 283, in main
    register(consul, args)
  File "/usr/local/lib/python2.7/dist-packages/consulate/cli.py", line 255, in register
    args.port, tags, check, interval, ttl)
  File "/usr/local/lib/python2.7/dist-packages/consulate/api/agent.py", line 181, in register
    raise ValueError('port must be an integer')
ValueError: port must be an integer
@gmr
Copy link
Owner

gmr commented Jul 16, 2015

That's pretty odd, since it should be forced to int by argparse I am forcing this to an int in CLI in an incoming patch, but I'm worried about TTL and other settings as well. What version of Python 2.7 are you running?

gmr added a commit that referenced this issue Jul 16, 2015
@gmr
Copy link
Owner

gmr commented Jul 16, 2015

This will be released with 0.6.0

@gmr gmr closed this as completed Jul 16, 2015
@withinboredom
Copy link
Author

I was running whatever the default was in the official ubuntu docker image.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants