search

tempesta-tech / tempesta-test

Test suite for Tempesta FW
11 stars 4 forks source link

plain configs for multiple listeners #275

Closed RomanBelozerov closed 2 years ago

RomanBelozerov commented 2 years ago

I added plain configs for multiple listeners. And also changed path for tls certificates to tmp/tempesta. I left all the tests disabled

nickzaev commented 2 years ago

Also don't forget to delete this test suite from tests_disabled.json when it passes.

RomanBelozerov commented 2 years ago

With the changes required multiple_listeners.test_multiple_listening passes, but two other ones fail. Do they pass for you?

No. Only `multiple_listeners.test_multiple_listening' passes for me.

nickzaev commented 2 years ago

No. Only `multiple_listeners.test_multiple_listening' passes for me.

Is there an issue filed regarding those fails? Do you have any clues on why them fail?

RomanBelozerov commented 2 years ago

No. Only `multiple_listeners.test_multiple_listening' passes for me.

Is there an issue filed regarding those fails? Do you have any clues on why them fail?

Yes, issue #1624. test_mixed failed for this reason. I have tried to understand the reason for fail of test_on_the_fly. But I don't know it yet.

RomanBelozerov commented 2 years ago

No need to fix anything for now, but please in future pull requests leave more detailed commit messages. For example returned client wrk in test_on_the_fly.py doesn't explain how this fixes the reported issue.

OK, I'll write more detailed commit messages. I rebased from master and did force push, it should work now.