Skip to content

Delegate refresh_ems class method to parent cloud manager #120

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
Open
nasark wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Delegate refresh_ems class method to parent cloud manager #120

nasark wants to merge 2 commits into from

Conversation

@nasark
Copy link
Member

@nasark nasark commented Oct 31, 2024

Reasoning here: ManageIQ/manageiq-providers-azure#564 (comment)

@miq-bot assign @agrare
@miq-bot add_label bug, radjabov/yes?
@miq-bot add_reviewer @agrare

nasark
nasark commented Oct 31, 2024
View reviewed changes
config/settings.yml
:ibm_power_vc_infra:
:is_admin: false
:cinder:
:ibm_power_vc_cinder:
Copy link
Member Author

@nasark nasark Oct 31, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note: key was named incorrectly

@miq-bot miq-bot added bug Something isn't working radjabov/yes? labels Oct 31, 2024
@miq-bot
Copy link
Member

miq-bot commented Feb 3, 2025

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).

@miq-bot miq-bot added the stale label Feb 3, 2025
@miq-bot
Copy link
Member

miq-bot commented Mar 5, 2025

Checked commits nasark/manageiq-providers-ibm_power_vc@0d5f7f3~...5f8fd46 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
3 files checked, 0 offenses detected
Everything looks fine. 🏆

@miq-bot
Copy link
Member

miq-bot commented Jun 9, 2025

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

Reviewers

@agrare agrare Awaiting requested review from agrare agrare is a code owner

@Fryguy Fryguy Awaiting requested review from Fryguy Fryguy is a code owner

@jaywcarman jaywcarman Awaiting requested review from jaywcarman jaywcarman is a code owner

Assignees

@agrare agrare

Labels

bug Something isn't working radjabov/yes? stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nasark @miq-bot @agrare