Skip to content

Conversation

@matejvasek
Copy link
Collaborator

@matejvasek matejvasek commented Apr 4, 2025

Highlights

  • Feature: build host mounts -- users now can bind local file/directory into the build container. This can be for instance used for inserting git credentials for private Go repositories.

  • Bugfix: fixed a bug where it was not possible to use external dependencies (non-Go stdlib) in Go function.

  • Bugfix: fixed scaffolding to use appropriate module name, this was causing troubles when using sub-modules in the Go function.

PRs picked

@openshift-ci openshift-ci bot added the approved label Apr 4, 2025
@matejvasek matejvasek changed the title Cherry picks for v1.16 [release-v.1.16] Cherry picks for v1.16 Apr 4, 2025
@matejvasek matejvasek requested review from dsimansk and lkingland April 4, 2025 10:35
@lkingland
Copy link

/approve
/lgtm
Hoping the tests pass 🤞🏻

@openshift-ci
Copy link

openshift-ci bot commented Apr 4, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lkingland, matejvasek

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

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [lkingland,matejvasek]

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

matejvasek and others added 13 commits April 4, 2025 14:43
Signed-off-by: Matej Vašek <mvasek@redhat.com>
Signed-off-by: Matej Vašek <mvasek@redhat.com>
* Use tiny instead of base BP builder.
* Decrease minimal required Go version.
* Make the self-signed certificate CA==true so it works with update-ca-trust utility.

Signed-off-by: Matej Vašek <mvasek@redhat.com>
* Allow host mounts for build pahse.

This is in particular useful for using paket bindings,
e.g. to inject git credentials into build process.

Signed-off-by: Matej Vašek <mvasek@redhat.com>

* Mark test as a integration test

Signed-off-by: Matej Vašek <mvasek@redhat.com>

* fixup: style

Signed-off-by: Matej Vašek <mvasek@redhat.com>

---------

Signed-off-by: Matej Vašek <mvasek@redhat.com>
Signed-off-by: Matej Vašek <matejvasek@gmail.com>
Signed-off-by: Matej Vašek <mvasek@redhat.com>
Signed-off-by: Matej Vašek <matejvasek@gmail.com>
The release-v1.16 branch uses older docker library version some minor
adjusment was needed.

Signed-off-by: Matej Vašek <matejvasek@gmail.com>
* Return error code when compilation was not done
* Run `go mod tidy` on scaffoded code

Signed-off-by: Matej Vašek <mvasek@redhat.com>
* Added tests for s2i build with private Go module

The test is for now skipped since host binds are not yet implemented for s2i.

Signed-off-by: Matej Vašek <mvasek@redhat.com>

* User "host" network mode for s2i build on Linux

Signed-off-by: Matej Vašek <mvasek@redhat.com>

* Implement host binds for local s2i build

Signed-off-by: Matej Vašek <mvasek@redhat.com>

---------

Signed-off-by: Matej Vašek <mvasek@redhat.com>
* Make build binds more similar to runtime binds

Signed-off-by: Matej Vašek <mvasek@redhat.com>

* Added comments on struct

Signed-off-by: Matej Vašek <mvasek@redhat.com>

---------

Signed-off-by: Matej Vašek <mvasek@redhat.com>
Signed-off-by: Matej Vašek <matejvasek@gmail.com>
Signed-off-by: Matej Vašek <mvasek@redhat.com>
Signed-off-by: Matej Vašek <matejvasek@gmail.com>
* Fix: Go scaffolding uses correct module name

Using incorrect name works for a functions with flat structure -- no
sub-packages. When sub-packages are used we need to refer the user
module by its true name.

Signed-off-by: Matej Vašek <mvasek@redhat.com>

* fixup tests

Signed-off-by: Matej Vašek <mvasek@redhat.com>

---------

Signed-off-by: Matej Vašek <mvasek@redhat.com>
Signed-off-by: Matej Vašek <matejvasek@gmail.com>
@matejvasek matejvasek force-pushed the cherry-picks-for-v1.16 branch from 155baf3 to d5c090d Compare April 4, 2025 13:17
@openshift-ci openshift-ci bot removed the lgtm label Apr 4, 2025
@openshift-ci
Copy link

openshift-ci bot commented Apr 4, 2025

New changes are detected. LGTM label has been removed.

@lkingland lkingland added the lgtm label Apr 4, 2025
@matejvasek
Copy link
Collaborator Author

/test Konflux kflux-prd-rh02 / kn-plugin-func-func-util-116-on-pull-request

@openshift-ci
Copy link

openshift-ci bot commented Apr 4, 2025

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

/test 417-e2e-oncluster-test
/test 417-images
/test 417-unit
/test 418-e2e-oncluster-test
/test 418-images
/test 418-unit

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

pull-ci-openshift-knative-kn-plugin-func-release-v1.16-417-e2e-oncluster-test
pull-ci-openshift-knative-kn-plugin-func-release-v1.16-417-images
pull-ci-openshift-knative-kn-plugin-func-release-v1.16-417-unit
pull-ci-openshift-knative-kn-plugin-func-release-v1.16-418-e2e-oncluster-test
pull-ci-openshift-knative-kn-plugin-func-release-v1.16-418-unit
Details

In response to this:

/test Konflux kflux-prd-rh02 / kn-plugin-func-func-util-116-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-sigs/prow repository.

@matejvasek
Copy link
Collaborator Author

/test "Konflux kflux-prd-rh02 / kn-plugin-func-func-util-116-on-pull-request"

@openshift-ci
Copy link

openshift-ci bot commented Apr 4, 2025

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

/test 417-e2e-oncluster-test
/test 417-images
/test 417-unit
/test 418-e2e-oncluster-test
/test 418-images
/test 418-unit

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

pull-ci-openshift-knative-kn-plugin-func-release-v1.16-417-e2e-oncluster-test
pull-ci-openshift-knative-kn-plugin-func-release-v1.16-417-images
pull-ci-openshift-knative-kn-plugin-func-release-v1.16-417-unit
pull-ci-openshift-knative-kn-plugin-func-release-v1.16-418-e2e-oncluster-test
pull-ci-openshift-knative-kn-plugin-func-release-v1.16-418-unit
Details

In response to this:

/test "Konflux kflux-prd-rh02 / kn-plugin-func-func-util-116-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-sigs/prow repository.

@matejvasek matejvasek requested review from gauron99 and removed request for jcrossley3 April 4, 2025 14:33
@openshift-merge-bot openshift-merge-bot bot merged commit 114a7ff into release-v1.16 Apr 4, 2025
44 of 45 checks passed
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