datalust / seq-tickets

Issues, design discussions and feature roadmap for the Seq log server
https://datalust.co/seq
98 stars 5 forks source link

Seq upgrade installer blew away service configuration #2245

Closed c0shea closed 3 months ago

c0shea commented 3 months ago

Describe the bug I ran the installer to upgrade to the latest version 2024.3.12021. The install itself worked fine. I left the box checked to run the service configuration tool and closed the install wizard. A new UI opened for service administration and it didn't remember any of my previous settings. It defaulted to the ProgramData directory to store data, even though my current install kept the data on the D drive. It also reset the URL to localhost:5341 and I had to re-enter our HTTPS URL.

To Reproduce

  1. Run the installer to upgrade from 2024.3.11914 to 2024.3.12021.

Expected behavior If the installer was going to uninstall and reinstall the Seq Windows Service, it should have kept my existing configuration to respect the data directory and URL.

Screenshots This is the new dialog that didn't respect my existing config. image

Environment (please complete the following information):

nblumhardt commented 3 months ago

Thanks for the report @c0shea; we've seen one other instance of this, I think I've figured out what's going on, we'll get this sorted out :+1:

c0shea commented 3 months ago

@nblumhardt Thanks! 🙏🏻

nblumhardt commented 3 months ago

We'll release a patch fixing this in the next week or so. Unfortunately, the bug is in the installer's remove logic, so going from any of the existing versions to the fixed version will still be fragile and require the storage location/listen URI to be re-entered.