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

Fix master branch #5461

Merged
merged 1 commit into from
Nov 25, 2019
Merged

Fix master branch #5461

merged 1 commit into from
Nov 25, 2019

Conversation

morgo
Copy link
Contributor

@morgo morgo commented Nov 25, 2019

The test broke because of the interaction between two PRs: #5346 and #5427

This restores the intended behavior, which is that Vitess does not package etcd and looks for it in the $PATH.

Signed-off-by: Morgan Tocker [email protected]

Signed-off-by: Morgan Tocker <[email protected]>
@morgo morgo requested a review from sougou as a code owner November 25, 2019 18:16
@morgo morgo changed the title Fixes master branch Fix master branch Nov 25, 2019
@morgo morgo merged commit c75a8e7 into vitessio:master Nov 25, 2019
@morgo morgo deleted the morgo-fix-master branch November 25, 2019 19:09
@morgo morgo mentioned this pull request Nov 26, 2019
systay pushed a commit that referenced this pull request Jul 22, 2024
* Vectors: Online DDL support

Signed-off-by: Shlomi Noach <[email protected]>

* Add support for vectors in vreplication workflows. unit and e2e tests. Update cnf file to add loose_with_ps_vectors

Signed-off-by: Rohit Nayak <[email protected]>

* Self review fixes. Add e2e TestVector to CI

Signed-off-by: Rohit Nayak <[email protected]>

* Simplify unit test CI workflow to debug failure

Signed-off-by: Rohit Nayak <[email protected]>

* Skip test on errors on setting with_ps_vectors

Signed-off-by: Rohit Nayak <[email protected]>

* Revert debug mods to unit_race.yml

Signed-off-by: Rohit Nayak <[email protected]>

* Unit test race is again failing with no logs. So modify CI workflow temporarily to get full detail to debug random failures apparently unrelated to this PR

Signed-off-by: Rohit Nayak <[email protected]>

* Setup templating for vectors vreplication build

Signed-off-by: Dirkjan Bussink <[email protected]>

* Add vcopier unit test. Insert additional rows in e2e test after copy phase to test replication phase

Signed-off-by: Rohit Nayak <[email protected]>

* Switch shard for vector test to correct one

Signed-off-by: Rohit Nayak <[email protected]>

* Use vectors build for unit tests

Signed-off-by: Dirkjan Bussink <[email protected]>

* Refactor tests: move vector unit tests to separate files

Signed-off-by: Rohit Nayak <[email protected]>

* endtoend suite: filter with ONLINEDDL_SUITE_TEST_FILTER

Signed-off-by: Shlomi Noach <[email protected]>

* recognize ERVectorDimensionsMismatch      = ErrorCode(4501) as unrecoverable

Signed-off-by: Shlomi Noach <[email protected]>

* add expected failign vector test. Rename vector tests

Signed-off-by: Shlomi Noach <[email protected]>

* recognize ERVectorDimensionsMismatch      = ErrorCode(4501) as unrecoverable

Signed-off-by: Shlomi Noach <[email protected]>

* Unit race still failing without details:-(. Commenting out the summary code which seems to be redacting the details

Signed-off-by: Rohit Nayak <[email protected]>

* 'onlineddl_vrepl_suite' now requires a vectors build

Signed-off-by: Shlomi Noach <[email protected]>

* use vectors build

Signed-off-by: Shlomi Noach <[email protected]>

* update mysql version

Signed-off-by: Shlomi Noach <[email protected]>

---------

Signed-off-by: Shlomi Noach <[email protected]>
Signed-off-by: Rohit Nayak <[email protected]>
Signed-off-by: Dirkjan Bussink <[email protected]>
Co-authored-by: Rohit Nayak <[email protected]>
Co-authored-by: Dirkjan Bussink <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants