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

Revert "out_kafka: make MSVC compatible (#1179)" #1832

Merged
merged 1 commit into from
Dec 23, 2019

Conversation

fujimotos
Copy link
Member

This reverts commit 69fffab.

Bakashar Kemerbay (and many other Windows users) reports that it
causes an immediate abort on Windows environments. In particular,
it causes abort even when they does not use out_kafka.

For example, I can confirm that even the following command does
not work on a vanilla Windows server.

$ fluent-bit.exe -i dummy -o stdout

This is because librdkafka is dependent on OpenSSL (ssleay32.dll
and libeay32.dll), which most Windows servers do not have. Hence
fluent-bit.exe always fails due to missing DLL dependencies.

So we have to revert this, and revisit it after resolving the
dependency issue.

Signed-off-by: Fujimoto Seiji fujimoto@clear-code.com

This reverts commit 69fffab.

Bakashar Kemerbay (and many other Windows users) reports that it
causes an immediate abort on Windows environments. In particular,
it causes abort even when they does not use out_kafka.

For example, I can confirm that even the following command does
not work on a vanilla Windows server.

    $ fluent-bit.exe -i dummy -o stdout

This is because librdkafka is dependent on OpenSSL (ssleay32.dll
and libeay32.dll), which most Windows servers do not have. Hence
fluent-bit.exe always fails due to missing DLL dependencies.

So we have to revert this, and revisit it after resolving the
dependency issue.

Signed-off-by: Fujimoto Seiji <fujimoto@clear-code.com>
@JungleGenius
Copy link
Contributor

Hello Can someone please tell me where I can download the Windows td-agent-bit installer for the version that doesn't have this missing DLL issue? I've got to get this agent installed this weekend.

@fujimotos
Copy link
Member Author

fujimotos commented Dec 22, 2019 via email

@JungleGenius
Copy link
Contributor

Thank you! Can you verify if I have the right link? Is this it?

https://ci.appveyor.com/project/fluent/fluent-bit-2e87g/builds/29673141/job/1l1dnwvwyv9tkygi/artifacts

Does it include all the supporting files needed for Windows or just the executable? Sorry, but I only have the build environment for Linux. I wish they would leave up the links to previous versions..

@fujimotos
Copy link
Member Author

fujimotos commented Dec 22, 2019 via email

@JungleGenius
Copy link
Contributor

Hi @fujimotos,

Are the "standard" plugins a part of this Windows build? Do I need to install the "include" directory with the C header files in the production server?

When I run it I'm getting the following:
"Input plugin 'cpu' cannot be loaded"
"Input plugin 'mem' cannot be loaded"
"Input plugin 'disk' cannot be loaded"
"Input plugin 'exec' cannot be loaded"
Error Configuration file contain errors. Aborting.

We use this same configuration file on Linux with no issues.

@edsiper
Copy link
Member

edsiper commented Dec 23, 2019

@JungleGenius the missing plugins mentioned above are Linux "only"

@edsiper edsiper merged commit b0191b6 into fluent:master Dec 23, 2019
phiremande pushed a commit to phiremande/fluent-bit that referenced this pull request Dec 24, 2019
This reverts commit 69fffab.

Bakashar Kemerbay (and many other Windows users) reports that it
causes an immediate abort on Windows environments. In particular,
it causes abort even when they does not use out_kafka.

For example, I can confirm that even the following command does
not work on a vanilla Windows server.

    $ fluent-bit.exe -i dummy -o stdout

This is because librdkafka is dependent on OpenSSL (ssleay32.dll
and libeay32.dll), which most Windows servers do not have. Hence
fluent-bit.exe always fails due to missing DLL dependencies.

So we have to revert this, and revisit it after resolving the
dependency issue.

Signed-off-by: Fujimoto Seiji <fujimoto@clear-code.com>
hassanfarid pushed a commit to hassanfarid/fluent-bit that referenced this pull request Dec 27, 2019
This reverts commit 69fffab.

Bakashar Kemerbay (and many other Windows users) reports that it
causes an immediate abort on Windows environments. In particular,
it causes abort even when they does not use out_kafka.

For example, I can confirm that even the following command does
not work on a vanilla Windows server.

    $ fluent-bit.exe -i dummy -o stdout

This is because librdkafka is dependent on OpenSSL (ssleay32.dll
and libeay32.dll), which most Windows servers do not have. Hence
fluent-bit.exe always fails due to missing DLL dependencies.

So we have to revert this, and revisit it after resolving the
dependency issue.

Signed-off-by: Fujimoto Seiji <fujimoto@clear-code.com>
Signed-off-by: Hassan Farid <hassan.seth@gmail.com>
@fujimotos fujimotos deleted the sf/win32-no-openssl branch January 7, 2020 06:52
edsiper pushed a commit that referenced this pull request Jan 23, 2020
This reverts commit 69fffab.

Bakashar Kemerbay (and many other Windows users) reports that it
causes an immediate abort on Windows environments. In particular,
it causes abort even when they does not use out_kafka.

For example, I can confirm that even the following command does
not work on a vanilla Windows server.

    $ fluent-bit.exe -i dummy -o stdout

This is because librdkafka is dependent on OpenSSL (ssleay32.dll
and libeay32.dll), which most Windows servers do not have. Hence
fluent-bit.exe always fails due to missing DLL dependencies.

So we have to revert this, and revisit it after resolving the
dependency issue.

Signed-off-by: Fujimoto Seiji <fujimoto@clear-code.com>
@fujimotos fujimotos mentioned this pull request Jul 26, 2021
8 tasks
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.

None yet

3 participants