Skip to content

Revert "add if statement for module control mode support"#1341

Merged
kcudnik merged 1 commit intosonic-net:masterfrom
dbarashinvd:revert-1305-dbarashi_indep_mode
Jan 29, 2024
Merged

Revert "add if statement for module control mode support"#1341
kcudnik merged 1 commit intosonic-net:masterfrom
dbarashinvd:revert-1305-dbarashi_indep_mode

Conversation

@dbarashinvd
Copy link
Copy Markdown
Contributor

@dbarashinvd dbarashinvd commented Jan 23, 2024

Reverts #1305

this logic is not needed. we'll change sai.profile directly

@keboliu
Copy link
Copy Markdown
Collaborator

keboliu commented Jan 29, 2024

@kcudnik would you please help to merge?

@kcudnik kcudnik merged commit 5b2a517 into sonic-net:master Jan 29, 2024
mssonicbld pushed a commit to mssonicbld/sonic-sairedis that referenced this pull request Jan 31, 2024
…1341)

Reverts sonic-net#1305

this logic is not needed. we'll change sai.profile directly
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202311: #1346

@mssonicbld
Copy link
Copy Markdown
Collaborator

@dbarashinvd cherry pick PR didn't pass PR checker. Please check!!!
#1346

6 similar comments
@mssonicbld
Copy link
Copy Markdown
Collaborator

@dbarashinvd cherry pick PR didn't pass PR checker. Please check!!!
#1346

@mssonicbld
Copy link
Copy Markdown
Collaborator

@dbarashinvd cherry pick PR didn't pass PR checker. Please check!!!
#1346

@mssonicbld
Copy link
Copy Markdown
Collaborator

@dbarashinvd cherry pick PR didn't pass PR checker. Please check!!!
#1346

@mssonicbld
Copy link
Copy Markdown
Collaborator

@dbarashinvd cherry pick PR didn't pass PR checker. Please check!!!
#1346

@mssonicbld
Copy link
Copy Markdown
Collaborator

@dbarashinvd cherry pick PR didn't pass PR checker. Please check!!!
#1346

@mssonicbld
Copy link
Copy Markdown
Collaborator

@dbarashinvd cherry pick PR didn't pass PR checker. Please check!!!
#1346

mssonicbld pushed a commit that referenced this pull request Feb 11, 2024
Reverts #1305

this logic is not needed. we'll change sai.profile directly
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.

5 participants