Kanagaraj M has posted comments on this change.

Change subject: webadmin: remove bricks with migration support
......................................................................


Patch Set 3:

(4 comments)

New patch set to follow

....................................................
Commit Message
Line 7: webadmin: remove bricks with migration support
Line 8: 
Line 9: Added a new Checkbox 'Migrate Data' to the Remove Bricks
Line 10: Confirmation popup. And a warning message in the bottom
Line 11: explaining the necessity to migrate the data before remove.
Its there
Line 12: 
Line 13: The bricks in the confirmation popup is wrapped inside
Line 14: a ScrollPanel to avoid the overflowing if more number of
Line 15: bricks are selected.


....................................................
File 
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
Line 1713:     @DefaultStringValue("Cannot remove more than one brick from a 
Replicate volume at a time")
Line 1714:     String cannotRemoveBricksReplicateVolume();
Line 1715: 
Line 1716:     @DefaultStringValue("Incorrect bricks selected for the removal 
in Distributed Replicate volume. <br/> Either all the selected bricks should be 
from the same sub volume or one brick each for every sub volume!")
Line 1717:     String cannotRemoveBricksDistributedReplicateVolume();
Done
Line 1718: 
Line 1719:     @DefaultStringValue("Incorrect bricks selected for the removal 
in Distributed Stripe volume. <br/> Selected bricks should be from the same sub 
volume!")
Line 1720:     String cannotRemoveBricksDistributedStripeVolume();
Line 1721: 


Line 1716:     @DefaultStringValue("Incorrect bricks selected for the removal 
in Distributed Replicate volume. <br/> Either all the selected bricks should be 
from the same sub volume or one brick each for every sub volume!")
Line 1717:     String cannotRemoveBricksDistributedReplicateVolume();
Line 1718: 
Line 1719:     @DefaultStringValue("Incorrect bricks selected for the removal 
in Distributed Stripe volume. <br/> Selected bricks should be from the same sub 
volume!")
Line 1720:     String cannotRemoveBricksDistributedStripeVolume();
Done
Line 1721: 
Line 1722:     @DefaultStringValue("Incorrect bricks selected for the removal 
in Striped Replicate volume. <br/> Selected bricks should be a multiple of 
stripe and replica count")
Line 1723:     String cannotRemoveBricksStripedReplicateVolume();
Line 1724: 


....................................................
File 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
Line 2681:     @DefaultStringValue("Ensure safe data migration before removing 
the bricks")
Line 2682:     String removeBricksMigrateDataInfo();
Line 2683: 
Line 2684:     @DefaultStringValue("Please note that data will be lost if 
choose not to migrate")
Line 2685:     String removeBricksWarning();
Done
Line 2686: 
Line 2687:     // Volume Brick Details
Line 2688:     @DefaultStringValue("General")
Line 2689:     String generalBrickAdvancedPopupLabel();


-- 
To view, visit http://gerrit.ovirt.org/19201
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8aae4548661ff1cb9f93a14ab66b6c41bc201bbd
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: Shubhendu Tripathi <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to