In T5586#160073, @Apachez wrote:How does FRR/vrrpd work regarding SNMP compatability?
Im thinking if the keepalived could be replaced in favour of FRR/vrrpd?
And for now keep keepalived around only for virtual-server (unless that too can be dealt with by FRR/vrrpd)?
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed All Stories
All Stories
All Stories
Sep 18 2023
Sep 18 2023
GitHub <[email protected]> committed rVYOSONEX9620a233c093: Merge pull request #2281 from nicolas-fort/T5594 (authored by c-po).
Viacheslav moved T5586: Disable by default SNMP for Keepalived VRRP from Open to Finished on the VyOS 1.4 Sagitta board.
PR for latest: https://github.com/vyos/vyos-1x/pull/2281
PR for Equuleus: https://github.com/vyos/vyos-1x/pull/2282
PR created: https://github.com/vyos/vyos-1x/pull/2280
Viacheslav triaged T5594: VRRP - Error if using IPv6 Link Local as hello source address as High priority.
n.fort changed the status of T5594: VRRP - Error if using IPv6 Link Local as hello source address from Open to In progress.
Apachez closed T5431: Services not enabled or configured are started anyway within frr-family as Wontfix.
I will put this as "wontfix" because a fix would need additional work of stopping/starting correct FRR-service (if this occurs in future then this task can be reopened).
I will close this task as a duplicate of https://vyos.dev/T4502 where work is in progress.
Using VyOS 1.5-rolling-202309170024.
So what remains is how to solve point 1.
It looks like point 3 can be taken care of:
r4# show version FRRouting 9.0.1 (r4) on Linux(6.1.53-amd64-vyos)
Still has bugs
For example with redistribute
r4# conf t r4(config)# router eigrp 65001 r4(config-router)# redistribute connected % Configuration failed.
Added PR https://github.com/vyos/vyos-1x/pull/2278 for smoketest fix.
Ok, I was thinking if that then waited for some password or such.
I haven't tried anything else since I rebooted back into 1.4, but I did try sudo su - which gave the same error.
How does a simple "sudo bash" work?
Cannot pass the smoketest:
DEBUG - ====================================================================== DEBUG - FAIL: test_01_dyndns_service_standard (__main__.TestServiceDDNS.test_01_dyndns_service_standard) DEBUG - ---------------------------------------------------------------------- DEBUG - Traceback (most recent call last): DEBUG - File "/usr/libexec/vyos/tests/smoke/cli/test_service_dns_dynamic.py", line 82, in test_01_dyndns_service_standard DEBUG - self.assertIn(f'use=if', ddclient_conf) DEBUG - AssertionError: 'use=if' not found in '### Autogenerated by dns_dynamic.py ###\ndaemon=300\nsyslog=yes\nssl=yes\npid=/run/ddclient/ddclient.pid\ncache=/run/ddclient/ddclient.cache\nweb=googledomains use=no \n # Web service dynamic DNS configuration for cloudflare: [cloudflare, test.ddns.vyos.io]\nusev4=ifv4, \\\nifv4=eth0, \\\nprotocol=cloudflare, \\\nzone=vyos.io, \\\npassword=paSS_@4ord \\\ntest.ddns.vyos.io' DEBUG - DEBUG - ---------------------------------------------------------------------- DEBUG - Ran 4 tests in 11.489s DEBUG - DEBUG - FAILED (failures=1)
Testing point 2 above with VyOS 1.5-rolling-202309170024.
I think this broke a whole lot of things for RADIUS users (these work fine in 1.4-rolling-202308040317, but are broken in 1.5-rolling-202309170024):
Sep 17 2023
Sep 17 2023
Apachez added a comment to T5532: After add system image the boot stuck and works again after the second reboot.
This is the one I was thinking of:
Apachez added a comment to T5532: After add system image the boot stuck and works again after the second reboot.
This seems to have been resolved since a few days (1.5-rolling nightly).
GitHub <[email protected]> committed rVYOSONEX665523ac1b00: Merge pull request #2251 from indrajitr/ddclient-cache-fix (authored by c-po).
c-po moved T5592: salt: upgrade minion to 3005.2 from Open to Finished on the VyOS 1.4 Sagitta board.
c-po moved T5592: salt: upgrade minion to 3005.2 from Open to Finished on the VyOS 1.5 Circinus board.
PR for sagitta https://github.com/vyos/vyos-build/pull/415
c-po renamed T5592: salt: upgrade minion to 3005.2 from salt: upgrade minion to 3006.3 to salt: upgrade minion to 3005.2.
@Apachez The empty table inet vyos_offload is to ensure the table exists before deleting its content. Regarding hardware offload, I don't have a hardware supporting that. The implementation is totally based on documentation and I don't add any checks before applying the nftables config.
Output of ps auxww | grep -i frr after proposed v3 is applied:
Update of proposed changes (Ill call them proposed v3).
c-po added a parent task for T3761: Bump salt-minion to 3000: T5592: salt: upgrade minion to 3005.2.
c-po added a parent task for T4364: salt-minion: Upgrade to 3004 and migrate to get_config_dict(): T5592: salt: upgrade minion to 3005.2.
Update of proposed changes (Ill call them proposed v2).
If we alter the Jinja2 file we should also update the syntax - just ospf an example but its valid for all other SNMP stuff:
mgmtd can not be disabled. It's a mandatory FRR process.
I can create a PR if everyone agrees upon the proposed changes above however I have limited capability to test each and everyone of these changes.
Proposed data/templates/frr/daemons.frr.tmpl:
Sep 16 2023
Sep 16 2023
sarthurdev changed the status of T5571: Firewall does not delete networks from the table raw from Confirmed to Needs testing.
Fixed in PR: https://github.com/vyos/vyos-1x/pull/2276
GitHub <[email protected]> committed rVYOSONEX8f79a5cb4ee9: Merge pull request #2267 from vyos/mergify/bp/sagitta/pr-2253 (authored by c-po).
GitHub <[email protected]> committed rVYOSONEX50bbac2a4227: Merge pull request #2275 from c-po/sagitta (authored by c-po).
Apachez added a comment to T5511: Cleanup of unused directories (and files) in order to shrink image-size.
PR created: https://github.com/vyos/vyos-build/pull/414
Apachez added a comment to T5511: Cleanup of unused directories (and files) in order to shrink image-size.
PR created: https://github.com/vyos/vyos-build/pull/413
Sep 15 2023
Sep 15 2023
GitHub <[email protected]> committed rVYOSONEX4ce8e3b54162: Merge pull request #2271 from vyos/mergify/bp/sagitta/pr-2270 (authored by c-po).
GitHub <[email protected]> committed rVYOSONEX41341f776cef: Merge pull request #2266 from vyos/mergify/bp/sagitta/pr-2255 (authored by c-po).
GitHub <[email protected]> committed rVYOSONEX768df77b078a: Merge pull request #2265 from vyos/mergify/bp/sagitta/pr-2262 (authored by Viacheslav).
- Using hardware flowtable (flags offload;) on an interface which doesnt support it returns an error:
c-po moved T5581: Add "show ip nht" op-mode command (IPv4 nexthop tracking table) from In Progress to Finished on the VyOS 1.4 Sagitta board.
Tried to enable both software and hardware flowtable with VyOS 1.5-rolling-202309151051:
How does FRR/vrrpd work regarding SNMP compatability?
GitHub <[email protected]> committed rVYOSONEX3cd0ef72825d: Merge pull request #2274 from vyos/mergify/bp/sagitta/pr-2273 (authored by c-po).
Viacheslav moved T5586: Disable by default SNMP for Keepalived VRRP from Open to Finished on the VyOS 1.5 Circinus board.
Fixed
Mergify <37929162+mergify[bot]@users.noreply.github.com> committed rVYOSONEX0c94a95a2de5: T5586: Disable by default SNMP for Keeplived VRRP service (authored by Viacheslav).
GitHub <[email protected]> committed rVYOSONEX8c3ce3cd28ed: Merge pull request #2273 from sever-sever/T5586 (authored by c-po).
GitHub <[email protected]> committed rVYOSONEX27ad9f5ba643: Merge pull request #2185 from sever-sever/T5261-new (authored by Viacheslav).
GitHub <[email protected]> committed rVYOSONEX87af54425973: Merge pull request #2272 from vfreex/fix-t4502 (authored by Viacheslav).
sarthurdev moved T5568: Install image from live ISO always defaults boot to KVM entry from Open to In Progress on the VyOS 1.4 Sagitta board.
https://github.com/vyos/vyos-1x/pull/2272 should fix this
Viacheslav changed the status of T5586: Disable by default SNMP for Keepalived VRRP from Open to In progress.
Viacheslav renamed T5586: Disable by default SNMP for Keepalived VRRP from Disable be default SNMP for Keepalived VRRP to Disable by default SNMP for Keepalived VRRP.
Some extra lines were mistakenly included during rebase:
Viacheslav changed the status of T5579: Log firewall - Wrong command after firewall refactor, a subtask of T5160: Firewall refactor, from In progress to Needs testing.
Viacheslav changed the status of T5579: Log firewall - Wrong command after firewall refactor from In progress to Needs testing.
Viacheslav changed the status of T5574: Support per-service cache management for dynamic dns providers from Open to Needs testing.
Viacheslav changed the status of T5585: Fix file access mode for dynamic dns configuration from Open to Needs testing.
Mergify <37929162+mergify[bot]@users.noreply.github.com> committed rVYOSONEX0ea915fc81bf: ddclient: T5585: Fix file access mode for dynamic dns configuration (authored by indrajitr).
GitHub <[email protected]> committed rVYOSONEX8f705fb8cb0c: Merge pull request #2270 from indrajitr/ddclient-config-permission (authored by c-po).
indrajitr updated the task description for T5574: Support per-service cache management for dynamic dns providers.
indrajitr renamed T5574: Support per-service cache management for dynamic dns providers from Inprove and refactor dns dynamic service configuration to Support per-service cache management for dynamic dns providers.