vm icon indicating copy to clipboard operation
vm copied to clipboard

[talk] Nextcloud Talk High Performance Back-end repo will change it's structure

Open morph027 opened this issue 3 years ago • 5 comments
trafficstars

Steps To Reproduce

Run apt-get update.

Expected Result

apt-get update finishes without error.

Actual Result

apt-get update finishes with error.

N: This must be accepted explicitly before updates for this repository can be applied. See apt-secure(8) manpage for details.

Screenshots, Videos, or Pastebins

No response

Additional Context

Not active yet, will happen with the next upstream or bugfix release

Announcement: https://gitlab.com/packaging/nextcloud-spreed-signaling/-/issues/5

Build Version

n/a

Environment

By using the scripts

Environment Details

No response

morph027 avatar Sep 04 '22 11:09 morph027

Not sure how to handle this. Adding apt-get update --allow-releaseinfo-change is insecure and should be executed manually, which is not ideal.

Does the VM have any possibility to announce news or notifications to the users? (motd, nextcloud notifications, ...)

morph027 avatar Sep 04 '22 11:09 morph027

Question is why you decided to change your repo in the first place? Is it so important that thousands of users won't be able to run 'apt-update' without an error?

enoch85 avatar Sep 04 '22 18:09 enoch85

Reason was to allow the repo to be flagged properly for unattended-upgrades, which requires these fields. Also, it adheres to best practices to supply them. Unfortunately, i forgot to add them in the past, which leads to this hassle now.

If a user runs apt-get update manually, i'll ask for the change to be committed, so no problem from this side. I've also created an issue in my repo to notice anyone who stumbles upon this.

It's just annoying when being used in non-interactive scripts. That's why i created this issue here, so hopefully everybody who runs into this get's the info.

morph027 avatar Sep 04 '22 19:09 morph027

@morph027 We could automatically accept it if TURN is installeld by adding a -y flag, right?

enoch85 avatar Sep 09 '22 11:09 enoch85

Installation from scratch would work without any issues. Updating an existing installation will suffer the problem mentioned, but will work if the user runs apt-get update --allow-releaseinfo-change

Just using -yflag does not work. Also it does not affect the TURN server but the nextcloud-spreed-signalingpackage.

morph027 avatar Sep 09 '22 11:09 morph027

We can check if TURN is installed during update, and just force it with apt-get update --allow-releaseinfo-change. It would be the least hassle for the users.

D you think you could prep a PR?

enoch85 avatar Sep 22 '22 18:09 enoch85

@morph027 As far as I can tell everything is still working as expected. Do you know when this will go live?

enoch85 avatar Dec 25 '22 16:12 enoch85

There is no new build since the announcement. So changes to the repo are not live. Other repos are working fine. Do we want to schedule the change? Or wait until the next upstream release?

morph027 avatar Dec 25 '22 21:12 morph027

@morph027 Did you have time to check my PR?

enoch85 avatar Dec 27 '22 11:12 enoch85

Will do tonight, was travelling yesterday...

morph027 avatar Dec 27 '22 13:12 morph027

Please go ahead and push your fix @morph027

enoch85 avatar Dec 27 '22 18:12 enoch85