Skip to content

Fix quay repository presence check for single target builds#993

Merged
nsoranzo merged 2 commits intogalaxyproject:masterfrom
mvdbeek:fix_registering_single_containers_with_different_suffix
Feb 5, 2020
Merged

Fix quay repository presence check for single target builds#993
nsoranzo merged 2 commits intogalaxyproject:masterfrom
mvdbeek:fix_registering_single_containers_with_different_suffix

Conversation

@mvdbeek
Copy link
Member

@mvdbeek mvdbeek commented Jan 30, 2020

Since the v2_image_name for a single target doesn't include the build
suffix given by bioconda (e.--hc9558a2_0) we don't get an exact
match from the quay.io api.

Since the v2_image_name for a single target doesn't include the build
suffix given by bioconda (e.`--hc9558a2_0`) we don't get an exact
match from the quay.io api.
@nsoranzo nsoranzo requested a review from bgruening January 30, 2020 10:03
mvdbeek added a commit to mvdbeek/planemo-monitor that referenced this pull request Jan 30, 2020
@mvdbeek
Copy link
Member Author

mvdbeek commented Jan 30, 2020

this isn't actually meant for merging, I think the logic actually belongs in galaxy-tool-util but I wanted to have a check to see if / what else is missing for planemo monitor.

@mvdbeek mvdbeek force-pushed the fix_registering_single_containers_with_different_suffix branch from 9885715 to f396d5b Compare January 30, 2020 10:49
@mvdbeek mvdbeek force-pushed the fix_registering_single_containers_with_different_suffix branch from f396d5b to 7e7a915 Compare January 30, 2020 10:50
mvdbeek added a commit to mvdbeek/planemo-monitor that referenced this pull request Jan 30, 2020
mvdbeek added a commit to mvdbeek/planemo-monitor that referenced this pull request Jan 30, 2020
@mvdbeek
Copy link
Member Author

mvdbeek commented Jan 30, 2020

OK, we already had everything in place to do this right in galaxy-tool-util.

bgruening added a commit to galaxyproject/planemo-monitor that referenced this pull request Jan 30, 2020
@mvdbeek mvdbeek requested a review from nsoranzo February 4, 2020 14:27
@nsoranzo nsoranzo merged commit ba7fb01 into galaxyproject:master Feb 5, 2020
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