feedback improvements: link todo to ticket

Author: filipcirtog

.evergreen.yml

@@ -749,7 +749,7 @@ task_groups:
       - e2e_sharded_cluster_scram_sha_256_switch_project
       - e2e_replica_set_scram_sha_1_switch_project
       - e2e_sharded_cluster_scram_sha_1_switch_project
-      # Disabled these tests as they don't use the password secret, and project migrations aren't fully supported yet.  
+      # TODO CLOUDP-349093 - Disabled these tests as they don't use the password secret, and project migrations aren't fully supported yet.  
       # e2e_sharded_cluster_x509_switch_project
       # e2e_replica_set_x509_switch_project
       # e2e_replica_set_ldap_switch_project

docker/mongodb-kubernetes-tests/tests/authentication/replica_set_ldap_switch_project.py

@@ -115,7 +115,7 @@ class TestReplicaSetLDAPProjectSwitch(KubernetesTester):
     def test_create_replica_set(self, testhelper: ReplicaSetSwitchProjectHelper):
         testhelper.test_create_replica_set()
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_create_ldap_user(self, user_ldap: MongoDBUser):
     #     user_ldap.assert_reaches_phase(Phase.Updated)
 
@@ -139,7 +139,7 @@ def test_ops_manager_state_correctly_updated_in_initial_replica_set(
     def test_switch_replica_set_project(self, testhelper: ReplicaSetSwitchProjectHelper):
         testhelper.test_switch_replica_set_project()
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_ops_manager_state_with_users_correctly_updated_after_switch(
     #     self, testhelper: ReplicaSetSwitchProjectHelper
     # ):

docker/mongodb-kubernetes-tests/tests/authentication/replica_set_scram_sha_1_switch_project.py

@@ -56,7 +56,7 @@ def test_ops_manager_state_correctly_updated_in_initial_replica_set(
     ):
         testhelper.test_ops_manager_state_with_expected_authentication(expected_users=0)
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_create_secret(self):
     #     create_or_update_secret(
     #         KubernetesTester.get_namespace(),
@@ -98,7 +98,7 @@ def test_replica_set_connectivity_after_switch(self, testhelper: ReplicaSetSwitc
     def test_ops_manager_state_correctly_updated_after_switch(self, testhelper: ReplicaSetSwitchProjectHelper):
         testhelper.test_ops_manager_state_with_expected_authentication(expected_users=0)
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_ops_manager_state_with_users_correctly_updated_after_switch(
     #     self, testhelper: ReplicaSetSwitchProjectHelper
     # ):

docker/mongodb-kubernetes-tests/tests/authentication/replica_set_scram_sha_256_switch_project.py

@@ -51,7 +51,7 @@ def test_replica_set_connectivity(self, testhelper: ReplicaSetSwitchProjectHelpe
     def test_ops_manager_state_correctly_updated(self, testhelper: ReplicaSetSwitchProjectHelper):
         testhelper.test_ops_manager_state_with_expected_authentication(expected_users=0)
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_create_secret(self):
     #     create_or_update_secret(
     #         KubernetesTester.get_namespace(),
@@ -93,7 +93,7 @@ def test_replica_set_connectivity_after_switch(self, testhelper: ReplicaSetSwitc
     def test_ops_manager_state_correctly_updated_after_switch(self, testhelper: ReplicaSetSwitchProjectHelper):
         testhelper.test_ops_manager_state_with_expected_authentication(expected_users=0)
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_ops_manager_state_with_users_correctly_updated_after_switch(
     #     self, testhelper: ReplicaSetSwitchProjectHelper
     # ):

docker/mongodb-kubernetes-tests/tests/authentication/sharded_cluster_ldap_switch_project.py

@@ -122,7 +122,7 @@ def wait_for_ac_pushed() -> bool:
     def test_sharded_cluster_CLOUDP_229222(self, sharded_cluster: MongoDB, ldap_mongodb_users: List[LDAPUser]):
         sharded_cluster.assert_reaches_phase(Phase.Running, timeout=800)
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_new_mdb_users_are_created(self, ldap_user_mongodb: MongoDBUser):
     #     ldap_user_mongodb.assert_reaches_phase(Phase.Updated)
 
@@ -134,7 +134,7 @@ def test_ops_manager_state_correctly_updated_in_initial_cluster(
     def test_switch_sharded_cluster_project(self, testhelper: ShardedClusterSwitchProjectHelper):
         testhelper.test_switch_sharded_cluster_project()
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_ops_manager_state_correctly_updated_after_switch(self, testhelper: ShardedClusterSwitchProjectHelper):
     #     testhelper.test_ops_manager_state_with_expected_authentication(expected_users=0)
     #     # There should be one user (the previously created user should still exist in the automation configuration). We need to investigate further to understand why the user is not being picked up.

docker/mongodb-kubernetes-tests/tests/authentication/sharded_cluster_scram_sha_1_switch_project.py

@@ -60,7 +60,7 @@ def test_ops_manager_state_correctly_updated_in_initial_sharded_cluster(
     ):
         testhelper.test_ops_manager_state_with_expected_authentication(expected_users=0)
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_create_secret(self):
     #     create_or_update_secret(
     #         KubernetesTester.get_namespace(),
@@ -103,7 +103,7 @@ def test_sharded_cluster_connectivity_after_switch(self, testhelper: ShardedClus
     def test_ops_manager_state_correctly_updated_after_switch(self, testhelper: ShardedClusterSwitchProjectHelper):
         testhelper.test_ops_manager_state_with_expected_authentication(expected_users=0)
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_ops_manager_state_with_users_correctly_updated_after_switch(
     #     self, testhelper: ShardedClusterSwitchProjectHelper
     # ):

docker/mongodb-kubernetes-tests/tests/authentication/sharded_cluster_scram_sha_256_switch_project.py

@@ -60,7 +60,7 @@ def test_ops_manager_state_correctly_updated_in_initial_sharded_cluster(
     ):
         testhelper.test_ops_manager_state_with_expected_authentication(expected_users=0)
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_create_secret(self):
     #     create_or_update_secret(
     #         KubernetesTester.get_namespace(),
@@ -103,7 +103,7 @@ def test_sharded_cluster_connectivity_after_switch(self, testhelper: ShardedClus
     def test_ops_manager_state_correctly_updated_after_switch(self, testhelper: ShardedClusterSwitchProjectHelper):
         testhelper.test_ops_manager_state_with_expected_authentication(expected_users=0)
 
-    # Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
+    # TODO CLOUDP-349093 - Disabled these tests because project migrations are not supported yet, which could lead to flaky behavior.
     # def test_ops_manager_state_with_users_correctly_updated_after_switch(
     #     self, testhelper: ShardedClusterSwitchProjectHelper
     # ):