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

Port to fedora-messaging #289

Merged
merged 6 commits into from
Feb 6, 2025
Merged

Port to fedora-messaging #289

merged 6 commits into from
Feb 6, 2025

Conversation

ralphbean
Copy link
Member

Fedora infrastructure is going to decommission the fedmsg-gateway endpoint we've been consuming, which requires us to migrate off that stack entirely.

@ralphbean ralphbean requested a review from Zyzyx as a code owner February 4, 2025 18:42
@ralphbean
Copy link
Member Author

In order to get this working locally I had to install the fedora-messaging rpm and edit the /etc/fedora-messaging/config.toml file to set the amqp_urlto"amqps://fedora:@rabbitmq.fedoraproject.org/%2Fpublic_pubsub"`.

@ralphbean ralphbean requested review from webbnh and removed request for Zyzyx February 4, 2025 18:44
webbnh

This comment was marked as resolved.

@ralphbean ralphbean requested a review from webbnh February 4, 2025 22:14
This is only necessary because fedmsg doesn't have durable queues, and
soon, we're going to move off of fedmsg to fedora-messaging/amqp which
does have durable queues.
Fedora infrastructure is going to decommission the fedmsg-gateway
endpoint we've been consuming, which requires us to migrate off that
stack entirely.
It is less relevant now that we're moving closer to fedora-messaging.
@ralphbean ralphbean force-pushed the drop-fedmsg-entirely branch from 27d5f48 to bf74801 Compare February 5, 2025 12:58
@ralphbean ralphbean merged commit 82111b1 into main Feb 6, 2025
6 checks passed
@ralphbean ralphbean deleted the drop-fedmsg-entirely branch February 6, 2025 13:16
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