Skip to content

Conversation

@ohnoitsyou
Copy link

Longhorn uses labels to find volumes to backup, plus having labels makes sense anyway
Add values field for data-dir labels
Add data-pvc labels field to README

Longhorn uses labels to find volumes to backup, plus having labels makes sense anyway
Add values field for data-dir labels
Add data-pvc labels field to README
Bump Chart version number to 2.0.2
@ohnoitsyou ohnoitsyou requested a review from SQLJames as a code owner October 24, 2024 04:40
@SQLJames
Copy link
Owner

SQLJames commented Nov 1, 2024

I was trying to fix the merge conflict and correct the version but I am getting an error for permissions on your branch
! [remote rejected] HEAD -> ohnoitsyou (permission denied)
error: failed to push some refs to 'github.com:ohnoitsyou/factorio-server-charts.git'

@SQLJames
Copy link
Owner

SQLJames commented Nov 1, 2024

@ohnoitsyou I opened PR #60 to get this merged in. Since the commit is missing from the chain I gave you a shout out in the change log

@SQLJames SQLJames closed this Nov 1, 2024
@ohnoitsyou
Copy link
Author

Ah, hadn't realize there had been new commits. Thanks for getting it merged!

@ohnoitsyou ohnoitsyou deleted the data-pvc-labels branch November 1, 2024 18:03
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.

2 participants