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

[nova] Add ProxySQL side-cars #3579

Merged
merged 1 commit into from
Jul 22, 2022
Merged

[nova] Add ProxySQL side-cars #3579

merged 1 commit into from
Jul 22, 2022

Conversation

fwiesel
Copy link
Member

@fwiesel fwiesel commented Jun 2, 2022

This makes use of the proxysql side-car snippets.

@fwiesel fwiesel force-pushed the nova_proxy_sql_side_car branch 2 times, most recently from 84860b5 to 1787bf7 Compare June 17, 2022 07:03
@fwiesel fwiesel changed the title WIP: [nova] Add ProxySQL side-cars [nova] Add ProxySQL side-cars Jul 6, 2022
@fwiesel fwiesel marked this pull request as ready for review July 6, 2022 11:28
@fwiesel fwiesel force-pushed the nova_proxy_sql_side_car branch 2 times, most recently from 796066f to e4b1de4 Compare July 13, 2022 09:13
openstack/nova/values.yaml Outdated Show resolved Hide resolved
openstack/nova/values.yaml Outdated Show resolved Hide resolved
openstack/nova/templates/cell2-conductor-deployment.yaml Outdated Show resolved Hide resolved
openstack/nova/templates/bin/_db-online-migrate.tpl Outdated Show resolved Hide resolved
openstack/nova/templates/bin/_db-update-cells.tpl Outdated Show resolved Hide resolved
@fwiesel fwiesel force-pushed the nova_proxy_sql_side_car branch 2 times, most recently from b01160e to 4638d80 Compare July 13, 2022 11:47
@fwiesel fwiesel added proxysql-sidecar Adding ProxySql as a Sidecar reliability enhancements Changes targeted to improve the reliabilty of the services labels Jul 20, 2022
@fwiesel fwiesel force-pushed the nova_proxy_sql_side_car branch 5 times, most recently from fa3a2ce to b54bb0e Compare July 21, 2022 14:56
@fwiesel fwiesel force-pushed the nova_proxy_sql_side_car branch 2 times, most recently from dcde56d to 662b17c Compare July 22, 2022 09:39
Since nova stores & reads the db url from the database,
one cannot simply set proxysql.mode=unix_socket,
as first all pods need to have a proxysql sidecar.
First one needs to roll-out with
  proxysql.mode=host_alias
and after all pods with db access have a proxysql sidecar,
one can switch with second deployment to unix_socket
@fwiesel fwiesel merged commit fab112f into master Jul 22, 2022
@fwiesel fwiesel deleted the nova_proxy_sql_side_car branch July 22, 2022 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proxysql-sidecar Adding ProxySql as a Sidecar reliability enhancements Changes targeted to improve the reliabilty of the services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants