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

Skip to content

[16.0] Fix a bug where edgeview tcp/kube set port number incorrectly#5334

Merged
eriknordmark merged 1 commit into
lf-edge:16.0from
naiming-zededa:naiming-port-5324-16.0
Oct 30, 2025
Merged

[16.0] Fix a bug where edgeview tcp/kube set port number incorrectly#5334
eriknordmark merged 1 commit into
lf-edge:16.0from
naiming-zededa:naiming-port-5324-16.0

Conversation

@naiming-zededa

Copy link
Copy Markdown
Contributor

Description

  • when edgeview is run in single instance, the instance number in server is not set explicitly. this breaks the tcp/kube when it will not get the TCP port number correctly
  • backport PR# 5324

(cherry picked from commit 4a3aede)

PR dependencies

How to test and validate this PR

Configure the Edgeview as a single instance for the project/device, and run the eve-k image, start the edgeview.sh
with 'tcp/kube' option, and use 'kubectl --kubeonconfig=/tmp/download/kube-config.yaml get node' to make
sure this is successful.

Changelog notes

Fix a bug where edgeview tcp/kube set port number incorrectly

PR Backports

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.

- when edgeview is run in single instance, the instance number in server
  is not set explicitly. this breaks the tcp/kube when it will not get
  the TCP port number correctly
- backport PR# 5324

Signed-off-by: naiming-zededa <[email protected]>
(cherry picked from commit 4a3aede)

@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

@eriknordmark eriknordmark merged commit e75b5c7 into lf-edge:16.0 Oct 30, 2025
56 of 61 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.

2 participants