-
Notifications
You must be signed in to change notification settings - Fork 83
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
Play ansible roles on host #936
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #936 +/- ##
==========================================
+ Coverage 92.19% 92.26% +0.07%
==========================================
Files 6 6
Lines 3037 3078 +41
==========================================
+ Hits 2800 2840 +40
- Misses 237 238 +1
☔ View full report in Codecov by Sentry. |
nailgun/entities.py
Outdated
@@ -4332,6 +4332,25 @@ def bulk_available_incremental_updates(self, synchronous=True, timeout=None, **k | |||
response = client.post(self.path('bulk/available_incremental_updates'), **kwargs) | |||
return _handle_response(response, self._server_config, synchronous, timeout) | |||
|
|||
def play_roles(self, synchronous=True, timeout=None, **kwargs): | |||
"""Play all assigned ansible roles |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As same endpoint for both host and host group, so it'd be nice to cover this func for hostgroup too, wdyt?
"""Play all assigned ansible roles | |
"""Play all assigned ansible roles on host |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would. I did what I needed for my tests, there's a lot of things that would be nice.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yup, asked it because its a same endpoint and would be just copy-pasta! :) non-blocking though
* Play ansible roles on host * Coverage * Actually returns task id (cherry picked from commit 995aa7f)
Add a method to play ansible roles on a host.
Example: