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

Upgrade Vitess dependency to 14.0.0 #280

Merged
merged 9 commits into from
Jul 7, 2022
Merged

Conversation

GuptaManan100
Copy link
Collaborator

Description

This PR upgrades the operator dependency to release 14.0.0. As part of this upgrade, we also clean up all the TODOs inserted during the version upgrade to 13.0.0 #263

@GuptaManan100 GuptaManan100 added the enhancement New feature or request label Jul 6, 2022
@GuptaManan100
Copy link
Collaborator Author

The changes in this PR are on top of #279 and should be merged after it.

@GuptaManan100 GuptaManan100 marked this pull request as ready for review July 6, 2022 12:23
Copy link
Contributor

@Verolop Verolop left a comment

Choose a reason for hiding this comment

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

@GuptaManan100 GuptaManan100 force-pushed the update-vitess-dependency-14 branch from f6b7fcb to ec2291f Compare July 7, 2022 12:39
@GuptaManan100 GuptaManan100 merged commit feb2c04 into main Jul 7, 2022
@GuptaManan100 GuptaManan100 deleted the update-vitess-dependency-14 branch July 7, 2022 14:47
@frouioui
Copy link
Member

frouioui commented Jul 7, 2022

This is awesome, thank you @GuptaManan100!

GuptaManan100 added a commit that referenced this pull request Jul 28, 2022
* feat: upgrade vitess dependency to v14

Signed-off-by: Manan Gupta <[email protected]>

* feat: fix build issues by removing call to SetKeyspaceDurabilityPolicy which is no longer required

Signed-off-by: Manan Gupta <[email protected]>

* feat: cleanup semi-sync parameters being passed. Use durability policies for the same now

Signed-off-by: Manan Gupta <[email protected]>

* feat: upgrade test file to reflect vitess version upgrade

Signed-off-by: Manan Gupta <[email protected]>

* ci: upgrade go version to 1.18.3 for ci

Signed-off-by: Manan Gupta <[email protected]>

* ci: upgrade go version to 1.18.3 for github actions

Signed-off-by: Manan Gupta <[email protected]>

* feat: increment go version in the docker file too

Signed-off-by: Manan Gupta <[email protected]>

* feat: also upgrade vitess-operator to 2.7 in upgrade test

Signed-off-by: Manan Gupta <[email protected]>

* feat: remove a deprecated flag

Signed-off-by: Manan Gupta <[email protected]>
GuptaManan100 added a commit that referenced this pull request Jul 28, 2022
* Upgrade Vitess dependency to 14.0.0 (#280)

* feat: upgrade vitess dependency to v14

Signed-off-by: Manan Gupta <[email protected]>

* feat: fix build issues by removing call to SetKeyspaceDurabilityPolicy which is no longer required

Signed-off-by: Manan Gupta <[email protected]>

* feat: cleanup semi-sync parameters being passed. Use durability policies for the same now

Signed-off-by: Manan Gupta <[email protected]>

* feat: upgrade test file to reflect vitess version upgrade

Signed-off-by: Manan Gupta <[email protected]>

* ci: upgrade go version to 1.18.3 for ci

Signed-off-by: Manan Gupta <[email protected]>

* ci: upgrade go version to 1.18.3 for github actions

Signed-off-by: Manan Gupta <[email protected]>

* feat: increment go version in the docker file too

Signed-off-by: Manan Gupta <[email protected]>

* feat: also upgrade vitess-operator to 2.7 in upgrade test

Signed-off-by: Manan Gupta <[email protected]>

* feat: remove a deprecated flag

Signed-off-by: Manan Gupta <[email protected]>

* feat: revert upgrading vitess versions in test

Signed-off-by: Manan Gupta <[email protected]>

* feat: upgrade go dependency

Signed-off-by: Manan Gupta <[email protected]>

* feat:upgrade vitess dependency

Signed-off-by: Manan Gupta <[email protected]>

* feat: fix ci workflows to also run on release-branches

Signed-off-by: Manan Gupta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants