Nodejs.dev: CI: /preview command on second time fail

Created on 7 May 2020  路  4Comments  路  Source: nodejs/nodejs.dev

Description

When commenting with /preview on the second time in a PR, the Upload to staging action fails with an AccessDeniedException: 403 error.

Steps to reproduce

  • Create a PR
  • Comment with /preview
  • Push a new change
  • Comment again with /preview

Expected result

Upload to staging without errors

Actual result

https://github.com/nodejs/nodejs.dev/runs/640799129?check_suite_focus=true

bug

Most helpful comment

This should be fixed now, I've extended the permissions. I'm not thrilled with how broad the IAM is and may play around with this during the week, but for now it might be good enough.

All 4 comments

The gsutil rsync command used to push file to staging uses checksum of file to check if the file is updated or not. If it finds a file to be updated, it will try to delete the older file.
Looks like the current IAM role has permissions to create a file but not delete a file, hence resulting in AccessDeniedException: 403 error.

@MylesBorins would you mind taking a look at this error?

This should be fixed now, I've extended the permissions. I'm not thrilled with how broad the IAM is and may play around with this during the week, but for now it might be good enough.

It works again thanks @MylesBorins

Was this page helpful?
0 / 5 - 0 ratings

Related issues

SMotaal picture SMotaal  路  4Comments

benhalverson picture benhalverson  路  3Comments

talarviv picture talarviv  路  4Comments

mmarchini picture mmarchini  路  4Comments

tstreamDOTh picture tstreamDOTh  路  3Comments