Skip to content

Conversation

@prashant-gurung899
Copy link
Contributor

@prashant-gurung899 prashant-gurung899 commented Jun 19, 2025

Description

This PR adds the following pipelines to purge the cache of browsers, playwright-tracing-results and opencloud-build:

  • purge-browser-build-cache
  • purge-playwright-tracing-cache
  • purge-opencloud-build-cache

Check these previous CIs to see the cache being deleted:

Related Issue

How Has This Been Tested?

  • 🤖

Types of changes

  • Bugfix
  • Enhancement (a change that doesn't break existing code or deployments)
  • Breaking change (a modification that affects current functionality)
  • Technical debt (addressing code that needs refactoring or improvements)
  • Tests (adding or improving tests)
  • Documentation (updates or additions to documentation)
  • Maintenance (like dependency updates or tooling adjustments)

@prashant-gurung899 prashant-gurung899 self-assigned this Jun 19, 2025
@prashant-gurung899 prashant-gurung899 force-pushed the flush-all-cache branch 28 times, most recently from 7911b2b to 7f8eb4e Compare June 20, 2025 09:05
@prashant-gurung899 prashant-gurung899 force-pushed the flush-all-cache branch 12 times, most recently from 1815979 to c210276 Compare June 25, 2025 09:42
@prashant-gurung899 prashant-gurung899 marked this pull request as ready for review June 25, 2025 09:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds new pipelines to purge various caches—specifically for Opencloud build, browsers, and playwright tracing—to improve the cleanup process after builds.

  • Updated the afterPipelines function to include purge functions for Opencloud build, browser, and tracing caches.
  • Refactored cache purging by replacing genericCachePurge with a more generic purgeCache function that accepts an additional flush_age parameter.
  • Introduced dedicated purge functions for Opencloud build, browser, and tracing caches.
Comments suppressed due to low confidence (3)

.woodpecker.star:247

  • Consider adding inline comments that explain the order and purpose of each cache purge function added here to help future maintainers understand the cache clearing sequence.
    return publishRelease(ctx) + [purgeBuildArtifactCache(ctx), purgeOpencloudBuildCache(ctx), purgeBrowserCache(ctx), purgeTracingCache(ctx)]

.woodpecker.star:1187

  • Verify that the pipeline executor accepts a dictionary for the 'steps' field instead of a list, as used in previous implementations. If this change is intentional, consider adding a brief comment outlining the rationale for future reference.
        "steps": {

.woodpecker.star:1209

  • Double-check that the hardcoded flush path 'cache/opencloud-eu/web' is intentional and aligns with the desired cache location for build artifacts, especially in comparison to other purge functions.
        return purgeCache("purge_build_artifact_cache", "cache/opencloud-eu/web", 1)

Signed-off-by: prashant-gurung899 <prasantgrg777@gmail.com>
@prashant-gurung899 prashant-gurung899 merged commit 29ece9a into main Jul 2, 2025
24 checks passed
@prashant-gurung899 prashant-gurung899 deleted the flush-all-cache branch July 2, 2025 10:47
@openclouders openclouders mentioned this pull request Jul 2, 2025
1 task
@openclouders openclouders mentioned this pull request Jul 21, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants