Bug 1855811: Allow Tekton Event Listeners to render without a full route

[andrewballantyne]

Fixes:
https://issues.redhat.com/browse/ODC-4202

Analysis / Root cause:
Tried to use the partial Route object to craft a route and it failed to do so.

Solution Description:
Make sure we have a valid structured route before trying to get a URL out of it.

Test setup:

• OpenShift Pipelines Operator
• Go to Event Listener Create YAML and add this:

apiVersion: triggers.tekton.dev/v1alpha1
kind: EventListener
metadata:
  name: event-listener
spec:
  serviceAccountName: pipeline
  triggers:
    - bindings:
        - name: generic-git
      template:
        name: pipeline-template

• Before this fix, on completion of the YAML it would throw the error as it had a non-exposed route object

Browser conformance:

• Chrome
• Firefox
• Safari
• Edge

[rottencandy]

Tested locally.
/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andrewballantyne, rottencandy

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• frontend/packages/dev-console/OWNERS [andrewballantyne]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[openshift-cherrypick-robot]

@andrewballantyne: #5802 failed to apply on top of branch "release-4.5":

error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	frontend/packages/dev-console/src/components/pipelines/utils/triggers.ts
Falling back to patching base and 3-way merge...
Auto-merging frontend/packages/dev-console/src/components/pipelines/utils/triggers.ts
CONFLICT (content): Merge conflict in frontend/packages/dev-console/src/components/pipelines/utils/triggers.ts
Patch failed at 0001 Allow Tekton Event Listeners to render without a full route

In response to this:

/cherry-pick release-4.5

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[openshift-ci-robot]

@andrewballantyne: All pull requests linked via external trackers have merged: . Bugzilla bug 1855811 has been moved to the MODIFIED state.

In response to this:

Bug 1855811: Allow Tekton Event Listeners to render without a full route

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.