Added tests EC policy #281

Merged
d.zayakin merged 1 commit from d.zayakin/frostfs-testcases:new-ec-rep-tests into master 2024-08-07 08:00:55 +00:00
Member

Signed-off-by: Dmitriy Zayakin d.zayakin@yadro.com

Signed-off-by: Dmitriy Zayakin <d.zayakin@yadro.com>
d.zayakin added 1 commit 2024-08-05 06:16:48 +00:00
[#281] Added tests EC policy
Some checks reported warnings
DCO check / Commits Check (pull_request) Has been cancelled
76a9b77e27
Signed-off-by: Dmitriy Zayakin <d.zayakin@yadro.com>
d.zayakin requested review from qa-committers 2024-08-05 06:20:22 +00:00
d.zayakin requested review from qa-developers 2024-08-05 06:20:22 +00:00
anurindm approved these changes 2024-08-05 12:35:41 +00:00
abereziny reviewed 2024-08-05 12:37:10 +00:00
@ -579,0 +878,4 @@
expect_chunks = 4 if object_size.name == "simple" else 16
assert len(chunks) == expect_chunks
@allure.title("Replication chunk after drop.(size={object_size})")
Member

Why dot ?

Why dot ?
d.zayakin marked this conversation as resolved
abereziny reviewed 2024-08-05 12:37:58 +00:00
@ -501,0 +506,4 @@
@reporter.step("Search container by name")
def search_container_by_name(name: str, node: ClusterNode):
Member

It's not a fixture to be in conftest
It's not even used!

It's not a fixture to be in conftest It's not even used!
d.zayakin marked this conversation as resolved
abereziny reviewed 2024-08-05 12:40:56 +00:00
@ -73,2 +78,4 @@
@pytest.mark.ec_replication
class TestECReplication(ClusterTestBase):
@wait_for_success(120, 5)
def wait_update_network_map(self, frostfs_cli: FrostfsCli, alive_node: ClusterNode, wait_nodes: int) -> bool:
Member

wait_update_network_map => wait_for_nodes_appears_in_map
wait_nodes => desired_nodes_count

`wait_update_network_map` => `wait_for_nodes_appears_in_map` `wait_nodes` => `desired_nodes_count`
d.zayakin marked this conversation as resolved
abereziny requested changes 2024-08-05 12:44:12 +00:00
@ -75,0 +87,4 @@
@wait_for_success(120, 5)
def wait_replication(self, total_chunks: int, local_cli: FrostfsCli, cid: str, oid: str, success: bool = True) -> None:
if not success:
Member

Why? Success not even used anywhere

Why? Success not even used anywhere
Author
Member

I don't specify this as an argument anywhere, since it leads by default. And added the ability to do False, for cases where you need to check that the number of replications does not match

I don't specify this as an argument anywhere, since it leads by default. And added the ability to do False, for cases where you need to check that the number of replications does not match
abereziny marked this conversation as resolved
d.zayakin force-pushed new-ec-rep-tests from 76a9b77e27 to e35e7e58d5 2024-08-05 12:57:28 +00:00 Compare
d.zayakin force-pushed new-ec-rep-tests from e35e7e58d5 to 262b78766b 2024-08-05 13:59:58 +00:00 Compare
d.zayakin requested review from abereziny 2024-08-06 06:42:58 +00:00
d.zayakin force-pushed new-ec-rep-tests from 262b78766b to 7a4144ce9e 2024-08-06 07:29:44 +00:00 Compare
d.zayakin force-pushed new-ec-rep-tests from 7a4144ce9e to bbc1a20a4d 2024-08-06 07:49:14 +00:00 Compare
d.zayakin referenced this pull request from a commit 2024-08-06 08:32:07 +00:00
d.zayakin force-pushed new-ec-rep-tests from bbc1a20a4d to 286ed5f9e7 2024-08-06 08:49:19 +00:00 Compare
abereziny approved these changes 2024-08-06 08:51:09 +00:00
d.zayakin force-pushed new-ec-rep-tests from 286ed5f9e7 to 0c881c6fc8 2024-08-06 08:54:12 +00:00 Compare
abereziny approved these changes 2024-08-06 08:57:00 +00:00
d.zayakin requested review from qa-committers 2024-08-06 12:14:04 +00:00
d.zayakin requested review from qa-developers 2024-08-06 12:14:04 +00:00
anurindm approved these changes 2024-08-07 07:54:32 +00:00
d.zayakin merged commit 0c881c6fc8 into master 2024-08-07 08:00:55 +00:00
d.zayakin referenced this pull request from a commit 2024-08-07 08:00:57 +00:00
d.zayakin deleted branch new-ec-rep-tests 2024-08-07 08:00:58 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-testcases#281
No description provided.