-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
deck should not assume authentication for logs #3407
Comments
/area prow |
This may also be useful for pods, since the GC interval is tight enough (24h today) but artifacts are likely longer-living and pods need to push artifacts out to somewhere anyway. |
Fixing this issue will also help in reusing ProwJobs for integrating any kind of external system by simply adding a new controller that will pick up plank's work for such external system, w/o having any external logic inside the core prow components. |
Another option to not assuming auth that generically covers at least basic auth and token auth is to have an |
Yeah, now I think that the best option is to expose the log endpoint from the jenkins operator where we already have auth, and use a k8s service. Deck then gets a map of {agent -> url} from the config where we have {"jenkins": "jenkins-operator"} and proxies the request. |
/assign |
deck today uses basic auth for getting the logs from Jenkins (this assumes that Jenkins is setup with restricted access) whereas since it runs inside a pod, it can reuse its service account token to authenticate for accessing pod logs. Instead of requiring credentials for Jenkins (or any other external system), it should use a location accessible to it w/o auth where the logs are accessible for jobs of agent != "kubernetes", eg. GCS, S3, or maybe the Jenkins server is publicly available. Given that, we could get rid of all Jenkins-specific options and simply use a URL template in the prow config similar to what plank uses today for setting up github context (see #3223)
@spxtr @fejta thoughts
The text was updated successfully, but these errors were encountered: