mbox series

[0/5] Flight Data Recorder Update

Message ID 20220130190611.12292-1-steved@redhat.com (mailing list archive)
Headers show
Series Flight Data Recorder Update | expand

Message

Steve Dickson Jan. 30, 2022, 7:06 p.m. UTC
Here a few updates to the FDR package that
will allow it to work out of the box when
the RPM is install... Kinda. 

When the rpm is installed, the daemon is not enable or
started and the config file is not enabled. But it 
makes much easier set a tracepoint.  Uncomment
the needed tracepoint and start the fdrd daemon.
Then do tail -f on /var/log/nfs.log.

The package is very light weight: one daemon, two config 
files and one example file. The config file that is installed
has all of the nfs,nfsv4 and sunrpc tracepoints, commented
out. 

The first 4 patches are clean up... Making things work
right out of the box and conforming to current
presidencies. The last patch, installing an nfsall.conf
config, could be questionable. Do the maintainers want
to maintain config files or have the distros do it?

I guess the bottom line is this. At the last bakeathon
we talked about moving away from rpcdebug setting 
dprintks to tracepoints (which is a good thing IMHO).
Now, it is not clear how it can be done, but I'm
hoping this is the first step in that direction.


Steve Dickson (5):
  Makefile: cleaned up the usage of RPMBUILD_DIR
  fdr.spec: logrotate clean up
  systemd: rename service file.
  saveto verb: Document updated
  configs: Add an nfsall.conf config file

 Makefile                    |   8 +-
 buildrpm/1.3/fdr.spec       |  25 ++-
 configs/nfsall.conf         | 319 ++++++++++++++++++++++++++++++++++++
 fdrd.man                    |   9 +-
 fdr.service => fdrd.service |   0
 samples/nfs                 |   2 +
 6 files changed, 349 insertions(+), 14 deletions(-)
 create mode 100644 configs/nfsall.conf
 rename fdr.service => fdrd.service (100%)