search

Cacti / plugin_syslog

Syslog Plugin for Cacti
GNU General Public License v2.0
21 stars 16 forks source link

Update setup.php #162

Closed bmfmancini closed 2 years ago

bmfmancini commented 2 years ago

Change Messege varchar from 1024 to 2048

+-------------+---------------------+------+-----+---------------------+----------------+
| Field       | Type                | Null | Key | Default             | Extra          |
+-------------+---------------------+------+-----+---------------------+----------------+
| facility_id | int(10) unsigned    | YES  | MUL | NULL                |                |
| priority_id | int(10) unsigned    | YES  | MUL | NULL                |                |
| program_id  | int(10) unsigned    | YES  | MUL | NULL                |                |
| host_id     | int(10) unsigned    | YES  | MUL | NULL                |                |
| logtime     | timestamp           | NO   | MUL | 0000-00-00 00:00:00 |                |
| message     | varchar(2048)       | NO   |     |                     |                |
| seq         | bigint(20) unsigned | NO   | PRI | NULL                | auto_increment |
+-------------+---------------------+------+-----+---------------------+----------------+

Install Tested Uninstall tested

netniV commented 2 years ago

That pull request got pulled first... double pull for the loss?

netniV commented 2 years ago

To comments for the next request, don't forget to include upgrade settings and a changelog entry.

bmfmancini commented 2 years ago

Damn you beat me thats why I posted the second PR for the change log

I thought I had got the upgrade settings let me double check

netniV commented 2 years ago

If you update your branch against the PR, it will update the PR. You don't need to close and re-create it. However, you do in your case because it was against master not develop.