Skip to content

Conversation

dirkkul
Copy link
Collaborator

@dirkkul dirkkul commented Jul 14, 2025

No description provided.

@dirkkul dirkkul requested a review from a team as a code owner July 14, 2025 11:38
Copy link

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 1   low 3   info 3 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca
🛡️ The following IaC misconfigurations have been detected
NAME FILE
medium Controller creating containers without dropped capabilities ...r-compose-backup.yml View in code
low Container Traffic Not Bound To Host Interface ...r-compose-backup.yml View in code
low Cpus Not Limited ...r-compose-backup.yml View in code
low Memory Not Limited ...r-compose-backup.yml View in code
info Healthcheck Not Set ...r-compose-backup.yml View in code
info Restart Policy On Failure Not Set To 5 ...r-compose-backup.yml View in code
info Ensure that, if applicable, SELinux security options are set (Automated) ...r-compose-backup.yml View in code

include_collections: Union[List[str], str, None] = None,
exclude_collections: Union[List[str], str, None] = None,
roles_restore: Optional[Literal["noRestore", "all"]] = None,
user_restore: Optional[Literal["noRestore", "all"]] = None,

Choose a reason for hiding this comment

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

I would suggest users_restore to be consistent with roles_restore

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 97.36842% with 1 line in your changes missing coverage. Please review.

Project coverage is 87.11%. Comparing base (bd57a8e) to head (a689f17).
Report is 66 commits behind head on main.

Files with missing lines Patch % Lines
integration/test_backup_v4.py 96.42% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1737      +/-   ##
==========================================
- Coverage   87.92%   87.11%   -0.81%     
==========================================
  Files         256      269      +13     
  Lines       17812    18701     +889     
==========================================
+ Hits        15661    16292     +631     
- Misses       2151     2409     +258     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dirkkul dirkkul merged commit 85c84e0 into main Jul 15, 2025
57 checks passed
@dirkkul dirkkul deleted the backup_user_roles branch July 15, 2025 11:26
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