Page MenuHomeVyOS Platform

Ansible impove support for new 1.4 Firewall Rules Syntax
Closed, ResolvedPublicBUG

Description

The ansible collection cannot handle the new syntax at all so a consumer will get various exceptions. Based on this docs PR, this syntax change happened in 1.4-rolling-202308040557.

Details

Version
1.4-rolling-202308040557
Is it a breaking change?
Perfectly compatible
Issue type
Bug (incorrect behavior)

Event Timeline

I spent some time tonight getting this into a better place with https://github.com/vyos/vyos.vyos/pull/354, but I don't know how to run the tests locally, so I can't verify that my test changes are good. It looks like I need someone to approve some workflows in order for me to get that CI signal.

Viacheslav renamed this task from Impove support for new 1.4 Firewall Rules Syntax to Ansible impove support for new 1.4 Firewall Rules Syntax.Sep 12 2024, 7:42 AM
Viacheslav triaged this task as Normal priority.

@sdwilsh Are you good to close this out? The change in ipv6 should be handled by the updated version now in main

@sdwilsh Unless you have an objection, I'm going to close this out as fixed by the current version in Main.

Are we closing this? Or we have some glowing example of the wrong processing?

This issue should have been resolved by a number of already implemented PRs - e.g. https://github.com/vyos/vyos.vyos/pull/354 or https://github.com/vyos/vyos.vyos/pull/368, therefore I am closing this ticket now. Please feel free to re-open if I am wrong