OpenConfig Platform Transceiver

[nakano-omw]

OpenConfig support for Platform Transceiver.

sonic-net/SONiC#1858

Add support for following features:

• serial-no
• connector-type
• vendor
• vendor-part
• date-code
• supply-voltage
• laser-temperature
• output-power
• input-power
• laser-bias-current

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

[mssonicbld]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[nakano-omw]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 160 in repo sonic-net/sonic-mgmt-common

[nakano-omw]

/azp run Azure.sonic-mgmt-common

[azure-pipelines]

Commenter does not have sufficient privileges for PR 160 in repo sonic-net/sonic-mgmt-common

[nakano-omw]

/azpw run Azure.sonic-mgmt-common\

[mssonicbld]

/AzurePipelines run Azure.sonic-mgmt-common\

[azure-pipelines]

No pipelines are associated with this pull request.

[hdwhdw]

@judyjoseph @vvolam are you the right (platform) person to review this?

[vvolam]

@hdwhdw @prgeor is the right person for xcvr.

[anders-nexthop]

Somehow each file in the review is showing up as being modified, even though there appears to be no difference to much of the file. Has the whitespace been changed perhaps? Can you figure out what was different and revert it, so that only actual changes show up in the diff here? It's really difficult to review as it is now, as there's no way to see what has the actual changes are.

[nakano-omw]

@anders-nexthop Thanks for your feedback. It seems the whitespace or encoding differences caused unnecessary diffs. I’ll recreate the PR with only the actual changes — sorry for the trouble.

[nakano-omw]

Hi @anders-nexthop,
I’ve created a new PR (#197) with the whitespace issues fixed so that only the actual changes appear in the diff. Could you please review it when you have a moment? Thanks again for pointing it out.