Skip to content

[action] [PR:17616] Do not pass the option "device" in rsyslog.conf by default when syslog server's source address is configured (#17616)#18518

Merged
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/17616
Apr 1, 2024
Merged

[action] [PR:17616] Do not pass the option "device" in rsyslog.conf by default when syslog server's source address is configured (#17616)#18518
mssonicbld merged 1 commit intosonic-net:202311from
mssonicbld:cherry/202311/17616

Conversation

@mssonicbld
Copy link
Collaborator

Why I did it

An in-band syslog server will not receive any syslog if it is configured without a VRF specified, which is because eth0 is always specified as the device of a syslog server and the syslog packets will be sent to eth0 regardless of its destination IP address.

How I did it

Pass the option "device" in rsyslog.conf only if when syslog server's source address is configured with a non-default VRF

How to verify it

Manually test:

  1. Configuring a syslog server without VRF specified or with default as the VRF: no device passed in rsyslog.conf
  2. Configuring a syslog server with non-default VRF: the configured VRF passed as device in rsyslog.conf

…g server's source address is configured (sonic-net#17616)

### Why I did it

An in-band syslog server will not receive any syslog if it is configured without a VRF specified, which is because `eth0` is always specified as the `device` of a syslog server and the syslog packets will be sent to `eth0` regardless of its destination IP address.

### How I did it

Pass the option "device" in rsyslog.conf only if when syslog server's source address is configured with a non-default VRF

#### How to verify it

Manually test:
1. Configuring a syslog server without VRF specified or with `default` as the VRF: no `device` passed in `rsyslog.conf`
2. Configuring a syslog server with non-default VRF: the configured VRF passed as `device` in `rsyslog.conf`
@mssonicbld
Copy link
Collaborator Author

Original PR: #17616

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants