Skip to content

Dev v0.2.28 Targeted to Be New Master Release#27

Merged
ExtremeFiretop merged 33 commits intomainfrom
dev
Dec 7, 2023
Merged

Dev v0.2.28 Targeted to Be New Master Release#27
ExtremeFiretop merged 33 commits intomainfrom
dev

Conversation

@ExtremeFiretop
Copy link
Owner

Dev v0.2.28 Targeted to Be New Master Release

ExtremeFiretop and others added 30 commits November 29, 2023 21:12
Document to keep track of known issues, areas to improve,  general ideas and possible enhancements.
Reverted some changes related to setting the directory paths for the ZIP & LOG files as it was getting the wrong paths when changing the default setting to a different directory. Some more code improvements.
Update MerlinAU.sh in Dev branch.
Code changes & improvements to handle the case where the expected USB-attached drive path set up to download the ZIP file is not found at the time the F/W update check is run. In other situations, if the directory is not located we select the $HOME path and ask the user to confirm when running in "menu mode." Otherwise, we continue wit the temporary selection made.
Fixed bug causing the ZIP file to be deleted even when located on a USB-attached drive where Entware is installed & some code improvements in "Get_Custom_Setting()" function. Also modified "_DoCleanUp_" function to try to keep the ZIP file from being deleted under some conditions. Minor cleanup.
@ExtremeFiretop ExtremeFiretop merged commit 4ee5128 into main Dec 7, 2023
@Martinski4GitHub
Copy link
Collaborator

Dev v0.2.28 Targeted to Be New Master Release

I just noticed in the README a note in the "Remaining/Planned Features" section saying "Be aware that JFFS partitions may not work post-upgrades in some cases."
What is that about? Have you had that happen to your router?
Just curious about where exactly that's coming from and what the context is/was.

@ExtremeFiretop
Copy link
Owner Author

Dev v0.2.28 Targeted to Be New Master Release

I just noticed in the README a note in the "Remaining/Planned Features" section saying "Be aware that JFFS partitions may not work post-upgrades in some cases." What is that about? Have you had that happen to your router? Just curious about where exactly that's coming from and what the context is/was.

Martinski! I gotchu, I had to do a bit of lookup to find where this "came from" but it came from Yota in the snbforums when we were discussing the original design and limitations of this script. It's not "new" in the readme, it's been there since the start.

Specifically this post: https://www.snbforums.com/threads/seeking-feedback-contributions-merlin-auto-update-solutions.87044/post-868240

Where he says:

"A suggested idea is: prerequisite the router must have a USB drive connected, and then save the configuration files, jffs backup and current and newer versions of firmware in the USB drive before each update, because this is the only persistent storage location that has enough storage space and will work when router jffs doesn't work. (Yes, we have to assume that in some cases the jffs partition does not work after upgrading, because in history, they have adjusted the jffs partition by upgrading the firmware, and this kind of thing may cause jffs to not be mounted or need to be initialized)."

I have not personally experienced this myself, but apparently it's a risk we had to identify/mention.

@Martinski4GitHub
Copy link
Collaborator

Martinski4GitHub commented Dec 8, 2023

"A suggested idea is: prerequisite the router must have a USB drive connected, and then save the configuration files, jffs backup and current and newer versions of firmware in the USB drive before each update, because this is the only persistent storage location that has enough storage space and will work when router jffs doesn't work. (Yes, we have to assume that in some cases the jffs partition does not work after upgrading, because in history, they have adjusted the jffs partition by upgrading the firmware, and this kind of thing may cause jffs to not be mounted or need to be initialized)."

I have not personally experienced this myself, but apparently it's a risk we had to identify/mention.

Ah OK, got it. Thanks for the clarification. I think the note in the README should be clearer about the context. The way it's written now makes it sound as if the F/W update process itself could cause JFFS to fail, whereas the situation is more nuanced.

The rare scenario can happen when a new F/W update includes a resizing or reconfiguration of the JFFS partition, in which case JFFS must be reformatted & reinitialized so any custom configs/scripts/files are deleted. IOW, it's not the case that "JFFS may not work" after the update; it's just that after JFFS has been reformatted & reinitialized during or following the F/W update, any custom configuration files, scripts, etc. are deleted so it "appears" as if "it's not working." I believe this happened once in the last 5 years, so yes it can happen but it's very rare, and the Release Notes usually point this out to users.

My 2 cents.

@ExtremeFiretop
Copy link
Owner Author

ExtremeFiretop commented Dec 8, 2023

"A suggested idea is: prerequisite the router must have a USB drive connected, and then save the configuration files, jffs backup and current and newer versions of firmware in the USB drive before each update, because this is the only persistent storage location that has enough storage space and will work when router jffs doesn't work. (Yes, we have to assume that in some cases the jffs partition does not work after upgrading, because in history, they have adjusted the jffs partition by upgrading the firmware, and this kind of thing may cause jffs to not be mounted or need to be initialized)."
I have not personally experienced this myself, but apparently it's a risk we had to identify/mention.

Ah OK, got it. Thanks for the clarification. I think the note in the README should be clearer about the context. The way it's written now makes it sound as if the F/W update process itself could cause JFFS to fail, whereas the situation is more nuanced.

The rare scenario can happen when a new F/W update includes a resizing or reconfiguration of the JFFS partition, in which case JFFS must be reformatted & reinitialized so any custom configs/scripts/files are deleted. IOW, it's not the case that "JFFS may not work" after the update; it's just that after JFFS has been reformatted & reinitialized following the F/W update, any custom configuration files, scripts, etc. are deleted so it "appears" as if "it's not working." I believe this happened once in the last 5 years, so yes it can happen but it's very rare, and the Release Notes usually point this out to users.

My 2 cents.

As always your 2 cents are more like 2 dollars! ;)
Would you keep the warning in the readme as a potential risk? But make it clearer? Or would you just say the risk is so low, we remove it completely and let someone report issues if it happens again in 5 years down the line?

@Martinski4GitHub
Copy link
Collaborator

Or would you just say the risk is so low, we remove it completely and let someone report issues if it happens again in 5 years down the line?

This would be my preference since the scenario is very rare and unrelated to the script itself. It's just one of those post-update situations caused by changes in the new F/W, regardless of how the update was triggered (manual or automatic).

@ExtremeFiretop
Copy link
Owner Author

Or would you just say the risk is so low, we remove it completely and let someone report issues if it happens again in 5 years down the line?

This would be my preference since the scenario is very rare and unrelated to the script itself. It's just one of those post-update situations caused by changes in the new F/W, regardless of how the update was triggered (manual or automatic).

Done deal. Updated in Dev and pushed through right away to Main here: #30

This kinda stuff is just documentation updates, don't think it requires approval lol. Especially considering we just discussed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants