Project

General

Profile

Actions

Bug #3345

closed

Search nodes should accept the "m" notation in memory size

Added by Arthur ANGLADE almost 10 years ago. Updated over 9 years ago.

Status:
Released
Priority:
2
Category:
Web - Nodes & inventories
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Regression:

Description

ie : when searching "8m" for memory size, an error occurs it should be working (it works in 2.3)

Actions #1

Updated by Arthur ANGLADE almost 10 years ago

  • Category set to Web - Nodes & inventories
Actions #2

Updated by Nicolas PERRON almost 10 years ago

  • Target version changed from 2.4.4 to 2.4.5
Actions #3

Updated by Nicolas PERRON over 9 years ago

  • Target version changed from 2.4.5 to 2.4.6
Actions #4

Updated by Vincent MEMBRÉ over 9 years ago

Using the notation "m" is not working in 2.3:

  • I have a Node with 91Mo of Ram
    • In both => searching node for 91mo returns my node => Working fine
    • In 2.4 => searching for 91m throws an error ("Invalide memory size : '496M', expecting '300 Mo', '16KB', etc")
    • In 2.3 => searching for 91m does not display any result.

As it is not working in 2.3, It's far better to have an error message.

Actions #5

Updated by Vincent MEMBRÉ over 9 years ago

We should add the possibility to handle 'm/k/g' and other unities (omitting the O/B)

Actions #6

Updated by Vincent MEMBRÉ over 9 years ago

  • Status changed from New to In progress
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Priority changed from N/A to 2
Actions #7

Updated by Vincent MEMBRÉ over 9 years ago

  • Pull Request set to https://github.com/Normation/ldap-inventory/pull/18
Actions #8

Updated by Vincent MEMBRÉ over 9 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to François ARMAND
Actions #9

Updated by Vincent MEMBRÉ over 9 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100

Applied in changeset ldap-inventory:commit:d148a765473a6064ff36c9c69c2478c97b792bde.

Actions #10

Updated by Anonymous over 9 years ago

Applied in changeset ldap-inventory:commit:980623ab1b9f9012d81d365cedffe0562ce804ac.

Actions #11

Updated by Nicolas PERRON over 9 years ago

  • Status changed from Pending release to Released
Actions

Also available in: Atom PDF