Project

General

Profile

Actions

Architecture #2062

closed

Clean up "big red button" code ?

Added by François ARMAND over 12 years ago. Updated almost 5 years ago.

Status:
Rejected
Priority:
4
Category:
Architecture - Code maintenance
Effort required:
Name check:
Fix check:
Regression:

Description

In the last time version, we choose to remove the big red button UI.

Do we want to completely clean it up, or to put it back ?

We should not let the 2.4 be released with the half-backed solution we had for 2.3.

My personal feeling is that "big red button" may be a killer features, but clearly not in the state it was : it should be accessible on a group by group basis, or node by node one.


Related issues 1 (0 open1 closed)

Is duplicate of Rudder - Architecture #14054: Remove red button code from rudderReleasedFrançois ARMANDActions
Actions #1

Updated by Jonathan CLARKE over 12 years ago

  • Target version changed from 2.4.0~alpha1 to 2.4.0~alpha2
Actions #2

Updated by Jonathan CLARKE over 12 years ago

  • Assignee set to François ARMAND
  • Target version changed from 2.4.0~alpha2 to 2.4.0~alpha3

François ARMAND wrote:

My personal feeling is that "big red button" may be a killer features, but clearly not in the state it was : it should be accessible on a group by group basis, or node by node one.

I agree with this. I can tell you that the code in Policy Templates to manage this is mostly ready to manage this on a node by node basis, so I don't think we should remove. I don't know about the code in the webapp though, I'll leave that up to you.

Actions #3

Updated by François ARMAND over 12 years ago

Jonathan CLARKE wrote:

I agree with this. I can tell you that the code in Policy Templates to manage this is mostly ready to manage this on a node by node basis, so I don't think we should remove. I don't know about the code in the webapp though, I'll leave that up to you.

For now, there is no way in the webapp to mark a group or a node as "not managed".
To be consistent with the real node state regarding CFEngine, we need to use the same logic to check if a node is currently in "not managed" mode or not. I believe that that state is marked thanks to a given file (promise) in some directory, and so we can't use any sort of memory/db cache of it (the "red button" feature could be use from the command line or the file put by hand for a given node)

Once that is done, we can start to think to a group-level "red button", whose actual state would be the state of each of its node. The problem is that checking that state may be quite inefficient, as it would imply to check for perhaps hundred of file presence.

So, in the end, I do think that not much of the actual logic in the webapp can be keep, and that their is quite some specification and implementation work remaining, even for the node-only "red button", or to find a 80%-good solution.

Actions #4

Updated by Jonathan CLARKE about 12 years ago

  • Target version changed from 2.4.0~alpha3 to 2.4.0~alpha4
Actions #5

Updated by Jonathan CLARKE about 12 years ago

  • Priority changed from 3 to 4
Actions #6

Updated by François ARMAND about 12 years ago

  • Target version changed from 2.4.0~alpha4 to 2.4.0~alpha5
Actions #7

Updated by François ARMAND about 12 years ago

  • Assignee changed from François ARMAND to Nicolas CHARLES
Actions #8

Updated by Nicolas CHARLES about 12 years ago

The CFEngine code is indeed valid (and on a node basis)
As for the webapp part, i'm afraid we are a bit late in the release cycle to safely remove it; even more since there are event related to it, and scripts run by it.

Actions #9

Updated by Jonathan CLARKE about 12 years ago

  • Target version changed from 2.4.0~alpha5 to 2.4.0~alpha6
Actions #10

Updated by Jonathan CLARKE almost 12 years ago

  • Target version changed from 2.4.0~alpha6 to 18
Actions #11

Updated by Jonathan CLARKE over 11 years ago

  • Target version changed from 18 to Ideas (not version specific)
Actions #12

Updated by Benoît PECCATTE almost 9 years ago

  • Tracker changed from Question to Architecture
Actions #13

Updated by Nicolas CHARLES almost 9 years ago

  • Assignee changed from Nicolas CHARLES to François ARMAND

i guess the webapp code has been rmoved, isn't it ?

Actions #14

Updated by Alexis Mousset almost 7 years ago

It is still there is the system techniques, displaying a pretty non-understandable:

E| compliant     Common            Red Button               Red Button is not in effect, continuing as normal...

at every run on every node.

Actions #15

Updated by Alexis Mousset almost 5 years ago

Done in 5.1 (#14045)

Actions #16

Updated by Alexis Mousset almost 5 years ago

Actions #17

Updated by Alexis Mousset almost 5 years ago

  • Status changed from Discussion to Rejected
Actions

Also available in: Atom PDF