Skip to content

[action] [PR:18659] Update cEOS configuration to use next-hop-self#19086

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/18659
Jun 19, 2025
Merged

[action] [PR:18659] Update cEOS configuration to use next-hop-self#19086
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/18659

Conversation

@mssonicbld
Copy link
Collaborator

Description of PR

Summary: Force the cEOS instances to use their own IP for next-hop
Fixes #18658

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
  • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

Fixing a bug introduced by using a newer version of cEOS for testing. While the current documentation doesn't specify a newer version of cEOS, it would be good to have this fix in place before people start using a newer version of cEOS.

How did you do it?

Adjust the cEOS configuration templates to force cEOS to rewrite the next-hop as itself

How did you verify/test it?

Normal sonic-mgmt test run

Any platform specific information?

Not platform-dependent.

Supported testbed topology if it's a new test case?

All

Documentation

Just a bugfix.

The upgraded cEOS instance also changes from single-agent to multi-agent mode.  This includes a behaviour change around the next-hop-self feature.

Fixes: BUG872880

What is the motivation for this PR?
Fixing a bug introduced by using a newer version of cEOS for testing. While the current documentation doesn't specify a newer version of cEOS, it would be good to have this fix in place before people start using a newer version of cEOS.

How did you do it?
Adjust the cEOS configuration templates to force cEOS to rewrite the next-hop as itself

How did you verify/test it?
Normal sonic-mgmt test run

Any platform specific information?
Not platform-dependent.

Supported testbed topology if it's a new test case?
All
@mssonicbld
Copy link
Collaborator Author

Original PR: #18659

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 12ccdfd into sonic-net:202505 Jun 19, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants