Skip to content
Snippets Groups Projects

environment tweaks

Merged environment tweaks
conus-dps-env-tweaks into conus-dps
3 unresolved threads
Merged gcorradini requested to merge conus-dps-env-tweaks into conus-dps
3 unresolved threads

I think there are some things we don't want in the env-feds.yaml that are too presumptuous about the context in which it's being installed in. Comments inline below

Merge request reports

Merged by gcorradinigcorradini 2 years ago (Feb 6, 2023 7:13pm UTC)

Loading

Pipeline #3931 passed

Stage:

    Pipeline passed for f78a8ba1 on conus-dps

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
    env-feds.yml
    1 name: null
    1 name: feds
  • gcorradini
    gcorradini @gcorradini started a thread on the diff
    • env-feds.yml
    1 name: null
    1 name: feds
    2 2 channels:
    3 3 - conda-forge
    4 - defaults
  • gcorradini
    gcorradini @gcorradini started a thread on the diff
    • env-feds.yml
    245 244 - zarr=2.12.0=pyhd8ed1ab_0
    246 245 - zeromq=4.3.4=h9c3ff4c_1
    247 246 - zlib=1.2.12=h166bdaf_2
    248 - zstd=1.5.2=h6239696_4
    249
    250 prefix: /projects/env-feds
    • gcorradini
      gcorradini @gcorradini ·
      Author Owner

      this is the big presumption, only the ADE images have a /projects/env-feds folder. since run-dps.sh already creates a different prefix for installations let's just leave this out, sound good?

      By Gregory on 2023-02-06T18:20:10 (imported from GitLab)

    • Please register or sign in to reply
  • gcorradini requested review from @eorland_gee

    requested review from @eorland_gee

    By Gregory on 2023-02-06T18:20:34 (imported from GitLab)

  • gcorradini approved this merge request

    approved this merge request

    By Eli Orland on 2023-02-06T19:13:47 (imported from GitLab)

  • gcorradini mentioned in commit f78a8ba1

    mentioned in commit f78a8ba1

    By Eli Orland on 2023-02-06T19:13:49 (imported from GitLab)

  • gcorradini merged

    merged

    • Please register or sign in to reply