open-traffic-generator / snappi

Open Traffic Generator SDK in Python and Go
MIT License
68 stars 7 forks source link

GoSnappi: proto extension number collision with usprotocol #122

Open rudranil-das opened 2 years ago

rudranil-das commented 2 years ago

When a subscriber uses both gosnappi and usprotocol (ixia-c), following warnings are observed repeatedly

2021/10/18 06:44:53 WARNING: proto: extension number 60001 is already registered on message google.protobuf.MessageOptions
        previously from: "github.com/open-traffic-generator/snappi/gosnappi/snappipb"
        currently from:  "keysight/athena/controller/pkg/stubs/usprotocol"
A future release will panic on registration conflicts. See:
https://developers.google.com/protocol-buffers/docs/reference/go/faq#namespace-conflict

2021/10/18 06:44:53 WARNING: proto: extension number 60002 is already registered on message google.protobuf.FieldOptions
        previously from: "github.com/open-traffic-generator/snappi/gosnappi/snappipb"
        currently from:  "keysight/athena/controller/pkg/stubs/usprotocol"
A future release will panic on registration conflicts. See:
https://developers.google.com/protocol-buffers/docs/reference/go/faq#namespace-conflict

2021/10/18 06:44:53 WARNING: proto: extension number 60003 is already registered on message google.protobuf.ServiceOptions
        previously from: "github.com/open-traffic-generator/snappi/gosnappi/snappipb"
        currently from:  "keysight/athena/controller/pkg/stubs/usprotocol"
A future release will panic on registration conflicts. See:
https://developers.google.com/protocol-buffers/docs/reference/go/faq#namespace-conflict

2021/10/18 06:44:53 WARNING: proto: extension number 60004 is already registered on message google.protobuf.MethodOptions
        previously from: "github.com/open-traffic-generator/snappi/gosnappi/snappipb"
        currently from:  "keysight/athena/controller/pkg/stubs/usprotocol"
A future release will panic on registration conflicts. See:
https://developers.google.com/protocol-buffers/docs/reference/go/faq#namespace-conflict
ashutshkumr commented 2 years ago

@rudranil-das controller will need to stop using usprotocol at some point as the protobuf stubs shall now be available via gosnappi.

That should resolve this warning. Please close the issue once resolved.