I think that OnUpdate would fire whenever your text field itself is updated. To have something update the clock periodically, try:
Lua Code:
EVENT_MANAGER:RegisterForUpdate("tkClockUpdate", 1000, function()
tkClock:SetText(GetTimeString())
end))
This will cause it to update text once per second (which is honestly all you really need to do for a clock)