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

feature/MV 1829 Use OpenEB Dockerfile to build and test on GitHubActions #105

Conversation

esecondy-psee
Copy link

No description provided.

@esecondy-psee esecondy-psee force-pushed the feature/MV-1829-Use-OpenEB-dockerfile-to-build-&-test-on-GitHubAction branch 18 times, most recently from 66de3f5 to c169b44 Compare February 5, 2024 10:28
@esecondy-psee esecondy-psee force-pushed the feature/MV-1829-Use-OpenEB-dockerfile-to-build-&-test-on-GitHubAction branch from c169b44 to 4e26156 Compare February 5, 2024 10:33
@esecondy-psee esecondy-psee force-pushed the feature/MV-1829-Use-OpenEB-dockerfile-to-build-&-test-on-GitHubAction branch 6 times, most recently from da87657 to 3d90ee5 Compare February 6, 2024 16:36
context:
file: .github/workflows/Dockerfile.OpenEB
push: true
tags: prophesee/openeb:openeb-ubuntu-20.04
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why not just this ?

Suggested change
tags: prophesee/openeb:openeb-ubuntu-20.04
tags: prophesee/openeb:ubuntu-20.04

Copy link
Author

Choose a reason for hiding this comment

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

Look at this link : https://hub.docker.com/repository/docker/prophesee/openeb/tags?page=1&ordering=last_updated
The first image is the one that I use and the second one is the suggestion that you put, and you will understand that if we don't specify openeb here we won't know the version of openeb. We know that we are in prophesee/openeb thanks to the repository, but the image will be called only ubuntu-20.04. I want the image to be called openeb-ubuntu-20.04 so we know we are talking about the openeb image

@esecondy-psee esecondy-psee force-pushed the feature/MV-1829-Use-OpenEB-dockerfile-to-build-&-test-on-GitHubAction branch from 3d90ee5 to c687142 Compare February 6, 2024 17:44
@esecondy-psee esecondy-psee force-pushed the feature/MV-1829-Use-OpenEB-dockerfile-to-build-&-test-on-GitHubAction branch from c687142 to 2d1abf3 Compare February 7, 2024 09:24
@esecondy-psee esecondy-psee force-pushed the feature/MV-1829-Use-OpenEB-dockerfile-to-build-&-test-on-GitHubAction branch from 2d1abf3 to 0cae898 Compare February 7, 2024 13:52
@esecondy-psee esecondy-psee force-pushed the feature/MV-1829-Use-OpenEB-dockerfile-to-build-&-test-on-GitHubAction branch from 449f221 to cbfaf4f Compare February 7, 2024 16:38
@esecondy-psee esecondy-psee force-pushed the feature/MV-1829-Use-OpenEB-dockerfile-to-build-&-test-on-GitHubAction branch from cbfaf4f to 34954a5 Compare February 7, 2024 16:39
@esecondy-psee esecondy-psee force-pushed the feature/MV-1829-Use-OpenEB-dockerfile-to-build-&-test-on-GitHubAction branch from c05bc73 to 3cc7935 Compare February 8, 2024 10:40
Copy link
Collaborator

@lbristiel-psee lbristiel-psee left a comment

Choose a reason for hiding this comment

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

Looks good and clean to me.
As far as I can tell there is no impact for users.
Now we need a technical approval from people skilled in the whole GitHub/CI system :-)

.github/workflows/Dockerfile.OpenEB Outdated Show resolved Hide resolved
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.

3 participants