Skip to content

feat: add deletion task to job definitions and update related configu…#66

Merged
razbroc merged 2 commits into
masterfrom
feat/config-deletion-task
May 18, 2026
Merged

feat: add deletion task to job definitions and update related configu…#66
razbroc merged 2 commits into
masterfrom
feat/config-deletion-task

Conversation

@razbroc

@razbroc razbroc commented May 17, 2026

Copy link
Copy Markdown
Contributor
Question Answer
Bug fix
New feature
Breaking change
Deprecations
Documentation
Tests added
Chore

This pull request introduces support for a new "deletion" task type throughout the codebase. The main changes involve updating configuration files, interfaces, and handlers to recognize and handle the new task type, ensuring it is integrated into the task flow and appropriately excluded where necessary.

Configuration and Task Flow Updates:

  • Added deletion as a recognized task type in configuration files, including helm/values.yaml, helm/templates/configmap.yaml, and config/custom-environment-variables.json. [1] [2] [3]
  • Included tiles-deletion in the taskFlowManager.ingestionTasksFlow in helm/values.yaml, integrating deletion into the ingestion workflow.

Interface and Handler Adjustments:

  • Updated the IJobDefinitionsConfig interface in src/common/interfaces.ts to include the new deletion task type.
  • Modified the IngestionJobHandler in src/tasks/handlers/ingestion/ingestionHandler.ts to add the deletion task type to the list of excluded types, preventing its duplication in certain scenarios.

Documentation:

  • Updated the README.md to document the new JOB_DEFINITIONS_TASK_DELETION environment variable.

…rations

Co-authored-by: Copilot <copilot@github.com>
@razbroc razbroc requested a review from CL-SHLOMIKONCHA May 17, 2026 13:53
Comment thread helm/values.yaml Outdated
@razbroc razbroc merged commit 3d03126 into master May 18, 2026
5 checks passed
@razbroc razbroc deleted the feat/config-deletion-task branch May 18, 2026 08:12
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