ci: Add missing CI for feast-operator image builds#4184
Conversation
|
@woop is this something you can help with? the creation of those repos that is. |
Sorry, something went wrong.
|
Checking this out right now. |
Sorry, something went wrong.
|
Just to confirm, is there a reason we'd use kf-feast on GCR instead of dockerhub? Also, @franciscojavierarceo can you confirm what access you have already? |
Sorry, something went wrong.
|
@woop its used in |
Sorry, something went wrong.
|
I have access to Dockerhub but not gcr |
Sorry, something went wrong.
|
Hey @franciscojavierarceo you should have GCR access now if you check your GCP console. |
Sorry, something went wrong.
070c8d4 to
1719d65
Compare
June 19, 2024 14:36
|
@franciscojavierarceo @woop were these repos ever added? should we revisit this? it may help us troubleshoot some of the release build issues we've been facing. don't forget to add write perms for the gh actions robot account. |
Sorry, something went wrong.
|
Sorry been out of the loop on this one @tchughesiv does this still need review or action from me? |
Sorry, something went wrong.
1719d65 to
894d280
Compare
December 29, 2024 17:07
Signed-off-by: Tommy Hughes <tohughes@redhat.com>
6b84a1a to
1f3c60a
Compare
December 29, 2024 17:10
|
@franciscojavierarceo fixed... |
Sorry, something went wrong.
|
@tchughesiv so the robot should have access. The |
Sorry, something went wrong.
|
@franciscojavierarceo i guess it's worth a shot, but my expectation is that those new repos will have to exist first. especially because this registry has been deprecated and there's a migration guide. |
Sorry, something went wrong.
franciscojavierarceo
left a comment
There was a problem hiding this comment.
let's give this a try
Sorry, something went wrong.
What this PR does / why we need it:
Missing CI related to #4145
This change will require that the following image repos are created (if they don't already exist) -
Which issue(s) this PR fixes:
#4144