| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
Fixes #15190
|
| |
|
|\
| |
| | |
bug fix changes Config class in network to only raise exceptions
|
| |
| |
| |
| |
| | |
The Config class should not call fail_json but simply raise exceptions
and allow the implementor to handle the exception
|
|\ \
| | |
| | | |
initial commit of vyos shared module with Cli transport
|
| | |
| | |
| | |
| | |
| | |
| | | |
This adds support for the VyOS network operating system using the
Cli transport. This module will simplify building VyOS based
modules in Ansible
|
| | |
| | |
| | |
| | | |
Fixes #16408
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* Remove unnecessary copying of values from parents to role deps, as
this can cause problems when roles have multiple parents (or the same
parents with different params speficied through deps)
* Since we're already checking the dep chain in the block for role
things (which every task in a role should have), it is not necessary
to check the role directly in case it improperly grabs something
Fixes #14438
|
|/ / |
|
| |
| |
| |
| |
| | |
Fix callback plugin path (per lib/ansible/constants.py)
(cherry picked from commit e877ea7f2beb8892cdfe0d8653ad932d119bc3ff)
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Our custom encoder for the to_json filter was simply returning the
object if it was not a HostVars object, leading in some cases to a
TypeError when the data contained an undefined variable. This lead
to an odd error message being propagated up, so we now properly catch
this as an undefined variable error.
Fixes #15610
|
| | |
|
| | |
|
|\ \
| | |
| | | |
Fixed bug in find_mount_point function
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The find_mount_point function does not resolve the mount point of paths with a soft-link correctly and returns the wrong mount-point.
I have mounted an NFS filesystem on /nfs-mount. This directory contains a directory called "directory". I also created a soft-link to this last directory: /soft-link-to-directory -> /nfs-mount/directory. I created the following task to copy a file into /soft-link-to-directory:
- name: copy file to nfs-mount
copy:
src: "file"
dest: "/soft-link-to-directory/file"
This throws an exception:
invalid selinux context: [Errno 95] Operation not supported
This is caused by the find_mount_point function to return '/' as the mount point for '/soft-link-to-directory/file'. This should have been /nfs-mount. Because the find_mount_point returns the wrong mount-point, the is_special_selinux_path function does not recognise the file is on an NFS mount and tries to set the default SELinux context (system_u:object_r:default_t:s0), which fails. The context should have been: system_u:object_r:nfs_t:s0
Full Ansible output:
TASK [copy file to nfs-mount] **************************************************
fatal: [hostname]: FAILED! => {"changed": false, "checksum": "f34b60930a5d6d689cf49a4c16bd7f9806be608c", "cur_context": ["system_u", "object_r", "nfs_t", "s0"], "failed": true, "gid": 24170, "group": "foundation", "input_was": ["system_u", "object_r", "default_t", "s0"], "mode": "0644", "msg": "invalid selinux context: [Errno 95] Operation not supported", "new_context": ["system_u", "object_r", "default_t", "s0"], "owner": "root", "path": "/soft-link-to-directory/.ansible_tmpWCT6Z4file", "secontext": "system_u:object_r:nfs_t:s0", "size": 37, "state": "file", "uid": 0}
|
| | | |
|
| | | |
|
| | | |
|
|\ \ \
| | | |
| | | | |
Add support for AWS_SESSION_TOKEN environment variable. (trivial patch)
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
According to http://blogs.aws.amazon.com/security/post/Tx3D6U6WSFGOK2H/A-New-and-Standardized-Way-to-Manage-Credentials-in-the-AWS-SDKs
the "official" environment variables that should be used for
AWS credentials should be:
AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY and AWS_SESSION_TOKEN
This patch adds support for the latter (the first two are already
supported).
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | | |
Related to #16361
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Remove break word for docsite tables
|
|/ / / /
| | | |
| | | | |
This still allows the lines to wrap with lots of text, like comments cells.
|
| |_|/
|/| |
| | | |
Fixes #16385
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- Do not update base image packages unnecessarily.
- Fix incorrect package names.
- Remove commented out RUN statements.
- Sort list of packages to install.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Again, as we're carrying failed/unreachable hosts forward from play to play via
internal structures, we need to remember which ones had previously failed so that
unrelated host failures don't inflate the numbers for a given serial batch in the
PlaybookExecutor causing a premature exit.
Fixes #16364
|
|\ \ \
| | | |
| | | | |
Make credstash lookup plugin support encryption contexts
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Previously, the lookup plugin passes all its keyword arguments to
credstash's `getSecret`; while this works for passing the standard
parameters (version, region and table), this does not allow passing
a dictionary of key-value pairs as `getSecret`'s context parameter.
Instead, pop `version`, `region` and `table` from `kwargs`, supplying
the default value if they are not defined, and pass the rest of the `kwargs`
as the `context` parameter.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The listen statement on handlers should have supported a list, however
it was broken in the revision of the pub/sub feature based on the handler
revamp. This patch corrects the bug, so this works again:
- name: some handler
...
listen:
- some target
- another target
Fixes #16378
|
| | | |
| | | |
| | | | |
Addresses https://github.com/ansible/ansible-modules-core/issues/1715
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Adding instance_states to gce inventory to align with other inventories.
|
| | | | | |
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fix quoting of args for old-style modules
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This removes the extra layer of quotes around values in the 'args' file.
These quotes were there before the pipes.quote() call was added, but
were not removed, resulting in too much quoting.
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
* add new module network
* move EOS to NetworkModule
* shell.py Python 3.x compatibility
* implements the Command class through the connection for eos
This implements a new Command class that specifies the cli command
and output format. This removes the need to batch commands through
the connection
* initial add of netcmd module
|
| | | | | | |
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fixes ansible/proposals#8
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Due to the fact that roles may be instantiated with different sets of
params (multiple inclusions of the same role or via role dependencies),
simply tracking notified handlers by name does not work. This patch
changes the way we track handler notifications by using the handler
object itself instead of just the name, allowing for multiple internal
instances. Normally this would be bad, but we also modify the way we
search for handlers by first looking at the notifying tasks dependency
chain (ensuring that roles find their own handlers first) and then at
the main list of handlers, using the first match it finds.
This patch also modifies the way we setup the internal list of handlers,
which should allow us to correctly identify if a notified handler exists
more easily.
Fixes #15084
|
|/ / / / /
| | | | |
| | | | |
| | | | |
| | | | | |
PyPI download stats have been disabled due to resource constraints (see Donald Stufft's comment, https://bitbucket.org/pypa/pypi/issues/396/download-stats-have-stopped-working-again#comment-27808922).
Thus the download stats badge counter does not show correct data.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
(#16329)
Fixes #16327
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Adds support for converting a MAC address to an integer
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Example:
---
- hosts: local
tasks:
- set_fact: mac_str="52:54:00:00:00:00"
- set_fact: base_mac="{{ mac_str | macaddr('int') }}"
- debug: msg="{{ (base_mac|int + 257) | macaddr('linux') }}" # 52:54:00:00:01:01
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Ansible-galaxy requirements support a include directive
|