New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Start simplify system tests #6906
Conversation
This requires parameterized on the windows workers. I opened a PR for the changes. |
1d372b4
to
ca73d74
Compare
@@ -65,7 +65,7 @@ def assert_no_logged_warnings(self, replace=None): | |||
""" | |||
log = self.get_log() | |||
|
|||
pattern = self.build_log_regex("[cfgwarn]") | |||
pattern = self.build_log_regex("\[cfgwarn\]") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was actually a bug in our code and removed all WARN and ERROR entries from the log :-(
The idea of this PR is to start to play around with ways to simplify our system tests are remove most of the duplicated code. * Use check_metricset method for aerospike test * Add fields check to check_metricset method * Use parameterized for ceph and couchbase tests Further fixes: * Increase timeout for k8s tests * Fix regexp to remove cfgwarn warnings. The removal removed all WARN / ERROR messages. * Fix flaky auditbeat test by skipping the error message.
self.assertTrue("pool_disk" in evt["ceph"]) | ||
|
||
self.assert_fields_are_documented(evt) | ||
self.check_metricset("ceph", metricset, self.get_hosts(), self.FIELDS) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️
jenkins, test this |
The idea of this PR is to start to play around with ways to simplify our system tests are remove most of the duplicated code.
Further fixes: