Merge lp:~rvb/maas/component-bug-1307415-1.5 into lp:maas/1.5

Proposed by Raphaël Badin
Status: Merged
Approved by: Raphaël Badin
Approved revision: no longer in the source branch.
Merged at revision: 2247
Proposed branch: lp:~rvb/maas/component-bug-1307415-1.5
Merge into: lp:maas/1.5
Diff against target: 18 lines (+3/-3)
1 file modified
src/maasserver/start_up.py (+3/-3)
To merge this branch: bzr merge lp:~rvb/maas/component-bug-1307415-1.5
Reviewer Review Type Date Requested Status
Raphaël Badin (community) Approve
Review via email: mp+215816@code.launchpad.net

Commit message

Backport revision 2265: Move the registration of components errors into the critical section. This is part of the start up code and thus outside of transaction management so moving this code inside the critical section avoids race conditions between WSGI processes.

To post a comment you must log in.
Revision history for this message
Raphaël Badin (rvb) wrote :

Simple backport, self-approving.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/maasserver/start_up.py'
--- src/maasserver/start_up.py 2014-03-25 19:21:59 +0000
+++ src/maasserver/start_up.py 2014-04-15 07:43:43 +0000
@@ -86,11 +86,11 @@
86 # Regenerate MAAS's DNS configuration. This should be reentrant, really.86 # Regenerate MAAS's DNS configuration. This should be reentrant, really.
87 write_full_dns_config(reload_retry=True)87 write_full_dns_config(reload_retry=True)
8888
89 # Check whether we have boot images yet.
90 update_import_script_error()
91
8992
90def post_start_up():93def post_start_up():
91 """Startup jobs that can run after the critical section."""94 """Startup jobs that can run after the critical section."""
92 # Check whether we have boot images yet.
93 update_import_script_error()
94
95 # Send secrets etc. to workers.95 # Send secrets etc. to workers.
96 NodeGroup.objects.refresh_workers()96 NodeGroup.objects.refresh_workers()

Subscribers

People subscribed via source and target branches

to all changes: