Skip to content

Conversation

@amimas
Copy link
Collaborator

@amimas amimas commented Jan 11, 2026

This change adds support for managing/configuring push mirrors in GitLab. The change includes necessary acceptance tests and documentation.

Note: GitLab currently has a limitation in their API for configuring SSH based mirrors. For this type of mirror, GitLab needs to know the "host key" of the target system. This option can only be configured when setting up the mirror manually using the UI. Due to lack of API options in GitLab side, SSH based mirror setup will not be functional.

resolves #873

@amimas amimas requested a deployment to Integrate Pull Request January 11, 2026 16:00 — with GitHub Actions Waiting
@amimas amimas requested a deployment to Integrate Pull Request January 11, 2026 16:00 — with GitHub Actions Waiting
@codecov
Copy link

codecov bot commented Jan 11, 2026

Codecov Report

❌ Patch coverage is 92.59259% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.54%. Comparing base (230c16a) to head (602229e).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...orm/processors/project/remote_mirrors_processor.py 92.53% 10 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #1157       +/-   ##
===========================================
+ Coverage   61.55%   88.54%   +26.98%     
===========================================
  Files          73       74        +1     
  Lines        3470     3605      +135     
===========================================
+ Hits         2136     3192     +1056     
+ Misses       1334      413      -921     
Flag Coverage Δ
integration 87.18% <92.59%> (+28.59%) ⬆️
unittests 37.69% <15.55%> (-0.87%) ⬇️
Files with missing lines Coverage Δ
gitlabform/processors/project/__init__.py 100.00% <100.00%> (ø)
...orm/processors/project/remote_mirrors_processor.py 92.53% <92.53%> (ø)

... and 41 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@amimas amimas temporarily deployed to Integrate Pull Request January 11, 2026 19:12 — with GitHub Actions Inactive
@amimas amimas temporarily deployed to Integrate Pull Request January 11, 2026 19:12 — with GitHub Actions Inactive
@amimas amimas temporarily deployed to Integrate Pull Request January 18, 2026 03:40 — with GitHub Actions Inactive
@amimas amimas temporarily deployed to Integrate Pull Request January 18, 2026 03:40 — with GitHub Actions Inactive
amimas added 18 commits January 24, 2026 12:23
Adjusted processor logic to ensure create mirror is working including sync validation.
Previously a single test tried to validate creating multiple mirrors
as well as validating sync function for one of them. To simplify the
test logic, they have been separated into different tests. Moving
the logic for validating sync function also caught bug in mirror
update logic. That has been addressed as well.
@amimas amimas force-pushed the issue-873-push-mirror branch from 8ab3864 to 602229e Compare January 24, 2026 17:23
@amimas amimas temporarily deployed to Integrate Pull Request January 24, 2026 17:23 — with GitHub Actions Inactive
@amimas amimas temporarily deployed to Integrate Pull Request January 24, 2026 17:23 — with GitHub Actions Inactive
@amimas amimas merged commit 37b78c9 into gitlabform:main Jan 24, 2026
20 checks passed
@amimas amimas deleted the issue-873-push-mirror branch January 24, 2026 17:54
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.

Support managing repository push mirroring settings

1 participant