Upstreamsvcs

[chrisadubois]

COMPLETES #< INSERT LINK TO ISSUE >

This pull request addresses

Making energy forecast data an option for the WDM register and refresh calls

all doesn't change the current request, it is the default as we have today, we can use it to expand and request only certain parts from WDM from upstream services in the future.

https://confluence-eng-gpk2.cisco.com/conf/pages/viewpage.action?spaceKey=WBXT&title=Making+WDM+Upstream+Requests+optional+for+clients

Change Type

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update
• Tooling change
• Internal code refactor

The following scenarios where tested

manual and unit tests

< ENUMERATE TESTS PERFORMED, WHETHER MANUAL OR AUTOMATED >

I certified that

• I have read and followed contributing guidelines
• I discussed changes with code owners prior to submitting this pull request
• I have not skipped any automated checks
• All existing and new tests passed
• I have updated the documentation accordingly

---

Make sure to have followed the contributing guidelines before submitting.

[aws-amplify-us-east-2]

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-3832.d3m3l2kee0btzx.amplifyapp.com

[sreenara]

all doesn't change the current request, it is the default as we have today, we can use it to expand and request only certain parts from WDM from upstream services in the future.

@chrisadubois the SDK doesn't send IncludeUpstreamService right now while doing WDM registration. Why are we adding this considering the link between WDM and U2C is going to be removed in the future. Reference

[sreenara]

Just one comment regarding the decision to add includeUpstreamServices

[chrisadubois]

@sreenara - that's actually exactly what this helps with for the future. upstream services are not limited to just the catalog. The default is all. Once the catalog is removed, that is fine, there are still settings, toggles, etc which are ALSO upstream services. In fact, we could build on this after this PR and remove the u2c reference right away via this query parameter and not include the catalog in the WDM response.

[chrisadubois]

We are setting it to all because today, that is existing functionality. Next step would be to provide the list of upstream services sans u2c/catalog

[chrisadubois]

@sreenara please also have a look at this wiki: https://confluence-eng-gpk2.cisco.com/conf/pages/viewpage.action?spaceKey=WBXT&title=Making+WDM+Upstream+Requests+optional+for+clients