Skip to content

Commit

Permalink
el9to10: actors: mysql: Replace subprocess run with leapp wrapper
Browse files Browse the repository at this point in the history
  • Loading branch information
SlouchyButton committed Jan 23, 2025
1 parent 83e4219 commit 72d24e7
Showing 1 changed file with 3 additions and 5 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from leapp.models import DistributionSignedRPM, MySQLConfiguration
from leapp.libraries.common.rpms import has_package
from leapp.libraries.stdlib import api
from leapp.libraries.stdlib import api, run

import subprocess

Expand Down Expand Up @@ -43,11 +43,9 @@ def _check_incompatible_config() -> set[str]:
"""

found_options = set()
out = subprocess.run(['mysqld', '--validate-config', '--log-error-verbosity=2'],
capture_output=True,
check=False)
stderr = run(['mysqld', '--validate-config', '--log-error-verbosity=2'],
checked=False)['stderr']

stderr = out.stderr.decode("utf-8")
if 'deprecated' in stderr:
found_options = {arg for arg
in REMOVED_ARGS
Expand Down

0 comments on commit 72d24e7

Please sign in to comment.