Skip to content
Browse files

Add varnishlog capability to varnishtest

What this adds:

- When given a second -v, varnishlog output is added to the overall test log.

- The varnish component gets a -logexpecttag action. It takes 2 arguments, a
number (0+) and a VSL tag. It will see how many of said tag have passed by
since the child was started, and match it with the number.

- And it gets a -logexpectregex action. Again 2 arguments, a number (0+) and a
regex. The regex is matched with the contents of the loglines, just like
varnishlog et al do.

Do give the varnishd a smidgen of time to write to the VSL before doing your
expects. How much is a bit of an inexact science, but 0.2 has been working for
me. (See a00012.vtc)

Any comments are welcome, as are suggestions for better alternatives to
-logexpect(tag|regex). They're a bit horrible in length.

Also, I'm tempted to scrap the second verbosity level and always include VSL
in the test log. Opinions on that?
  • Loading branch information...
1 parent 6447d64 commit 8c2c9f21f6ab9225d55d4c6c687c348d8d9ebcda @drwilco committed Sep 13, 2011
Showing with 298 additions and 3 deletions.
  1. +94 −0 bin/varnishtest/tests/a00012.vtc
  2. +1 −0 bin/varnishtest/vtc.h
  3. +3 −2 bin/varnishtest/vtc_main.c
  4. +200 −1 bin/varnishtest/vtc_varnish.c
View
94 bin/varnishtest/tests/a00012.vtc
@@ -0,0 +1,94 @@
+varnishtest "Test varnishlog function"
+
+server s1 {
+ rxreq
+ txresp -hdr "X-Foo: Bar" -hdr "Cache-Control: max-age=0"
+} -start
+server s2 {
+ rxreq
+ txresp -hdr "X-Foo: Bar" -hdr "Cache-Control: max-age=0"
+} -start
+
+varnish v1 -vcl+backend {
+import std from "${topbuild}/lib/libvmod_std/.libs/libvmod_std.so";
+ sub vcl_recv {
+ set req.backend = s2;
+ set req.backend = s1;
+ std.log("this is v1");
+ }
+} -start
+
+varnish v2 -vcl+backend {
+import std from "${topbuild}/lib/libvmod_std/.libs/libvmod_std.so";
+ sub vcl_recv {
+ set req.backend = s1;
+ set req.backend = s2;
+ std.log("this is v2");
+ std.log("no really!");
+ }
+} -start
+
+client c1 {
+ txreq
+ rxresp
+ expect resp.http.X-Foo == "Bar"
+} -run
+
+client c2 -connect ${v2_sock} {
+ txreq
+ rxresp
+ expect resp.http.X-Foo == "Bar"
+} -run
+
+# give the log a moment to catch up
+delay 0.2
+
+# there should be no crosstalk
+varnish v1 -logexpecttag 1 VCL_Log
+varnish v2 -logexpecttag 2 VCL_Log
+
+# shouldn't be case sensitive
+varnish v1 -logexpecttag 1 vcl_log
+varnish v1 -logexpecttag 1 vcl_lOG
+
+varnish v1 -logexpecttag 1 RxURL
+
+server s1 -wait
+
+varnish v1 -stop -start
+
+server s1 {
+ rxreq
+ txresp -hdr "X-Foo: Bar"
+} -start
+
+varnish v1 -logexpecttag 0 RxURL
+
+client c1 {
+ txreq
+ rxresp
+ expect resp.http.X-Foo == "Bar"
+} -run
+
+delay 0.2
+
+varnish v1 -logexpectregex 3 "X-Foo: Bar"
+varnish v1 -logexpectregex 3 "X-Foo: B.."
+
+# check proper multiline handling
+varnish v1 -logexpectregex 3 "^X-Foo: Bar"
+varnish v1 -logexpectregex 3 "X-Foo: Bar$"
+varnish v1 -logexpectregex 0 "^Bar"
+varnish v1 -logexpectregex 0 "X-Foo$"
+
+# there should be no crosstalk
+varnish v1 -logexpectregex 1 "^this is v1$"
+varnish v1 -logexpectregex 0 "^this is v2$"
+varnish v2 -logexpectregex 0 "^this is v1$"
+varnish v2 -logexpectregex 1 "^this is v2$"
+
+# and finally, nothing should be found after a restart
+varnish v1 -stop -start
+delay 0.2
+varnish v1 -logexpectregex 0 "X-Foo: Bar"
+varnish v1 -logexpecttag 0 VCL_Log
View
1 bin/varnishtest/vtc.h
@@ -61,6 +61,7 @@ cmd_f cmd_sema;
extern volatile sig_atomic_t vtc_error; /* Error, bail out */
extern int vtc_stop; /* Abandon current test, no error */
extern pthread_t vtc_thread;
+extern int vtc_verbosity;
void init_sema(void);
View
5 bin/varnishtest/vtc_main.c
@@ -83,11 +83,12 @@ static int njob = 0;
static int npar = 1; /* Number of parallel tests */
static unsigned vtc_maxdur = 60; /* Max duration of any test */
static int vtc_continue; /* Continue on error */
-static int vtc_verbosity = 1; /* Verbosity Level */
static int vtc_good;
static int vtc_fail;
static int leave_temp;
+int vtc_verbosity = 1; /* Verbosity Level */
+
/**********************************************************************
* Parse a -D option argument into a name/val pair, and insert
* into extmacro list
@@ -370,7 +371,7 @@ main(int argc, char * const *argv)
vtc_maxdur = strtoul(optarg, NULL, 0);
break;
case 'v':
- if (vtc_verbosity < 2)
+ if (vtc_verbosity < 3)
vtc_verbosity++;
break;
default:
View
201 bin/varnishtest/vtc_varnish.c
@@ -37,6 +37,7 @@
#include <signal.h>
#include <poll.h>
#include <inttypes.h>
+#include <ctype.h>
#include <sys/types.h>
#include <sys/wait.h>
@@ -50,15 +51,18 @@
#include "cli_common.h"
#include "vss.h"
#include "vsb.h"
+#include "vsl.h"
#include "vtc.h"
struct varnish {
unsigned magic;
#define VARNISH_MAGIC 0x208cd8e3
char *name;
+ /* one log per thread */
struct vtclog *vl;
struct vtclog *vl1;
+ struct vtclog *vl2;
VTAILQ_ENTRY(varnish) list;
struct vsb *storage;
@@ -68,16 +72,28 @@ struct varnish {
pid_t pid;
pthread_t tp;
+ pthread_t ltp;
+ pthread_mutex_t lsmtx;
+ pthread_cond_t lscnd;
int cli_fd;
int vcl_nbr;
char *workdir;
struct VSM_data *vd;
+
+ /* XXX merge lvd with vd once VSM_Setup allows it */
+ struct VSM_data *lvd;
+ int lr;
+ int ts[256];
+ struct VSM_data *revd;
+ int reh;
};
#define NONSENSE "%XJEIFLH|)Xspa8P"
+#define SLEEP_USEC (50*1000)
+
static VTAILQ_HEAD(, varnish) varnishes =
VTAILQ_HEAD_INITIALIZER(varnishes);
@@ -116,6 +132,51 @@ varnish_ask_cli(const struct varnish *v, const char *cmd, char **repl)
}
/**********************************************************************
+ * Varnishlog gatherer + thread
+ */
+
+static int
+h_addlog(void *priv, enum VSL_tag_e tag, unsigned fd, unsigned len,
+ unsigned spec, const char *ptr, uint64_t bitmap)
+{
+ struct varnish *v;
+ int type;
+
+ (void) bitmap;
+
+ type = (spec & VSL_S_CLIENT) ? 'c' :
+ (spec & VSL_S_BACKEND) ? 'b' : '-';
+ CAST_OBJ_NOTNULL(v, priv, VARNISH_MAGIC);
+
+ v->ts[tag]++;
+
+ if (vtc_verbosity > 2)
+ vtc_log(v->vl2, 4, "varnishlog| %5u %-12s %c %.*s", fd,
+ VSL_tags[tag], type, len, ptr);
+ return (0);
+}
+
+static void *
+varnishlog_thread(void *priv)
+{
+ struct varnish *v;
+
+ CAST_OBJ_NOTNULL(v, priv, VARNISH_MAGIC);
+ while (v->lr && VSL_Open(v->lvd, 0) != 0) {
+ assert(usleep(SLEEP_USEC) == 0 || errno == EINTR);
+ }
+ while (v->lr) {
+ (void) VSL_Dispatch(v->lvd, h_addlog, v);
+ /* tell main thread we're at the end of the log */
+ pthread_mutex_lock(&v->lsmtx);
+ pthread_cond_signal(&v->lscnd);
+ pthread_mutex_unlock(&v->lsmtx);
+ assert(usleep(SLEEP_USEC) == 0 || errno == EINTR);
+ }
+ return (NULL);
+}
+
+/**********************************************************************
* Allocate and initialize a varnish
*/
@@ -157,6 +218,19 @@ varnish_new(const char *name)
VSB_printf(v->storage, "-sfile,%s,10M", v->workdir);
AZ(VSB_finish(v->storage));
+ v->vl2 = vtc_logopen(name);
+ AN(v->vl2);
+
+ v->lvd = VSM_New();
+ VSL_Setup(v->lvd);
+
+ (void)VSL_Arg(v->lvd, 'n', v->workdir);
+ (void)VSL_Arg(v->lvd, 'd', NULL);
+ v->lr = 1;
+ AZ(pthread_mutex_init(&v->lsmtx, NULL));
+ AZ(pthread_cond_init(&v->lscnd, NULL));
+ AZ(pthread_create(&v->ltp, NULL, varnishlog_thread, v));
+
v->cli_fd = -1;
VTAILQ_INSERT_TAIL(&varnishes, v, list);
@@ -171,13 +245,23 @@ varnish_new(const char *name)
static void
varnish_delete(struct varnish *v)
{
+ void *p;
CHECK_OBJ_NOTNULL(v, VARNISH_MAGIC);
+ v->lr = 0;
+ AZ(pthread_join(v->ltp, &p));
+ AZ(pthread_mutex_destroy(&v->lsmtx));
+ AZ(pthread_cond_destroy(&v->lscnd));
vtc_logclose(v->vl);
+ vtc_logclose(v->vl1);
+ vtc_logclose(v->vl2);
free(v->name);
free(v->workdir);
+ VSB_delete(v->storage);
if (v->vd != NULL)
VSM_Delete(v->vd);
+ if (v->lvd != NULL)
+ VSM_Delete(v->lvd);
/*
* We do not delete the workdir, it may contain stuff people
@@ -353,7 +437,7 @@ varnish_launch(struct varnish *v)
vtc_log(v->vl, 0, "CLI auth command failed: %u %s", u, r);
free(r);
- (void)VSL_Arg(v->vd, 'n', v->workdir);
+ (void)VSC_Arg(v->vd, 'n', v->workdir);
AZ(VSC_Open(v->vd, 1));
}
@@ -371,6 +455,7 @@ varnish_start(struct varnish *v)
varnish_launch(v);
if (vtc_error)
return;
+ memset(&v->ts, 0, sizeof v->ts);
vtc_log(v->vl, 2, "Start");
u = varnish_ask_cli(v, "start", &resp);
if (vtc_error)
@@ -663,6 +748,103 @@ varnish_expect(const struct varnish *v, char * const *av) {
}
/**********************************************************************
+ * Check varnishlog for VSL tags
+ */
+
+static void
+varnish_logexpecttag(struct varnish *v, const int exp, const char *tag)
+{
+ int i, j, tl, ul;
+ const char *t, *u;
+
+ /* Make sure log is caught up */
+ pthread_mutex_lock(&v->lsmtx);
+ pthread_cond_wait(&v->lscnd, &v->lsmtx);
+ pthread_mutex_unlock(&v->lsmtx);
+
+ tl = strlen(tag);
+ for (i = 0; i < 256; i++) {
+ if (VSL_tags[i] == NULL)
+ continue;
+ u = VSL_tags[i];
+ ul = strlen(u);
+ if (ul != tl)
+ continue;
+ t = tag;
+ for (j = 0; j < tl; j++)
+ if (tolower(*t++) != tolower(*u++))
+ break;
+ if (j == tl) {
+ j = v->ts[i]; /* single read, so no mutex */
+ if (j == exp)
+ vtc_log(v->vl, 2, "as expected, found %s %d "
+ "times", VSL_tags[i], exp);
+ else
+ vtc_log(v->vl, 0, "Found %s %d times, expected"
+ " %d times", VSL_tags[i], j, exp);
+ return;
+ }
+ }
+ vtc_log(v->vl, 0, "Unknown tag: %s", tag);
+}
+
+/**********************************************************************
+ * Check varnishlog for regexes
+ */
+
+static int
+h_count(void *priv, enum VSL_tag_e tag, unsigned fd, unsigned len,
+ unsigned spec, const char *ptr, uint64_t bitmap)
+{
+ struct varnish *v;
+
+ (void) tag;
+ (void) fd;
+ (void) len;
+ (void) spec;
+ (void) ptr;
+ (void) bitmap;
+ CAST_OBJ_NOTNULL(v, priv, VARNISH_MAGIC);
+ v->reh++;
+ return (0);
+}
+
+static void
+varnish_logexpectregex(struct varnish *v, const int exp, const char *pat)
+{
+
+ v->revd = VSM_New();
+ VSL_Setup(v->revd);
+
+ (void)VSL_Arg(v->revd, 'n', v->workdir);
+ (void)VSL_Arg(v->revd, 'd', NULL);
+ if (!VSL_Arg(v->revd, 'I', pat)) {
+ vtc_log(v->vl, 0, "Regex error: %s", pat);
+ return;
+ }
+
+ v->reh = 0;
+
+ if (VSL_Open(v->revd, 0)) {
+ vtc_log(v->vl, 0, "VSL_Open failed");
+ return;
+ }
+ if (VSL_Dispatch(v->revd, h_count, v) != -1) {
+ vtc_log(v->vl, 0, "VSL_Dispatch failed");
+ return;
+ }
+
+ VSM_Delete(v->revd);
+
+ if (exp == v->reh)
+ vtc_log(v->vl, 2, "as expected, matched \"%s\" %d times", pat,
+ exp);
+ else
+ vtc_log(v->vl, 0, "Matched \"%s\" %d times, expected %d", pat,
+ v->reh, exp);
+
+}
+/**********************************************************************
* Varnish server cmd dispatch
*/
@@ -764,11 +946,28 @@ cmd_varnish(CMD_ARGS)
continue;
}
if (!strcmp(*av, "-expect")) {
+ AN(av[1]);
+ AN(av[2]);
+ AN(av[3]);
av++;
varnish_expect(v, av);
av += 2;
continue;
}
+ if (!strcmp(*av, "-logexpecttag")) {
+ AN(av[1]);
+ AN(av[2]);
+ varnish_logexpecttag(v, atoi(av[1]), av[2]);
+ av += 2;
+ continue;
+ }
+ if (!strcmp(*av, "-logexpectregex")) {
+ AN(av[1]);
+ AN(av[2]);
+ varnish_logexpectregex(v, atoi(av[1]), av[2]);
+ av += 2;
+ continue;
+ }
vtc_log(v->vl, 0, "Unknown varnish argument: %s", *av);
}
}

0 comments on commit 8c2c9f2

Please sign in to comment.
Something went wrong with that request. Please try again.