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

[LASKER] Delay WaitForUpdates #787

Merged
merged 1 commit into from
Feb 2, 2022

Conversation

agrare
Copy link
Member

@agrare agrare commented Feb 2, 2022

No description provided.

@agrare agrare requested a review from Fryguy as a code owner February 2, 2022 21:27
Delay WaitForUpdates and save in one thread to prevent "update storm" causing saver thread to be backed up

(cherry picked from commit a37bc28)
@agrare agrare force-pushed the lasker_add_threaded_saver_setting branch from d019fef to dbf88a2 Compare February 2, 2022 22:14
@Fryguy
Copy link
Member

Fryguy commented Feb 2, 2022

Ok, I ran the script to backport 754 and 753 so it would take care of all the labeling...surprised this conflicted, but it should be good it you rebase and the first 2 commits should go away

@miq-bot
Copy link
Member

miq-bot commented Feb 2, 2022

Checked commit agrare@dbf88a2 with ruby 2.6.3, rubocop 1.13.0, haml-lint 0.35.0, and yamllint
6 files checked, 0 offenses detected
Everything looks fine. 🏆

@Fryguy Fryguy self-assigned this Feb 2, 2022
@Fryguy Fryguy merged commit 088cac3 into ManageIQ:lasker Feb 2, 2022
@Fryguy Fryguy added the bug label Feb 2, 2022
@agrare agrare deleted the lasker_add_threaded_saver_setting branch February 3, 2022 13:15
@agrare
Copy link
Member Author

agrare commented Feb 3, 2022

Yeah I was surprised it conflicted also since there were no other commits missing from master 🤷 was just this one line https://github.com/ManageIQ/manageiq-providers-vmware/pull/787/files#diff-c3abdb8e922b3a9bbdf85dab00139f5bc1ca94e548558b488b2f5e7e24be790eL19-R19

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

Successfully merging this pull request may close these issues.

None yet

3 participants