Skip to content

Conversation

@DXCanas
Copy link
Member

@DXCanas DXCanas commented Jan 29, 2026

Summary

Initially put in place during #5596

The idea being that the CD pipelines wouldn't have to updated on the infra side in coordination with the merge of this PR.

But it didn't work:


David Cañas  [3:21 PM]
I actually added symlinks specifically to avoid having to do this  

David Cañas  [3:30 PM]
Oh. Just got to look at the error message.The problem here is that it is docker inside of docker and docker does not like sym links

twice

Marcella Maki  [4:14 PM] @david the build failed and I don't know if it's me

Already have image (with digest): gcr.io/cloud-builders/docker
unable to prepare context: unable to evaluate symlinks in Dockerfile path: lstat /workspace/k8s/images/app/docker: no such file or directory

if this is a larger issue it does not need to be remediated right now, it's fine.

David Cañas  [4:24 PM]
I had this lined up yesterday, and then was like "nahhh they're not merging it up yet and I might brake build for any other changes" and then completely forgot when you told me you were merging up


References

Resolves https://github.com/learningequality/infrastructure/issues/660

Reviewer guidance

Enjoy the red 🤩

Studio is now free of deprecated infra code!
We think

Resolves learningequality/infrastructure#660
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant