Skip to content

Allow MySQL test image to be configured via env var#3280

Merged
havetisyan merged 2 commits intoAthenZ:masterfrom
Bhuff1:feature/testcontainers_mysql_env
Apr 6, 2026
Merged

Allow MySQL test image to be configured via env var#3280
havetisyan merged 2 commits intoAthenZ:masterfrom
Bhuff1:feature/testcontainers_mysql_env

Conversation

@Bhuff1
Copy link
Copy Markdown
Contributor

@Bhuff1 Bhuff1 commented Apr 6, 2026

Description

Certain CI/CD environments cannot pull from Docker Hub. This PR allows the option to specify a different endpoint and image for MySQL. Solution for issue opened here.

Contribution Checklist:

  • The pull request does not introduce any breaking changes
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.

Attach Screenshots (Optional)

image

Signed-off-by: Benjamin Huff <benjaminehuff@gmail.com>
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces the ability to configure the MySQL Docker image used in ZMS tests via the ZMS_TEST_MYSQL_IMAGE environment variable, falling back to a default version if the variable is not set. A review comment suggests using the newly defined constant MYSQL_IMAGE_ENV_VAR within the getMySQLImage method to ensure consistency and maintainability.

Signed-off-by: Benjamin Huff <benjaminehuff@gmail.com>
Copy link
Copy Markdown
Collaborator

@havetisyan havetisyan left a comment

Choose a reason for hiding this comment

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

Thanks

@havetisyan havetisyan merged commit d8f5119 into AthenZ:master Apr 6, 2026
3 checks 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.

2 participants