-
Notifications
You must be signed in to change notification settings - Fork 22
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
support k8s 1.30 and 1.31 #727
Conversation
72fe9bf
to
a331901
Compare
5aa4569
to
42d6033
Compare
d2a05f8
to
652434e
Compare
52f8e38
to
b749124
Compare
b749124
to
99eb50c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
.github/workflows/ci-e2e.yaml
Outdated
@@ -20,12 +20,12 @@ jobs: | |||
name: Integration tests with MySQL | |||
strategy: | |||
matrix: | |||
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.0"] | |||
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.0", "8.4.2"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't our policy support only the latest LTS release?
I mean, we do not need to keep 8.4.0
here.
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.0", "8.4.2"] | |
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.2"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also write 8.0.39
.github/workflows/ci-e2e.yaml
Outdated
if: steps.changed-files.outputs.any_changed == 'true' | ||
|
||
e2e-mysql: | ||
name: Supported MySQL versions End-to-End Tests | ||
strategy: | ||
matrix: | ||
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.0"] | ||
k8s-version: ["1.29.4"] | ||
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.0", "8.4.2"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
DEVELOP.md
Outdated
@@ -84,23 +84,23 @@ MySQL versions appear twice: | |||
name: Integration tests with MySQL | |||
strategy: | |||
matrix: | |||
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.0"] | |||
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.0", "8.4.2"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
DEVELOP.md
Outdated
... | ||
# Matrix tests for different MySQL versions on the latest supported Kubernetes version. | ||
e2e-mysql: | ||
name: Supported MySQL versions End-to-End Tests | ||
strategy: | ||
matrix: | ||
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.0"] | ||
k8s-version: ["1.29.4"] | ||
mysql-version: ["8.0.28", "8.0.36", "8.0.37", "8.4.0", "8.4.2"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
README.md
Outdated
@@ -26,8 +26,8 @@ Blog article: [Introducing MOCO, a modern MySQL operator on Kubernetes](https:// | |||
|
|||
## Supported software | |||
|
|||
- MySQL: 8.0.28, 8.0.36, 8.0.37, 8.4.0 | |||
- Kubernetes: 1.27, 1.28, 1.29 | |||
- MySQL: 8.0.28, 8.0.36, 8.0.37, 8.4.0, 8.4.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- MySQL: 8.0.28, 8.0.36, 8.0.37, 8.4.0, 8.4.2 | |
- MySQL: 8.0.28, 8.0.36, 8.0.37, 8.4.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
1f2cc5f
to
90c91a6
Compare
90c91a6
to
3f003fe
Compare
There are no changes that will cause StatafulSet to restart since latest release. So I reverted update of moco-agent. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.