| Age | Commit message (Collapse) | Author |
|---|
|
CodeRabbit will reference core Ansible framework and netcommon base classes
when reviewing this collection modules, plugins, and test patterns.
🤖 Generated by [robots](https://vyos.io)
|
|
- Expand changelog fragment glob to *.{yaml,yml} (18 of 19 fragments use .yml)
- Add known_issues and release_summary to valid changelog keys per changelogs/config.yaml
- Remove vyos_vrf from legacy module list — it has resource module infrastructure (argspec, config, facts)
🤖 Generated by [robots](https://vyos.io)
|
|
Add .coderabbit.yaml with path-specific guidelines for automated code
review, docstring generation, and unit test generation covering all
module types (resource and legacy), plugin layers, and test infrastructure.
🤖 Generated by [robots](https://vyos.io)
|
|
* Add easy-wins improvement spec for vyos.vyos collection
Covers five phases: formatting compliance, runtime.yml redirect fix,
deprecated feature cleanup, missing unit tests, and template deduplication.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
* Add implementation plan for easy-wins improvements
15 tasks across 5 phases: formatting compliance, runtime.yml bugfix,
deprecated code cleanup, missing unit tests, template deduplication.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
* Update spec and plan with architect review corrections
Key changes:
- Defer Phase 5 (template deduplication) to v7.0.0: route_maps
are not identical, BGP dedup blocked by Python module-level
scoping, OSPF has fundamentally different command paradigms
- Add .git-blame-ignore-revs step to Phase 1
- Add missing test cases: overridden/rendered for resource modules,
aggregate/purge/with_address for vyos_vlan
- Fix incorrect claim that version.py LooseVersion is unused
- Add sequential merge requirement to preamble
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
* chore: add .worktrees/ and .claude/ to .gitignore
* T8523: add Copilot custom review instructions
Add .github/copilot-instructions.md and path-specific instruction
files under .github/instructions/ to teach Copilot project conventions:
copyright attribution, changelog fragment key selection, test mocking
patterns, required state coverage for resource modules, and module
option description standards.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* T8523: remove copyright instructions from Copilot config
Copyright attribution rules are internal guidance kept in Claude
config, not in Copilot review instructions.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* T8523: remove docs/superpowers files inadvertently included
These spec/plan files came from local main commits that were
pulled in during rebase. They are unrelated to this PR.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* T8523: fix inaccuracies flagged by Copilot review
- CLI quoting: clarify that boolean flags and bare keywords are
unquoted; only string/address values use single quotes
- Test mocking: describe get_device_data / execute_show_command as
the primary resource module pattern (used by most existing tests);
note get_resource_connection_facts approach as an alternative
- modules.instructions.md: broaden applyTo to include meta/runtime.yml
so redirect guidance fires when that file is edited
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* T8523: fix legacy module mocking guidance in tests.instructions.md
Existing legacy tests use side_effect for run_commands, not
return_value. Updated to describe side_effect as the standard
approach, note return_value as also acceptable for fixed responses,
and clarify that the patched helper varies by module (run_commands
for command/facts/ping, load_config/get_config for vyos_config).
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* T8523: fix four more inaccuracies flagged by Copilot review
- CLI quoting: address values in firewall groups are unquoted in
fixtures; instruct reviewers to align with surrounding fixture
style rather than flagging all unquoted addresses
- Resource module mocking: add .start()/.stop() calls to the snippet
so it matches the actual setUp/tearDown pattern in existing tests
- terminal plugin versions: reference README.md instead of hardcoding
a version list that diverges from README
- changelog fragment: remove stale "copyright headers" claim (that
section was dropped from the instruction files)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
|
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 5 to 6.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/codecov/codecov-action/compare/v5...v6)
---
updated-dependencies:
- dependency-name: codecov/codecov-action
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: omnom62 <75066712+omnom62@users.noreply.github.com>
|
|
* sanity tests 2.22 ignore
* Ignore 2.22 for sanity
|
|
in… (#445)
* Ensure the vyos_user module works when user properties are defined in aggregate
Ensure the vyos_user module works when user properties are defined in aggregate. Previously the value variable is not filled when a property is configured as a property of a user. This gives a python error when the value variable is called.
* Add unit test for aggregate vyos_user module
* Create vyos_user_aggregate_fix.yml
Create changelog for vyos_user aggregate bugfix
* Rename vyos_user_aggregate_fix.yml to T8205_vyos_user_aggregate_fix.yml
* Rename T8205_vyos_user_aggregate_fix.yml to T8205_vyos_user_aggregate_fix.yml
Remove unintended whitespace in changelog fragment: T8205_vyos_user_aggregate_fix.yml
---------
Co-authored-by: omnom62 <75066712+omnom62@users.noreply.github.com>
Co-authored-by: Daniil Baturin <daniil@baturin.org>
|
|
* Init for the fixes related to ADE
* testing new tox.ini
* tox ade fixes
* test tox
* test tox
* testing tox
* test tox
* test tox
* testing tox
* testing fix
* testing a fix
* testing tox
* testing fix
* bindep.txt test
* test with tox
|
|
* Mid fixes
* Ready for testing
* Fixing SR interface
* Fix is ready
* changelog
* Integration tests for static_routes interface next-hop v.1.4+ WIP
* branching 1.3- and 1.4+
* Integration tests for T8104 complete
* Comments are resolved
* Comments are resolved
|
|
* t6836 vfr module init
* argspec facts module for VRF
* T6834 static_route module to support interface_route (#398)
* t6834 init
* facts fixes, WIP
* T6834 - facts updates
* static_routes 1.3- config, facts, args and sanit and unit tests
* changelog
* changelog updates
* static_routes 1.4+
* static_routes v1.4 unit tests
* comment removed
* T6829 (area interfaces in OSPFv3 1.3-1.5) re-init (#399)
* T6829 re-init
* re-init T6829 ospfv3 area interface for 1.3-
* changelog
* changelog fix
* Doc is updated
* adding 1.4+ support
* ospfv3 area interfaces for 1.4+
* fixed ospfv3 facts and added area interface unit test for 1.4
* ospfv3 clean-up
* doc updates
* Resolve merge conflicts
* vrf parsers set_table
* more vrf paresers added
* vrf facts main code
* vrf data structures
* simple vrf config push
* simple vrf config push
* some work on vrf
* basic facts done
* vrf instance parsers wif
* ongoing parser testing
* vrf dev work
* testing with paresers
* all basic settings
* merged for vrf
* more states for VRF
* vrf deleted, overriden
* overriden
* sanity
* rst for vrf
* changelog
* fix changelog
* doc fix
* vrf delete idempotent
* deleted and overridden for VFR
* overriden
* overriden vrf idempotency
* asdvanced vrf settings
* afi for VRF facts
* VRF AFI config
* VRF AFI tests
* vrf dev work
* afi dev work
* afi and deep_merge fixes for VRF
* afi basic provisioning
* changelog
* vrf sanity fixes
* VRF AFI protocols and route-maps
* vrf parsed, gathered, rendered fixes
* docstrings VRF
* docstrings VRF
* docstring VRF
* module doc update
* vrf module examples
* VRF doc examples
* linter
* VRF unit tests init
* VRF unit tests init
* VRF unit tests updates
* fixed VRF overridden and deleted unit test cases
* VRF integration tests init
* VRF int tests rtt, merged, replaced
* VRF int testt gathered, rendered
* VRF int tests deleted, rendered, overridden and empty_config
* linter
* VRF int test misc fixes
* VRF global idepmotency
* fixing py3.13 runner deps
* fixing lint
* fixing sanity
* reverting previous fixes
* VRF protocols load_module and provisioning flow
* fixes to bgp VRF protocol methods
* VRF protocol facts init
* VRF protocol load_module works
* reworked VRF load_module routine
* vrf protocl static fixed
* Fixed static and ospf VRF protocol provisioning
* Fixed static and ospf VRF protocol provisioning
* VRF doc update
* VRF docs update
* VRF doc updates
* unit tests for VRF protocols
* VRF BGP protcol sanity check
* VRF protocol fixed sanity checks (DOCUMENTATION)
* sanity and doc for VRF fixed
* VRF unit tests mock
* VRF protocol integration parsed, rtt and merged init
* VRF protocols - rendered, overridden
* VRF protocols replaced
* fix accidental changes to test_vyos_config.py
* PR description
* fix image
* README fix
* Fixing linter
* Mid-result UAT overriden VRF
* Mid-result for VRF UAT
* VRF uat
* VRF obsolete features removed
|
|
* added vyos_config confirm options
* fix: update commit comment to match main
* fix: documentation example for automatic confirm
* fix: add tests for commit confirm and confirm_timeout options
* fix: add config_confirm changelog fragment
* fix: lint missing diff parameter for edit_config
* fix: add commit confirm integration test
---------
Co-authored-by: omnom62 <75066712+omnom62@users.noreply.github.com>
|
|
Bumps [ansible/team-devtools/.github/workflows/ah_token_refresh.yml](https://github.com/ansible/team-devtools) from 25.12.8 to 26.1.0.
- [Release notes](https://github.com/ansible/team-devtools/releases)
- [Commits](https://github.com/ansible/team-devtools/compare/v25.12.8...v26.1.0)
---
updated-dependencies:
- dependency-name: ansible/team-devtools/.github/workflows/ah_token_refresh.yml
dependency-version: 26.1.0
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
|
|
Bumps [ansible/team-devtools/.github/workflows/ah_token_refresh.yml](https://github.com/ansible/team-devtools) from 25.7.0 to 25.12.8.
- [Release notes](https://github.com/ansible/team-devtools/releases)
- [Commits](https://github.com/ansible/team-devtools/compare/v25.7.0...v25.12.8)
---
updated-dependencies:
- dependency-name: ansible/team-devtools/.github/workflows/ah_token_refresh.yml
dependency-version: 25.12.8
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
|
|
* Init T7933 bugfix
* offload-target
* Argspsec fixed
* Documentation fixes
* fw_rules config fpor offload_target
* fact fixes
* offload unit test merge
* offload unit tests for v1.3- overriden, replaced
* Offload unit test v13 complete
* v14+ offload unit tests
* v1.4.+ unit tests for offload
|
|
* t7621 vyos_firewall_global diff mode
* vyos_firewall_global integration test for diff
* enhanced integration tests with diff
* diff integration tests
* merged diff test
* 1.3- specific changes to integration tests
* data/config separated
---------
Co-authored-by: Gaige B Paulsen <gaige@cluetrust.com>
|
|
* T6823 init fix
* slaac and integration tests
* reaplced integration test
* WIP integration tests
* changelog and tweaks
* rendered, gathered, parsed, integration tests for SLAAC
* l3_interfaces gathered, emty, deleted
* deleted integration test
* vif processing
* vif tests
* vir related integration test cases
* ignore ansible-bad-import-from
* ansible-bad-import-from ingore
* ignore ansible-bad-import-from for all
* test ansible-bad-import-from skip
* ingore file
* Fix bugs with configuration remove
* Delete testcase fixed
* SLAAC integration tests
* Sanity check fixes
* remove_config formatted
* l3 slaac eth0 removed from SIT
* Remove ignore 2.20
* ignore2.20 recovered
* Remove residue
|
|
Bumps [ansible/team-devtools](https://github.com/ansible/team-devtools) from 22.5.0 to 25.7.0.
- [Release notes](https://github.com/ansible/team-devtools/releases)
- [Commits](https://github.com/ansible/team-devtools/compare/v22.5.0...v25.7.0)
---
updated-dependencies:
- dependency-name: ansible/team-devtools
dependency-version: 25.7.0
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniil Baturin <daniil@baturin.org>
Co-authored-by: Gaige B Paulsen <gaige@cluetrust.com>
Co-authored-by: omnom62 <75066712+omnom62@users.noreply.github.com>
|
|
* T7964 - Add failing test.
* T7964 - Fix "IndexError: list index out of range".
|
|
* T7496 Fix disabling src route (failing tests)
* T7496 Fix disabling src route (bugfix)
* T7496 Fix disabling src route (Changelog)
---------
Co-authored-by: omnom62 <75066712+omnom62@users.noreply.github.com>
Co-authored-by: Gaige B Paulsen <gaige@cluetrust.com>
|
|
Co-authored-by: Gaige B Paulsen <gaige@cluetrust.com>
|
|
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 5 to 6.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](https://github.com/actions/setup-python/compare/v5...v6)
---
updated-dependencies:
- dependency-name: actions/setup-python
dependency-version: '6'
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniil Baturin <daniil@baturin.org>
Co-authored-by: Gaige B Paulsen <gaige@cluetrust.com>
|
|
* fix: T7943 fix sanity checks
* ci: update changelog
|
|
* init for upstream sanity checks
* PY3 removed
* procenv.py PY3 fix
* changelog
* Bulk commit of linter changes
* Clean-up
|
|
* Fixes to T7221 and greedy matching
|
|
* sanity doc fix
* changelog
* testing fix
---------
Co-authored-by: evgmol-wbio <evgeny.molotkov@whalebone.io>
|
|
* T7579: added workflow for CLA
* T7579: fix of the run trigger for CLA
* added a new line
* added the changelog fragment
* fix a new line character at the end of file
* fixed new line issues (#427) - https://github.com/vyos/vyos.vyos/pull/427
* removed blank lines (#427)
---------
Co-authored-by: Yevhen Bondarenko <evgeniy.bondarenko@sentrium.io>
|
|
* Fix typing_extensions requirement
* linter fix
|
|
|
|
* pre release doc updates
* changelog
* module doc updates
|
|
Fixing the Automation Hub token refresh workflow
|
|
bytes." (#422)
* Revert "T7320: Fixed compatibility with Paramiko by removing all "null" bytes…"
This reverts commit f02bc8b2af6de47962e3670e719d756df0cba029.
* Changelog to enable the branch revert
* Fixing lint
---------
Co-authored-by: Om Nom <omnom62@outlook.com>
Co-authored-by: Gaige B Paulsen <gaige@cluetrust.com>
|
|
* Add GitHub Actions workflow to refresh automation hub token
* Update refresh_token.yml
https://github.com/vyos/vyos.vyos/pull/423#discussion_r2140946852
As per @andamasov
* Fix indentation in refresh_token.yml cron schedule
|
|
Fixes issue found in this comment: https://github.com/vyos/vyos.vyos/pull/407#issuecomment-2774775677
Co-authored-by: omnom62 <75066712+omnom62@users.noreply.github.com>
|
|
* t7391 init
* sanity
* vyos_system unit test
* added vyos_system v14 test cases
* vyos_system integration tests fix
* changelog
* domain search integration test
* Update plugins/modules/vyos_system.py
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
|
* clean-up for strreamlines e2e integration tests
* 1.4+ fix for snmp_server
* changes to integratio tests
* changelog fix
* fix ampty json on vyos.py get_config
* fix vyos get_config
* quick fix for teardwon in vyos_system test
* Update plugins/module_utils/network/vyos/vyos.py
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* copilot recommendations incorporated
* remove copilot mess
---------
Co-authored-by: John Doe <johndoe@email.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
|
* t6883 as_prepend_path fix
* chaneglog
* draft for rm community handling
* rm updates
* unit and integration tests for route maps
* rm unit tests fixed
* 1.4+ rm integration tests draft
* rm set_community parser
* large-community 1.4+
* unit tests fix for rm
* extcommunity config
* pre-commit
* cleanup
* RM bandwidth clauses
* fixing parsers index for 1.3 RM
* updated dic
* fix unit tests
* unit tests for RM
* RM unit tests
* dict literal for reserved keywords in unit tests
* set_src ipv6 replaced unit test
* unit test case for RM call
* RM action type unit tests
* rm match protocol code and unit test
* doc update
* set_table RM
* set_table RM unit test
* unit tests refactor
* copilot recommendations
|
|
* T7259 get_config() fix
* fix get_config calls
* vyos_config match=none integration test case
* get_config() update
|
|
* T7162: interface preflight
* fix: update with changes from PR 396
* fix: re-run pre-commit for missed updates
* fix: typo
|
|
Co-authored-by: omnom62 <75066712+omnom62@users.noreply.github.com>
|
|
Co-authored-by: omnom62 <75066712+omnom62@users.noreply.github.com>
|
|
* TestModule profile patch
* changelog
* linter fix
|
|
|
|
|
|
* fw_rules diff init
* comment support
* sanity fix
* remove redundant
* integration tests for fw_rules diff mode
* 1.3- integration tests for firewall_rule diff
* 1.4+ long string linter fix
* 1.3- long string linter fix
* long str rework
* typo
* Remove commented-out diff block in YAML file
removed comments
---------
Co-authored-by: Daniil Baturin <daniil@baturin.org>
|
|
* T7284 (Delete firewall description not possible) Added failing tests.
* T7284 (Delete firewall description not possible) Functional code.
* T7284 (Delete firewall description not possible) Fixed pep8.
* T7284 (Delete firewall description not possible) Added changelog.
* T7284 (Delete firewall description not possible) Fixed changelog.
|
|
* init for ospfv2
* ospfv2 rtt and merge and merge_update tests fixed
* ospfv2 integration test partial fix
* ospfv2 integration tests new structure and 1.4+ fixes
* ospfv2 1.3- unit tests
* ospfv2 changelog
* Added unit tests suite for 1.4+ and vyos_ospfv2
* fixed some netsted dict, added pasive-interface default, added passive-interface exclude for 1.4+
* passive-interface-exclude support for 1.4+
* fix remove func for passive-interface-exclude in 1.4+ ospfv2
* parsed.yaml update
* leftovers removed
* prepare_vyous_tests.yaml to use network_cli and fail fast
* ospfv2 integration tests pre-requisite scripts
* reverted prepare_vyos_tests script
* facts regex corrected
* reworked facts regex
---------
Co-authored-by: Daniil Baturin <daniil@baturin.org>
|
|
* t6831_ospf_vif init
* ospf_vif 1.3 provisioning workflow
* changelog
* ospf-interface unit tests fix
* parser tests
* ospf vif parsers updated
* added vif unt test cases for ospf_interfaces
* ospf vif integration tests
* fixing eth names and utils func
* ospf vif prep/demolih updates
* added new vif test cases
|
|
* T6829 re-init
* re-init T6829 ospfv3 area interface for 1.3-
* changelog
* changelog fix
* Doc is updated
* adding 1.4+ support
* ospfv3 area interfaces for 1.4+
* fixed ospfv3 facts and added area interface unit test for 1.4
* ospfv3 clean-up
* doc updates
|
|
* t6834 init
* facts fixes, WIP
* T6834 - facts updates
* static_routes 1.3- config, facts, args and sanit and unit tests
* changelog
* changelog updates
* static_routes 1.4+
* static_routes v1.4 unit tests
* comment removed
|
|
* T7012 init strucuture for ospf_interfaces integration tests
* ospf_interfaces unit and integration tests fixes for 1.3-
* fixes to v1.4 ospf_interfaces parsers
* T7012 integration tests and 1.4+ support for ospf_interfaces
|
