Hi Rainer, Thanks for confirming! I will add more tests when I'll have time to do so.
Please let me know if I can help by making changes - like you said, they should work on a default system. So I can add a check and skip each ES test (how can a test be skipped?) if ES isn't there. I can also make the tests install ES with a couple of lines. The trouble is, there are two such tests now (and hopefully more in the future), and we don't want to download, install and start ES for each of those tests. Grouping tests would be nice, but I'm not sure how one would do that nicely. Making a script that runs all omelasticsearch tests might work - and then ES install and cleanup can happen only in the "master" script. Best regards, Radu 2013/7/19 Rainer Gerhards <[email protected]> > I can't look in-depth right now, but new tests are always useful. I > probably need to see that I can make them only execute if ES is there (and > the other components). A key point is that they must be able to work on a > more or less default system OR carry everything that is needed to setup > them within themselfes. > > If I manage to have a deeper look and merge before I leave, I'll do. > Otherwise after my vacation. But please keep them flowing :-) > > Rainer > > > On Fri, Jul 19, 2013 at 2:54 PM, Radu Gheorghe <[email protected] > >wrote: > > > Same for omelasticsearch: > > > > > https://github.com/sematext/rsyslog/commit/5213a7d1791549151698c66cddaf1e1ec6ed665b > > > > I'd like to know if anyone finds these tests useful. > > > > > > 2013/7/19 Radu Gheorghe <[email protected]> > > > > > Hello again :) > > > > > > The same seems to apply for mmjson parse. I've added a test here: > > > > > > > > > https://github.com/sematext/rsyslog/commit/30970c67ef4451aee73545c26c43a527c658097e > > > > > > I hope I'm on the right path with these :) > > > > > > Best regards, > > > Radu > > > > > > > > > 2013/7/18 Radu Gheorghe <[email protected]> > > > > > >> Hello, > > >> > > >> I didn't find any tests for RELP in the testbench so I thought I'd > make > > >> one. Here are the files: > > >> https://github.com/sematext/rsyslog/blob/master/tests/sndrcv_relp.sh > > >> > > >> > > > https://github.com/sematext/rsyslog/blob/master/tests/testsuites/sndrcv_relp_sender.conf > > >> > > >> > > > https://github.com/sematext/rsyslog/blob/master/tests/testsuites/sndrcv_relp_rcvr.conf > > >> > > >> Can they be merged in rsyslog? Or is there something missing/needed? > > >> > > >> Best regards, > > >> Radu > > >> > > > > > > > > _______________________________________________ > > rsyslog mailing list > > http://lists.adiscon.net/mailman/listinfo/rsyslog > > http://www.rsyslog.com/professional-services/ > > What's up with rsyslog? Follow https://twitter.com/rgerhards > > NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad > > of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you > > DON'T LIKE THAT. > > > _______________________________________________ > rsyslog mailing list > http://lists.adiscon.net/mailman/listinfo/rsyslog > http://www.rsyslog.com/professional-services/ > What's up with rsyslog? Follow https://twitter.com/rgerhards > NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad > of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you > DON'T LIKE THAT. > _______________________________________________ rsyslog mailing list http://lists.adiscon.net/mailman/listinfo/rsyslog http://www.rsyslog.com/professional-services/ What's up with rsyslog? Follow https://twitter.com/rgerhards NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T LIKE THAT.
