* Put '.zip' to suffix_order list as well as to suffix_map.
* Update test
* Added changelog.
* Handle .pyx modules as well + test
* Test fix: guard pyximport assertions
* Ran pre-commit hook
For some of the more complex salt code--or when salt functions lack
timing stats themselves--it's desirable to be able to do thus:
```sls
{%- profile as 'inner test' %}
< jinja-code >
{%- endprofile %}
```
And get a log line like thus:
```
2019-08-05 22:10:59,531 [salt.utils.jinja ][PROFILE ][26803] Time (in seconds) to render profile block 'inner test': 5.96046447754e-06
```
This commit adds a new `profile` block tag that emits these log
statements.
The same machinery used to support the `profile` block tag is also wired
into `import_*` blocks. For example:
```sls
{%- import grains.sls as 'blah' %}
```
results in the following log statement:
```
2019-08-05 22:10:59,542 [salt.utils.jinja ][PROFILE ][26803] Time (in seconds) to render import_yaml 'grains.sls': 0.0106961727142
```
* Make proxy_config read in the proxy specific configuration which is typically found in /etc/salt/proxy.d/minionid/.
* Adding changelog.
* Fixed auto py3 update hook result
* Only lay down the required config for tests using it
* Running pre-commit bits manually.
Co-authored-by: Dmitry Kuzmenko <dmitry.kuzmenko@dsr-corporation.com>
Co-authored-by: Pedro Algarvio <pedro@algarvio.me>
* Adding a check when a source is an HTTP or FTP URL to do a query to ensure the URL is valid before returning, then we know if we need to move onto to the next source in the list or not. Adding tests.
* Adding changelog.
* Removing change from another PR.
* Fixing tests to be cross platform.
* add in a ValueError catch for BigSur services, also removes py2 code
* pylint fix
* full pyupgrade
* pylint improvements
* add changelog
* remove import comments
* remove comments in test
Co-authored-by: Gareth J. Greenaway <gareth@wiked.org>
Co-authored-by: Sage the Rage <36676171+sagetherage@users.noreply.github.com>
* add in a check so if someone passes in a service name like salt-master the module converts over to the macOS naming convention of com.saltstack.salt.master
* add note to documentation
* blacken mac_service
* add changelog for fixing 57878
* remove comments in test
Co-authored-by: Gareth J. Greenaway <gareth@wiked.org>
Co-authored-by: Sage the Rage <36676171+sagetherage@users.noreply.github.com>
- The first positional parameter of get_repo was changed
from "name" to "repo".
- aptpkg, zypperpkg, opkg, already use "repo" as the first
parameter
- Fixes#57778
Some recent change in Salt is now doing the right thing, and calling the
different states with separated args and kwargs. This change trigger a
hidden bug in the __mount_device decorator, that expect those parameter
to be in kwargs, as is happening during the test.
This patch change the way that the wrapper inside the decorator search
for the name and device parameters, first looking into kwargs and later
in args if possible. A new test is introduced to exercise both cases.
Fix#58012
salt-run $server spacewalk.api allows users to run arbitrary Spacewalk
API functions through Salt. These are passed in a namespace.method
notation and may use nested namespaces. Previously only methods in a
top-level namespace were supported.
Fixes https://github.com/saltstack/salt/issues/57442
Co-authored-by: Wayne Werner <wwerner@saltstack.com>
* delete_schedule_conf_on_yaml_parse_error
* delete_schedule_conf_on_yaml_parse_error
* delete_schedule_conf_on_yaml_parse_error
* doc
* changelog
* delete_schedule_conf_on_yaml_parse_error
* delete_schedule_conf_on_yaml_parse_error
* delete_schedule_conf_on_yaml_parse_error
* doc
* changelog
* Add some tests
* Rename instead of remove
* doc
* no sleep
* pylint
* Use direct import instead of __utils__
* Remove __utils__ from other modules __virtual__ func
* Remove py2 code
* Revert boto_ec2
There's a problem with the assign_funcs util that I don't know how to
fix
* Rever boto_sns
There a problems with the assign_funcs util that I don't know how to fix
* Remove py2 code form elbv2
* delete_schedule_conf_on_yaml_parse_error
* delete_schedule_conf_on_yaml_parse_error
* delete_schedule_conf_on_yaml_parse_error
* doc
* changelog
* Add some tests
* Rename instead of remove
* doc
* no sleep
* pylint
* Remove py2
* Fix format strings
* Fix some black
* fix some lint
* Fix failing test
* Added changelogs
* Moved test code of Twangboy, help of DmitryKuzmenko
* Redundant import
* Fix black
* Fix issue with iterator
Co-authored-by: twangboy <slee@saltstack.com>
because the future was generated from tornado.gen.sleep, its eventual timer
still fires regardless of if the future is manually set with set_result. When
set_result is called twice (once manually by saltnado, once by tornado call_later),
a stacktrace like so is thrown:
[ERROR ] Uncaught exception POST / (::1)
Traceback (most recent call last):
File "/home/mphillips81/.pyenv/versions/3.6.5/lib/python3.6/site-packages/tornado/web.py", line 1369, in _stack_context_handle_exception
raise_exc_info((type, value, traceback))
File "<string>", line 3, in raise_exc_info
File "/home/mphillips81/.pyenv/versions/3.6.5/lib/python3.6/site-packages/tornado/stack_context.py", line 314, in wrapped
ret = fn(*args, **kwargs)
File "/home/mphillips81/.pyenv/versions/3.6.5/lib/python3.6/site-packages/tornado/gen.py", line 771, in later
f.set_result(None)
File "/home/mphillips81/.pyenv/versions/3.6.5/lib/python3.6/site-packages/tornado/concurrent.py", line 254, in set_result
self._set_done()
File "/home/mphillips81/.pyenv/versions/3.6.5/lib/python3.6/site-packages/tornado/concurrent.py", line 298, in _set_done
for cb in self._callbacks:
TypeError: 'NoneType' object is not iterable
This patch attempts to fix that.
This patch ensures that the extents file attribute is ignored when
comparing file attributes. This is necessary because this attribute is
set by the underlying file system and can never be reset.
The libvirt xen driver does not handle disk of 'volume' type. We thus
need to convert them into their equivalent using the 'file' or 'block'
type (issue #58333).
