Skip to content

feat: add Index field to Task parameters for merging items - #97

Merged
IvanJosipovic merged 1 commit into
alphafrom
task-key-required
Jun 23, 2026
Merged

feat: add Index field to Task parameters for merging items#97
IvanJosipovic merged 1 commit into
alphafrom
task-key-required

Conversation

@IvanJosipovic

Copy link
Copy Markdown
Collaborator
  • Introduced an Index field in TaskInitParameters, TaskObservation, and TaskParameters across v1alpha1 and v1beta1 APIs to facilitate merging of items in parent object lists.
  • Updated DeepCopy methods to handle the new Index field.
  • Adjusted job types to replace deprecated list map keys with the new Index key for tasks, environments, and job clusters.
  • Modified server-side apply merge strategies to utilize the new Index field for better item merging.
  • Updated CRDs to reflect the changes in the structure and added descriptions for the new Index field.

Description of your changes

Fixes #

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

- Introduced an `Index` field in TaskInitParameters, TaskObservation, and TaskParameters across v1alpha1 and v1beta1 APIs to facilitate merging of items in parent object lists.
- Updated DeepCopy methods to handle the new `Index` field.
- Adjusted job types to replace deprecated list map keys with the new `Index` key for tasks, environments, and job clusters.
- Modified server-side apply merge strategies to utilize the new `Index` field for better item merging.
- Updated CRDs to reflect the changes in the structure and added descriptions for the new `Index` field.
@IvanJosipovic
IvanJosipovic requested a review from glalanne as a code owner June 23, 2026 00:14
@IvanJosipovic
IvanJosipovic merged commit 785b759 into alpha Jun 23, 2026
7 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.5.0-alpha.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

1 participant