Skip to content

Security pipeline update - #1781

Merged
AlexanderBarabanov merged 12 commits into
open-edge-platform:developfrom
AlexanderBarabanov:develop
Jul 10, 2025
Merged

Security pipeline update#1781
AlexanderBarabanov merged 12 commits into
open-edge-platform:developfrom
AlexanderBarabanov:develop

Conversation

@AlexanderBarabanov

@AlexanderBarabanov AlexanderBarabanov commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

Summary

This PR updates security pipeline, in particular:

  • security checks is implemented based on reusable actions from Anomalib repo (later we will move it to more appropriate repo like reusable-workflows) - this will simplify maintenance and support across Geti repos.
  • bandit configuration moved to pyproject.toml
  • .pre-commit-config.yaml - added zizmor and bandit pre-commit checks
  • removed unnecessary files

Tested on local fork.

How to test

Tested in fork https://github.com/AlexanderBarabanov/datumaro/actions

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have added the description of my changes into CHANGELOG.​
  • I have updated the documentation accordingly

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below).
# Copyright (C) 2025 Intel Corporation
#
# SPDX-License-Identifier: MIT

Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
@github-advanced-security

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Signed-off-by: Barabanov <alexander.barabanov@intel.com>
Signed-off-by: Barabanov <alexander.barabanov@intel.com>
@AlexanderBarabanov
AlexanderBarabanov marked this pull request as ready for review July 10, 2025 07:49
@AlexanderBarabanov
AlexanderBarabanov requested a review from gdlg July 10, 2025 07:50
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@gdlg gdlg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks Alexander!

@AlexanderBarabanov
AlexanderBarabanov merged commit 301b93c into open-edge-platform:develop Jul 10, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants