Skip to content

Allow two concurrent tasks on workers - #657

Merged
nforro merged 2 commits into
packit:mainfrom
nforro:deployment
Jul 7, 2026
Merged

nforro merged 2 commits into
packit:mainfrom
nforro:deployment

Conversation

@nforro

@nforro nforro commented Jul 7, 2026

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Nikola Forró <nforro@redhat.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request increases the maximum concurrent tasks from 1 to 2 in the OpenShift configmap environment configuration. However, this change introduces a critical race condition where concurrent tasks processing the same package will attempt to use and delete the same shared directory path, potentially causing build failures or file corruption. It is recommended to keep the concurrency limit at 1 or update the agent code to use unique directories per merge request.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread openshift/configmap-agents-env.yml
@nforro

nforro commented Jul 7, 2026

Copy link
Copy Markdown
Member Author

/gemini review

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request increases the maximum concurrent tasks to 2 in the OpenShift configuration and appends a merge request suffix to the Fedora clone directory path. Feedback highlights a critical concurrency issue in rebase_agent.py triggered by increasing concurrent tasks, and suggests a code improvement to reuse local_clone.name instead of parsing the merge request URL again.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread openshift/configmap-agents-env.yml
Comment thread ymir/agents/tasks.py Outdated
Signed-off-by: Nikola Forró <nforro@redhat.com>
Assisted-by: Claude Opus 4.6 via Claude Code
@nforro

nforro commented Jul 7, 2026

Copy link
Copy Markdown
Member Author

/gemini review

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request increases the maximum concurrent tasks in the OpenShift configuration, localizes the local_tool_options dictionary inside the workflow functions of the merge request, rebase, and rebuild agents to prevent shared state issues, and updates the Fedora clone directory path to include the local clone name to avoid directory conflicts. There are no review comments, and I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@majamassarini majamassarini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤞🏻

@lbarcziova lbarcziova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@nforro
nforro merged commit b86d8a1 into packit:main Jul 7, 2026
11 checks passed
@nforro
nforro deleted the deployment branch July 7, 2026 13:29
This was referenced Aug 6, 2026
@qodo-for-packit qodo-for-packit Bot mentioned this pull request Sep 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants