Skip to content
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

Exception in public_access_update when called in syncmodifyapi #1687

Closed
mikkonie opened this issue May 31, 2023 · 3 comments
Closed

Exception in public_access_update when called in syncmodifyapi #1687

mikkonie opened this issue May 31, 2023 · 3 comments
Assignees
Labels
app: samplesheets Issue in the samplesheets app bug Something isn't working
Milestone

Comments

@mikkonie
Copy link
Contributor

mikkonie commented May 31, 2023

As a part of working on #1617 I noticed a crash with a certain project, when performing public_access_update when called from syncmodifyapi. This may be related to recent changes within that issue, or something from before I simply didn't notice.

Full dump of error pasted below. I'll have to try and reproduce and see if this requires syncmodifyapi or can be triggered without it.

Facts

  • Public guest access is not enabled
  • Sample sheets exist
  • iRODS collections have not been created
    • I have a hunch this may be the root cause..
@mikkonie mikkonie added bug Something isn't working app: samplesheets Issue in the samplesheets app labels May 31, 2023
@mikkonie mikkonie added this to the v0.14.0 milestone May 31, 2023
@mikkonie mikkonie self-assigned this May 31, 2023
@mikkonie
Copy link
Contributor Author

Log dump:

2023-05-31 11:00:29,602 [DEBUG] projectroles.management.commands.syncmodifyapi: Synchronizing project: Example Projects / Becnel Cohort
(Taskflow sync flows called here)
2023-05-31 11:00:30,015 [DEBUG] projectroles.views: Calling perform_project_sync() in plugin "samplesheets"
2023-05-31 11:00:30,083 [INFO] taskflowbackend.api: Building flow "public_access_update"..
2023-05-31 11:00:30,083 [INFO] taskflowbackend.api: Building flow OK
2023-05-31 11:00:30,083 [INFO] taskflowbackend.flows: Running flow "public_access_update"
2023-05-31 11:00:30,087 [INFO] taskflowbackend.tasks: Executed: <iRODS> Set collection user group access (SetAccessTask)
2023-05-31 11:00:30,089 [INFO] taskflowbackend.tasks: Reverted: <iRODS> Set collection user group access (SetAccessTask)
2023-05-31 11:00:30,090 [ERROR] taskflowbackend.flows: Exception in run_flow(): 
2023-05-31 11:00:30,091 [ERROR] taskflowbackend.api: Error running flow: 
2023-05-31 11:00:30,091 [ERROR] samplesheets.plugins: Public status update taskflow failed: Error running flow: 
2023-05-31 11:00:30,091 [ERROR] projectroles.views: Exception in perform_project_sync() for plugin "samplesheets": Error running flow: 
2023-05-31 11:00:30,091 [ERROR] projectroles.management.commands.syncmodifyapi: Exception in project sync: Error running flow: 
2023-05-31 11:00:30,091 [ERROR] projectroles.management.commands.syncmodifyapi: Project sync failed! Unable to continue, exiting..

@mikkonie
Copy link
Contributor Author

When triggering public guest access from the project update menu, we get the following:

2023-05-31 12:06:59,212 [DEBUG] projectroles.views: Calling perform_project_modify() in plugin "samplesheets"
2023-05-31 12:06:59,221 [DEBUG] samplesheets.plugins: Skipping: Investigation collections not created in iRODS

That is correct. I guess the check is simply missing/broken in perform_project_sync().

@mikkonie
Copy link
Contributor Author

This turned out to be an existing bug. Fixed in the #1617 work branch, to be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app: samplesheets Issue in the samplesheets app bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant