nephio-project / nephio

Nephio is a Kubernetes-based automation platform for deploying and managing highly distributed, interconnected workloads such as 5G Network Functions, and the underlying infrastructure on which those workloads depend.
Apache License 2.0
104 stars 53 forks source link

porch: creating a revision of a package that contains a inaccessible function results in empty pipeline #682

Open liamfallon opened 5 months ago

liamfallon commented 5 months ago

Original issue URL: https://github.com/kptdev/kpt/issues/3598 Original issue user: https://github.com/droot Original issue created at: 2022-10-04T21:56:17Z Original issue last updated at: 2022-11-15T22:01:15Z Original issue body: I have a package called base-app with published version v2. The revision is published through kpt CLI.

This package uses a function that is not publicly accessible. When I create a new revision using CaD UI, the new revision's Kptfile's pipeline is wiped out completely. I do not see any errors also in the porch logs.

I think most likely function-runner is failing to execute this function, but then the expected behavior is porch to return with error.

Original issue comments: Comment user: https://github.com/droot Comment created at: 2022-10-04T21:56:32Z Comment last updated at: 2022-10-04T21:56:32Z Comment body: /cc @ChristopherFry

Comment user: https://github.com/droot Comment created at: 2022-10-04T22:07:00Z Comment last updated at: 2022-10-04T22:07:00Z Comment body: Quick update: i published another version v3 through kpt CLI and this time using a public version of the function, but the behavior still persists.

Comment user: https://github.com/ChristopherFry Comment created at: 2022-10-04T23:30:49Z Comment last updated at: 2022-10-04T23:30:49Z Comment body: > Quick update: i published another version v3 through kpt CLI and this time using a public version of the function, but the behavior still persists.

Confirmed this is a Porch issue. At the moment, Porch does not like any packages that were created using the kpt cli and tagged manually. See #3599.