Project

General

Profile

Actions
Copy link

Bug #3345

closed

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

Added by Arthur ANGLADE over 11 years ago. Updated over 11 years ago.

Status:
Released
Priority:
2
Assignee:
François ARMAND
Category:
Web - Nodes & inventories
Target version:
2.4.6
Pull Request:
https://github.com/Normation/ldap-inv...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

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

Actions
Copy link
 #1

Updated by Arthur ANGLADE over 11 years ago

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

Updated by Nicolas PERRON over 11 years ago

  • Target version changed from 2.4.4 to 2.4.5
Actions
Copy link
 #3

Updated by Nicolas PERRON over 11 years ago

  • Target version changed from 2.4.5 to 2.4.6
Actions
Copy link
 #4

Updated by Vincent MEMBRÉ over 11 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
Copy link
 #5

Updated by Vincent MEMBRÉ over 11 years ago

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

Actions
Copy link
 #6

Updated by Vincent MEMBRÉ over 11 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
Copy link
 #7

Updated by Vincent MEMBRÉ over 11 years ago

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

Updated by Vincent MEMBRÉ over 11 years ago

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

Updated by Vincent MEMBRÉ over 11 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
Copy link
 #10

Updated by Anonymous over 11 years ago

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

Actions
Copy link
 #11

Updated by Nicolas PERRON over 11 years ago

  • Status changed from Pending release to Released
Actions
Copy link

Also available in: Atom PDF