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.

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

Adding support for self-managed GitLab instances #21274

Merged
merged 3 commits into from May 13, 2023

Conversation

darbyfrey
Copy link
Contributor

@darbyfrey darbyfrey commented May 12, 2023

This MR resolves this issue by adding a config option for gitlab_host. The option defaults to the gitlab.com host, but can be configured for folks running self-managed gitlab instances.

Checklist

  • I've run bundle exec rspec from the root directory to see all new and existing tests pass
  • I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
  • I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary.

Motivation and Context

The first iteration of the gitlab_secure_files storage backend only supported gitlab.com since that feature wasn't available for self managed instances. It's now been available for several months, so it's about time we added support here as well. This limitation was also reported in #21197.

Description

This change adds an additional configuration set to the match init process and adds the gitlab_host option to the Matchfile

Testing Steps

You can test this change by running fastlane match init and verifying that the gitlab_host option is written to the Matchfile

The configuration is also printed in the output when running fastlane match. For example:

 Initializing match for GitLab project root/ios-demo-components-demo on https://gitlab.com/api/v4

@darbyfrey
Copy link
Contributor Author

Hey @getaaron, I've got an update on the gitlab backend here, mind giving it a look when you have some time?

@getaaron getaaron merged commit 7b3e224 into fastlane:master May 13, 2023
8 checks passed
markhomoki added a commit to markhomoki/fastlane that referenced this pull request Jul 5, 2023
Improvements

* [core][match] remove obsolete and expired WWDR G1 certificate (fastlane#21271) via Frederik Seiffert (@triplef)
* [action][ensure_git_status_clean] new ignore_files option for explicitly ignoring files (fastlane#21283) via Josh Holtz (@joshdholtz)
* [scan] run simulator destination with arch=x86_64 for Xcode 14.3 and up if on Intel (fastlane#21284) via Josh Holtz (@joshdholtz)
* [match] adding support for self-managed GitLab instances (fastlane#21274) via Darby Frey (@darbyfrey)
* [pilot] fix increase limit for build query (fastlane#21212) via Eric Lindvall (@eric)
* [dependency] relax `multipart_post` dependency version requirement (fastlane#20870) via Edouard Brière (@edouard)

https://github.com/fastlane/fastlane/releases/tag/2.213.0
smalbs added a commit to potatotrain/fastlane that referenced this pull request Nov 16, 2023
* commit '0ede7e768d97d7e22e576d72ab8de96272c479a5':
  Version bump to 2.213.0 (fastlane#21286)
  [core] Remove obsolete and expired WWDR G1 certificate (fastlane#21271)
  [action][ensure_git_status_clean] new ignore_files option for explicitly ignoring files (fastlane#21283)
  [scan] run simulator destination with arch=x86_64 for Xcode 14.3 and up if on Intel (fastlane#21284)
  [match] Adding support for self-managed GitLab instances (fastlane#21274)
  [pilot] Fix increase limit for build query (fastlane#21212)
  [dependency] relax `multipart_post` dependency version requirement (fastlane#20870)
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.

fastlane match gitlab_secure_files does not work with private gitlab instance
2 participants