jenkins-bot has submitted this change and it was merged.
Change subject: Cache project name
......................................................................
Cache project name
This should eliminate many, many keystone calls.
Change-Id: I7f12acdefaf287bff42c2232e8cac74ebc5db546
---
M nova/OpenStackNovaProject.php
1 file changed, 11 insertions(+), 2 deletions(-)
Approvals:
Andrew Bogott: Looks good to me, approved
jenkins-bot: Verified
diff --git a/nova/OpenStackNovaProject.php b/nova/OpenStackNovaProject.php
index 544237c..dc826aa 100644
--- a/nova/OpenStackNovaProject.php
+++ b/nova/OpenStackNovaProject.php
@@ -70,9 +70,18 @@
function loadProjectName() {
global $wgOpenStackManagerLDAPProjectBaseDN;
+ global $wgMemc;
- $controller = OpenstackNovaProject::getController();
- $this->projectname = $controller->getProjectName(
$this->projectid );
+ $key = wfMemcKey( 'openstackmanager', 'projectname',
$this->projectid );
+ $this->projectname = $wgMemc->get( $key );
+ if ( ! $this->projectname ) {
+ $controller = OpenstackNovaProject::getController();
+ $this->projectname = $controller->getProjectName(
$this->projectid );
+
+ # Projectname doesn't ever change once a project is
created, so
+ # we can cache this a good long time.
+ $wgMemc->set( $key, $this->projectname );
+ }
# We still keep things like sudoers in ldap, so we need a
unique dn for this
# project to keep things under.
--
To view, visit https://gerrit.wikimedia.org/r/270499
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7f12acdefaf287bff42c2232e8cac74ebc5db546
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/OpenStackManager
Gerrit-Branch: wmf/1.27.0-wmf.13
Gerrit-Owner: Andrew Bogott <[email protected]>
Gerrit-Reviewer: Andrew Bogott <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
