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

Migrate experimental flags in robustness tests #19354

Open
3 tasks
siyuanfoundation opened this issue Feb 6, 2025 · 4 comments
Open
3 tasks

Migrate experimental flags in robustness tests #19354

siyuanfoundation opened this issue Feb 6, 2025 · 4 comments
Assignees

Comments

@siyuanfoundation
Copy link
Contributor

siyuanfoundation commented Feb 6, 2025

What would you like to be added?

With #19141 and #18023, we will be removing all experimental flags in 3.7.

The robustness tests are used by different binary versions, for the same parameter, we would need to use the experimental flag in 3.5 and 3.4, while using the non-experimental flag in 3.6.

The affected flags include:

  • experimental-snapshot-catchup-entries
  • experimental-compaction-batchLimit
  • experimental-watch-process-notify-interval

Why is this needed?

The robustness test would fail when we delete the deprecated flags if we don't fix this.

@kavirajk
Copy link

@siyuanfoundation I would like to pick this up if that's ok.

Also quick clarification. I see few more experimental flags are to be removed in v1.37.

-bash-5.2$ rg "TODO: Delete in v3.7" -A 3 | rg "Experimental" | rg "json" | awk '{print$2}'
ExperimentalTxnModeWriteWithSharedBuffer
ExperimentalEnableLeaseCheckpointPersist
ExperimentalCompactionBatchLimit
ExperimentalCompactionSleepInterval
ExperimentalWatchProgressNotifyInterval
ExperimentalWarningApplyDuration
ExperimentalBootstrapDefragThresholdMegabytes
ExperimentalWarningUnaryRequestDuration
ExperimentalMaxLearners
ExperimentalPeerSkipClientSanVerification
ExperimentalMemoryMlock
ExperimentalTxnModeWriteWithSharedBuffer
ExperimentalStopGRPCServiceOnDefrag

Those things can also cause failure in robustness test IIUC? no? But I see you mentioned only subset of it.

@siyuanfoundation
Copy link
Contributor Author

[@siyuanfoundation](https://github.com/siyuanfoundation) I would like to pick this up if that's ok.

Also quick clarification. I see few more experimental flags are to be removed in v1.37.

-bash-5.2$ rg "TODO: Delete in v3.7" -A 3 | rg "Experimental" | rg "json" | awk '{print$2}'
ExperimentalTxnModeWriteWithSharedBuffer
ExperimentalEnableLeaseCheckpointPersist
ExperimentalCompactionBatchLimit
ExperimentalCompactionSleepInterval
ExperimentalWatchProgressNotifyInterval
ExperimentalWarningApplyDuration
ExperimentalBootstrapDefragThresholdMegabytes
ExperimentalWarningUnaryRequestDuration
ExperimentalMaxLearners
ExperimentalPeerSkipClientSanVerification
ExperimentalMemoryMlock
ExperimentalTxnModeWriteWithSharedBuffer
ExperimentalStopGRPCServiceOnDefrag
Those things can also cause failure in robustness test IIUC? no? But I see you mentioned only subset of it.

These flags are not currently used in robustness tests. If they are being used, then yes they would break.

@ivanvc
Copy link
Member

ivanvc commented Feb 27, 2025

Covered in our triage meeting. @kavirajk, are you interested in working on this? If so, please comment /assign so the k8s-bot can assign it to you

@kavirajk
Copy link

/assign

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants