diff --git a/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureProxiedRepositoryAction-validation.xml b/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureProxiedRepositoryAction-validation.xml
index f72ec85c8..1b49d7d0d 100644
--- a/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureProxiedRepositoryAction-validation.xml
+++ b/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureProxiedRepositoryAction-validation.xml
@@ -19,7 +19,6 @@
 
 <!-- TODO! see MRM-150 for progress on these -->
 <validators>
-  <!-- TODO: constrain more -->
   <field name="id">
     <field-validator type="requiredstring">
       <message>You must enter the repository identifier.</message>
@@ -35,7 +34,6 @@
     </field-validator>
   </field>
 
-  <!-- deng todo: check if the entered repo url exists -->
   <field name="url">
     <field-validator type="requiredstring">
       <message>You must enter the repository URL.</message>
@@ -54,7 +52,7 @@
     </field-validator>
   </field>
 
-  <!-- deng todo: check if the interval validator is still valid -->
+  <!-- TODO: is the validation correct? -->
   <validator type="interval">
     <message/>
   </validator>
@@ -87,9 +85,4 @@
     </field-validator>
   </field>
 
-  <!-- TODO: validate managed repository -->
-  <!-- TODO: validate layout -->
-  <!-- TODO: validate policies -->
-  <!-- TODO: validate that intervals are integers -->
-  <!-- TODO: validate that intervals are empty if policy is not interval -->
 </validators>
\ No newline at end of file
diff --git a/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureRepositoryAction-validation.xml b/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureRepositoryAction-validation.xml
index 8de49142b..158527497 100644
--- a/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureRepositoryAction-validation.xml
+++ b/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureRepositoryAction-validation.xml
@@ -18,8 +18,7 @@
     "http://www.opensymphony.com/xwork/xwork-validator-1.0.2.dtd">
 
 <!-- TODO! see MRM-150 for progress on these -->
-<validators>
-  <!-- TODO: constrain more -->
+<validators>  
   <field name="id">
     <field-validator type="requiredstring">
       <message>You must enter the repository identifier.</message>
@@ -30,7 +29,6 @@
       <message>You must enter the repository name.</message>
     </field-validator>
   </field>
-  <!-- deng todo: validate if the specified directory exists -->
   <field name="directory">
     <field-validator type="requiredstring">
       <message>You must enter the repository directory.</message>
diff --git a/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureSyncedRepositoryAction-addSelectedSyncedRepository-validation.xml b/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureSyncedRepositoryAction-addSelectedSyncedRepository-validation.xml
index 10ff70b01..1cd7245ff 100644
--- a/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureSyncedRepositoryAction-addSelectedSyncedRepository-validation.xml
+++ b/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureSyncedRepositoryAction-addSelectedSyncedRepository-validation.xml
@@ -47,8 +47,5 @@
   <validator type="syncedrepo">
     <message/>
   </validator>
-
-  <!-- TODO: validate managed repository -->
-  <!-- TODO: validate layout -->
-  <!-- TODO: validate sync settings, depending on what method -->
+    
 </validators>
\ No newline at end of file
diff --git a/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureSyncedRepositoryAction-validation.xml b/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureSyncedRepositoryAction-validation.xml
index 43ea651aa..a842b3670 100644
--- a/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureSyncedRepositoryAction-validation.xml
+++ b/archiva-webapp/src/main/resources/org/apache/maven/archiva/web/action/admin/ConfigureSyncedRepositoryAction-validation.xml
@@ -18,8 +18,7 @@
     "http://www.opensymphony.com/xwork/xwork-validator-1.0.2.dtd">
 
 <!-- TODO! see MRM-150 for progress on these -->
-<validators>
-  <!-- TODO: constrain more -->
+<validators>  
   <field name="method">
     <field-validator type="requiredstring">
       <message>You must enter the synchronization method.</message>