-
Notifications
You must be signed in to change notification settings - Fork 465
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
Add software to "No Team" #19550
Comments
@noahtalerman Would you consider this workflow blocking? Customer-preston cannot use app management without this ability (as we discussed on the roadmap call) |
Hey @nonpunctual and @zayhanlon let's chat about this during this week's product office hours. I added an agenda item (internal): https://docs.google.com/document/d/1Znyp2a9qcM9JdYHrzLudvcPwEdhnCg7RiKi22s8yGWw/edit |
Hi @noahtalerman @noahtalerman Currently, in Fleet, the "no-team" feature is very usefull and present everywhere, with this feature we are able to manage :
Almost all of the Fleet's features are available in the "no team" tab, we build our entire system around it since it's available, and it works very well. The fact that the Software feature does not have this "no team" element is blocking us to use it, since we do not use teams at all. On a side note : |
Hey @valentinpezon-primo thanks for the feedback! The plan is to bring this story into the current design sprint and get started on drafting/wireframing. cc @nonpunctual |
@noahtalerman @nonpunctual Would adding software to "All teams" be distinct from this request? I understand "No Team" would be useful for customers that aren't using teams, but what about customers who use Teams that want to upload software once and have it apply to all organization wide to all their Teams? |
@noahtalerman @marko-lisica it is distinct for the reason you mention. customer-preston does not use Teams, so, assignment to "All Teams" would not help them. I think there is a tacit assumption that assignment to "No Team" means it would be automagically assigned to "every Host".
I am not entirely sure about this "No Team" feature from an Engineering persepctive, i.e., is it hard? Impossible? Is there a way to satisfy it that's easier by allowing App assignement via Label & that somehow a Label could be created that would be equivalent to the set of Hosts that matches "every Host"? If so, I think that would be equivalent to this FR. related: #20805 |
Jumping in @ddribeiro @nonpunctual What we want is to have the "Software" tab working the same way as the "Controls" tab. We would like to be able to use software like we use script basically Also, for the "Queries" and "Policies" tab, you have the "All teams" wording instead of "No team", but the behavior are the same since I can use queries and policies stored on "All team" on devices that do not have teams, so it looks like "All Teams" also works when device's team is Hope that helps ! |
@sharon-fdm This will need to come over to Endpoint ops for capacity let me know if you have any questions |
Hey @mostlikelee and @RachelElysia Figma is now updated w/ a dev note that clarifies that we want to add "No team" filtering to all tabs on the software page: I also realized we were missing wireframes for adding "No team" to the details pages. That's now in Figma: Also, I opened a PR w/ proposed API design here: #20489
cc @sharon-fdm |
UPDATE: Hey @mostlikelee, @RachelElysia, and @sharon-fdm, the GitOps changes are specified in this PR to the GitOps reference docs: #20502 We want to add support for I updated the issue description with this. |
Thanks @noahtalerman. |
@sharon-fdm, @mostlikelee, and @RachelElysia. Also heads up that this is a Fleet Premium only feature. I updated the issue description to call this out. |
@RachelElysia i'm realizing we'll need to expand scope to include "team 0" support for all the pages downstream from the software tab (all the detail pages): /software/titles/{id} |
@mostlikelee @sharon-fdm Good thing I got some wiggle points this sprint. This might add another 2-3 points of FE work. |
Thanks @RachelElysia! Great catch.
Let's do this 👍 I added an item to the agenda (internal): https://docs.google.com/document/d/1AduqZ9yuMQ8uvC5Z6GJFJtE0pbdqdX9zHIau_VCOqGI/edit For other folks looking at this comment, all UI changes go through design review. Any contributor at Fleet can bring items to design review. |
Update: Design decision during 7/24/24 design review to not include team reference in empty state e.g. "No hosts have this OS installed." will be the generic empty state for any team, no team, and free version where there is no team. |
Here's what this will look like: Furthermore, in this pass we decided to just make this change on OS, Software version, Software title, Vulnerabilities detail pages. @RachelElysia, when you get the chance, can you please help track an issue for other places so we can come back and don’t forget? I think the issue can be really generic, quick/dirty and link to this comment. Thanks :) |
Hey @mostlikelee, I added the expected behavior (above) we discussed during standup today to the issue description. cc @getvictor |
Hey @zayhanlon and @pintomi1989 heads up that this customer request was shipped in 4.55 🎉 |
"No Team" expands, |
Goal
Context
Changes
Product
software
top-level key indefault.yml
. PR to reference docs is here. Software specified indefault.yml
will be applied to "No team."software
is a required field. Show an easy to understand error message ifsoftware
isn't specified in thedefault.yml
or team YAML files. If the user is a Fleet Free user, don't error because they can't usesoftware
. More generally, all top level keys in thedefault.yml
and team YAML files are required.Engineering
QA
Risk assessment
Manual testing steps
Testing notes
Confirmation
The text was updated successfully, but these errors were encountered: