-
Notifications
You must be signed in to change notification settings - Fork 908
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
Some miscellaneous integration test fixes (SC-606) #1103
Some miscellaneous integration test fixes (SC-606) #1103
Conversation
ab3c669
to
f6a40cd
Compare
- Added to list of expected warnings on Oracle when opc user has no ssh key - Added retries to tests that read from syslog as that can sometimes take time to reflect in the log - Updated test_apt.py to remove proxy info into its own test as that can cause failures in updating, which will immediately traceback out of the module and prevent us from running further class tests - Updated test_apt.py to use a more updated ppa in the test_keyserver - Added basic rsyslog test to test_combined.py - Added basic puppet test as test_puppet.py
f6a40cd
to
78c8019
Compare
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.
Nice integration test adds. Minor typo, and future suggestion for cloud-awareness in verify_clean_logs.
# - name: opc | ||
# ssh_redirect_user: true | ||
# This can trigger a warning about opc having no public key | ||
warning_texts.append( |
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.
Not a big deal for this integration test, but seems like we might want instance.cloud awareness in verify_clean_logs for azure vs oracle cases. But, this can wait until we have a third use-case.
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.
Looks good to me. Glad to see the retry decorators and addition of the puppet tests. The rest looks good too. Nothing critical from me. Thanks!
@blackboxsw , Fixed the typo |
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.
+1
Proposed Commit Message