Skip to content

[tests-only][full-ci] Add cli test for deleting stale uploads #11277

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

Merged
merged 1 commit into from
May 8, 2025

Conversation

amrita-shrestha
Copy link
Contributor

@amrita-shrestha amrita-shrestha commented Apr 28, 2025

Description

This PR adds tests for listing and deleting stale uploads.
To create stale upload POSTPROCESSING_DELAY env has been used to start processing after some time so that we can stop ocis and delete *.info file inside .ocis/storage/users/uploads

Scenario covered

  • list and delete all stale uploads (which list and delete every user stale upload)
  • list and delete stale upload of specific space (which list and delete stale upload of specific user specific space)

Related Issue

Motivation and Context

How Has This Been Tested?

  • CI

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@amrita-shrestha amrita-shrestha self-assigned this Apr 28, 2025
@amrita-shrestha amrita-shrestha force-pushed the check-base-path-is-accessible-or-not branch 3 times, most recently from 453d4bb to d242ccd Compare April 28, 2025 11:13
@amrita-shrestha amrita-shrestha changed the title [tests-only][full-ci] Add ci test for deleting stale uploads [tests-only][full-ci] Add cli test for deleting stale uploads Apr 29, 2025
@amrita-shrestha amrita-shrestha force-pushed the check-base-path-is-accessible-or-not branch 14 times, most recently from 8fe4f82 to bf92f4a Compare May 5, 2025 08:53
@amrita-shrestha amrita-shrestha marked this pull request as ready for review May 5, 2025 08:56
@amrita-shrestha amrita-shrestha removed the request for review from phil-davis May 5, 2025 08:57
@amrita-shrestha amrita-shrestha force-pushed the check-base-path-is-accessible-or-not branch from 9986f32 to 68faf09 Compare May 6, 2025 04:21
@amrita-shrestha amrita-shrestha force-pushed the check-base-path-is-accessible-or-not branch 2 times, most recently from 0c5327f to a1da767 Compare May 8, 2025 08:37
Copy link
Member

@saw-jan saw-jan left a comment

Choose a reason for hiding this comment

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

others look good 👍

@amrita-shrestha amrita-shrestha force-pushed the check-base-path-is-accessible-or-not branch from a1da767 to 6c9a98a Compare May 8, 2025 10:22
@amrita-shrestha amrita-shrestha requested a review from saw-jan May 8, 2025 10:22
@amrita-shrestha amrita-shrestha enabled auto-merge May 8, 2025 10:27
Copy link

sonarqubecloud bot commented May 8, 2025

@amrita-shrestha amrita-shrestha merged commit ed6c0e5 into master May 8, 2025
4 checks passed
ownclouders pushed a commit that referenced this pull request May 8, 2025
…-or-not

[tests-only][full-ci] Add cli test for deleting stale uploads
@nirajacharya2 nirajacharya2 deleted the check-base-path-is-accessible-or-not branch May 9, 2025 03:29
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.

5 participants