fix: activation of flows dependent on actions #79
Conversation
This file contains 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
|
Hey @ewingjm, some rapid-fire contributions this week! Tests failed with: |
|
@tdashworth should all be sorted now. Gonna put a pre-release package out there to test it on our project first. |
Tests have shown that a maximum of 30 seconds is not sufficient
@ewingjm let me know how the pre-release goes and I'll approve. |
|
Kudos, SonarCloud Quality Gate passed! |
|
@ewingjm can you remember how this went? |
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.
Purpose
Activating flows immediately after actions they're dependent on fails due to them being unable to retrieve the metadata. There is a short delay after activating an action before which these flows are able to be activated.
Approach
Introduces multiple attempts to activate processes and will delay for 10 seconds where there are any failure responses due to the above issue.
TODOs