Skip to content

Comments

Initial support for running tests on AIX#52568

Merged
Ch3LL merged 1 commit intosaltstack:2018.3from
dmurphy18:aix_ssh_tests
Apr 17, 2019
Merged

Initial support for running tests on AIX#52568
Ch3LL merged 1 commit intosaltstack:2018.3from
dmurphy18:aix_ssh_tests

Conversation

@dmurphy18
Copy link
Contributor

What does this PR do?

Adds support for running tests on AIX

What issues does this PR fix or reference?

None, desire to get tests/runtests.py working on AIX to allow for ssh testing

Previous Behavior

python tests/runtests.py would fail due to inability to open a port on the localhost

New Behavior

tests/runtests.py can now run tests on AIX.
Note UTF-8 has to be installed on the AIX system, for example:
installp -acXYg -d /stage/middleware/AIX/bos/bos.loc.utf.EN_US.7.1.0.0.I all
and /etc/environment updated with LC_ALL=EN_US and the machine rebooted

Tests written?

No, now able to run python tests/runtests.py --help and other unit tests
python test/runtests.py -v -n unit/client/test_ssh.py

Commits signed with GPG?

No

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

@dmurphy18 dmurphy18 requested review from Ch3LL and thatch45 April 16, 2019 21:26
@Ch3LL Ch3LL merged commit e906aff into saltstack:2018.3 Apr 17, 2019
garethgreenaway added a commit to garethgreenaway/salt that referenced this pull request Sep 19, 2019
dwoz added a commit that referenced this pull request Dec 17, 2019
@dmurphy18 dmurphy18 deleted the aix_ssh_tests branch August 4, 2020 17:47
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.

3 participants