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

Refactor initial refresh queueing for VMware Refresh Worker #14813

Closed
agrare opened this issue Apr 19, 2017 · 5 comments · Fixed by ManageIQ/manageiq-providers-vmware#414
Closed

Refactor initial refresh queueing for VMware Refresh Worker #14813

agrare opened this issue Apr 19, 2017 · 5 comments · Fixed by ManageIQ/manageiq-providers-vmware#414
Assignees
@agrare
Labels
pinned providers/infrastructure refactoring

Comments

@agrare
Copy link
Member

agrare commented Apr 19, 2017

Don't rely on the broker to queue the initial refresh for EMSs when it starts up.
@agrare agrare mentioned this issue Apr 19, 2017
Merged
@agrare
Copy link
Member Author

agrare commented Apr 19, 2017

ManageIQ/manageiq-providers-vmware#41

@agrare agrare self-assigned this Apr 19, 2017
@miq-bot
Copy link
Member

miq-bot commented Oct 21, 2017

This issue has been automatically marked as stale because it has not been updated for at least 6 months.

If you can still reproduce this issue on the current release or on master, please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions!

@miq-bot miq-bot added the stale label Oct 21, 2017
@JPrause
Copy link
Member

JPrause commented Jan 23, 2019

@agrare is this still a valid issue. If not can you close.
If there's no update by next week, I'll be closing this issue.

@agrare
Copy link
Member Author

agrare commented Jan 23, 2019

This is still a valid issue

@agrare agrare added pinned and removed stale labels Jan 23, 2019
@JPrause
Copy link
Member

JPrause commented Jan 23, 2019

Thanks @agrare for the info and removing the label.

agrare added a commit to agrare/manageiq-providers-vmware that referenced this issue Jul 19, 2019
@agrare
Let the RefreshWorker queue the first full refresh
5ccdd73 
Move VMware more in line with the other providers by allowing the
RefreshWorker to queue the first full when it starts up.

This used to be done by the VimBrokerWorker because the RefreshWorkers
restarted every 2 hours. Now that
ManageIQ/manageiq#19010 does restart the
refresh_worker every 2 hours we can go back to just handling this in the
RefreshWorker.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1439387
Fixes ManageIQ/manageiq#14813
@agrare agrare mentioned this issue Jul 19, 2019
Merged
agrare added a commit to agrare/manageiq-providers-vmware that referenced this issue Jul 19, 2019
@agrare
Let the RefreshWorker queue the first full refresh
33a1565 
Move VMware more in line with the other providers by allowing the
RefreshWorker to queue the first full when it starts up.

This used to be done by the VimBrokerWorker because the RefreshWorkers
restarted every 2 hours. Now that
ManageIQ/manageiq#19010 does restart the
refresh_worker every 2 hours we can go back to just handling this in the
RefreshWorker.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1439387
Fixes ManageIQ/manageiq#14813
agrare added a commit to agrare/manageiq-providers-vmware that referenced this issue Jul 19, 2019
@agrare
Let the RefreshWorker queue the first full refresh
f940b29 
Move VMware more in line with the other providers by allowing the
RefreshWorker to queue the first full when it starts up.

This used to be done by the VimBrokerWorker because the RefreshWorkers
restarted every 2 hours. Now that
ManageIQ/manageiq#19010 does restart the
refresh_worker every 2 hours we can go back to just handling this in the
RefreshWorker.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1439387
Fixes ManageIQ/manageiq#14813
agrare added a commit to agrare/manageiq-providers-vmware that referenced this issue Aug 2, 2019
@agrare
Let the RefreshWorker queue the first full refresh
0acf5aa 
Move VMware more in line with the other providers by allowing the
RefreshWorker to queue the first full when it starts up.

This used to be done by the VimBrokerWorker because the RefreshWorkers
restarted every 2 hours. Now that
ManageIQ/manageiq#19010 does restart the
refresh_worker every 2 hours we can go back to just handling this in the
RefreshWorker.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1439387
Fixes ManageIQ/manageiq#14813
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@agrare agrare

Labels
pinned providers/infrastructure refactoring
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Let the RefreshWorker queue the first full refresh agrare/manageiq-providers-vmware
4 participants
@Fryguy @miq-bot @JPrause @agrare