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: perconas haproxy default cfg #1211

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

fitbeard
Copy link
Contributor

Since we are now using version 1.14.0 of the operator and default haproxy tag from it is also 1.14.0 we need to update haproxy config to new version. Without kubernetes resolver startup of haproxy pods takes much longer. And yes, it has nothing to do with crVersion: "1.13.0" from spec itself:)

@mnaser
Copy link
Member

mnaser commented May 14, 2024

ugh, I used to actually have a golang unit test that isn't running between the GHA/Jenkins/Zuul dance that would have caught this.

Do you have time to port that test to Python instead or would you like me to if you don't have spare time @fitbeard ?

@fitbeard
Copy link
Contributor Author

@mnaser I need a working example to understand whole logic. Maybe you can create tests this time? When I have source for reference i'm very productive copy-cat'er:)

@okozachenko1203
Copy link
Member

@fitbeard there are var_test.go files in roles and have a task in earthfile https://github.com/vexxhost/atmosphere/blob/main/Earthfile#L3-L14
This is an example test file for pxc. https://github.com/vexxhost/atmosphere/blob/main/roles/percona_xtradb_cluster/vars_test.go

btw, @mnaser i updated the current go unit test code to avoid version mismatching between image ref and cluster spec.CRVersion bf5ddfa
And also added go unit test in the zuul CI. #1983

@mnaser
Copy link
Member

mnaser commented Oct 21, 2024

/cherry-pick stable/2024.1

@vexxhost-bot
Copy link
Collaborator

@mnaser: once the present PR merges, I will cherry-pick it on top of stable/2024.1 in a new PR and assign it to you.

In response to this:

/cherry-pick stable/2024.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@mnaser
Copy link
Member

mnaser commented Oct 21, 2024

/cherry-pick stable/2023.2

@vexxhost-bot
Copy link
Collaborator

@mnaser: once the present PR merges, I will cherry-pick it on top of stable/2023.2 in a new PR and assign it to you.

In response to this:

/cherry-pick stable/2023.2

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@mnaser
Copy link
Member

mnaser commented Oct 21, 2024

/cherry-pick stable/2023.1

@vexxhost-bot
Copy link
Collaborator

@mnaser: once the present PR merges, I will cherry-pick it on top of stable/2023.1 in a new PR and assign it to you.

In response to this:

/cherry-pick stable/2023.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@mnaser mnaser added the gate label Oct 21, 2024
@atmosphere-ci atmosphere-ci bot merged commit c7a54c0 into vexxhost:main Oct 21, 2024
4 checks passed
@vexxhost-bot
Copy link
Collaborator

@mnaser: new pull request created: #1988

In response to this:

/cherry-pick stable/2024.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@vexxhost-bot
Copy link
Collaborator

@mnaser: new pull request created: #1989

In response to this:

/cherry-pick stable/2023.2

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@vexxhost-bot
Copy link
Collaborator

@mnaser: new pull request created: #1990

In response to this:

/cherry-pick stable/2023.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

atmosphere-ci bot pushed a commit that referenced this pull request Oct 21, 2024
This is an automated cherry-pick of #1211
/assign mnaser
atmosphere-ci bot pushed a commit that referenced this pull request Oct 21, 2024
This is an automated cherry-pick of #1211
/assign mnaser
atmosphere-ci bot pushed a commit that referenced this pull request Oct 21, 2024
This is an automated cherry-pick of #1211
/assign mnaser
atmosphere-ci bot pushed a commit that referenced this pull request Oct 30, 2024
remove 1.22 from TestKubeconform test (node-feature-discovery uses grpc in probe which is available since 1.22)
add a workaround values in TestKubeconform for loki chart
skip CephFilesystemSubVolumeGroup kind for ceph chart
fix oslo_db test expected values
update TestImageExist test to support image tags without digest
fix pxc test codes
supplement placement chart value for database max_retries param

depends-on: #1977
depends-on: #1211
depends-on: #1987 #1211 #1977

Reviewed-by: Mohammed Naser <mnaser@vexxhost.com>
Reviewed-by: Oleksandr K.
okozachenko1203 added a commit that referenced this pull request Oct 31, 2024
remove 1.22 from TestKubeconform test (node-feature-discovery uses grpc in probe which is available since 1.22)
add a workaround values in TestKubeconform for loki chart
skip CephFilesystemSubVolumeGroup kind for ceph chart
fix oslo_db test expected values
update TestImageExist test to support image tags without digest
fix pxc test codes
supplement placement chart value for database max_retries param

depends-on: #1977
depends-on: #1211
depends-on: #1987 #1211 #1977

Reviewed-by: Mohammed Naser <mnaser@vexxhost.com>
Reviewed-by: Oleksandr K.
okozachenko1203 added a commit that referenced this pull request Oct 31, 2024
remove 1.22 from TestKubeconform test (node-feature-discovery uses grpc in probe which is available since 1.22)
add a workaround values in TestKubeconform for loki chart
skip CephFilesystemSubVolumeGroup kind for ceph chart
fix oslo_db test expected values
update TestImageExist test to support image tags without digest
fix pxc test codes
supplement placement chart value for database max_retries param

depends-on: #1977
depends-on: #1211
depends-on: #1987 #1211 #1977

Reviewed-by: Mohammed Naser <mnaser@vexxhost.com>
Reviewed-by: Oleksandr K.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants