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

Pin version of fastavro in use in alert_stream

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: alert_stream
    • Labels:
      None

      Description

      Per https://github.com/lsst-dm/alert_stream/issues/24, the latest release of fastavro is incompatible with the serialized alert data in lsst-dm/alert_stream.

      Long term, we need to fix the data. Short term, pin the version of fastavro to one that works in the Dockerfile.

        Attachments

          Issue Links

            Activity

            No builds found.
            swinbank John Swinbank created issue -
            swinbank John Swinbank made changes -
            Field Original Value New Value
            Status To Do [ 10001 ] In Progress [ 3 ]
            swinbank John Swinbank made changes -
            Description Per https://github.com/lsst-dm/alert_stream/issues/24, |https://github.com/lsst-dm/alert_stream/issues/24, the latest release of fastavro is incompatible with the serialized alert data in lsst-dm/alert_stream.

            Long term, we need to fix the data. Short term, pin the version of fastavro to one that works in the Dockerfile.
            Perhttps://github.com/lsst-dm/alert_stream/issues/24, the latest release of fastavro is incompatible with the serialized alert data in lsst-dm/alert_stream.

            Long term, we need to fix the data. Short term, pin the version of fastavro to one that works in the Dockerfile.
            swinbank John Swinbank made changes -
            Description Perhttps://github.com/lsst-dm/alert_stream/issues/24, the latest release of fastavro is incompatible with the serialized alert data in lsst-dm/alert_stream.

            Long term, we need to fix the data. Short term, pin the version of fastavro to one that works in the Dockerfile.
            Per https://github.com/lsst-dm/alert_stream/issues/24, the latest release of fastavro is incompatible with the serialized alert data in lsst-dm/alert_stream.

            Long term, we need to fix the data. Short term, pin the version of fastavro to one that works in the Dockerfile.
            Hide
            swinbank John Swinbank added a comment -

            PR: https://github.com/lsst-dm/alert_stream/pull/25

            This is a trivial change, so I propose to merge without further review... but Eric Bellm, I'll give you a few minutes to shout if you disagree before I do so.

            Show
            swinbank John Swinbank added a comment - PR: https://github.com/lsst-dm/alert_stream/pull/25 This is a trivial change, so I propose to merge without further review... but Eric Bellm , I'll give you a few minutes to shout if you disagree before I do so.
            swinbank John Swinbank made changes -
            Reviewers Eric Bellm [ ebellm ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            swinbank John Swinbank made changes -
            Link This issue relates to DM-17551 [ DM-17551 ]
            Hide
            swinbank John Swinbank added a comment -

            The sample alert data should be updated to the new schema on DM-17551.

            Show
            swinbank John Swinbank added a comment - The sample alert data should be updated to the new schema on DM-17551 .
            Hide
            swinbank John Swinbank added a comment -

            Merged.

            Show
            swinbank John Swinbank added a comment - Merged.
            swinbank John Swinbank made changes -
            Resolution Done [ 10000 ]
            Status In Review [ 10004 ] Done [ 10002 ]
            swinbank John Swinbank made changes -
            Epic Link DM-19979 [ 307530 ]

              People

              Assignee:
              swinbank John Swinbank
              Reporter:
              swinbank John Swinbank
              Reviewers:
              Eric Bellm
              Watchers:
              Eric Bellm, John Swinbank
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.