Skip to content

[backport for 11.0-stable] Fix an issue in Edgeview not getting App i… #4818

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: 11.0-stable
Choose a base branch
from

Conversation

naiming-zededa
Copy link
Contributor

@naiming-zededa naiming-zededa commented May 12, 2025

…nterface IP address

Description

port the PR #4352 to 11.0-stable

PR dependencies

How to test and validate this PR

Need to configure an app on the edge-node, enable edgeview, and use the
tcp command such as tcp/{app-ipaddr}:22 to verify if this is ok.

Testing Done

tested this patch on 11.0-stable, with edgeview, and application.
was able to edgeview using tcp/{app-ip-address}:22 to ssh into the app.

Changelog notes

[backport to 11.0-stable] Fix an issue in Edgeview not getting App interface IP address

PR Backports

Checklist

  • [ x ] I've provided a proper description
  • I've added the proper documentation (when applicable)
  • [ x ] I've tested my PR on amd64 device(s)
  • I've tested my PR on arm64 device(s)
  • I've written the test verification instructions
  • I've set the proper labels to this PR

…nterface IP address

- this is backport the PR lf-edge#4352 to 11.0-stable
- didn't do the cherry-pick, since it would cause massive vendor file
  conflicts.
- due to the change of multiple port in zedrouter, the
  AppNetAdapterList.AllocatedIPv4Addr was changed to AssignedAddresses,
  do the same change

Signed-off-by: naiming-zededa <[email protected]>
@naiming-zededa naiming-zededa force-pushed the naiming-11.0-stable-port4352 branch from add783b to 5a8b272 Compare May 12, 2025 22:49
@OhmSpectator
Copy link
Member

11.0 is end of support, we do not need to backport there

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.

4 participants