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

Hotfix/deprecated analysis states #1006

Merged

Conversation

apetkau
Copy link
Member

@apetkau apetkau commented May 19, 2021

Description of changes

Fixed issue with analyses pages not displaying by adding back in DOWNLOADING and FINISHED_DOWNLOADING states.

To test this you can setup a new instance of IRIDA 21.01.4 and run an analysis pipeline. Then, you can go into the database and switch one of the analysis states in the audit tables to DOWNLOADING. For example:

1. analysis_submission_AUD before change

+----+------------------------------------+------------------+------------------------+
| id | name                               | analysis_state   | analysis_cleaned_state |
+----+------------------------------------+------------------+------------------------+
|  1 | bio_hansel_20210518_AE014613       | NEW              | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | PREPARING        | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | PREPARED         | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | SUBMITTING       | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | RUNNING          | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | FINISHED_RUNNING | NOT_CLEANED            |
+----+------------------------------------+------------------+------------------------+

2. Switch a state to DOWNLOADING

update analysis_submission_AUD set analysis_state = 'DOWNLOADING' where id = 1 and analysis_state = 'FINISHED_RUNNING';

3. analysis_submission_AUD after change

+----+------------------------------------+------------------+------------------------+
| id | name                               | analysis_state   | analysis_cleaned_state |
+----+------------------------------------+------------------+------------------------+
|  1 | bio_hansel_20210518_AE014613       | NEW              | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | PREPARING        | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | PREPARED         | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | SUBMITTING       | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | RUNNING          | NOT_CLEANED            |
|  1 | bio_hansel_20210518_AE014613       | DOWNLOADING      | NOT_CLEANED            |
+----+------------------------------------+------------------+------------------------+

4. Try to access analyes

Now, go to Analyses > My Analyses and the page should fail to load (as described in #1004 ). Applying this fix (checking out this branch and re-running IRIDA) will fix the issue and the page will load again.

Related issue

Fix for #1004

Checklist

Things for the developer to confirm they've done before the PR should be accepted:

  • CHANGELOG.md (and UPGRADING.md if necessary) updated with information for new change.
  • Tests added (or description of how to test) for any new features.
  • [ ] User documentation updated for UI or technical changes.

@apetkau apetkau changed the title Hotfix/deprecated analysis states WIP:Hotfix/deprecated analysis states May 19, 2021
@apetkau apetkau changed the title WIP:Hotfix/deprecated analysis states Hotfix/deprecated analysis states May 20, 2021
@apetkau apetkau requested a review from tom114 May 20, 2021 14:04
@apetkau apetkau changed the base branch from master to development May 20, 2021 15:05
Copy link
Contributor

@tom114 tom114 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works as described. Tested the issue with a manually edited database as suggested. Thanks.

@tom114 tom114 added the back-end Issues relating to the service layer, repository, model, configuration, or database label May 20, 2021
@tom114 tom114 self-assigned this May 20, 2021
@tom114 tom114 merged commit 7dfb548 into phac-nml:development May 20, 2021
@apetkau apetkau deleted the hotfix/deprecated-analysis-states branch May 20, 2021 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-end Issues relating to the service layer, repository, model, configuration, or database
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants