Skip to content
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

DHCP_RELAY: use config interface instead of ifconfig #16648

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

illia-kotvitskyi
Copy link
Contributor

use config interface command instead of ifconfig to shut down/bring up the interface

the motivation to make the command compatible with interfaces, not only portchannels

Change-Id: I09dad3c7283787c3cb8ceea36ef64608342df0ba

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

the motivation is to make the command compatible with interfaces, not only portchannels

How did you do it?

How did you verify/test it?

Ran test_dhcp_relay_after_link_flap on t0-128 topology

Any platform specific information?

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

Documentation

…p the interface

the motivation to make the command compatible with interfaces, not only portchannels

Change-Id: I09dad3c7283787c3cb8ceea36ef64608342df0ba
@mssonicbld
Copy link
Collaborator

/azp run

@illia-kotvitskyi illia-kotvitskyi changed the title DCHP_RELAY: use config interface instead of ifconfig DHCP_RELAY: use config interface instead of ifconfig Jan 23, 2025
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@illia-kotvitskyi
Copy link
Contributor Author

@roy-sror @nhe-NV, could you please review and approve this?

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.

3 participants