Skip to content

Commit

Permalink
Comment rewording
Browse files Browse the repository at this point in the history
  • Loading branch information
aiden3c committed Oct 8, 2024
1 parent aa0f49f commit 13e2614
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/api2/test_openssl.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
@pytest.mark.skipif(not ha, reason='Test only valid for HA')
def test_fips_version():
# The reason we have a set of commands in a payload is because of some annoying FIPS technicalities.
# Basically, when FIPS is enabled, we can't use SSH because the SSH key used by root isn't using a FIPS provided algorithm.
# Basically, when FIPS is enabled, we can't use SSH because the SSH key used by root isn't using a FIPS provided algorithm. (this might need to be investigated further)
# To allow testing, we write our FIPS information to a file during this phase, and then go disable FIPS to get SSH back all in one joint command.
payload = """midclt call --job system.security.update '{"enable_fips": true}' && openssl list -providers > /root/osslproviders && midclt call system.reboot.info >> /root/osslproviders && midclt call --job system.security.update '{"enable_fips": false}'"""
for i in range(retry):
Expand All @@ -24,7 +24,7 @@ def test_fips_version():
request = ssh(payload, complete_response=True, timeout=300) # fail the test

enabled_info = ssh("cat /root/osslproviders")
# Check that things are what we expect when fips was enabled
# Check that things are what we expect when FIPS was enabled
assert "3.0.9" in enabled_info
assert "FIPS configuration was changed." in enabled_info

Expand Down

0 comments on commit 13e2614

Please sign in to comment.