MAAS

Merge lp://staging/~mpontillo/maas/packaging-fix-db-purge into lp://staging/~maas-maintainers/maas/packaging

  1. packaging-fix-db-purge
  2. Merge into packaging
Proposed by Mike Pontillo
Status: Merged
Approved by: Mike Pontillo
Approved revision: no longer in the source branch.
Merged at revision: 376
Proposed branch: lp://staging/~mpontillo/maas/packaging-fix-db-purge
Merge into: lp://staging/~maas-maintainers/maas/packaging
Diff against target: 25 lines (+4/-0)
2 files modified
debian/changelog (+3/-0)
debian/maas-region-controller.prerm (+1/-0)
To merge this branch: bzr merge lp://staging/~mpontillo/maas/packaging-fix-db-purge
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Raphaël Badin (community) Approve
Review via email: mp+255330@code.staging.launchpad.net

Commit message

Fix bug #1044559 (services must be stopped, or database cannot properly be purged)

Description of the change

Before a postgresql database can be dropped, nothing can be using it. This patch stops maas-clusterd in prerm, so that it's more likely that the purge will succeed.

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

Looks good to me. I'd like Andres opinion about this but you can probably drop the "invoke-rc.d apache2 stop || true" bit now…

review: Approve
Revision history for this message
Mike Pontillo (mpontillo) wrote :

> Looks good to me. I'd like Andres opinion about this but you can probably
> drop the "invoke-rc.d apache2 stop || true" bit now...

Good point. Maybe the behavior changed when we switched from Apache. Part of me would rather leave it in, in case someone is using an old Apache config...

Revision history for this message
Andres Rodriguez (andreserl) wrote :

lgtm!

review: Approve
Revision history for this message
Andres Rodriguez (andreserl) wrote :

also, I don't think we care about stopping apache2 anymore since we don';t really use it anymore.

Revision history for this message
Mike Pontillo (mpontillo) wrote :

Should we keep it in, in case we're purging an old version that is still running?

Revision history for this message
Andres Rodriguez (andreserl) wrote :

Yeah let's keep it in for the time being!

Thanks!
On Apr 8, 2015 14:59, "Mike Pontillo" <email address hidden> wrote:

> Should we keep it in, in case we're purging an old version that is still
> running?
> --
>
> https://code.launchpad.net/~mpontillo/maas/packaging-fix-db-purge/+merge/255330
> You are reviewing the proposed merge of
> lp:~mpontillo/maas/packaging-fix-db-purge into
> lp:~maas-maintainers/maas/packaging.
>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches