Date: Sat, 13 May 2017 10:09:52 -0700 From: "Ngie Cooper (yaneurabeya)" <yaneurabeya@gmail.com> To: Warner Losh <imp@bsdimp.com> Cc: "Rodney W. Grimes" <rgrimes@freebsd.org>, Ngie Cooper <ngie@freebsd.org>, src-committers <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r318250 - in head: etc etc/newsyslog.conf.d etc/syslog.d tools/build/mk Message-ID: <B486F745-A3A2-4DCB-A453-907C3D73AE12@gmail.com> In-Reply-To: <CANCZdfpmcfU9YCxgZ%2Bi3AnPW77TPk1MRu2msWJ4t-rPE90SCiw@mail.gmail.com> References: <201705131537.v4DFbgWV045290@pdx.rh.CN85.dnsmgr.net> <E6B91EBC-7E7B-4A0B-9807-506A786EDC45@gmail.com> <E92037DB-3739-4BA9-BC96-A50855F19B04@gmail.com> <CANCZdfpmcfU9YCxgZ%2Bi3AnPW77TPk1MRu2msWJ4t-rPE90SCiw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] > On May 13, 2017, at 9:39 AM, Warner Losh <imp@bsdimp.com> wrote: > > On Sat, May 13, 2017 at 10:32 AM, Ngie Cooper <yaneurabeya@gmail.com <mailto:yaneurabeya@gmail.com>> wrote: >> >>> On May 13, 2017, at 09:29, Ngie Cooper <yaneurabeya@gmail.com> wrote: >>> >>> On May 13, 2017, at 08:37, Rodney W. Grimes <freebsd@pdx.rh.CN85.dnsmgr.net> wrote: >>> >>>>> Author: ngie >>>>> Date: Sat May 13 03:10:50 2017 >>>>> New Revision: 318250 >>>>> URL: https://svnweb.freebsd.org/changeset/base/318250 >>>>> >>>>> Log: >>>>> Handle the logfiles in newsyslog and syslogd conditionally, based on >>>>> src.conf(5) knobs >>>>> >>>>> This will allow consumers of FreeBSD to use the unmodified configuration >>>>> files out of the box more than previously. >>>> >>>> What about simply generating proper newsyslog.conf and syslog.conf based >>>> on the the MK_ values rather than change the visible administration >>>> interface that has finger memory, ansible, and puppet support? >>> >>> - My employer doesn't use puppet on its appliances. >>> - Files snapshotted in time bitrot (which was the case with our configs, which is part of the reason behind this change. > > couldn't you have an Isilion config file in the appropriate directory > instead? And use the default newsyslog / syslog config files? I’m not sure I’m in disagreement with this statement, but the fact is that there are a handful of config files, about 200 lines each, which drive newsyslog/syslogd on OneFS, with a fair amount of duplication. I’m trying to simplify the mess I’m presented with at work, one bit at a time. >>> - It's really easy to screw up a mergemaster call if you edit the files, and install the stock version which removes the edits. >> >> Also, programmatically removing the entries means you have to bake the metadata into etc/Makefile, which is already complicated enough as-is. > > Why do you care about removing them at all? They are no-ops if the > files don't exist. Why not just always install all these files is > where I'm going with this… I don’t follow. Could you please restate this paragraph? Thanks, -Ngie [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- iQIcBAEBCgAGBQJZFz3gAAoJEPWDqSZpMIYV5dAP/REv7uL4OCv1mMMMShFpHQgG v1P+4qx9JRw+o+AEIuk9FNN5we2Utz4chTSYsoZzidrEJoqIj4vQZHVLVpiy+8/p TnqHguzPI6NawkjJIy1u0/cQS0esrOr808wuYpS2pxlQUllZNA1VQ2Sztgdd6LU9 zL2GMeRKRWLUQlUkImgCz1Nmgky+2lB+dL3kR4yuAf2b3Lnqr5YIfrMX0fhMYEmw Pp11ybVuDRBEMvSzWLGh/tCTyj6L3EuwJEU5F6JjUrgZAlrao6sKSNeZnrJk6ORa p2o2TnTFlV5lZouPwDnP7MLTK9/gt5z1qvv7Wbuil0vjNJRbQNDHWj0bdCEsU/ze 0Wu1vnVDPy5R5jBv9ZGTZcge08mQEeiu0/0G9JFn76na8IudSG8HmhPgeQvy3BXH 5sr+S90JTVz+IRO98SroK4qy+2n1UZw5BL0/2nVvLBhRDNmYrPL9MD9UpdTZGbgI 4P/6vLZpYGT2DCwpMGTDqY/hRpesLR9G0BiHBgKY4M29UqOItJuXG8A0PYOTDlM2 tDzH2lpMRnfHjhriIDg8qzTfaNUpf9dqDjYAR07Se1hcHnwUYU86Qo28g5CEPBbE uL+DvO8/s19Ye2Cv3GYdFwabEVnBCTZPAI+9KA1Qa7Od90rV+137Z843YkDaoDjS vQhg1DBofkOi1S0CQAjj =LJZA -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?B486F745-A3A2-4DCB-A453-907C3D73AE12>