Plugins » Katello

this playbook should recreate the issue:

---
- hosts: localhost
  collections:
    - theforeman.foreman
  gather_facts: false
  vars:
    foreman_username: admin
    foreman_password: "changeme" 
    foreman_server_url: "https://centos7-katello-3-17.yatsu.example.com" 
    foreman_validate_certs: false
    organization_name: "Test Organization" 
    product_name: "Test Product" 
    repository_name: "Test Repository Justin" 
    repository_content_type: "yum" 
    repository_label: "just_a_test_repo_justin" 
    repository_mirror_on_sync: false
    repository_url: "https://repos.fedorapeople.org/pulp/pulp/demo_repos/zoo/" 
    repository_state: present
  tasks:
    - name: create repo
      repository:
        username: "{{ foreman_username }}" 
        password: "{{ foreman_password }}" 
        server_url: "{{ foreman_server_url }}" 
        validate_certs: "{{ foreman_validate_certs }}" 
        organization: "{{ organization_name }}" 
        name: "{{ repository_name }}" 
        label: "{{ repository_label | default(omit) }}" 
        mirror_on_sync: False
        state: "{{ repository_state }}" 
        product: "{{ product_name }}" 
        content_type: "{{ repository_content_type }}" 
        url: "{{ repository_url | default(omit) }}" 

    - name: update repo
      repository:
        username: "{{ foreman_username }}" 
        password: "{{ foreman_password }}" 
        server_url: "{{ foreman_server_url }}" 
        validate_certs: "{{ foreman_validate_certs }}" 
        organization: "{{ organization_name }}" 
        name: "{{ repository_name }}" 
        label: "{{ repository_label | default(omit) }}" 
        mirror_on_sync: True
        state: "{{ repository_state }}" 
        product: "{{ product_name }}" 
        content_type: "{{ repository_content_type }}" 
        url: "{{ repository_url | default(omit) }}" 

We should also handle this more gracefully to try to handle the situation where the distribution reference doesn't exist, but the distribution does exist in pulp3. Today, this will always error.

The best path forward would be to make the create/update apis async and handle the race condition by trying to look up the distribution in pulp by the base path.

Edit: on second thought, this would still require pre-creating an empty distribution. Another solution could be to edit RefreshDistribution to lookup the distribution in Pulp first to check if it was created, and go from there.