projectriff / riff

riff is for functions
https://projectriff.io
Apache License 2.0
799 stars 64 forks source link

public CI with github status #258

Closed jldec closed 6 years ago

jldec commented 6 years ago

workflow

questions / caveats

jchesterpivotal commented 6 years ago

As I see it:

jchesterpivotal commented 6 years ago

Added github status resource to pipeline, which tells Github about pending, failed and succeeded statuses per-commit.

jchesterpivotal commented 6 years ago

@jldec I think I've got the "workflow" bit done. I might suggest we split out PRs as another issue. Anything else you'd need before we close this one?

jchesterpivotal commented 6 years ago

Actually, scratch that, I have a bit more to do. It currently only applies status on the build-container jobs at the start of the pipeline. It'd be nice if it also updated it for other jobs, especially system tests.

trisberg commented 6 years ago

should we move the pivotal-cf/pfs-system-test to projectriff/acceptance-test?

change the sk8s docker hub user to riffci for images we use?

jchesterpivotal commented 6 years ago

No objection, but this issue has been fairly busy and I'm anxious to close it in its own right.

jchesterpivotal commented 6 years ago

Except, actually, you're right about both of those being private bits.

Ugh.

jchesterpivotal commented 6 years ago

Infinite todos

jchesterpivotal commented 6 years ago

Actually, I have a hunch that I am wrong about not being able to set nice commit statuses for every job / commit permutation. The theory would be:

  1. Set the status for every commit in every job.
  2. Rely on the context key to distinguish what is going on (so a commit might be passed for ci/riff/build-http-gateway-container but pending on run-riff-system-tests-gke).
  3. Use version: every to ensure we are testing all the permutations. Right now we don't.
jchesterpivotal commented 6 years ago

blocking on projectriff/ci#50

jchesterpivotal commented 6 years ago

I'm going to call this done. We have a public CI, images are pushed to public repos and so on. I'll break out missing FATS tests into another issue.