Update ctrl_bps_panda/config/bps_usdf.yaml to allow for local custom setup.

Description

Can we change the setup in bps_usdf.yaml to have a hook for custom setup. I.e., so that it looks like this:

Issue Matrix

hide

Activity

Show:
Zhaoyu Yang
March 1, 2023 at 4:27 PM

I merged the PR, so the hook will be available in next weekly.

Zhaoyu Yang
February 28, 2023 at 4:43 PM

Thanks Michelle. For this ticket, I'll add the doc/changes file, and also move "pwd; ls -al;" to runnerCommand as you suggested.

About change bps_idf.yaml and bps_panda.yaml, we should do it in a seperate ticket? I'll check more on those.

Michelle Gower
February 28, 2023 at 4:02 PM

Merge approved (after adding doc/changes file)

  • Needs doc/changes file.

  • Is there documentation somewhere (panda.lsst.io maybe) where custom_lsst_setup needs to be mentioned?

  • I think this meets the user's request. However, in this or some other ticket, similar changes should be made to the bps_idf.yaml as this feature is not usdf specific (even though the value of custom_lsst_setup is different). Also, after pulling this out, can runnerCommand now only appear in the central bps_panda.yaml? (maybe another "extra_docker_env" bps setting needs to be introduced and only have a value at IDF?).

Zhaoyu Yang
February 28, 2023 at 3:40 AM

I added the hook as Eric requested. From my test it works. However I'm not quite sure this is what we should/want to have in the bps_usdf.yaml in general. 

 Can you help review this ticket? I remember you have some comments at last week's CM meeting about this.

Done
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Reviewers

Michelle Gower

RubinTeam

Components

Checklist

Created February 27, 2023 at 5:52 PM
Updated March 1, 2023 at 4:27 PM
Resolved March 1, 2023 at 4:26 PM