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

fix(gitmodules): change URL of a git submodule from SSH to HTTPS #203

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

VincentCauchois
Copy link
Member

@VincentCauchois VincentCauchois commented Dec 5, 2023

DESCRIPTION

Change the URL for the git submodule "dependencies/GeoNature" from [email protected]/PnX-SI/GeoNature.git to https://github.com/PnX-SI/GeoNature.git so that the command git submodule update works properly instead of causing the following error :

[email protected]: Permission denied (publickey).
fatal: Impossible de lire le dépôt distant.

Veuillez vérifier que vous avez les droits d'accès
et que le dépôt existe.
fatal: Le clonage de '[email protected]:PnX-SI/GeoNature.git' dans le chemin de sous-module '/home/geonatureadmin/gn_module_export/dependencies/GeoNature' a échoué
Impossible de cloner 'dependencies/GeoNature'. Réessai prévu

ADDITIONAL INFORMATION

Use the following command, after retrieving the present commit, to repair after a failed git submodule update: git submodule sync --recursive.

Change the URL for the submodule "dependencies/GeoNature" from "git@[...]" to "https://[...].

Signed-off-by: VincentCauchois <[email protected]>
@VincentCauchois VincentCauchois added the bug Something isn't working label Dec 5, 2023
@VincentCauchois VincentCauchois self-assigned this Dec 5, 2023
Copy link

codecov bot commented Dec 5, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2adb037) 62.48% compared to head (135bd58) 62.48%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #203   +/-   ##
========================================
  Coverage    62.48%   62.48%           
========================================
  Files           12       12           
  Lines          685      685           
========================================
  Hits           428      428           
  Misses         257      257           
Flag Coverage Δ
pytest 62.48% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jacquesfize
Copy link
Contributor

Tu as la même erreur avec monitorings ?

https://github.com/PnX-SI/gn_module_monitoring/blob/main/.gitmodules

@jacquesfize jacquesfize merged commit caec603 into develop Dec 7, 2023
8 checks passed
@VincentCauchois
Copy link
Member Author

Tu as la même erreur avec monitorings ?

https://github.com/PnX-SI/gn_module_monitoring/blob/main/.gitmodules

Côté GINCO DEPOBIO, le module Monitoring n'est installé que sur une instance qui est une instance de Test avec Docker. Je n'y ai pas observé le problème, mais il faudrait que je vérifie en rentrant dans le container si le sous-module est bien initialisé correctement.

@VincentCauchois
Copy link
Member Author

Tu as la même erreur avec monitorings ?

https://github.com/PnX-SI/gn_module_monitoring/blob/main/.gitmodules

Pour Monitoring voir la PR suivante : PnX-SI/gn_module_monitoring#294

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants