Skip to content

Switch Trigger Service main loop data structure #1206

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pawiecz
Copy link
Contributor

@pawiecz pawiecz commented Jun 18, 2025

Build configurations list will not be known up front when external configuration files are enabled in Maestro. Dropping build_configs filter will allow simplifying Trigger Service logic taking into account only parameters that are still relevant.

@pawiecz pawiecz added the staging-skip Don't test automatically on staging.kernelci.org label Jun 18, 2025
@pawiecz pawiecz force-pushed the switch-trigger-loop-structure branch 2 times, most recently from 6d51916 to f3930de Compare June 19, 2025 18:57
@pawiecz pawiecz removed the staging-skip Don't test automatically on staging.kernelci.org label Jun 19, 2025
@pawiecz pawiecz requested a review from nuclearcat June 19, 2025 18:57
@pawiecz pawiecz marked this pull request as ready for review June 19, 2025 18:58
@pawiecz pawiecz requested a review from JenySadadia June 19, 2025 18:58
@pawiecz pawiecz added the staging-skip Don't test automatically on staging.kernelci.org label Jun 22, 2025
@pawiecz pawiecz force-pushed the switch-trigger-loop-structure branch from f3930de to ff30182 Compare June 23, 2025 07:39
@pawiecz pawiecz removed the staging-skip Don't test automatically on staging.kernelci.org label Jun 23, 2025
pawiecz added 2 commits July 7, 2025 13:28
This reverts commit 81387ab.

Build configurations list will not be known up front when external
configuration files are enabled in Maestro. Dropping this filter will
allow simplifying Trigger Service logic taking into account only
parameters that are still relevant.

Signed-off-by: Paweł Wieczorek <pawiecz@collabora.com>
Build configurations list will not be known up front when external
configuration files are enabled in Maestro. Switching main loop to
iterate over monitored trees will allow obtaining this information right
before its use.

Signed-off-by: Paweł Wieczorek <pawiecz@collabora.com>
@pawiecz pawiecz force-pushed the switch-trigger-loop-structure branch from ff30182 to a2f30dd Compare July 7, 2025 11:28
Copy link
Collaborator

@JenySadadia JenySadadia left a comment

Choose a reason for hiding this comment

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

LGTM.

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