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

Fix attribute for Smart Proxy Server #3538

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

Conversation

maximiliankolb
Copy link
Contributor

What changes are you introducing?

I came to fix the example user input but stayed to fix the attribute.

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

You can synchronized a CV to a Smart Proxy Server but not from your Foreman Server onto your Smart Proxy that's on your Foreman Server.

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

Checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.13/Katello 4.15
  • Foreman 3.12/Katello 4.14 (Satellite 6.16)
  • Foreman 3.11/Katello 4.13 (orcharhino 6.11 on EL8 only; orcharhino 7.0 on EL8+EL9)
  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (Satellite 6.15; orcharhino 6.8/6.9/6.10)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • We do not accept PRs for Foreman older than 3.7.

Copy link

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

I'm not entirely sure I agree that you want to sync to a specific server. Server feels like an implementation detail. In theory there could be a whole highly available Pulp cluster running in Kubernetes behind it. Not something we support today, but code should be there to make it a reality.

@maximiliankolb
Copy link
Contributor Author

In the docs, we always differentiate between "Smart Proxies" and "Smart Proxy Servers". With content views, you can use the Web UI or Hammer CLI to sync them from your Foreman+Katello Server to a Smart Proxy Server. You cannot sync them from your Foreman+Katello Server to the Smart Proxy that runs on your Foreman+Katello Server; it actually comes from there.

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