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

Composite workflow #2

Open
wants to merge 18 commits into
base: composite-workflow
Choose a base branch
from

Conversation

stevanbz
Copy link

@stevanbz stevanbz commented Mar 3, 2023

Issue #, if available:

Description of changes:
Contains the changes for CRUD-ing the workflow and changes for executing the workflow

CheckList:
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…nitor index is not initialized yet. Added workflow crud test cases

Signed-off-by: Stevan Buzejic <[email protected]>
… consider the workflow execution id

Added worfklow service used for retrieving monitors and their findings. Added business logic for considering the chained monitors

Signed-off-by: Stevan Buzejic <[email protected]>
Signed-off-by: Stevan Buzejic <[email protected]>
…when loading the cluster

Signed-off-by: Stevan Buzejic <[email protected]>
…esponse class

Code adjusted according to comments

Signed-off-by: Stevan Buzejic <[email protected]>
…dation if the query monitor is part of the workflow chain

Signed-off-by: Stevan Buzejic <[email protected]>
Signed-off-by: Stevan Buzejic <[email protected]>
… checking workflow metadata. Changed flow of workflow execution

Signed-off-by: Stevan Buzejic <[email protected]>
@stevanbz stevanbz force-pushed the feature/composite-workflow-execution-v1 branch from 20de168 to 8e0d28d Compare March 9, 2023 21:17
…hat verify that workflow metadata is not created

Signed-off-by: Stevan Buzejic <[email protected]>
…he monitors once the workflow is updated

Signed-off-by: Stevan Buzejic <[email protected]>
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.

1 participant