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

Place lsst.alert.packet in lsst_distrib

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Story Points:
      2
    • Epic Link:
    • Sprint:
      AP S20-6 (May)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      On https://jira.lsstcorp.org/browse/DM-24324 we are beginning to incorporate the lsst.alert.packet code into the production pipelines. We need to make it readily available to systems like Jenkins.

      This ticket is to

      • move the sample_avro_alert repository to the lsst Github organization
      • rename the repo alert_packet for consistency
      • add it to lsst_distrib

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            It'd be a good idea to update the text of DMTN-093 on this ticket so that it refers to the new package.

            Show
            swinbank John Swinbank added a comment - It'd be a good idea to update the text of DMTN-093 on this ticket so that it refers to the new package.
            Hide
            swnelson Spencer Nelson added a comment -

            Not the world's most exciting review, but I've got a PR that replaces the string "sample-avro-alert" with "alert_packet" everywhere in github.com/lsst/alert_packet.

            Show
            swnelson Spencer Nelson added a comment - Not the world's most exciting review, but I've got a PR that replaces the string "sample-avro-alert" with "alert_packet" everywhere in github.com/lsst/alert_packet.
            Hide
            swinbank John Swinbank added a comment -

            The changes in alert_packet look fine to me. There's an obvious typo in lsst/repos... but I bet you're way ahead of me on fixing that.

            Any chance of a DMTN-093 update? Or would you prefer to defer that to another ticket? I'm just thinking of this line.

            Show
            swinbank John Swinbank added a comment - The changes in alert_packet look fine to me. There's an obvious typo in lsst/repos... but I bet you're way ahead of me on fixing that. Any chance of a DMTN-093 update? Or would you prefer to defer that to another ticket? I'm just thinking of this line .
            Hide
            swnelson Spencer Nelson added a comment -

            John Swinbank, good catch on DMTN-093! I'll do that now.

            Show
            swnelson Spencer Nelson added a comment - John Swinbank , good catch on DMTN-093! I'll do that now.
            Hide
            swnelson Spencer Nelson added a comment -

            DMTN-093 is updated.

            Show
            swnelson Spencer Nelson added a comment - DMTN-093 is updated.
            Hide
            swnelson Spencer Nelson added a comment -

            This appears to have worked. DM-24324 was able to use the package.

            Show
            swnelson Spencer Nelson added a comment - This appears to have worked. DM-24324 was able to use the package.
            Hide
            swnelson Spencer Nelson added a comment -

            John Swinbank, ping on review - I think this is done.

            Show
            swnelson Spencer Nelson added a comment - John Swinbank , ping on review - I think this is done.
            Hide
            swinbank John Swinbank added a comment -

            Sorry, slipped my mind.

            Show
            swinbank John Swinbank added a comment - Sorry, slipped my mind.

              People

              • Assignee:
                swnelson Spencer Nelson
                Reporter:
                ebellm Eric Bellm
                Reviewers:
                John Swinbank
                Watchers:
                Chris Morrison, Eric Bellm, John Swinbank, Spencer Nelson
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel