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

--allow-releaseinfo-change #2208

Merged
merged 2 commits into from
Sep 16, 2021
Merged

Conversation

jneilliii
Copy link
Contributor

possible fix for recent install errors, #2206, #2207

possible fix for recent install errors, UnchartedBull#2206, UnchartedBull#2207
@UnchartedBull
Copy link
Owner

Thanks for leading all the discussions in the issues! Was kinda busy the last couple of weeks and didn't really had time to check in.

Just one question: Does this have any side implications, i.e. are there users that might be locked onto a different release-info?

@UnchartedBull
Copy link
Owner

Just searched around the internet on how to handle that. Information seems fairly sparse, but I was able to find this:

--allow-releaseinfo-change
  Allow the update command to continue downloading data from a repository which changed its information of the release 
  contained in the repository indicating e.g a new major release. APT will fail at the update command for such 
  repositories until the change is confirmed to ensure the user is prepared for the change. 
  See also apt-secure(8) for details on the concept and configuration.

So this would indicate, that this shouldn't be done by some script, that is being downloaded from the internet. I thought that maybe pinning the issue for a couple of weeks and adding an entry to the troubleshooting guide might also work?

@jneilliii
Copy link
Contributor Author

jneilliii commented Sep 8, 2021

From what we've seen it appears to only error out with apt-get, and not with just apt, so maybe that is an option. only issue is that it complains about not having a good CLI implementation, but from what I've seen with the CustoPiZer fixes with this implemented there were no bad side effects. I totally get not wanting to add it, but just trying to eliminate that from your support overhead, because it will continue...

@UnchartedBull
Copy link
Owner

Yeah I totally get that with the support request. Those are usually easy to handle, but I'm afraid of maybe a ticket opening up that basically says: "hey guys, you crashed my linux installation. Thank you very much."

Interesting finding with apt and apt-get though. The script is already using apt though, so this might not apply in this case?

Another option would be to check for that error message and directly tell the user: Hey this happened, here is how to fix that. Might need to investigate this a little.

@jneilliii
Copy link
Contributor Author

Yeah, a sane error is definitely something to consider and I get your point. Guysoft is struggling with the same thing and I believe wants to see if this is an upstream issue with the pi-gen project.

@jneilliii
Copy link
Contributor Author

from what I understand from the linked issues, some people have mentioned just running the sudo apt update command ahead of the install script is enough to allow the install script to work, so maybe catch the error and re-run is an option.

@UnchartedBull
Copy link
Owner

UnchartedBull commented Sep 15, 2021

Hi, I updated your branch with the error message (hope that is ok). It's fairly dumb and shows the error message if any error occurs with apt, but it should be fine I guess. It now looks like this if apt throws an error:

Screenshot 2021-09-15 at 13 30 01

This can be merged if you're also ok with the changes :)

@jneilliii
Copy link
Contributor Author

No problem on my end.

@UnchartedBull UnchartedBull merged commit 5598378 into UnchartedBull:main Sep 16, 2021
@jneilliii jneilliii deleted the patch-1 branch December 31, 2022 22:17
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.

None yet

2 participants