-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
tests(clustering/rpc): enable 09-hybrid_mode/01-sync_spec.lua for incremental sync #13959
Conversation
-- TODO: The API output does include labels and certs when the | ||
-- incremental sync is enabled. | ||
if inc_sync == "off" then | ||
assert.equal("mycloud", v.labels.deployment) | ||
assert.equal("us-east-1", v.labels.region) | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For perfect solution, we need to add a feature for incremental sync: https://konghq.atlassian.net/browse/KAG-5935
Cherry-pick failed for Please cherry-pick the changes locally. git remote add upstream https://github.com/kong/kong-ee
git fetch upstream master
git worktree add -d .worktree/cherry-pick-13959-to-master-to-upstream upstream/master
cd .worktree/cherry-pick-13959-to-master-to-upstream
git checkout -b cherry-pick-13959-to-master-to-upstream
ancref=$(git merge-base 35aa60546de8d313c792f32afcb3ec71fd179520 8c8da0e653655f9a70bc1f854d92e4a547cad3aa)
git cherry-pick -x $ancref..8c8da0e653655f9a70bc1f854d92e4a547cad3aa manual cherry-pick: https://github.com/Kong/kong-ee/pull/10922 |
Summary
Enable all tests of 09-hybrid_mode/01-sync_spec.lua under incremental sync on mode
Checklist
changelog/unreleased/kong
orskip-changelog
label added on PR if changelog is unnecessary. README.mdIssue reference
Fix KAG-5951