jetstack / navigator

Managed Database-as-a-Service (DBaaS) on Kubernetes
Apache License 2.0
271 stars 31 forks source link

Add validation to only allow Cassandra major version 3 #321

Closed wallrj closed 6 years ago

wallrj commented 6 years ago

Fixes: #320 Release note:

NONE
wallrj commented 6 years ago

/retest

wallrj commented 6 years ago

Hey @munnerz.

Please take another look.

I can follow this up with PRs to

wallrj commented 6 years ago

/retest

wallrj commented 6 years ago

/retest

wallrj commented 6 years ago

The navigator-e2e-v1-9 E2E test failure was a flake, described here: https://github.com/jetstack/test-infra/issues/180

munnerz commented 6 years ago

/lgtm /approve

jetstack-bot commented 6 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: munnerz

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/jetstack/navigator/blob/master/OWNERS)~~ [munnerz] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment