deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michal Fojtik <mfoj...@redhat.com>
Subject Re: [PATCH core 7/7] CIMI: Print warning for pending migrations
Date Tue, 19 Feb 2013 09:10:20 GMT
On 02/18, David Lutterkort wrote:
> This strikes me as more clever than it needs to be:
>       * we should exit(1) if there are pending migrations - there's no
>         point in starting the server otherwise.

Yes, makes sense to me.

>       * can't deltacloud-db-upgrade just blindly run
>         Sequel::Migrator.run ? There shouldn't be any harm even if there

Fixed :)

>         are no migrations to apply
>       * do we really need a separate deltacloud-db-upgrade script rather
>         than just the rake task ? I don't have an issue with requiring
>         that people have rake on their prod systems (what's good enough
>         for rails ought to be good enough for us ;)

The reason I put deltacloud-db-upgrade here is that the 'gem' users will
not have access to rake tasks. If you do: gem install deltacloud-core
or you do gem update deltacloud-core, you need some commands to run
migrations.

  -- Michal

-- 
Michal Fojtik <mfojtik@redhat.com>
Deltacloud API, CloudForms

Mime
View raw message