search

nextstrain / seasonal-flu

Scripts. config, and snakefiles for seasonal-flu nextstrain builds
44 stars 26 forks source link

profiles/nextstrain-public: Update build/auspice names #153

Closed joverlee521 closed 4 months ago

joverlee521 commented 6 months ago

Description of proposed changes

We will be changing the URLs for seasonal flu from nextstrain.org/flu/seasonal/... to nextstrain.org/seasonal-flu/... to match changes for avian-flu.

This commit changes the build_name and auspice_name for the nextstrain-public builds to reflect this change.

Related issue(s)

Resolves #152

Checklist

joverlee521 commented 6 months ago

Ah, will need to update AWS IAM permissions to allow the uploads to seasonal-flu* prefix.

Edit: Updated the NextstrainSeasonalFluUploader policy to allow reading/writing the seasonal-flu* prefix.

joverlee521 commented 6 months ago

Running test builds in https://github.com/nextstrain/seasonal-flu/actions/runs/7981622939.

The test builds should be deployed with the new Auspice JSON names seasonal-flu_*.json, but won't be visible on nextstrain.org (yet). The manifest_core.json is being updated by @lmoncla on the avian-flu-restructure branch in the nextstrain.org repo.

joverlee521 commented 4 months ago

Avian flu CI changes in https://github.com/nextstrain/avian-flu/pull/12 prompted me to take another look here. It's a nonissue here because the CI uses a completely separate profile https://github.com/nextstrain/seasonal-flu/blob/5606d218741abe3c333743c57ce69d6f84cb6829/.github/workflows/ci.yaml#L14

joverlee521 commented 4 months ago

Merging ahead of https://github.com/nextstrain/nextstrain.org/pull/811.

joverlee521 commented 4 months ago

Instead of running builds again, I've just copied over the flu_seasonal_*.json files to seasonal-flu_*.json using https://gist.github.com/joverlee521/dedba0a47e7fd04ef708c470df802769.

The builds that we will trigger in our new updates on Thursday will upload to the new paths.