-
Notifications
You must be signed in to change notification settings - Fork 12
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
v0.6.2 fails when trying to run entrypoint.sh #50
Comments
Hey @hgaiser thank you for reporting this. May I know if you're using a custom Github runner or is it default? The change that you have linked checks for the |
Hey @BartoszBlizniak , thanks for the quick reply :). I am using |
The best approach would be to In your case it should be under: |
Hey @hgaiser - just wanted to check if you're consistently facing this issue? I wasn't able to replicate the behaviour using Perhaps if you can share your workflow I could take a look at replicating the issue with that. |
I was getting the same error in two different repositories. The workflow is as follows:
I removed some actions related to Poetry, I don't think they are relevant for this issue. I didn't output the variable yet. Do you still think it's relevant? |
Thank you @hgaiser - I was able to replicate the issue. It appears that this part of the code breaks the deployment:
I will have to investigate as to why it's doing that. Removing that line should still work and download the necessary python dependencies. |
Thanks for looking into this, happy to hear you could replicate the issue :). |
Hey @hgaiser - the "breaking" change was actually implemented as step to remove the need of having a docker container being pulled to setup the Cloudsmith GHA, we use "composite" instead of "docker". I think the correct way forward for time being would be to remove the container flag in general and use the defaults. I tried to dig around and play around with the code to get this work within a container but no luck. I've also stumbled across this Github Issue thread that to this day hasn't been properly addressed and it looks like it's contributing to the issue that we are seeing around this. |
Hmm but does that mean this action will be incompatible when using a container? For this specific workflow that's probably not an issue, but what about workflows where a container is required? For example for a project I'm running I require to compile in an Arch Linux environment. I use a docker container for that purpose. Does it mean that I can't use this action for that workflow? If so, I think the issue should remain open (with maybe a changed title) until this is resolved. Let me know what you think 👍 |
When using
cloudsmith-io/[email protected]
I get the following error:Running
cloudsmith-io/[email protected]
does work however.I noticed the following release message:
However I assume this didn't quite work.
The text was updated successfully, but these errors were encountered: