feat(traffic-graph): add trafficType prop to TrafficGraphWithDaySelect#25235
Merged
laura-b-g merged 7 commits intoinputLicensefrom Mar 31, 2026
Merged
feat(traffic-graph): add trafficType prop to TrafficGraphWithDaySelect#25235laura-b-g merged 7 commits intoinputLicensefrom
laura-b-g merged 7 commits intoinputLicensefrom
Conversation
…tinction and update related tests
Update the License plugin type definition to use new component names: - EnterpriseTrafficGraph → TrafficGraph - LicenseGraphWithMetrics → TrafficGraphWithLicenseMetrics Update GraylogClusterOverview.tsx to reference the new plugin export names. These changes mirror the enterprise plugin refactoring that renames the components to better reflect their responsibility and unifies draw-down behavior across all pages. Validation: GraylogClusterOverview tests passing, no type errors. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
AntonEbel
added a commit
that referenced
this pull request
Apr 8, 2026
* add index mapping for gl2_input_message_size field * add inputMessageSize field to JournalMessages * feat(traffic-graph): add trafficType prop to TrafficGraphWithDaySelect (#25235) * Add input-indexed traffic counter for license accounting (#25501) --------- Co-authored-by: Laura Bergenthal-Grotlüschen <197286649+laura-b-g@users.noreply.github.com> Co-authored-by: Florian Petersen <188503754+fpetersen-gl@users.noreply.github.com> Co-authored-by: Maxwell Anipah <maxwell.anipah@graylog.com> Co-authored-by: Anton Ebel <anton.ebel@graylog.com> Co-authored-by: Ismail Belkacim <ismail.belkacim@graylog.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
closes https://github.com/Graylog2/graylog-plugin-enterprise/issues/13243
/nocl
/jpd Graylog2/graylog-plugin-enterprise#13438
Add a
trafficTypeprop ('input' | 'output') toTrafficGraphWithDaySelectso callers can control which traffic direction is displayed. The component derives its title and telemetry section from this prop, defaulting to'output'for backward compatibility.The cluster overview graph is switched to show input traffic. Enterprise plugin type definitions are updated to match component renames in the companion PR.
Motivation and Context
The traffic graph previously hardcoded output traffic. Different license types measure usage against different directions, so the component needs to be configurable. The companion enterprise PR (
/jpd) uses this to show the correct graph per license type.How can this be tested?
/jpdenterprise PR must be applied togetherTrafficGraphWithDaySelect.test.tsxexpanded to cover both traffic types;GraylogClusterOverview.test.tsxupdatedScreenshots (if appropriate):
Types of changes
Checklist: