Skip to content

fix: add rust installation to dockerfile#282

Open
claudiunicolaa wants to merge 1 commit into
blockchain-certificates:masterfrom
claudiunicolaa:fix-dockerfile
Open

fix: add rust installation to dockerfile#282
claudiunicolaa wants to merge 1 commit into
blockchain-certificates:masterfrom
claudiunicolaa:fix-dockerfile

Conversation

@claudiunicolaa

Copy link
Copy Markdown

related #281

@lemoustachiste

Copy link
Copy Markdown
Member

do you know why rust is needed?

@claudiunicolaa

Copy link
Copy Markdown
Author

do you know why rust is needed?

I don't know. I tried to install the project locally via Dockerfile and ran into this problem (see issue assigned to PR)

@lemoustachiste

Copy link
Copy Markdown
Member

I'm not sure this is a cert-issuer only issue and I don't have leeway at this time to try and replicate it, nor to understand it.

As such I am reluctant to add this into the codebase. This is a python project and not a rust project so this feels a bit off.

If you have managed to get cert-issuer working on your machine by applying this fix, then great.
I would like to see more people facing this issue before introducing it into the code base.

@NachoG13

Copy link
Copy Markdown

I have the same problem

@gaojindev

Copy link
Copy Markdown

I ran into the same problem

@lemoustachiste

Copy link
Copy Markdown
Member

@NachoG13 @gaojindev did this fix unblock you?

@gaojindev

Copy link
Copy Markdown

It seems to been fixed by that but after building the container the pc got very slow and i found a strange process
image

@lemoustachiste

Copy link
Copy Markdown
Member

Thanks for testing and reporting @gaojindev

I'm still focused on other fronts at the moment but I'll try to make some room to investigate the issue. Not merging this PR yet.

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