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

Feat : draft gogs integration (WIP) #166

Draft
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

AlexFernandes-MOVAI
Copy link
Contributor

@AlexFernandes-MOVAI AlexFernandes-MOVAI commented Jul 13, 2022

3.1 - GIT server on CE

New service to be installed on Manager / Flow :

  • GIT server : gogs
  • Single authentication : to be checked
  • Allow anonymous access on GIT : to be checked

Launch it :

export APP_PATH=$PWD
/scripts/movai-flow-launch

Configure Gogs : http://localhost:30080/

Database infos : gitdb, user: gitmovai, password: gitmovai

SSH access : localhost:30022

Challenges:

  • initial SSH configuration for GitServer, MovaiBackend:

    • validate DAL can access the GIT server using SSH / HTTPS (user, password)
  • initial MOVAI repositories creation :

    • populate the server with MOVAI baselines (nodes, flow, cbs, ...)
    • import or fork the necessary repositories
    • create users repositories

    once the process is clear a master git server can then be deployed for whole R&D

  • plug backend, spawner, health-node containers to these repositories local clones

    • replace redis-local / redis-master
    • handle git commit / git stash / git checkout .... operations
  • packages compilation and generation will be handled by some complementary orchestration (mobros spawnable containers):

    • multiple ros distributions workspaces creation
    • c++ compilation
    • deb pack ===> apt repository
    • docker build ===> docker registry

  • Make sure you are opening from a topic/feature/bugfix branch
  • Ensure that the PR title represents the desired changes
  • Ensure that the PR description detail the desired changes
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@guide-bot
Copy link

guide-bot bot commented Jul 13, 2022

Thanks for opening this Pull Request!
We need you to:

  1. Complete the activities.

    Action: Complete Link to relevant pull requests, esp. upstream and downstream changes
    Action: Complete Ensure you have provided tests - that demonstrates feature works or fixes the issue

    If an activity is not applicable, use '~activity description~' to mark it not applicable.

@AlexFernandes-MOVAI AlexFernandes-MOVAI self-assigned this Jul 13, 2022
@AlexFernandes-MOVAI AlexFernandes-MOVAI added enhancement New feature or request help wanted Extra attention is needed status: in progress Issue is currently in progress of being addressed labels Jul 13, 2022
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@engdorm
Copy link
Contributor

engdorm commented Jul 13, 2022

didn't we want to use git tea?

@AlexFernandes-MOVAI
Copy link
Contributor Author

didn't we want to use git tea?

@Mograbi told us the 2 tools were similar and a fork of each other.

Here we just want to POC the GIT solution and communication mechanism don't we ?

After we can change the git backend server without impacts ? right ?

@Mograbi
Copy link

Mograbi commented Aug 14, 2022

@AlexFernandes-MOVAI I think this should be open to a seperate branch first and not dev

@MithunKinarullathil MithunKinarullathil marked this pull request as draft September 27, 2022 16:01
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 6, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed status: in progress Issue is currently in progress of being addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants