Rework the pull status and subscription notification to remove the top
level instance tags that were returned with the notification. These tags
were originally introduced to identify which PTP instance a given
notification was associated with. Updates to the ORAN standard have
clarified that the instance identification should instead be included in
the Resource Address field.
This commit includes the following changes:
- Update notification format to place instance identifier in Resource
Address instead of a top level tag (default behaviour)
- Provide a helm override to enable legacy notification format with top
level instance tags for compatibility. This can be enabled by setting
the override "notification_format=legacy"
- Ensure GM clockClass is correctly reported for both BC and GM configs
- Changes result in new images for notificationclient-base and
notificationservice-base-v2
- Pull status notifications are now delivered as a list of dicts when
there are multiple instances in a request. The "overall sync status"
message is delivered as a dict because it only ever has a single
notification
- Push notifications are delivered as individual dicts for each
notification, this behaviour remains the same
Test plan:
Pass: Verify ptp-notification builds and deploys
Pass: Verify container image builds
Pass: Verify ptp-notification operations (pull status, subscribe, list,
delete)
Pass: Verify standard and legacy format output for each notification
type
Partial-bug: 2089035
Signed-off-by: Cole Walker <cole.walker@windriver.com>
Change-Id: Ied6674a02b41ed31079a291fc9bace74d95d467a
bbeef95744