Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): update rhtap references #95

Merged
merged 2 commits into from
Mar 28, 2024
Merged

Conversation

red-hat-konflux[bot]
Copy link
Contributor

@red-hat-konflux red-hat-konflux bot commented Feb 22, 2024

This PR contains the following updates:

Package Change Notes
quay.io/redhat-appstudio-tekton-catalog/task-buildah 00402c6 -> e1e6c63
quay.io/redhat-appstudio-tekton-catalog/task-clair-scan 9d0f4fa -> 5097b69
quay.io/redhat-appstudio-tekton-catalog/task-clamav-scan 144c1ab -> 628847d
quay.io/redhat-appstudio-tekton-catalog/task-deprecated-image-check 724c2c0 -> ba55ff5
quay.io/redhat-appstudio-tekton-catalog/task-git-clone e13f6e9 -> fffe623
quay.io/redhat-appstudio-tekton-catalog/task-inspect-image 9aa6f81 -> c9e991c
quay.io/redhat-appstudio-tekton-catalog/task-prefetch-dependencies a0d054a -> e29adab
quay.io/redhat-appstudio-tekton-catalog/task-sast-snyk-check e06d355 -> 422177f
quay.io/redhat-appstudio-tekton-catalog/task-sbom-json-check be8d2e7 -> e5202b2
quay.io/redhat-appstudio-tekton-catalog/task-source-build 3ad20ad -> dce619d
quay.io/redhat-appstudio-tekton-catalog/task-summary 0.1 -> 0.2 ⚠️migration⚠️

To execute skipped test pipelines write comment /ok-to-test

Copy link

openshift-ci bot commented Feb 22, 2024

Hi @red-hat-konflux[bot]. Thanks for your PR.

I'm waiting for a redhat-appstudio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch 3 times, most recently from 7c77a03 to a1646ea Compare February 28, 2024 04:29
@tkdchen
Copy link
Contributor

tkdchen commented Feb 28, 2024

/test image-controller-on-pull-request

Copy link

openshift-ci bot commented Feb 28, 2024

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

  • /test image-controller-e2e
  • /test images

Use /test all to run the following jobs that were automatically triggered:

  • pull-ci-redhat-appstudio-image-controller-main-images

In response to this:

/test image-controller-on-pull-request

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.

@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch 5 times, most recently from 8aa1b62 to d525a45 Compare March 5, 2024 12:30
@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch 3 times, most recently from d9612d7 to 3e9767d Compare March 14, 2024 08:40
@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch 4 times, most recently from 29862f5 to a7e0a83 Compare March 21, 2024 14:23
@red-hat-konflux red-hat-konflux bot force-pushed the konflux/references/main branch from a7e0a83 to f69d598 Compare March 21, 2024 15:26
@chmeliik
Copy link
Contributor

The original EC check failed because Tekton Chains took too long to generate the signature and attestation, and the integration service has a known bug where it doesn't always wait for them properly

After re-running the EC tests according to https://redhat-appstudio.github.io/docs.appstudio.io/Documentation/main/how-to-guides/testing_applications/proc_retriggering_integration_tests/, there are new violations:

  violations:
  - metadata:
      code: cve.cve_results_found
      collections:
      - minimal
      - redhat
      depends_on:
      - attestation_type.known_attestation_type
      description: Confirm that clair-scan task results are present in the SLSA Provenance
        attestation of the build pipeline. To exclude this rule add "cve.cve_results_found"
        to the `exclude` section of the policy configuration.
      solution: Make sure there is a successful task in the build pipeline that runs
        a Clair scan and creates a task result called `CLAIR_SCAN_RESULT`.
      title: CVE scan results found
    msg: Clair CVE scan results were not found

The clair scan task did indeed fail in the build pipeline. Retrying

@chmeliik
Copy link
Contributor

/retest

Copy link
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

kubernetes-sigs/controller-runtime#2720

The minimum required Go version for setup-envtes was bumped to 1.22,
breaking us. Pin to release-0.16, which doesn't use 1.22 yet. It uses
1.21, which is still higher than what we use, but back then Go didn't
enforce the version requirement yet.

Signed-off-by: Adam Cmiel <acmiel@redhat.com>
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link

openshift-ci bot commented Mar 28, 2024

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

  • /test image-controller-e2e
  • /test images

Use /test all to run all jobs.

In response to this:

/test ci/prow/images

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.

@mmorhun
Copy link
Collaborator

mmorhun commented Mar 28, 2024

/test images

@openshift-ci openshift-ci bot added the lgtm label Mar 28, 2024
@mmorhun mmorhun merged commit 5a05b87 into main Mar 28, 2024
16 checks passed
@mmorhun mmorhun deleted the konflux/references/main branch March 28, 2024 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants