Skip to content

Conversation

@seebi
Copy link
Member

@seebi seebi commented Nov 26, 2025

No description provided.

seebi and others added 15 commits November 26, 2025 16:07
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Signed-off-by: Thomas Telleis <thomas.telleis@eccenca.com>
Copy link
Member

@sobo sobo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

partial review


## Linked Data delivery mode

TODO: What is this section? Still relevant? Also rename to explore?
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we still need this part?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove todo, leave unchanged

- The other client is used as a technical user with the command line interface (typically named `cmem-service-account`).
Depending on the environment, there might be other use cases, when running background schedules, then a third client, also as technical user, might be useful.
- The second client is used as a technical user with the command line interface (typically named `cmem-service-account`).
- (optional, when using Graph Insights) similar to the `cmem` client you can configure Graph Insights to use a separate client (usally name `graph-insights`)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo, named

Depending on the environment, there might be other use cases, when running background schedules, then a third client, also as technical user, might be useful.
- The second client is used as a technical user with the command line interface (typically named `cmem-service-account`).
- (optional, when using Graph Insights) similar to the `cmem` client you can configure Graph Insights to use a separate client (usally name `graph-insights`)
- (optional, when using Graph Insights) similar to the `cmem-service-account` client you can configure Graph Insights to communicate with a separate client (usally name `graph-insights-service-account`)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo named

## Helm based Kubernetes deployment

Note: During the update procedure, the directory `WEB-INF` is created. Due to security concerns the update mechanism does not delete this directory. You can delete it after the update process is finished.
Most production deployments are in Kubernetes cluster.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: clusters


Clients are used to link users and groups managed in Keycloak to Corporate Memory.
There are two different clients used by Corporate Memory:
There are two different type of clients used by Corporate Memory:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: types

Copy link
Contributor

@rpietzsch rpietzsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me.

@rpietzsch rpietzsch merged commit c5cc670 into main Dec 1, 2025
2 checks passed
@rpietzsch rpietzsch deleted the feature/graphInsightsDocu-CMEM-7120 branch December 1, 2025 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants