Skip to content

Conversation

@KelvinMutuma
Copy link
Contributor

123Haynes and others added 4 commits September 18, 2015 19:49
We didn't specify that the db user needs the right to create
temporary tables before.
That's why the migration will fail on most installations.
This commits removes the "TEMPORARY" from the migration
and changes the documentation to include the necessary rights for the future.
Fix failing migration on mysql

We didn't specify that the db user needs the right to create temporary
tables before. That's why the migration will fail on most installations
that installed gitlab according to our documentation. This MR removes
the "TEMPORARY" from the migration and changes the documentation to
include the necessary rights for the future.

Fixes #2590

See merge request !1351
@KelvinMutuma
Copy link
Contributor Author

@dosire

Copy link
Member

Choose a reason for hiding this comment

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

You can't modify an existing migration I think.

Copy link
Member

Choose a reason for hiding this comment

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

This looks like a merge from a22b6c5. @KelvinMutuma, can you rebase from master again?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@stanhu I have since made a rebased merge request on gitlab.

However I am getting strange build errors:

Progress: |WARNING: This version of Gitlab depends on gitlab-shell 2.6.5, but you are running on Unknown. Please update gitlab-shell. -> That's seems a problem of a outdated gitlab-shell on the build machine.

screen shot 2015-09-21 at 10 56 10 am

screen shot 2015-09-21 at 10 56 14 am

screen shot 2015-09-21 at 10 55 48 am

On the gitlab merge request I also refactored the code to take note of @dosire's scalability concern and uploaded screenshots of the changes as stated on the contribution guidelines.

I have restarted the builds to see if they will pass.

PS: I felt that gitlab would be a better place to push a merge request but I should have been prudent enough to cancel this pull request since.

@stanhu
Copy link
Member

stanhu commented Sep 21, 2015

Can you post screenshots of this change? I'm not sure if the total repository size needs to be shown in the dashboard page, where commonly-used data is presented. Maybe it's sufficient to show it only in the groups view page.

@stanhu
Copy link
Member

stanhu commented Sep 21, 2015

Closing this in favor of https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/1366/.

@stanhu stanhu closed this Sep 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants