Skip to content

[fix][ci] Fix running OWASP for PRs that change dependencies - #18892

Merged
lhotari merged 1 commit into
apache:masterfrom
lhotari:lh-fix-running-owasp-for-prs-that-change-deps
Dec 13, 2022
Merged

[fix][ci] Fix running OWASP for PRs that change dependencies#18892
lhotari merged 1 commit into
apache:masterfrom
lhotari:lh-fix-running-owasp-for-prs-that-change-deps

Conversation

@lhotari

@lhotari lhotari commented Dec 12, 2022

Copy link
Copy Markdown
Member

Motivation

Modifications

  • Fix the problem that causes "OWASP dependency check" job to always get skipped.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository: lhotari#111 (added some additional changes that contain pom.xml changes)

@lhotari lhotari added this to the 2.12.0 milestone Dec 12, 2022
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Dec 12, 2022
@lhotari

lhotari commented Dec 12, 2022

Copy link
Copy Markdown
Member Author

/pulsarbot rerun-failure-checks

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #18892 (5d12642) into master (3180a4a) will decrease coverage by 9.11%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #18892      +/-   ##
============================================
- Coverage     46.17%   37.05%   -9.12%     
+ Complexity    10359     1968    -8391     
============================================
  Files           703      209     -494     
  Lines         68845    14431   -54414     
  Branches       7382     1574    -5808     
============================================
- Hits          31788     5348   -26440     
+ Misses        33448     8497   -24951     
+ Partials       3609      586    -3023     
Flag Coverage Δ
unittests 37.05% <0.00%> (-9.12%) ⬇️

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

Impacted Files Coverage Δ
...va/org/apache/pulsar/client/impl/ConsumerImpl.java 15.09% <0.00%> (-0.04%) ⬇️
...he/pulsar/client/impl/MultiTopicsConsumerImpl.java 22.78% <0.00%> (-0.09%) ⬇️
.../org/apache/pulsar/client/impl/ConnectionPool.java 37.43% <0.00%> (-1.03%) ⬇️
.../main/java/org/apache/pulsar/PulsarStandalone.java
...broker/intercept/ManagedLedgerInterceptorImpl.java
.../pulsar/broker/service/AbstractBaseDispatcher.java
...sistent/PersistentDispatcherMultipleConsumers.java
...nsaction/pendingack/impl/PendingAckHandleImpl.java
...er/systopic/NamespaceEventsSystemTopicFactory.java
...apache/bookkeeper/mledger/ManagedLedgerConfig.java
... and 487 more

@lhotari
lhotari merged commit eb3a4bf into apache:master Dec 13, 2022
Demogorgon314 pushed a commit to Demogorgon314/pulsar that referenced this pull request Dec 26, 2022
Demogorgon314 pushed a commit to Demogorgon314/pulsar that referenced this pull request Dec 29, 2022
lifepuzzlefun pushed a commit to lifepuzzlefun/pulsar that referenced this pull request Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci doc-not-needed Your PR changes do not impact docs ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants