Skip to content

fix: add index cleanup cron#396

Merged
eric-zaharia merged 3 commits intomasterfrom
fix/cleanup-test-indexes
Apr 21, 2026
Merged

fix: add index cleanup cron#396
eric-zaharia merged 3 commits intomasterfrom
fix/cleanup-test-indexes

Conversation

@eric-zaharia
Copy link
Copy Markdown
Contributor

@eric-zaharia eric-zaharia commented Apr 21, 2026

Q A
Bug fix? no
New feature? no
BC breaks? no
Need Doc update no

Describe your change

Adds .github/workflows/cleanup-test-indexes.yml: a weekly cron (Sundays 03:00 UTC, also workflow_dispatch) that deletes every Algolia index prefixed with sf_phpunit_ using the existing CI secrets.

What problem is this fixing?

Test indexes leak forever. The prefix bakes in GITHUB_RUN_NUMBER, so every CI run creates a fresh set of ~30 unique indexes and nothing ever deletes them.

@eric-zaharia eric-zaharia self-assigned this Apr 21, 2026
@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Apr 21, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 7 complexity · 0 duplication

Metric Results
Complexity 7
Duplication 0

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@eric-zaharia eric-zaharia marked this pull request as ready for review April 21, 2026 09:21
@eric-zaharia eric-zaharia marked this pull request as draft April 21, 2026 09:25
@eric-zaharia eric-zaharia marked this pull request as ready for review April 21, 2026 10:05
Copy link
Copy Markdown
Contributor

@Fluf22 Fluf22 left a comment

Choose a reason for hiding this comment

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

Great

@eric-zaharia eric-zaharia merged commit a500c04 into master Apr 21, 2026
7 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.

2 participants