I don't know what you mean by not getting the desired output. More details would be helpful.
But six things about this applet. First, are these switches using AAA command authorization? If so, then you also need to configure:
event manager session cli username USER
Where USER is a username authorized to run all of the commands.
Second, a wait within a policy can be bad. It can cause the policy to die because it hits its maxrun. You may want to increase maxrun to something like 30 seconds to account for the added delay (add "maxrun 30" to the end of the "event none").
Third, I never recommend using a hostname for an SMTP server since that adds a bit more delay and has been known to lock up policies in older versions of IOS.
Fourth, you need an action "end" after your mail action. You start an if block, you need to end it.
Fifth, you shouldn't mix labels like you're doing. Continue the theme, and use a number for your mail action. This will make adding the "end" action easier.
And sixth, you should also use a fully qualified email address for "from".
