Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(iota-benchmark/tests): fix test_simulated_load_reconfig_restarts #3853

Conversation

Dkwcs
Copy link
Contributor

@Dkwcs Dkwcs commented Nov 1, 2024

Description of change

test_simulated_load_reconfig_restarts passed (green) without any changes.

Links to any relevant issues

Fix #3852 .

Type of change

  • Bug fix (a non-breaking change which fixes an issue)

How the change has been tested

cargo simtest test_simulated_load_reconfig_restarts

@Dkwcs Dkwcs added the vm-language Issues related to the VM & Language Team label Nov 1, 2024
@Dkwcs Dkwcs self-assigned this Nov 1, 2024
@Dkwcs Dkwcs marked this pull request as ready for review November 1, 2024 13:54
@Dkwcs Dkwcs requested review from a team as code owners November 1, 2024 13:54
crates/iota-benchmark/tests/simtest.rs Outdated Show resolved Hide resolved
@Dkwcs Dkwcs marked this pull request as draft November 5, 2024 14:50
@Dkwcs Dkwcs marked this pull request as ready for review November 6, 2024 19:10
@miker83z miker83z changed the title fix(iota-benchmark/tests): fix test_simulated_load_reconfig_restarts, access_clock_object_test fix(iota-benchmark/tests): fix test_simulated_load_reconfig_restarts Nov 7, 2024
…_simulated_load_reconfig_restarts,-access_clock_object_test)
@@ -468,10 +468,9 @@ async fn call_shared_object_contract() {
);
}

#[ignore("Disabled due to flakiness - re-enable when failure is fixed")]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of this comment, can you put in a link to a tracking GH issue?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean create new one? Because it's original Sui's comment and there is no appropriate issue for it as I see

Copy link
Contributor

@miker83z miker83z Nov 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes you could create one issue in the Product Backlog with low priority

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

…d_reconfig_restarts,-access_clock_object_test)' of github.com:iotaledger/kinesis into sc-platfrom/fix-iota-benchmark-tests(test_simulated_load_reconfig_restarts,-access_clock_object_test)
@Dkwcs Dkwcs requested a review from DaughterOfMars November 12, 2024 09:24
…_simulated_load_reconfig_restarts,-access_clock_object_test)
…_simulated_load_reconfig_restarts,-access_clock_object_test)
@Dkwcs Dkwcs merged commit 7796881 into develop Nov 12, 2024
39 of 40 checks passed
@Dkwcs Dkwcs deleted the sc-platfrom/fix-iota-benchmark-tests(test_simulated_load_reconfig_restarts,-access_clock_object_test) branch November 12, 2024 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vm-language Issues related to the VM & Language Team
Projects
None yet
7 participants