Comment 3 for bug 1660646

Revision history for this message
Jim Rollenhagen (jim-rollenhagen) wrote :

discussion in irc with perhaps a good/easy fix for this:

16:47:15 jroll | sdague: so we learned we are doing it wrong (tm) in our grenade jobs, but were getting lucky until recently, now it's broken. this hacks around it until we can really fix it: https://review.openstack.org/#/c/427248
16:47:52 jroll | the bug trail can be followed for details if you need more
16:52:26 sdague | jroll: if you are +1 on that, I'll +A it. Hopefully it unsticks you.
16:52:44 sdague | We do need to figure out how to deal with that in a more real way later
16:53:02 jroll | sdague: +1'd, yeah I'm good with it
16:53:06 sdague | I was really hoping we wouldn't have to build a full dependency graph
16:53:13 sdague | but... that might be dead hope
16:53:30 jroll | I guess there's the alternative of support upgrades in either order, but that makes other things harder
16:56:30 sdague | so... maybe we could cheat a little
16:57:18 sdague | register_project_for_upgrade is currently just adding things to a list
16:57:57 sdague | ensure_project_before ironic nova - that moves the ironic entry to be right before the nova entry
16:58:09 sdague | and we just do a list adjust early on
16:58:48 sdague | I'm trying to think about if that could go horribly wrong
17:06:08 jroll | sdague: oh, your other idea is interesting, we could play with it
17:07:24 sdague | jroll: yeh, if you or someone else on the ironic side wanted to implement that ensure_project_before function, which would be an explicit call after the register_project... call, that would be cool
17:07:35 sdague | we'll still land the other patch to fix you
17:07:48 sdague | but that would be the more correct way to handle things
17:09:03 jroll | sdague: yeah, thanks, I'll post in the bug about it and I think someone can get to it soon
17:09:10 sdague | jroll: cool
17:09:14 jroll | maybe next week, we're freezing this week