[ZBXNEXT-8570] Remove TimescaleDB v1.x support Created: 2023 Jul 12 Updated: 2024 Jun 10 Resolved: 2023 Sep 26 |
|
Status: | Closed |
Project: | ZABBIX FEATURE REQUESTS |
Component/s: | Frontend (F), Server (S) |
Affects Version/s: | None |
Fix Version/s: | 7.0.0alpha5, 7.0 (plan) |
Type: | Change Request | Priority: | Trivial |
Reporter: | Alex Kalimulin | Assignee: | dimir |
Resolution: | Fixed | Votes: | 1 |
Labels: | None | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Attachments: |
![]() ![]() |
||||||||||||||||||||
Issue Links: |
|
||||||||||||||||||||
Team: | |||||||||||||||||||||
Sprint: | Sprint 102 (Jul 2023), Sprint 103 (Aug 2023), Sprint 104 (Sep 2023) | ||||||||||||||||||||
Story Points: | 2 |
Description |
Remove support for TSDB 1.x from Zabbix server and frontend. This means bumping ZBX_TIMESCALE_MIN_VERSION to 20000 (or 20001) and removing all the code that is surrounded by ZBX_DB_TSDB_V1 from the server. The frontend must also remove any TSDB v1.x conditional code (e.g. isCompressed() in PostgresqlDbBackend). The docs must remove mentions of TSDB v1.x (if any, I didn’t find ones). |
Comments |
Comment by dimir [ 2023 Aug 31 ] | ||||||||||||||||||||
As I understand we drop support for versions <= 1.7.5 (Feb 16, 2021) | ||||||||||||||||||||
Comment by dimir [ 2023 Sep 01 ] | ||||||||||||||||||||
Talking about TimescaleDB versions that Zabbix supports, in 7.0 we will increase the absolute minimum supported version to 2.0.1 and recommended minimum to 2.10.0:
| ||||||||||||||||||||
Comment by dimir [ 2023 Sep 06 ] | ||||||||||||||||||||
Fixed in development branch for pre-7.0. Before this fix there were 2 separate functions:
The latter function was reporting error when TSDB version was too old, while details were already collected (by the previous function). The first function was using different SQL queries when talking to v1 and v2 TSDB (for getting license and compressed chunk flags). Since we removed support for v1 we must perform the version check first. This is why it was decided to combine those 2 functions and go one by one, getting a detail, reporting error if it isn't satisfactory. One issue, I'm not sure if the function zbx_tsdb_get_license() that was moved from zbxdb to zbxdbhigh should be using zbx_db_*() or zbx_db_*_basic() functions. | ||||||||||||||||||||
Comment by dimir [ 2023 Sep 25 ] | ||||||||||||||||||||
OverviewThe TimescaleDB version requirements from Zabbix perspective where changed in this commit.
Fixed in
|