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

add reorg tests for v23 #13749

Merged
merged 2 commits into from
Jul 17, 2024
Merged

add reorg tests for v23 #13749

merged 2 commits into from
Jul 17, 2024

Conversation

shileiwill
Copy link
Contributor

@shileiwill shileiwill commented Jul 2, 2024

@shileiwill shileiwill requested a review from anirudhwarrier July 2, 2024 22:11
@shileiwill shileiwill marked this pull request as ready for review July 16, 2024 17:13
@shileiwill shileiwill requested review from a team as code owners July 16, 2024 17:13
@cl-sonarqube-production
Copy link

@shileiwill shileiwill enabled auto-merge July 16, 2024 18:31
@@ -240,8 +240,9 @@ func (k *KeeperBenchmarkTest) Run() {
txKeyId = 0
}
kr := k.keeperRegistries[rIndex]
// TODO: need to add the LINK, WETH and WETH/USD feed to support v23
Copy link
Contributor

Choose a reason for hiding this comment

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

is this not done?
if so, can u create a ticket to track

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, this is not done yet, Sync-ed with Anirudh this morning, and he is aware and gonna tackle this soon. Here is the ticket (AUTO-11119) for benchmark testing.

@shileiwill shileiwill added this pull request to the merge queue Jul 16, 2024
Merged via the queue into develop with commit e28f8a4 Jul 17, 2024
115 checks passed
@shileiwill shileiwill deleted the chaos_reorg branch July 17, 2024 00:04
@shileiwill shileiwill restored the chaos_reorg branch July 18, 2024 17:30
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.

3 participants