server icon indicating copy to clipboard operation
server copied to clipboard

MDEV-36531 Enable Feedback Plugin for DEB Packages

Open vuvova opened this issue 8 months ago • 2 comments

Create a user facing question during installation step to allow users to choose whether to enable the feedback plugin or not.

vuvova avatar Apr 27 '25 22:04 vuvova

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Apr 27 '25 22:04 CLAassistant

Note we have had https://salsa.debian.org/mariadb-team/mariadb-server/-/merge_requests/16 open for a long time. It was waiting for @cvicentiu to confirm what is the correct API URL as there was some confusion which of the alternatives is correct, and perhaps some confirmation that the system actually works end-to-end to be deployed now. Seems there are some stats now at https://mariadb.org/feedback-plugin/ so it is up and running again?

ottok avatar Apr 27 '25 23:04 ottok

Note we have had https://salsa.debian.org/mariadb-team/mariadb-server/-/merge_requests/16 open for a long time. It was waiting for @cvicentiu to confirm what is the correct API URL as there was some confusion which of the alternatives is correct, and perhaps some confirmation that the system actually works end-to-end to be deployed now. Seems there are some stats now at https://mariadb.org/feedback-plugin/ so it is up and running again?

If you @vuvova or @cvicentiu have time to continue on this, it would be good to have feedback plugin enabled upstream so I can see that it works, what the correct API URL is etc and then follow-up by finalizing https://salsa.debian.org/mariadb-team/mariadb-server/-/merge_requests/16

ottok avatar Jun 12 '25 07:06 ottok

I plan to have it in the next release, that is, by the end of July.

@ottok , please say if you need it earlier, for example, to prepare or to try it out or to update a merge request in salsa or something

vuvova avatar Jun 12 '25 10:06 vuvova

I plan to have it in the next release, that is, by the end of July.

@ottok , please say if you need it earlier, for example, to prepare or to try it out or to update a merge request in salsa or something

No urgency on this one.

ottok avatar Jun 19 '25 19:06 ottok

thanks

vuvova avatar Jun 20 '25 15:06 vuvova

Seems this was merged in https://github.com/MariaDB/server/commit/5d1e485e0a1aa4b98f50439c2dfc70ef3a57651a but using a different version than in this Pull Request.

ottok avatar Aug 07 '25 22:08 ottok