-
Notifications
You must be signed in to change notification settings - Fork 286
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 c10s nodejs 22 #443
Conversation
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
…on 22-minimal Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
Signed-off-by: Petr "Stone" Hracek <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
1 similar comment
This comment was marked as outdated.
This comment was marked as outdated.
c413ec5
to
7af11f7
Compare
Signed-off-by: Petr "Stone" Hracek <[email protected]>
7af11f7
to
53172ce
Compare
This comment was marked as outdated.
This comment was marked as outdated.
1 similar comment
This comment was marked as outdated.
This comment was marked as outdated.
e013ba0
to
0e3cd0a
Compare
Signed-off-by: Petr "Stone" Hracek <[email protected]>
0e3cd0a
to
5f1200a
Compare
[test] |
1 similar comment
[test] |
@mhdawson @lholmquist NodeJS 22 test is failing:
|
06efaf1
to
84ef93c
Compare
[test] |
1 similar comment
[test] |
@hhorak @mhdawson @lholmquist @pkubatrh What do you think about merging this pull request? Fedora 22 is GREEN, CEntOS Stream 10 22-minimal is GREEN but CentOS Stream 22 is failing on reason mentioned in comment #443 (comment). Thank you for your response. |
@pacostas and I looked a bit at #443 (comment) today. I'll let Costas add more details (he'll do that tomorrow as it was his end of day when we looked at it) but the short answer is that we don't believe it is related to Node.js, instead the base library used by all language clients that re-use the common librdkafka library will be broken. In that context I think it should not hold up landing this PR. My guess is that CEntOS Stream 10 22-minimal is GREEN because we don't test packages which need compilers on it. |
Yes, the error is happening on the librdkafka level on this line https://github.com/confluentinc/librdkafka/blob/9416dd80fb0dba71ff73a8cb4d2b919f54651006/src/rdkafka_conf.h#L40 Practically, the node-rdkafka npm package is a wrapper of the librdkafka library. This error is not specific to nodejs as a result affects all the packages that wrap this functionality e.g. python https://docs.confluent.io/kafka-clients/python/current/overview.html#python-installation Go https://github.com/confluentinc/confluent-kafka-go etc. |
This pull request introduces NodeJS 22 for CentOS Stream 10.
The pull request is separated into several commits.