Gaige B Paulsen
Gaige B Paulsen
@andamasov This should be moved over to vyos.dev ; I partially did this in the `README` in a pending PR (not out of draft yet), but it doesn't specifically talk...
Hi @sdwilsh Can you try using the code from #352 and see if that resolves the problems you are seeing with the 1.4 firewall rules? I'm working on getting this...
> @gaige, I left a comment there on at least one thing that this fixes that #352 does not fix, with the line that needs to be updated. thanks, I’ll...
@sdwilsh I think I've got the bulk of your changes in the PR we've been working on. I'm awaiting final review, we've tested against 1.2, 1.3, 1.4, and 1.5 rolling...
@andamasov I've asked @sdwilsh to look at this and make sure we're not leaving anything unfixed based on his checks. However, I don't think we need to bring this over...
The specific change in here to address the new address-family specific naming is already handled in the previously-merged PR. I'm going to close this as redundant.
This is a significant change in behavior. We should document clearly and probably shouldn’t do this on less than a major release due to the non-backwards-compatible behavior. I’m dubious of...
@omnom62 did you run tests against 1.3.8? For 6.x, we need to still ensure these. If you did, please update the PR text. If not, please do.
@omnom62 Can you update the PR template, it seems rather empty.
RST file is missing as well, which is odd because that should be auto-created. Sanity checks are failing as well. This seem s not quite ready, should it be in...