Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

[BUG] Aggregate EIP is not allowing nested steps #808

Closed
lordrip opened this issue Jul 25, 2023 · 1 comment · Fixed by #825
Closed

[BUG] Aggregate EIP is not allowing nested steps #808

lordrip opened this issue Jul 25, 2023 · 1 comment · Fixed by #825
Assignees
Labels
bug Something isn't working
Milestone

Comments

@lordrip
Copy link
Member

lordrip commented Jul 25, 2023

Describe the bug
The aggregate EIP can have nested steps, but currently, Kaoto doesn't offer a mechanism to allow it.

To Reproduce
Steps to reproduce the behavior:

  1. Starting with a start step (timer)
  2. Add an aggregate EIP
  3. Click on the branches tab from the mini-catalog
  4. See that the branches tab is disabled
    image

Expected behavior
The aggregate EIP should allow to define nested steps

Additional context
Simple YAML structure showcasing aggregate nested steps:

- route:
    id: route-8e8b
    from:
      uri: kamelet:timer-source
      id: from-af58
      parameters:
        message: ${message.body}
      steps:
        - aggregate:
            id: aggregate-4bbc
            correlationExpression: {}
            steps:
              - log:
                  message: ${body}
                  id: log-2fdd
        - log:
            message: ${body}
            id: log-e936

after a sync, this is the result

- route:
    id: route-8e8b
    from:
      id: from-af58
      uri: kamelet:timer-source/
      parameters:
        message: ${message.body}
      steps:
      - aggregate:
          id: aggregate-4bbc
      - log:
          id: log-e936
          message: ${body}

notice that the following part is not supported by Kaoto yet:

            correlationExpression: {}
            steps:
              - log:
                  message: ${body}
                  id: log-2fdd
@lordrip lordrip added the bug Something isn't working label Jul 25, 2023
@igarashitm
Copy link
Contributor

related to #767, we should extract those step metadata from schema and catalog.

@Delawen Delawen self-assigned this Aug 3, 2023
Delawen added a commit to Delawen/zimara-backend that referenced this issue Aug 3, 2023
@Delawen Delawen added this to the 1.2.0 milestone Aug 3, 2023
Delawen added a commit to Delawen/zimara-backend that referenced this issue Aug 3, 2023
Delawen added a commit that referenced this issue Aug 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants