<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 17, 2014 at 8:56 PM, Farah Juma <span dir="ltr"><<a href="mailto:fjuma@redhat.com" target="_blank">fjuma@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><br>
> From: "Matthias Wessendorf" <<a href="mailto:matzew@apache.org">matzew@apache.org</a>><br>
> To: "Farah Juma" <<a href="mailto:fjuma@redhat.com">fjuma@redhat.com</a>><br>
> Cc: "AeroGear Developer Mailing List" <<a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a>><br>
> Sent: Monday, March 17, 2014 2:55:01 PM<br>
<div class="">> Subject: Re: [aerogear-dev] UPS OpenShift cartridge 0.10.1 test results<br>
><br>
> On Mon, Mar 17, 2014 at 7:27 PM, Farah Juma <<a href="mailto:fjuma@redhat.com">fjuma@redhat.com</a>> wrote:<br>
><br>
> ><br>
> > > From: "Matthias Wessendorf" <<a href="mailto:matzew@apache.org">matzew@apache.org</a>><br>
> > > To: "AeroGear Developer Mailing List" <<a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a>><br>
> > > Cc: "Farah Juma" <<a href="mailto:fjuma@redhat.com">fjuma@redhat.com</a>><br>
> > > Sent: Monday, March 17, 2014 1:14:05 PM<br>
> > > Subject: Re: [aerogear-dev] UPS OpenShift cartridge 0.10.1 test results<br>
> > ><br>
> > > On Mon, Mar 17, 2014 at 6:10 PM, Karel Piwko <<a href="mailto:kpiwko@redhat.com">kpiwko@redhat.com</a>> wrote:<br>
> > ><br>
> > > > Hey,<br>
> > > ><br>
> > > > status looks like:<br>
> > > ><br>
> > > > * Integration tests - passed<br>
> > > > * UI tests - failed - UI glitches in FF<br>
> > > ><br>
> > ><br>
> > > known problem; not sure - but there might be already a ticket for it<br>
> > ><br>
> > ><br>
> > > > * SP tests - passed<br>
> > > > * Migration tests - failed - scheme problems.<br>
> > > ><br>
> > ><br>
> > > possible - we might have changed the data model between 0.9.x and 0.10.x<br>
> ><br>
> > Interesting, this would cause issues with modifying existing data/adding<br>
> > new data. As Karel noted on AGPUSH-563, it looks like we would need a<br>
> > migration strategy for DB schema changes.<br>
> ><br>
><br>
> yep - I was thought that's part of the upgrade work; but we can file a<br>
> ticket to explicitly cover this<br>
<br>
</div>That would be good. The database part of the upgrade is currently handled by the MySQL cartridge's "post_restore" action hook - this just restores the database to whatever it was before and doesn't handle schema changes:<br>
<br>
<a href="https://github.com/openshift/origin-server/blob/master/cartridges/openshift-origin-cartridge-mysql/bin/control#L140" target="_blank">https://github.com/openshift/origin-server/blob/master/cartridges/openshift-origin-cartridge-mysql/bin/control#L140</a></blockquote>
<div><br></div><div><br></div><div>ok</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><br>
<br>
Once the necessary MySQL commands for migrating data from 0.9.x to 0.10.x are defined, we could try adding them to the AeroGear cartridge's "post_restore" action hook.<br></blockquote><div><br></div><div>With "necessary MySQL commands" you mean those that take care of the schema changes? I think generally having a migration script for the schema is a nice thing to have, regardless of OpenShift. Once we are closer to our 1.0.0 that's something we have to have.</div>
<div><br></div><div>Are you planing on adding these necessary MySQL commands (assuming they are relevant for taking care of schema changes ;-)) for our cartridge? Or should we just track that requirement in a JIRA for now ? </div>
<div><br></div><div>-Matthias</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class=""><div class="h5"><br>
> ><br>
> > > ><br>
> > > > Migration is not a part of cartridge but I guess that UI should be<br>
> > fixed.<br>
> > > ><br>
> > ><br>
> > > It's minor, not sure we should really stop the process; especially the<br>
> > UPS<br>
> > > is now on staging/hold since one entire week<br>
> > ><br>
> > ><br>
> > > ><br>
> > > > I have filled following bugs:<br>
> > > > * <a href="https://issues.jboss.org/browse/AGPUSH-562" target="_blank">https://issues.jboss.org/browse/AGPUSH-562</a><br>
> > > > * <a href="https://issues.jboss.org/browse/AGPUSH-563" target="_blank">https://issues.jboss.org/browse/AGPUSH-563</a><br>
> > > ><br>
> > > > Note, as for Firefox DNS cache, just set<br>
> > > > network.dnsCacheExpirationGracePeriod=0 in about:config.<br>
> > > ><br>
> > > > Karel<br>
> > > > _______________________________________________<br>
> > > > aerogear-dev mailing list<br>
> > > > <a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
> > > > <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
> > > ><br>
> > ><br>
> > ><br>
> > ><br>
> > > --<br>
> > > Matthias Wessendorf<br>
> > ><br>
> > > blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
> > > sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
> > > twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a><br>
> > ><br>
> ><br>
><br>
><br>
><br>
> --<br>
> Matthias Wessendorf<br>
><br>
> blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
> sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>
> twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a><br>
><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Matthias Wessendorf <br><br>blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
</div></div>