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

core - lazy load resources #5032

Open
wants to merge 8 commits into
base: master
from

Conversation

@kapilt
Copy link
Collaborator

kapilt commented Nov 5, 2019

lots of stuff here, but the core is to enable resource module code loading lazily instead of eagerly. for all providers installed this is a 4x latency on the cli win, as well as 2x serverless cold start latency win.

this changes registry notify to drop separate event types (ie no more final event), there is only the per resource type event now, a subscriber on registration gets invoked separately for any existing resource types in the registry.

this also switches tox to using devmode environments else it was picking up the last release of c7n for other providers.

this also drops the notion of out of tree plugins, as that seems to be getting people into more trouble then value chain wrt to how to approach modeling domain problems.

closes #585

@kapilt kapilt mentioned this pull request Nov 6, 2019
@kapilt kapilt force-pushed the kapilt:core/lazy-load-resources branch from 91334c7 to b8d0c74 Nov 22, 2019
@kapilt

This comment has been minimized.

Copy link
Collaborator Author

kapilt commented Nov 25, 2019

i've got a followup branch that does the actual wiring into serverless execution modes, this branch is probably at a size limit that its worth separating them.

@kapilt kapilt added this to In progress in roadmap Jan 14, 2020
@kapilt kapilt force-pushed the kapilt:core/lazy-load-resources branch from 869ff83 to 32d6ac4 Jan 19, 2020
@kapilt kapilt moved this from In progress to Review in progress in roadmap Jan 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
roadmap
  
Review in progress
1 participant
You can’t perform that action at this time.