Skip to content

Fix: Use legacy Bitnami registry for MySQL dependency to resolve CI failure#47

Merged
tananaev merged 2 commits into
traccar:mainfrom
nareddyt:fix-mysql-legacy-image
Jun 12, 2026
Merged

Fix: Use legacy Bitnami registry for MySQL dependency to resolve CI failure#47
tananaev merged 2 commits into
traccar:mainfrom
nareddyt:fix-mysql-legacy-image

Conversation

@nareddyt

Copy link
Copy Markdown
Contributor

Fixes #42.

This PR updates the default MySQL image repository to bitnamilegacy/mysql to resolve the ImagePullBackOff failure in CI caused by Bitnami's repository restructuring (which moved version-pinned tags to a legacy namespace).

This maintains compatibility for existing users without forcing a major version upgrade of MySQL.

@nareddyt

Copy link
Copy Markdown
Contributor Author

PTAL @tananaev

mysql:
enabled: true
image:
repository: bitnamilegacy/mysql

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Isn't this basically no longer supported?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yup.

But I don't know enough about mysql to fix this. I don't even use this feature of the chart (I use postgres).

My intention is to fix CI without breaking any users of mysql.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

#44 by @rossigee is probably a better attempt to fix this, but is completely breaks the values for pre-existing users.

I suggest: Let's focus this PR on getting CI unblocked

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I agree that we should focus on unblocking CI, but this doesn't seem like a good fix. This seems like a simpler hack and not a proper solution.

@nareddyt nareddyt Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The issue is that Bitnami removed all free versioned tags from their main repo, so the default chart is currently broken. https://www.reddit.com/r/kubernetes/comments/1mjx86p/comment/n7ep0il/

Two options to fix it:

  1. This PR (non-breaking): Use bitnamilegacy. It's a workaround, but keeps the chart backward-compatible. IMO it is not a hack, just a badly maintained solution.
  2. PR Fix: Replace bitnami/mysql with groundhog2k/mysql (official Docker image) #44: Replace the subchart with groundhog2k/mysql (uses official MySQL image). This is cleaner long-term, but changes the configuration schema (breaking existing values).

If you want to fix this the proper way, let's merge #44 and ensure users know about the breaking change.

I still prefer we fix it with this workaround so users don't need to update their values. Keep in mind that nobody is updating the bitnami chart dependencies regularly in this repo anyways. So moving to a proper maintained helm chart doesn't really add much benefit...

@nareddyt
nareddyt requested a review from tananaev June 12, 2026 15:52
@tananaev

Copy link
Copy Markdown
Member

Looks like it's failing.

@nareddyt

Copy link
Copy Markdown
Contributor Author

Fixed, PTAL

@tananaev
tananaev merged commit fe5033e into traccar:main Jun 12, 2026
1 check passed
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.

CI tests failing due to missing mysql image

2 participants