search

kbaseapps / kb_staging_exporter

MIT License
0 stars 3 forks source link

DATAUP-515: Fix staging service 500 on deleting export #26

Closed MrCreosote closed 3 years ago

MrCreosote commented 3 years ago

The staging service cannot currently delete any files or folders created by the exporter because the staging service runs as user 800 (at least in CI) and this app runs as root (yuck). However, both the service and this app are part of the root group, so adding write permissions to that group should allow the staging service to delete files. This is a shorter term fix - the longer term fix would be to use sane user names for both processes, and possibly have the exporter chown to the staging service.

MrCreosote commented 3 years ago

I'll need to test in the various environments before I can be sure about the fix, obviously

MrCreosote commented 3 years ago

@Tianhao-Gu I realized this had a nasty bug, fixed now - can you take another look?