rsync + raw_ssh by default, with gevent backend for celery #397
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dshulyak
force-pushed
the
multi_transports
branch
2 times, most recently
from
0b48652
to
511d90c
Compare
dshulyak
changed the title
dshulyak
force-pushed
the
multi_transports
branch
2 times, most recently
from
ea6d2e0
to
b67fee9
Compare
|
pushed image with tag - solarproject/solar-celery:gevent |
dshulyak
force-pushed
the
multi_transports
branch
from
b67fee9
to
0d5de7d
Compare
| def is_connection(arg): | ||
| if isinstance(arg, basestring) and '::' in arg: | ||
| return True | ||
| return False | ||
|
|
||
|
|
||
| def parse_dicts_as_collection(child_input, arg): |
There was a problem hiding this comment.
We will have now parse_dict_input and parse_dicts_as_collection. Can you merge them?
dshulyak
force-pushed
the
multi_transports
branch
from
0d5de7d
to
6c4948e
Compare
dshulyak
force-pushed
the
multi_transports
branch
from
6c4948e
to
8cb61d6
Compare
dshulyak
force-pushed
the
multi_transports
branch
from
8cb61d6
to
b894c1d
Compare
|
|
||
| hosts1 = vr.create('hosts_file1', 'resources/hosts_file', {})[0] | ||
| hosts2 = vr.create('hosts_file2', 'resources/hosts_file', {})[0] | ||
| node1.connect(hosts1, { |
There was a problem hiding this comment.
It's done in templates/nodes.yaml
There was a problem hiding this comment.
there is just too much of duplication + new transport
|
gevent was not added in this PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.