Skip to content

Conversation

@rylnd
Copy link
Contributor

@rylnd rylnd commented Dec 4, 2020

Summary

This addresses two minor bugs that are logically independent, but both fall generally under "configuration issues," and are tiny, so I've put them together here:

  1. Fixes an error raised when running the FTR server with our API integration config
  2. Removes duplicated context from our logger instance
    • This context used to be required, but in new platform it's provided automatically, which lead to duplicated log tags:
      server    log   [12:36:07.138] [debug][plugins][plugins][securitySolution][securitySolution] ...
      
    • They now appear without the duplication:
      server    log   [12:41:52.112] [debug][plugins][securitySolution] ...
      

For maintainers

rylnd added 2 commits December 4, 2020 12:35
These paths (or rather their absence) breaks the functional_tests_server script.

An analogous problem for cases was fixed in elastic#79127.
This context is already provided to the logger by kibana; adding it a
second time leads to duplicated log tags, e.g.:

```
server    log   [12:36:07.138] [debug][plugins][plugins][securitySolution][securitySolution] ...
```

This is now fixed:

```
server    log   [12:41:52.112] [debug][plugins][securitySolution] ...
```
@rylnd rylnd added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.11.0 Team:Detections and Resp Security Detection Response Team labels Dec 4, 2020
@rylnd rylnd requested a review from FrankHassanabad December 4, 2020 18:56
@rylnd rylnd requested a review from a team as a code owner December 4, 2020 18:56
@rylnd rylnd self-assigned this Dec 4, 2020
@rylnd rylnd requested a review from a team as a code owner December 4, 2020 18:56
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Copy link
Contributor

@FrankHassanabad FrankHassanabad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rylnd rylnd merged commit 918dbb1 into elastic:master Dec 4, 2020
@rylnd rylnd deleted the quick_bugfixes branch December 4, 2020 21:57
rylnd added a commit that referenced this pull request Dec 5, 2020
* Remove nonexistent paths from our API tests config

These paths (or rather their absence) breaks the functional_tests_server script.

An analogous problem for cases was fixed in #79127.

* Remove duplicated context from logger

This context is already provided to the logger by kibana; adding it a
second time leads to duplicated log tags, e.g.:

```
server    log   [12:36:07.138] [debug][plugins][plugins][securitySolution][securitySolution] ...
```

This is now fixed:

```
server    log   [12:41:52.112] [debug][plugins][securitySolution] ...
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Detections and Resp Security Detection Response Team v7.11.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants