Bug 52121 - Remove univention-snmp*
Remove univention-snmp*
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: General
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0
Assigned To: Philipp Hahn
Florian Best
:
Depends on:
Blocks: 52980
  Show dependency treegraph
 
Reported: 2020-09-21 10:44 CEST by Philipp Hahn
Modified: 2021-05-25 16:03 CEST (History)
0 users

See Also:
What kind of report is it?: Development Internal
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2020-09-21 10:44:56 CEST
- `snmp` is the "Client configuration package", which ships an empty CFG file and only opens the firewall.
- `snmpd` the the "Server configuration package" and mostly adds UCRV `autostart/*` handling.
Neither of those two package is mentioned in the manual, which would match our definition of `not supported`.
This is about removing the "integration" package - the Debian packages `snmp*` will still be available (as `unmaintained`).








Comment 1 Philipp Hahn univentionstaff 2020-09-21 10:47:38 CEST
Bug #52121: Remove univention-snmp*
Comment 2 Florian Best univentionstaff 2020-10-19 17:20:21 CEST
Shouldn't we remove the UCR variables:

-ucr set security/packetfilter/package/univention-snmp/udp/161/all=ACCEPT \
-               security/packetfilter/package/univention-snmp/udp/161/all/en="SNMP"
-ucr set security/packetfilter/package/univention-snmpd/tcp/161/all="ACCEPT" \
-    security/packetfilter/package/univention-snmpd/tcp/161/all/en="SNMP" \
-    security/packetfilter/package/univention-snmpd/udp/161/all="ACCEPT" \
-    security/packetfilter/package/univention-snmpd/udp/161/all/en="SNMP" \
-    security/packetfilter/package/univention-snmpd/tcp/162/all="ACCEPT" \
-    security/packetfilter/package/univention-snmpd/tcp/162/all/en="SNMP" \
-    security/packetfilter/package/univention-snmpd/udp/162/all="ACCEPT" \
-    security/packetfilter/package/univention-snmpd/udp/162/all/en="SNMP"

I added the package to check.sh:update_check_old_packages in git:9172ea858aea
Comment 3 Florian Best univentionstaff 2021-02-18 10:22:15 CET
The packages were still part of the repositories:

Removing: /var/univention/buildsystem2/apt/ucs_5.0-0/source/univention-snmpd_11.0.0-1A~5.0.0.202006052240.tar.gz
Removing: /var/univention/buildsystem2/apt/ucs_5.0-0/all/univention-snmpd_11.0.0-1A~5.0.0.202006052240_all.deb
Removing: /var/univention/buildsystem2/apt/ucs_5.0-0/source/univention-snmpd_11.0.0-1A~5.0.0.202006052240.dsc

Removing: /var/univention/buildsystem2/apt/ucs_5.0-0/source/univention-snmp_10.0.0-1A~5.0.0.202006052238.dsc
Removing: /var/univention/buildsystem2/apt/ucs_5.0-0/source/univention-snmp_10.0.0-1A~5.0.0.202006052238.tar.gz
Removing: /var/univention/buildsystem2/apt/ucs_5.0-0/all/univention-snmp_10.0.0-1A~5.0.0.202006052238_all.deb
Comment 4 Florian Best univentionstaff 2021-02-18 10:47:49 CET
(In reply to Florian Best from comment #2)
> Shouldn't we remove the UCR variables:
> 
> -ucr set security/packetfilter/package/univention-snmp/udp/161/all=ACCEPT \
> -              
> security/packetfilter/package/univention-snmp/udp/161/all/en="SNMP"
> -ucr set security/packetfilter/package/univention-snmpd/tcp/161/all="ACCEPT"
> \
> -    security/packetfilter/package/univention-snmpd/tcp/161/all/en="SNMP" \
> -    security/packetfilter/package/univention-snmpd/udp/161/all="ACCEPT" \
> -    security/packetfilter/package/univention-snmpd/udp/161/all/en="SNMP" \
> -    security/packetfilter/package/univention-snmpd/tcp/162/all="ACCEPT" \
> -    security/packetfilter/package/univention-snmpd/tcp/162/all/en="SNMP" \
> -    security/packetfilter/package/univention-snmpd/udp/162/all="ACCEPT" \
> -    security/packetfilter/package/univention-snmpd/udp/162/all/en="SNMP"
> 
> I added the package to check.sh:update_check_old_packages in git:9172ea858aea

Should those also be added in doc/extended-docs/ucr-deprecated.yaml ?
Comment 5 Philipp Hahn univentionstaff 2021-02-18 12:20:44 CET
(In reply to Florian Best from comment #4)
> (In reply to Florian Best from comment #2)
> > Shouldn't we remove the UCR variables:
...
> Should those also be added in doc/extended-docs/ucr-deprecated.yaml ?

Good idea now that we have that place:

[5.0-0] 106e1e6196 Bug #39281 doc: Obsolete UCR variables
 doc/extended-docs/ucr-deprecated.schema.json |  35 +++++++
 doc/extended-docs/ucr-deprecated.yaml        | 244 ++++++++++++++++++++++++++++++++---------------
 2 files changed, 204 insertions(+), 75 deletions(-)
Comment 6 Florian Best univentionstaff 2021-03-23 15:12:10 CET
OK: package removal
OK: changelog entry
Comment 7 Florian Best univentionstaff 2021-05-25 16:03:07 CEST
UCS 5.0 has been released:
 https://docs.software-univention.de/release-notes-5.0-0-en.html
 https://docs.software-univention.de/release-notes-5.0-0-de.html

If this error occurs again, please use "Clone This Bug".