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

Enable additional ssh arguments #53

Closed
wants to merge 2 commits into from
Closed

Enable additional ssh arguments #53

wants to merge 2 commits into from

Conversation

ankanm
Copy link

@ankanm ankanm commented Feb 12, 2015

Added ability to provide any additional ssh arguments.
This is to fix - #54
See this instead - #55

@@ -33,6 +33,9 @@
help="""Name of the task."""
).completer = completion_helpers.task

parser.add_argument(
Copy link
Contributor

Choose a reason for hiding this comment

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

As there are no arguments for this that are mesos specific yet, I'd like to just pass all the options directly to ssh itself. That way, you don't need to remember to pass in a special arg.

Copy link
Author

Choose a reason for hiding this comment

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

Ok. Makes sense. Will look into this.

@ankanm
Copy link
Author

ankanm commented Feb 13, 2015

Will send a different PR.

@ankanm ankanm closed this Feb 13, 2015
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

Successfully merging this pull request may close these issues.

2 participants