Skip to content

N°9604 - add "flowmaps" as a new neighbour#914

Merged
v-dumas merged 7 commits into
developfrom
feature/9604-FlowMap
May 21, 2026
Merged

N°9604 - add "flowmaps" as a new neighbour#914
v-dumas merged 7 commits into
developfrom
feature/9604-FlowMap

Conversation

@v-dumas
Copy link
Copy Markdown
Contributor

@v-dumas v-dumas commented May 20, 2026

Base information

Question Answer
Related to Another PR
Type of change? Enhancement

Objective (enhancement)

Cf Bug N°9604 - List dataflow with impacted CIs in the Tickets.

Proposed solution (bug and enhancement)

Allow be able to create a Ticket from a DataFlow, to report an Incident, request a evolution or plan a Change
Allow to see the impacted DataFlow from a Ticket, not only in the impact analysis graph but linked to the Ticket somewhere.
Allow DataFlow to be used within a new neighborhood relation "dataflows" to display Flow maps (The display of those Flow maps will be improved in a later iTop version)

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have tested all changes I made on an iTop instance
  • I have added a unit test, otherwise I have explained why I couldn't
  • Is the PR clear and detailed enough so anyone can understand without digging in the code?

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 20, 2026

Greptile Summary

This PR adds DataFlow objects to the CMDB relation model and UI. It changes:

  • DataFlow now extends FunctionalCI and can participate in CI neighbourhoods.
  • Source and destination filters prevent selecting another DataFlow in the UI.
  • New dataflows relations and dashboard entries expose inbound and outbound flows.
  • DataFlowType is added as a typology with dictionary labels.
  • Container and flow-map dashboard badge identifiers are normalized.

Confidence Score: 4/5

This is close, but the summary attribute typo should be fixed before merging.

  • DataFlow summary rendering can fail on the new frequency item.

  • The dictionary changes in the latest push are text-only.

  • The other summary items checked here point to real DataFlow or inherited FunctionalCI attributes.

  • datamodels/2.x/itop-flow-map/datamodel.itop-flow-map.xml

Important Files Changed

Filename Overview
datamodels/2.x/itop-flow-map/datamodel.itop-flow-map.xml Defines the DataFlow inheritance, validation, presentations, and neighbourhood relations; the summary zlist contains one invalid attribute code.

Reviews (3): Last reviewed commit: "N°9604 - Improve labels and summary" | Re-trigger Greptile

Comment thread datamodels/2.x/itop-flow-map/datamodel.itop-flow-map.xml
Comment thread datamodels/2.x/itop-flow-map/datamodel.itop-flow-map.xml Outdated
Comment thread datamodels/2.x/itop-flow-map/datamodel.itop-flow-map.xml Outdated
Comment thread datamodels/2.x/itop-flow-map/datamodel.itop-flow-map.xml Outdated
Comment thread datamodels/2.x/itop-flow-map/datamodel.itop-flow-map.xml
@v-dumas
Copy link
Copy Markdown
Contributor Author

v-dumas commented May 20, 2026

@greptile

Comment thread datamodels/2.x/itop-flow-map/datamodel.itop-flow-map.xml Outdated
v-dumas and others added 2 commits May 20, 2026 18:22
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@v-dumas
Copy link
Copy Markdown
Contributor Author

v-dumas commented May 21, 2026

@greptile

Comment thread datamodels/2.x/itop-flow-map/datamodel.itop-flow-map.xml Outdated
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@v-dumas v-dumas merged commit ba2af7e into develop May 21, 2026
@v-dumas v-dumas deleted the feature/9604-FlowMap branch May 21, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Work made by Combodo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants