Skip to content

Fixes #39126 - provide empty provider_vm_specific_fields in CR::Base#650

Merged
stejskalleos merged 1 commit intomasterfrom
i39126
Mar 3, 2026
Merged

Fixes #39126 - provide empty provider_vm_specific_fields in CR::Base#650
stejskalleos merged 1 commit intomasterfrom
i39126

Conversation

@evgeni
Copy link
Copy Markdown
Member

@evgeni evgeni commented Mar 3, 2026

No description provided.

Fields::Field.new(label: _('Url'), path: [:url])
]
end
def provider_vm_specific_fields; []; end
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is this needed?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In don't see any relations with mentioned theforeman/foreman_kubevirt#192

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Because otherwise hammer is broken when hammer_cli_foreman_kubevirt is installed, as described in https://projects.theforeman.org/issues/39126

@stejskalleos stejskalleos merged commit a7ebf01 into master Mar 3, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants