#12921 Request RabbitMQ queue osci-pipelines-queue-rmdepcheck
Closed: Fixed by lecris. Opened by lecris.

Describe what you would like us to do:

Add a new RabbitMQ queue for rmdepcheck runs as osci-pipelines-queue-rmdepcheck. The implementation details of the job are at https://github.com/fedora-ci/rmdepcheck-trigger/pull/1. Not sure what information is relevant to give here.


When do you need this to be done by? (YYYY/MM/DD)



Metadata Update from @james:
- Issue priority set to: Waiting on Assignee (was: Needs Review)
- Issue tagged with: low-trouble, medium-gain

Hmm, looking in ansible-infra, the pipelines appear to be numbered, not named:

- name: OSCI
  run_once: true
  include_role:
    name: rabbit/queue
  vars:
    queue_username: osci-pipelines{{ env_suffix }}
    queue_name: "{{ osci_pipeline_queue }}{{ env_suffix }}"
    queue_message_ttl: 864000000
    user_sent_topics: ^org\.(fedoraproject|centos)\.{{ env_short }}\.ci\..*
  loop:
    - "osci-pipelines{{ env_suffix }}-queue-2"
    - "osci-pipelines{{ env_suffix }}-queue-4"
    - "osci-pipelines{{ env_suffix }}-queue-5"
    - "osci-pipelines{{ env_suffix }}-queue-7"
    - "osci-pipelines{{ env_suffix }}-queue-9"
    - "osci-pipelines{{ env_suffix }}-queue-10"
    - "osci-pipelines{{ env_suffix }}-queue-11"
    - "osci-pipelines{{ env_suffix }}-queue-14"
    - "osci-pipelines{{ env_suffix }}-queue-15"
    - "osci-pipelines{{ env_suffix }}-queue-16"
  loop_control:
    loop_var: osci_pipeline_queue
  tags:
    - osci-pipelines

The most recent was added by @mvadkert in https://pagure.io/fedora-infra/ansible/pull-request/1079# . So should we just do the same and add a -17 ?

Explicitly wanted to avoid the numbering because it is a nightmare tracking it back when one of them has an issue. But also weird, se of them should have been disabled

yeah, this is in roles/rabbitmq_cluster/tasks/apps.yml in ansible.
We can use the name, that seems like a much better idea to me.

we basically need:

queue_username: some user for the queue
queue_name: name
user_sent_topics: what topics

Could you do a pr? or else I can try and get it.

I will try to do a pr for it. Will need to familiarize with some of these anyway

Ok PR open: https://pagure.io/fedora-infra/ansible/pull-request/2992

But about the order, I guess it would be safe to merge https://github.com/fedora-ci/rmdepcheck-trigger/pull/1 and have it listen even before the queue is made?

Working now

Metadata Update from @lecris:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

Metadata
Related Pull Requests