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

Improve GitLab support #66

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Improve GitLab support #66

wants to merge 20 commits into from

Conversation

erral
Copy link
Member

@erral erral commented Oct 10, 2024

With this PR I bring back GitLab CI/CD support with the following features:

Add a question whether to use GitLab based deployment. If so, add a .gitlab-ci.yml file that will add the required job pipelines which check, test, build, release and deploy the containers.

I have slightly changed the generated docker image names only when GitLab deployment is enabled because of GitLab peculiarities: in a single GitLab repo you can save different docker files, but their name must match the structure gitlab_url/gitlab_username/project_slug/docker_image/variant so, instead of having registry.gitlab.com/codesyntax/project-backend we are having registry.gitlab.com/codesyntax/project/backend (- vs. /)

I am open to discuss the release procedure stated in the CI file, which I tried to copy from the GHA procedure. But we can discuss or change it later too.

@erral erral requested a review from ericof as a code owner October 10, 2024 06:49
@erral erral requested a review from sneridagh as a code owner October 14, 2024 06:23
@erral erral changed the title Improve GitLab support [DRAFT] Improve GitLab support Oct 14, 2024
@davisagli
Copy link
Member

@erral What's the status of this one? I'd like to include it, but I can't easily test it myself.

@erral
Copy link
Member Author

erral commented Nov 6, 2024 via email

@erral
Copy link
Member Author

erral commented Nov 8, 2024

@stevepiercy can you review the documentation I added in the README_GITLAB.md file please?

@erral erral changed the title [DRAFT] Improve GitLab support Improve GitLab support Nov 8, 2024
@erral
Copy link
Member Author

erral commented Nov 8, 2024

I have tested this on gitlab, with the following result:

2024-11-08 17-14-46

Copy link
Contributor

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

Some grammar fixes, GitHub Markdown syntax flavoring, and a few questions. Otherwise docs look good.

project/cookiecutter.json Outdated Show resolved Hide resolved
project/hooks/post_gen_project.py Outdated Show resolved Hide resolved
Copy link
Contributor

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

Docs and comments LGTM. Needs a technical review. Thank you!

@ericof
Copy link
Member

ericof commented Nov 13, 2024

@erral Could you sync this with the main branch?

@erral
Copy link
Member Author

erral commented Nov 13, 2024

@erral Could you sync this with the main branch?

done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants