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

[data-plane]: Bump logstash-logback-encoder from 6.6 to 7.0 in /data-plane #1491

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Nov 15, 2021

Bumps logstash-logback-encoder from 6.6 to 7.0.

Release notes

Sourced from logstash-logback-encoder's releases.

logstash-logback-encoder-7.0

Overview

Version 7.0 is a major release with new enhancements, bugfixes, and several backwards incompatibilities mentioned in the sections below.

I'd also like to introduce @​brenuart as a new maintainer! logstash-logback-encoder now has a bus factor of 2. Welcome Bertrand!

We've moved the logstash-logback-encoder repository to the new logfellow GitHub organization. See #700 for more info.

Enhancements

  • #364 #365 #377 Switch FastDateFormat to DateTimeFormatter (@​J-Jimmy, @​philsttr)
  • #461 #472 #630 Reduce memory allocations by writing directly into the output stream and reusing Jackson JsonGenerators between invocations. These improvements should greatly reduce pressure on the garbage collector when logging at high rate. (@​brenuart)
  • #465 Remove support for logback 1.1 (@​philsttr)
  • #470 Remove deprecated functionality
  • #496 Allow customizing pattern of "message" field for AccessEvents (@​philsttr)
  • #506 Documentation does not mention StackHashJsonProvider (@​philsttr)
  • #511 Avoid calling 'toArray' with pre-sized array argument (@​ArthurGazizov)
  • #559 #565 #619 Allow async appenders to block when buffer is full instead of dropping event (@​brenuart)
  • #567 #570 Create Disruptor with a ThreadFactory instead of ExecutorService (@​brenuart )
  • #568 #571 Release/clear LogEvent early before end of batch. Thanks to this change space becomes available in the RingBuffer as soon as an event is successfully async processed. This should reduce the time a logging thread must wait to enqueue its log event when the ring buffer is at or close to maximum capacity. (@​brenuart)
  • #575 Move servlet-api to test scope (@​philsttr)
  • #576 Eager validation of configuration parameters (@​brenuart)
  • #613 #617 Refresh LogstashBasicMarker implementation and remove synchronisation (@​brenuart)
  • #616 LogstashMarker#equals() and #hashCode() are unstable (@​brenuart)
  • #619 #620 Implement exponential backoff for the retry strategy (@​brenuart)
  • #630 #631 #646 #667 #670 Implement a pool of reusable JsonGenerator (@​brenuart)
  • #636 LogstashMarker refactoring (@​brenuart)
  • #649 #329 Drop event when Encoder fails to encode it before it becomes a "poison" event (@​brenuart)
  • logfellow/logstash-logback-encoder@903b77a Be more resilient to exceptions thrown by listeners when firing events (@​brenuart)
  • #663 New JSON providers have been added to the AccessCompositeJsonProvider:
    • #660 threadName: Name of the thread from which the event was logged (@​brenuart).
    • #661 sequence: Output an incrementing sequence number for every log event (@​brenuart).
    • #662 uuid: Outputs random UUID as field value. Handy to provide unique identifier for log lines (@​brenuart).
  • #650 #655 FormattedTimestampJsonProvider#setTimeZone defaults to GMT if the zoneId is not a valid zone id (@​brenuart)
  • #660 Allow ThreadNameJsonProvider to be used with IAccessEvent (@​brenuart)
  • #661 Allow SequenceJsonProvider to be used with IAccessEvent (@​brenuart)
  • #679 #680 #681 jsonproviders no auto start (@​brenuart)
  • #686 #691 PatternJsonProvider now detects errors in pattern at configuration time instead of silently ignoring the field at runtime. The provider emits an ERROR status at startup when it encounters an invalid Logback pattern layout and reverts to producing nothing at runtime.
  • #688 Enhance performance of AbstractPatternJsonProvider (@​brenuart)
  • #690 #696 Allow masking of multiple substrings (@​philsttr)

... (truncated)

Commits
  • 05af6c5 [maven-release-plugin] prepare release logstash-logback-encoder-7.0
  • c936fbe [release]
  • 06ee7bb Remove reference to GITHUB_ACTOR.
  • 38a5e7b Try to fix releases again.
  • d4e62b2 [release]
  • e377fb1 Revert to checkout action 1.1.0 to fix git auth issue when the maven release ...
  • 6fe594c [release]
  • 6f198a0 Fix release java version in workflow
  • 230c87a [release]
  • 4aa0f83 Remove ignore path filters. The are preventing the empty release commit
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [logstash-logback-encoder](https://github.com/logstash/logstash-logback-encoder) from 6.6 to 7.0.
- [Release notes](https://github.com/logstash/logstash-logback-encoder/releases)
- [Commits](logfellow/logstash-logback-encoder@logstash-logback-encoder-6.6...logstash-logback-encoder-7.0)

---
updated-dependencies:
- dependency-name: net.logstash.logback:logstash-logback-encoder
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Nov 15, 2021
@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Nov 15, 2021
@knative-prow-robot knative-prow-robot added area/data-plane size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Nov 15, 2021
@knative-prow-robot
Copy link
Contributor

Hi @dependabot[bot]. Thanks for your PR.

I'm waiting for a knative-sandbox member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Signed-off-by: Knative Automation <automation@knative.team>
@codecov
Copy link

codecov bot commented Nov 15, 2021

Codecov Report

Merging #1491 (3a50aa1) into main (f4a1fb7) will not change coverage.
The diff coverage is n/a.

❗ Current head 3a50aa1 differs from pull request most recent head 10e5015. Consider uploading reports for the commit 10e5015 to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##               main    #1491   +/-   ##
=========================================
  Coverage     73.48%   73.48%           
  Complexity      585      585           
=========================================
  Files           101      101           
  Lines          3975     3975           
  Branches        165      165           
=========================================
  Hits           2921     2921           
  Misses          815      815           
  Partials        239      239           
Flag Coverage Δ
java-unittests 82.26% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f4a1fb7...10e5015. Read the comment docs.

@pierDipi
Copy link
Member

/ok-to-test

@knative-prow-robot knative-prow-robot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Nov 15, 2021
@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 16, 2021
@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dependabot[bot], pierDipi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow-robot knative-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 16, 2021
@knative-prow-robot knative-prow-robot merged commit 09d6a05 into main Nov 16, 2021
@dependabot dependabot bot deleted the dependabot/maven/data-plane/main/net.logstash.logback-logstash-logback-encoder-7.0 branch November 16, 2021 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/data-plane cla: yes Indicates the PR's author has signed the CLA. dependencies Pull requests that update a dependency file lgtm Indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants