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

Add performance tuning and reporting in Ansible config #166

Merged
merged 3 commits into from
Jul 19, 2024
Merged

Conversation

m-bull
Copy link
Contributor

@m-bull m-bull commented Jul 18, 2024

  • Enable SSH pipelining: This results in almost a 50% reduction in runtime on a no-op deployment
  • Add task profiling: Allows simple viewing of how long tasks are taking
  • Add stdout callback formatting: Format task stdouts as YAML

Add these configurations to example and CI environments.

@m-bull m-bull requested a review from mkjpryor July 18, 2024 08:34
m-bull added 2 commits July 18, 2024 10:28
When the profile-tasks callback is enabled, it adds non-JSON lines
to the JSON stdout callback output. Set JSON_INDENT to 0 to
ensure that ansible JSON is output on a single line, and make jq
robust to parsing mixed JSON/non-JSON data.
Copy link
Collaborator

@mkjpryor mkjpryor left a comment

Choose a reason for hiding this comment

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

LGTM. Test failure is not related.

@mkjpryor mkjpryor merged commit df47e80 into devel Jul 19, 2024
1 of 2 checks passed
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.

2 participants