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

OSSM-1805: Requeue patching Kiali if Kiali resource is not available #946

Conversation

jewertow
Copy link
Member

@jewertow jewertow commented Jul 7, 2022

While testing #910, I observed that the operator skips patching Kiali if this resource does not exist, but it may occur temporarily, so I think we should requeue reconciliation to wait for Kiali and patch it properly.

Signed-off-by: Jacek Ewertowski jewertow@redhat.com

@jewertow jewertow changed the title Requeue patching Kiali if Kiali resource is not available OSSM-1805: Requeue patching Kiali if Kiali resource is not available Jul 26, 2022
@jewertow jewertow requested review from rcernich and luksa July 26, 2022 11:21
@jewertow jewertow force-pushed the OSSM-218-requeue-reconciliation-if-kiali-is-not-available branch from a46abd3 to c81581f Compare July 28, 2022 09:22
@maistra-bot maistra-bot added size/L and removed size/S labels Jul 28, 2022
@jewertow jewertow force-pushed the OSSM-218-requeue-reconciliation-if-kiali-is-not-available branch from c81581f to 287cf1a Compare July 28, 2022 09:24
@maistra-bot maistra-bot added size/S and removed size/L labels Jul 28, 2022
@jewertow
Copy link
Member Author

@jwendell I guess you should cherry-pick #956 to maistra-2.3.

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>
@jewertow jewertow force-pushed the OSSM-218-requeue-reconciliation-if-kiali-is-not-available branch from 287cf1a to 75f6796 Compare July 28, 2022 13:38
@maistra-bot maistra-bot merged commit 724e8c1 into maistra:maistra-2.3 Jul 28, 2022
@maistra-bot
Copy link
Contributor

In response to a cherrypick label: new pull request created: #958

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants