ci: update header for run_role_with_clear_facts [citest_skip]#22
Closed
richm wants to merge 1 commit into
Closed
Conversation
Update header for run_role_with_clear_facts to indicate not editable and clarify what it does Signed-off-by: Rich Megginson <rmeggins@redhat.com>
Reviewer's GuideAligns repository references and CI/tmt configuration with the renamed trustee_client role and introduces a centrally managed Ansible helper task file to clear facts and run the trustee_client role with optional failure suppression. Sequence diagram for tft CI workflow running trustee_client planssequenceDiagram
actor Developer
participant GitHub
participant GitHubActions
participant TestingFarm
participant tmt
participant trustee_client_role
Developer->>GitHub: Push_commit_or_open_PR
GitHub-->>GitHubActions: Trigger_tft_workflow
GitHubActions->>TestingFarm: Request_tests(tf_scope=private, tmt_plan_filter="tag:playbooks_parallel,trustee_client")
TestingFarm->>tmt: Start_tmt_multihost_pipeline
tmt->>trustee_client_role: Run_plans_tagged_playbooks_parallel_and_trustee_client
trustee_client_role-->>tmt: Report_role_test_results
tmt-->>TestingFarm: Report_plan_results
TestingFarm-->>GitHubActions: Publish_results_and_status
GitHubActions-->>GitHub: Set_final_commit_status
Flow diagram for run_role_with_clear_facts helper taskflowchart TD
A[Start_run_role_with_clear_facts] --> B[Clear_Ansible_facts_on_managed_nodes]
B --> C[Include_role_trustee_client]
C --> D{allow_failure_enabled}
D -- yes --> E[Ignore_role_failure_in_test_outcome]
D -- no --> F[Propagate_role_failure_to_test]
E --> G[End_run_role_with_clear_facts]
F --> G[End_run_role_with_clear_facts]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- In README.md the main heading and the tox workflow badge URL still reference
trustee_attestation_client; if the role is being renamed, consider updating those remaining identifiers totrustee_clientfor consistency.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In README.md the main heading and the tox workflow badge URL still reference `trustee_attestation_client`; if the role is being renamed, consider updating those remaining identifiers to `trustee_client` for consistency.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update header for run_role_with_clear_facts to indicate not editable
and clarify what it does
Signed-off-by: Rich Megginson rmeggins@redhat.com
Summary by Sourcery
Rename remaining references from trustee_attestation_client to trustee_client and add a reusable task file to clear facts and run the trustee_client role with configurable error handling.
Enhancements:
Documentation: