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

"Automatic upgrades" web GUI option in pre-release versions is misleading #4216

Closed
ProactiveServices opened this issue Jun 15, 2017 · 9 comments
Labels
bug A problem with current functionality, as opposed to missing functionality (enhancement) frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion good first issue Good starting points for new contributors
Milestone

Comments

@ProactiveServices
Copy link
Member

Usually I stick to the release channel and upgrade manually. I wanted to take advantage of a recent bug fix in a pre-release version.

I set the Options to permit release candidates and after upgrading I wanted to set automatic upgrades back to "disabled". The web UI lets me do this, asked for a Syncthing restart, then the automatic upgrades are forced back on, as per pre-release behaviour. I assumed something had caused the config save to fail and tried to diagnose the "problem" :-)

It would be helpful if the Options->Automatic upgrades option showed "Stable and release candidates" and locked the menu to make it clear that the option cannot be changed, perhaps with a tooltip or a brief explanation below.

@calmh
Copy link
Member

calmh commented Jun 15, 2017

Changing to "stable releases only" is possible and legitimate so the control itself shouldn't be locked down. But "no upgrades" isn't a valid choice from a pre-release, so that option shouldn't be selectable, indeed.

@calmh calmh added bug A problem with current functionality, as opposed to missing functionality (enhancement) good first issue Good starting points for new contributors labels Jun 15, 2017
@calmh calmh added this to the Unplanned (Contributions Welcome) milestone Jul 10, 2017
@danielmreck
Copy link

While in the Options screen I accidentally changed the Automatic Upgrades option to allow release candidates. Syncthing almost immediately upgraded and now I cannot switch back to stable releases.

@calmh as of v0.14.34-rc.1 the "Automatic Upgrades" UI element is completely missing from the Options screen, as seen in this screenshot. Where can I change the setting back to only automatically upgrade to stable releases? I cannot find this in config.xml or anywhere else, and I really don't want to go through the pain of completely reinstalling and reconfiguring this machine across the entire mesh.

Allowing users to go down a one-way street into unstable candidate releases is both confusing and frustrating.

Thanks!

capture

@ProactiveServices
Copy link
Member Author

To get back to a stable release:
Stop Syncthing
Edit Syncthing's config.xml , upgradeToPreReleases entry to false
Drop the latest release binary over the top of the existing pre-release
Start Syncthing

@danielmreck
Copy link

Thanks, @ProactiveServices. I will do that when I'm back in front of this machine. I recommend that the UI be corrected. (Or, if there's some reason we want to make it extra difficult to go back to stable from release, these instructions should be provided in the release release UI.)

@calmh
Copy link
Member

calmh commented Aug 1, 2017 via email

@danielmreck
Copy link

danielmreck commented Aug 2, 2017

@ProactiveServices I was able to get back to the stable release tree with the instructions above. Thank you!

@calmh Thank you for this additional detail about what causes the upgrade box to disappear. In my case, the only variable that changed was an accidental upgrade to the release track. This shouldn't have changed the environment or build option, right?

Access to the upgrade server should not have been a problem because all the machines on the stable track were still showing the Automatic Upgrades dropdown box -- only the two systems that were on the candidate release track were missing the UI box, which implies a bug unless I'm missing another nuance of intended behavior. These machines were on the same local network, so they all had the same access conditions.

@calmh
Copy link
Member

calmh commented Aug 2, 2017 via email

@danielmreck
Copy link

All the machines affected by the accidental move to the candidate release track are 64-bit Windows. There is one 32-bit Windows machine and one BSD machine in the mesh, but they were unaffected.

From a UI perspective, it is better that all options are visible, lest we confuse users when the documentation refers them to invisible elements. Then we can disable specific UI elements with an explanation as appropriate.

(And for goodness of all, let's not bury options under 82 levels of swipe gestures, unlabeled icons, and thin low-contrast/non-accessible links -- looking at you, Google and Microsoft!)

@calmh
Copy link
Member

calmh commented Jan 15, 2018

The latter problems mentioned here were #4654 and are now fixed. The original issue remains.

@calmh calmh removed this from the Unplanned (Contributions Welcome) milestone Feb 11, 2018
calmh added a commit to calmh/syncthing that referenced this issue Jan 14, 2019
…fixes syncthing#4216)

This adds booleans to the /system/version response to advice the GUI
whether the running version is a candidate release or not. (We could
parse it from the version string, but why duplicate the logic.)

Additionally the settings dialog locks down the upgrade and usage
reporting options on candidate releases. This matches the current
behavior, it just makes it obvious what actually *can* be chosen.
@calmh calmh closed this as completed in f4bf18c Jan 15, 2019
@calmh calmh modified the milestones: v1.0.2, v1.0.1 Jan 15, 2019
@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label Jan 16, 2020
@syncthing syncthing locked and limited conversation to collaborators Jan 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug A problem with current functionality, as opposed to missing functionality (enhancement) frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion good first issue Good starting points for new contributors
Projects
None yet
Development

No branches or pull requests

4 participants