Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge upstream changes #5

Merged
merged 39 commits into from
Jan 15, 2015
Merged

Merge upstream changes #5

merged 39 commits into from
Jan 15, 2015

Conversation

ipolishchuk
Copy link
Owner

No description provided.

PedroMiguelFigueiredo and others added 30 commits November 21, 2014 23:00
Case corrections.
This way it also works with Latin1_General_BIN collation SQL Server.
Various CentOS/Redhat flavours require a tty to be allocated when using sudo. Document how to by-pass this requirement per user.
Also add safeguards in case of crazy amount of queues/topics/subscribers
The builds must not rely on sudo from now on.
That means either fetching pre-compiled binaries from
upstream or fetching the sources and compiling directly
on the build machine.
However, as the released upstream sources do not change
we can benefit from the caching system and cache the
binaries once we have them.
All these software are in $INTEGRATIONS_DIR

Introduction also of running the integration tests against
different versions of the software described in the build
matrix.

Also a bit of documentation on how to use the new test system.

+ Several more fixes of the test suite codebase
Adjust the test to conditions for postgres needed to
be >= 9.1 and >= 9.2
When using an external Elasticsearch cluster, we should tag the
hostname by the node's given hostname in the stats output. Otherwise
we end up using only the last node's values because each subsequent
gauge metric will stomp on the one before it.

The test is updated to make sure the hostname given in the stats
(which should be the FQDN) matches what we output at the end.

Finally, this change also splits the cluster health metrics into a
seperate dict. This has two benefits: (a) it will clean up a lot of
debugging logging and extra work for `_process_health_data` and
(b) makes it easy to check against just the stats metrics for tests.
[elasticsearch] Fix host tagging for external clusters.
[ci/tests] New build. New Infrastructure. New Flavors
subprocess.PIPE is limited in size because the data is buffered in
memory. When running the Agent on a host that has many containers,
the number of mountpoints can be very large. Because of this buffer limit,
the shell call can hang for these huge outputs.

To workaround this, we will write the output to a temporary file and
read it from there.

This is a "quick" fix for a specific case and there might be a better
general approach. But either way, we will still need to handle the
case of large outputs from our subprocess commands otherwise the
whole Agent collector simply hangs.

ref: http://thraxil.org/users/anders/posts/2008/03/13/Subprocess-Hanging-PIPE-is-your-enemy/
Use a temporary file for df output to avoid PIPE limits.
[snmp] configuration option to ignore non-increasing OIDs
[marathon] service checks and improvements
Remi Hakim and others added 9 commits January 14, 2015 19:52
[leo@datadoghq.com: Rebased to pass CI, cosmetic changes]
[netowrk] freebsd: '-W' in netstat for full names
[leo@datadoghq.com] rebased for new CI skeleton and no sudo use
Squashed a bunch of commits too
[postfix] Document requiretty requirement on Redhat/Centos for using sudo.
Merge upstream changes
ipolishchuk pushed a commit that referenced this pull request Jan 15, 2015
@ipolishchuk ipolishchuk merged commit 7c387a7 into londiste Jan 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants