Skip to content

Modified the ecn profiles in script to LOSSLESS as all platforms doesnt have LOSSY profile #747

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.

Sign up for GitHub

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

Merged
stcheng merged 2 commits into from
Jul 17, 2019
Merged

Modified the ecn profiles in script to LOSSLESS as all platforms doesnt have LOSSY profile #747

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
852f74a
Modified the ecn profiles in script to LOSSLESS as all platforms doe…
chitra-raghavan Dec 5, 2018
4e0bf8f
Merge branch 'master' into ecn
chitra-raghavan Dec 6, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 4 additions & 5 deletions ansible/roles/test/tasks/ecn_wred.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@
test_wred_values="[ '491520', '516096', '430080' ]"
tags: always

# Read and store original rmin value for AZURE_LOSSY
# Read and store original rmin value for AZURE_LOSSLESS
- name: Get red_min_threshold
shell: ecnconfig -l | grep -A20 AZURE_LOSSY | grep red_min_threshold | awk '{print $2}'
shell: ecnconfig -l | grep -A20 AZURE_LOSSLESS | grep red_min_threshold | awk '{print $2}'
register: wred_value
failed_when: wred_value.rc != 0

Expand Down Expand Up @@ -42,8 +42,7 @@

always:
- name: Restore original value
shell: ecnconfig -p AZURE_LOSSLESS -rmin {{ red_min_threshold }}
become: yes
shell: ecnconfig -p AZURE_LOSSY -rmin {{ red_min_threshold }}
register: ecn_restore
failed_when: ecn_restore.rc != 0

failed_when: ecn_restore.rc != 0
2 changes: 1 addition & 1 deletion ansible/roles/test/tasks/ecn_wred_worker.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@

# Set value...
- name: Set WRED value {{ item }}
shell: ecnconfig -p AZURE_LOSSLESS -rmin {{ item }}
become: yes
shell: ecnconfig -p AZURE_LOSSY -rmin {{ item }}
register: rc
failed_when: rc.rc != 0

Expand Down