Skip to content

[Infra] Use callback for logger.trace calls#199805

Merged
dgieselaar merged 4 commits intoelastic:mainfrom
dgieselaar:trace-logs-with-callback
Nov 19, 2024
Merged

[Infra] Use callback for logger.trace calls#199805
dgieselaar merged 4 commits intoelastic:mainfrom
dgieselaar:trace-logs-with-callback

Conversation

@dgieselaar
Copy link
Contributor

@dgieselaar dgieselaar commented Nov 12, 2024 •
edited by kibanamachine
Loading

Use a callback for logger.trace() so big objects only get stringified when trace logging is enabled.

@dgieselaar dgieselaar requested a review from a team as a code owner November 12, 2024 14:43
@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Nov 12, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Nov 12, 2024

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

dominiqueclarke
dominiqueclarke approved these changes Nov 14, 2024
View reviewed changes
Copy link
Contributor

@dominiqueclarke dominiqueclarke left a comment

Choose a reason for hiding this comment

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

LGTM

@dgieselaar dgieselaar added the release_note:skip Skip the PR/issue when compiling release notes label Nov 14, 2024
@dgieselaar
Copy link
Contributor Author

dgieselaar commented Nov 14, 2024

@elasticmachine merge upstream

@dgieselaar
Copy link
Contributor Author

dgieselaar commented Nov 19, 2024

@elasticmachine merge upstream

@dgieselaar dgieselaar enabled auto-merge (squash) November 19, 2024 07:02
@elasticmachine
Copy link
Contributor

elasticmachine commented Nov 19, 2024 •
edited
Loading

💚 Build Succeeded

  • Buildkite Build
  • Commit: 4bcf7db
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-199805-4bcf7dbcf3cf

Metrics [docs]

✅ unchanged

History

@dgieselaar dgieselaar merged commit b8e6bbd into elastic:main Nov 19, 2024
@kibanamachine
Copy link
Contributor

kibanamachine commented Nov 19, 2024

Starting backport for target branches: 8.15, 8.16, 8.x

https://github.com/elastic/kibana/actions/runs/11909417661

kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Nov 19, 2024
@dgieselaar @elasticmachine @kibanamachine
[Infra] Use callback for logger.trace calls (elastic#199805)
7fe617f 
Use a callback for logger.trace() so big objects only get stringified
when trace logging is enabled.

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit b8e6bbd)
@kibanamachine kibanamachine mentioned this pull request Nov 19, 2024
Closed
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Nov 19, 2024
@dgieselaar @elasticmachine @kibanamachine
[Infra] Use callback for logger.trace calls (elastic#199805)
cf8f08a 
Use a callback for logger.trace() so big objects only get stringified
when trace logging is enabled.

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit b8e6bbd)
@kibanamachine kibanamachine mentioned this pull request Nov 19, 2024
Merged
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Nov 19, 2024
@dgieselaar @elasticmachine @kibanamachine
[Infra] Use callback for logger.trace calls (elastic#199805)
1b37274 
Use a callback for logger.trace() so big objects only get stringified
when trace logging is enabled.

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit b8e6bbd)
@kibanamachine kibanamachine mentioned this pull request Nov 19, 2024
Merged
@kibanamachine
Copy link
Contributor

kibanamachine commented Nov 19, 2024

💚 All backports created successfully

Status Branch Result
✅ 8.15
✅ 8.16
✅ 8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine mentioned this pull request Nov 19, 2024
Merged
2 tasks
kibanamachine added a commit that referenced this pull request Nov 19, 2024
@kibanamachine @dgieselaar
[8.x] [Infra] Use callback for logger.trace calls (#199805) (#200684)
99e3f67 
# Backport

This will backport the following commits from `main` to `8.x`:
- [[Infra] Use callback for logger.trace calls
(#199805)](#199805)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Dario
Gieselaar","email":"dario.gieselaar@elastic.co"},"sourceCommit":{"committedDate":"2024-11-19T08:51:55Z","message":"[Infra]
Use callback for logger.trace calls (#199805)\n\nUse a callback for
logger.trace() so big objects only get stringified\r\nwhen trace logging
is enabled.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"b8e6bbd4610b50169f1d19a0929d0db68aa82ad7","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-major","ci:project-deploy-observability","v8.17.0"],"title":"[Infra]
Use callback for logger.trace
calls","number":199805,"url":"https://github.com/elastic/kibana/pull/199805","mergeCommit":{"message":"[Infra]
Use callback for logger.trace calls (#199805)\n\nUse a callback for
logger.trace() so big objects only get stringified\r\nwhen trace logging
is enabled.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"b8e6bbd4610b50169f1d19a0929d0db68aa82ad7"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199805","number":199805,"mergeCommit":{"message":"[Infra]
Use callback for logger.trace calls (#199805)\n\nUse a callback for
logger.trace() so big objects only get stringified\r\nwhen trace logging
is enabled.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"b8e6bbd4610b50169f1d19a0929d0db68aa82ad7"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Dario Gieselaar <dario.gieselaar@elastic.co>
kibanamachine added a commit that referenced this pull request Nov 19, 2024
@kibanamachine @dgieselaar
[8.16] [Infra] Use callback for logger.trace calls (#199805) (#200683)
44bfa2d 
# Backport

This will backport the following commits from `main` to `8.16`:
- [[Infra] Use callback for logger.trace calls
(#199805)](#199805)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Dario
Gieselaar","email":"dario.gieselaar@elastic.co"},"sourceCommit":{"committedDate":"2024-11-19T08:51:55Z","message":"[Infra]
Use callback for logger.trace calls (#199805)\n\nUse a callback for
logger.trace() so big objects only get stringified\r\nwhen trace logging
is enabled.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"b8e6bbd4610b50169f1d19a0929d0db68aa82ad7","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-major","ci:project-deploy-observability","v8.17.0"],"title":"[Infra]
Use callback for logger.trace
calls","number":199805,"url":"https://github.com/elastic/kibana/pull/199805","mergeCommit":{"message":"[Infra]
Use callback for logger.trace calls (#199805)\n\nUse a callback for
logger.trace() so big objects only get stringified\r\nwhen trace logging
is enabled.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"b8e6bbd4610b50169f1d19a0929d0db68aa82ad7"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199805","number":199805,"mergeCommit":{"message":"[Infra]
Use callback for logger.trace calls (#199805)\n\nUse a callback for
logger.trace() so big objects only get stringified\r\nwhen trace logging
is enabled.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>\r\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"b8e6bbd4610b50169f1d19a0929d0db68aa82ad7"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Dario Gieselaar <dario.gieselaar@elastic.co>
@kibanamachine kibanamachine added v8.16.1 backport missing Added to PRs automatically when the are determined to be missing a backport. labels Nov 19, 2024
@kibanamachine
Copy link
Contributor

kibanamachine commented Nov 20, 2024

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

1 similar comment
@kibanamachine
Copy link
Contributor

kibanamachine commented Nov 21, 2024

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@dgieselaar dgieselaar removed backport missing Added to PRs automatically when the are determined to be missing a backport. backport:prev-major labels Nov 21, 2024
@dgieselaar dgieselaar deleted the trace-logs-with-callback branch November 21, 2024 11:54
@mistic
Copy link
Contributor

mistic commented Nov 21, 2024

This PR didn't make it into the latest BC of v8.16.1. Updating the labels.

@mistic mistic added v8.16.2 and removed v8.16.1 labels Nov 21, 2024
@maryam-saeidi maryam-saeidi mentioned this pull request Nov 25, 2024
Closed
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
@dgieselaar @elasticmachine
@kibanamachine @CAWilson94
[Infra] Use callback for logger.trace calls (elastic#199805)
7c4e25e 
Use a callback for logger.trace() so big objects only get stringified
when trace logging is enabled.

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dominiqueclarke dominiqueclarke dominiqueclarke approved these changes

Assignees

No one assigned

Labels

ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes v8.16.2 v8.17.0 v9.0.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dgieselaar @elasticmachine @kibanamachine @mistic @dominiqueclarke