Feature/flexible dependency specification #3
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
The dep chain for roles created during the compile step had bugs, in which the dep chain was overwriten and the original tasks in the role were not assigned a dep chain. This lead to problems in determining whether roles had already run when in a "diamond" structure, and in some cases roles were not correctly getting variables from parents. Fixes ansible#14046
Fix server numbers in first 10 next 10 example
Role definitions typically require params to be different from those which are specified as FieldAttributes on the playbook classes used for roles, however a certain subset should be allowed (typically those used for connection stuff). Fixes ansible#14095
Fixes documentation formatting
Fixes a bug inroduced in fb797a9 where included tasks in a role were not being executed because the child blocks had no dep chain set.
pkg_add has been obsoleted in all released versions of FreeBSD for several years. Change the example to use the appropriate pkg(8) command line.
Change example to use pkg rather than the obsolete pkg_add
Fixes documentation formatting
Note "ssh" deprecation for inventory variables
it is now called for every event prior to any other callbacks fixes ansible#14114
also removed cruft about localhost as if used it is specified inline fixes ansible#13848
Patch to remove dependency on boto when only using boto3
* In 2.0.0.x become was reversed for synchronize. It was happening on the local machine instead of the remote machine. This restores the ansible-1.9.x behaviour of doing become on the remote machine. However, there's aspects of this that are hacky (no hackier than ansible-1.9 but not using 2.0 features). The big problem is that it does not understand any become method except sudo. I'm willing to use a partial fix now because we don't want people to get used to the reversed semantics in their playbooks. * synchronize copying to the wrong host when inventory_hostname is localhost * Fix problem with unicode arguments (first seen as a bug on synchronize) Fixes ansible#14041 Fixes ansible#13825
…h a raw native string that we make unicode in py2.
…eversed synchronize: First cut at not doing sudo on the control machine but on the remote machine instead.
So far, when a 'diff' dict is returned with module results, it is checked for 'before' and 'after' texts, which are processed in _get_diff() by python difflib. This generates the changes to display when CLI users specify --diff. However, some modules will generate changes that cannot easily be expressed in a conventional diff. One example is the output of the synchronize module, which presents changed files in a common log format as in `rsync --itemize-changes`. Add a check for a diff['prepared'] key, which can contain prepared diff text from modules.
Handle the key 'custom' in the 'diff' result from modules
Adding support for scm-based role source urls (ansible-galaxy)
functions same, just keeping names from confusing people
Linode Dynamic Inventory: Add public IP's as hostvars
…sing_exception Display the string which is causing an exception to be raised.
Some typos in CHANGELOG.md
Cleanup for networking common bits.
remove brackets on links
profile_tasks README.md
moved to base class for shell plugins
* Make sure dep chains are checked recursively for nested blocks * Fixing iterator is_failed() check to make sure we're not in a rescue block before returning True * Use is_failed() to test whether a host should be added to the TQM failed_hosts list * Use is_failed() when compiling the list of hosts left to iterate over in both the linear and free strategies Fixes ansible#14222
Initial roadmap to be published. 2.1 release
This adds a new action plugin iosxr_template that allows the iosxr_template module to pass network device configurations through the template engine. It also allows configurations to be backed up.
* Fix the way task_include fields were created and copied * Have blocks get_dep_chain() look at task_include's blocks for proper dep chain inheritance * Fix the way task_include fields are copied to prevent a recursive degradation Fixes ansible#14460
this should simplify api calls
also adopted to less requried options
When working around "bad systems that insist on not allowing updates in an atomic manner", we should not run previous exception management code that tries to perform atomic move in case of exception since the dirty non atomic move has already been performed.
Update playbooks_lookups.rst
document exit status for ansible-playbook
Fix related to ansible#13981
try to copy into place first
javiplx
force-pushed
the
feature/flexible_dependency_specification
branch
3 times, most recently
from
2da5a34
to
5da8e22
Compare
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.
Issue Type:
Ansible Version:
Summary:
Example output: