tektoncd / catalog

Catalog of shared Tasks and Pipelines.
Apache License 2.0
666 stars 577 forks source link

task: Add generic RHACS task #1267

Closed rhybrillou closed 5 months ago

rhybrillou commented 5 months ago

Changes

Add a generic task for custom interactions with Red Hat Advanced Cluster Security.

This allows users to perform any action with the roxctl cli.

Submitter Checklist

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

See the contribution guide for more details.

tekton-robot commented 5 months ago

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

rhybrillou commented 5 months ago

/kind feature

vinamra28 commented 5 months ago

/test pull-catalog-catlin-lint

tekton-robot commented 5 months ago

@vinamra28: The specified target(s) for /test were not found. The following commands are available to trigger required jobs:

Use /test all to run all jobs.

In response to [this](https://github.com/tektoncd/catalog/pull/1267#issuecomment-2143291961): >/test pull-catalog-catlin-lint Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
rhybrillou commented 5 months ago

/retest

tekton-robot commented 5 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vdemeester, vinamra28

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: - ~~[OWNERS](https://github.com/tektoncd/catalog/blob/main/OWNERS)~~ [vdemeester,vinamra28] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
rhybrillou commented 5 months ago

/test pull-catalog-catlin-lint

tekton-robot commented 5 months ago

@rhybrillou: The specified target(s) for /test were not found. The following commands are available to trigger required jobs:

Use /test all to run all jobs.

In response to [this](https://github.com/tektoncd/catalog/pull/1267#issuecomment-2146779700): >/test pull-catalog-catlin-lint Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
vdemeester commented 5 months ago

/lgtm

rhybrillou commented 5 months ago

/test all

vdemeester commented 5 months ago

/test pull-catalog-catlin-lint

tekton-robot commented 5 months ago

@vdemeester: The specified target(s) for /test were not found. The following commands are available to trigger required jobs:

Use /test all to run all jobs.

In response to [this](https://github.com/tektoncd/catalog/pull/1267#issuecomment-2147330889): >/test pull-catalog-catlin-lint Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
vdemeester commented 5 months ago

[post-comment : post-comment] Sending this data to GitHub: 
[post-comment : post-comment] {'body': ''}
[post-comment : post-comment] Error: 422
[post-comment : post-comment] b'{"message":"Validation Failed","errors":[{"resource":"IssueComment","code":"unprocessable","field":"data","message":"Body cannot be blank"}],"documentation_url":"https://docs.github.com/rest/issues/comments#create-an-issue-comment"}'

Hum.. 🤔

rhybrillou commented 5 months ago

/test pull-tekton-catalog-integration-tests

rhybrillou commented 5 months ago

/test pull-tekton-catalog-integration-tests

vdemeester commented 5 months ago
 FAILED: redhat-dependency-analytics/0-2 task has failed to comeback properly 

It might be a legit failure isn't it ?

rhybrillou commented 5 months ago
 FAILED: redhat-dependency-analytics/0-2 task has failed to comeback properly 

It might be a legit failure isn't it ?

That is correct, but I fear the dependency analytics task is out of my realm. The redhat-dependency-analytics/0.2 task also failed in #1255 and #1257

vdemeester commented 5 months ago

It might be a legit failure isn't it ?

That is correct, but I fear the dependency analytics task is out of my realm. The redhat-dependency-analytics/0.2 task also failed in #1255 and #1257

But it passed at some point ?

vdemeester commented 5 months ago

ah yeah it did there..

afrittoli commented 5 months ago

/test pull-catalog-catlin-lint

tekton-robot commented 5 months ago

@afrittoli: The specified target(s) for /test were not found. The following commands are available to trigger required jobs:

Use /test all to run all jobs.

In response to [this](https://github.com/tektoncd/catalog/pull/1267#issuecomment-2150063887): >/test pull-catalog-catlin-lint Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
afrittoli commented 5 months ago

/test pull-catalog-catlin-lint

tekton-robot commented 5 months ago

@afrittoli: The specified target(s) for /test were not found. The following commands are available to trigger required jobs:

Use /test all to run all jobs.

In response to [this](https://github.com/tektoncd/catalog/pull/1267#issuecomment-2150175131): >/test pull-catalog-catlin-lint Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
afrittoli commented 5 months ago

/test pull-catalog-catlin-lint

tekton-robot commented 5 months ago

@afrittoli: The specified target(s) for /test were not found. The following commands are available to trigger required jobs:

Use /test all to run all jobs.

In response to [this](https://github.com/tektoncd/catalog/pull/1267#issuecomment-2151751278): >/test pull-catalog-catlin-lint Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.