Skip to content
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

Merged
merged 2 commits into from
Nov 15, 2021

Conversation

TheRealFalcon
Copy link
Member

@TheRealFalcon TheRealFalcon commented Nov 9, 2021

Proposed Commit Message

Some miscellaneous integration test fixes

- 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

@TheRealFalcon TheRealFalcon added the wip Work in progress, do not land label Nov 9, 2021
@TheRealFalcon TheRealFalcon force-pushed the integration-test-fixes branch 4 times, most recently from ab3c669 to f6a40cd Compare November 11, 2021 22:39
- 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
@TheRealFalcon TheRealFalcon removed the wip Work in progress, do not land label Nov 14, 2021
@TheRealFalcon TheRealFalcon changed the title WIP: Some miscellaneous integration test fixes Some miscellaneous integration test fixes Nov 14, 2021
@TheRealFalcon TheRealFalcon changed the title Some miscellaneous integration test fixes Some miscellaneous integration test fixes (SC-606) Nov 14, 2021
Copy link
Collaborator

@blackboxsw blackboxsw left a 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.

tests/integration_tests/modules/test_puppet.py Outdated Show resolved Hide resolved
# - name: opc
# ssh_redirect_user: true
# This can trigger a warning about opc having no public key
warning_texts.append(
Copy link
Collaborator

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.

Co-authored-by: Chad Smith <chad.smith@canonical.com>
Copy link
Member

@holmanb holmanb left a 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!

@TheRealFalcon
Copy link
Member Author

@blackboxsw , Fixed the typo

Copy link
Collaborator

@blackboxsw blackboxsw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@blackboxsw blackboxsw merged commit f0af9f7 into canonical:main Nov 15, 2021
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