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

Default to use streaming_refresh for vSphere #434

Merged
merged 2 commits into from
Dec 3, 2019

Conversation

agrare
Copy link
Member

@agrare agrare commented Aug 8, 2019

Switch to using streaming refresh by default

@agrare agrare added the wip label Aug 8, 2019
@agrare agrare force-pushed the default_to_streaming_refresh branch from 04972db to 39ce0ce Compare August 8, 2019 17:52
@agrare agrare force-pushed the default_to_streaming_refresh branch 2 times, most recently from 34255af to 9f17756 Compare August 14, 2019 20:59
@agrare agrare force-pushed the default_to_streaming_refresh branch 2 times, most recently from e1a762b to b02de0a Compare September 26, 2019 17:47
@agrare agrare changed the title [WIP] Default to use streaming_refresh for vSphere Default to use streaming_refresh for vSphere Sep 26, 2019
@agrare agrare removed the wip label Sep 26, 2019
@agrare agrare force-pushed the default_to_streaming_refresh branch from b02de0a to 1d63553 Compare November 11, 2019 18:51
@agrare agrare force-pushed the default_to_streaming_refresh branch from 1d63553 to b47e7a0 Compare November 19, 2019 19:35
@agrare agrare requested a review from Fryguy as a code owner November 19, 2019 19:35
@agrare
Copy link
Member Author

agrare commented Nov 19, 2019

@Fryguy probably time to pull the trigger on this one

@agrare agrare force-pushed the default_to_streaming_refresh branch 2 times, most recently from f8729f2 to 2665804 Compare December 2, 2019 16:37
@miq-bot
Copy link
Member

miq-bot commented Dec 2, 2019

Checked commits agrare/manageiq-providers-vmware@b061569~...2665804 with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
8 files checked, 0 offenses detected
Everything looks fine. 🍪

@djberg96
Copy link
Contributor

djberg96 commented Dec 3, 2019

👍

@Fryguy Fryguy merged commit ef00d2e into ManageIQ:master Dec 3, 2019
@Fryguy Fryguy added this to the Sprint 126 Ending Dec 9, 2019 milestone Dec 3, 2019
@agrare agrare deleted the default_to_streaming_refresh branch December 3, 2019 16:31
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.

4 participants