feature request: make test URLs point directly to pantheon bucket |
||||
Issue descriptionThe test logs contain lists of URLs like these: security_RuntimeExecStack http://cautotest/tko/retrieve_logs.cgi?job=/results/98016880-chromeos-test/ security_OpenFDs http://cautotest/tko/retrieve_logs.cgi?job=/results/98016881-chromeos-test/ kernel_ConfigVerify http://cautotest/tko/retrieve_logs.cgi?job=/results/98016883-chromeos-test/ security_EnableChromeTesting http://cautotest/tko/retrieve_logs.cgi?job=/results/98016884-chromeos-test/ security_SymlinkRestrictions http://cautotest/tko/retrieve_logs.cgi?job=/results/98016885-chromeos-test/ Those URLs are immediately redirected to their pantheon buckets, like this one: https://pantheon.corp.google.com/storage/browser/chromeos-autotest-results/98016931-chromeos-test/ but the redirection takes many extra seconds (and occasionally fails completely). I can copy-paste the job ID, but it would be simpler and faster to produce the pantheon URL directly, if possible.
,
Jan 26 2017
The reason for this redirection is that while the tests are running and for a small time after the tests have finished running, the logs are only available on the autotest servers. The URLs point to the autotest AFE and the AFE decides about whether the file is present locally. If it isn't, you are redirected to GS. My position has been in the past that the dependence on correct functioning of AFE to get to the logs on GS is broken. If we can: - Provide better guarantees on the time it takes for us to offload logs to GS after a test finishes - Remove the requirement of being able to look at logs while a job is running. Then, we can just start pointing to GS instead of redirecting through AFE.
,
Jan 26 2017
+ triaging queue. Try to use go/cros-infra-bug to file bugs to get the correct labels :)
,
Jan 26 2017
FYI: I have never seen a case in which I was redirected to a "local" file. There have been many occasions in which I was redirected to GS and got the message "there are no objects in this folder" (which should really be "this folder doesn't exist", I opened a GS bug for this). Thank you for the go/cros-infra-bug advice :)
,
Jan 26 2017
Oh yeah, you're right. That indirection has been broken for a while. And since we haven't fixed it right now, I'm going to use that as proof that we don't need it. +jrbarnette: because I think you've had concerns about my request in the past to do pretty much what this bug wants.
,
Feb 1 2017
Discussed further, not possible to fix.
,
Feb 1 2017
Would it be possible to add a brief summary of the discussion here FMI? Thanks!
,
Feb 1 2017
I think the conclusion was: We can not support both at the same time: (1) Don't redirect via the AFE when logs are available on GS (what this bugs wants) (2) Provide access to partial logs on the lab servers while the test is running / before logs are offloaded. (This is currently broken as #4 states, and as I was aware of). I think the intended behaviour is (2), and should be fixed. So this bug, which asks for (1) has been marked WontFix. |
||||
►
Sign in to add a comment |
||||
Comment 1 by semenzato@chromium.org
, Jan 26 2017