Uploaded image for project: 'VOLTHA'
  1. VOLTHA
  2. VOL-5190

voltha-protos version should be consistent across all repositories

    XMLWordPrintable

    Details

    • Story Points:
      0

      Description

      We have a mix of voltha-protos version in use across repositories.

      Problem can be corrected at release time but this is too far downstream, problems can manifest and will surface at the worst possible time:

       

      ./voltha-lib-go/go.mod:	github.com/opencord/voltha-protos/v5 v5.4.6
      ./voltha-lib-go/go.mod:replace github.com/opencord/voltha-protos/v5 => github.com/opencord/voltha-protos/v5 v5.4.7
      ./voltha-eponolt-adapter/go.mod:	github.com/opencord/voltha-protos/v3 v3.4.1
      ./bbsim/go.mod:	github.com/opencord/voltha-protos/v5 v5.2.3
      ./voltha-epononu-adapter/go.mod:	github.com/opencord/voltha-protos/v3 v3.3.9
      ./voltha-go-controller/go.mod:	github.com/opencord/voltha-protos/v5 v5.2.4
      ./voltha-api-server/vendor/github.com/opencord/voltha-protos/v2/go.mod:module github.com/opencord/voltha-protos/v2
      ./voltha-api-server/go.mod:	github.com/opencord/voltha-protos/v2 v2.0.1
      ./voltha-openolt-adapter/go.mod:	github.com/opencord/voltha-protos/v5 v5.4.9
      ./voltha-protos/go.mod:module github.com/opencord/voltha-protos/v5
      ./voltha-go/go.mod:	github.com/opencord/voltha-protos/v5 v5.4.9
      ./voltha-openonu-adapter-go/go.mod:	github.com/opencord/voltha-protos/v5 v5.4.9
      ./voltha-northbound-bbf-adapter/go.mod:	github.com/opencord/voltha-protos/v5 v5.2.

       

      Changes needed:

      • Update all proto strings to v5.4.10 (latest release for v2.12)
      • Create a "make lint" target that will infer latest version from git and err when discrepancies are detected.
      • Afterwards wire the voltha-protos version check into JJB pipelines so repositories can be kept up-to-date.

        Attachments

        # Subject Branch Project Status CR V

          Activity

            People

            Assignee:
            joey Joey Armstrong
            Reporter:
            joey Joey Armstrong
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: