-
Notifications
You must be signed in to change notification settings - Fork 450
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MSP dashboard: API endpoint to get all software available for install for all teams #23824
Comments
Specification:
Example API response[
{
"software_title_name": "zoom.us.app",
"installer_name": "Zoom.pkg",
"software_title_id": 849,
"installer_version": "6.2.3.40682",
"teams": [
{
"id": 3,
"team_name": "macOS Workstations"
},
{
"id": 9,
"team_name": "Team 2"
}
...
]
},
{
...
}
] |
Hey @eashaw I think let's go w/ What do you think? cc @rachaelshaw |
@noahtalerman Makes sense! I updated the example API response. |
@eashaw @noahtalerman I am trying to use https://fleetdm.com/guides/building-an-effective-dashboard-with-fleet-rest-api-flask-and-plotly to set up an MSP Dashboard but are there different instructions I should be following? Assistance would be appreciated. Thanks! |
@jmwatts Sorry about this, I met with Mike to review this PR last night and I made some changes that affected the QA steps in this issue. I updated the steps to reflect the new behavior. The MSP dashboard is a separate node app in the ee/bulk-operations-dashboard/ folder. To QA this change, you should be able to follow the steps in this section of the app's README to start the dashboard. I can also help you get started if you want to schedule some time with me. |
QA Notes: Verified new /api/v1/list-software endpoint is available and returns results based on the specs above. @noahtalerman @eashaw One thing to note:
|
@jmwatts thanks! I don't think that's accetable. @pintomi1989 I don't think this improvement is ready for @lukeheath, for when you're back, can you please work with Scott and Eric on improvements that could speed this up? |
@eashaw or @noahtalerman can this be moved back to "Waiting" or "In Progress" as there isn't further QA action to be taken at this time? |
@jmwatts Issues can be moved back to "In Progress" if they don't clear QA. I've updated. @sgress454 If you are online before next sprint kick off could you chat with @eashaw to brainstorm how we could improve response time? If it's straight-forward you can go ahead and implement, but if it's a heavier lift we should discuss the architectural changes. Thanks! |
@jmwatts I updated the PR to have the new endpoint send multiple requests to the Fleet instance at once to improve the response time. |
Goal
Key result
Deliver customer promises and prioritized requests
Original requests
Context
Changes
Product
QA
Manual testing steps
/api/v1/list-software
with no authorization header./api/v1/list-software
with the authorization header set toBearer ${apiToken}
platform
query parameter to filter the results and confirm thatGET /api/v1/list-software?platform=darwin
returns a list of macOS software on the Fleet instanceteams
array in the response contains the name and ID of each team the software is assigned to.GET /api/v1/list-software?platofrm=windows
Returns a list of Windows software on the Fleet instanceteams
array in the response contains the name and ID of each team the software is assigned to.GET /api/v1/list-software?platofrm=linux
Returns a list of Linux software on the Fleet instanceteams
array in the response contains the name and ID of each team the software is assigned to.GET /api/v1/list-software?platofrm=chrome
Returns a 400 bad request response.Testing notes
Confirmation
The text was updated successfully, but these errors were encountered: