Go Premium for a chance to win a PS4. Enter to Win

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 616
  • Last Modified:

Help with VBScript

Hi there

I'm trying to use the following script to give me a report on Server Uptime and I can get it to work using a single instance but can't get it to read the MachineList.Txt file

Can anyone help.

Cheers

strComputer = "MachineList.Txt"
'strComputer = InputBox("Enter Machine Name")

Set objWMIService = GetObject("winmgmts:" & "{impersonationLevel=impersonate}!\\" & strComputer & "\root\cimv2")

Set objExcel = CreateObject("Excel.Application")
objExcel.Visible = True
objExcel.Workbooks.Add
intRow = 2

objExcel.Cells(1, 1).Value = "Machine Name"
objExcel.Cells(1, 2).Value = "IP Address"
objExcel.Cells(1, 3).Value = "MAC Address"
objExcel.Cells(1, 4).Value = "Days"
objExcel.Cells(1, 5).Value = "Hours"
objExcel.Cells(1, 6).Value = "Minutes"
objExcel.Cells(1, 7).Value = "Report Time Stamp"

Set colAdapters = objWMIService.ExecQuery ("SELECT * FROM Win32_NetworkAdapterConfiguration WHERE IPEnabled = True")
For Each objAdapter in colAdapters
objExcel.Cells(intRow, 1).Value = objAdapter.DNSHostName
If Not IsNull(objAdapter.IPAddress) Then
For i = 0 To UBound(objAdapter.IPAddress)
objExcel.Cells(intRow, 2).Value = objAdapter.IPAddress(i)
Next
End If
objExcel.Cells(intRow, 3).Value = objAdapter.MACAddress
Next

Set colObjects = objWMIService.ExecQuery ("SELECT * FROM Win32_PerfRawData_PerfOS_System")
For Each objWmiObject In colObjects
intPerfTimeStamp = objWmiObject.Timestamp_Object
intPerfTimeFreq = objWmiObject.Frequency_Object
intCounter = objWmiObject.SystemUpTime
Next
iUptimeInSec = (intPerfTimeStamp - intCounter)/intPerfTimeFreq
sUptime = ConvertTime(iUptimeInSec)

Function ConvertTime(seconds)
ConvDays = seconds \ (3600 * 24)
ConvHour = (seconds Mod (3600 * 24)) \ 3600
ConvMin = (seconds Mod 3600) \ 60

objExcel.Cells(intRow, 4).Value = ConvDays
objExcel.Cells(intRow, 5).Value = ConvHour
objExcel.Cells(intRow, 6).Value = ConvMin
End Function

objExcel.Cells(intRow, 7).Value = Now()
intRow = intRow + 1

objExcel.Range("A1:G1").Select
objExcel.Selection.Interior.ColorIndex = 19
objExcel.Selection.Font.ColorIndex = 11
objExcel.Selection.Font.Bold = True
objExcel.Cells.EntireColumn.AutoFit

Set objWMIService = Nothing
Set objExcel = Nothing
Set colAdapters = Nothing
Set colObjects = Nothing

Wscript.Echo "Done"
0
matrixnz
Asked:
matrixnz
  • 4
  • 3
1 Solution
 
sirbountyCommented:
This is untested, but it should work...

Dim objFSO, strComputer, ts, strData, strFile
strFile = "MachineList.Txt"

Set objFSO = CreateObject("Scripting.FileSystemObject")
Set ts = objFso.OpenTextFile(strFile)
strData=Split(ts.ReadAll,vbNewLine)

Set objExcel = CreateObject("Excel.Application")
objExcel.Visible = True
objExcel.Workbooks.Add
intRow = 2
objExcel.Cells(1, 1).Value = "Machine Name"
objExcel.Cells(1, 2).Value = "IP Address"
objExcel.Cells(1, 3).Value = "MAC Address"
objExcel.Cells(1, 4).Value = "Days"
objExcel.Cells(1, 5).Value = "Hours"
objExcel.Cells(1, 6).Value = "Minutes"
objExcel.Cells(1, 7).Value = "Report Time Stamp"

For each strComputer in strData  'Loop through each element in the strData array
  Set objWMIService = GetObject("winmgmts:" & "{impersonationLevel=impersonate}!\\" & strComputer & "\root\cimv2")
  Set colAdapters = objWMIService.ExecQuery ("SELECT * FROM Win32_NetworkAdapterConfiguration WHERE IPEnabled = True")
  For Each objAdapter in colAdapters
    objExcel.Cells(intRow, 1).Value = objAdapter.DNSHostName
    If Not IsNull(objAdapter.IPAddress) Then
      For i = 0 To UBound(objAdapter.IPAddress)
        objExcel.Cells(intRow, 2).Value = objAdapter.IPAddress(i)
      Next
    End If
    objExcel.Cells(intRow, 3).Value = objAdapter.MACAddress
  Next

  Set colObjects = objWMIService.ExecQuery ("SELECT * FROM Win32_PerfRawData_PerfOS_System")
  For Each objWmiObject In colObjects
    intPerfTimeStamp = objWmiObject.Timestamp_Object
    intPerfTimeFreq = objWmiObject.Frequency_Object
    intCounter = objWmiObject.SystemUpTime
  Next

  iUptimeInSec = (intPerfTimeStamp - intCounter)/intPerfTimeFreq
  sUptime = ConvertTime(iUptimeInSec)
Next

Function ConvertTime(seconds)
ConvDays = seconds \ (3600 * 24)
ConvHour = (seconds Mod (3600 * 24)) \ 3600
ConvMin = (seconds Mod 3600) \ 60

objExcel.Cells(intRow, 4).Value = ConvDays
objExcel.Cells(intRow, 5).Value = ConvHour
objExcel.Cells(intRow, 6).Value = ConvMin
End Function

objExcel.Cells(intRow, 7).Value = Now()
intRow = intRow + 1

objExcel.Range("A1:G1").Select
objExcel.Selection.Interior.ColorIndex = 19
objExcel.Selection.Font.ColorIndex = 11
objExcel.Selection.Font.Bold = True
objExcel.Cells.EntireColumn.AutoFit

Set objWMIService = Nothing
Set objExcel = Nothing
Set colAdapters = Nothing
Set colObjects = Nothing

Wscript.Echo "Done"
0
 
matrixnzAuthor Commented:
Hi Sirbounty

Thanks for your quick response

When I use your script, it opens Excel, and inserts the first server, and than replaces it with the second server and so on.  So I only get the first row.

I also receive an error
Script: C:\Uptime.vbs
Line: 21
Char: 3
Error: 0x80041021
Code: 80041021
Source: (null)

This seems to refer to the following line:
  Set objWMIService = GetObject("winmgmts:" & "{impersonationLevel=impersonate}!\\" & strComputer & "\root\cimv2")

Cheers
0
 
sirbountyCommented:
Okay, the counter's not incrementing properly then.
As for line 21 - it would appear that for whatever reason strComputer was empty...
You could troubleshoot it by placing a
  wscript.echo strComputer
just before that line (or msgbox in place of wscript.echo).

Aside from that - can you verify that your text file is seperated by hard carriage returns (or if they're delimeted some other way, let me know).

I'll look at the counter now...
0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
sirbountyCommented:
I don't completely follow the logic, but I believe this would remedy the counter problem...

Move these lines:

objExcel.Cells(intRow, 7).Value = Now()
intRow = intRow + 1

just after your

  sUptime = ConvertTime(iUptimeInSec)

(line 41)

Let me know if that fixes things...
0
 
matrixnzAuthor Commented:
Hi Sirbounty
Hard Carriage returns

Also thanks for the tip regarding "wscript.echo strComputer" the issue was ending I had a hard carriage return after the last server, so it read this line as well, removing the last HCR allowed the script to complete successfully.

Cheers
0
 
matrixnzAuthor Commented:
Perfect!!

Cheers for that
0
 
sirbountyCommented:
Happy to have helped - thanx!
~sirbounty
0

Featured Post

Concerto Cloud for Software Providers & ISVs

Can Concerto Cloud Services help you focus on evolving your application offerings, while delivering the best cloud experience to your customers? From DevOps to revenue models and customer support, the answer is yes!

Learn how Concerto can help you.

  • 4
  • 3
Tackle projects and never again get stuck behind a technical roadblock.
Join Now