Skip to content

Commit 4caf235

Browse files
authored
docs(PYEGERIA_ISSUES): ISSUE-84 live-verified by dwolfson-1b/trellis (#339)
* docs(PYEGERIA_ISSUES): ISSUE-84 live-verified by dwolfson-1b/trellis contentStatus: DRAFT inside properties on NewElementRequestBody confirmed working end-to-end against a live Egeria server (quickstart-egeria-main, once recovered from an unrelated Postgres pool exhaustion): creates without error, round-trips correctly on read-back, elementHeader.status stays ACTIVE as expected (different axis). BlueprintMaterializer upgraded to use it; ComponentMaterializer's identical gap tracked separately, still open on the trellis side. Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com> * docs(PYEGERIA_ISSUES): ISSUE-84 -- add materialized guid from live verification dwolfson-1b/trellis confirmed the real-application-path verification with a concrete guid, independently queried back to confirm contentStatus persisted as DRAFT. Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com> --------- Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>
1 parent 0b064fe commit 4caf235

1 file changed

Lines changed: 27 additions & 8 deletions

File tree

‎PYEGERIA_ISSUES.md‎

Lines changed: 27 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -631,9 +631,13 @@ into the entry now, so it isn't rediscovered from scratch later.
631631

632632
### ISSUE-84: `SolutionArchitect.create_solution_blueprint`'s own docstring documents a `NewSolutionElementRequestBody` body (with `initialStatus`) for Draft-status creation — that class does not exist as a pydantic model, so the documented shape fails client-side validation before any HTTP call
633633

634-
**Status:** fixed 2026-09-03 — docstring-only fix (see "Fix landed" below).
635-
Reported by a consumer (trellis/Resource Explorer's `BlueprintMaterializer`,
636-
`docs/blueprint-materialization-plan.md` Phase A).
634+
**Status:** fixed and live-verified 2026-09-03 — docstring-only fix (see
635+
"Fix landed" below), merged upstream via
636+
[PR #338](https://github.com/odpi/egeria-python/pull/338). Reported by a
637+
consumer (trellis/Resource Explorer's `BlueprintMaterializer`,
638+
`docs/blueprint-materialization-plan.md` Phase A), who also live-verified
639+
it once their shared `quickstart-egeria-main` instance recovered from an
640+
unrelated Postgres connection-pool exhaustion (`jdbcMaximumPoolSize` bump).
637641

638642
**Confirmed in both**: this checkout's installed pyegeria (5.3.4.23, via
639643
the trellis venv) and this repo's own working tree (6.1.9-dev) —
@@ -770,11 +774,26 @@ still fails as expected (proving the understanding of the original bug is
770774
correct), and the corrected shape
771775
(`{"class": "NewElementRequestBody", "properties": {...,
772776
"contentStatus": "DRAFT"}}`) validates cleanly with `contentStatus`
773-
preserved intact through to the serialized JSON. **Not live-verified
774-
against a real Egeria server** — `dwolfson-1b`/trellis has live access;
775-
recommend they confirm `contentStatus: "DRAFT"` at creation actually lands
776-
as the blueprint/component's status server-side before switching
777-
`BlueprintMaterializer` off its ACTIVE-only workaround.
777+
preserved intact through to the serialized JSON.
778+
779+
**Live-verified 2026-09-03 by `dwolfson-1b`/trellis**, once their shared
780+
`quickstart-egeria-main` instance recovered from unrelated Postgres pool
781+
exhaustion: `contentStatus: "DRAFT"` inside `properties` on
782+
`NewElementRequestBody` creates without error and round-trips correctly on
783+
read-back (`properties.contentStatus == "DRAFT"`), no separate request-body
784+
class needed — confirmed twice, once as a standalone probe and once through
785+
Resource Explorer's actual application path (accepting a real candidate
786+
blueprint end-to-end via the Curate UI's API, materialized element guid
787+
`809025b5-cca9-4e9a-a2f7-3a5104138f67`, independently queried back by that
788+
guid to confirm). `elementHeader.status` stays `ACTIVE` regardless, as
789+
expected — a different axis (header status vs. `contentStatus`, distinct
790+
concepts in Egeria's status model), not a sign the fix is incomplete.
791+
792+
`BlueprintMaterializer.materialize_blueprint_element` has been upgraded to
793+
send `contentStatus: "DRAFT"` in `properties`, replacing its earlier
794+
ACTIVE-only workaround. `ComponentMaterializer`'s identical gap is
795+
**still open** — same fix applies, not yet applied there, tracked
796+
separately on the trellis/Resource Explorer side.
778797

779798
### ISSUE-82: `pyegeria/omvs/valid_metadata.py` sends the literal query string `typeName=None` whenever `type_name` is Python `None` — breaks every Type-Name-omitted (global) Valid Metadata Value, in 12 of 14 methods across `ValidMetadataManager`
780799

0 commit comments

Comments
 (0)