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

Add documentation regarding conflicting event notification rules #17949

Open
jasonpanosso opened this issue Nov 4, 2024 · 3 comments · May be fixed by #19130
Open

Add documentation regarding conflicting event notification rules #17949

jasonpanosso opened this issue Nov 4, 2024 · 3 comments · May be fixed by #19130
Assignees
Labels
content:edit Request for content edits DevRel Tasks that need support from developer relations. documentation Documentation edits product:r2 R2 object storage: https://developers.cloudflare.com/r2

Comments

@jasonpanosso
Copy link

jasonpanosso commented Nov 4, 2024

Existing documentation URL(s)

What changes are you suggesting?

Provide documentation regarding when event notification rules conflict with other event notification rules

Additional information

e.g. when adding an event notification rule on some bucket for some queue, you can not have multiple event notifications configured for the same action, e.g. 'PutObject'. It will give you an error mentioning that there are conflicting rules.

@jasonpanosso jasonpanosso added content:edit Request for content edits documentation Documentation edits labels Nov 4, 2024
@github-actions github-actions bot added the product:r2 R2 object storage: https://developers.cloudflare.com/r2 label Nov 4, 2024
@dcpena dcpena added the DevRel Tasks that need support from developer relations. label Dec 6, 2024
@harshil1712 harshil1712 self-assigned this Jan 3, 2025
@harshil1712
Copy link
Contributor

harshil1712 commented Jan 8, 2025

Hey @jasonpanosso, I tried the following and got no errors. The event notification rules were created successfully.

  • Create a rule for object-create for Queue A
  • Create a rule for object-create for Queue B
  • Create a rule for object-delete for Queue B

All the above cases were a success. Can you please share what exact error you got? Were you using prefix or suffix with your rules? Or were you trying to use the same Queue for two different buckets?

@jasonpanosso
Copy link
Author

jasonpanosso commented Jan 8, 2025

Hey @jasonpanosso, I tried the following and got no errors. The event notification rules were created successfully.

* Create a rule for `object-create` for Queue A

* Create a rule for `object-create` for Queue B

* Create a rule for `object-delete` for Queue B

All the above cases were a success. Can you please share what exact error you got? Were you using prefix or suffix with your rules? Or were you trying to use the same Queue for two different buckets?

I was specifically getting an error regarding "conflicting event notification rules" when creating rules that are effectively the same using this api endpoint.

Do you get an error in this situation?

  • create a rule for PutObject + DeleteObject for Queue A for Bucket A
  • create a rule for PutObject + CopyObject for Queue A for Bucket A

I would have expected that to work considering the API endpoint was a PUT, but it seems that it acts more like a POST considering that it doesn't seem to overwrite/replace existing rules and isn't idempotent

@harshil1712
Copy link
Contributor

@jasonpanosso, I was able to replicate your issue. I am not sure if it is a limitation or a bug. I am checking with the team to get a clarity. Post that we will update the documentation, if needed. Thank you again for reporting it!

@harshil1712 harshil1712 linked a pull request Jan 10, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content:edit Request for content edits DevRel Tasks that need support from developer relations. documentation Documentation edits product:r2 R2 object storage: https://developers.cloudflare.com/r2
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants