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

Fails again with read only config #9164

Closed
2 of 7 tasks
brandonkal opened this issue Nov 26, 2019 · 5 comments · Fixed by #9171
Closed
2 of 7 tasks

Fails again with read only config #9164

brandonkal opened this issue Nov 26, 2019 · 5 comments · Fixed by #9171
Labels
issue/duplicate The issue has already been reported.

Comments

@brandonkal
Copy link

  • Gitea version (or commit ref):
  • Git version:
  • Operating system:
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:

Description

Config should be considered immutable. Gitea should never attempt to save a value to app.ini

Error saving generated INTERNAL_TOKEN to custom config: open /data/gitea/c │
│ onf/app.ini: read-only file system

Gitea should exit when it finds a fatal error. There is no sense looping over that issue.

Screenshots

@lunny
Copy link
Member

lunny commented Nov 26, 2019

I think there is already an issue to report that.

@lunny lunny added the issue/duplicate The issue has already been reported. label Nov 26, 2019
@bagasme
Copy link
Contributor

bagasme commented Nov 26, 2019

@lunny any issue that duplicate this?

@davidsvantesson
Copy link
Contributor

I think previous issues/prs are closed and considered resolved.
@bagasme did you read under Create required directory structure? https://docs.gitea.io/en-us/install-from-binary/

@zeripath
Copy link
Contributor

This is specific to docker. Gitea does exit when there is a fatal error.

The problem is that the way the docker is structured it will keep on trying to restart Gitea indefinitely.

@brandonkal
Copy link
Author

Yes. The docker container should exit and leave the restart logic to the orchestrator (docker-compose/kubernetes)

@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
issue/duplicate The issue has already been reported.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants