-
Notifications
You must be signed in to change notification settings - Fork 78
Publish OCI images for teleport plugins #438
Comments
For the CI portion of this. I noticed both drone and cloudbuild being used in this repository. Should I lean towards one or the other?
Can you clarify what you mean by this? |
Use cloudbuild for CI. I think we still prefer Drone for publishing, as the point of this project isn't to refactor the publishing/promotion pipleline.
Ideally, we'd build the docker images before merge, such that if there was a syntax error in the dockerfiles, we'd catch it. My general thought is:
My concern here would be: Can we safely build an image at the CI stage. DinD hasn't been sufficient isolation previously, and I'm not sure what options GCB offers for building images at the CI phase. Build after approval (for external contributors) is fine. |
Yeah. The eventual goal is GCB for everything, but we're a long, long way from there.
GCB can definitely build/push images to a docker repo, so the question becomes can we do it safely. Reasonably sure that it's done by mounting the docker socket into the build container (i.e. Docker-out-of-Docker), with whatever implications that has (and I don't know enough about that to comment on off the top of my head). |
The question I'd like to attempt to answer is: Is each build isolated from other builds? For instance, is the lint and unit-tests-linux builds run in separate environments but the steps in each build are isolated only by containers? The machineType option discusses setting the type of virtual machine your build is run on. This to me seems to indicate a virtual machine is used. |
An update: Merging via tag and promoting to quay have been merged. See #456 and #466 #460 tracks the issue of building and pushing docker images on push to master. This would allow developers to test their changes without having to create a tag. I'm independently verified the pipeline but am waiting for #469 (Release 9.0.2) to be merged in order to verify if it "just works" for other people. I will start work on documenting these changes now. |
Docs Update: Merged: #473 which contained updates for README files in the different plugin directories Descriptions in Quay now available for following repositories:
Next Steps: |
With:
Documentation + backports for the above repos is added to goteleport.com in their corresponding guide sections. |
Summary
This is a sub-issue of making our teleport plugins more user friendly (#160). This ticket tracks the following dev tasks:
The text was updated successfully, but these errors were encountered: