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

[TT-1209] use the same CL node config in k8s as in docker #13411

Merged
merged 16 commits into from
Jun 25, 2024

Conversation

Tofel
Copy link
Contributor

@Tofel Tofel commented Jun 4, 2024

We just read it from TOML instead of using hardcoded ones. I also split v1/v2 ocr and forwarder tests into separate ones so that we can read correct TOML config for them. Makefile commands were modified and command names changed!

Tests... tested:

  • automation load -> simulated
  • keeper benchmark -> simulated
  • ocr soak -> simulated
  • ocr2 soak -> simulated
  • forwarder ocr soak -> simulated
  • forwarder ocr2 soak -> simulated

@Tofel Tofel marked this pull request as ready for review June 20, 2024 12:22
@Tofel Tofel requested review from a team as code owners June 20, 2024 12:22
@cl-sonarqube-production
Copy link

@@ -104,6 +84,30 @@ LimitDefault = 5_000_000
* Upkeeps are expected to be performed during the reorg.
*/
func TestAutomationReorg(t *testing.T) {
c, err := tc.GetConfig("Reorg", tc.Automation)
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

@anirudhwarrier anirudhwarrier left a comment

Choose a reason for hiding this comment

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

Verified that all Automation tests work as before without any changes.

@Tofel Tofel enabled auto-merge June 24, 2024 14:50
@chudilka1 chudilka1 self-requested a review June 25, 2024 09:55
Copy link
Contributor

@chudilka1 chudilka1 left a comment

Choose a reason for hiding this comment

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

Does it require updating README?

@Tofel Tofel added this pull request to the merge queue Jun 25, 2024
Merged via the queue into develop with commit e61d256 Jun 25, 2024
124 checks passed
@Tofel Tofel deleted the tt_1209_k8s_cl_node_toml_from_test_config branch June 25, 2024 10:16
ferglor added a commit that referenced this pull request Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants