-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
OS-level notifications for all local instances when upgrading a single instance to a new version #1054
Comments
This is really strange. I just launched UTM (a macOS GUI for QEMU), and I got the same thing — notifications for all of my EdgeDB instances. 🤔 Just closed it and re-launched it, and I didn't get the notifications. Seems it was only on initial launch of UTM. |
I don't see any code that touches other services on upgrade. So probably this is some MacOS bug that shows notifications of all services when one is changed, as confirmed by UTM launch. Can you also try upgrading an instance after UTM is installed and look whether UTM is also shown in notifications? (i.e. confirm that this is not EdgeDB-specific issue). |
This should suppress unintended notifications by MacOS. Fixes #1054
Looks like #1058 should fix it. Would you mind testing it git version? We do rewrite |
I don't mind testing it. How would I do that? I'm not sure how to run the CLI out of a particular branch. |
If you have Rust installed run in the edgedb-cli repository:
You may also run |
This should suppress unintended notifications by MacOS. Fixes #1054
This should suppress unintended notifications by MacOS. Fixes #1054
This looks great, @tailhook! When I upgraded the instance, I got only the single notification. 🎉 |
It appears this didn't fix the issue after all, @tailhook (or else there has been a regression). This morning, I upgraded the CLI and created a new instance. Got a notification for each instance on my machine again. |
Did you upgrade to the nightly? I think we didn't make a stable release. |
I'm on 3.2.0-dev.1003+c9675ed |
That might be because this fix only applies to newly created instances, i.e. it does not retroactively apply to all existing service units. IIRC we regenerate unit files on a |
OK, so I would stop/start all of my instances to regenerate all unit files, upgrade the CLI again, and create a new instance? Am I understanding correctly, @elprans? |
It seems this is also happening for instances a user has already destroyed according to a Discord user. See the thread attached to this message: https://discord.com/channels/841451783728529451/849377751370432573/1119215483183452180 |
Hm, non-removing service files is weird indeed. Can you try destroying the instance with logging enabled?
Otherwise, I'll try reproduce it on monday. |
I can't reproduce it myself. Service files are removed as expected on my MacOS 11.2. |
For those seeing the issue, does running |
When upgrading a local instance to a new server version, I get one OS-level notification that a background item has been added for each instance running on my system. I would expect to get only one for the newly upgraded instance.
Update: I've just noticed that I also get these notifications after restarting my system. I do not get any notifications for other background services, but I get one per EdgeDB instance upon restart. Added this as an alternative reproduction.
Steps to Reproduce:
or
The text was updated successfully, but these errors were encountered: