<div dir="ltr">Just following up. This was fixed earlier on today and the various CI upgrade jobs are now passing. I've marked the ticket as Fix Released so that this issue no longer blocks merges.</div><div class="gmail_extra"><br><div class="gmail_quote">On 19 December 2014 at 09:40, Menno Smits <span dir="ltr"><<a href="mailto:menno.smits@canonical.com" target="_blank">menno.smits@canonical.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On 19 December 2014 at 06:02, Dimiter Naydenov <span dir="ltr"><<a href="mailto:dimiter.naydenov@canonical.com" target="_blank">dimiter.naydenov@canonical.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
All this is great work! Thanks for the write-up as well.<br>
<br>
I think I discovered an issue with it - take a look at this bug<br>
<a href="http://pad.lv/1403738" target="_blank">http://pad.lv/1403738</a>. It seems machine.SetAgentVersion() should be<br>
handled specially by the multi-env transaction runner, as only after<br>
calling it the upgrade actually starts and the steps to add env-uuids<br>
to state collections are executed.</blockquote><div><br></div></span><div>Sorry - I neglected to do a manual upgrade test before pushing this change. Ensuring that code that runs before database migrations have occurred still works as it should has been pain point for us while doing the multi-environment work.</div><div><br></div><div>I will get this sorted. Thanks for saving me some time by doing the initial analysis.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>- Menno</div><div><br></div><div><br></div><div> </div></font></span></div></div></div>
</blockquote></div></div>