Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-32448

Backport incorrect GAaP field name

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      Yusra AlSayyad found out that in obs_lsst/policy/imsim/Object.yaml has `ext_gaap_GaapFlux_1_15x_0_5_instFlux` mapped to `gaap3p0Flux`. This should be backported.

        Attachments

          Activity

          Hide
          jcarlin Jeffrey Carlin added a comment -

          Looks good - that was a lot of text that changed in the previous ticket, so I'm not surprised that one got mis-typed...

          Show
          jcarlin Jeffrey Carlin added a comment - Looks good - that was a lot of text that changed in the previous ticket, so I'm not surprised that one got mis-typed...
          Hide
          kannawad Arun Kannawadi added a comment -

          Thank you for that swift review, Jeff! Merging it now since Jenkins passed.

          https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/35316/pipeline/

          Show
          kannawad Arun Kannawadi added a comment - Thank you for that swift review, Jeff! Merging it now since Jenkins passed. https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/35316/pipeline/
          Hide
          kannawad Arun Kannawadi added a comment -
          Show
          kannawad Arun Kannawadi added a comment - The backporting branch has also passed Jenkins:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/35358/pipeline
          Hide
          yusra Yusra AlSayyad added a comment -

          Great. Next steps would be to make a PR for DM-32448-v23 and merge to v23.0.x

          Show
          yusra Yusra AlSayyad added a comment - Great. Next steps would be to make a PR for DM-32448 -v23 and merge to v23.0.x
          Hide
          kannawad Arun Kannawadi added a comment -

          PR here: https://github.com/lsst/obs_lsst/pull/371 . I'll go ahead and merge it once I get an approval from you, Yusra AlSayyad

          Show
          kannawad Arun Kannawadi added a comment - PR here: https://github.com/lsst/obs_lsst/pull/371  . I'll go ahead and merge it once I get an approval from you, Yusra AlSayyad
          Hide
          kannawad Arun Kannawadi added a comment -

          Backporting complete

          Show
          kannawad Arun Kannawadi added a comment - Backporting complete
          Hide
          yusra Yusra AlSayyad added a comment -

          That's usually communicated by adding the 'backport-done' label (step 6 in https://developer.lsst.io/work/backports.html), which moves it from the "to be backported' query to the "backport done" query.

          Show
          yusra Yusra AlSayyad added a comment - That's usually communicated by adding the 'backport-done' label (step 6 in https://developer.lsst.io/work/backports.html ), which moves it from the "to be backported' query to the "backport done" query.

            People

            Assignee:
            kannawad Arun Kannawadi
            Reporter:
            kannawad Arun Kannawadi
            Reviewers:
            Jeffrey Carlin
            Watchers:
            Arun Kannawadi, Jeffrey Carlin, Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.