02a7875eb8
refactor: remove last dependency of crypto on LocalConnection
Matthias Radestock2015-04-27 07:27:31 +01:00
ea4e84cdd3
refactor: (almost) remove dependency of crypto on LocalConnection
Matthias Radestock2015-04-27 07:14:30 +01:00
29c02d2724
refactor: lift error handling out of Encryptor.Bytes() in order to reduce dependency on LocalConnection
Matthias Radestock2015-04-27 07:12:21 +01:00
fcd44c37e1
refactor: reduce coupling between connections and crypto
Matthias Radestock2015-04-27 06:13:41 +01:00
ce4c79ed93
refactor: extract commonQueryHandler to remove duplication between queryHandler and rdnsHandler
Matthias Radestock2015-04-26 12:40:34 +01:00
cb42ec358e
even more alignment between queryHandler and rdnsHandler in preparation of extracting common helper function
Matthias Radestock2015-04-26 11:40:28 +01:00
1a35b5aa2b
align queryHandler and rdnsHandler increasing their similarity in preparation of refactor
Matthias Radestock2015-04-26 11:20:44 +01:00
58c8945b32
Support "insistent" queries in the mDNS client, used for obtaining all possible responses from mDNS servers. Support for multiple responses from Lookup*() methods Support for multiple records when building packets with make*() methods Use a common "ZoneRecord" interface for structs that contain names and IPs
Alvaro Saurin2015-04-14 15:26:44 +02:00
22c04bfc24
don't use $(exec_on ...) since that will cause the test to exit on failure
Matthias Radestock2015-04-22 20:09:34 +01:00
eebb674232
simplify dns tests - use 'host' to lookup MX record. It's shorter - use getent everywhere else for dns lookups, instead of dig; this tests more of the o/s name resolution machinery and is generally shorter. - introduce assert_dns_record helper that performs forward and reverse - lookup introduce some vars for holding constants, so we don't have to repeat them
Matthias Radestock2015-04-22 19:38:04 +01:00
5749d19fee
Merge pull request #594 from awh/591_smoke_test_robustness
rade2015-04-22 16:52:37 +01:00
6a06d0f95f
Simplify ping tests to avoid set -e precipitated exit in failure case
Adam Harrison2015-04-22 15:37:50 +01:00
d850ea41a2
Report overall status after running all suites
Adam Harrison2015-04-22 15:02:08 +01:00
849bd52096
Merge pull request #590 from rade/simplify_test_docker_exec
Adam Harrison2015-04-22 10:31:37 +01:00
a7c9fd272c
add exec_on helper so we don't repeat the remote docker invocation command (complete with hard-coded port) over and over again.
Matthias Radestock2015-04-22 09:25:42 +01:00
b4c00050e0
simplify dns-args it makes more sense to strip off the --with-dns at the call site. weave dns-args now does something useful, instead of having to be invoked as weave dns-args --with-dns.
Matthias Radestock2015-04-19 13:14:59 +01:00
53e145bea7
simplify local portion of weave run pull all the local weave run execution into one place, thus dispensing with the need to have lots of tests for whether we are executing in the context of "--local". Also, we only remote exec dns-args if "--with-dns" is supplied, which is more efficient in the common case (and also more obvious).
Matthias Radestock2015-04-19 12:52:13 +01:00
0a2f341c6b
cosmetic: move DOCKERHUB_USER setting for more logical grouping
Matthias Radestock2015-04-19 10:42:43 +01:00
fbf7612726
only check for command / print usage at remote This is desirable since we want to do as little as possible locally, and not duplicate any checks.
Matthias Radestock2015-04-19 10:32:13 +01:00
7752d6b102
refactor: move "need to run as root" check to a more obvious place
Matthias Radestock2015-04-19 13:18:22 +01:00
57b7276ceb
create forwarders before AddConnection, if we can This is beneficial for reasons explained in the comments. And we used to do that previously, when initHeartbeats was called before AddConnection.
Matthias Radestock2015-04-15 20:55:42 +01:00
9b25815641
don't send heartbeats on new but ultimately invalid connections It is wasteful and can trip up crypto at the other end. See comment for why.
Matthias Radestock2015-04-15 20:04:40 +01:00
Matthias Radestock