Skip to content

Conversation

@sever-sever
Copy link
Member

@sever-sever sever-sever commented Oct 27, 2025

Change summary

Bump hsflowd version to v2.1.17-1

$ git log --oneline dffc277..v2.1.17-1
6949d81 (HEAD, tag: v2.1.17-1, origin/master, origin/HEAD, master) bump version to 2.1.17-1
1edc4d4 fix file-descriptor leak when mod_tcp opens netlink sockets in other network namespaces
e984add improve logging
d521a7a readVNIC logging
ab27cd3 clean up file descriptors in readVNIC
016b990 simplify EVSocket lifecycle
3f9dfdd mod_nlroute improve logging
ba0d630 improve mod_pcap logging
835fcde check FD_SETSIZE
a6a00ed (tag: v2.1.16-1) bump version
f89573d avoid SIGSEGV triggered by retrying collector socket opening which config is missing (e.g. when last SONiC collector is removed)
b6f02df more Dockerfiles add clang
1eec6a5 allow mod_psample to compile on ubuntu20 with newer fields
a0020c9 (tag: v2.1.15-1) reduce mod_epcap logging
998a600 tweak Docker build files
93d7c54 update docker build files
358001e bump version to 2.1.15-1
590656f Fix Debian 12 build and add Debian 13 (#83)
c601aed Accommodate PSAMPLE_ATTR_SAMPLE_PROBABILITY if it appears and warn that it is problematic
db237d2 mod_epcap use BUSY event to allow for default notification from libbpf
0195232 Add EVEventRxOff to unsubscribe,  and BUSY event for bursts of rapid polling


Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe): Update hsflowd version

Related Task(s)

Related PR(s)

check

vyos@r14:~$ show version all | match hsflow
ii  hsflowd                          2.1.17-1                                 amd64        sFlow(R) monitoring agent
vyos@r14:~$ 

set system sflow agent-address '10.0.0.1'
set system sflow interface 'eth0'
set system sflow sampling-rate '1'
set system sflow server 10.0.0.2

Check traffic on the sFlow server:

vyos@r16# sudo goflow2 | jq
time=2025-10-27T12:05:06.736Z level=INFO msg="starting GoFlow2"
time=2025-10-27T12:05:06.736Z level=INFO msg="starting collection" scheme=sflow hostname="" port=6343 count=1 workers=2 blocking=false queue_size=1000000
time=2025-10-27T12:05:06.738Z level=INFO msg="starting collection" scheme=netflow hostname="" port=2055 count=1 workers=2 blocking=false queue_size=1000000
{
  "type": "SFLOW_5",
  "time_received_ns": 1761566707347156000,
  "sequence_num": 102,
  "sampling_rate": 1,
  "sampler_address": "10.0.0.1",
  "time_flow_start_ns": 1761566707347156000,
  "time_flow_end_ns": 1761566707347156000,
  "bytes": 102,
  "packets": 1,
  "src_addr": "192.168.122.14",
  "dst_addr": "1.1.1.1",
  "etype": "IPv4",
  "proto": "ICMP",
  "src_port": 0,
  "dst_port": 0,
  "in_if": 1073741823,
  "out_if": 2,
  "src_mac": "52:54:00:8d:67:6e",
  "dst_mac": "52:54:00:d4:d9:a1",
  "src_vlan": 0,
  "dst_vlan": 0,
  "vlan_id": 0,
  "ip_tos": 0,
  "forwarding_status": 0,
  "ip_ttl": 64,
  "ip_flags": 2,
  "tcp_flags": 0,
  "icmp_type": 0,
  "icmp_code": 0,
  "ipv6_flow_label": 0,
  "fragment_id": 18888,
  "fragment_offset": 0,
  "src_as": 0,
  "dst_as": 0,
  "next_hop": "",
  "next_hop_as": 0,
  "src_net": "0.0.0.0/0",
  "dst_net": "0.0.0.0/0",
  "bgp_next_hop": "",
  "bgp_communities": [],
  "as_path": [],
  "mpls_ttl": [],
  "mpls_label": [],
  "mpls_ip": [],
  "observation_domain_id": 0,
  "observation_point_id": 0,
  "layer_stack": [
    "Ethernet",
    "IPv4",
    "ICMP"
  ],
  "layer_size": [
    14,
    20,
    8
  ],
  "ipv6_routing_header_addresses": [],
  "ipv6_routing_header_seg_left": 0
}
...

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

@github-actions
Copy link

👍
No issues in PR Title / Commit Title

@github-actions github-actions bot added the current VyOS rolling release label Oct 27, 2025
@sever-sever sever-sever requested review from c-po and dmbaturin October 27, 2025 10:16
Copy link
Member

@dmbaturin dmbaturin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If it works and creates no issues, I'm certainly not against updating it in rolling.

Copy link
Contributor

@jestabro jestabro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No objections to update.

@jestabro jestabro merged commit 3e81f2e into current Oct 28, 2025
8 checks passed
@jestabro jestabro deleted the T7958 branch October 28, 2025 15:21
@vyosbot vyosbot added mirror-initiated This PR initiated for mirror sync workflow mirror-completed and removed mirror-initiated This PR initiated for mirror sync workflow labels Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

current VyOS rolling release mirror-completed

Development

Successfully merging this pull request may close these issues.

5 participants