Skip to content

SAT-40191 Compute resources & plugins#344

Merged
evgeni merged 1 commit into
theforeman:masterfrom
stejskalleos:ls/provisioning_plugins
Dec 3, 2025
Merged

SAT-40191 Compute resources & plugins#344
evgeni merged 1 commit into
theforeman:masterfrom
stejskalleos:ls/provisioning_plugins

Conversation

@stejskalleos

Copy link
Copy Markdown
Contributor

Allow enabling and installing of two features:
foreman_azure_rm & foreman_google

This includes foreman plugins and hammer-cli plugins.

How to test:

foremanctl deploy --add-feature foreman_azure_rm
foremanctl deploy --add-feature foreman_google

Related tasks

Comment thread src/requirements.yml Outdated
@arvind4501

arvind4501 commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

Can we also enable foreman_kubevirt as it comes under computer resources as its part of available plugins(https://github.com/theforeman/foreman-oci-images/blob/master/images/foreman/Containerfile#L19C101-L19C117) . or we don't support it?

Comment thread .github/workflows/test.yml Outdated
@stejskalleos

Copy link
Copy Markdown
Contributor Author

foreman_kubevirt is a technical preview, and its current state is "questionable".
I'd like to keep it out of the CI & our daily testing.

@evgeni

evgeni commented Dec 2, 2025

Copy link
Copy Markdown
Member

foreman_kubevirt is a technical preview, and its current state is "questionable". I'd like to keep it out of the CI & our daily testing.

I agree, let's keep it disabled for now

Comment thread tests/foreman_compute_resources_test.py
Comment thread tests/foreman_plugins_test.py
@evgeni evgeni force-pushed the ls/provisioning_plugins branch from 2274654 to 90973de Compare December 3, 2025 09:49
@evgeni

evgeni commented Dec 3, 2025

Copy link
Copy Markdown
Member

(should be squashed on merge)

Allow enabling and installing of two features:
foreman_azure_rm  & foreman_google

This includes foreman plugins and hammer-cli plugins.

Related tasks
* theforeman/foreman-oci-images@5c09bfb
@evgeni evgeni force-pushed the ls/provisioning_plugins branch from 90973de to 96a2e8f Compare December 3, 2025 12:45

@shubhamsg199 shubhamsg199 left a comment

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.

Ack

@evgeni evgeni merged commit 30da213 into theforeman:master Dec 3, 2025
10 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.

4 participants