suppress both the index and the index column of the data_frame in output #51

tkishel · 2022-06-02T17:09:09Z

Description

Suppress both the index and the index column of the data_frame in output.

Motivation and Context

The indexes are not part of the data returned by the APIs, rather they are an artifact of pandas.

How Has This Been Tested?

Before:

# pc --output text settings
      index    disconnectPeriodDays    listeningPort  admissionControlEnabled    admissionControlWebhookSuffix    hostCustomComplianceEnabled    automaticUpgrade
--  -------  ----------------------  ---------------  -------------------------  -------------------------------  -----------------------------  ------------------
 0        0                       1             9998  True                       8is8ilgnnl579305k25ic4mfi...     True                           False

After:

# pc --output text settings
  disconnectPeriodDays    listeningPort  admissionControlEnabled    admissionControlWebhookSuffix    hostCustomComplianceEnabled    automaticUpgrade
----------------------  ---------------  -------------------------  -------------------------------  -----------------------------  ------------------
                     1             9998  True                       8is8ilgnnl579305k25ic4mfi...     True                           False

And for other formats:

# pc --output csv settings
disconnectPeriodDays,listeningPort,admissionControlEnabled,admissionControlWebhookSuffix,hostCustomComplianceEnabled,automaticUpgrade
1,9998,True,8is8ilgnnl579305k25ic4mfirfz,True,False


# pc --output json settings
[{"disconnectPeriodDays":"1","listeningPort":"9998","admissionControlEnabled":"True","admissionControlWebhookSuffix":"8is8ilgnnl579305k25ic4mfirfz","hostCustomComplianceEnabled":"True","automaticUpgrade":"False"}]


# pc --output json settings | jq
[
  {
    "disconnectPeriodDays": "1",
    "listeningPort": "9998",
    "admissionControlEnabled": "True",
    "admissionControlWebhookSuffix": "8is8ilgnnl579305k25ic4mfirfz",
    "hostCustomComplianceEnabled": "True",
    "automaticUpgrade": "False"
  }
]


# pc --output html settings
<html>
<head>
    <link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap@4.0.0/dist/css/bootstrap.min.css" integrity="sha384-Gn5384xqQ1aoWXA+058RXPxPg6fy4IWvTNh0E263XmFcJlSAwiGgFAW/dAiS6JXm" crossorigin="anonymous">
</head>
                
<table border="1" class="dataframe table table-sm table-striped text-left">
  <thead>
    <tr style="text-align: left;">
      <th>disconnectPeriodDays</th>
      <th>listeningPort</th>
      <th>admissionControlEnabled</th>
      <th>admissionControlWebhookSuffix</th>
      <th>hostCustomComplianceEnabled</th>
      <th>automaticUpgrade</th>
    </tr>
  </thead>
  <tbody>
    <tr>
      <td>1</td>
      <td>9998</td>
      <td>True</td>
      <td>8is8ilgnnl579305k25ic4mfirfz</td>
      <td>True</td>
      <td>False</td>
    </tr>
  </tbody>
</table>
<script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.2/js/bootstrap.min.js"></script>
</html>

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

I have updated the documentation accordingly.
I have read the CONTRIBUTING document.
I have added tests to cover my changes if appropriate.
All new and existing tests passed.

…utputting

welcome-to-palo-alto-networks · 2022-06-02T17:09:11Z

🎉 Thanks for opening this pull request! We really appreciate contributors like you! 🙌

steven-deboer

Thank you Tom!!!

welcome-to-palo-alto-networks · 2022-06-02T19:40:30Z

🎉 Congrats on getting your first pull request merged! We here at Palo Alto Networks are so grateful! ❤️

suppress both the index and the index column of the data_frame when o…

c926103

…utputting

tkishel requested a review from steven-deboer June 2, 2022 17:13

steven-deboer approved these changes Jun 2, 2022

View reviewed changes

steven-deboer merged commit a83b2fc into PaloAltoNetworks:main Jun 2, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

suppress both the index and the index column of the data_frame in output #51

suppress both the index and the index column of the data_frame in output #51

tkishel commented Jun 2, 2022 •

edited

Loading

welcome-to-palo-alto-networks bot commented Jun 2, 2022

steven-deboer left a comment

welcome-to-palo-alto-networks bot commented Jun 2, 2022

suppress both the index and the index column of the data_frame in output #51

suppress both the index and the index column of the data_frame in output #51

Conversation

tkishel commented Jun 2, 2022 • edited Loading

Description

Motivation and Context

How Has This Been Tested?

Types of changes

Checklist

welcome-to-palo-alto-networks bot commented Jun 2, 2022

steven-deboer left a comment

Choose a reason for hiding this comment

welcome-to-palo-alto-networks bot commented Jun 2, 2022

tkishel commented Jun 2, 2022 •

edited

Loading