-
Notifications
You must be signed in to change notification settings - Fork 263
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
terminal prompts disabled #728
Comments
I'm having the same problem with original issue #621 in v0.13, after update to v0.13.1, I get the "terminal prompts disabled" error. |
@pasha-codefresh It's a big change, and imho should go into 0.14. We could go ahead and release a 0.14 soon, though. I do not see any issue with that :) |
same here. automatic upgrades are completely broken for me in v0.13.1. |
Since #726 is merged, I thought I could build and test the latest Am I missing something alongside changes in that PR? @jannfis / @pasha-codefresh could you help? Image Updater Logs
Thank you! |
I am experiencing the same issue on |
For some reason release 0.13.1 works for me, so looks like #726 is specifically what caused this issue 🤷 |
Found the issue and have a fix up. |
I think this bug hits only if you use an HTTPS based repository configuration. @Sovietaced, are you by chance using SSH connected repositories? |
I am using repositories setup with GitHub app credentials with an HTTPS URL. |
Is there any workaround for that? |
Is this fixed anyway? Running the latest build I'm still getting:
Annotations:
|
Thanks for testing and reporting, @alexismaior. Will dig into again. |
@alexismaior @jannfis could you test with the latest image? I tested this with my PR #756 and this should be working now. |
Yep. Tested with both HTTPS and SSH git credentials and the image updater was able to write back to git. Thanks, folks, for working on this. |
Are there any plans to release v0.13.2 in the short term with this fix? :D |
See #748 (comment) |
Describe the bug
The original issue was #621 but that's been closed. The newest release does not panic anymore, but as indicated in that now closed issue it left it's place to a new problem :
Downgrading back to 0.12 fixes the issue.
Version
v0.13.1
The text was updated successfully, but these errors were encountered: