kloczek / zabbix-templates

Zabbix templates
111 stars 65 forks source link

OS Windows updated #1

Closed Chewbakka-Wakka closed 6 years ago

Chewbakka-Wakka commented 6 years ago

First test, more items need adding and descriptions also need to be done still.

Chewbakka-Wakka commented 6 years ago

Also updated README, to correct few spelling typos, and slight changes to wording.

kloczek commented 6 years ago

Merged but I need to review those changes one more time on my local copy of the repo.

About naming convention. Adding 3-4 characters should be added definitely to the trigger names. In case of the items this seems is redundant as in host->items view each item always always listed with template name and in Latest data view is listed under exact Application so now SYS::Uptime key is listed under SYS Application which seems duplicates this SYS. In items list view it will be no problem with for example multiple items like Uptime when they will be listed as for example: OS Linux: Uptime Service MySQL: Uptime

Only thing which is missing is have similar view in Latest data detailed view. However I just located place in php code which needs to be modified to have template or LLD name in item name in Latest data view like it is in Items list view, I'm going to submit this change to zabbix main source tree.

Another thing is submitting changes. Please try to push changes in smaller chunks with even more pull requests. With less changes introduced in each step it will be easier to make decision about committing the change or not.

Chewbakka-Wakka commented 6 years ago

Ok, think I understand most of that. Should I continue to name items the way I have been or do another way?

This is what I see in my Zabbix when I search host -> items

OS Windows http://192.168.77.206/items.php?hostid=10265&filter_set=1: SYS::Uptime http://192.168.77.206/items.php?form=update&hostid=10261&itemid=28714

Actually it should be 2-4 characters, to cover HW::

I do see what you mean about Latest data detailed view, so yes ok.

Ok, I will do this with making changes/pushing from now on.

For OS Windows first I am adding items, and then after create some trigger conditions when I learn and observe more about what is happening on my system.

On Windows template, these counters are possible to add. Should these be under MEM or SYS? I'm thinking MEM as it is memory related primarily. (or some of these)

\Memory\System Code Total Bytes \Memory\System Code Resident Bytes \Memory\System Driver Total Bytes \Memory\System Driver Resident Bytes \Memory\System Cache Resident Bytes

Also on OS Windows this item is always as value 1. Another on my todo list

On 14 December 2017 at 22:35, kloczek notifications@github.com wrote:

Merged but I need to review those changes one more time on my local copy of the repo.

About naming convention. Adding 3-4 characters should be added definitely to the trigger names. In case of the items this seems is redundant as in host->items view each item always always listed with template name and in Latest data view is listed under exact Application so now SYS::Uptime key is listed under SYS Application which seems duplicates this SYS. In items list view it will be no problem with for example multiple items like Uptime when they will be listed as for example: OS Linux: Uptime Service MySQL: Uptime

Only thing which is missing is have similar view in Latest data detailed view. However I just located place in php code which needs to be modified to have template or LLD name in item name in Latest data view like it is in Items list view, I'm going to submit this change to zabbix main source tree.

Another thing is submitting changes. Please try to push changes in smaller chunks with even more pull requests. With less changes introduced in each step it will be easier to make decision about committing the change or not.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/kloczek/zabbix-templates/pull/1#issuecomment-351857140, or mute the thread https://github.com/notifications/unsubscribe-auth/APv6zvE2KgIDK4p8PGFKmRYCY9sJF6NBks5tAaKmgaJpZM4QvujA .