Skip to content

Conversation

@brbzull0
Copy link
Contributor

The idea is to handle the interrupt signal(Ctrl+C) if you just cancel the metric monitor command, this will basically work as the ping command where as soon as you SIGINT then you get the data collected so far. This also exposes a shared flag to capture the handled signal.

traffic_monitor

@brbzull0 brbzull0 added Tools traffic_ctl traffic_ctl related work. labels Mar 31, 2023
@brbzull0 brbzull0 added this to the 10.0.0 milestone Mar 31, 2023
@brbzull0 brbzull0 self-assigned this Mar 31, 2023
@brbzull0 brbzull0 force-pushed the traffic_ctl_monitor_signal_handling branch from 7dd8011 to d7267d2 Compare March 31, 2023 14:36
@brbzull0 brbzull0 marked this pull request as ready for review March 31, 2023 14:42
@brbzull0 brbzull0 force-pushed the traffic_ctl_monitor_signal_handling branch from d7267d2 to b8dfea3 Compare March 31, 2023 15:28
@brbzull0
Copy link
Contributor Author

brbzull0 commented Apr 3, 2023

[approve ci cmake]

@bneradt bneradt requested a review from cmcfarlen April 3, 2023 22:22
@bneradt
Copy link
Contributor

bneradt commented Apr 8, 2023

[approve ci fedora]

cmcfarlen
cmcfarlen previously approved these changes Apr 20, 2023
@brbzull0 brbzull0 force-pushed the traffic_ctl_monitor_signal_handling branch from b8dfea3 to 111d941 Compare April 20, 2023 17:02
@brbzull0 brbzull0 requested a review from cmcfarlen April 20, 2023 17:42
@brbzull0 brbzull0 merged commit a4729e8 into apache:master Apr 20, 2023
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
* asf/master: (40 commits)
  Change remap filter behavior to match ip_allow.yaml (apache#9631)
  Cleanup: Get rid of dead code from Cache (apache#9621)
  Replace obsolete Debug() macro with Dbg() in SocksProxy.cc. (apache#9613)
  Updates for the new go-httpbin v2.6.0 release. (apache#9633)
  Fix debian symbol not found for test_HttpTransact (apache#9617)
  add traffic_ctl to cmake (apache#9628)
  Fix Proxy Protocol outbound (apache#9632)
  DOC: Fix variable name `proxy.config.exec_thread.autoconfig.enabled`. (apache#9629)
  traffic_ctl: metric monitor. Handle SIGINT to drop collected stats. (apache#9570)
  traffic_ctl: plugin msg command, print out the response from server. (apache#9610)
  Doc: document IP allow filter for remap. (apache#9626)
  Cleanup: Rename d with vol (apache#9619)
  Ensure a reason phrase when sending an HTTP/1 response (apache#9615)
  Cmake plugins and install things (apache#9597)
  quic: Fix session cleanup assert. (apache#9622)
  Enables switching SSL certificates on QUIC with QUICHE (apache#9347)
  Use FetchSM for OCSP HTTP requests (apache#9591)
  Make a couple of the threads configs correct (apache#9604)
  Change submit_and_wait to take ink_hrtime. Fix test_AIO for io_uring. (apache#9555)
  Update build_h3_tools for mac (apache#9608)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tools traffic_ctl traffic_ctl related work.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants