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.
osci-pipelines-queue-rmdepcheck
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)