Skip to content

Update all IUC tools on test.galaxyproject.org#202

Merged
mvdbeek merged 2 commits intogalaxyproject:masterfrom
mvdbeek:update_iuc_tools_on_test
Jun 23, 2020
Merged

Update all IUC tools on test.galaxyproject.org#202
mvdbeek merged 2 commits intogalaxyproject:masterfrom
mvdbeek:update_iuc_tools_on_test

Conversation

@mvdbeek
Copy link
Copy Markdown
Member

@mvdbeek mvdbeek commented Jun 16, 2020

No description provided.

@mvdbeek mvdbeek requested a review from a team as a code owner June 16, 2020 19:53
@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 17, 2020

@galaxybot deploy this

1 similar comment
@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 17, 2020

@galaxybot deploy this

@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 17, 2020

Arg, this fails for revisions that appear both in toolshed and testtoolshed entries.

@mvdbeek mvdbeek force-pushed the update_iuc_tools_on_test branch from 06cfc3c to bdc5de9 Compare June 17, 2020 12:59
@mvdbeek mvdbeek requested a review from a team as a code owner June 17, 2020 16:42
@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 18, 2020

Hmm, with extended debugging I see this is the state for one of the failing repositories:

[{'id': '1dc8dee747cb1130',
  'tool_shed': 'toolshed.g2.bx.psu.edu',
  'name': 'gemini_roh',
  'owner': 'iuc',
  'installed_changeset_revision': 'd65ca2fa673b',
  'changeset_revision': '345d41f3f574',
  'ctx_rev': '7',
  'includes_datatypes': False,
  'tool_shed_status': {'latest_installable_revision': 'True',
   'repository_deprecated': 'False',
   'revision_update': 'False',
   'revision_upgrade': 'False'},
  'deleted': False,
  'uninstalled': False,
  'dist_to_shed': False,
  'status': 'Installed',
  'error_message': '',
  'description': 'Wrapper for the gemini tool suite: GEMINI roh',
  'url': '/api/tool_shed_repositories/1dc8dee747cb1130'}]

So it looks like the repo was actually installed correctly in the first call, but uwsgi closed the connection before the response was sent. I guess we need to also handle the repository state being terminal during the exception handling.

@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 18, 2020

@galaxybot deploy this

2 similar comments
@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 18, 2020

@galaxybot deploy this

@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 22, 2020

@galaxybot deploy this

@mvdbeek mvdbeek force-pushed the update_iuc_tools_on_test branch from a0c2080 to 33d2ae9 Compare June 23, 2020 10:24
@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 23, 2020

@galaxybot deploy this

@mvdbeek mvdbeek merged commit 101b584 into galaxyproject:master Jun 23, 2020
@martenson
Copy link
Copy Markdown
Member

@mvdbeek you ended up using your fork for ephemeris, what was the difference?

@mvdbeek
Copy link
Copy Markdown
Member Author

mvdbeek commented Jun 23, 2020

Need to include terminal states in possible states to check after a request timeout -- galaxyproject/ephemeris#161

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants