Skip to content

Route endpoints Egeria removed to their replacements, deprecated (ISSUE-165) - #485

Merged
dwolfson merged 2 commits into
odpi:mainfrom
dwolfson:fix/issue-165-replacements
Oct 24, 2026
Merged

dwolfson merged 2 commits into
odpi:mainfrom
dwolfson:fix/issue-165-replacements

Conversation

@dwolfson

Copy link
Copy Markdown
Member

Summary

6.2.10 documented 17 SDK methods that call endpoints Egeria no longer serves (ISSUE-165). The Egeria team (Mandy Chessell) named the replacements. Each obsolete method is now a deprecated wrapper (DeprecationWarning) that delegates to its replacement, following the ISSUE-153 pattern, so existing callers keep working. Removal is planned for a later release.

Obsolete Now delegates to
DataDiscovery analysis-report create/update/delete/get/find (8 methods) AssetMaker asset calls with type SurveyReport (SurveyReport now inherits from Report). A body naming AnalysisReportProperties is rewritten to SurveyReportProperties.
link/detach_analysis_report_to/from_asset AssetMaker.link_report_subject / unlink_report_subject (ReportSubject)
ClassificationExplorer.add/remove_element_to/from_subject_area CollectionManager membership (SubjectArea is now a Collection). New optional subject_area_guid; without it the call raises an error naming the replacement.
GovernanceOfficer.find_governance_action_processes find_governance_definitions(metadata_element_type="GovernanceActionProcess")
GovernanceOfficer.get_gov_def_in_context get_governance_definition_by_guid, which returns the context by default. Dr.Egeria's View Governance Definition Context now calls it directly.
DataDesigner data-classification create/update/link/detach data value specification create/update and link/detach_data_value_assignment. Best-effort mapping: Mandy expected this but wasn't certain.

Still open: get_platform_organization (removed upstream in 816076ab50; no replacement named).

Test plan

  • Micro-tests: 1,483 pass. New: tests/micro-tests/test_issue165_replacements.py (8). DataDiscovery entries were removed from two generic by-name tests, because those methods now delegate to AssetMaker.
  • omvs_audit.py: JAVA-MISSING is down to 4: get_platform_organization and 3 known false positives.
  • Live on quickstart 9443 (Egeria 81beb8e5, freshly reset), one peer-cleared run with throwaway pyeg-test::issue165:: elements; all 5 deleted and read back as NotFound:
    • Process search returns GovernanceActionProcess and its subtypes. The context lookup equals the by-GUID result, context included.
    • Subject-area add without a GUID raises the guidance error. With a GUID, the term is a member after add and gone after remove.
    • The analysis report is created as a SurveyReport and found by get and find. ReportSubject is present after link and gone after detach.
    • The data classification is created as a DataClass. It is in the field's assignedDataValueSpecifications after link and gone after detach.

Details: PYEGERIA_ISSUES.md ISSUE-165.

🤖 Generated with Claude Code

…UE-165)

Per the Egeria team (Mandy Chessell, 2026-10-23), each obsolete method now
warns (DeprecationWarning) and delegates, following the ISSUE-153 pattern:
- analysis reports -> SurveyReport assets via AssetMaker (SurveyReport now
  inherits from Report); link/detach -> ReportSubject (link/unlink_report_subject).
- add/remove_element_to/from_subject_area -> CollectionManager membership on
  the subject area (SubjectArea now inherits from Collection); new optional
  subject_area_guid, without which they raise an error naming the replacement.
- find_governance_action_processes -> find_governance_definitions for type
  GovernanceActionProcess (now a GovernanceDefinition).
- get_gov_def_in_context -> get_governance_definition_by_guid (context is
  returned by default); Dr.Egeria View Governance Definition Context too.
- data classifications -> data value specifications and DataValueAssignment
  (a best-effort mapping; the Egeria team was not certain).

8 micro-tests; two generic by-name tests no longer list the delegating
DataDiscovery wrapper (its target is listed). Not verified live: Egeria was
being reset.

Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>
Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>
@dwolfson
dwolfson merged commit 2d0ec8c into odpi:main Oct 24, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant