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

Fixes #32233 - Katello should send user-repo info to container gateway #9267

Conversation

ianballou
Copy link
Member

@ianballou ianballou commented Apr 6, 2021

Causes Katello to send user and repo info to the container gateway during proxy sync time. Katello will only send the user-repo mapping for users who exist in the Container Gateway's database.

Related Container Gateway PR: Katello/smart_proxy_container_gateway#13

TODO:

  • Write tests

@ianballou ianballou force-pushed the 32233-send-user-repo-mapping-container-gateway branch from 947dcfc to 2395e74 Compare April 6, 2021 22:26
@ianballou ianballou changed the title Katello should send user-repo info to container gateway Fixes #32233 - Katello should send user-repo info to container gateway Apr 6, 2021
@ianballou ianballou force-pushed the 32233-send-user-repo-mapping-container-gateway branch from 2395e74 to 07eb359 Compare April 6, 2021 22:27
@ianballou ianballou force-pushed the 32233-send-user-repo-mapping-container-gateway branch from 07eb359 to 92c0b3e Compare April 14, 2021 20:03
@ianballou ianballou marked this pull request as ready for review April 14, 2021 20:07
@ianballou ianballou force-pushed the 32233-send-user-repo-mapping-container-gateway branch from 92c0b3e to a9c522b Compare April 14, 2021 20:21
@ianballou ianballou force-pushed the 32233-send-user-repo-mapping-container-gateway branch from a9c522b to 6a48ee1 Compare April 15, 2021 15:33
@jjeffers
Copy link
Member

Tested this with docker client and podman via a smart proxy updated with Katello/smart_proxy_container_gateway#13. Worked as expected!

@ianballou ianballou merged commit 628ee7e into Katello:master Apr 21, 2021
@ianballou ianballou deleted the 32233-send-user-repo-mapping-container-gateway branch April 26, 2021 17:00
ianballou added a commit to ianballou/katello that referenced this pull request Jun 17, 2021
ianballou added a commit to ianballou/katello that referenced this pull request Jun 17, 2021
Katello#9267)

(cherry picked from commit 628ee7e)

Conflicts:
	app/controllers/katello/api/registry/registry_proxies_controller.rb
	app/models/katello/authorization/content_view.rb
	app/models/katello/concerns/smart_proxy_extensions.rb
ianballou added a commit that referenced this pull request Jun 19, 2021
* Fixes #32233 - Katello should send user-repo info to container gateway (#9267)

(cherry picked from commit 628ee7e)

Conflicts:
	app/controllers/katello/api/registry/registry_proxies_controller.rb
	app/models/katello/authorization/content_view.rb
	app/models/katello/concerns/smart_proxy_extensions.rb

* Bump version to 4.0.1.2

Pins foreman-tasks gem (#4)

Pin foreman-tasks related gems (#5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants