Skip to content

feat(source-gitlab): Increase default concurrency from 8 to 10 (4.4.23-rc.3)#75537

Merged
sophiecuiy merged 2 commits intomasterfrom
devin/1774637715-source-gitlab-concurrency-10
Mar 27, 2026
Merged

feat(source-gitlab): Increase default concurrency from 8 to 10 (4.4.23-rc.3)#75537
sophiecuiy merged 2 commits intomasterfrom
devin/1774637715-source-gitlab-concurrency-10

Conversation

@sophiecuiy
Copy link
Copy Markdown
Contributor

What

Increases the default concurrency level for source-gitlab from 8 to 10 as the next step in the progressive rollout concurrency tuning process (epic).

RC.2 (which commented out HTTPAPIBudget) showed stable or slightly improved performance vs RC.1 across all pinned actors with zero failures. This RC bumps concurrency to measure the effect of higher parallelism without rate limiting.

How

  • manifest.yaml: Change default_concurrency fallback from 8 to 10 (users who set num_workers in config are unaffected)
  • metadata.yaml: Bump dockerImageTag from 4.4.23-rc.2 to 4.4.23-rc.3
  • gitlab.md: Add changelog entry

Review guide

  1. manifest.yaml — single-line concurrency default change
  2. metadata.yaml — version bump
  3. docs/integrations/sources/gitlab.md — changelog entry (⚠️ verify the PR number 75535 matches this PR's actual number)

User Impact

No immediate user impact — this RC will only be deployed to ~15 pinned actors via progressive rollout. Users who have explicitly configured num_workers are unaffected. If the concurrency increase causes issues (rate limiting, failures), it will be caught during rollout monitoring before wider release.

Can this PR be safely reverted and rolled back?

  • YES 💚

Link to Devin session: https://app.devin.ai/sessions/4462c8953c3c483084a62a7240f6973f
Requested by: sophiecuiy

…3-rc.3)

Co-Authored-By: sophie.cui@airbyte.io <sophie.cui@airbyte.io>
@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@octavia-bot octavia-bot bot marked this pull request as draft March 27, 2026 18:57
@octavia-bot
Copy link
Copy Markdown
Contributor

octavia-bot bot commented Mar 27, 2026

Note

📝 PR Converted to Draft

More info...

Thank you for creating this PR. As a policy to protect our engineers' time, Airbyte requires all PRs to be created first in draft status. Your PR has been automatically converted to draft status in respect for this policy.

As soon as your PR is ready for formal review, you can proceed to convert the PR to "ready for review" status by clicking the "Ready for review" button at the bottom of the PR page.

To skip draft status in future PRs, please include [ready] in your PR title or add the skip-draft-status label when creating your PR.

Co-Authored-By: sophie.cui@airbyte.io <sophie.cui@airbyte.io>
@github-actions
Copy link
Copy Markdown
Contributor

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

PR Slash Commands

Airbyte Maintainers (that's you!) can execute the following slash commands on your PR:

  • 🛠️ Quick Fixes
    • /format-fix - Fixes most formatting issues.
    • /bump-version - Bumps connector versions, scraping changelog description from the PR title.
  • ❇️ AI Testing and Review (internal link: AI-SDLC Docs):
    • /ai-prove-fix - Runs prerelease readiness checks, including testing against customer connections.
    • /ai-canary-prerelease - Rolls out prerelease to 5-10 connections for canary testing.
    • /ai-review - AI-powered PR review for connector safety and quality gates.
  • 🚀 Connector Releases:
    • /publish-connectors-prerelease - Publishes pre-release connector builds (tagged as {version}-preview.{git-sha}) for all modified connectors in the PR.
    • /bump-progressive-rollout-version - Bumps connector version with an RC suffix (2.16.10-rc.1) for progressive rollouts (enableProgressiveRollout: true).
      • Example: /bump-progressive-rollout-version changelog="Add new feature for progressive rollout"
  • ☕️ JVM connectors:
    • /update-connector-cdk-version connector=<CONNECTOR_NAME> - Updates the specified connector to the latest CDK version.
      Example: /update-connector-cdk-version connector=destination-bigquery
  • 🐍 Python connectors:
    • /poe connector source-example lock - Run the Poe lock task on the source-example connector, committing the results back to the branch.
    • /poe source example lock - Alias for /poe connector source-example lock.
    • /poe source example use-cdk-branch my/branch - Pin the source-example CDK reference to the branch name specified.
    • /poe source example use-cdk-latest - Update the source-example CDK dependency to the latest available version.
  • ⚙️ Admin commands:
    • /force-merge reason="<REASON>" - Force merges the PR using admin privileges, bypassing CI checks. Requires a reason.
      Example: /force-merge reason="CI is flaky, tests pass locally"
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 27, 2026

Deploy preview for airbyte-docs ready!

✅ Preview
https://airbyte-docs-cpqsq7zvw-airbyte-growth.vercel.app

Built with commit d879a79.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 27, 2026

source-gitlab Connector Test Results

35 tests   31 ✅  1m 23s ⏱️
 2 suites   4 💤
 2 files     0 ❌

Results for commit d879a79.

♻️ This comment has been updated with latest results.

@sophiecuiy sophiecuiy marked this pull request as ready for review March 27, 2026 19:07
@sophiecuiy sophiecuiy merged commit 49d3750 into master Mar 27, 2026
49 checks passed
@sophiecuiy sophiecuiy deleted the devin/1774637715-source-gitlab-concurrency-10 branch March 27, 2026 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants