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

Add support for building the enterprise edition #1477

Closed
wants to merge 1 commit into from

Conversation

grimmy
Copy link

@grimmy grimmy commented Jan 19, 2018

This is a repost of PR 1428 on it's own branch this time.

This PR adds support for building the enterprise edition as well. This is controlled via a Docker build argument. It defaults to building the community edition but you can build the enterprise edition with

docker build --build-arg GITLAB_EDITION=ee -t sameersbn/gitlab:10.2.2-ee .

I kept the documentation changes light as I figured you might want to tweak them a bit.

@lordvlad
Copy link
Contributor

would love to see this merged

@tomislater
Copy link
Contributor

Any news? It would be a great option.

@snics
Copy link

snics commented Sep 11, 2018

Yeah, would it be nice if someone could comment on that?

@martinspaeth
Copy link

I would like to give ee a try using this patch. Do one know if the container ce and ee are compatible? I want to be able to switch back to ce if ee din’t Match our expectations...

Thank you for your help

@tomislater
Copy link
Contributor

We have been using it for three months without a hitch.

@martinspaeth
Copy link

Great! Thanks for your fast response @tomislater. Two more questions. If I run the EE container do I need to provide a license key directly or does it run with the ce features until I provide the corresponding key? And what Happens if I use it as EE and the key expires.

@tomislater
Copy link
Contributor

@martinspaeth
Copy link

Would be great to see this accepted.

I have struggle to get the image built (#1743). Having the original image from this project would be way nicer!

gjrtimmer referenced this pull request in gjrtimmer/docker-gitlab Dec 14, 2018
Fixes #1477

Allow commandline override for GITLAB_EDITION to config.yml
@reynold-lariza
Copy link
Contributor

reynold-lariza commented Mar 9, 2019

Hi @grimmy and everyone, I was able to build a container using this patch -- but when I tried this on to our existing Gitlab CE 11.8.1, Gitlab stuck on the temp page (the page when its still just starting up), also workhorse and gitaly failed to start. Was able to revert back to CE, but unable to migrate to EE :(

@martinspaeth
Copy link

Is anyone here running ee containerized with those images lately? I would love to switch to ee but I don’t want to give up the current docker setup...

@DZDomi
Copy link

DZDomi commented Mar 27, 2019

We are currently running a GitLab EE instance (11.6) with this image and also this PR. Everything is working great so far. I only had to change 1 setting in the nginx configuration cause we are using Gitlab Pages with custom domain names and TLS. But besides that all features are working as intended.

@Maghin
Copy link

Maghin commented Apr 3, 2019

Hi,

I've been running Gitlab-EE 11.3.2 for 6 month with this image. You just have to replace your CE container with a new one created from this image. Nothing more to do. This works perfectly except that I lost the user SSH keys but I think it's not related.

I just want to warning about the license model Gitlab-EE uses : you have to pay a license for every "active user". An active user is in fact an "active account", no matter if he has already connected to Gitlab or not. So before migrating be sure to block every non used account.
On our little business it means we can no longer use automatic/bot user, LDAP authentication, or create a temporary access to a client to manage an issues (I can not explain my boss that he as to pay an annual license for a 5 minutes usage...).

Also good to know : Gitlab does not answer to questions about the "starter license" (probably not profitable enough).

It's a bit disappointing and I'm now going back to CE...

@elitvinov
Copy link

@grimmy, can you please merge upstream and resolve conflicts.
Your patch to use Gitlab EE edition seem to be still working

@@ -50,6 +50,7 @@ RUN apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv E1DD270288B4E60
&& rm -rf /var/lib/apt/lists/*

COPY assets/build/ ${GITLAB_BUILD_DIR}/
ARG GITLAB_EDITION=ce
Copy link
Contributor

@splattael splattael Feb 7, 2020

Choose a reason for hiding this comment

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

Since GitLab moved to a single codebase (see https://about.gitlab.com/blog/2019/08/23/a-single-codebase-for-gitlab-community-and-enterprise-edition/) the naming has changed (although the redirects still work!). It's now gitlab-org/gitlab-foss (suffix is -foss; former -ce) and gitlab-org/gitlab (no suffix; former -ee).

FYI removing the EE folder from gitlab-org/gitlab (rm -fr ./ee) gives you basically the FOSS version.

Edit: It's a bit more than just removing ee/: https://gitlab.com/gitlab-org/merge-train/-/blob/034051459339b14d4244e557f6c6429a5d0273fd/bin/merge-train#L141-145

So, maybe it's safer to clone twice.

Choose a reason for hiding this comment

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

Correct, both EE => CE and CE => EE procedures require some manual work:

  1. https://gitlab.com/gitlab-org/gitlab/-/blob/master/doc/downgrade_ee_to_ce/README.md
  2. https://gitlab.com/gitlab-org/gitlab/-/blob/master/doc/update/upgrading_from_ce_to_ee.md

Some or procedures are not applicable to docker environments as they are applied at container start or build (cache clearing, deps installation, db migrations). So migrations are also the discussion point for test and documenting using sameersbn docker images.

I mean now conflict resolving is mainly use the new url for CE, as it made in master at 7ca29df

@stale
Copy link

stale bot commented May 6, 2020

This issue has been automatically marked as stale because it has not had any activity for the last 60 days. It will be closed if no further activity occurs during the next 7 days. Thank you for your contributions.

@stale stale bot added the wontfix label May 6, 2020
@stale stale bot closed this May 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants