Skip to content

[vs] Skip MACsec clean up if /sbin/ip is not accessible#750

Merged
lguohan merged 1 commit intosonic-net:masterfrom
kcudnik:clean
Dec 17, 2020
Merged

[vs] Skip MACsec clean up if /sbin/ip is not accessible#750
lguohan merged 1 commit intosonic-net:masterfrom
kcudnik:clean

Conversation

@kcudnik
Copy link
Copy Markdown
Collaborator

@kcudnik kcudnik commented Dec 16, 2020

Some build containers for sonic-buildimage may not have ip command, and for non MACsec test's this is not required to be present.

@kcudnik
Copy link
Copy Markdown
Collaborator Author

kcudnik commented Dec 16, 2020

@Pterosaur This is just temporary solution here to make things going on sonic buildimage, since you designed this, it would be best if you will handle proper fix for that.

@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Dec 16, 2020

retest this please

@kcudnik
Copy link
Copy Markdown
Collaborator Author

kcudnik commented Dec 16, 2020

retest this please

2 similar comments
@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Dec 17, 2020

retest this please

@kcudnik
Copy link
Copy Markdown
Collaborator Author

kcudnik commented Dec 17, 2020

retest this please

@lguohan lguohan merged commit a43bf56 into sonic-net:master Dec 17, 2020
@kcudnik kcudnik deleted the clean branch December 17, 2020 17:22
kktheballer pushed a commit to kktheballer/sonic-sairedis that referenced this pull request Mar 10, 2021
Some build containers for sonic-buildimage may not have ip command, and for non MACsec test's this is not required to be present.
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
Some build containers for sonic-buildimage may not have ip command, and for non MACsec test's this is not required to be present.
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