-
Notifications
You must be signed in to change notification settings - Fork 362
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
S3 Bucket constantly calls PutBucketNotification #1165
Comments
@empath-nirvana are you using |
We aren't using that.
the bucket does work, it's just resyncing 1000 times a day and causing test messages to be sent. Is there a way to get detailed logging so i can see what it thinks the difference is? |
Thanks @empath-nirvana. I was able to reproduce the bug with your YAML. It seems the cause of this issue is that the []types.QueueConfiguration{
{
... // 1 ignored and 1 identical fields
Filter: &types.NotificationConfigurationFilter{
Key: &types.S3KeyFilter{
FilterRules: []types.FilterRule{
{
- Name: "Prefix",
+ Name: "prefix",
... // 2 ignored fields
},
},
... // 1 ignored field
},
... // 1 ignored field
},
... // 2 ignored fields
},
} This seems to be an issue on AWS side: {
"QueueConfigurations": [
{
"Id": "...",
"QueueArn": "...",
"Events": [
"s3:ObjectCreated:*"
],
"Filter": {
"Key": {
"FilterRules": [
{
"Name": "Prefix",
"Value": "some-prefix"
}
]
}
}
}
]
} The SDK itself also only defines |
Is there a way to work around this that doesn't involve waiting for a patch to the aws sdk for go? |
Its a bit difficult because the controller uses @muvaf what would be the best way to circumvent this issue? |
What happens if you change the enum to allow "Prefix" and "Suffix"? Does s3 only accept lower case? |
Given that aws-sdk has said they won't fix the issue, is there some way this can be fixed by the cross-plane provider? |
Since its a bug at AWS it would be best to wait for them to fix it on their side. On provider level you could only add a hacky workaround by converting the respective fields to lowercase strings or find a way to ignore case when doing |
I talked to someone on the S3 team. It’s extremely unlikely they’re going
to modify S3 api behavior.
On Thu, Mar 10, 2022 at 6:32 AM MisterMX ***@***.***> wrote:
Since its a bug at AWS it would be best to wait for them to fix it on
their side.
On provider level you could only add a hacky workaround by converting the
respective fields to lowercase strings or find a way to ignore case when
doing cmp.Equal.
—
Reply to this email directly, view it on GitHub
<#1165 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AUVYTL5PMTCDTPKHU6PIIUTU7HMTHANCNFSM5O7OYNCQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
“The world is not comprehensible, but it is embraceable.”
|
…guration.Filter.Key.FilterRules.Name Signed-off-by: Paul Schroeder <paul.schroeder-extern@deutschebahn.com>
…guration.Filter.Key.FilterRules.Name Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
…guration.Filter.Key.FilterRules.Name Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
…guration.Filter.Key.FilterRules.Name Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
…guration.Filter.Key.FilterRules.Name Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
…guration.Filter.Key.FilterRules.Name Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
…-instead-of-prefix fix(s3.bucket): Fixes #1165, discrepancy in QueueConfiguration.Filter.Key.FilterRules.Name
…guration.Filter.Key.FilterRules.Name (with extensive corner case testing) Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
…guration.Filter.Key.FilterRules.Name (with extensive corner case testing) Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
…guration.Filter.Key.FilterRules.Name (with extensive corner case testing) Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
…-instead-of-prefix fix(s3.bucket): Fixes #1165, discrepancy QueueConfiguration.Filter.Ke…
…guration.Filter.Key.FilterRules.Name Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
…guration.Filter.Key.FilterRules.Name (with extensive corner case testing) Signed-off-by: Paul Schroeder <97968850+schroeder-paul@users.noreply.github.com>
Facing same issue but with TopicConfiguration.Filter.Key.FilterRules.Name. Crossplane is infinitely calling the PutBucketNotification on our bucket. Seems the same fix need to be applied to TopicConfiguration as well. |
What happened?
The issue is similar to this issue, however, I've hardcoded the ID in the notifications queue configuration and we're using version 0.21.2 which should have this fix in place:
#476
I've created a queue and it continuously tries to put in the notifications configuration, generating a test message every time.
How can we reproduce it?
Create a bucket with a notifications queue, check cloudtrail logs to see that it's making an api call every minute.
What environment did it happen in?
Crossplane version:
The text was updated successfully, but these errors were encountered: