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

Restart loop if update failed and suspicious binary_name mismatch in log-file #206

Open
dragetd opened this issue Dec 2, 2020 · 3 comments
Labels
quality Improve repurposability of the project without changing its behaviour windows specific

Comments

@dragetd
Copy link
Member

dragetd commented Dec 2, 2020

Describe the bug
A user reported a restart-loop and sent in some interesting logs. The logfiles cannot be attached to this issue because they container information not intended for public use.

I have the log saved next to my trivrost development directory and we can have a look next time we dive into the issues.

This bug serves as a reminder for us to look into it. Once we figured out what happened here, we can close this issue and open more relevant issues or fix the things right away. :)

@dragetd dragetd added this to the v1.5.0 milestone Dec 2, 2020
@dragetd dragetd modified the milestones: v1.5.0, v1.5.x Apr 14, 2021
@JulindM
Copy link

JulindM commented Sep 6, 2021

The problem with the restart loop is because Windows does not allow having 2 Folders with the same name and, to top it off, it is case insensitive! So for windows, having a folder named TEST and test it considers them the same.

This is the culprit to the problem! The vendorname was in our case company but an old artifact had the vendor Company. This mismatch caused the launcher to keep restarting, because it could not write into the folder company because it was occupied by Company.

@dragetd dragetd added the quality Improve repurposability of the project without changing its behaviour label Sep 7, 2021
@dragetd
Copy link
Member Author

dragetd commented Sep 7, 2021

Thank you very much for the input! This could very well be the issue.

Martin, we should go through all code path and check if we have messed up the case sensitivity handling.

@MMulthaupt
Copy link
Collaborator

MMulthaupt commented Sep 8, 2021

I completely missed this issue. We rushed in a fix here 8107351 which fixed the bug.

@theJuli I believe we wrote somewhere, that Vendor name and Product name should not change after the first release; i.e. I am not sure if above commit would fix your issue as well. You can try with v1.5.0-rc1 which is the first (and latest) release to contain the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
quality Improve repurposability of the project without changing its behaviour windows specific
Projects
None yet
Development

No branches or pull requests

3 participants