fix: ensure wftmplLifecycleHook wait for each dag task

[toyamagu-2021]

Signed-off-by: toyamagu-2021 toyamagu2021@gmail.com

Fixes #12120

Motivation

• Ensure WorkflowTemplateLevelLifeCycleHook wait for each dag task
• Almost same as onExit handler fix: Ensure task dependencies run after onExit handler is fulfilled #3435

Modifications

• Add CheckAllHooksFullfilled to check child hooked nodes.

Verification

This PR resolve #12120

yaml

spec:
  templates:
    - name: main
      inputs: {}
      outputs: {}
      metadata: {}
      dag:
        tasks:
          - name: some-task
            template: some-task
            arguments: {}
            hooks:
              failure:
                template: exit-handler
                arguments: {}
                expression: tasks["some-task"].status == "Failed"
          - name: finish
            template: finish
            arguments: {}
            dependencies:
              - some-task
    - name: some-task
      inputs: {}
      outputs: {}
      metadata: {}
      container:
        name: ''
        image: alpine:latest
        command:
          - sh
          - '-c'
        args:
          - |
            echo "Doing great things";
            echo "Failing...";
            exit 1;
        resources: {}
    - name: exit-handler
      inputs: {}
      outputs: {}
      metadata: {}
      container:
        name: ''
        image: alpine:latest
        command:
          - sh
          - '-c'
        args:
          - |
            echo "Sending updates to github";
            sleep 5;
            echo "Done!";
        resources: {}
      retryStrategy:
        limit: 1
        retryPolicy: Always
    - name: finish
      inputs: {}
      outputs: {}
      metadata: {}
      container:
        name: ''
        image: alpine:latest
        command:
          - sh
          - '-c'
        args:
          - |
            echo "Finished!";
        resources: {}
  entrypoint: main

• Success hook case

• Failed hook case

[terrytangyuan]

LGTM. Thanks!