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.

Sign up for GitHub

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

Jump to bottom

[branch/v7] Backport #9193 #9339

Merged
merged 1 commit into from
Dec 16, 2021
Merged

[branch/v7] Backport #9193 #9339

merged 1 commit into from
Dec 16, 2021

Conversation

espadolini
Copy link
Contributor

@espadolini espadolini commented Dec 10, 2021 •
edited by russjones
Loading

Backport #9193

@espadolini espadolini self-assigned this Dec 10, 2021
@github-actions github-actions bot requested review from jakule and zmb3 December 10, 2021 11:30
@espadolini espadolini changed the title Ensure we don't miss the resolution of an access request (#9193) [branch/v7] Backport (#9193) Dec 10, 2021
@espadolini espadolini changed the title [branch/v7] Backport (#9193) [branch/v7] Backport #9193 Dec 10, 2021
@espadolini espadolini force-pushed the espadolini/v7-tsh-watcher-race branch from dd26899 to 625e9bd Compare December 14, 2021 07:55
@espadolini espadolini enabled auto-merge (squash) December 14, 2021 07:56
@espadolini
Ensure we don't miss the resolution of an access request (#9193)
282c502 
This makes it so that tsh will watch for access request resolution on the
correct (root) cluster, and it will not create access requests before the event
watcher is ready.

Fixes #9003 and #9244.

Includes v7 backport fixups.
@espadolini espadolini force-pushed the espadolini/v7-tsh-watcher-race branch from 625e9bd to 282c502 Compare December 16, 2021 10:01
@espadolini espadolini merged commit c04761c into branch/v7 Dec 16, 2021
@espadolini espadolini deleted the espadolini/v7-tsh-watcher-race branch December 16, 2021 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@russjones russjones russjones approved these changes

@r0mant r0mant r0mant approved these changes

@jakule jakule Awaiting requested review from jakule

@zmb3 zmb3 Awaiting requested review from zmb3

Assignees

@espadolini espadolini

Labels
backport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@espadolini @r0mant @russjones