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

Feature request for protobuf ignore files that starts with dot #3138

Open
dubyte opened this issue Jan 23, 2025 · 0 comments
Open

Feature request for protobuf ignore files that starts with dot #3138

dubyte opened this issue Jan 23, 2025 · 0 comments
Labels
bughancement It's kind of a bug, but mostly an enhancement. needs investigation It looks as though have all the information needed but investigation is required processors Any tasks or issues relating specifically to processors

Comments

@dubyte
Copy link

dubyte commented Jan 23, 2025

The feature is alrady trying to fix the issue I will describe later, So if there is other way to resolve the problem please do.

When using kafka connect on kubernetes if we add proto files in a configMap and we mount that for example in /proto folder

Kubernetes create symlinks so if there are:
person.proto in /proto folder
kubernetes created that file as a symlink to /proto/..timestamp_person.proto

so if we use in protobuf processor the import path /proto it fails becasue it complains that person defined in /proto/person.proto is already defined in /proto/..timestamp_person.proto

So I think we could add a filter to ignore files starting with dot
or we can pass specifig list of files instead of a a dir

Other option is add flag that ignoreSymlinks so if we use /proto import path by ignoring symlinks it could find the defintions on the ../..timestamp... protos

do you think in other ways?

Currently I noticed that if I use mounting path as a specifc file and use subpath there is no symlinks but that is not an ideal fix for me.

@mihaitodor mihaitodor added processors Any tasks or issues relating specifically to processors bughancement It's kind of a bug, but mostly an enhancement. needs investigation It looks as though have all the information needed but investigation is required labels Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bughancement It's kind of a bug, but mostly an enhancement. needs investigation It looks as though have all the information needed but investigation is required processors Any tasks or issues relating specifically to processors
Projects
None yet
Development

No branches or pull requests

2 participants