Reusing logic from kubernetes/test-infra? about prow-github-actions HOT 7 OPEN

I'm super into this investigation! Absolutely happy to chime in as well!

from prow-github-actions.

Hi @ashleypafko - I personally haven't been investigating this (I wish I had more time to do open source stuff 😭)

I'm open to any contributions on this and I'd be super interested to hear what could work. If you want to pick this up, I'd be happy to see what you come up with!

from prow-github-actions.

@carolynvs - Hmmmm. I'm not sure; I had looked at re-using some Go code from prow and test-infra early on, but at the time, the Go octokit API library was being archived. But seems there is a google maintained go-github library nowadays that may work for our needs.

I'm also not sure what real benefit we would get from a full re-write vs a port. I'd consider this github action fairly stable and I worry about introducing more complexity for users and maintainers if we wanted to re-using much logic from kubernetes in Go.

Generally, I'm not a fan of re-writing this action in Go. I like having nock and jest for API mocking and in my experience, that's painful in Go.

But it sounds like you're asking if we could look into somehow importing that Go code and wrapping it around executors we could use in this javascript action? I'd be all for investigating that and if it's stable, let's do it!

Opinions welcome!

from prow-github-actions.

Maybe a simple example of how we could do that with a small go library would help me wrap my head around what you're suggesting. If I had to read between the lines, sounds like how you'd do that is

• import the necessary libraries with public methods into our own go program
• build/compile that binary
• have our action (written in TS) execute that binary with some inputs, deriving the outputs

?

from prow-github-actions.

Oh yeah sorry, I totally am not interesting in reimplementing the gh-action in Go! 😬 What you suggested above was exactly what I was thinking for how to reuse test-infra without having to port chunks of it from go to typescript. I'm not sure which is more effort to maintain over time so I may explore both options.

I mostly wanted to make sure that if one was super distasteful that I knew not to waste time going down that path.

from prow-github-actions.

Hey @carolynvs & @jpmcb ! I've been keeping an eye on this issue and pretty interested in this feature... just curious, are you guys currently working on this / is it close to being done? Love the project 🙂

from prow-github-actions.

Same, I haven't had time to work on this but would be super interested in seeing it implemented!

from prow-github-actions.