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

Use the new after_update_endpoints callback for RefreshWorker #841

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

agrare
Copy link
Member

@agrare agrare commented Dec 6, 2022

Restart the RefreshWorker after endpoints are updated

Related:

@agrare agrare requested a review from Fryguy as a code owner December 6, 2022 19:22
@agrare agrare force-pushed the add_after_update_endpoints branch 2 times, most recently from 0af2d55 to e4374e3 Compare December 7, 2022 15:04
@Fryguy Fryguy self-assigned this Dec 7, 2022
@Fryguy Fryguy added the bug label Dec 7, 2022
@miq-bot miq-bot added the stale label Mar 13, 2023
@miq-bot
Copy link
Member

miq-bot commented Mar 13, 2023

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s)

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@agrare agrare force-pushed the add_after_update_endpoints branch from 293625e to 45445ef Compare March 14, 2023 15:53
@miq-bot
Copy link
Member

miq-bot commented Jun 19, 2023

This pull request has been automatically closed because it has not been updated for at least 3 months.

Feel free to reopen this pull request if these changes are still valid.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@miq-bot miq-bot closed this Jun 19, 2023
@agrare agrare reopened this Jun 19, 2023
@agrare agrare removed the stale label Jun 19, 2023
Restart the RefreshWorker after endpoints are updated
@agrare agrare force-pushed the add_after_update_endpoints branch from 45445ef to 1a491c5 Compare June 19, 2023 13:59
@miq-bot
Copy link
Member

miq-bot commented Jun 19, 2023

Checked commits agrare/manageiq-providers-vmware@76aeb6c~...1a491c5 with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@miq-bot miq-bot added the stale label Sep 25, 2023
@miq-bot
Copy link
Member

miq-bot commented Sep 25, 2023

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s)

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@miq-bot
Copy link
Member

miq-bot commented Jan 1, 2024

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).

1 similar comment
@miq-bot
Copy link
Member

miq-bot commented Apr 8, 2024

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).

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

3 participants