Skip to content

Conversation

@tobiashuste
Copy link
Member

@tobiashuste tobiashuste commented Apr 28, 2025

Starting with GitLab 18.0 git_data_dirs will be removed. They need to be replaced according to: https://docs.gitlab.com/omnibus/settings/configuration/#migrating-from-git_data_dirs

@Normo Do we want to provide a fallback for older GitLab versions? In principle the new method is now supported for quite a couple of releases.

Related to #366

@tobiashuste tobiashuste self-assigned this Apr 28, 2025
@tobiashuste tobiashuste requested a review from a team as a code owner April 28, 2025 12:51
@github-actions github-actions bot added the gitlab Related to hifis.gitlab role label Apr 28, 2025
@tobiashuste tobiashuste force-pushed the replace-git-data-dirs branch from 1edd95f to 73e9ad5 Compare April 28, 2025 12:59
@Normo Normo self-requested a review April 28, 2025 13:05
@tobiashuste
Copy link
Member Author

@Normo Could you please review this one?

@Normo
Copy link
Member

Normo commented May 19, 2025

@tobiashuste Thanks, that looks reasonable to me.

One thing we definitely need to consider: gitaly[‘configuration’] is also used to configure authentication and network listeners for example. So far, we have done this via the gitlab_additional_configurations variable as follows:

gitlab_additional_configurations:
  - gitaly:
      - key: ‘configuration’
        type: ‘plain’
        value: ‘{{ gitaly_configuration }}’

We need to make sure that it is merged and not overwritten.

@tobiashuste
Copy link
Member Author

@tobiashuste Thanks, that looks reasonable to me.

One thing we definitely need to consider: gitaly[‘configuration’] is also used to configure authentication and network listeners for example. So far, we have done this via the gitlab_additional_configurations variable as follows:

gitlab_additional_configurations
  - gitaly:
      - key: ‘configuration’
        type: ‘plain’
        value: ‘{{ gitaly_configuration }}’

We need to make sure that it is merged and not overwritten.

That's a good point. I'll have a look how this could be done.

@tobiashuste tobiashuste force-pushed the replace-git-data-dirs branch 11 times, most recently from fdebc31 to 7b621d1 Compare May 20, 2025 12:50
@tobiashuste tobiashuste force-pushed the replace-git-data-dirs branch from 7b621d1 to b4f3654 Compare May 21, 2025 07:16
@tobiashuste tobiashuste requested a review from Normo May 21, 2025 07:19
@Normo Normo added the breaking Backwards incompatible change label May 21, 2025
@Normo Normo added this to the v6.0.0 milestone May 21, 2025
@Normo Normo linked an issue May 21, 2025 that may be closed by this pull request
Starting with GitLab 18.0 git_data_dirs will be removed. They need to be
replaced according to: https://docs.gitlab.com/omnibus/settings/configuration/#migrating-from-git_data_dirs
This can happen if it is already present in
gitlab_additional_configurations. In this case send a warning that users
have to add the gitaly storage path themselves.
@Normo Normo force-pushed the replace-git-data-dirs branch from b4f3654 to c721b78 Compare May 21, 2025 08:58
@Normo Normo merged commit 177e410 into hifis-net:main May 21, 2025
9 checks passed
@Normo Normo removed the breaking Backwards incompatible change label May 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gitlab Related to hifis.gitlab role

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecate git_data_dirs setting

2 participants