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.

Sign up for GitHub

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

Jump to bottom

K8SPXC-1071 - Add percona-scheduler-admin for proxysql #1198

Closed
wants to merge 33 commits into from
Closed

K8SPXC-1071 - Add percona-scheduler-admin for proxysql #1198

wants to merge 33 commits into from

Conversation

tplavcic
Copy link
Member

@tplavcic tplavcic commented Aug 4, 2022 •
edited by jira bot
Loading

@pull-request-size pull-request-size bot added the size/S 10-29 lines label Aug 4, 2022
@tplavcic tplavcic mentioned this pull request Aug 4, 2022
Draft
@tplavcic tplavcic requested review from egegunes and hors August 4, 2022 08:33
egegunes
egegunes previously approved these changes Aug 16, 2022
View reviewed changes
@tplavcic tplavcic force-pushed the proxysql-scheduler branch from 3cd8fd8 to 9ba4227 Compare August 23, 2022 16:23
@pull-request-size pull-request-size bot added size/M 30-99 lines size/XL 500-999 lines and removed size/S 10-29 lines size/M 30-99 lines labels Aug 23, 2022
@it-percona-cla
Copy link

it-percona-cla commented Aug 24, 2022 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@tplavcic tplavcic marked this pull request as ready for review August 24, 2022 17:12
@tplavcic tplavcic changed the title K8SPXC-1071 - Replace proxysql-admin with percona-scheduler-admin K8SPXC-1071 - Add percona-scheduler-admin for proxysql Aug 25, 2022
hors
hors requested changes Aug 29, 2022
View reviewed changes
Copy link
Collaborator

@hors hors left a comment

Choose a reason for hiding this comment

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

@tplavcic please check tests

@pull-request-size pull-request-size bot added size/L 100-499 lines size/XL 500-999 lines and removed size/XL 500-999 lines size/L 100-499 lines labels Sep 8, 2022
github-actions[bot]
github-actions bot reviewed Sep 16, 2022
View reviewed changes
e2e-tests/scheduled-backup/run Outdated Show resolved Hide resolved
e2e-tests/scheduled-backup/run Outdated Show resolved Hide resolved
egegunes
egegunes requested changes Oct 4, 2022
View reviewed changes
pkg/apis/pxc/v1/pxc_types.go Outdated Show resolved Hide resolved
@egegunes egegunes added this to the v1.12.0 milestone Oct 6, 2022
@pull-request-size pull-request-size bot added size/L 100-499 lines and removed size/XL 500-999 lines labels Nov 7, 2022
@egegunes
Copy link
Contributor

@tplavcic please fix conflicts

cap1984
cap1984 previously approved these changes Nov 16, 2022
View reviewed changes
Copy link
Contributor

@cap1984 cap1984 left a comment

Choose a reason for hiding this comment

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

just resolve minor conflict

inelpandzic
inelpandzic previously approved these changes Nov 16, 2022
View reviewed changes
@pull-request-size pull-request-size bot added size/XL 500-999 lines and removed size/L 100-499 lines labels Nov 16, 2022
@hors hors removed this from the v1.12.0 milestone Nov 16, 2022
@tplavcic tplavcic marked this pull request as draft November 16, 2022 16:14
@pull-request-size pull-request-size bot added size/L 100-499 lines and removed size/XL 500-999 lines labels Nov 24, 2022
@JNKPercona
Copy link
Collaborator

Test name Status
affinity-8-0 passed
auto-tuning-8-0 passed
cross-site-8-0 passed
demand-backup-8-0 passed
demand-backup-cloud-8-0 failed
demand-backup-encrypted-with-tls-8-0 passed
haproxy-5-7 failed
haproxy-8-0 passed
init-deploy-5-7 passed
init-deploy-8-0 passed
limits-8-0 passed
monitoring-2-0-8-0 passed
one-pod-5-7 passed
one-pod-8-0 passed
pitr-8-0 passed
proxy-protocol-8-0 passed
proxysql-sidecar-res-limits-8-0 passed
recreate-8-0 passed
restore-to-encrypted-cluster-8-0 passed
scaling-8-0 passed
scaling-proxysql-8-0 passed
scheduled-backup-5-7 passed
scheduled-backup-8-0 passed
security-context-8-0 passed
smart-update1-8-0 passed
smart-update2-8-0 passed
storage-8-0 passed
tls-issue-cert-manager-8-0 passed
tls-issue-cert-manager-ref-8-0 passed
tls-issue-self-8-0 passed
upgrade-consistency-8-0 failed
upgrade-haproxy-8-0 passed
upgrade-proxysql-8-0 failed
users-5-7 passed
users-8-0 passed
validation-hook-8-0 passed
We run 36 out of 36

commit: 6f56ee0
image: perconalab/percona-xtradb-cluster-operator:PR-1198-6f56ee03

@tplavcic tplavcic closed this Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@nmarukovich nmarukovich nmarukovich left review comments

@inelpandzic inelpandzic inelpandzic left review comments

@cap1984 cap1984 cap1984 left review comments

@pooknull pooknull Awaiting requested review from pooknull pooknull is a code owner

@hors hors Awaiting requested review from hors hors is a code owner

@egegunes egegunes Awaiting requested review from egegunes egegunes is a code owner

@ptankov ptankov Awaiting requested review from ptankov ptankov will be requested when the pull request is marked ready for review ptankov is a code owner

@jvpasinatto jvpasinatto Awaiting requested review from jvpasinatto jvpasinatto will be requested when the pull request is marked ready for review jvpasinatto is a code owner

@eleo007 eleo007 Awaiting requested review from eleo007 eleo007 will be requested when the pull request is marked ready for review eleo007 is a code owner

Assignees
No one assigned
Labels
size/L 100-499 lines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@tplavcic @it-percona-cla @egegunes @JNKPercona @hors @inelpandzic @nmarukovich @cap1984