Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Cannot run multiple batch-enabled instances on separate target branches #267

Open
valentindavid opened this issue Aug 19, 2020 · 0 comments · May be fixed by #271
Open

Cannot run multiple batch-enabled instances on separate target branches #267

valentindavid opened this issue Aug 19, 2020 · 0 comments · May be fixed by #271

Comments

@valentindavid
Copy link

In a project we use a marge bot instance per release branch so that we can run merges in parallel. However in that configuration, only one instance can do batches. This is because the branch name used for the batch is constant: marge_bot_batch_merge_job.

Instead it should be using a name that depends on the target branch. So for example for target branch release/20.08, it should use marge_bot_batch_merge_job/release/20.08.

However there is no way to configure the name of the batch branch.
@valentindavid valentindavid linked a pull request Aug 31, 2020 that will close this issue
Open
sg70 added a commit to sg70/marge-bot that referenced this issue Oct 1, 2021
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
d0022b0 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
sg70 added a commit to sg70/marge-bot that referenced this issue Mar 30, 2022
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
9262ded 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
sg70 added a commit to sg70/marge-bot that referenced this issue Mar 30, 2022
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
f16d589 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
sg70 added a commit to sg70/marge-bot that referenced this issue Mar 30, 2022
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
35d813a 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
sg70 added a commit to sg70/marge-bot that referenced this issue Mar 30, 2022
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
6dd4d53 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
sg70 added a commit to sg70/marge-bot that referenced this issue Mar 30, 2022
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
3712840 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
sg70 added a commit to sg70/marge-bot that referenced this issue Mar 31, 2022
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
a68549c 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
sg70 added a commit to sg70/marge-bot that referenced this issue Mar 31, 2022
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
a0e44b7 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
sg70 added a commit to sg70/marge-bot that referenced this issue Mar 31, 2022
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
3d8c496 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
sg70 added a commit to sg70/marge-bot that referenced this issue Mar 31, 2022
@sg70
feat: enable multiple target branches for batching MRs (smarkets#267)
d9354ac 
In order to support batch jobs in parallel, the batch branches
need to be tied to the target branch name.

Signed-off-by: Sascha Binckly <[email protected]>
@nanonyme nanonyme mentioned this issue Dec 16, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use target branch name in batch merge to allow multiple instances valentindavid/marge-bot
1 participant
@valentindavid