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

First draft for GitLab CE app #1252

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hartzell
Copy link

First draft of an app for GitLab Community Edition.

I can run:

.github/scripts/ci.py --app gitlab-ce --train community --test-file basic-values.yaml

successfully.

I can also run the result docker-compose.yaml file manually and interact with the service.

I believe I have all of the weirdness knocked out, but it's been a game of whack-a-mole, rather than first-principles.

TODO: it could use a bit of instructional information about e.g. changing the initial root password and/or working with the gitlab.rb file. The app has taken ownership of the GITLAB_OMNIBUS_CONFIG environment variable, it might be nice to let users append their own bits to it.

I'd appreciate any feedback on the app.

First draft of an app for GitLab Community Edition.

I can run:

  .github/scripts/ci.py --app gitlab-ce --train community --test-file basic-values.yaml

successfully.

I can also run the result docker-compose.yaml file manually and
interact with the service.

I believe I have all of the weirdness knocked out, but it's been a
game of whack-a-mole, rather than first-principles.

TODO: it could use a bit of instructional information about
e.g. changing the initial root password and/or working with the
gitlab.rb file.  The app has taken ownership of the
`GITLAB_OMNIBUS_CONFIG` environment variable, it might be nice to let
users append their own bits to it.

I'd appreciate any feedback on the app.
@hartzell hartzell marked this pull request as ready for review December 29, 2024 16:40
I failed to clean up all of the bits in the questions.yaml file when I
stole it from another app.

This fixes those bits.
@kmoore134 kmoore134 requested a review from stavros-k January 2, 2025 13: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.

1 participant