Project

General

Profile

Actions

Bug #4643

closed

Search Nodes: Search button grayed-out after search

Added by Dennis Cabooter about 7 years ago. Updated over 6 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Nodes & inventories
Target version:
Severity:
User visibility:
Effort required:
Priority:

Description

Scenario

  • I'm searching for nodes matching the following criteria:
Match the criteria below with this operand: AND
Software Name = rudder-agent
Software Version Not matching regex 2.9.4.*
  • I upgrade rudder-agent on a node and run "cf-agent -KI -D force_inventory" to update the inventory
  • I want to search again to get an up to data search result

Problem

The search button is grayed-out, so it's impossible to perform the same search without re-doing the search criteria.

Work-around

Since the Rudder server was upgraded first and so rudder-agent, ticking or unticking "Include policy servers" makes the search button useable again.


Related issues

Has duplicate Rudder - Bug #5626: Greyed-out Search button after a search should not be greyed-outRejectedFrançois ARMAND2014-10-09Actions
Actions #1

Updated by Nicolas CHARLES about 7 years ago

  • Category set to Web - Nodes & inventories
  • Assignee set to François ARMAND

Ha, this is indeed quite unfriendly.
Francois, I'm sure you have a solution for this (maybe detect when node changes?)

Actions #2

Updated by François ARMAND about 7 years ago

Well, I'm not sur it's a technical problem, and perhaps the real solution is to just let the user click as much as he wants on the button. Moreover, as depicted here, the workaround exists and so, we don't really gard people from doing that action.

So, I find more and more than we are going to far in the patronization of our user, and that the limits (like that one) we arbitrary impose bite them much more than they help.

So, perhaps we should allows any search, and allows "save" only if the nodelist changed.

Actions #3

Updated by François ARMAND about 7 years ago

  • Assignee changed from François ARMAND to Jonathan CLARKE

Jon, what so you thing about that?

Actions #4

Updated by Jonathan CLARKE about 7 years ago

  • Status changed from New to 8
  • Assignee changed from Jonathan CLARKE to François ARMAND
  • Target version set to 2.10.0

François ARMAND wrote:

So, perhaps we should allows any search, and allows "save" only if the nodelist changed.

Yup, agreed. Let's do it!

Since this is a behavioural change, let's introduce it in 2.10, not released versions.

Actions #5

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 2.10.0 to 2.11.0~beta1
Actions #6

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 2.11.0~beta1 to 2.11.0~beta2
Actions #7

Updated by Matthieu CERDA almost 7 years ago

  • Target version changed from 2.11.0~beta2 to 2.11.0~rc1
Actions #8

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 2.11.0~rc1 to 2.11.0~rc2
Actions #9

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 2.11.0~rc2 to 2.11.0
Actions #10

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 2.11.0 to 2.11.1
Actions #11

Updated by Nicolas PERRON almost 7 years ago

  • Target version changed from 2.11.1 to 2.11.2
Actions #12

Updated by Nicolas PERRON over 6 years ago

  • Target version changed from 2.11.2 to 2.11.3
Actions #13

Updated by Matthieu CERDA over 6 years ago

  • Target version changed from 2.11.3 to 2.11.4
Actions #14

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 2.11.4 to 2.11.5
Actions #15

Updated by Jonathan CLARKE over 6 years ago

  • Target version changed from 2.11.5 to 3.0.0~beta2
Actions #16

Updated by Nicolas CHARLES over 6 years ago

  • Status changed from 8 to Pending technical review
  • Pull Request set to https://github.com/Normation/rudder/pull/691
Actions #17

Updated by Nicolas CHARLES over 6 years ago

  • Status changed from Pending technical review to 8
  • Assignee changed from François ARMAND to Nicolas CHARLES

oh, it should have been fixed in branch 3.0 :(

Actions #18

Updated by Nicolas CHARLES over 6 years ago

  • Status changed from 8 to Pending technical review
  • Assignee changed from Nicolas CHARLES to François ARMAND
  • Pull Request changed from https://github.com/Normation/rudder/pull/691 to https://github.com/Normation/rudder/pull/692
Actions #19

Updated by Nicolas CHARLES over 6 years ago

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

Updated by Vincent MEMBRÉ over 6 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 3.0.0~beta2, which were these days.

Actions

Also available in: Atom PDF