PR#1610: Allow distrobuildsync-eln to send to rebuilder.eln - fedora-infra/ansible

fedora-infra / ansible

#1610 Allow distrobuildsync-eln to send to rebuilder.eln

Opened 7 months ago by abompard. Modified 3 months ago

fedora-infra/ abompard/ansible ticket-11585 into main

Add two more subscriptions for distrobuildsync-eln

Aurélien Bompard • 7 months ago

1ff6d1e

Fix spaces for CI

Aurélien Bompard • 7 months ago

8f82110

Allow distrobuildsync-eln to send to rebuilder.eln

Aurélien Bompard • 7 months ago

5d8ac9d

roles/rabbitmq_cluster/tasks/apps.yml

file modified

+27 -19

		`@@ -60,9 +60,9 @@`
		`include_role:`
		`name: rabbit/queue`
		`vars:`
		`- username: fedora-build-checks{{ env_suffix }}`
		`- queue_name: fedora-build-checks{{ env_suffix }}`
		`- routing_keys:`
		`+ username: fedora-build-checks{{ env_suffix }}`
		`+ queue_name: fedora-build-checks{{ env_suffix }}`
		`+ routing_keys:`
		`- "org.centos.ci.#"`

		`- name: copr`
		`@@ -86,7 +86,7 @@`
		`# This routing key is for testing purposes only`
		`- "org.centos.stream.#"`
		`thresholds:`
		`- warning: 1000`
		`+ warning: 1000`
		`critical: 10000`

		`# CENTOS GITLAB BEGIN`
		`@@ -160,31 +160,39 @@`
		`name: rabbit/queue`
		`vars:`
		`username: distrobuildsync-eln`
		`- sent_topics: ^$`
		`+ sent_topics: ^org\.fedoraproject\.{{ env_short }}\.rebuilder\.eln\..*`
		`queue_name: distrobuildsync-eln`
		`# TTL: 10 days (in miliseconds)`
		`message_ttl: 864000000`
		`routing_keys:`
		`+ - "#.buildsys.repo.init"`
		`- "#.buildsys.repo.done"`
		`- "#.buildsys.tag"`
		`+ - "#.buildsys.task.state.change"`
		`+ - "org.fedoraproject.*.rebuilder.eln.enqueue"`
		`+ - "org.fedoraproject.*.rebuilder.eln.batch.create"`
		`+ - "org.fedoraproject.*.rebuilder.eln.batch.state_change"`
		`+ - "org.fedoraproject.*.rebuilder.eln.attempt.create"`
		`+ - "org.fedoraproject.*.rebuilder.eln.attempt.state_change"`
		`+`

		`# ELN END`

		`# ELN CS BEGIN`
		`#`
		`- #- name: eln cs queue`
		`- # run_once: true`
		`- # include_role:`
		`- # name: rabbit/queue`
		`- # vars:`
		`- # username: distrobuildsync-cs`
		`- # sent_topics: ^$`
		`- # queue_name: distrobuildsync-cs`
		`- # # TTL: 10 days (in miliseconds)`
		`- # message_ttl: 864000000`
		`- # routing_keys:`
		`- # - "#.buildsys.repo.init"`
		`- # - "#.buildsys.repo.done"`
		`- # - "#.buildsys.tag"`
		`+ # - name: eln cs queue`
		`+ # run_once: true`
		`+ # include_role:`
		`+ # name: rabbit/queue`
		`+ # vars:`
		`+ # username: distrobuildsync-cs`
		`+ # sent_topics: ^$`
		`+ # queue_name: distrobuildsync-cs`
		`+ # # TTL: 10 days (in miliseconds)`
		`+ # message_ttl: 864000000`
		`+ # routing_keys:`
		`+ # - "#.buildsys.repo.init"`
		`+ # - "#.buildsys.repo.done"`
		`+ # - "#.buildsys.tag"`
		`#`
		`# ELN END`

abompard commented 7 months ago

See: https://pagure.io/fedora-infrastructure/issue/11585

zuul commented 7 months ago

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/4b2d2f7782ba4e04bc02c0cb023fdb38

fi-ansible-lint-diff : SUCCESS in 4m 20s
fi-yamllint-diff : FAILURE in 2m 41s

1 new commit added

Fix spaces for CI

7 months ago

2 new commits added

Fix spaces for CI
Allow distrobuildsync-eln to send to rebuilder.eln

7 months ago

zuul commented 7 months ago

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/3e8b6b59d6874833a527105f5340fa40

fi-ansible-lint-diff : SUCCESS in 5m 40s
fi-yamllint-diff : SUCCESS in 3m 17s

sgallagh commented on line 32 of roles/rabbitmq_cluster/tasks/apps.yml 7 months ago

I thought these were limited by the routing_keys argument in the client's config file. That would explain why we had to add in a hack to get properly notified about buildsys.repo.init and buildsys.task.state.change. I thought adding them to the fedora-messaging config file would be enough.

Could we please get "#.buildsys.repo.init" and "#.buildsys.task.state.change" added here as well?

1 new commit added