Optimize tenant, flavor and template selection

dice-cyfronet / atmosphere

Atmosphere cloud platform

MIT License

7 stars 3 forks source link

Optimize tenant, flavor and template selection #279

Closed nowakowski closed 8 years ago

nowakowski commented 8 years ago

Completely eliminates the n+1 query problem in optimizer.select_tmpl_and_flavor_and_tenant, thus fixing a nasty performance bug in PL-Grid Atmosphere deployment.

nowakowski commented 8 years ago

@mkasztelnik Can you help out with this? It's quite urgent (btw., we probably need an "Urgent" label -- perhaps you could define one).

mkasztelnik commented 8 years ago

Can you describe how can I help with it? What is missing? Only review or something more?

nowakowski commented 8 years ago

@mkasztelnik I just need a code review.

mkasztelnik commented 8 years ago

First of all can you squash all commits into one with meaningful name? Commits like "Interim commit." or "Woof." say nothing about what was changed and why.

nowakowski commented 8 years ago

I believe I'm done with all requested changes.

mkasztelnik commented 8 years ago

:+1: from me. Please merge all commits into one before merge.