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 documentation on how to update Gitea to new version #4500

Merged
merged 5 commits into from
Jul 30, 2018
Merged

Add documentation on how to update Gitea to new version #4500

merged 5 commits into from
Jul 30, 2018

Conversation

bkroll
Copy link
Contributor

@bkroll bkroll commented Jul 23, 2018

Providing documentation for updating gitea, as asked in #3817

@codecov-io
Copy link

codecov-io commented Jul 23, 2018

Codecov Report

Merging #4500 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #4500   +/-   ##
======================================
  Coverage    20.2%   20.2%           
======================================
  Files         156     156           
  Lines       31140   31140           
======================================
  Hits         6292    6292           
  Misses      23890   23890           
  Partials      958     958

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b11ddd8...1f002e1. Read the comment docs.

@bkcsoft bkcsoft added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 23, 2018
@lafriks
Copy link
Member

lafriks commented Jul 23, 2018

I think it should note to do backup and stop gitea before replacing binary

@lafriks lafriks added the type/docs This PR mainly updates/creates documentation label Jul 23, 2018
@lafriks lafriks changed the title documentation on updating as asked in go-gitea/gitea#3817 Add documentation on how to update Gitea to new version Jul 23, 2018
@@ -90,6 +90,20 @@ cp gitea /usr/local/bin/gitea

See how to create [Linux service]({{< relref "run-as-service-in-ubuntu.en-us.md" >}})

## Updating to a new version

You can update to a new version of gitea by stopping gitea, replacing the binary at `/usr/local/bin/gitea` and restarting the instance.
Copy link
Member

Choose a reason for hiding this comment

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

What if the gitea binary is at another location?

Copy link
Member

Choose a reason for hiding this comment

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

@JonasFranzDEV installation from binary recommends this location so we should stick to this location

Copy link
Member

Choose a reason for hiding this comment

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

Fair enough

@ghost
Copy link

ghost commented Jul 25, 2018

@bkroll please fix build error

@appleboy appleboy added this to the 1.6.0 milestone Jul 26, 2018
Copy link
Member

@sapk sapk left a comment

Choose a reason for hiding this comment

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

Thanks for the contrib.

@bkcsoft bkcsoft added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jul 27, 2018
binary.

To solve this, go to the admin options and run the task `Resynchronize pre-receive,
update and post-receive hooks of all repositories` to update all hooks to contain
Copy link
Member

Choose a reason for hiding this comment

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

Please add warning that if a user has custom Git hooks this will overwrite them (I know that the admin dashboard has it, but more warning is better).

Also, another debugging step would be to re-write the authorized key file, as it puts the path to the binary in that file and if the binary name changes then this will also have issues when pushing via SSH.

@bkroll
Copy link
Contributor Author

bkroll commented Jul 30, 2018

OK I'm off at this point. I just tried to contribute that piece of documentation on how to update in general, as #3817 was the only instruction on this that I could find. I'm afraid I can't deliver a more detailed documentation than I did so far, because of a lack of both Gitea experience and time.

I have never used custom git hooks and I'm not sure what you would like to see written on the authorized key file (#4500 (review)), so someone else would need to chime in here.

@techknowlogick
Copy link
Member

@bkroll I can add the documentation I requested. Is it ok if I modify your PR?

@bkroll
Copy link
Contributor Author

bkroll commented Jul 30, 2018

Yes, sure @techknowlogick. Thanks!

@bkcsoft bkcsoft added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jul 30, 2018
@techknowlogick
Copy link
Member

Thanks @bkroll 😄

@techknowlogick techknowlogick merged commit d572336 into go-gitea:master Jul 30, 2018
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants