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

Scheduling event on s3 bucket fails #1442

Open
jesse-peters opened this issue Mar 14, 2018 · 5 comments
Open

Scheduling event on s3 bucket fails #1442

jesse-peters opened this issue Mar 14, 2018 · 5 comments
Labels

Comments

@jesse-peters
Copy link
Contributor

Context

Using zappa on python 3.6 inside a venv.

Expected Behavior

I have an s3 bucket which has a few existing event subscriptions. There is no overlap with the event subscription that zappa is trying to manage. I expect to be able to schedule a new subscription with zappa, even if there are already some on the bucket.

Actual Behavior

It looks at the bucket, and says that the event schedule already exists.

Possible Fix

I suspect it is only looking at the event types, and does not actually look at the filters when looking to see if an s3 event subscription already exists.

Steps to Reproduce

  1. Add an event subscription on prefix A manually to an s3 bucket
  2. Attempt to schedule zappa with an event subscription on prefix B. It should tell you that the event subscription already exists.

Your Environment

  • Zappa version used: 0.45.1
  • Your zappa_settings.py:
{
			"function": "app.function",
			"event_source": {
				"arn": "arn:aws:s3:::s3bucketname",
				"key_filters": [{
					"type": "prefix",
					"value": "A"
				}],
				"events": [
					"s3:ObjectCreated:*"
				]
}
@richiverse
Copy link
Contributor

richiverse commented Mar 14, 2018 via email

@jesse-peters
Copy link
Contributor Author

jesse-peters commented Mar 14, 2018

I can have different prefixes hit different lambda functions if I set it up manually on the bucket though.

(those are different functions)

image

@scoates scoates reopened this Mar 14, 2018
@scoates
Copy link
Collaborator

scoates commented Mar 14, 2018

Sorry if I closed this too early. (-: reopened.

@nrowland
Copy link

I also am seeing this issue where zappa updates are deleting other pre-existing events on an s3 bucket. Is this going to be resolved any time soon?

@farridav
Copy link

*Bump this is affecting me too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants