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

prepare 5.6.1 release #72

Merged
merged 183 commits into from
Aug 6, 2019
Merged

prepare 5.6.1 release #72

merged 183 commits into from
Aug 6, 2019

Conversation

eli-darkly
Copy link
Contributor

[5.6.1] - 2019-08-05

Fixed:

  • Enabling TLS for Redis as a separate option was not working; it would only work if you specified a rediss:// secure URL. Both methods now work. (#71)

atrakh and others added 30 commits July 20, 2018 11:25
The results are unpredictable when we do.
Make packages we don't intend for public use internal
Also start using testify/require and wait an extra moment to flush exporters that have startedin a flaky test.
…igurable-global-datadog

Add global tag configuration for datadog exporter
Add smoke tests for debian, centos and docker builds.
Git tag may not be up-to-date until we tag the new release.
…s-nesting-in-metrics-tests

[ch20947] Flatten metrics tests
…and-public-metric-views

[ch21033] Separate private and public metrics
Fix event exporter views and rename private metric view names
logging.InitLogging needs to be callable externally
eli-darkly and others added 26 commits July 23, 2019 16:08
Add proxy options and NTLM proxy auth support
Fixed docker-entrypoint
Moved config value check to relay package
Removed unneeded default value
Proper error handling on HTTP server
Do a status check on HTTP server startup
Add nolint for TLS startup check
Error check response body and server shutdown
Fix logging
Escape shell quotes
Add in escaped docker-entrypoint, missed in previous commit
Move err channel creation to main
Defer channel close
Explicit channel in function
Cleaner goroutine error handling
Remove server shutdown logic
break out config structs into their own file and simplify how defaults work
* allow relay to be configured with environment vars and no file

* remove Docker script, use vars directly

* update goreleaser Dockerfile to match main Dockerfile

* rm another obsolete file reference

* allow both file and vars

* describe new parameter in readme

* rm obsolete test of Docker conf file

* typo

* try making output from test job visible

* DataDog address parameters shouldn't be nullable

* comments

* don't background the job, do display logs

* better config logic that's more backward-compatible

* update readme for new config behavior

* misc fixes + tests

* optional file syntax for full backward-compatibility in Docker

* add some variables we forgot to support

* add another variable we forgot to support

* fix command line

* misc cleanup

* major rewrite of readme: config options, examples, copyedit (#92)

* change syntax for optional config file
change Redis config logic so TLS option is always respected
@eli-darkly eli-darkly requested a review from bwoskow-ld August 6, 2019 00:49
@eli-darkly eli-darkly merged commit 08d2523 into v5 Aug 6, 2019
@eli-darkly eli-darkly deleted the 5.6.1 branch August 6, 2019 01:03
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.

7 participants