Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

standard models - asset consumer more-information moved from common to framework services #52

Closed
planetf1 opened this issue Mar 16, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@planetf1
Copy link
Member

planetf1 commented Mar 16, 2023

In cell #6 of the 'working with standard models' notebook, we have a call to :

# Get the link to the top-level category
getGlossaryMoreInfoURLRoot = cocoMDS2PlatformURL + '/servers/' + cocoMDS2Name + '/open-metadata/common-services/asset-consumer/connected-asset/users/' +  erinsUserId + '/referenceables/'
getGlossaryMoreInfoURLTail = '/more-information?elementStart=0&maxElements=50'

However this fails with a 404 as this endpoint has now changed to


# Get the link to the top-level category
getGlossaryMoreInfoURLRoot = cocoMDS2PlatformURL + '/servers/' + cocoMDS2Name + '/open-metadata/framework-services/asset-consumer/connected-asset/users/' +  erinsUserId + '/referenceables/'
getGlossaryMoreInfoURLTail = '/more-information?elementStart=0&maxElements=50'

ie for this api call common-services -> framework-services in Egeria 4.0

@planetf1
Copy link
Member Author

There are many other calls to connected-asset especially in common-functions, which all of the notebooks except this one use. These still refer to common-services

@planetf1
Copy link
Member Author

Automated curation also has the wrong url

"    metadataStoreURL = serverPlatformURL + '/servers/' + serverName + '/open-metadata/common-services/' + serviceURLMarker + '/open-metadata-store/users/' + userId \n",

As open-metadata-store is now in the framework services too

@planetf1 planetf1 added the bug Something isn't working label Mar 16, 2023
@dwolfson
Copy link
Member

I submitted a PR with proposed fixes.

@planetf1
Copy link
Member Author

this is now complete & merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Archived in project
Development

No branches or pull requests

2 participants