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

Support for AMQP #248

Closed
wants to merge 1 commit into from
Closed

Support for AMQP #248

wants to merge 1 commit into from

Conversation

mmailhos
Copy link

@mmailhos mmailhos commented Jan 9, 2019

No description provided.

Signed-off-by: Mathieu Mailhos <mathieu@datarepublic.com>
@jojotheboss
Copy link

Hi, I really need that PR. Please merge :-)

@warmfusion
Copy link

Out of my own curiosity, any reason this PR doesn't use the giraffi based fluent-plugin-amqp - I'm a contributor on that project and am about to start deploying the plugin onto our own kubernetes clusters so if you're aware of a known problem with it, i'll need to go fix it :-)

@jojotheboss
Copy link

Any news concerning this PR ?

@maxpain
Copy link

maxpain commented Mar 28, 2019

Any news?

@jojotheboss
Copy link

Still no merge ?

@repeatedly
Copy link
Member

Yes. Currently, we don't want to merge such patches because it is not good for users. We need to find good way for maintaining third party images.
Maybe, patch author will publish on docker hub.

@repeatedly
Copy link
Member

I just opened new issue for current concern: #293

@repeatedly repeatedly added the pending Can't merge by some reason label May 28, 2019
@mmailhos mmailhos closed this Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending Can't merge by some reason
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants