local-clock-identity duplicates

openBackhaul / synchronization

Synchronization model according to ITU-T G.7721-2018

Apache License 2.0

1 stars 0 forks source link

local-clock-identity duplicates #52

Open michbin opened 1 year ago

michbin commented 1 year ago

The following attributes seem to be duplicates:

ptp-sync-pac/ptp-local-clock-identity
ptp-sync-pac/ptp-default-dataset/local-clock-identity

openBackhaul commented 1 year ago

ptp-sync-pac/ptp-local-clock-identity:ClockIdentity [1] = 00:00:00:00:00:00:00:00 with comment "Identifier of the “physical” clock"
ptp-sync-pac/ptp-default-dataset/local-clock-identity:ClockIdentity [1] = 00:00:00:00:00:00:00:00 with comment "Identifier of the “physical” local clock."

If two different attributes, respective purpose should be explicitly described in the comments.
Otherwise ptp-sync-pac/ptp-default-dataset/local-clock-identity should be deleted.

demx8as6 commented 1 year ago

Feedback: agree with Michael comments that attributes are redundant. Sync #52: We also think it is duplicated. But we don’t have opinion on which attribute to delete.

demx8as6 commented 1 year ago

Decision during the 5Gxhaul call on 19th of July 2023:

The ptp-sync-pac/ptp-default-dataset/local-clock-identity should be deleted from the SyncModel.