Instead it should show a red message saying the update failed.
Also, once the user clicks the Save button to save a config, we should display a modal window saying "Configuration update in progress...", since config updates can take a few seconds and to prevent the user from trying to make further edits in the config editor. The modal would go away once the update completed, and at that point the user would be redirected to the History subtab and shown the message indicating whether the update succeeded or failed.
I don't think a modal dialog is the best option, since people should be allowed to navigate to other parts of the app while the config update is being executed, we just dont want to let them edit the config anymore. So can we not just make the readonly as soon as Save is clicked?
The problem here is that the UI message is wrong and attempts to do too much. The message is thrown by the pluginConfigurationEditor when an asynchronous call to the server side is not successful. The message content is wrong because it attempts to indicate the completed status of the asynch method that can realistically take a while to complete and got through 'in progress', 'failed', etc. The most that we can do here is say that we 'initiated' the configuration update process and expect the user to go to the Config History section to monitor the progress later, much like a scheduled operation. Fixed in master with commit: 8e5afecf3ca30d98f5d6711f18e70060b8d95a14 Changed ui message to indicate whether initiation successful only. Moving to ON_QA.
Verified on build#123 (Version: 4.1.0-SNAPSHOT Build Number: a6d2d56) Created a compatible group of datasource resources and changed a config value in the configuration tab of the group. Clicking on the Save button navigates to the history tab and displays a message 'The group configuration updates have been initiated'. Also verified this by shutting down one resource and doing a config update. It displays the message 'The group configuration updates have been initiated' and on refreshing the history list, the failure status of the config update is displayed. Marking as verified.
Bulk closing of old issues that are in VERIFIED state.