Skip to content

Conversation

@cardil
Copy link
Member

@cardil cardil commented Feb 14, 2025

For vendorless projects we need to collect the test images from deps. This does that by scanning imports (usually the hack/tools.go file).

Part of SRVCLI-405

/kind enhancement

@openshift-ci
Copy link

openshift-ci bot commented Feb 14, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: cardil
Once this PR has been reviewed and has the lgtm label, please assign lberk for approval. For more information see the Code Review Process.

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

@cardil cardil force-pushed the feature/collect-test-images-vendorless branch from 8cbbe55 to 4db5dfd Compare February 14, 2025 15:23
@cardil
Copy link
Member Author

cardil commented Feb 14, 2025

/cc @pierDipi
/cc @creydr

@cardil cardil marked this pull request as ready for review February 14, 2025 15:52
@openshift-ci openshift-ci bot requested review from creydr and pierDipi February 14, 2025 15:52
@openshift-ci openshift-ci bot requested review from matzew and skonto February 14, 2025 15:52
@pierDipi
Copy link
Member

pierDipi commented Feb 18, 2025

As discussed on Slack, for kn-event-plugin we can leverage the reconciler-test file resolver https://github.com/knative-extensions/reconciler-test/tree/main/pkg/images/file with the help of the generate command as used here https://github.com/openshift-knative/eventing-istio/blob/e8680a4b989140e96986488dfbd76d1028f4a557/openshift/generate.sh#L48-L49

The benefits are:

  • faster
  • less resources
  • less builds (big benefit as Konflux scalability is still a big challenge)
  • more environmental friendly
  • Cost savings
  • less code to maintain as we wouldn't need scanning
  • Reuse an existing feature

In the meantime, we decided that:

@cardil
Copy link
Member Author

cardil commented Feb 19, 2025

/close

@openshift-ci
Copy link

openshift-ci bot commented Feb 19, 2025

@cardil: Closed this PR.

Details

In response to this:

/close

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-sigs/prow repository.

@openshift-ci openshift-ci bot closed this Feb 19, 2025
@cardil cardil deleted the feature/collect-test-images-vendorless branch February 19, 2025 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants