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

Add support for rpm_repository remote and repo_config #150

Merged

Conversation

netsandbox
Copy link
Contributor

No description provided.

@netsandbox netsandbox force-pushed the rpm_repository_remote_repo_config branch from 9edf22e to 64d1aba Compare March 5, 2024 17:39
Comment on lines +264 to +278
- hosts: localhost
gather_facts: false
vars_files:
- vars/server.yaml
module_defaults:
<<: *pulp_module_defaults
tasks:
- name: Make repository absent
pulp.squeezer.rpm_repository:
name: test_rpm_repository
state: absent
- name: Make remote absent
pulp.squeezer.rpm_remote:
name: test_rpm_remote
state: absent
Copy link
Member

Choose a reason for hiding this comment

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

Nice catch!

Comment on lines +128 to +131
if module.params["remote"] is not None:
remote = PulpRpmRemote(module, {"name": module.params["remote"]})
remote.find(failsafe=False)
desired_attributes["remote"] = remote.href
Copy link
Member

Choose a reason for hiding this comment

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

Too bad this module wasn't yet translated to using glue...
I hope that won't become more complicated now.

@mdellweg mdellweg merged commit ee28b8d into pulp:develop Mar 6, 2024
12 checks passed
@netsandbox netsandbox deleted the rpm_repository_remote_repo_config branch March 6, 2024 12:39
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