Ansible Network Collection for VyOS (mirror of https://github.com/vyos/vyos.vyos.git) https://git.amelek.net/vyos/vyos.vyos.git/atom?h=dependabot%2Fgithub_actions%2Fansible%2Fteam-devtools%2Fdot-github%2Fworkflows%2Fah_token_refresh.yml-26.2.0 2026-04-26T14:15:30+00:00 2026-04-26T14:15:30+00:00 Yuriy Andamasov yuriy@vyos.io 2026-04-26T14:15:30+00:00 urn:sha1:3f4304b00af36fe3bcdeff1357da582fda5d8011 2026-04-26T14:11:07+00:00 Yuriy Andamasov yuriy@vyos.io 2026-04-26T14:11:07+00:00 urn:sha1:24814ecbe107e2aa9198d0a96361c7c5b0b0378c * T8584: add CodeRabbit review configuration 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) * T8584: fix changelog glob, valid keys, and vyos_vrf classification - 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) * T8584: add ansible/ansible and ansible.netcommon as linked repositories 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) 2026-04-17T12:33:40+00:00 omnom62 75066712+omnom62@users.noreply.github.com 2026-04-17T12:33:40+00:00 urn:sha1:2a5f7b17d0d3b18ae73c338971f3e933f0768f15 2026-04-17T12:22:01+00:00 Yuriy Andamasov yuriy@vyos.io 2026-04-17T12:22:01+00:00 urn:sha1:db645c3fc5a66f0fdadd03b5096937f70787e3b0 * 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> 2026-04-16T03:38:44+00:00 Yuriy Andamasov yuriy@vyos.io 2026-04-16T03:38:44+00:00 urn:sha1:e79e6658fde365a783b35cd8f99345209d67c15a 2026-04-16T03:38:28+00:00 dependabot[bot] 49699333+dependabot[bot]@users.noreply.github.com 2026-04-16T03:38:28+00:00 urn:sha1:1bee3f7d8e1dca104c0b5c5d7a7833946aa7514a 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> 2026-04-14T19:31:42+00:00 omnom62 75066712+omnom62@users.noreply.github.com 2026-04-14T19:31:42+00:00 urn:sha1:fd6dac425eb92cb39d4df2e3e955b7b2abac27a5 2026-04-14T17:51:12+00:00 omnom62 75066712+omnom62@users.noreply.github.com 2026-04-14T17:51:12+00:00 urn:sha1:61e70a448db1381695700eb0c7701e68230a0b9d * sanity tests 2.22 ignore * Ignore 2.22 for sanity 2026-04-07T19:18:50+00:00 omnom62 75066712+omnom62@users.noreply.github.com 2026-04-07T19:18:50+00:00 urn:sha1:cf76cff9a8f56dad3d65aab8e34dda8b4d94d95a 2026-04-07T11:46:41+00:00 gideon-kuijt-northwave gideon.kuijt@northwave-cybersecurity.com 2026-04-07T11:46:41+00:00 urn:sha1:16e6f21de8e90833e2284aeab6cd39d2e1482155 * 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>