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

gitops summary breaks when there a dummy app directory under uptick-cluster/apps #64

Closed
mdesouky opened this issue Dec 7, 2023 · 2 comments
Closed

gitops summary breaks when there a dummy app directory under uptick-cluster/apps #64

mdesouky opened this issue Dec 7, 2023 · 2 comments

Comments

@mdesouky
Copy link
Collaborator

mdesouky commented Dec 7, 2023

While running gitops summary it breaks with the error message There's no app with the name '<dummy-directory>', silly.
@mdesouky
Copy link
Collaborator Author

mdesouky commented Dec 7, 2023

This is fixed by removing the dummy directory from the local clone of the uptick-cluster repo

@mdesouky
Copy link
Collaborator Author

mdesouky commented Dec 8, 2023 •
edited
Loading

Probably need to modify this to detect empty directories and not process them as an app

mdesouky added a commit that referenced this issue Dec 8, 2023
@mdesouky
fix: 🐛 Fix issue #64
e1f3b45 
Fixes erroring out when there are empty directories under uptick-cluster/apps
@mdesouky mdesouky mentioned this issue Dec 8, 2023
Merged
mdesouky added a commit that referenced this issue Dec 11, 2023
@mdesouky
Merge pull request #66 from uptick/bugfix/fix-issue-#64
b7bde5d 
Fix issue #64
This was referenced Nov 15, 2024
Closed
Closed
Merged
This was referenced Jan 23, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mdesouky @uptickmetachu