-
Notifications
You must be signed in to change notification settings - Fork 498
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
Make StsWebhook and PodWebhook work #2664
Conversation
debug in ci debug ci fix e2e checking add log E2e CI job use memory storage for etcd (pingcap#2633) * support memory storage for etcd in kind * export ETCD_STORAGE_TYPE=memory in e2e_kind.groovy * fix a bug * use KIND_ETCD_DATADIR instead of ETCD_STORAGE_TYPE * fix typos * fix bugs * Revert "fix bugs" This reverts commit 3ae083b. * revert format and simplified extraMounts Hide datasource information for backup job (pingcap#2652) Co-authored-by: pingcap-github-bot <sre-bot@pingcap.com> use crd Update pingcap_tidb_operator_build_kind.groovy
tc := fixture.GetTidbCluster(ns, "admission", utilimage.TiDBV3Version) | ||
tc.Spec.PD.Replicas = 3 | ||
tc.Spec.TiKV.Replicas = 3 | ||
tc.Spec.TiDB.Replicas = 2 |
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.
use crd instead
return fmt.Errorf("fail to set annotation for [%s/%s], component: %s, partition: %d, err: %v, output: %s", namespace, tcName, component, ordinal, err, string(output)) | ||
} | ||
return nil | ||
} |
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.
use framework.RunKubectl
or related functions to run kubectl
command, e.g.
tidb-operator/tests/e2e/tidbcluster/tidbcluster.go
Lines 1223 to 1232 in 46ad199
args := []string{ | |
"exec", "-n", ns, | |
fmt.Sprintf("%s-0", controller.TiCDCMemberName(fromTCName)), | |
"--", | |
"/cdc", "cli", "changefeed", "create", | |
fmt.Sprintf("--sink-uri=tidb://root:@%s:4000/", controller.TiDBMemberName(toTCName)), | |
fmt.Sprintf("--pd=http://%s:2379", controller.PDMemberName(fromTCName)), | |
} | |
data, err := framework.RunKubectl(args...) | |
framework.ExpectNoError(err, fmt.Sprintf("failed to create change feed task: %s, %v", string(data), err)) |
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.
updated.
/merge |
/run-all-tests |
/run-all-tests |
Signed-off-by: sre-bot <sre-bot@pingcap.com>
cherry pick to release-1.1 in PR #2676 |
What problem does this PR solve?
Close #2095
Currently sts webhook and pod webhook couldn't work at the same time because when pod webhook enabled, the sts partition would direct be set as 0.
This pr fix this and make e2e test to ensure it.
Related changes
Does this PR introduce a user-facing change?: