Skip to content

Conversation

@zwoop
Copy link
Contributor

@zwoop zwoop commented Oct 8, 2024

The point of this exercise is to keep the "copied" plugins on the filesystem, for the lifetime of their use. This makes it possible to get full backtrace from a crash / core file, before this patch all debug info is lost with the .so being removed immediately on load / reload.

@zwoop zwoop added the Plugins label Oct 8, 2024
@zwoop zwoop added this to the 10.1.0 milestone Oct 8, 2024
@zwoop zwoop requested review from bryancall and cmcfarlen October 8, 2024 20:10
@zwoop zwoop self-assigned this Oct 8, 2024
@zwoop zwoop force-pushed the KeepReloadablePlugins branch from 777284a to 5f568cd Compare October 8, 2024 20:10
@zwoop zwoop force-pushed the KeepReloadablePlugins branch from 1299bad to 3147d34 Compare October 8, 2024 20:28
@zwoop zwoop force-pushed the KeepReloadablePlugins branch from cdfc391 to e9b32e1 Compare October 9, 2024 16:57
@zwoop zwoop force-pushed the KeepReloadablePlugins branch from e9b32e1 to f4030ff Compare October 10, 2024 14:49
@zwoop
Copy link
Contributor Author

zwoop commented Oct 11, 2024

[approve ci autest]

@zwoop zwoop merged commit 71293ea into apache:master Oct 15, 2024
@zwoop zwoop deleted the KeepReloadablePlugins branch October 15, 2024 19:56
masaori335 pushed a commit to masaori335/trafficserver that referenced this pull request May 29, 2025
* Defer deletion of reloadable remap plugins

* Fix tests, since plugins now remains

* Put the startup cleanup into a try-catch

* Make autest happier during shutdown

(cherry picked from commit 71293ea)

Co-authored-by: Leif Hedstrom <zwoop@apache.org>
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.

3 participants