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

Hide datasource information for backup job #2652

Merged
merged 2 commits into from
Jun 9, 2020

Conversation

Yisaer
Copy link
Contributor

@Yisaer Yisaer commented Jun 8, 2020

What problem does this PR solve?

close #2505

As each backup job's target tidbcluster have already been set in its spec. We don't need to expose its datasource connection info in the log.

  • Need to update the documentation

Does this PR introduce a user-facing change?:

Make datasource information hidden in log for `Backup` and `Restore` job

Copy link
Contributor

@zjj2wry zjj2wry left a comment

Choose a reason for hiding this comment

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

lgtm

@Yisaer
Copy link
Contributor Author

Yisaer commented Jun 9, 2020

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Jun 9, 2020

/run-all-tests

@sre-bot sre-bot merged commit 39a8b7c into pingcap:master Jun 9, 2020
sre-bot pushed a commit to sre-bot/tidb-operator that referenced this pull request Jun 9, 2020
Signed-off-by: sre-bot <sre-bot@pingcap.com>
@sre-bot
Copy link
Contributor

sre-bot commented Jun 9, 2020

cherry pick to release-1.1 in PR #2667

Yisaer pushed a commit to Yisaer/tidb-operator that referenced this pull request Jun 9, 2020
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
Yisaer added a commit that referenced this pull request Jun 9, 2020
Signed-off-by: sre-bot <sre-bot@pingcap.com>

Co-authored-by: Song Gao <disxiaofei@163.com>
sre-bot added a commit that referenced this pull request Jun 10, 2020
* remove statefulsets webhook

* fix webhook partition

* check in e2e

debug in ci

debug ci

fix e2e checking

add log

E2e CI job use memory storage for etcd (#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 (#2652)

Co-authored-by: pingcap-github-bot <sre-bot@pingcap.com>

use crd

Update pingcap_tidb_operator_build_kind.groovy

* use RunKubectl

Co-authored-by: pingcap-github-bot <sre-bot@pingcap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

backup job will log the mysql password dsn when connection failed
4 participants