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

Development environment not correctly using development versions of replace_landsurface #16

Closed
aidanheerdegen opened this issue Dec 10, 2024 · 1 comment · Fixed by #17

Comments

@aidanheerdegen
Copy link
Member

Problem description: a new PR was merged into replace_landsurface. This triggered a new deployment of the development conda environment but the updated version of replace_landsurface was not installed in the development environment, instead the version deployed to the accessnri conda channel was present.

Details:

PR: ACCESS-NRI/replace_landsurface#49

Deployment workflow: https://github.com/ACCESS-NRI/access-ram-condaenv/actions/runs/12246525050

Workflow correctly identified the version from the PR above

https://github.com/ACCESS-NRI/access-ram-condaenv/actions/runs/12246525050/job/34162575850#step:4:87

Getting latest "replace_landsurface" dependency ...
replace_landsurface: Found PR #49.
  PR url: https://github.com/ACCESS-NRI/replace_landsurface/pull/49
  Commit ref: 9c7945fcf7ccb31454f9d87baec89a409fa16cbe
replace_landsurface: Found successful workflow run for the PR.
  Run ID: 12246488622
  Run url: https://github.com/ACCESS-NRI/replace_landsurface/actions/runs/12246488622
replace_landsurface: Successfully downloaded artifact "replace-landsurface-0+untagged.1.g51d97c1-py_0.tar.bz2".
Getting latest "era5_grib_parallel" dependency ...
era5_grib_parallel: No useful open PR found. Dependency will be installed from the latest release.
era5_grib_parallel: Successfully downloaded latest release: "era5grib-parallel-1.0-py_0.tar.bz2".
gh: No commit found for the ref 9c7945fcf7ccb31454f9d87baec89a409fa16cbe (HTTP 404)

However the install_dependencies step doesn't seem to find the version dowloaded above, but installs from the accessnri channel:

https://github.com/ACCESS-NRI/access-ram-condaenv/actions/runs/12246525050/job/34162575850#step:7:87

  + replace-landsurface        1.0  py_0                     accessnri         18kB

I think it might just be a channel ordering issue, as accessnri is before the local cache:

micromamba install replace-landsurface -c accessnri -c conda-forge -c coecms -c file:///home/runner/work/access-ram-condaenv/access-ram-condaenv/replace_landsurface_channel -c file:///home/runner/work/access-ram-condaenv/access-ram-condaenv/era5_grib_parallel_channel  -y
@aidanheerdegen
Copy link
Member Author

If I am correct it should be sufficient to just rearrange the ordering here to put the local cache first in the list of channels

https://github.com/ACCESS-NRI/access-ram-condaenv/blob/main/.github/workflows/deploy-prerelease.yml#L178-L179

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 a pull request may close this issue.

1 participant