Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: Update fmf plan to add a separate job to prepare managed nodes #708

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

spetrosi
Copy link
Collaborator

  • Add a preparation job to run on managed nodes
  • Update instructions to run tests locally
  • From tft workflow, remove condition for commenter author_association.
    This would require us to add all trusted users, running workflow is not dangerous.
  • Keep the plan minimal, only run prep steps required to install beakerlib

Signed-off-by: Sergei Petrosian spetrosi@redhat.com

* Add a preparation job to run on managed nodes
* Update instructions to run tests locally
* From tft workflow, remove condition for commenter author_association.
  This would require us to add all trusted users, running workflow is not dangerous.
* Keep the plan minimal, only run prep steps required to install beakerlib

Signed-off-by: Sergei Petrosian <spetrosi@redhat.com>
@spetrosi spetrosi requested a review from liangwen12year as a code owner July 30, 2024 14:14
@spetrosi spetrosi merged commit 773d1c1 into main Jul 30, 2024
19 of 22 checks passed
@spetrosi spetrosi deleted the ci-tf-prep-managed-nodes branch July 30, 2024 15:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant