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.

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

[Merged by Bors] - Fix Fluvio log directory on K8 #1043

Closed
wants to merge 4 commits into from

Conversation

nicholastmosher
Copy link
Contributor

@nicholastmosher nicholastmosher commented Apr 30, 2021

Fixes #539.

  • Changes remaining Specs to use /var/lib/fluvio/data
  • Changes local development data directory to ~/.fluvio/data/

@nicholastmosher
Copy link
Contributor Author

Will update to use ~/.fluvio/data as the development data directory.

Copy link
Contributor

@sehz sehz left a comment

Choose a reason for hiding this comment

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

LGTM

@sehz
Copy link
Contributor

sehz commented Apr 30, 2021

Update chanelog and bump up platform version

@sehz
Copy link
Contributor

sehz commented Apr 30, 2021

And bump up version for this crate as well

Copy link
Contributor

@nacardin nacardin left a comment

Choose a reason for hiding this comment

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

K8 and Local stores data at
/var/lib/fluvio/data and ~/.fluvio/data respectively

@nicholastmosher
Copy link
Contributor Author

bors r+

bors bot pushed a commit that referenced this pull request Apr 30, 2021
Fixes #539.

- Changes remaining Specs to use `/var/lib/fluvio/data`
- Changes local development data directory to `~/.fluvio/data/`
@bors bors bot changed the title Fix Fluvio log directory on K8 [Merged by Bors] - Fix Fluvio log directory on K8 Apr 30, 2021
@bors bors bot closed this Apr 30, 2021
@nicholastmosher nicholastmosher deleted the logdir branch May 18, 2021 14:09
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.

[Bug]: SPU on K8 uses /tmp/fluvio as data dir
3 participants