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

fix: evaluate all possible refresh reasons for multi-source apps (#12379) (cherry-pick #12576) #12609

Merged
merged 1 commit into from Feb 24, 2023

Conversation

gcp-cherry-pick-bot[bot]
Copy link

Cherry-picked fix: evaluate all possible refresh reasons for multi-source apps (#12379) (#12576)

Signed-off-by: Michael Crenshaw 350466+crenshaw-dev@users.noreply.github.com

  • remove redundant parentheses

Signed-off-by: Michael Crenshaw 350466+crenshaw-dev@users.noreply.github.com

  • tests

Signed-off-by: Michael Crenshaw 350466+crenshaw-dev@users.noreply.github.com

  • don't auto-sync, it makes tests flaky

Signed-off-by: Michael Crenshaw 350466+crenshaw-dev@users.noreply.github.com

  • auto-sync because sync CLI doesn't work for multi-source apps

Signed-off-by: Michael Crenshaw 350466+crenshaw-dev@users.noreply.github.com

  • don't require out-of-sync - app may sync quickly

Signed-off-by: Michael Crenshaw 350466+crenshaw-dev@users.noreply.github.com

  • timeout 60

Signed-off-by: Michael Crenshaw 350466+crenshaw-dev@users.noreply.github.com

  • fix timeout

Signed-off-by: Michael Crenshaw 350466+crenshaw-dev@users.noreply.github.com


Signed-off-by: Michael Crenshaw 350466+crenshaw-dev@users.noreply.github.com

) (#12576)

* fix: evaluate all possible refresh reasons for multi-source apps (#12379)

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* remove redundant parentheses

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* tests

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* don't auto-sync, it makes tests flaky

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* auto-sync because sync CLI doesn't work for multi-source apps

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* don't require out-of-sync - app may sync quickly

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* timeout 60

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* fix timeout

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

---------

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
@crenshaw-dev
Copy link
Collaborator

Good bot.

@crenshaw-dev
Copy link
Collaborator

I wonder if we should enable more checks for the cherry-pick PRs.

@crenshaw-dev crenshaw-dev merged commit a5fb6ff into release-2.6 Feb 24, 2023
@crenshaw-dev crenshaw-dev deleted the cherry-pick-17ec58-release-2.6 branch February 24, 2023 18:39
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.

None yet

1 participant