New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 10
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment
link

Issue 920114: Failed to upload ninja_log from https://chromium-build-stats.appspot.com/ninja_log/

Reported by tikuta@chromium.org, Jan 9 Project Member

Issue description

I got following error when uploading ninja log.

```
upload status: https://www.googleapis.com/upload/storage/v1/b/chromium-build-stats.appspot.com/o?uploadType=media&name=upload/ninja_log.NLoPcyIfdM-WCDsTVlsmKh1WRJgwyodk0o5plSMHrbM=.gz 403 403 Forbidden: {
 "error": {
  "errors": [
   {
    "domain": "global",
    "reason": "insufficientPermissions",
    "message": "Insufficient Permission"
   }
  ],
  "code": 403,
  "message": "Insufficient Permission"
 }
}
: <nil>
```
 

Comment 1 by tikuta@chromium.org, Jan 9

Status: Started (was: Untriaged)

Comment 2 by bugdroid1@chromium.org, Jan 10

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/7566ce97769c1d428a658785fac7a54cbb78978b

commit 7566ce97769c1d428a658785fac7a54cbb78978b
Author: Takuto Ikuta <tikuta@chromium.org>
Date: Thu Jan 10 06:52:07 2019

[chromium_build_stats] use storage library for logstore.Upload

To fix permission issue, let me use storage library for upload.

I confirmed this works in
https://19897-b0f13f6-tikuta-storage-lib-dot-chromium-build-stats.appspot.com/ninja_log/upload/ninja_log.38llWvLNwxIfW96zlGW7R4MVV-oGPoNRwYQPC_GbKpo=.gz

Bug:  920114 
Change-Id: Ifd85d1bf76dda5b5fbb8523242c40f31b9ca0b68
Reviewed-on: https://chromium-review.googlesource.com/c/1401974
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Fumitoshi Ukai <ukai@chromium.org>
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#19886}
[modify] https://crrev.com/7566ce97769c1d428a658785fac7a54cbb78978b/go/src/infra/appengine/chromium_build_stats/app/ninja_log.go
[modify] https://crrev.com/7566ce97769c1d428a658785fac7a54cbb78978b/go/src/infra/appengine/chromium_build_stats/logstore/upload.go

Comment 3 by tikuta@chromium.org, Jan 10

Status: Fixed (was: Started)

Sign in to add a comment