-
Notifications
You must be signed in to change notification settings - Fork 16
ci: rollout several recent changes to CI testing #180
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
Conversation
* Pass in a YAML true value as `__bootc_validation: true` using
the --extra-vars option to ensure that `__bootc_validation` is
treated as a boolean and not a string value.
`-e "__bootc_validation: true"`
You can also use JSON format:
`-e '{"__bootc_validation": true}'`
but YAML is simpler in this case.
* Use tox-lsr version 3.11.1
* Ensure the citest bad comment works when the test was cancelled in
addition to the failure case.
* Update contributing.md documentation
* Update number of nodes to use in testing farm, if needed
* remove unnecessary ansible-lint skips
Signed-off-by: Rich Megginson <rmeggins@redhat.com>
Reviewer's GuideThis PR refines CI workflows by upgrading tox-lsr to 3.11.1 across all tests, changing the __bootc_validation extra-vars flag to use a YAML boolean, extending the contributing guide with local CI instructions, and updating the citest bad-comment workflow to account for cancelled runs. Sequence diagram for citest bad-comment workflow handling cancelled runssequenceDiagram
participant GitHub Actions
participant "gh api"
participant jq
participant User
User->>GitHub Actions: Trigger citest bad-comment workflow
GitHub Actions->>"gh api": Request workflow runs for PR
"gh api"->>jq: Filter runs by display_title and conclusion
jq-->>GitHub Actions: Return run ID for failure or cancelled
GitHub Actions->>User: Notify if failed/cancelled workflow found or exit
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
__bootc_validation: trueusingthe --extra-vars option to ensure that
__bootc_validationistreated as a boolean and not a string value.
-e "__bootc_validation: true"You can also use JSON format:
-e '{"__bootc_validation": true}'but YAML is simpler in this case.
Use tox-lsr version 3.11.1
Ensure the citest bad comment works when the test was cancelled in
addition to the failure case.
Update contributing.md documentation
Update number of nodes to use in testing farm, if needed
remove unnecessary ansible-lint skips
Signed-off-by: Rich Megginson rmeggins@redhat.com
Summary by Sourcery
Refine CI testing by upgrading tox-lsr, ensuring proper boolean handling, enhancing workflows, and updating documentation
Bug Fixes:
Enhancements:
Documentation: