-
Notifications
You must be signed in to change notification settings - Fork 5
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
[WIP] debug InstanceProfileCredentialsProvider usage #397
base: main
Are you sure you want to change the base?
Conversation
Running the code local yields to this error:
I guess this will be the same on your side @sugarcrm-jgminder. /cc @embano1 |
@agebhar1 thx! I guess the error is due to the fact that you don't run in an EKS IMDS environment? |
@sugarcrm-jgminder You can download the build for this test PR here: https://github.com/awslabs/eventbridge-kafka-connector/actions/runs/11048078087?pr=397 |
The error is more a result of the missing profile file, introduced by the change of v1.3.1 |
@agebhar1 got it! @maschnetwork this comparison of the change between the versions from Andi was helpful. |
Yes, the difference is in the behavior of My understanding is they behave the same in absence of package software.amazon.event.kafkaconnector;
import software.amazon.awssdk.profiles.ProfileFile;
import software.amazon.awssdk.profiles.ProfileFileSupplier;
public class Test {
public static void main(String[] args) {
try {
var supplier = ProfileFileSupplier.defaultSupplier();
var profileFile = supplier.get();
System.out.println(profileFile);
} catch (Exception e) {
System.out.println(e); // java.lang.NullPointerException
}
var profileFile = ProfileFile.defaultProfileFile();
System.out.println(profileFile); // ProfileFile(sections=[])
}
} IDK if one can call it a bug? |
02d6a4c
to
422ba5e
Compare
Description
Currently only for debugging broken authentication with kube2iam (InstanceProfileCredentialsProvider).
Test Steps
Build
mvn clean package -Drevision=$(git describe --tags --always)
and deploy artifact to Kafka Connect.
Checklist:
Related Issue
#396
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.