You have commit messages with errors ⧗ input: bgpd: Don't mark nexthop as changed if a set next-hop unchanged is applied An inbound policy preserving the received nexthop into the RIB shouldn't prevent the outbound decision to set next-hop-self or reset the nexthop for EBGP. Those are independent decisions. If an operator wants the nexthop preserved on outbound, they configure neighbor X next-hop-unchanged or an outbound route-map with set ip next-hop unchanged (which would set the flag in out_rmap_flags). Fixes: a725bf4bbb ("bgpd:Fixing a conflict issue between rmap_in_change_flag and rmap_out_change_flag.") Signed-off-by: Martin Winter <mwinter@opensourcerouting.org> Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit e14d1bd9b1f0ce4a84b90156676de45654256c48) ✖ header must not be longer than 72 characters, current length is 74 [header-max-length] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: tests: Check if route-map with set nexthop unchanged does not prevent outgoing Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit 8de3f049fcd80d705e1e717866f626e731b24451) ✖ header must not be longer than 72 characters, current length is 78 [header-max-length] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint

.github/workflows/commitlint.yml (Line: 14, Col: 9): Conditional expression contains literal text outside replacement tokens. This will cause the expression to always evaluate to truthy. Did you mean to put the entire expression inside ${{ }}?