mark transceiver serial-no and vendor-part as deprecated

[SydneyCaulfeild]

Change Scope

This change is backwards compatible.

This change is marking the components/component/transceiver/state/serial-no and components/component/transceiver/state/vendor-part leaves as deprecated. The components/component/state/serial-no and components/component/state/part-no leaves should be used instead going forward.

These duplicate serial and part number leaves are duplicate/redundant, and the top-level component one is preferred.

Platform Implementations

nvidia has hardware that is using the components/component/transceiver/state/serial-no and components/component/transceiver/state/vendor-part instead of components/component/state/serial-no and components/component/state/part-no.

[dplore]

/gcbrun

[OpenConfigBot]

No major YANG version changes in commit 27e943e

[ElodinLaarz]

/gcbrun

[ElodinLaarz]

You included a single vendor in your initial comment, and usually it helps to have more than one vendor to show neutrality? In this case, it's probably fine since the leaves exist in the model.

You mention a different leaf in your initial comment than the one you list in the description, though? components/component/state/serial-no vs. transceiver/state/serial-no (which I assume is /components/component/transceiver/state/serial-no.)

Do we want to distinguish / add a comment about this vs. /components/component/state/serial-no?

[SydneyCaulfeild]

You included a single vendor in your initial comment, and usually it helps to have more than one vendor to show neutrality? In this case, it's probably fine since the leaves exist in the model.

You mention a different leaf in your initial comment than the one you list in the description, though? components/component/state/serial-no vs. transceiver/state/serial-no (which I assume is /components/component/transceiver/state/serial-no.)

Do we want to distinguish / add a comment about this vs. /components/component/state/serial-no?

I'm only aware of this one vendor, but I figured since this involves marking something as deprecating and not actually yet removing anything that this would be ok.

@ElodinLaarz Could you please elaborate on your concern on distinguishing the two serial-no leaves? Do you mean add a note to /components/component/transceiver/state/serial-no in the .yang to explain that it is just a duplicate of components/component/state/serial-no? Or cleaning up the PR description?