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

create services/install-gitlab.sh (fixes #2139) #2159

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

Conversation

JLKwong
Copy link
Member

@JLKwong JLKwong commented Apr 4, 2021

­

@JLKwong JLKwong closed this Apr 4, 2021
@JLKwong JLKwong reopened this Apr 4, 2021
@JLKwong JLKwong changed the title create tests/resolution.bats create services/install-gitlab.sh (fixes #2139) Apr 4, 2021
@JLKwong JLKwong marked this pull request as draft April 4, 2021 03:48
mkdir -p /srv/gitlab

# create yml(s)
{version: '3.3'
Copy link
Member

Choose a reason for hiding this comment

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

I get an error when I run services gitlab install:
gitlab-review-error
Maybe related to this line not having a space (check codeclimate https://codeclimate.com/github/treehouses/cli/pull/2159).

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed the Code Climate error. 😌🎉

@JLKwong JLKwong marked this pull request as ready for review April 5, 2021 18:20
Copy link
Member

@rjpadilla rjpadilla left a comment

Choose a reason for hiding this comment

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

Good: It works
Bad:

  • It takes about four minutes to boot up to the login screen.
  • My pi ends up bricked several minutes later.
  • Can't connect to port 443 and 2222

All in all, the bad points come from my end (I'm surprised I even made it to the gitlab page with my 2gbs of ram), so I'll give it my 👍

Copy link
Member

@rjpadilla rjpadilla left a comment

Choose a reason for hiding this comment

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

FXML flashbacks intensifies

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.

2 participants