Skip to content

[fix][ci] Fix OWASP dep check GH actions workflow - #21831

Merged
lhotari merged 13 commits into
apache:masterfrom
lhotari:lh-fix-ci-owasp-dep-check
Jan 2, 2024
Merged

lhotari merged 13 commits into
apache:masterfrom
lhotari:lh-fix-ci-owasp-dep-check

Conversation

@lhotari

@lhotari lhotari commented Jan 2, 2024

Copy link
Copy Markdown
Member

Motivation

Modifications

  • if: !cancelled() -> if: ${{ !cancelled() }}
  • Modify tiered-storage and pulsar-io scanning to scan each individual project.
  • Add caching for vulnerability database download

Documentation

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

- Fix "Invalid workflow file" error
- follow up for apache#21826
- GHA requires "if: ${{ !cancelled() }}"
@lhotari lhotari added this to the 3.3.0 milestone Jan 2, 2024
@lhotari
lhotari requested a review from tisonkun January 2, 2024 07:47
@lhotari lhotari self-assigned this Jan 2, 2024
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Jan 2, 2024
@lhotari

lhotari commented Jan 2, 2024

Copy link
Copy Markdown
Member Author

@lhotari
lhotari requested a review from Technoboy- January 2, 2024 07:49

@tisonkun tisonkun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@lhotari
lhotari marked this pull request as draft January 2, 2024 08:51
@lhotari

lhotari commented Jan 2, 2024

Copy link
Copy Markdown
Member Author

One more test run here with more useful scanning for tiered-storage/offloaders and pulsar-io connectors: https://github.com/lhotari/pulsar/actions/runs/7383922078/job/20085915493

@lhotari
lhotari marked this pull request as ready for review January 2, 2024 09:03
@lhotari

lhotari commented Jan 2, 2024

Copy link
Copy Markdown
Member Author

/pulsarbot rerun-failure-checks

@lhotari
lhotari merged commit 92e0e47 into apache:master Jan 2, 2024
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.

2 participants