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

Create docker images #194

Merged
merged 9 commits into from
May 27, 2024
Merged

Create docker images #194

merged 9 commits into from
May 27, 2024

Conversation

antarcticrainforest
Copy link
Member

1 this PR simply adds the auto creation of the freva-dev (container with conda , MySQL and solr without freva pre-installed) and the freva container (samve as frva-dev but freva lib installed).

@MoSHad91 you can use these containers (once merged) in the gitlab-ci for the metadata-crawler and your plugin template.

@mo-dkrz
Copy link
Contributor

mo-dkrz commented May 17, 2024

Great @antarcticrainforest, this afternoon I go for reviewing and taking a test on CI of metadata-crawler

Copy link
Contributor

@mo-dkrz mo-dkrz left a comment

Choose a reason for hiding this comment

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

Look good to me, also u can resolve this comment without any changes. it's more about making the code more neat and readable ... deserve to be ignored :)

@mo-dkrz
Copy link
Contributor

mo-dkrz commented May 17, 2024

@antarcticrainforest btw i forgot to say, when u merged the PR, I would take a test on the metadata-crawler CI using the newest image ...

@antarcticrainforest
Copy link
Member Author

Any more suggestions?

@mo-dkrz
Copy link
Contributor

mo-dkrz commented May 27, 2024

all good imo

@antarcticrainforest antarcticrainforest merged commit debce12 into main May 27, 2024
21 checks passed
@antarcticrainforest antarcticrainforest deleted the create-docker-images branch May 27, 2024 12:10
@antarcticrainforest antarcticrainforest restored the create-docker-images branch May 27, 2024 12:14
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.

2 participants