Skip to content

Add Templates feature to foreman-proxy#519

Open
shubhamsg199 wants to merge 2 commits into
theforeman:masterfrom
shubhamsg199:templates-feature
Open

Add Templates feature to foreman-proxy#519
shubhamsg199 wants to merge 2 commits into
theforeman:masterfrom
shubhamsg199:templates-feature

Conversation

@shubhamsg199

Copy link
Copy Markdown
Contributor

Why are you introducing these changes? (Problem description, related links)

What are the changes introduced in this pull request?

  • Enable templates feature for foreman-proxy

How to test this pull request

Steps to reproduce:

  • foremanctl deploy --add-feature templates

Checklist

  • Tests added/updated (if applicable)
  • Documentation updated (if applicable)

@shubhamsg199 shubhamsg199 self-assigned this May 25, 2026
@shubhamsg199 shubhamsg199 marked this pull request as draft May 25, 2026 17:31
Comment thread src/playbooks/deploy/metadata.obsah.yaml
@shubhamsg199 shubhamsg199 marked this pull request as ready for review June 3, 2026 09:30
@shubhamsg199 shubhamsg199 force-pushed the templates-feature branch 3 times, most recently from 77b2cc9 to 6be7b5b Compare June 5, 2026 15:23
Comment thread .github/workflows/test.yml
@ehelms

ehelms commented Jun 10, 2026

Copy link
Copy Markdown
Member

Does templates need to be in the features.yaml? I do not see how the feature in this PR ensures foreman-proxy gets added.

@shubhamsg199

shubhamsg199 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Does templates need to be in the features.yaml? I do not see how the feature in this PR ensures foreman-proxy gets added.

Yes, templates is declared in features.yaml under the foreman_proxy key (same pattern as bmc). I don't think we have a way to ensure if foreman-proxy gets added.

@shubhamsg199 shubhamsg199 force-pushed the templates-feature branch 4 times, most recently from 713b3d9 to ded1e9b Compare June 11, 2026 17:00
@shubhamsg199

shubhamsg199 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

@ehelms Made some updates that ensures foreman-proxy gets added on the feature install. Not sure if that should be part of this PR.

Comment thread tests/foreman_proxy_test.py Outdated
Signed-off-by: Shubham Ganar <shubhamsg123m@gmail.com>
Signed-off-by: Shubham Ganar <shubhamsg123m@gmail.com>
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