Conditionals: Support chaining conditions

[dibyom]

Expected Behavior

Users should be able to specify multiple conditions to gate a task and chain them using a type: "AND", "OR"
Part of implementation for #27

Additional Info

Strawperson of the potential API: https://gist.github.com/dibyom/557c9abc50f607e3e0cf5a8125adcb62 based on https://docs.google.com/a/google.com/document/d/1iTb0rDDlgQcPbiYIGJ-atkaML8aCyPCyhfucCPJmNLs/edit?disco=AAAADFnE658

ref: #1137

[vincent-pli]

@dibyom
Do we plan to support nested conditions:

conditions:
  - type: AND
    checks:
      - type: OR
        checks: 
          - conditinRef: foo
          - conditinRef: foo_1
      - conditionRef: stringsEqual
        params: 
          - name: expected
            value: master
          - name: actual
            value: ${input.workspace.revision}

      - conditionRef: stringsEqual
        params:
          - name: expected
            value: foo
          - name: actual
value: ${input.workspace.somethingElse}

If yes, I think it's will be a big topic 😄

[dibyom]

I was planning on only supporting basic AND or OR for top level conditions with no nesting.
Mainly because like you said, it will be a complex API and these scenarios can be also be implemented by writing your own Condition check container and referencing those in your pipelines.

[vincent-pli]

Make these logic in container will make them hard to reuse.
need some research to find a solution 🤔

[tekton-robot]

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.

/lifecycle rotten

Send feedback to tektoncd/plumbing.

[bobcatfish]

Conditions are being deprecated in https://github.com/tektoncd/community/blob/master/teps/0007-conditions-beta.md

( + @jerop )