Skip to content

feature: new task to run camel routes via camel jbang#1364

Closed
mcarlett wants to merge 1 commit intotektoncd:mainfrom
mcarlett:camel-run
Closed

feature: new task to run camel routes via camel jbang#1364
mcarlett wants to merge 1 commit intotektoncd:mainfrom
mcarlett:camel-run

Conversation

@mcarlett
Copy link

@mcarlett mcarlett commented Feb 6, 2026

Changes

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Follows the authoring recommendations
  • Includes docs (if user facing)
  • Includes tests (for new tasks or changed functionality)
  • Meets the Tekton contributor standards (including functionality, content, code)
  • Commit messages follow commit message best practices
  • Has a kind label. You can add one by adding a comment on this PR that
    contains /kind <type>. Valid types are bug, cleanup, design, documentation,
    feature, flake, misc, question, tep
  • Complies with Catalog Organization TEP, see example. Note An issue has been filed to automate this validation
    • File path follows <kind>/<name>/<version>/name.yaml

    • Has README.md at <kind>/<name>/<version>/README.md

    • Has mandatory metadata.labels - app.kubernetes.io/version the same as the <version> of the resource

    • Has mandatory metadata.annotations tekton.dev/pipelines.minVersion

    • mandatory spec.description follows the convention

        ```
      
        spec:
          description: >-
            one line summary of the resource
      
            Paragraph(s) to describe the resource.
        ```
      

See the contribution guide for more details.

@tekton-robot tekton-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 6, 2026
@tekton-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign vinamra28 after the PR has been reviewed.
You can assign the PR to them by writing /assign @vinamra28 in a comment when ready.

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

Details Needs approval from an approver in each of these files:

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

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 6, 2026

CLA Not Signed

@tekton-robot tekton-robot added do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Feb 6, 2026
@mcarlett
Copy link
Author

mcarlett commented Feb 6, 2026

/kind feature

@tekton-robot tekton-robot added the kind/feature Categorizes issue or PR as related to a new feature. label Feb 6, 2026
@tekton-robot
Copy link

Catlin Output
FILE: task/camel-run/0.1/camel-run.yaml
WARN : Step "execute" uses image "$(params.camel-jbang-image)" that contains variables; skipping validation
Catlin script lint Output
ERROR: /usr/bin/shellcheck, [-s bash] failed:

In camel-run-execute line 7:
[[ ! "$PARAM_FILENAME" == "" ]] && CAMEL_RUN_ARGS+=($PARAM_FILENAME)
                                                    ^-------------^ SC2206 (warning): Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.


In camel-run-execute line 44:
[[ ! "$PARAM_EXTRA_ARGS" == "" ]] && CAMEL_RUN_ARGS+=($PARAM_EXTRA_ARGS)
                                                      ^---------------^ SC2206 (warning): Quote to prevent word splitting/globbing, or split robustly with mapfile or read -a.


In camel-run-execute line 46:
echo "Running: camel run ${CAMEL_RUN_ARGS[@]}"
                         ^------------------^ SC2145 (error): Argument mixes string and array. Use * or separate argument.

For more information:
  https://www.shellcheck.net/wiki/SC2145 -- Argument mixes string and array. ...
  https://www.shellcheck.net/wiki/SC2206 -- Quote to prevent word splitting/g...

@tekton-robot
Copy link

Catlin Output
FILE: task/camel-run/0.1/camel-run.yaml
WARN : Step "execute" uses image "$(params.camel-jbang-image)" that contains variables; skipping validation

@tekton-robot
Copy link

catlin.txt

@tekton-robot tekton-robot removed the do-not-merge/invalid-owners-file Indicates that a PR should not merge because it has an invalid OWNERS file in it. label Feb 12, 2026
@tekton-robot
Copy link

catlin.txt

@tekton-robot
Copy link

Catlin Output
FILE: task/camel-run/0.1/camel-run.yaml
WARN : Step "execute" uses image "$(params.camel-jbang-image)" that contains variables; skipping validation

@tekton-robot
Copy link

Catlin Output
FILE: task/camel-run/0.1/camel-run.yaml
WARN : Step "execute" uses image "$(params.camel-jbang-image)" that contains variables; skipping validation

@mcarlett
Copy link
Author

mcarlett commented Mar 4, 2026

superseded by #1366

@mcarlett mcarlett closed this Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. kind/feature Categorizes issue or PR as related to a new feature. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants