-
-
Notifications
You must be signed in to change notification settings - Fork 398
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Synchronization broken? #44
Comments
Hi @gurneyalex, that it's because there is no synchronization at all. We let the debate about what is the best strategy to do: replay commits or rebase, and we didn't conclude anything. I think it's time for retake this question. |
ah ok. I had assumed this had been dealt with in july while I was on holiday 😎 |
I think I'll just start doing this with replays, and if we notice this screws up everything later, it should be simple to switch to the rebase approach. So if nobody comes up with good reasons why that's the source of all evil in the world, I'm going to do it next Monday. |
I made a script on OCA/maintainer-tools#35 for the replay process. Today, I experimented a bit with rebases but that didn't work the way I wanted. The script as it is works for 8.0 without doing anything, for 7.0 there are 4 very trivial conflicts and one somewhat trivial conflict to resolve by hand. |
@hbrunn there is a leftover trivial merge conflict in the 7.0-synced branch in addons/account_voucher/account_voucher.py |
There is also another question: ¿how do we get and integrate specific 7.0 OCB revisions that are not on 8.0? |
so, I think Odoo merges 7.0 into master regularly, maybe they will also merge 7.0 into 8.0 ? |
@clonedagain Yeah, I think so. It can be the solution. @OCA board members, are you going to host sync script in any place? |
On 10/09/2014 11:36, Pedro M. Baeza wrote:
Alexandre Fayolle Camptocamp France SAS |
I just created a github user called oca-git-bot: https://github.com/OCA-git-bot/OCB. Here I also did the initial conflict resolving. My proposal is to review that and merge it into OCA/OCB On one of Therp's servers, this user will run the ocb-sync script every night and push to OCA-git-bot/OCB. As soon as we are convinced that this works as it should, we can have the bot commit to OCA/OCB and we're done. The only thing left to do then is to add the git bot to the project's commiters. I'll be happy to provide the password to every OCA board member interested (please provide your GPG key then) |
It looks good! Let's see how it works for a couple of weeks. Have you tried to merge 7.0 into 8.0? |
not yet |
Tonight for example, we had another conflict. Output looks like this then:
So I could simply go to the bot's account on our server, fix the conflict and push it, which I did just now. |
OK, these are the steps I see now:
What do you think, @gurneyalex and rest of the OCA board members? |
Is the first step a blocking one? I think Therp is more than happy to keep doing this for the time being |
I just tried the merge and we get a lot of conflicts we don't want here. Thinking it through, that's also not what we want to do. We want to merge the missing commits between OCB/7.0 and odoo/7.0 into OCB/8.0. I also think this is a problem we can better postpone until the sync process is implemented. My proposal of steps:
|
|
@hbrunn, I agree. Let's start with the PR, start with the synchronization in Therp's servers, and wait until OCA organizes the rest of the things. |
@pedrobaeza @hbrunn agree: these kind of synchronizations aren't large piece of infrastructure, it's not like if that would put a lock in on OCB, so we can perfectly delay doing the sync' on an OCA server and do the important steps before. |
@hbrunn Agree. |
I've just updated manually 8.0 OCB branch to be at "final" state label. When do we start the synchronization? @StefanRijnhart, have we got any list of commits which OCB 7.0 branch differs from official one? |
@pedrobaeza: for addons and web, you can extract that by modifying http://bazaar.launchpad.net/~therp-nl/lp-community-utils/replay_ocb2git/view/head:/ocb2git.py to just list the missing commits. For server, that is not possible due to the push overwrite which happened by accident during the 7.0 live cycle. You'll have to take the patch, which is not unmanageably large. |
OK, I'll start to work on these changes to be applied in 8.0 branch. Can you please activate the synchronization script? Thanks. |
Before we can do that, please review #51. And one of the OCA admin would have to add oca-git-bot to the OCA members for the push permissions. |
Initial merge #51 has been applied to the 7.0 main branch. I have invited @OCA-git-bot to the OCB maintainers team, so that @hbrunn can set up automatic synchronization for OCB 7.0 and 8.0. |
Thanks, @StefanRijnhart! Now I'm examining the diff commits between odoo/7.0 and OCB/7.0 to move them to 8.0 branch, and I'll propose a methodology. I think we can keep this issue open for that discussion, or may I open a new one? |
Thanks Pedro! Please open a new issue for that. |
OK, I have opened #52 for that and I'm closing this one. |
Last night we got the first run with the bot committing into the OCB branches. Everything went fine and 7.0 as well as 8.0 are up to date now. |
[FIX] set_message_last_post needs to be called in post step
It seems that the sync from odoo/odoo is broken.
@hbrunn @StefanRijnhart could you tell us what is failing so we can restore the sync?
Thanks
The text was updated successfully, but these errors were encountered: