Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Option to set default view for argocd apps #12019

Closed
codestalkerr opened this issue Jan 18, 2023 · 5 comments
Closed

Option to set default view for argocd apps #12019

codestalkerr opened this issue Jan 18, 2023 · 5 comments
Labels
enhancement New feature or request

Comments

@codestalkerr
Copy link

Summary

It would be really helpful give option to set config for default UI element.
For example today the default view is "APPLICATION DETAILS TREE" but there is no way to change that default view to something else with help of configmap or some settings in yaml. (If there is already then please let me know)

Motivation

Started looking into this because many developers complained that while loading the app view the UI was not responding well and hanging a bit (we have around 1200 different application there). We did try to improve the performance by updating different settings but didnt see much difference. And noticed a nice work around is to change the default view to "APPLICATION DETAILS NETWORK" which loads faster.

Proposal

Maybe an option in externalArgs or configmap?
@codestalkerr codestalkerr added the enhancement New feature or request label Jan 18, 2023
@alexef
Copy link
Member

alexef commented Jan 18, 2023 •
edited
Loading

Wow, I was thinking of the same thing right now, had similar issues in the past days.

This makes even more sense as view extensions just landed.

What I would like to be able to configure is:

  • default view (built-in or extension)
  • default group by option in the pods view

@alexef
Copy link
Member

alexef commented Jan 23, 2023 •
edited
Loading

I am also thinking that a different view might make sense on an application basis. So in addition to supporting this in argocd-cm maybe we can support it also as an argoproj.io/Application metadata annotation?

@alexmt / @rbreeze as you're working on view extensions, what do you think? see also initial PR: #12080 (we really need it internally)

alexef added a commit to alexef/argo-cd that referenced this issue Jan 23, 2023
@alexef
feat(UI): allow applications to declare default view preferences (arg…
4a00bb1 
…oproj#12019)

Signed-off-by: Alex Eftimie <alex.eftimie@getyourguide.com>
@alexef alexef mentioned this issue Jan 23, 2023
Merged
10 tasks
@rbreeze
Copy link
Member

rbreeze commented Jan 23, 2023

@alexef I agree, this makes sense on a per app basis. Taking a look at the PR

rbreeze added a commit that referenced this issue Jan 23, 2023
@alexef @rbreeze
feat(UI): allow applications to declare default view preferences (#12019
4a50114 
) (#12080)


Signed-off-by: Alex Eftimie <alex.eftimie@getyourguide.com>
Co-authored-by: Remington Breeze <remington@breeze.software>
alexef added a commit to alexef/argo-cd that referenced this issue Jan 24, 2023
@alexef
feat(UI): allow applications to declare default view preferences (arg…
179ca2c 
…oproj#12019)

Signed-off-by: Alex Eftimie <alex.eftimie@getyourguide.com>
emirot pushed a commit to emirot/argo-cd that referenced this issue Jan 27, 2023
@alexef @rbreeze @emirot
feat(UI): allow applications to declare default view preferences (arg…
9afb524 
…oproj#12019) (argoproj#12080)

Signed-off-by: Alex Eftimie <alex.eftimie@getyourguide.com>
Co-authored-by: Remington Breeze <remington@breeze.software>
Signed-off-by: emirot <emirot.nolan@gmail.com>
schakrad pushed a commit to schakrad/argo-cd that referenced this issue Mar 14, 2023
@alexef @rbreeze @schakrad
feat(UI): allow applications to declare default view preferences (arg…
bfe5038 
…oproj#12019) (argoproj#12080)

Signed-off-by: Alex Eftimie <alex.eftimie@getyourguide.com>
Co-authored-by: Remington Breeze <remington@breeze.software>
Signed-off-by: schakrad <chakradari.sindhu@gmail.com>
@shuker85
Copy link
Contributor

shuker85 commented Aug 3, 2023

It would be great if we can override for all apps from a argocd-cm param too.It can be tedious to change in case of hundreds apps.

@crenshaw-dev
Copy link
Member

Closing this since the original ask is done: https://argo-cd.readthedocs.io/en/latest/operator-manual/ui-customization/#default-application-details-view

@shuker85 would you mind opening a new issue for the global setting?

@shuker85 shuker85 mentioned this issue Aug 3, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@alexef @crenshaw-dev @rbreeze @shuker85 @codestalkerr