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

Import as crash #4075

Closed
wants to merge 2 commits into from
Closed

Import as crash #4075

wants to merge 2 commits into from

Conversation

dominikl
Copy link
Member

Fix for Ticket 12929

Test:

  • Log in as an admin user
  • Kick off an "import as" process which takes some time, so you'll have time for the next step. Important: Really start the import process, don't just add it to the queue.
  • Go back to the import step, and start another "import as" job for another user. But now create a Project and/or Dataset for the import. Without the PR this step was enough to crash Insight, although the crash could happen delayed by some time, depending on how "heavy" the previous started import jobs are.
  • Make sure both import jobs are passing like expected.

Note:
The first commit is the actualy bugfix. The second commit just adds a couple of toString methods to some Gateway/Pojo classes, as I found this makes debugging much easier; can remove this commit again if needed.

@jburel jburel added the develop label Aug 24, 2015
@jburel
Copy link
Member

jburel commented Aug 25, 2015

No problem noticed when I tested it yesterday

@dominikl
Copy link
Member Author

--exclude until #4092 is merged

@dominikl
Copy link
Member Author

I don't understand it, I'm sure I tested it and it worked, but now I can't sudo create projects/dataset because I get

2015-08-26 13:43:12,815 INFO  [       ome.services.sessions.SessionBean] (l.Server-3) Handling session exception: 
ome.conditions.SecurityViolation: Group sudo is not permitted for group group1 (gid=3)

@dominikl
Copy link
Member Author

I'll keep that one excluded for now, probably will close it due to the problem there.

@dominikl
Copy link
Member Author

Closing the PR, there must be some deeper problem within the Gateway code (but it's not introduced by the Gateway decoupling PR, as it also happens in the 5.1 line).

@dominikl dominikl closed this Aug 28, 2015
@dominikl dominikl deleted the import_as_crash branch October 27, 2015 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants