Project

General

Profile

Bug #33852

Specify replication bandwidth limit in kbps

Added by Steven Nyhof 11 months ago. Updated 10 months ago.

Status:
Done
Priority:
No priority
Assignee:
William Grzybowski
Category:
Middleware
Target version:
Seen in:
Severity:
Low
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

Units appear to be wrong for the bandwidth limit in both the UI (both main Replication Tasks screen & task edit screen) as well as the documentation. UI & documentation specify kB/s however testing seems to indicate this should be kbps.

How I tested:

Created a new dataset with a 3.1 Gb compressed file.
Created a new snapshot.
Created a new replication task with limit = 300,000.
Timed the replication to be approximately 90s (this was over gigabit ethernet).
Did the math: 300,000 kilobits / s * 90 s = 27,000,000 kilobits = 3375000 kB = 3.2 Gb - close enough for me


Related issues

Copied to FreeNAS - Bug #34702: Show replication bandwidth in kbpsDone

Associated revisions

Revision 49abe381 (diff)
Added by William Grzybowski 11 months ago

fix(gui): kB/s -> kbps; kilobytes -> kilobits

Ticket: #33852

Revision e0215a56 (diff)
Added by William Grzybowski 11 months ago

fix(gui): kB/s -> kbps; kilobytes -> kilobits

Ticket: #33852

History

#1 Updated by Dru Lavigne 11 months ago

  • Reason for Blocked set to Need additional information from Author

Steven: to clarify, did you try in the old UI, the new UI, or both?

#2 Updated by Steven Nyhof 11 months ago

Dru Lavigne wrote:

Steven: to clarify, did you try in the old UI, the new UI, or both?

Hi Dru

This was in the old UI, however I just tried the beta UI and it has the same incorrect information both in the field label (KB/s) and in the help hover text (kilobytes/second). Should be, I believe, (kbps) and (kilobits/second), respectively. The Guide also has the incorrect units.

#3 Updated by Dru Lavigne 11 months ago

  • Category changed from GUI (new) to Middleware
  • Assignee changed from Release Council to William Grzybowski

#4 Updated by William Grzybowski 11 months ago

  • Status changed from Unscreened to In Progress
  • Target version changed from Backlog to 11.2-BETA1
  • Reason for Blocked deleted (Need additional information from Author)
  • Needs QA changed from Yes to No
  • Needs Merging changed from Yes to No

#5 Updated by William Grzybowski 11 months ago

#6 Updated by William Grzybowski 11 months ago

  • Status changed from In Progress to Ready for Testing

#7 Updated by Dru Lavigne 11 months ago

  • Subject changed from Wrong units on replication bandwidth limit to Replication bandwidth limit is in kbps

#8 Avatar?id=55038&size=24x24 Updated by Zackary Welch 10 months ago

  • Needs QA changed from No to Yes

So I see the change in the old UI to kbps, but the new UI still has KB/s on the Replication task scene and kilobytes/second in the tooltip. Docs still has KB/s and kilobytes/second as well on the Tasks page.

#9 Updated by Dru Lavigne 10 months ago

  • Status changed from Ready for Testing to Passed Testing

#10 Updated by Dru Lavigne 10 months ago

  • Copied to Bug #34702: Show replication bandwidth in kbps added

#11 Updated by Dru Lavigne 10 months ago

  • Subject changed from Replication bandwidth limit is in kbps to Specify replication bandwidth limit in kbps

#12 Updated by Dru Lavigne 10 months ago

  • Needs QA changed from Yes to No

#13 Updated by Michael Reynolds 10 months ago

  • File Storage-RepTasks-Limit-kbps-2018-06-25.png added
  • File Storage-RepTasks-Limit-kbps-webui-2018-06-25.png added
  • File RepTaskLegacyDocs-2018-06-25.png added

Legacy UI
Storage:Replication Tasks -> Limit (kbps) (Is a column)
See Storage-RepTasks-Limit-kbps-2018-06-25.png

WebUI
Tasks:Replication Tasks -> Add Replication -> Limit (kbps) (is a field)
Storage-RepTasks-Limit-kbps-webui-2018-06-25.png
See Storage-RepTasks-Limit-kbps-webui-2018-06-25.png

Legacy docs do not refer to kbps see RepTaskLegacyDocs-2018-06-25.png
WebUI docs do not mention kbps see RepTaskWebUI-Docs-2018-06-25.png

#14 Updated by Michael Reynolds 10 months ago

  • File RepTaskWebUI-Docs-2018-06-25.png added

#15 Updated by Dru Lavigne 10 months ago

  • File deleted (Storage-RepTasks-Limit-kbps-2018-06-25.png)

#16 Updated by Dru Lavigne 10 months ago

  • File deleted (Storage-RepTasks-Limit-kbps-webui-2018-06-25.png)

#17 Updated by Dru Lavigne 10 months ago

  • File deleted (RepTaskLegacyDocs-2018-06-25.png)

#18 Updated by Dru Lavigne 10 months ago

  • File deleted (RepTaskWebUI-Docs-2018-06-25.png)

#19 Updated by Dru Lavigne 10 months ago

  • Status changed from Passed Testing to Done
  • Needs Doc changed from Yes to No

Also available in: Atom PDF