Skip to content

Commit

Permalink
Fix: ui_context: crm configure up prompt ClusterLabs#1466 (ClusterLab…
Browse files Browse the repository at this point in the history
…s#1481)

The regression was introduced in beb26f3
Before beb26f3 it was like
```
crm(live/15sp5-1)configure# primitive d Dummy
crm(live/15sp5-1)configure# up
There are changes pending. Do you want to commit them (y/n)?
```
After beb26f3 there is no prompt `There are changes pending. Do you
want to commit them (y/n)?`
This change brings the prompt back.
  • Loading branch information
liangxin1300 authored Jul 8, 2024
2 parents 3b3d7ee + ac4a532 commit 3e96f90
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion crmsh/ui_context.py
Original file line number Diff line number Diff line change
Expand Up @@ -326,7 +326,7 @@ def up(self):
'''
ok = True
if len(self.stack) > 1:
if self.command_name and self.command_name not in constants.NON_FUNCTIONAL_COMMANDS:
if ServiceManager().service_is_active("pacemaker.service"):
ok = self.current_level().end_game(no_questions_asked=self._in_transit) is not False
self.stack.pop()
self.clear_readline_cache()
Expand Down

0 comments on commit 3e96f90

Please sign in to comment.