Skip to content

Conversation

@brayan-trejo
Copy link
Contributor

@brayan-trejo brayan-trejo commented Aug 14, 2025

What type of Pull Request is this? (check all applicable)

  • Refactor
  • [X ] Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Describe your changes in brief

  • Added /cmd/config.go (source code, called by using cloudfuse config)
  • Creates config.aes in project root.

FUTURE CHANGES:

  • Add dry run mount call to make sure configuration was successful before letting the user go.
  • Add credential key validation. There are some built-in validation when calling getContainerListAzure() or getContainerListS3() but need to do further testing.
  • Add a check that makes sure the available cache size is at least XGB when caching is enabled. Otherwise, recommend that user disables caching. Where X is the minimum required capacity for caching.

Checklist

  • [X ] Tested locally
  • [X ] Added new dependencies
  • Updated documentation
  • Added tests

Related Issues

  • Related Issue #
  • Closes #

Copy link
Contributor

@jfantinhardesty jfantinhardesty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a few issues to resolve. Love the look of the TUI!

@brayan-trejo brayan-trejo changed the base branch from main to cloudfuse-2.0 August 18, 2025 15:23
Copy link
Contributor

@jfantinhardesty jfantinhardesty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Just left one more comment. And then looks like their are some lint issues.

Copy link
Contributor

@foodprocessor foodprocessor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This. Is clean, well-organized and feature-rich.
Thank you so much for making this, and also thank you for your incredible patience with the feature creep I've caused here!
Very good work!

Copy link
Contributor

@foodprocessor foodprocessor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great!

Copy link
Contributor

@foodprocessor foodprocessor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work!

@brayan-trejo brayan-trejo merged commit 7f50d23 into cloudfuse-2.0 Sep 2, 2025
37 of 42 checks passed
@brayan-trejo brayan-trejo deleted the add-tui branch September 2, 2025 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants