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

Use charmed-mysql rock instead of the docker image #55

Closed
wants to merge 1 commit into from

Conversation

shayancanonical
Copy link
Contributor

dpe-1628

Issue

We are currently using a docker image in the mysqlrouter charm, but we would instead like to use the charmed-mysql snap instead

Solution

Use the charmed-mysql rock by:

  1. running mysqlrouter --bootstrap when the database is ready (WIP)
  2. running mysqlrouter --config <config_file> with pebble

@@ -30,7 +30,6 @@ resources:
mysql-router-image:
type: oci-image
description: OCI image for mysql-router
# TODO: replace with canonical maintained image
upstream-source: dataplatformoci/mysql-router:8.0-22.04_edge
upstream-source: ghcr.io/canonical/charmed-mysql@sha256:924cb913733002e0936a36b6edab0bb8a4d72f1b92be0e0b7a933e797b7ef215
Copy link
Contributor Author

Choose a reason for hiding this comment

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

TODO: update the image hash once canonical/charmed-mysql-rock#23 is merged

--conf-set-option http_server.bind_address=127.0.0.1
--conf-use-gr-notifications
--force
""".split()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we need to bootstrap mysqlrouter when the application is scaled to more than 1 unit

carlcsaposs-canonical added a commit that referenced this pull request Apr 25, 2023
carlcsaposs-canonical added a commit that referenced this pull request Apr 25, 2023
@carlcsaposs-canonical carlcsaposs-canonical mentioned this pull request Apr 25, 2023
@carlcsaposs-canonical
Copy link
Contributor

Superseded by #51

@carlcsaposs-canonical carlcsaposs-canonical deleted the feature/charmed_mysql_rock branch May 15, 2023 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants