MWA - Add Manifest Column

[natewalck]

Issue by natewalck
Friday Sep 19, 2014 at 07:06 GMT
Originally opened as munki/munki#207

---

From [email protected] on November 01, 2012 15:03:00

This is a request to add a column to the MunkiWebAdmin "Reports" and/or "Inventory Clients" view. It would be tremendously helpful in our environment to have a quick glance view of which clients are using which manifest.

Original issue: http://code.google.com/p/munki/issues/detail?id=207

[natewalck]

Comment by natewalck
Friday Sep 19, 2014 at 07:06 GMT

---

From [email protected] on March 24, 2013 11:28:32

Could you be a little more specific? The manifest that the client is using is listed at the top-right of the reports page. Were you looking for something more in depth, like a list of inherited manifests?

I also realize that quite a few changes have been made since you filed this issue, so if it no longer applies just let me know.

Owner: [email protected]

[natewalck]

Comment by natewalck
Friday Sep 19, 2014 at 07:06 GMT

---

From [email protected] on March 26, 2013 15:35:35

Ideally, I would be able to pull a report of all Macs that are using a particular manifest. For example, as Macs are deployed into our environment, they get a generic organization-wide client identifier. If everyone in the field is doing their job, these client identifiers will be changed (by MCX) to reflect the particular site that they are installed in. Sometimes they get missed and having a way to see which clients are still using the generic manifest would be a helpful way for me to catch and correct those errors.

Unfortunately, MunkiWebAdmin got extremely slow after our client base passed about 2,000 Macs. I may need to scrap it and try re-installing with the latest code updates.

[natewalck]

Comment by natewalck
Friday Sep 19, 2014 at 07:06 GMT

---

From [email protected] on April 10, 2013 12:54:53

I added a 'Manifest' column to my 'Reports' view for this exact reason. Our old image was not correctly updating the ClientIdentifier, so we have a bunch of machines checking the wrong manifests. The correct way to do this would be to have the manifest name/clientID stored in its own column in the database, but since I was trying to do this quickly as a temporary measure to fix all the broken ClientIDs, I am pulling the name from the plist data instead. This significantly slows down the 'Reports' page though as it has to process the entire ManagedInstalls.plist for each host. If you are still interested, I can post my code changes, but there is definitely a better way to do it.

[natewalck]

Comment by natewalck
Friday Sep 19, 2014 at 07:06 GMT

---

From [email protected] on May 24, 2013 08:20:32

I'd like the code you used to add a manifest column, if you don't mind. We don't have many computers and it would certainly be useful, at least as a patch.

[natewalck]

Comment by natewalck
Friday Sep 19, 2014 at 07:06 GMT

---

From [email protected] on September 26, 2013 05:13:49

It's not part of MWA, but Dan McClellan posted a script which does seem to be inline with the details of this issue:

Discussion: https://groups.google.com/forum/#!topic/munki-web-admin/5CxyiHx03Jw Script: http://pastebin.com/xDQ0RmHd

[natewalck]

Comment by natewalck
Friday Sep 19, 2014 at 07:06 GMT

---

From [email protected] on October 18, 2013 17:07:07

Gah, so sorry for not replying back in May. I've been logging in with different accounts and didn't see any notifications about replies. I've since removed the column from our MWA to speed the reports page back up. I'm going to pull a clean copy of MWA and post a diff of the view and the template with the manifest column added in case anyone wants the slow version (if you have a small number of machines.) If I have some time next week I can work on the larger updates to the model and scripts that actually add a manifest column to the database table.

[natewalck]

Comment by natewalck
Friday Sep 19, 2014 at 07:06 GMT

---

From [email protected] on October 18, 2013 17:47:41

diff --git a/reports/templates/reports/clienttable.html b/reports/templates/reports/clienttable.html
index 6c1e6b8..5515585 100644
--- a/reports/templates/reports/clienttable.html
+++ b/reports/templates/reports/clienttable.html
@@ -9,6 +9,7 @@

Machine
User

•        <th>Manifest  </th>
       <th>IP        </th>
       <th>Latest Run</th>
     </tr>
  

  @@ -27,6 +28,7 @@
  
  
  {{ report.console_user }}

•          <td>{{ report.manifest_name }}</td>
         <td class='ip'>{{ report.machine.remote_ip }}</td>
         <td>
             {{ report.timestamp|date:"Y-m-d H:i" }}
  

  diff --git a/reports/views.py b/reports/views.py
  index b6a7e7b..9783b83 100644
  --- a/reports/views.py
  +++ b/reports/views.py
  @@ -171,6 +171,11 @@ def index(request):

  if model is not None:
  reports = reports.filter(machine__machine_model__exact=model)
  +

• Used to add a 'manifest name' column to the report index page

• This will significantly increase page load time for the index

• for report in reports:

•    report.manifest_name = report.get_report().get('ManifestName', '')
  

  return render_to_response('reports/index.html',
  {'reports': reports,