-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
opensearch: Aoss master #6612
opensearch: Aoss master #6612
Conversation
please fix the conflicts on tests/internal/fuzzers/signv4_fuzzer.c |
ping |
Signed-off-by: Matthew Fala <falamatt@amazon.com>
1bfba07
to
2edb926
Compare
Rebased, thank you! |
@@ -1063,6 +1063,11 @@ static struct flb_config_map config_map[] = { | |||
0, FLB_FALSE, 0, | |||
"External ID for the AWS IAM Role specified with `aws_role_arn`" | |||
}, | |||
{ | |||
FLB_CONFIG_MAP_STR, "aws_service_name", "es", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the service name for aws opensearch serverless is "aoss". "es" doesn't work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This plugin is meant to be used with serverless and non-serverless. Non-serverless (es) has always been the default. This change allows you to configure something different.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@matthewfala minor change to get this merged, please fix the subjects for these commits:
|
Signed-off-by: Matthew Fala <falamatt@amazon.com>
Signed-off-by: Matthew Fala <falamatt@amazon.com>
2edb926
to
41bf918
Compare
Please see notes and add comments to : #6448
Enter
[N/A]
in the box, if an item is not applicable to your change.Testing
Before we can approve your change; please submit the following in a comment:
If this is a change to packaging of containers or native binaries then please confirm it works for all targets.
Documentation
Backporting
Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.