Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[16.0] Use Pubsub Large to handle large KubeClusterInfo structures.#5347

Merged
rene merged 1 commit into
lf-edge:16.0from
andrewd-zededa:16.0-backport-5336
Nov 5, 2025
Merged

[16.0] Use Pubsub Large to handle large KubeClusterInfo structures.#5347
rene merged 1 commit into
lf-edge:16.0from
andrewd-zededa:16.0-backport-5336

Conversation

@andrewd-zededa

@andrewd-zededa andrewd-zededa commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

Description

Pods, VMIs, Volumes, and Replicas have the potential to grow this structure past the 64k pubsub limit.

(cherry picked from commit 58ce32e)

Backport of #5336

PR dependencies

None

How to test and validate this PR

  • deploy at least one hv=k node
  • deploy a large number of volume instances, they can be empty
  • Verify /run/zedkube/KubeClusterInfo/global.json is over 65K

Changelog notes

None

PR Backports

This is the backport.

Checklist

  • I've provided a proper description
  • I've added the proper documentation
  • I've tested my PR on amd64 device
  • I've tested my PR on arm64 device
  • I've written the test verification instructions
  • I've set the proper labels to this PR

For backport PRs (remove it if it's not a backport):

  • I've added a reference link to the original PR
  • PR's title follows the template

And the last but not least:

  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

Please, check the boxes above after submitting the PR in interactive mode.

Pods, VMIs, Volumes, and Replicas have the potential to grow this
structure past the 64k pubsub limit.

Signed-off-by: Andrew Durbin <[email protected]>
(cherry picked from commit 58ce32e)

@eriknordmark eriknordmark left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@rene rene merged commit 048d9d0 into lf-edge:16.0 Nov 5, 2025
44 of 45 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.

3 participants