Commit Graph

109 Commits (ce5e10c00468b24628113ed8cdb1d545b6b78b9c)

Author SHA1 Message Date
Ashwini Mhatre ce5e10c004
Add ipcut filter plugin (#291)
* Add ipcut filter plugin

* Add testcases

* Add testcases

* Fix sanity

* Fix sanity

* Fix sanity

* Fix ansible-lint

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update ipcut.py

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix sanity

* Fix sanity

* iFix sanity

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ashwini Mhatre <amhatre@amhatre-thinkpadt14sgen2i.pnq.csb>
2023-08-07 16:19:33 +05:30
Ashwini Mhatre 827e88dfbf
Bugfix ip4 hex (#292)
* Fix bug in ipv4_hex filter.

* Fix issue in ipv4_hex

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update tests/integration/targets/utils_ipaddr_filter/tasks/ip4_hex.yaml

Co-authored-by: Sagar Paul <sagpaul@redhat.com>

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Sagar Paul <sagpaul@redhat.com>
2023-07-20 11:46:34 +05:30
pre-commit-ci[bot] 1c31436512
[pre-commit.ci] pre-commit autoupdate (#290)
* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/asottile/add-trailing-comma: v2.5.1 → v3.0.0](https://github.com/asottile/add-trailing-comma/compare/v2.5.1...v3.0.0)
- [github.com/pre-commit/mirrors-prettier: v3.0.0-alpha.9-for-vscode → v3.0.0](https://github.com/pre-commit/mirrors-prettier/compare/v3.0.0-alpha.9-for-vscode...v3.0.0)
- [github.com/psf/black: 23.3.0 → 23.7.0](https://github.com/psf/black/compare/23.3.0...23.7.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
2023-07-19 09:29:35 -04:00
Sebastian Wiesinger c2231a6b20
Update jsonschema validator (#226)
* Update jsonschema validator

Update jsonschema validator to support newer drafts. It now supports
drafts 2019-09 and 2020-12.

The logic for choosing the jsonschema validator class has changed so
that the following enhancements are available:

- When no draft is explicitly specified we now use use the validator
  draft that is specified in the "$schema" field of the criteria. This
  is done by the jsonschema module by default and should support
  possible future drafts without any changes to this code.

- Optionally allow to disable format checks in the code. As format
  checks are not required by the spec there might be situations where
  people want to disable them.

* Update requirements.txt

* Skip all tests which dependas on jsonschema 4.5

* jsonschema: Refactor code to support python 3.6

* Fix jsonschema requirements for python<3.7

* jsonschema: Update code for compatibility

* Better documentation and error handling for missing schema specifications

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ashwini Mhatre <amhatre@redhat.com>
Co-authored-by: ashwini-mhatre <mashu97@gmail.com>
2023-04-05 11:01:12 -04:00
Vinay M ed6555526d
Fixed the netaddr error msg for clarity (#244)
* Fixed the netaddr error msg for clarity

* added changelog

* Used missing_required_lib

---------

Co-authored-by: Kate Case <kcase@redhat.com>
2023-04-05 18:54:10 +05:30
Kate Case 9668f26a26
Update pre-commit config (#245)
* Update pre-commit config
2023-03-29 18:18:57 -04:00
Ashwini Mhatre fca1229097
Added support to disable xml declartion with full_document flag (#237) 2022-12-28 19:47:34 +05:30
Kate Case 24e53efe5e
Add documentation for IPv6 in ipsubnet filter (#212)
* fix typo. Add example for IPV6

* add unit tests for IPv6

Co-authored-by: serosset <serosset@cisco.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
2022-12-05 16:00:19 -05:00
Kate Case f407bbdb92
Update regexes to work around Python 3.11 changes (#231) 2022-12-05 13:32:45 -05:00
Ashwini Mhatre baba9ca7ea
Accept int input for ipaddr ipv4 and ipv6 filters (#224)
* Accept int input for ipaddr ipv4 and ipv6 filters

* Fix fragment

* Fix tests
2022-11-21 15:18:35 +05:30
Federico Chiacchiaretta 0aee02eb97
Welcome to support for indentation with spaces in to_xml plugin (#192)
* Welcome to support for indentation with spaces in to_xml plugin

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Changelog fragment

* Unit tests for:

* Indent with spaces and explicit indent_width (4)
* Invalid indent

I also slightly modified VALID_DATA to produce output with indentation,
so existing test_valid_data and OUTPUT has been updated accordingly.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix changelog section name to minor_changes

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
2022-11-04 14:13:13 -04:00
Kate Case 6f0c83be4d
Add PersistentConnectionBase (#213) 2022-11-03 16:00:17 +00:00
Rohit Thakur 7de75fbd9e
add validated content plugin support (#201)
* add validated content plugin support

Signed-off-by: Rohit Thakur <rohitthakur2590@outlook.com>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Signed-off-by: Rohit Thakur <rohitthakur2590@outlook.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
2022-11-03 14:43:04 +00:00
Felix Fontein b73ed61a47
Fix filter and test docs (#215)
* Fix filter and test documentation syntax errors.

Most cases were strings that were YAML-parsable as dictionaries due to colons,
sometimes of the form `str: str`, and sometimes of the form `str: list`.

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
2022-10-14 13:02:22 -04:00
artraf 61665ea45a
Fix not optimal variable assignment (#207) 2022-10-14 15:35:33 +00:00
Harm Geerts e3e4f7595d
Fix #209 ansible.utils filters should raise AnsibleFilterError (#210)
* Fix #209 ansible.utils filters should raise AnsibleFilterError

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
2022-10-14 10:55:05 -04:00
Kate Case 462e4e7f71
Clean up test failures (#216)
* Fix arguments-renamed

* Handle changed behavior in 2.15
2022-10-13 11:01:42 -04:00
Tim Way 57c5cd4336
Switch To Calculating Networking Information Directly For Performance (#146)
* Add Unit Tests To Capture Failures From 'subnet' Generator

The netaddr library returns a generator for the 'subnet' call. This works great until you use larger networks. While it is uncommon to encounter it in IPv4 usage it is trivial to hit it in IPv6.

* Switch To Calculating Networking Information Directly For Performance

This replaces the inefficient generator for 'subnet' and uses math to determine the result directly. Since a list is not returned directly to the client in the implemented cases this works great and is fast.

A further optimization at least on the logic of this might be to break the different cases implemented by the filter out into unique functions. I did not do this yet because I wanted to get feedback on this direction.

* Changelog Fragment For PR / Bugfix

Adding changelog fragment that references source issue.

* Dropping Python 3.7 Bypass Removes Need For 'sys' Module

A test for ipsubnet was bypassed under 3.7 because of an inconsistent return value w/3.6 and 2.7. I removed the bypass and changed the behavior of the filter to raise an AnsibleFilterError in all versions of Python.

* Add A Pair of Integration Tests

These demonstrate the issue with the current implementation and would normally stall out while building the list of possible subnets from the generator.

* Address Changelog Feedback

I kept the performance item as a bugfix but bumped the typing to a minor change.

* Add 'netaddr' To Integration Test 'requirements.txt'

* The `ansible-test integration --docker` requires this line in requirements.txt to pass the 'netaddr' related tests

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Replace str -> to_text

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Kate Case <this.is@katherineca.se>
2022-08-31 12:09:35 -04:00
Kate Case e7143c16bd
Drop 2.7 shim (#202) 2022-08-24 09:28:22 -04:00
Bradley A. Thornton a0622d06ce
Add trailing comma support to pre-commit (#190) 2022-06-01 18:05:10 +05:30
Bradley A. Thornton 2541cb3f6a
Add isort, increase black line length (#189) 2022-05-26 13:18:57 -04:00
Bradley A. Thornton 79a12c3c43
Enable update docs via pre-commit (#186)
* Enable update docs

* Remove foo

* Changelog

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
2022-05-25 15:53:41 -07:00
Bradley A. Thornton 50201a1ad9
Enable integration tests with Github action (#180) 2022-05-18 11:06:59 -07:00
Bradley A. Thornton 5a53c19804
Github action for unit and sanity tests (#172) 2022-05-16 09:52:15 -07:00
Ashwini Mhatre 0c77d18f32
Fix validate-module sanity for stable 2.13 sanity job (#164)
* Fix validate-module sanity

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Bradley A. Thornton <bthornto@redhat.com>
2022-04-13 15:14:05 -07:00
KB-perByte 6110081f19
sanity fix 2022-04-11 17:20:49 +05:30
KB-perByte ca31a4e474
update docs 2022-04-11 16:13:46 +05:30
KB-perByte 7801c67a5c
update err code 2022-04-11 14:57:00 +05:30
KB-perByte 1ce17fab54
update tests 2022-04-09 11:28:35 +05:30
KB-perByte 595c9ee38f
review pt 1 2022-04-08 23:31:45 +05:30
KB-perByte 89efc29ab4
update added version 2022-04-06 19:25:56 +05:30
KB-perByte f3e851d544
update review 1 2022-04-06 19:19:48 +05:30
KB-perByte ae4bfa421c
remove strings 2022-04-06 11:20:16 +05:30
KB-perByte 3f98e8ae7a
sanity and tox fixed 2022-04-05 15:44:37 +05:30
KB-perByte 02779ae85a
UTs added 2022-04-05 12:23:31 +05:30
KB-perByte 38986edc8e
doc fix 2022-04-04 21:09:45 +05:30
KB-perByte 1b34ebd896
tox fixed and docstrings 2022-04-04 21:06:58 +05:30
KB-perByte 0cbb257293
update 1 2022-04-04 20:56:07 +05:30
KB-perByte f337f82d59
docs and examples 2022-04-04 20:02:43 +05:30
KB-perByte 141c175cff
sanity fixex updated 2022-04-04 17:58:09 +05:30
KB-perByte 66bd592df4
init code 2022-04-04 17:55:01 +05:30
Ashwini Mhatre 0716010fe3
Fix bugs in Ipaddr filters (#149)
Fix bugs in Ipaddr filters

SUMMARY

ipaddr - Fix input validation issues in ipaddr,ipv4,ipv6,ipwrap filters.
ipaddr - Add valid network for link-local
fixes: #148
fixes: ansible-collections/ansible.netcommon#375
fixes: ansible-collections/ansible.netcommon#350

ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Nilashish Chakraborty <nilashishchakraborty8@gmail.com>
Reviewed-by: Sagar Paul <sagpaul@redhat.com>
2022-03-04 11:43:59 +00:00
Ashwini Mhatre 9f510450f8
Improve ut coverage (#145)
Improve ut coverage

SUMMARY

Add more testcases for ipaddr filter

ISSUE TYPE

COMPONENT NAME

ipaddr.py
ADDITIONAL INFORMATION

Reviewed-by: Nilashish Chakraborty <nilashishchakraborty8@gmail.com>
Reviewed-by: Ashwini Mhatre <mashu97@gmail.com>
Reviewed-by: Sagar Paul <sagpaul@redhat.com>
2022-03-01 05:06:47 +00:00
Sagar Paul bd95db581e
minor doc fix (#140)
minor doc fix

SUMMARY

in_any_network, doc fix for redundant line

ISSUE TYPE


Docs Pull Request

COMPONENT NAME

in_any_network
ADDITIONAL INFORMATION

Reviewed-by: Nilashish Chakraborty <nilashishchakraborty8@gmail.com>
2022-02-10 17:35:19 +00:00
Sagar Paul 9294a2df8f
doc fixes (#134)
Minor documentation fixes

SUMMARY

Minor documentation fixes and updated doc-string

ISSUE TYPE


Docs Pull Request

Reviewed-by: Ashwini Mhatre <mashu97@gmail.com>
Reviewed-by: None <None>
2022-01-31 10:30:34 +00:00
Nathaniel Case ad9d3e1399
New validate sub-plugin "config" (#112)
New validate sub-plugin "config"

SUMMARY


Implement ansible-collections/ansible.network#15 as a validate sub-plugin.
ISSUE TYPE


Feature Pull Request

COMPONENT NAME

validate

Reviewed-by: Ganesh Nalawade <None>
Reviewed-by: Nilashish Chakraborty <nilashishchakraborty8@gmail.com>
Reviewed-by: Nathaniel Case <this.is@nathanielca.se>
Reviewed-by: None <None>
2022-01-28 00:02:36 +00:00
Sagar Paul 624bc76e26
Recursive filter plugins remove_keys | replace_keys | keep_keys (#110)
Recursive filter plugins remove_keys | replace_keys | keep_keys

SUMMARY

New plugins

remove_keys
replace_keys
keep_keys


ISSUE TYPE


New Module Pull Request

COMPONENT NAME


remove_keys
replace_keys
keep_keys

ADDITIONAL INFORMATION

Reviewed-by: Ashwini Mhatre <mashu97@gmail.com>
Reviewed-by: Nilashish Chakraborty <nilashishchakraborty8@gmail.com>
Reviewed-by: None <None>
2022-01-27 06:52:38 +00:00
Ashwini Mhatre ea54e97359
Add hwaddr, macaddr, slaac filter plugin (#129)
Add hwaddr, macaddr, slaac filter plugin

SUMMARY

Added hwaddr, macaddr, slaac filter plugin

ISSUE TYPE

New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Nathaniel Case <this.is@nathanielca.se>
Reviewed-by: Ashwini Mhatre <mashu97@gmail.com>
Reviewed-by: Sagar Paul <sagpaul@redhat.com>
Reviewed-by: None <None>
2022-01-24 21:43:18 +00:00
Ashwini Mhatre 4488b79b04
Add ipaddr Filter plugin (#115)
Add ipaddr Filter plugin

SUMMARY

depend-on: #113

ISSUE TYPE


New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Nathaniel Case <this.is@nathanielca.se>
Reviewed-by: Ashwini Mhatre <mashu97@gmail.com>
Reviewed-by: Sagar Paul <sagpaul@redhat.com>
Reviewed-by: None <None>
2022-01-24 11:15:16 +00:00
Ashwini Mhatre cfc7965400
Add ipwrap filter plugin (#118)
Add ipwrap filter plugin

SUMMARY


ISSUE TYPE


New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Priyam Sahoo <None>
Reviewed-by: Sagar Paul <sagpaul@redhat.com>
Reviewed-by: None <None>
2022-01-22 11:27:27 +00:00