Skip to content

Conversation

@rnagy
Copy link

@rnagy rnagy commented Dec 10, 2025

What does this PR do?

Fixes the version report by using the proper attribute for MySQLdb.

Previous Behavior

$ salt -V | grep mysql-pyt
  mysql-python: Not Installed

New Behavior

$ salt -V | grep mysql-pyt
  mysql-python: 2.2.7.final.0

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

No

@rnagy rnagy requested a review from a team as a code owner December 10, 2025 10:36
@twangboy twangboy added the test:full Run the full test suite label Dec 10, 2025
@twangboy twangboy added this to the Sulfur v3006.18 milestone Dec 10, 2025
@twangboy
Copy link
Contributor

Could you rebase this PR on the 3006.x branch and create a test for it, please?

@rnagy rnagy changed the base branch from master to 3006.x December 12, 2025 12:55
@rnagy
Copy link
Author

rnagy commented Dec 12, 2025

rebased on 3006.x but i am unsure if this warrants an actual test case, if it does please tell me where exactly

@twangboy
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants