etcd-druid
etcd-druid copied to clipboard
Enable Druid to use mock GCS server to run e2e tests
How to categorize this PR?
/area dev-productivity /area testing /kind test
What this PR does / why we need it:
This PR enables druid to run e2e tests for GCS locally with the help of a GCS Emulator called fake-gcs-server
.
To run e2e tests with GCS Emulator as object store, do make ci-e2e-kind-gcs
.
Which issue(s) this PR fixes: Fixes #738
Special notes for your reviewer:
Release note:
Enable Druid to use mock GCS server to run e2e tests using "make ci-e2e-kind-gcs"
@anveshreddy18: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
pull-etcd-druid-e2e-kind-nondistroless-etcd | 917b4b7ede70aeb36a2b13677fe65c22bb3034ad | link | true | /test pull-etcd-druid-e2e-kind-nondistroless-etcd |
Full PR test history. Your PR dashboard. Command help for this repository. Please help us cut down on flakes by linking this test failure to an open flake report or filing a new flake report if you can't find an existing one. Also see our testing guideline for how to avoid and hunt flakes.
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.
@anveshreddy18 You need rebase this pull request with latest master branch. Please check.
Thanks for the review @seshachalam-yv, can you pls test now. Earlier there was a hashtag infront of the svcacc.json field in the secret, hence you saw the failure to read the creds. I've now removed that, and updated the docs. Pls follow the latest doc to run it successfully.
Could you create a new PR to bump the version of etcd-backup-restore?
/retest
/hold
Needs some changes as etcdbr
will be modified to depend on mounted secret instead of env vars from druid for endpoint
& emulatorEnabled
vars. That needs to be accommodated here as well.