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

chore: [WIP] Moving off and off-pro to off2 #243

Merged
merged 70 commits into from
Nov 23, 2023
Merged

Conversation

alexgarel
Copy link
Member

We are moving off and off-pro install to off2 !

sudo chmod g+rwX -R .
```

I will generaly work to modify / commit to the repository using my user alex, while using off only to push.
Copy link
Contributor

Choose a reason for hiding this comment

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

I tried doing that but I'm getting permission issues:

stephane@off-pro:/srv/off-pro$ git commit -m "fix: migration of off-pro import scripts (done by alex)" scripts
fatal: Unable to create '/srv/off-pro/.git/index.lock': Permission denied

If I add myself to the "off" group, I get a little bit past but there's a file not writable by the group:

stephane@off-pro:/srv/off-pro$ git commit -m "fix: migration of off-pro import scripts (done by alex)" scripts
fatal: could not open '.git/COMMIT_EDITMSG': Permission denied

Copy link
Contributor

Choose a reason for hiding this comment

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

ok I'm not sure how you manage to commit with the alex user. I managed to do it after adding myself to the off group and changing some permissions:

root@off-pro:/srv/off-pro/.git# ls -lrt | grep r--r--
-rw-r--r--  1 off off     25 Sep 14 13:33 COMMIT_EDITMSG
-rw-r--r--  1 off off  34684 Sep 14 14:57 FETCH_HEAD
root@off-pro:/srv/off-pro/.git# sudo chmod g+rwX -R COMMIT_EDITMSG FETCH_HEAD 

but then the ownership of the index file is changed to stephane...

I must be missing something.

Copy link
Member Author

Choose a reason for hiding this comment

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

No I myself generally commit with off, because I too have problem with permission… my comment above is wrong 😬

alexgarel added a commit to openfoodfacts/openfoodfacts-server that referenced this pull request Nov 23, 2023
…roxmox) (#8827)

- Migration of Open Food Facts (site and pro platform) from off1 (bare metal) to off2 (proxmox)
- Modifications linked to moving off and off-pro to off2 server. 
- This includes a lot of path modifications:
  - some directories where removed from the repository and are created dynamically
  - we try to differentiate folders containing code (only this is in git), data, cache data and dist files
  - we control this by using a fixed set of base path (see Paths.pm)
- Most of server configurations are now in the git repository
- Import scripts where reworked 

See also: openfoodfacts/openfoodfacts-infrastructure#243

Part of:
- openfoodfacts/openfoodfacts-infrastructure#182
@alexgarel alexgarel marked this pull request as ready for review November 23, 2023 13:35
Copy link
Contributor

@stephanegigandet stephanegigandet left a comment

Choose a reason for hiding this comment

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

Thank you!

@alexgarel alexgarel merged commit 54435e7 into develop Nov 23, 2023
5 checks passed
@alexgarel alexgarel deleted the off2-off-reinstall branch November 23, 2023 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants